Bug#614332: libsdl1.2: Build corrections and improvements
Guillem Jover
guillem at debian.org
Mon Feb 21 04:39:21 UTC 2011
Source: libsdl1.2
Source-Version: 1.2.14-6.1
Severity: normal
Tags: patch
Hi!
Here are several fixes to the build infrastructure, I'm attaching only
three patches because several of the changes touch the same lines, so
this avoids ordering and conflict resolution from your side, if you'd
prefer me to split each logical change into different patches, please
say so, and I'll resubmit.
Here's a description of each fix/change:
* Build-Depends:
- Remove libarts1-dev only libartsc0-dev seems to be needed, this
also matches the dependency from the -dev package.
- Change nasm arches from i386 kfreebsd-i386 to any-i386
- Change libasound2-dev arches from !kfreebsd-i386 !kfreebsd-amd64
!hurd-i386 to linux-any.
- Change libusbhid-dev arches from kfreebsd-i386 kfreebsd-amd64 to
kfreebsd-any.
- Line-wrapped the field, so that changes are easier to see in the
future.
* Use DEB_HOST_ARCH* variables instead of DEB_BUILD_ARCH* (HOST is
the target system, BUILD is the building one). This will fix
cross-compilation support (although I've not tested if there's
anything else to fix for that).
* Fix configure flags:
- Update --enable-dlopen to --enable-sdl-dlopen (although it's yes
by default).
- Remove inexistent --enable-dependency-tracking option.
- Refactor --disable-video-ggi into confflags.
- Add --enable-nas-shared=no to be consistent with the other audio
plugins.
- Add --enable-x11-shared=no, the reason for this one is that this way
we get proper dependencies, which otherwise we lack. The fact the
the -all package is getting an libx11 dependency at all is actually
a bug in the aalib package, which once fixed will imply libsdl does
not depend on libx11 at all.
- Add a new flavour_conffflags variable with all video plugins
disabled except for x11, which will match with the description
of the packages. And use the flag on the flavour configure calls.
- Add a print line stating the current flavour being configured and
its configure flags, to ease tracking what's going on.
I've not removed the inexistent --enable-debug option because the code
handling noop should probably set the correct CFLAGS then, which I've
not had time to implement and test.
thanks,
guillem
More information about the Pkg-sdl-maintainers
mailing list