[Ffmpeg-devel] configure script broken with dash as sh
Tue Aug 15 17:34:33 CEST 2006
John Dalgliesh said:
> On Mon, 14 Aug 2006, Alan Curry wrote:
>> Diego Biurrun writes:
>>> On Mon, Aug 14, 2006 at 11:56:26PM +0200, matthieu castet wrote:
>>>> if dash is used as sh instead of bash, the configure script put in
>>>> #define ASMALIGN(ZEROBITS) ".align 1<<" #ZEROBITS "
>>>> instead of :
>>>> #define ASMALIGN(ZEROBITS) ".align 1<<" #ZEROBITS "\n\t"
>> You can't portably echo a backslash.
> OK. Does the attached patch fix the problem?
>> printf would be better. If there's some printf-phobia, post-process the echo
>> through tr or sed or use a heredoc.
> As printf isn't yet used in the configure script, this patch uses tr. In
> bash you cannot escape a single quote within single quotes, even with a
> backslash. If your shell differs then another backslash could be safely
> added to the last arg to tr.
Please use printf instead. It's much cleaner.
mru at inprovide.com
More information about the ffmpeg-devel