--libs-only-L and --libs-only-l should be additive, not exclusive #73

Closed
opened 2014-08-18 22:16:46 +00:00 by dankegel · 1 comment
dankegel commented 2014-08-18 22:16:46 +00:00 (Migrated from github.com)

I had a build failure today that turned out to be an incompatibility in pkgconf.
Turns out
pkg-config --libs-only-L --libs-only-l
is supposed to print both -L and -l things, not just one of them.

http://kegel.com/pkgconf-libs-only.patch passes all tests (including my new ones).
Caveats:
--cflags-only-I might want same treatment
-L flags should be output before -l flags, but at moment pkgconf seems to not do this right for transitive dependencies? I should file a separate bug for that.

I had a build failure today that turned out to be an incompatibility in pkgconf. Turns out pkg-config --libs-only-L --libs-only-l is supposed to print both -L and -l things, not just one of them. http://kegel.com/pkgconf-libs-only.patch passes all tests (including my new ones). Caveats: --cflags-only-I might want same treatment -L flags should be output before -l flags, but at moment pkgconf seems to not do this right for transitive dependencies? I should file a separate bug for that.

I merged a patch based on yours in commit 025fc092.

Thanks.

I merged a patch based on yours in commit 025fc092. Thanks.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: ariadne/pkgconf#73
There is no content yet.