[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [FYI] {maint,master} test defs: allow overriding of `$me'
From: |
Stefano Lattarini |
Subject: |
Re: [FYI] {maint,master} test defs: allow overriding of `$me' |
Date: |
Mon, 18 Apr 2011 10:27:04 +0200 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
On Monday 18 April 2011, Ralf Wildenhues wrote:
> Hi Stefano,
>
> * Stefano Lattarini wrote on Sun, Apr 17, 2011 at 09:36:42PM CEST:
> > On Sunday 17 April 2011, Ralf Wildenhues wrote:
> >
> > > am_test_name is better, but doesn't explain either why it would be
> > > needed in the first place.
> > >
> > Second patch of:
> > <http://lists.gnu.org/archive/html/automake-patches/2011-02/msg00044.html>
> > And possible similar patches in the future.
>
> That explains why, from within the testsuite, you'd like to be able to
> override it for some tests. It doesn't explain why an override from the
> user calling 'make check' should be possible. (IOW, I understand the
> "this would be convenient" aspect, but it seems it should be possible to
> construct the testsuite in a way to still not allow user overrides.)
>
Honestly, I'd not worry about this ATM (but I share your concerns about
the lack of namespace cleanliness). Presently, a determined user can
anyway wreak havoc by exporting variables such as 'required', 'MISSING'
and 'parallel_tests' (and there are even more in the master branch:
'original_AUTOMAKE', 'am__using_gmake', 'instspc_action'). A first
step would IMHO be making these variables at least namespace-safe.
Should I write a patch?
Regards,
Stefano
- [FYI] {maint,master} test defs: allow overriding of `$me', Stefano Lattarini, 2011/04/17
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Ralf Wildenhues, 2011/04/17
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Stefano Lattarini, 2011/04/17
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Ralf Wildenhues, 2011/04/17
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Stefano Lattarini, 2011/04/17
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Ralf Wildenhues, 2011/04/18
- Re: [FYI] {maint,master} test defs: allow overriding of `$me',
Stefano Lattarini <=
- Re: [FYI] {maint,master} test defs: allow overriding of `$me', Ralf Wildenhues, 2011/04/18
- New release wishlist (was: Re: [FYI] {maint, master} test defs: allow overriding of `$me'), Stefano Lattarini, 2011/04/18
- Re: New release wishlist, Ralf Wildenhues, 2011/04/18
- Re: New release wishlist, Stefano Lattarini, 2011/04/18
- [PATCH] test defs: don't allow `$me' to be overridden from the environment (was: Re: [FYI] {maint, master} test defs: allow overriding of `$me'), Stefano Lattarini, 2011/04/17
- Re: [PATCH] test defs: don't allow `$me' to be overridden from the environment, Ralf Wildenhues, 2011/04/18
- Re: [PATCH] test defs: don't allow `$me' to be overridden from the environment, Stefano Lattarini, 2011/04/18
- Re: [PATCH] test defs: don't allow `$me' to be overridden from the environment, Ralf Wildenhues, 2011/04/18
- Re: [PATCH] test defs: don't allow `$me' to be overridden from the environment, Stefano Lattarini, 2011/04/18
- Re: [PATCH] test defs: don't allow `$me' to be overridden from the environment, Stefano Lattarini, 2011/04/18