Bug 193189

Summary: should have libpcap-devel
Product: [Fedora] Fedora Reporter: Jeremy Katz <katzj>
Component: tcpdumpAssignee: Martin Stransky <stransky>
Status: CLOSED RAWHIDE QA Contact:
Severity: medium Docs Contact:
Priority: medium    
Version: rawhideCC: dennis
Target Milestone: ---Keywords: Reopened
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2006-06-04 18:02:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 150224, 195764    

Description Jeremy Katz 2006-05-25 22:05:35 UTC
As libpcap is intended for developing against, there should be a -devel subpackage

Comment 1 Martin Stransky 2006-05-30 18:19:40 UTC
modified in rawhide...

Comment 2 Ville Skyttä 2006-06-03 18:38:34 UTC
There is one issue with the implementation as of 14:3.9.4-6: libpcap.so is in
the main libpcap package, and libpcap-devel does not have a dependency on libpcap.

The result is that packages that have "BuildRequires: libpcap-devel" will not
have libpcap pulled in in various build roots, and because the *.so symlink is
in the main package and thus not available, will become statically linked with
libpcap.a, which is unexpected.

Fix: move the libpcap.so symlink to libpcap-devel and/or add a dependency on
libpcap to libpcap-devel.  Additionally, is there a need to ship the static lib
in the first place?

One more thing while at it: libpcap-devel ships the empty /usr/include/net
directory which seems odd, perhaps that can be dropped?

Comment 3 Dennis Gilmore 2006-06-03 19:14:29 UTC
going by Fedora Extras policy,  which new Core packages must use.  you need to 
put the .so files in the devel packages  and you need to have the 
libpcap-devel package Require a versioned libpcap package

Comment 4 Jeremy Katz 2006-06-04 18:02:16 UTC
Agreed -- fixed these up in CVS