Skip Menu |

This queue is for tickets about the Glib CPAN distribution.

Report information
The Basics
Id: 44055
Status: resolved
Priority: 0/
Queue: Glib

People
Owner: Nobody in particular
Requestors: dmacks [...] netspace.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Missing space between linker flags
Date: Thu, 12 Mar 2009 02:01:26 -0400
To: bug-Glib [...] rt.cpan.org
From: Daniel Macks <dmacks [...] netspace.org>
Building Glib-1.201 on OS X 10.4 with either Apple's perl5.8.6 or fink's perl5.8.8 and all other dependencies (glib, etc) via at the level of latest GNOME-2.24, and ExtUtils::PkgConfig-1.12, I get a warning during Makefile.PL: Including generated API documentation... WARNING: EXTRA_META is not a known parameter. Checking if your kit is complete... Looks good 'EXTRA_META' is not a known MakeMaker parameter name. Note (probably harmless): No library found for -lintl-L/sw/lib Writing Makefile for Glib Should be "-lintl -L/sw/lib", two separate flags. Reason they're together is a bug in Makefile.PL: $glib->set_inc (' -I. ' . $glibcfg{cflags} . $gthreadcfg{cflags}); $glib->set_libs ($glibcfg{libs} . $gthreadcfg{libs}); Need a space between the $glibcfg and $gthreadcfg strings: $glib->set_inc (' -I. ' . $glibcfg{cflags} . ' ' . $gthreadcfg{cflags}); $glib->set_libs ($glibcfg{libs} . ' ' . $gthreadcfg{libs}); so the last cflags or libs of one doesn't merge with the first of the other. dan -- Daniel Macks dmacks@netspace.org http://www.netspace.org/~dmacks
On Thu Mar 12 02:02:12 2009, dmacks@netspace.org wrote: Show quoted text
> Need a space between the $glibcfg and $gthreadcfg strings: > > $glib->set_inc (' -I. ' . $glibcfg{cflags} . ' ' . $gthreadcfg{cflags}); > $glib->set_libs ($glibcfg{libs} . ' ' . $gthreadcfg{libs}); > > so the last cflags or libs of one doesn't merge with the first of the > other.
Ah, yep. Apparently, the pkg-config on my system always adds padding space at the end of these flags -- I think that's why nobody noticed yet. Change committed. Thanks for the report.