[Buildroot] [PATCH 2/2] Add test infrastructure

Arnout Vandecappelle arnout at mind.be
Sun Mar 4 13:01:40 UTC 2012


On Sunday 04 March 2012 01:13:27 Yann E. MORIN wrote:

[snip]
> > +testdir=$(dirname $0)
> > +topdir=$(cd $testdir/..; pwd)
> 
> I would highly suggest using ${foo} instead of $foo because it makes
> it explicit what the variable name is. Also, enclose variable between
> double-quotes, in case a vicious user uses a dir with a space in it.

 Spaces will make 'make' barf, so aren't supported anyway.  But * and
other ugly stuff would indeed be possible, so I've added quotes
everywhere.

[snip]
> See the "The Open Group Base Specifications Issue 7", utilities chapter 2,
> dealing about "Shell Command Language":
>     http://pubs.opengroup.org/onlinepubs/9699919799/toc.htm
>     http://pubs.opengroup.org/onlinepubs/9699919799/utilities/V3_chap02.html

 That's a useful reference!  I'll look at that more often, thanks!

 Regards,
 Arnout

-- 
Arnout Vandecappelle                               arnout at mind be
Senior Embedded Software Architect                 +32-16-286540
Essensium/Mind                                     http://www.mind.be
G.Geenslaan 9, 3001 Leuven, Belgium                BE 872 984 063 RPR Leuven
LinkedIn profile: http://www.linkedin.com/in/arnoutvandecappelle
GPG fingerprint:  7CB5 E4CC 6C2E EFD4 6E3D A754 F963 ECAB 2450 2F1F



More information about the buildroot mailing list