[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <517D32AE.503@mojatatu.com>
Date: Sun, 28 Apr 2013 10:31:10 -0400
From: Jamal Hadi Salim <jhs@...atatu.com>
To: Alexander Duyck <alexander.h.duyck@...el.com>
CC: shemminger@...tta.com, netdev@...r.kernel.org,
jeffrey.t.kirsher@...el.com, Hasan Chowdhury <shemonc@...il.com>
Subject: Re: [IPROUTE PATCH] iproute2: act_ipt fix xtables breakage on older
versions.
Hi Alex,
On 13-04-26 11:58 AM, Alexander Duyck wrote:
> Getting it to compile is kind of important since it was preventing me
> from completing the build for iproute2 as I had another fix I needed to
> test.
So i just went back and looked. I was under the impression the original
patch from Hasan was not incorporated, it turns out it was (I just didnt
get the usual "applied" email; ands unfortunately for some reason i
showed up as the author instead of Hasan). It is the patch you are
referring to.
Second - I did compile this on Debian squeeze which is iptables 1.4.8
based; lucky, I still have that machine around, so i just verified again
that the current git tree compiles fine there.
So this could be a fedora specific issue.
What does the generated Config file have for you?
on squeeze:
IPT_LIB_DIR:=/lib/xtables
>
> If you have some test case you want me to run just let me know. All I
> did here is revert the changes that I believe were made incorrectly for
> versions prior to 1.4.10 in "iproute2: act_ipt fix xtables breakage".
> It states it was fixing it for versions starting in 1.4.10 which I am
> assuming is the XTABLES_VERSION_CODE >= 6 case so it should not have
> modified the 3 parameter case for versions prior to 6 which is what I
> corrected.
>
Are you sure about that? Please double check again, i see 4 parameters.
Having said that the mystery and breakages may have to do with the way
the different distros package iptables.
cheers,
jamal
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists