Re: [Ntop-dev] [patch] fix broken configure option handling

2005-12-31 Thread Roland Illig
Mike Frysinger wrote: AC_ARG_ENABLE(i18n, [ --enable-i18n Enable (limited) internationalization [[default=disabled]]], - ac_enable_i18n=yes, + ac_enable_i18n=$enableval, ac_enable_i18n=no) You can also just use

Re: [Ntop-dev] [patch] fix broken configure option handling

2005-12-31 Thread Mike Frysinger
On Saturday 31 December 2005 05:39, Roland Illig wrote: Mike Frysinger wrote: AC_ARG_ENABLE(i18n, [ --enable-i18n Enable (limited) internationalization [[default=disabled]]], - ac_enable_i18n=yes, + ac_enable_i18n=$enableval,

Re: [Ntop-dev] [patch] fix broken configure option handling

2005-12-31 Thread Mike Frysinger
On Saturday 31 December 2005 07:36, Burton Strauss wrote: We also need to be careful and make sure this works with older versions of auto*. We've had problems in the past with this, since we don't require that the auto* tools be present, and people use old (friendly, well understood) versions

RE: [Ntop-dev] [patch] fix broken configure option handling

2005-12-31 Thread Burton Strauss
as the autoconf 2.13 - 2.50 changes however. -Burton -Original Message- From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] On Behalf Of Mike Frysinger Sent: Saturday, December 31, 2005 6:41 AM To: ntop-dev@Unipi.IT Cc: Burton Strauss Subject: Re: [Ntop-dev] [patch] fix broken configure

Re: [Ntop-dev] [patch] fix broken configure option handling

2005-12-29 Thread Mike Frysinger
On Thursday 29 December 2005 20:46, Mike Frysinger wrote: all of the AC_ARG_ENABLE() macros used in configure.in set the 3rd option incorrectly ... find attached a patch to fix the issue actually that patch is slightly broken as well ... at first glance i thought they were all AC_ARG_ENABLE()

RE: [Ntop-dev] [patch] fix broken configure option handling

2005-12-29 Thread Burton Strauss
@Unipi.IT Subject: Re: [Ntop-dev] [patch] fix broken configure option handling On Thursday 29 December 2005 20:46, Mike Frysinger wrote: all of the AC_ARG_ENABLE() macros used in configure.in set the 3rd option incorrectly ... find attached a patch to fix the issue actually that patch

Re: [Ntop-dev] [patch] fix broken configure option handling

2005-12-29 Thread Mike Frysinger
On Thursday 29 December 2005 21:58, Burton Strauss wrote: So that let's us use --enable-i18n, --disable-i18n and even --enable-i18n=no ? And have them work as expected?? exactly -mike ___ Ntop-dev mailing list Ntop-dev@unipi.it