This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: enable Ada test suite


Laurent GUERBY <laurent@guerby.net> writes:
> On Fri, 2003-10-31 at 21:16, Alexandre Oliva wrote:
>> On Oct 29, 2003, "Zack Weinberg" <zack@codesourcery.com> wrote:

>>> I see extensive use of this construct in run_all.sh.

>> It looks hopelessly unportable :-(  If it's meant to be portable, it
>> almost has to be rewritten from scratch. [...]

> It has to work where GNAT works (that does not include old UNIXes), the
> original version was not written to be portable, and I don't believe it
> needs to be that portable.

> On the bright side, all the people who used the original scripts seemed
> to have GNU bash around and succeeded running the thing without too much
> hassle, so I don't expect that much problems.

If it's intentionally not portable, perhaps someone could change the first
line of the script to be:

    #!/bin/bash

instead?  I doubt this would significantly reduce its current portability,
and it would give poor confused people with Bourne shells some clue as to
what they need to do to get it to work.

-- 
Russ Allbery (rra@stanford.edu)             <http://www.eyrie.org/~eagle/>


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]