search

PerlGameDev / SDL

Rehashing the old perl SDL binding on cpan.org
http://search.cpan.org/dist/SDL
GNU General Public License v2.0
81 stars 29 forks source link

Build.PL ignores some options because of Getopt::Long #272

Open Civil opened 9 years ago

Civil commented 9 years ago

It seems, that Debian guys haven't reported that bug upstream, but they've got a patch that's essential for many distributions: http://anonscm.debian.org/cgit/pkg-perl/packages/libalien-sdl-perl.git/tree/debian/patches/fix-build-option

Otherwise it's impossible to create normal package - build system ignores "--with-sdl-config" option, tries to fetch SDLs sources (even if SDL is already installed) and tries to install package to /usr/local, which is wrong.

This affects both Alien-SDL 1.444.0 and 1.445_1 dev release from CPAN.

dod38fr commented 9 years ago

Oops, my bad. I should have reported that upstream... Thanks for following up.

jtpalmer commented 9 years ago

So, should we merge that?

dod38fr commented 9 years ago

@jtpalmer : yes. This patch will help other downstream distro (like Fedora, Mandriva) to build SDL perl using already compiled sdl libraries.

All the best

jtpalmer commented 9 years ago

@dod38fr I've patched the experimental branch and uploaded a dev release.