Patches to allow specific build flags, different installation directories and fix gcc45 build issues.

For the gcc-4.5 patch, I suggest to use my version in #255 as it does not contain the whitespace change. ;)

http://subversion.ffado.org/attachment/ticket/255/ffado-gcc45.patch

I don't really understand the purpose (use case) of ffado-2.0.0-sconinstall.patch. Shouldn't DESTDIR do the trick?

Just my €0.02

the gcc-fixes are applied in r1824 and r1825

The sconinstall-patch is nonsense (unless you can prove me wrong that is). if you want to change the install paths to be different from $prefix/lib or $prefix/bin, use BINDIR=... at the scons command.

While the upper-case variants of the variables are the ones for usage in scons and honor the DESTDIR, the lower-case variants of these variables are only to be used in the source. And there it doesn't make any sense to define them to something different the the upper-case variants. Except for the missing DESTDIR.

I will look at the env-variables thing.

Sure enough its the other way round: Upper-case for in-code usage, lower-case for scons (and following destdir).

Still doesn't make any sense to have libdir any different then $DESTDIR/$LIBDIR as this will break finding files...

When I used DESTDIR=$RPM_BUILD_ROOT with rpmbuild on my system, the binaries ended up in my system's /usr/lib64 and /usr/bin and only files under /usr/share where installed in the buildroot. That was the reason I patched in separate paths for the lowercase install paths. There is also a source rpm produced from the openSUSE build and we can't have files outside of the buildroot when a user tries to build the package with a source rpm. As for the flags, I and others have had to patch in build flags in other scons built packages so somewhere there must be a bug in scons. import os env = Environment(ENV = os.environ) is supposed to propagate the entire environment but it doesn't pass any of the CC,CXX or CFLAGS although it could pass xdg the environment. The only package I know of where flags can be passed is blender and that package has specific options for CFLAGS= and CCFLAGS= where CFLAGS sets only CFLAGS and CCFLAGS set both CXX and CFLAGS. I will try the DESTDIR method again and report exactly what happens. I'll also make sure I'm not doing anything wrong.

So its more like a DESTDIR-bug. Which is strange as it seems to work well for the debian-/ubuntu-, gentoo-, arch- and probably more- guys. At least they didn't yet complain.

Anyway I will check the DESTDIR stuff the next days. And re-read the scons manpage for hints on importing the environment. In the past we had chosen to only import the variables we thought where important. But maybe its better to just import the whole environment, if scons allows that.

Hm, configuring my 2.0-branch checkout to have PREFIX=/usr I get the following: Doing "scons install" as users fails (naturally) because of missing rights. Doing "scons install DESTDIR=/home/arnold/tmp" installs everything correctly to DESTDIR/PREFIX while using PREFIX for the compiled-in paths.

I wonder what your problem there is?

It could be me, I will try all possibilities tomorrow and report back.

It was me, I specified DESTDIR at build time and not install time, I've removed that patch. I only have to move the usr/share/libffado/icon directory to usr/share/ now. I didn't have to do that with the install directory patch but that's acceptable.

Hm, somehow scons seems to have a problem import C*FLAGS from the environment.

What about adding an option to define extra flags and stuff to be added to the environment via env.MergeFlags()?

env.MergeFlags() has the advantage that it more or less automatically does the right thing.

Fullfill distributors dreams of custom flags?

Might the patch fulfill the needs of distributions (and gentoo-users)?

This seems to me much less hacky then importing a bunch of C*FLAGS variables that scons resets upon its initialization. And it states the fact that there is a supported setup of compiler flags and modifying it is asking for non-support.

Should be fixed r1852 and r1853.