On 2016-08-03 at 10:58:01 +0200, Vadim Kochan <vadi...@gmail.com> wrote:
> Add command line parsing function which allows to compile tools
> (trafgen, netsniff-ng) without libnl-xxx libraries.
> 
> Option --disable-libnl sets CONFIG_LIBNL=0 which means compile tools
> without libnl dependencies.
> 
> Signed-off-by: Vadim Kochan <vadi...@gmail.com>
> ---
>  configure | 47 ++++++++++++++++++++++++++++++++++++++++++++---
>  1 file changed, 44 insertions(+), 3 deletions(-)
> 
> diff --git a/configure b/configure
> index 105b1ec..61b2e31 100755
> --- a/configure
> +++ b/configure
> @@ -18,8 +18,36 @@ HAVE_LIBGEOIP=0
>  HAVE_LIBZ=0
>  HAVE_TPACKET3=0
>  
> +CONFIG_LIBNL=1
> +
>  # use "CROSS_COMPILE=<prefix> SYSROOT=<path> ./configure && make" for cross 
> compilation
>  
> +usage()
> +{
> +     echo "Usage: ./configure [OPTIONS]"

echo "Usage: ./configure [OPTION]..."

> +     echo -e  "\t-h, --help      - print usage"
> +     echo -ne "\t--disable-libnl - compile without libnl."

Please avoid using echo -e and tabs, just use two spaces to indent
(also check usage message format of autotools configure).

btw, I just noticed we already consider the DISABLE_GEOIP and
DISABLE_ZLIB environment variables. I will apply the option handling
part of this patch first and convert the existing DISABLE_* to
parameters. Then you may implement your libnl changes on top of mine.

> +     echo "Some features (rfraw, nlmsg dissect) will not work in trafgen, 
> netsniff-ng."

I don't think this message is necessary. If at all it should be added to
INSTALL (and the "required/optional libraries" section updated
accordingly).

-- 
You received this message because you are subscribed to the Google Groups 
"netsniff-ng" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to netsniff-ng+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to