Matthieu Moy <[email protected]> writes: > Junio C Hamano <[email protected]> writes: > >> David Aguilar <[email protected]> writes: >> >>> +IFS=' >>> +' >>> +git ls-files *.h ewah/*.h vcs-svn/*.h xdiff/*.h | >> >> Hmm. This is only for true developers (not one who merely compiles >> after expanding a tarball), so "git ls-files" may probably be OK. >> >> But "/bin/ls" would be equally fine for that, no? > > Actually, since this is "| while read header", I have to wonder why this > is not written as > > for header in .h ewah/*.h vcs-svn/*.h xdiff/*.h > do > ... > done
Yes, that would be even better. Then you wouldn't even have to worry about $IFS dance. -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
