Bug 233925 - broken pkgconfig file
Summary: broken pkgconfig file
Alias: None
Product: Fedora
Classification: Fedora
Component: compat-libosip2   
(Show other bugs)
Version: 6
Hardware: All Linux
Target Milestone: ---
Assignee: Jeffrey C. Ollie
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2007-03-25 21:09 UTC by Patrice Dumas
Modified: 2007-11-30 22:12 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2007-04-17 15:59:17 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)
correct pkgconfig file (340 bytes, patch)
2007-03-25 23:55 UTC, Patrice Dumas
no flags Details | Diff
corresponding spec file patch (1.14 KB, patch)
2007-03-25 23:56 UTC, Patrice Dumas
no flags Details | Diff

Description Patrice Dumas 2007-03-25 21:09:51 UTC
Description of problem:
The cflags is not right, since the includedir is missing:
pkg-config --cflags libosip2-2.2.2
should be 
-I/usr/include/osip2-2.2.2 -DOSIP_MT

I guess that one should have
Cflags: -I${includedir} -DOSIP_MT

I can do a patch if you want.

Version-Release number of selected component (if applicable):


How reproducible:

Steps to Reproduce:
Actual results:

Expected results:

Additional info:

Comment 1 Patrice Dumas 2007-03-25 21:11:27 UTC
This is unrelated, but I suggest using
sed -i.compat -e
instead of
%{__perl} -pi.compat -e

Comment 2 Patrice Dumas 2007-03-25 22:01:33 UTC
I verified that the same bug is present in libosip2.pc too
(although it cannot show up in fedora), I won't open another 
bug, though.

Comment 3 Jeffrey C. Ollie 2007-03-25 23:34:30 UTC
That'd be great if you could generate a patch...

Comment 4 Patrice Dumas 2007-03-25 23:55:57 UTC
Created attachment 150863 [details]
correct pkgconfig file

Comment 5 Patrice Dumas 2007-03-25 23:56:37 UTC
Created attachment 150864 [details]
corresponding spec file patch

Comment 6 Patrice Dumas 2007-03-26 00:01:26 UTC
Also unrelated, I don't think that %{?dist} should be in the
Provides: and Obsoletes:, otherwise the obsoleted version
will depend on the current %{?dist} value which is wrong 
(it is less problematic for the Provides:).

I also thing that those Provides and Obsoletes shouldn't be
there. Provides are automatically handled by the dependencies 
on the soname, and, in my opinion, upgrading from a previous
version should lead to the latest version, not the compat-

Comment 7 Jeffrey C. Ollie 2007-04-17 15:59:17 UTC
Fixed in compat-libosip2-2.2.2-11

Note You need to log in before you can comment on or make changes to this bug.