[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] {tests-init} Tests defs: de-uglify code for requiring an XSI
From: |
Stefano Lattarini |
Subject: |
Re: [PATCH] {tests-init} Tests defs: de-uglify code for requiring an XSI-capable shell. |
Date: |
Mon, 13 Dec 2010 20:16:47 +0100 |
User-agent: |
KMail/1.13.3 (Linux/2.6.30-2-686; KDE/4.4.4; i686; ; ) |
On Monday 13 December 2010, Ralf Wildenhues wrote:
> * Stefano Lattarini wrote on Sun, Dec 12, 2010 at 10:06:58PM CET:
> > This is just a cosmetic patch. I will push in 72 hours
> > (Wednesday evening) if nobody objects.
>
> Please don't. It's a feature that the code is the same as the one
> tested in Libtool _LT_CHECK_SHELL_FEATURES: bug-tested and all.
>
Ah, ok, What about adding a comment about this fact then? So that
someone reading the code can tell "OK, this is ugly, but for a good
reason indeed; better not touch it."
> I don't see this fixing a bug,
>
It doesn't.
> so it can only regress.
>
> Thanks,
> Ralf
>
Regards,
Stefano