[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug-inetutils] Bug in --with-path-foo in inetutils-1.4.2

From: Dan Kegel
Subject: [bug-inetutils] Bug in --with-path-foo in inetutils-1.4.2
Date: Thu, 25 Sep 2003 11:38:37 -0700
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.4) Gecko/20030624

(Originally posted to address@hidden by Milan Bobde yesterday.)

Looks like maybe the configure.ac rewrite described at
missed a trick.

There seems to be an inconsistency in the configure script for inetutils-1.4.2.

./configure --help  says:
 --with-PATHVAR=PATH     Set the value of PATHVAR to PATH
                          PATHVAR is the name of a \`PATH_FOO' variable,
                          downcased, with \`_' changed to \`-'

i.e. if you want to set the path used to invoke cp to /bin/cp, you'd
use --with-path-cp=/bin/cp.  Sadly, this fails silently.  You have
to use --with-PATH-CP=/bin/cp.  If this is the intent, the --help
message needs changing.  But there's more to this bug!  Looking
into acinclude.m4, we found that it's still trying to convert 'foo'
in --with-foo to lowercase; see the lines

    iu_pathvar="`echo $iu_path  | sed y/${IU_UCASE}/${iu_lcase}/`"
    AC_MSG_CHECKING(for value of $iu_path)

However, IU_UCASE is not defined anywhere!  The ChangeLog says:

2002-04-21  Alfred M. Szmidt  <address@hidden>
        * acinclude.m4 (IU_UCASE, iu_lcase): Variables removed.
(And the patch that did it seems to be
http://mail.gnu.org/archive/html/bug-inetutils/2002-04/msg00051.html )

So -- either IU_UCASE needs to be reinstated, or the code
needs to be changed to not refer to it, and the doc needs
to be changed to reflect the need for uppercase when setting paths.

- Dan & Milan

Dan Kegel

reply via email to

[Prev in Thread] Current Thread [Next in Thread]