Opened 17 years ago

Closed 17 years ago

Last modified 17 years ago

#620 closed defect (fixed)

change all 'echo -n' to 'echo ${_echo_n}' to match in configure script

Reported by: sci-fi@… Owned by: diego@…
Priority: unimportant Component: build system
Version: HEAD Severity: trivial
Keywords: Cc:
Blocked By: Blocking:
Reproduced by developer: no Analyzed by developer: no

Description

Hi,

On OSX and probably other systems, we're seeing some odd spacings in the
configure text presented on the console/terminal, plus a stray -n is printed in
a few places. On OSX anyway, bash's built-in echo doesn't honour -n, it is
printed instead; I see configure already has logic to discern this, but not all
echo stmts are using the same convention therein.

Attachments (1)

mplayer_configure_echo-n.patch (914 bytes ) - added by sci-fi@… 17 years ago.
change all 'echo -n' to 'echo ${_echo_n}'

Download all attachments as: .zip

Change History (3)

by sci-fi@…, 17 years ago

change all 'echo -n' to 'echo ${_echo_n}'

comment:1 by sci-fi@…, 17 years ago

Here is a hopefully trivial patch to make things look a wee bit better. ;)

comment:2 by diego@…, 17 years ago

Resolution: fixed
Status: newclosed

Patch applied, thanks.

Note: See TracTickets for help on using tickets.