[Bug 553483] Review Request: dogtag-pki-console-ui - The Dogtag PKI Console User Interface

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=553483

--- Comment #2 from Matthew Harmsen <mharmsen at redhat.com> 2010-01-13 16:47:22 EST ---
Parag Nemade wrote:
> Hi,
>      I am not sure but I think all these dogtag*-ui packages should require some basic pki framework providing package, otherwise if someone just installs say dogtag-pki-console-ui package then he will just get some files installed from that package only. I think pki-common-ui should be Requires in all these -ui packages. Am I right? if so you need to add them to SPEC then.

Actually -- the intent of the UI packages was merely to separate brand-specific
"text/graphics" from their associated non-UI component (much like a branding
'skin'):

    * 'pki-common' REQUIRES 'pki-common-ui' which is PROVIDED by EITHER
'dogtag-pki-common-ui' OR 'redhat-pki-common-ui'
    * 'pki-ca' REQUIRES 'pki-ca-ui' which is PROVIDED by EITHER
'dogtag-pki-ca-ui' OR 'redhat-pki-ca-ui'
    * 'pki-kra' REQUIRES 'pki-kra-ui' which is PROVIDED by EITHER
'dogtag-pki-kra-ui' OR 'redhat-pki-kra-ui'
    * 'pki-ocsp' REQUIRES 'pki-ocsp-ui' which is PROVIDED by EITHER
'dogtag-pki-ocsp-ui' OR 'redhat-pki-ocsp-ui'
    * 'pki-tks' REQUIRES 'pki-tks-ui' which is PROVIDED by EITHER
'dogtag-pki-tks-ui' OR 'redhat-pki-tks-ui'
    * 'pki-ra' REQUIRES 'pki-ra-ui' which is PROVIDED by EITHER
'dogtag-pki-ra-ui' OR 'redhat-pki-ra-ui'
    * 'pki-tps' REQUIRES 'pki-tps-ui' which is PROVIDED by EITHER
'dogtag-pki-tps-ui' OR 'redhat-pki-tps-ui'
    * 'pki-console' REQUIRES 'pki-console-ui' which is PROVIDED by EITHER
'dogtag-pki-console-ui' OR 'redhat-pki-console-ui'

Additionally, the 'pki-common' component is ONLY REQUIRED by the Java/Tomcat
packages 'pki-ca', 'pki-kra', 'pki-ocsp', 'pki-tks', and 'pki-console'; it is
NOT REQUIRED by the Perl/Apache package 'pki-ra' NOR the C/C++/Apache package
'pki-tps'.

>      I tried to install all reviewed packages in Fedora but I see conflicts is not allowing me to install pki-common. May I know specific reason to have it conflicts with tomcat-native?I see both packages provides different files at different installation paths. As per https://fedoraproject.org/wiki/Packaging:Conflicts#Other_Uses_of_Conflicts: , we need to add some information in SPEC file. Sorry that I missed this in package review.

>      Can we have pki-common package updated with this conflict information in its spec file?

The Conflict problems associated with "tomcat-native" are detailed in
https://bugzilla.redhat.com/show_bug.cgi?id=441974, and appeared to stem from
attempting to run Tomcat using a different security model ( e. g. - openSSL
(tomcat-native) conflicting with NSS (tomcatjss->JSS->NSS)).  Although I had
originally suggested placing the "Conflicts:" keyword in the 'pki-common'
package, I believe that it may be more appropriate to place it within the
"tomcatjss" package?

Regardless of which package is chosen, are you suggesting that we need a
clearer explanation regarding the reason for this conflict under the
"%description" section?

>      Please remove "Requires: bash" as it looks unnecessary.

We used "Requires: bash" because I believe that rpmbuild always needs at least
one "Requires:", and we chose "Requires: bash" as the lone "Requires" statement
in many of the UI- packages.  Is something else more appropriate (as this is
what we ended up using in the "dogtag-pki-ui-ca" package)?

>
>
> Regards,
> Parag

Thanks,
-- Matt

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.