Cross-compilation Improvements

[Guillermo Amaral <g@…>]

I noticed the cross-compilation support in the Bitlbee configure script is a bit strict.

I've modified it to make it easier to integrate into Buildroot.

The modification are still backward compatible with how it worked before.

[Guillermo Amaral <g@…>]

I can't seem to attach files. I'm getting: OSError: [Errno 13] Permission denied: '/home/wilmer/trac/bitlbee/files/attachments/ticket/29f'

I've pastebinned the patch instead: ​http://pastebin.com/6gYStSAV

[Guillermo Amaral <g@…>]

Changes allow users to specify a sysroot which is used to prepend system libs. It could also be used to specify --sysroot via CFLAGS but I decided no to add it just yet. It may cause compilations to break in some older compilers.

STRIP is now also prepended with the TARGET, same as CC and LD.

We prevent PKG_CONFIG_LIBDIR from being overwritten, it's only set if it's not found. This could be the only thing that might break the previous cross-compilation setups, if they have PKG_CONFIG_LIBDIR in the environment, it will NOT be overwritten.

[dx]

Thanks for the patch! Looks mostly good. One issue though, systemlibdirs_cross+="$sysroot$i " seems to be a bashism, I'm getting errors when running it under dash (which is the default /bin/sh of debian systems)

$ dash ./configure --target='asd'
BitlBee configure
./configure: 248: ./configure: systemlibdirs_cross+=/lib64 : not found
./configure: 248: ./configure: systemlibdirs_cross+=/usr/lib64 : not found
./configure: 248: ./configure: systemlibdirs_cross+=/usr/local/lib64 : not found
./configure: 248: ./configure: systemlibdirs_cross+=/lib : not found
./configure: 248: ./configure: systemlibdirs_cross+=/usr/lib : not found
./configure: 248: ./configure: systemlibdirs_cross+=/usr/local/lib : not found

[Guillermo Amaral <g@…>]

You're right. /me shakes first at BASH.

Here's another version (dash tested): ​http://pastebin.com/YYErCUHL

[dx]

Applied. Thanks!

​https://github.com/bitlbee/bitlbee/commit/f5bbaba3b2015f039221945431f98027cb3d2d49

(btw, pull requests are fine too)

[wilmer]

FWIW I've also restored permissions so that attachments will hopefully work again.

Though for patches likely a pull request is better anyway yes.