All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@computergmbh.de>
Cc: Netfilter Developer Mailing List <netfilter-devel@vger.kernel.org>
Subject: Re: [PATCH 1/5][IPTABLES]: Import netfilter.h
Date: Tue, 29 Jan 2008 15:34:06 +0100	[thread overview]
Message-ID: <479F395E.2000305@trash.net> (raw)
In-Reply-To: <Pine.LNX.4.64.0801291528180.1185@fbirervta.pbzchgretzou.qr>

Jan Engelhardt wrote:
> On Jan 29 2008 14:10, Patrick McHardy wrote:
>> Subject: Re: [PATCH 1/5][IPTABLES]: Import netfilter.h
>>
>> Jan Engelhardt wrote:
>>> Import netfilter.h from kernel to get hold of union nf_inet_addr
>>> and fix a compile error in current iptable tarball releases when
>>> used with non-development kernels.
>> We already have this exact file in the current sources, without
>> the #ifdef __KERNEL__ parts.
>>
> 
> Where? If you mean the sanitized headers in 
> /usr/include/linux/netfilter.h -- this file is outdated and does not 
> ship the nf_inet_addr definition.

You're right, I only added it locally, but forgot "svn add" :)
Committed now.

> Since we copy all xt_{match,target}*.h header files too, I thought doing 
> the same for netfilter.h is ok.
> 
> I do believe it is, because the sanitized headers in 
> /usr/include/linux/netfilter/ are _also_ out-of-date.
> BTW, could we, perhaps, remove headers-y for 
> /usr/include/linux/netfilter/, since iptables ships it anyway?

No, there are other users of these files. I recall there is
some perl interface for iptables for example.


      reply	other threads:[~2008-01-29 14:34 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-01-24 19:01 [PATCH 1/5][IPTABLES]: Import netfilter.h Jan Engelhardt
2008-01-24 19:01 ` [PATCH 2/5][IPTABLES]: Give preference to iptables header files Jan Engelhardt
2008-01-29 13:15   ` Patrick McHardy
2008-01-24 19:02 ` [PATCH 3/5][IPTABLES]: Build adjustments Jan Engelhardt
2008-01-29 13:16   ` Patrick McHardy
2008-01-24 19:02 ` [PATCH 4/5][IPTABLES]: libxt_CONNMARK revision 1 Jan Engelhardt
2008-01-29 13:19   ` Patrick McHardy
2008-01-24 19:02 ` [PATCH 5/5][IPTABLES]: libxt_hashlimit " Jan Engelhardt
2008-01-29 13:10 ` [PATCH 1/5][IPTABLES]: Import netfilter.h Patrick McHardy
2008-01-29 14:31   ` Jan Engelhardt
2008-01-29 14:34     ` Patrick McHardy [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=479F395E.2000305@trash.net \
    --to=kaber@trash.net \
    --cc=jengelh@computergmbh.de \
    --cc=netfilter-devel@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.