[Buildroot] [PATCH 2/2] Prefer 'printf' over 'echo -e' (for portability)

Peter Korsgaard jacmet at uclibc.org
Tue Jan 21 08:40:30 UTC 2014


>>>>> "Thomas" == Thomas De Schampheleire <patrickdepinguin at gmail.com> writes:

Hi,

 > I find it very strange that we're mixing printf and echo in this same
 > file. I would suggest to either use echo only, or printf only. The
 > statement
 > echo -e '\nblaat'
 > is functionally equivalent to:
 > echo
 > echo blaat

 > Your point about printf being more portable is valid, and has been
 > raised in the past by Yann (I believe) as well.
 > This means that moving to printf iso echo makes sense, but then this
 > patch should be reworked as to fix the entire file.

 > Note that this is just my opinion, I can't speak for the entire community.

Yes, I agree. I would also like to see the same style used everywhere in
the file.

-- 
Bye, Peter Korsgaard



More information about the buildroot mailing list