Bug 671030 - Review Request: dogtag-pki-theme - Certificate System, Dogtag PKI Theme Components
Summary: Review Request: dogtag-pki-theme - Certificate System, Dogtag PKI Theme Compo...
Keywords:
Status: CLOSED RAWHIDE
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
low
medium
Target Milestone: ---
Assignee: Parag AN(पराग)
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: Dogtag-to-Fedora
TreeView+ depends on / blocked
 
Reported: 2011-01-19 23:38 UTC by Kevin Wright
Modified: 2015-07-13 04:57 UTC (History)
6 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2011-04-27 20:45:05 UTC
Type: ---
Embargoed:
panemade: fedora-review+
j: fedora-cvs+


Attachments (Terms of Use)

Description Kevin Wright 2011-01-19 23:38:16 UTC
Need to get dogtag-pki-theme built in koji.

Summary: Certificate System - Dogtag PKI Theme Components

spec file and src.rpm will be posted shortly.

Comment 2 Parag AN(पराग) 2011-01-20 10:14:44 UTC
1) Package failed to build as its missing BR: java
2) some packages owns /usr/share/pki and some not. But i think its already owned by pki-setup now.
3) Add following in %install
chmod 755 %{buildroot}%{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl

and remove from %files of dogtag-pki-tps-theme
%attr(00755,root,root) %{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl

This will make sure, duplicate files warning will not come in build.log

Comment 3 Matthew Harmsen 2011-01-20 18:34:14 UTC
(In reply to comment #2)
> 1) Package failed to build as its missing BR: java
> 2) some packages owns /usr/share/pki and some not. But i think its already
> owned by pki-setup now.
> 3) Add following in %install
> chmod 755 %{buildroot}%{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl
> 
> and remove from %files of dogtag-pki-tps-theme
> %attr(00755,root,root) %{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl
> 
> This will make sure, duplicate files warning will not come in build.log

Updated SPEC and SRPM specified below address comments 1) and 3):

Spec URL:
http://mharmsen.fedorapeople.org/dogtag-pki-theme.spec

SRPM URL:
http://mharmsen.fedorapeople.org/dogtag-pki-theme-9.0.0-2.fc12.src.rpm

In reference to comment 2), Dogtag consists of numerous subsystems which can be installed independently of each other.  As a consequence of this flexibility, several components may need to have ownership of the '/usr/share/pki' directory, as they may be the only component on a given system.  Where possible, we have tried to isolate this to the lowest common dependency (e. g. - 'pki-setup' is required by all six top-level Dogtag subsystems).  However, in the case of "theme" packages such as "dogtag-pki-theme", no package requires this at build time, and "theme" packages may be installed without requiring other Dogtag packages such as "pki-setup" (actually, packages such as 'pki-ca' require "pki-ca-theme" which is something provided by "theme" packages such as "dogtag-pki-ca-theme").

Comment 4 Parag AN(पराग) 2011-01-21 09:23:43 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > 1) Package failed to build as its missing BR: java
> > 2) some packages owns /usr/share/pki and some not. But i think its already
> > owned by pki-setup now.
> > 3) Add following in %install
> > chmod 755 %{buildroot}%{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl
> > 
> > and remove from %files of dogtag-pki-tps-theme
> > %attr(00755,root,root) %{_datadir}/pki/tps-ui/cgi-bin/sow/cfg.pl
> > 
> > This will make sure, duplicate files warning will not come in build.log
> 
> Updated SPEC and SRPM specified below address comments 1) and 3):
> 
> Spec URL:
> http://mharmsen.fedorapeople.org/dogtag-pki-theme.spec
> 
> SRPM URL:
> http://mharmsen.fedorapeople.org/dogtag-pki-theme-9.0.0-2.fc12.src.rpm
> 
> In reference to comment 2), Dogtag consists of numerous subsystems which can be
> installed independently of each other.  As a consequence of this flexibility,
> several components may need to have ownership of the '/usr/share/pki'
> directory, as they may be the only component on a given system.  Where
> possible, we have tried to isolate this to the lowest common dependency (e. g.
> - 'pki-setup' is required by all six top-level Dogtag subsystems).  However, in
> the case of "theme" packages such as "dogtag-pki-theme", no package requires
> this at build time, and "theme" packages may be installed without requiring
> other Dogtag packages such as "pki-setup" (actually, packages such as 'pki-ca'
> require "pki-ca-theme" which is something provided by "theme" packages such as
> "dogtag-pki-ca-theme").

In that case, I think other subpackages  dogtag-pki-tks-theme, dogtag-pki-ocsp-theme, dogtag-pki-kra-theme, dogtag-pki-ca-theme should also own '/usr/share/pki'

Good to add comment somewhere in spec about directory ownership.

Comment 5 Matthew Harmsen 2011-01-22 00:27:02 UTC
(In reply to comment #4) 
> In that case, I think other subpackages  dogtag-pki-tks-theme,
> dogtag-pki-ocsp-theme, dogtag-pki-kra-theme, dogtag-pki-ca-theme should also
> own '/usr/share/pki'
> 

These four packages all require "dogtag-pki-common-theme", and so for these four packages we simply provide ownership of "/usr/share/pki" to "dogtag-pki-common-theme". 

> Good to add comment somewhere in spec about directory ownership.

Added comment in newly attached spec file:

Spec URL:
http://mharmsen.fedorapeople.org/dogtag-pki-theme.spec

SRPM URL:
http://mharmsen.fedorapeople.org/dogtag-pki-theme-9.0.0-3.fc12.src.rpm


Additionally, on-going development required me to add the following (all referenced in the latest changelog):

    * Augmented overview description.
    * Isolated and corrected EPEL information
    * fixed two additional bugs (source tarball was recreated)

P. S. - Although the source tarball was altered, the original tarball was simply disposed of because we do not yet have an official package.  Once an "official" package exists, all of the source tarballs will only be modified via external stand-alone "patch" files or recreated with a different version name once they become "official".

Comment 6 Parag AN(पराग) 2011-01-24 03:44:37 UTC
Thanks for the nice update. I am still on vacation for next few days. But I will not make this package to wait.
APPROVED.

Comment 7 Kevin Wright 2011-01-24 18:23:51 UTC
New Package CVS Request
=======================
Package Name: dogtag-pki-theme
Short Description: Certificate System - Dogtag PKI Theme Components
Owners: kwright
Branches: F-13, F-14, EL-5
InitialCC: ausil

Comment 8 Jason Tibbitts 2011-01-24 21:54:42 UTC
Git done (by process-git-requests).

Comment 9 Parag AN(पराग) 2011-04-27 07:05:03 UTC
Can this review be closed now?

Comment 10 Matthew Harmsen 2011-04-27 20:38:18 UTC
(In reply to comment #9)
> Can this review be closed now?

Yes

Comment 11 Kevin Wright 2011-04-27 20:45:05 UTC
closing


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