[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is uncondition
From: |
Ralf Wildenhues |
Subject: |
Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted. |
Date: |
Mon, 6 Sep 2010 19:49:18 +0200 |
User-agent: |
Mutt/1.5.20 (2010-04-22) |
Hi Stefano,
* Stefano Lattarini wrote on Mon, Sep 06, 2010 at 11:28:27AM CEST:
> On Monday 06 September 2010, Ralf Wildenhues wrote:
> > Well, in v1.11-139-g6fee87c and before that in v1.11-138-gc6bbc45
> > the above line was move resp. introduced. Was that a buglet in the
> > latter patch?
> No, there's no bug to be fixed that I know of. It's just that we
> really need $testsrcdir to be the *real* absolute path of $(srcdir),
> so there's no need to read it from the environment when we can just
> AC_SUBST it (also, currently we have to absolutize $testsrcdir, which
> add a fork for every test).
I gather that you are saying that the
address@hidden@}
part was never needed since v1.11-138-gc6bbc45, and at that time already
should have been
address@hidden@
(modulo quoting). In that case, the patch is fine, thanks.
> > If yes, then this patch is fine.
> "Unfortunately", there is no bug fixed by this patch. I can only
> offer the above argumentation (plus consistency of $testsrcdir and
> $top_testsrcdir).
Your verbose explanation above and this seem to contradict each other.
;-)
Thanks,
Ralf
- [PATCH 0/4] {branch "tests-init"} Tests defs: renamings, tweakings and extensions in some global variables, Stefano Lattarini, 2010/09/02
- [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Stefano Lattarini, 2010/09/02
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Ralf Wildenhues, 2010/09/05
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Stefano Lattarini, 2010/09/05
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Ralf Wildenhues, 2010/09/06
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Stefano Lattarini, 2010/09/06
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted.,
Ralf Wildenhues <=
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Stefano Lattarini, 2010/09/06
- Re: [PATCH 1/4] {branch "tests-init"} Tests defs: $srcdir is unconditionally substituted., Ralf Wildenhues, 2010/09/06
Re: [PATCH 2/4] {branch "tests-init"} Tests defs: new variable $top_srcdir., Stefano Lattarini, 2010/09/02
[PATCH 3/4] {branch "tests-init"} Tests defs: remove variable $testaclocaldir., Stefano Lattarini, 2010/09/02
[PATCH 4/4] {branch "tests-init"} Tests defs: $srcdir and $top_srcdir renaming., Stefano Lattarini, 2010/09/02
[PUSH] {branch "tests-init"} Tests defs: renamings, tweakings and extensions in some global variables, Stefano Lattarini, 2010/09/06