netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
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/


  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).