Closed
Bug 1268883
Opened 9 years ago
Closed 9 years ago
build/autoconf/nss.m4: support pkg-config
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
WONTFIX
People
(Reporter: wiz, Unassigned)
Details
Attachments
(1 file)
1.99 KB,
patch
|
mshal
:
review-
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; NetBSD amd64; rv:46.0) Gecko/20100101 Firefox/46.0
Build ID: 20160428084422
Steps to reproduce:
nss does not install the nss-config script any longer. Instead if supplies a pkg-config .pc file.
build/autoconf/nss.m4 does not know about this and still tries to use nss-config.
Reporter | ||
Updated•9 years ago
|
Attachment #8747063 -
Attachment is patch: true
Attachment #8747063 -
Attachment mime type: application/x-m4 → text/plain
Attachment #8747063 -
Flags: review?(mshal)
Reporter | ||
Updated•9 years ago
|
OS: Unspecified → All
Hardware: Unspecified → All
Version: 46 Branch → Trunk
Comment 1•9 years ago
|
||
Where do nss.pc and nss-config come from? I don't see either of them in the nss repo or tarballs. Are they normally added on by the distro or something? If so, do we need to support both nss-config and pkg-config?
Flags: needinfo?(wiz)
Reporter | ||
Comment 2•9 years ago
|
||
Sorry, my original bug report was incorrect, since I was only looking at pkgsrc. That provides a pkgconfig file, but no nss-config script.
It seems that some distributions provide a nss-config script, because I can't explain why the code using it would be there otherwise.
So it would probably be best to support both (and/or make nss install one of them by default).
What do you think?
Flags: needinfo?(wiz) → needinfo?(mshal)
Comment 3•9 years ago
|
||
It seems to me we'd either need to support both, or you'd have to work with wherever nss is being packaged to add the nss-config script there. I'll defer to ted or glandium though - they should have more insight here.
Flags: needinfo?(mshal) → needinfo?(mh+mozilla)
Comment 4•9 years ago
|
||
We have --with-system-foo=prefix things for iirc libpng and/or libjpeg. We should probably have something like that for nss. That could fallback to pkg-config in cases we know pkg-config scripts exist. Either way, we shouldn't touch nss.m4 at this point. See bug 1251497 and bug 1269513
Flags: needinfo?(mh+mozilla)
Comment 5•9 years ago
|
||
Comment on attachment 8747063 [details] [diff] [review]
Patch switching from nss-config to pkg-config.
r- since it seems this is all changing soon, though presumably we'll need to support pkg-config in the new system. Can you work with chmanchester and glandium to find out how this should be addressed nowadays? You can find them in #build or needinfo them.
Attachment #8747063 -
Flags: review?(mshal) → review-
Reporter | ||
Comment 6•9 years ago
|
||
I did a minimal effort try at converting from old-configure to moz.configure, but nss in old-configure sets NSS_LIBS and NSS_CFLAGS and I wasn't sure how to do that properly in moz.configure, and without the settings the build failed.
Since upstream nss provides neither pkg-config nor nss-config, this is not a proper solution anyway.
I'll try to get upstream to provide either or both of them and then we can revisit this topic.
Thanks for the feedback!
Status: UNCONFIRMED → RESOLVED
Closed: 9 years ago
Resolution: --- → WONTFIX
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•