automake-patches
[Top][All Lists]
Advanced

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

Re: Correct expansion of auxdir; have missing use it


From: Tom Tromey
Subject: Re: Correct expansion of auxdir; have missing use it
Date: 14 Aug 2001 10:11:03 -0600

>>>>> "Tim" == Tim Van Holder <address@hidden> writes:

Tim>    * m4/auxdir.m4: Use a subshell so our cwd doesn't
Tim>    get changed.
Tim>    * m4/missing.m4: Require AM_AUX_DIR_EXPAND and
Tim>    use $am_aux_dir.

Tim> -am_aux_dir=`CDPATH=:; cd $ac_aux_dir && pwd`
Tim> +am_aux_dir=`(CDPATH=:; cd $ac_aux_dir && pwd)`

I think a subshell is implicit in backquotes.
So I don't think this patch should be required.

I'm assuming you're doing this on Windows?
What shell are you using?

It is (mostly) harmless to apply this patch, but if we really need it
then we need to audit the code to make sure this construct is used
everywhere, and then we need to add something to maintainer-check so
that we don't inadvertantly reintroduce the problem.

Tom



reply via email to

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