All of lore.kernel.org
 help / color / mirror / Atom feed
From: Patrick McHardy <kaber@trash.net>
To: Jan Engelhardt <jengelh@medozas.de>
Cc: netfilter-devel@vger.kernel.org
Subject: Re: nftables-goes-automake patch series
Date: Wed, 25 Mar 2009 15:33:22 +0100	[thread overview]
Message-ID: <49CA40B2.5010300@trash.net> (raw)
In-Reply-To: <alpine.LSU.2.00.0903251447320.13559@fbirervta.pbzchgretzou.qr>

Jan Engelhardt wrote:
> On Wednesday 2009-03-25 14:11, Patrick McHardy wrote:
>> Jan Engelhardt wrote:
>>> Not much more to elaborate on, this was straight forward to do
>>> and only a matter of time, and really needed :)
>> Patch 1 looks fine, although I might take the liberty to rename the
>> file again :)
>>
>> About automake - I'm glad you're fixing up the makefiles, but I'm
>> quite attached to the silent build and the automake version that
>> comes with Debian unstable is 1.10, so I assume it won't be
>> silent anymore.
> 
> I do have a patch for 1.10 too, if you need.

I don't really want to compile my own automake. Can't we just keep
the silent build in combination with automake? I'm fine with doing
the final conversion once that version of automake is commonly
available in distributions.

>> It also doesn't build anymore on my system, the
>> parser.h file is not generated:
>>
>> ...
>> scanner.l:23:20: Fehler: parser.h: Datei oder Verzeichnis nicht gefunden
>> <stdout>:1395: Fehler: expected specifier-qualifier-list before »YYSTYPE«
>> <stdout>:1454: Fehler: expected »=«, »,«, »;«, »asm« or »__attribute__« before
>> »*« token...
> 
> It does do it here:
> 
> make[2]: Entering directory `/home/jengelh/Coding/nftables/src'
> flex --header-file=scanner.h -t scanner.l >scanner.c
> /bin/sh ../ylwrap parser.y y.tab.c parser.c y.tab.h parser.h y.output parser.output -- bison -y  -d --skeleton=../../../../../../../..//home/jengelh/Coding/nftables/src/parser-skeleton.c
> updating parser.h
> depbase=`echo scanner.o | sed 's|[^/]*$|.deps/&|;s|\.o$||'`;\
>         gcc -DHAVE_CONFIG_H -I. -I..    -D_FILE_OFFSET_BITS=64 -D_REENTRANT -Wall -Waggregate-return    -Wbad-function-cast -Wcast-align -Wdeclaration-after-statement  -Werror -Wformat=2 -Winit-self -Winline -Wmissing-declarations  -Wmissing-format-attribute -Wmissing-prototypes         -Wshadow -Wsign-compare -Wstrict-prototypes -Wundef -Wunused    -Wwrite-strings -fno-strict-aliasing -pipe -I../include -DDEFAULT_INCLUDE_PATH=\"/usr/local/etc/nftables\" -g -O2 -MT scanner.o -MD -MP -MF $depbase.Tpo -c -o scanner.o scanner.c &&\
>         mv -f $depbase.Tpo $depbase.Po
> 
> Maybe you need a distclean? Are you sure you are on commit
> v0.01-alpha1-23-g85bf078?

I tried both pulling your tree into mine, as well as a fresh clone of
your tree. Let me know if you need more information.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  reply	other threads:[~2009-03-25 14:33 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-03-24 23:31 nftables-goes-automake patch series Jan Engelhardt
2009-03-25 13:11 ` Patrick McHardy
2009-03-25 14:02   ` Jan Engelhardt
2009-03-25 14:33     ` Patrick McHardy [this message]
2009-03-25 14:44       ` Jan Engelhardt
2009-03-25 16:30         ` Patrick McHardy
2009-03-26  9:45       ` Jan Engelhardt

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=49CA40B2.5010300@trash.net \
    --to=kaber@trash.net \
    --cc=jengelh@medozas.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.