Opened 14 years ago

Closed 14 years ago

Last modified 14 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: Analyzed by developer:



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@… 14 years ago.
change all 'echo -n' to 'echo ${_echo_n}'

Download all attachments as: .zip

Change History (3)

Changed 14 years ago by sci-fi@…

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

comment:1 Changed 14 years ago by sci-fi@…

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

comment:2 Changed 14 years ago by diego@…

  • Resolution set to fixed
  • Status changed from new to closed

Patch applied, thanks.

Note: See TracTickets for help on using tickets.