Thread (6 messages) 6 messages, 4 authors, 2019-11-07

Re: [PATCH] powerpc/tools: Don't quote $objdump in scripts

From: Michael Ellerman <mpe@ellerman.id.au>
Date: 2019-10-30 11:58:03

Segher Boessenkool [off-list ref] writes:
On Thu, Oct 24, 2019 at 11:47:30AM +1100, Michael Ellerman wrote:
quoted
Some of our scripts are passed $objdump and then call it as
"$objdump". This doesn't work if it contains spaces because we're
using ccache, for example you get errors such as:

  ./arch/powerpc/tools/relocs_check.sh: line 48: ccache ppc64le-objdump: No such file or directory
  ./arch/powerpc/tools/unrel_branch_check.sh: line 26: ccache ppc64le-objdump: No such file or directory

Fix it by not quoting the string when we expand it, allowing the shell
to do the right thing for us.
This breaks things for people with spaces in their paths.
Spaces in their what? Who does that? :)

Also we don't support it:

  $ pwd
  $ /home/michael/foo bar
  $ make clean
  Makefile:147: *** source directory cannot contain spaces or colons.  Stop.

cheers
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help