Bug 879978 - Install nssck.api template in a place where mod_revocator can access it
Summary: Install nssck.api template in a place where mod_revocator can access it
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Fedora
Classification: Fedora
Component: nss
Version: 18
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: Elio Maldonado Batiz
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: 782671
TreeView+ depends on / blocked
 
Reported: 2012-11-25 19:33 UTC by Elio Maldonado Batiz
Modified: 2013-03-11 23:13 UTC (History)
7 users (show)

Fixed In Version: nss-3.14-11.fc18
Clone Of:
Environment:
Last Closed: 2013-03-11 23:13:56 UTC
Type: Bug
Embargoed:


Attachments (Terms of Use)
spec file changes to install nsscki.api for mod_revocator - in patch form (2.98 KB, patch)
2012-11-26 03:19 UTC, Elio Maldonado Batiz
no flags Details | Diff
spec file changes to install nsscki.pai in /usr/includes/nss3 (1.78 KB, patch)
2012-11-28 06:39 UTC, Elio Maldonado Batiz
no flags Details | Diff

Description Elio Maldonado Batiz 2012-11-25 19:33:27 UTC
Description of problem: mod_revocator needs to acccess the ck.api header template.

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

How reproducible: always

Steps to Reproduce:
1. Search for ck.api anywhere in /usr/lib{64}
3.
  
Actual results: It cannot be found
Expected results: I shoulld be found in some nested directory

Additional info: Bug 7862677 requests that mod_revocator replace its use of nss private headers with public ones. mod_revocator is implemented as an NSS pkcs #11 module that uses NSS's own ckfw facilites. ckfw is NSS's cryptokey framework and it provides the ck.api header. Via a perl script mod_revocator build fileswill generate its own module private header from the ck.api template. The ck.api file needs to be installed in an appropriate directory in the system for mod_revocator for to access it.

Comment 1 Elio Maldonado Batiz 2012-11-25 19:44:34 UTC
This ck.api is not a header properly but a template from which to generate a header so /usr/includes/nss3 may not be a good location and possibly be contrary to fedora packaging guidelines. Nathan Kinder suggested installing in an nss direcory nested within /usr/share/docs. I propose that it be installed as %{_usr}/share/doc/nss3/templates/ck.api.

Comment 2 Elio Maldonado Batiz 2012-11-26 03:19:01 UTC
Created attachment 651733 [details]
spec file changes to install nsscki.api for mod_revocator - in patch form

This actullay installs the nsscki.api in the two alternative locations mentioned before, the neted directory in the doc area and also in the nss incules. While doing some quick testing with mod_revocator I found that the latter location makes the work of modifying the mod_revocator spec file easier - at least for me. We should choose the most apprpriate method.

Comment 3 Elio Maldonado Batiz 2012-11-26 18:11:26 UTC
Comment on attachment 651733 [details]
spec file changes to install nsscki.api for mod_revocator - in patch form

Which of the two locations do you recommend?

Comment 4 Tom "spot" Callaway 2012-11-27 14:49:32 UTC
I don't see any issue with that template being in /usr/include/nss3, especially if other code is using the template to generate a header for its own use.

That said, I don't mind if you prefer to put it in a docsdir, but unless you need to have it in two places, just pick one. (If you really do need it in multiple places, please symlink it.)

Also, please don't ever use things like this in the Release field:

10.1%{?dist}.installnssck.api.2

Just increment from:

10%{?dist}

to

11%{?dist}

And explain it as thoroughly as you'd like in the %changelog entry. :)

Comment 5 Elio Maldonado Batiz 2012-11-27 17:20:58 UTC
(In reply to comment #4)

> I don't see any issue with that template being in /usr/include/nss3,
> especially if other code is using the template to generate a header for its
> own use.

That's good to know. Makes it simpler for mod_revocator.spec.

> 
> That said, I don't mind if you prefer to put it in a docsdir, but unless you
> need to have it in two places, just pick one. (If you really do need it in
> multiple places, please symlink it.)
> 
Doing it in the headers is my preference. I showed for the docs option also as it was suggested in case the headers option would not be allowed but its okay. I propose then going with the headers option.

> Also, please don't ever use things like this in the Release field:
> 
> 10.1%{?dist}.installnssck.api.2

I never do, only when doing scratch builds to install in my system or to share with others for evaluation while working this out. We spot the scratch builds and their intent easily that way.
> 
> Just increment from:
> 
> 10%{?dist}
> 
> to
> 
> 11%{?dist}

Will certainly do with the official build.

> 
> And explain it as thoroughly as you'd like in the %changelog entry. :)

Will certainly do.

Thank you Spot for your prompt review.

Comment 6 Elio Maldonado Batiz 2012-11-28 06:39:23 UTC
Created attachment 653311 [details]
spec file changes to install nsscki.pai in /usr/includes/nss3

Keeps changes to mod_revocator.spec to a minimum.

Comment 7 Elio Maldonado Batiz 2013-03-11 23:13:56 UTC
Verified with Matt's help hat mod_revocator can indeed access the file from that location and can use it.


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