[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: 02-lineno.patch
From: |
Raja R Harinath |
Subject: |
Re: 02-lineno.patch |
Date: |
Thu, 27 Sep 2001 12:20:22 -0500 |
User-agent: |
Gnus/5.090004 (Oort Gnus v0.04) Emacs/21.0.105 |
Hi,
Akim Demaille <address@hidden> writes:
> | Well one other case this matters is in the computation of $srcdir.
> | configure supports the --srcdir option and also uses `dirname $0`.
> | The only issue is whether --srcdir was meant to support 'configure'
> | invocation from $PATH.
>
> Ah, I missed that, thanks! Hm... Which means the test suite will soon
> fail for someone, as it does exercise this, but my host is too good to
> see that.
The bash man page says:
... If LINENO is unset, it loses its special properties, even if it
is subÂsequently reset.
So, maybe one of the testcases should 'unset LINENO' early on, so that
you could test the no-$LINENO case.
> | > Raja> You also need to capture the directory of the original
> | > Raja> 'configure' call so that you can figure out $srcdir from it.
[snip]
> Thanks for pointing this out! I don't know how to handle this
> properly though... Maybe we should pass something like
> AS_ORIGINAL_INVOCATION_NAME?
>
> exec AS_ORIGINAL_INVOCATION_NAME=$0 $as_me.lineno
>
> what do you think about that? It should be configure-independent, as
> it is M4sh code, not dedicated to Autoconf.
Yep. That seems the only option. We can't assume the script has a
--srcdir option.
It would be nice if autoconf/general.m4 didn't have to compute $srcdir
on it's own. Maybe m4sh should prove $as_mydir (== `dirname $0`) in
parallel to $as_me (== `basename $0`) and autoconf/general.m4 just
says:
if test -z "$srcdir"; then
ac_srcdir_default=yes
srcdir=$as_mydir
...
Then, AS_ORIGINAL_INVOCATION_NAME would be internal to m4sh.
- Hari
--
Raja R Harinath ------------------------------ address@hidden
"When all else fails, read the instructions." -- Cahn's Axiom
"Our policy is, when in doubt, do the right thing." -- Roy L Ash
Re: 02-lineno.patch, Akim Demaille, 2001/09/27