From: Eric Leblond <eric@regit.org>
To: Felix Janda <felix.janda@posteo.de>, netfilter-devel@vger.kernel.org
Subject: Re: [ulogd2 PATCH 2/4] Use stdint types everywhere
Date: Tue, 23 Jun 2015 14:37:37 +0200 [thread overview]
Message-ID: <1435063057.4241.5.camel@regit.org> (raw)
In-Reply-To: <20150516130207.GC14201@euler>
> Hello Felix,
Sorry for the delay in the answer.
Other patches of the series are ok, comments on this one inline.
On Sat, 2015-05-16 at 15:02 +0200, Felix Janda wrote:
> Signed-off-by: Felix Janda <felix.janda@posteo.de>
> ---
> filter/raw2packet/ulogd_raw2packet_BASE.c | 42 +++++++++++++++------
> ----------
> filter/ulogd_filter_HWHDR.c | 6 ++---
> filter/ulogd_filter_IP2HBIN.c | 4 +--
> filter/ulogd_filter_IP2STR.c | 2 +-
> filter/ulogd_filter_MARK.c | 4 +--
> filter/ulogd_filter_PWSNIFF.c | 4 +--
> include/libipulog/libipulog.h | 4 +--
> include/ulogd/addr.h | 2 +-
> include/ulogd/conffile.h | 8 +++---
> include/ulogd/ipfix_protocol.h | 36 +++++++++++++--------
> -----
> include/ulogd/ulogd.h | 42 +++++++++++++++------
> ----------
> input/packet/ulogd_inppkt_NFLOG.c | 16 ++++++------
> input/packet/ulogd_inppkt_UNIXSOCK.c | 34 ++++++++++++---------
> ----
> libipulog/libipulog.c | 8 +++---
> output/mysql/ulogd_output_MYSQL.c | 2 +-
> output/ulogd_output_IPFIX.c | 20 +++++++--------
> src/addr.c | 4 +--
> util/db.c | 2 +-
> util/printpkt.c | 6 ++---
> 19 files changed, 123 insertions(+), 123 deletions(-)
I'm ok with the idea of the patch but this is not building anymore on
my Debian.
|| make[2]: Entering directory
'/home/eric/git/netfilter/ulogd2/libipulog'
|| CC libipulog.lo
include/libipulog/libipulog.h|23 col 1| error: unknown type name
'uint32_t'
|| uint32_t ipulog_group2gmask(uint32_t group);
|| ^
include/libipulog/libipulog.h|23 col 29| error: unknown type name
'uint32_t'
|| uint32_t ipulog_group2gmask(uint32_t group);
|| ^
int32_t group);
|| ^
I've got plenty of similar error.
Could you fix this an resubmit ?
BR,
--
Eric Leblond <eric@regit.org>
Blog: https://home.regit.org/
next prev parent reply other threads:[~2015-06-23 13:02 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-05-16 13:02 [ulogd2 PATCH 2/4] Use stdint types everywhere Felix Janda
2015-06-23 12:37 ` Eric Leblond [this message]
2015-06-23 17:26 ` Felix Janda
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=1435063057.4241.5.camel@regit.org \
--to=eric@regit.org \
--cc=felix.janda@posteo.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).