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