Hi Kartik,

>>> [...]
>>>
>>> I had to add a "my"at line 50:
>>>
>>> my $cflags = `sdl-config --cflags`;
>>>
>>> and line 54
>>>
>>> my $libs = `sdl-config  --libs`;
>>>
>>> [...]
>>>       

As for the 2 mentioned lines from inc/My/Builder/Darwin.pm I propose the
following change:

+ use Alien::SDL;
...
- my $cflags = `sdl-config --cflags`;
+ my $cflags = Alien::SDL->config('cflags');
...
- my $cflags = `sdl-config --libs`;
+ my $cflags = Alien::SDL->config('libs');

The idea of the new Alien::SDL is to keep all similar stuff in
Alien::SDL. The other thing is that pure calling of `sdl-config ...`
very likely will not work properly when you choose in Alien::SDL's
Build.PL to build SDL libs from sources. In this scenario we are not
installing SDL binaries in system location (e.g. /usr/local) but into so
called "share dir" of Alien-SDL distribution (which means sdl-config not
being in PATH).

--
kmx

Reply via email to