Opened 12 years ago

Closed 12 years ago

Last modified 11 months ago

#8734 closed patch

make cross-compiling sane out of the box; blackfin support

Reported by: SF/vapier Owned by: fingolfin
Priority: normal Component: Ports
Keywords: Cc:
Game:

Description

the configure script forces runtime tests in order to detect endian, type sizes, and other features that can just as easily be done by compile-only tests. once i cleaned that up, it allows the configure script to provide a pretty sane setup by default without having to hardcode a crazy list of hosts in the file.

it may look like a lot, but really most of the changes are to move code out of the 'if ! cross-compiling' block into common build time detection for everyone.

now building for Blackfin is trivial:
$ ./configure --host=bfin-linux-uclibc
Running ScummVM configure...
Looking for C++ compiler... bfin-linux-uclibc-g++
Checking for compiler version... 4.1.2, ok
Checking endianness... little
Type with 1 byte... char
Type with 2 bytes... short
Type with 4 bytes... int
Running on x86... no
Checking hosttype... linux-gnu
Checking whether building plugins was requested... no
Checking for Ogg Vorbis... no
Checking for Tremor... no
Checking for FLAC >= 1.0.1... no
Checking for MAD... no
Checking for ALSA >= 0.9... no
Checking for zlib... no
Checking for libmpeg2 >= 0.3.2... no
Checking for libfluidsynth... no
....

you also need to import config.sub from the GNU config project just like you've imported config.guess

Ticket imported from: #1821100. Ticket imported from: patches/839.

Attachments (1)

scummvm-cross-compile.patch (8.5 KB ) - added by SF/vapier 12 years ago.
scummvm-cross-compile.patch

Download all attachments as: .zip

Change History (6)

comment:1 by fingolfin, 12 years ago

I didn't have a chance to actually review your patch, but a quick test on my OS X machine shows that it's not quite ready for inclusion ;-)

$ ./configure
Running ScummVM configure...
Looking for C++ compiler... none found!
$

by SF/vapier, 12 years ago

Attachment: scummvm-cross-compile.patch added

scummvm-cross-compile.patch

comment:2 by SF/vapier, 12 years ago

the new one attached should fix it
File Added: scummvm-cross-compile.patch

comment:3 by fingolfin, 12 years ago

Owner: set to fingolfin
Status: newclosed
Summary: make cross-compiling sane out of the boxmake cross-compiling sane out of the box; blackfin support

comment:4 by fingolfin, 12 years ago

Works great now. Very useful addition indeed, thanks!

I also added config.sub and updated config.guess to the latest official version.

comment:5 by digitall, 11 months ago

Component: Ports
Note: See TracTickets for help on using tickets.