Bug 147510 - NSS has broken includes, or pkg-config, or something
NSS has broken includes, or pkg-config, or something
Product: Fedora
Classification: Fedora
Component: mozilla (Show other bugs)
All Linux
medium Severity medium
: ---
: ---
Assigned To: Christopher Aillon
Ben Levenson
Depends On:
  Show dependency treegraph
Reported: 2005-02-08 12:51 EST by Bill Nottingham
Modified: 2014-03-16 22:52 EDT (History)
3 users (show)

See Also:
Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2008-02-08 11:17:50 EST
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

  None (edit)
Description Bill Nottingham 2005-02-08 12:51:12 EST
nss/cdbhdl.h includes mcom_db.h, which is in dbm/

dbm/ is not searched when you use "pkg-config --cflags mozilla-nss"
Comment 1 Christopher Aillon 2005-03-31 06:17:01 EST
wtc, thoughts?
Comment 2 Wan-Teh Chang 2005-03-31 13:51:43 EST
The bug description is a little terse for me,
so my comments may be a little off.

In recent versions of NSS, cdbhdl.h is a private
header file that is only used by files in the
mozilla/security/nss/lib/softoken directory.
cdbhdl.h should not be included in the NSS "devel"
package because NSS users don't need this header

There is no need to release a dbm "devel" package
because NSS's "softoken" shared library is statically
linked with dbm.  In other words, dbm can be considered
an internal component of NSS even though in the Mozilla
source tree dbm lives outside NSS.

mcom_db.h doesn't need to be included in the NSS
"devel" package, either.  There is one exception: NSS
allows alternative cert and key database plugin (the
"rdb" shared library), and cert/key database
plugin implementors need the mcom_db.h file.

Bob Relyea and I should review the contents of the
NSS "devel" package.
Comment 3 Bill Nottingham 2005-03-31 14:08:52 EST
Where it came from was a program (mutt) that had NSS code that tried to include
cdbhdl.h. (It didn't build anyway for other reasons.)
Comment 4 Wan-Teh Chang 2005-03-31 14:21:23 EST
Does mutt have code that uses NSS, or code copied from
NSS?  If the former, the code shouldn't need to include
Comment 5 Bill Nottingham 2005-03-31 14:34:13 EST
It uses NSS (or, at least, tries to.)
Comment 6 Bob Relyea 2005-03-31 16:14:08 EST
Sounds like it's trying to include private NSS headers if it's getting cdbhdl.h.
Is there something Mutt is doing that it can't get to from the public API, or
was the public API just not clear. (or, alternately, what the public API was was
not clear;).

Comment 7 Bill Nottingham 2005-03-31 17:05:04 EST
Put it this way: once the includes are fixed, the mutt code doesn't even build
with current NSS. Hence, I'm guessing it was doing something wrong to begin
with, or was based on really ancient code. I haven't done much work trying to
fix the NSS support yet - for now, it's still built with openssl.
Comment 8 Matěj Cepl 2008-02-08 11:17:50 EST
Since this bugzilla report was filed, we have seriously upgraded Gecko-related
packages in Rawhide, which may have resolved this issue. Users who have
experienced this problem are encouraged to upgrade their system to the latest
version of their distribution available.

Closing this bug as CANTFIX. Please, reopen, if this bug is still reproducable
on the latest update of your distribution.

[This is mass-closing of bugs which seem to be too old and irrelevant anymore;
we are sorry, if we are closing your bug in mistake; please, don't hesitate to
reopen, if it is still alive issue.]

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