[PATCH][RESEND] Print what mainboard coreboot is being built for during build

[Nickolas Lloyd]

Sorry about the mangling, I've attached the patch this time.

Nickolas

[Marc Jones]

Hi Nickolas,

On Wed, Apr 6, 2011 at 10:37 AM, Nickolas Lloyd  wrote:
> Sorry about the mangling, I've attached the patch this time.
> @echo "Built coreboot for $(CONFIG_MAINBOARD_VENDOR) $(CONFIG_MAINBOARD_PART_NUMBER)"

I think that you want the variables outside the "".

Marc

[Russell Whitaker]

On Thu, 7 Apr 2011, Marc Jones wrote:
> Hi Nickolas,
>
> On Wed, Apr 6, 2011 at 10:37 AM, Nickolas Lloyd  wrote:
> > Sorry about the mangling, I've attached the patch this time.
> > @echo "Built coreboot for $(CONFIG_MAINBOARD_VENDOR) $(CONFIG_MAINBOARD_PART_NUMBER)"
>
> I think that you want the variables outside the "".

At least here the echo command ignores the quotes.
Try this:
   export ARCH=i686
then
   echo my arch is $ARCH
   echo "my arch is $ARCH"
   echo "my arch" is $ARCH
all yeald the same result: my arch is i686

Russ

[Nickolas Lloyd]

In bash at least using double-quotes results in the evaluation of
symbols, whereas single-quotes doesn't.  I suspect it's the same for
make.  I can change it if you like though.

Nick

On Thu, Apr 7, 2011 at 6:56 PM, Russell Whitaker  wrote:
> On Thu, 7 Apr 2011, Marc Jones wrote:
> > Hi Nickolas,
> >
> > On Wed, Apr 6, 2011 at 10:37 AM, Nickolas Lloyd  wrote:
> > > Sorry about the mangling, I've attached the patch this time.
> > > @echo "Built coreboot for $(CONFIG_MAINBOARD_VENDOR)
> > >
> > > $(CONFIG_MAINBOARD_PART_NUMBER)"
> >
> > I think that you want the variables outside the "".
>
> At least here the echo command ignores the quotes.
> Try this:
>  export ARCH=i686
> then
>  echo my arch is $ARCH
>  echo "my arch is $ARCH"
>  echo "my arch" is $ARCH
> all yeald the same result: my arch is i686
>
> Russ

[PATCH][RESEND] Print what mainboard coreboot is being built for during build

[Peter Stuge]

Nickolas Lloyd wrote:
> In bash at least using double-quotes results in the evaluation of
> symbols, whereas single-quotes doesn't.  I suspect it's the same
> for make.

make expands $() regardless of whether it is inside quotes or not.
make doesn't do much parsing, after expansion the command is passed
to sh:

$ cat Makefile 
X:=bar

.PHONY: x
x:
    @echo "foo $(X) baz"
$ strace -fF make 2>&1 | grep echo
read(3, "X:=bar\n\n.PHONY: x\nx:\n\t@echo "foo "..., 4096) = 43
[pid 29660] execve("/bin/sh", ["/bin/sh", "-c", "echo "foo bar baz""], [/* 51 vars */]) = 0
> I can change it if you like though.

Better keep it inside the quotes, so that weird names will not mess
up the shell, if we'll ever have any names like that.

Acked-by: Peter Stuge

//Peter