/usr/local stuff added unconditionnally in configure.in

Hello,

Around lines 77-84 in configure.in lie these lines:

for path in /usr/local; do
if test -d $path/include; then
EXTRA_CFLAGS="$EXTRA_CFLAGS -I$path/include"
fi
if test -d $path/lib; then
EXTRA_LDFLAGS="$EXTRA_LDFLAGS -L$path/lib"
fi
done

When cross-compiling, this is definitely something we don’t want (mixing
build and host/target libraries/include).–
Patrice Mandin
WWW: http://pmandin.atari.org/
Programmeur Linux, Atari
Sp?cialit?: D?veloppement, jeux

good call

personally, i dont think these lines should ever be added … a properly
configure toolchain will search /usr/local automatically anyways
-mikeOn Thursday 11 May 2006 17:30, Patrice Mandin wrote:

Hello,

Around lines 77-84 in configure.in lie these lines:

for path in /usr/local; do
if test -d $path/include; then
EXTRA_CFLAGS="$EXTRA_CFLAGS -I$path/include"
fi
if test -d $path/lib; then
EXTRA_LDFLAGS="$EXTRA_LDFLAGS -L$path/lib"
fi
done

When cross-compiling, this is definitely something we don’t want (mixing
build and host/target libraries/include).

I’m pretty sure I added this to fix building on *BSD, but I’ll disable
it for now, until I figure out why that was needed.

BTW, what would cause a properly configured toolchain to look in /usr/local?
Or /opt/pkg? Or any other non-system path?

-Sam Lantinga, Senior Software Engineer, Blizzard Entertainment> On Thursday 11 May 2006 17:30, Patrice Mandin wrote:

Hello,

Around lines 77-84 in configure.in lie these lines:

for path in /usr/local; do
if test -d $path/include; then
EXTRA_CFLAGS="$EXTRA_CFLAGS -I$path/include"
fi
if test -d $path/lib; then
EXTRA_LDFLAGS="$EXTRA_LDFLAGS -L$path/lib"
fi
done

When cross-compiling, this is definitely something we don’t want (mixing
build and host/target libraries/include).

good call

personally, i dont think these lines should ever be added … a properly
configure toolchain will search /usr/local automatically anyways

I’m pretty sure I added this to fix building on *BSD, but I’ll disable
it for now, until I figure out why that was needed.

You can do that, to check for cross-compiling:

if test $build != $host; then
do_that_usr_local_stuff
fi> De : “Sam Lantinga”

I’m pretty sure I added this to fix building on *BSD, but I’ll disable
it for now, until I figure out why that was needed.

You can do that, to check for cross-compiling:

better to check $cross_compiling

imho, it still doesnt change the fact that the stuff shouldnt be there at all,
even for native builds
-mikeOn Friday 12 May 2006 03:19, Patrice MANDIN wrote:

De : “Sam Lantinga”