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]

Re: [v3] testsuite_flags absolute path fix

To: Rainer Orth <ro at TechFak dot Uni-Bielefeld dot DE>
Subject: Re: [v3] testsuite_flags absolute path fix
From: Alexandre Oliva <aoliva at redhat dot com>
Date: 24 May 2001 00:09:31 -0300
Cc: Richard Henderson <rth at redhat dot com>, Benjamin Kosnik <bkoz at redhat dot com>, gcc-patches at gcc dot gnu dot org, zackw at Stanford dot EDU
Organization: GCC Team, Red Hat
References: <200105180718.f4I7IJw05673@fillmore.constant.com><20010518012608.B27094@redhat.com><ydd3d9vfzqg.fsf@xayide.TechFak.Uni-Bielefeld.DE>

On May 23, 2001, Rainer Orth <ro@TechFak.Uni-Bielefeld.DE> wrote:

> dir=`case $dir in /*) echo $dir ;; *) echo \`cd $dir; ${PWDCMD:-pwd}\` ;; esac`

> Thoughts, comments?

Are don't nest properly in all shells.  :- isn't portable.  Instead, use:

case $dir in
[\\/$]* | ?:[\\/]*) ;;
*) dir=`cd "$dir" && ${PWDCMD-pwd} || echo "$dir"` ;;
esac

-- 
Alexandre Oliva   Enjoy Guarana', see http://www.ic.unicamp.br/~oliva/
Red Hat GCC Developer                  aoliva@{cygnus.com, redhat.com}
CS PhD student at IC-Unicamp        oliva@{lsd.ic.unicamp.br, gnu.org}
Free Software Evangelist    *Please* write to mailing lists, not to me

References:
- [v3] testsuite_flags absolute path fix
  - From: Benjamin Kosnik
- Re: [v3] testsuite_flags absolute path fix
  - From: Richard Henderson
- Re: [v3] testsuite_flags absolute path fix
  - From: Rainer Orth

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