Re: [Gnash-commit] gnash libbase/image.h libbase/log.cpp server/as...

[zou lunkai]

> >> -abs_mediadir = $(shell cd $(srcdir)/../media; pwd)
> >> +abs_mediadir = `cd $(srcdir)/../media; pwd`

I used to take the first version in my makefile. Isn't  $(shell
command) the traditional way in a makefile?

>    Because otherwise a newer automake complains, and I like to get rid
> of warnings whenever possible. It should work on Fedora 6, but obviously
> doesn't. There are probably other ways to get rid of the automake error
> about variables names not being POSIX compliant.

I don't know much about the scripts.  I simply want  'make' and 'make
check' works here.  What should I do then? update the autotools?

--zou




On Dec 11, 2007 1:02 PM, Rob Savoye <address@hidden> wrote:
> zou lunkai wrote:
> >> -abs_mediadir = $(shell cd $(srcdir)/../media; pwd)
> >> +abs_mediadir = `cd $(srcdir)/../media; pwd`
> >
> > after the above patch,  'make check ' in testsuite/misc-swfc.all, got:
> >
> > cpp -P -I. -DMEDIADIR='`cd ./../media; pwd`' sound.sc > pp_sound.sc
> > /usr/local/bin/swfc  -o sound.swf pp_sound.sc
> > Syntax error in line 145, 14: `cd ./../media; pwd`/sound1.wav
>
>
> > The second version does not work here.  Why we need this change?
>
>    Because otherwise a newer automake complains, and I like to get rid
> of warnings whenever possible. It should work on Fedora 6, but obviously
> doesn't. There are probably other ways to get rid of the automake error
> about variables names not being POSIX compliant.
>
>         - rob -
>