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=668839 Matthew Harmsen <mharmsen@xxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |MODIFIED --- Comment #7 from Matthew Harmsen <mharmsen@xxxxxxxxxx> 2011-01-14 11:42:27 EST --- # cd pki/specs # svn diff pki-core.spec Index: pki-core.spec =================================================================== --- pki-core.spec (revision 1731) +++ pki-core.spec (working copy) @@ -1,10 +1,6 @@ -############################################################################### -### P A C K A G E H E A D E R ### -############################################################################### - Name: pki-core Version: 9.0.0 -Release: 1%{?dist} +Release: 3%{?dist} Summary: Certificate System - PKI Core Components URL: http://pki.fedoraproject.org/ License: GPLv2 @@ -12,13 +8,17 @@ BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) +# jss requires versioning to meet both build and runtime requirements +# tomcatjss requires versioning since version 2.0.0 requires tomcat6 +# pki-common-theme requires versioning to meet runtime requirements +# pki-ca-theme requires versioning to meet runtime requirements BuildRequires: cmake BuildRequires: java-devel >= 1:1.6.0 BuildRequires: jpackage-utils BuildRequires: jss >= 4.2.6-12 BuildRequires: ldapjdk -BuildRequires: nspr-devel >= 4.6.99 -BuildRequires: nss-devel >= 3.12.3.99 +BuildRequires: nspr-devel +BuildRequires: nss-devel BuildRequires: openldap-devel BuildRequires: osutil BuildRequires: pkgconfig @@ -84,7 +84,7 @@ For deployment purposes, Certificate System requires ONE AND ONLY ONE \ of the following "Mutually-Exclusive" PKI Theme packages: \ \ - * ipa-pki-theme (IPA deployments - IPA Theme) \ + * ipa-pki-theme (IPA deployments) \ * dogtag-pki-theme (Dogtag Certificate System deployments) \ * redhat-pki-theme (Red Hat Certificate System deployments) \ \ @@ -93,14 +93,6 @@ %description %{overview} -############################################################################### -### S U B P A C K A G E H E A D E R S ### -############################################################################### - -######################## -## pki-setup ## -######################## - %package -n pki-setup Summary: Certificate System - PKI Instance Creation & Removal Scripts Group: System Environment/Base @@ -119,10 +111,6 @@ %{overview} -######################## -## pki-symkey ## -######################## - %package -n pki-symkey Summary: Symmetric Key JNI Package Group: System Environment/Libraries @@ -130,7 +118,7 @@ Requires: java >= 1:1.6.0 Requires: jpackage-utils Requires: jss >= 4.2.6-12 -Requires: nss >= 3.12.3.99 +Requires: nss Provides: symkey = %{version}-%{release} @@ -145,17 +133,14 @@ %{overview} -######################## -## pki-native-tools ## -######################## - %package -n pki-native-tools Summary: Certificate System - Native Tools Group: System Environment/Base Requires: openldap-clients -Requires: nss >= 3.12.3.99 -Requires: nss-tools >= 3.12.3.99 +Requires: nss +Requires: nss-tools +Requires: pki-setup = %{version}-%{release} %description -n pki-native-tools These platform-dependent PKI executables are used to help make @@ -166,10 +151,6 @@ %{overview} -######################## -## pki-util ## -######################## - %package -n pki-util Summary: Certificate System - PKI Utility Framework Group: System Environment/Base @@ -211,10 +192,6 @@ %{overview} -######################## -## pki-java-tools ## -######################## - %package -n pki-java-tools Summary: Certificate System - PKI Java-Based Tools Group: System Environment/Base @@ -251,10 +228,6 @@ %{overview} -######################## -## pki-common ## -######################## - %package -n pki-common Summary: Certificate System - PKI Common Framework Group: System Environment/Base @@ -306,10 +279,6 @@ %{overview} -######################## -## pki-selinux ## -######################## - %package -n pki-selinux Summary: Certificate System - PKI Selinux Policies Group: System Environment/Base @@ -327,10 +296,6 @@ %{overview} -######################## -## pki-ca ## -######################## - %package -n pki-ca Summary: Certificate System - Certificate Authority Group: System Environment/Daemons @@ -361,10 +326,6 @@ %{overview} -######################## -## pki-silent ## -######################## - %package -n pki-silent Summary: Certificate System - Silent Installer Group: System Environment/Base @@ -391,10 +352,6 @@ %{overview} -############################################################################### -### P A C K A G E P R O C E S S I N G ### -############################################################################### - %prep @@ -412,29 +369,11 @@ %{__make} VERBOSE=1 %{?_smp_mflags} -############################################################################### -### P A C K A G E I N S T A L L A T I O N ### -############################################################################### - %install %{__rm} -rf %{buildroot} cd build -%{__make} install DESTDIR=%{buildroot} +%{__make} install DESTDIR=%{buildroot} INSTALL="install -p" - -############################################################################### -### S U B P A C K A G E I N S T A L L A T I O N ### -############################################################################### - -######################## -## pki-setup ## -######################## - - -######################## -## pki-symkey ## -######################## - cd %{buildroot}%{_libdir}/symkey %{__rm} symkey.jar %{__ln_s} symkey-%{version}.jar symkey.jar @@ -444,79 +383,6 @@ %{__ln_s} %{_libdir}/symkey/symkey.jar symkey.jar -######################## -## pki-native-tools ## -######################## - - -######################## -## pki-util ## -######################## - - -######################## -## pki-java-tools ## -######################## - - -######################## -## pki-common ## -######################## - - -######################## -## pki-selinux ## -######################## - - -######################## -## pki-ca ## -######################## - - -######################## -## pki-silent ## -######################## - - -############################################################################### -### P R E & P O S T I N S T A L L / U N I N S T A L L S C R I P T S ### -############################################################################### - -######################## -## pki-setup ## -######################## - - -######################## -## pki-symkey ## -######################## - - -######################## -## pki-native-tools ## -######################## - - -######################## -## pki-util ## -######################## - - -######################## -## pki-java-tools ## -######################## - - -######################## -## pki-common ## -######################## - - -######################## -## pki-selinux ## -######################## - %pre -n pki-selinux %saveFileContext targeted @@ -539,13 +405,6 @@ fi -######################## -## pki-ca ## -######################## - -%pre -n pki-ca - - %post -n pki-ca # This adds the proper /etc/rc*.d links for the script /sbin/chkconfig --add pki-cad || : @@ -564,32 +423,18 @@ fi -######################## -## pki-silent ## -######################## - - -############################################################################### -### I N V E N T O R Y O F F I L E S A N D D I R E C T O R I E S ### -############################################################################### - -######################## -## pki-setup ## -######################## - %files -n pki-setup %defattr(-,root,root,-) %doc base/setup/LICENSE %{_bindir}/pkicreate %{_bindir}/pkiremove %dir %{_datadir}/pki -%{_datadir}/pki/scripts/ +%dir %{_datadir}/pki/scripts +%{_datadir}/pki/scripts/pkicommon.pm +%dir %{_localstatedir}/lock/pki +%dir %{_localstatedir}/run/pki -######################## -## pki-symkey ## -######################## - %files -n pki-symkey %defattr(-,root,root,-) %doc base/symkey/LICENSE @@ -597,10 +442,6 @@ %{_libdir}/symkey/ -######################## -## pki-native-tools ## -######################## - %files -n pki-native-tools %defattr(-,root,root,-) %doc base/native-tools/LICENSE base/native-tools/doc/README @@ -610,28 +451,23 @@ %{_bindir}/setpin %{_bindir}/sslget %{_bindir}/tkstool -%dir %{_datadir}/pki %{_datadir}/pki/native-tools/ -######################## -## pki-util ## -######################## - %files -n pki-util %defattr(-,root,root,-) %doc base/util/LICENSE -%{_javadir}/pki/ +%dir %{_javadir}/pki +%{_javadir}/pki/pki-cmsutil-%{version}.jar +%{_javadir}/pki/pki-cmsutil.jar +%{_javadir}/pki/pki-nsutil-%{version}.jar +%{_javadir}/pki/pki-nsutil.jar %files -n pki-util-javadoc %defattr(-,root,root,-) %{_javadocdir}/pki-util-%{version}/ -######################## -## pki-java-tools ## -######################## - %files -n pki-java-tools %defattr(-,root,root,-) %doc base/java-tools/LICENSE @@ -654,23 +490,27 @@ %{_bindir}/PrettyPrintCert %{_bindir}/PrettyPrintCrl %{_bindir}/TokenInfo -%{_javadir}/pki/ +%{_javadir}/pki/pki-tools-%{version}.jar +%{_javadir}/pki/pki-tools.jar %files -n pki-java-tools-javadoc %defattr(-,root,root,-) %{_javadocdir}/pki-java-tools-%{version}/ -######################## -## pki-common ## -######################## - %files -n pki-common %defattr(-,root,root,-) %doc base/common/LICENSE -%{_javadir}/pki/ -%dir %{_datadir}/pki -%{_datadir}/pki/scripts/ +%{_javadir}/pki/pki-certsrv-%{version}.jar +%{_javadir}/pki/pki-certsrv.jar +%{_javadir}/pki/pki-cms-%{version}.jar +%{_javadir}/pki/pki-cms.jar +%{_javadir}/pki/pki-cmsbundle-%{version}.jar +%{_javadir}/pki/pki-cmsbundle.jar +%{_javadir}/pki/pki-cmscore-%{version}.jar +%{_javadir}/pki/pki-cmscore.jar +%{_datadir}/pki/scripts/functions +%{_datadir}/pki/scripts/pki_apache_initscript %{_datadir}/pki/setup/ %files -n pki-common-javadoc @@ -678,26 +518,18 @@ %{_javadocdir}/pki-common-%{version}/ -######################## -## pki-selinux ## -######################## - %files -n pki-selinux %defattr(-,root,root,-) %doc base/selinux/LICENSE %{_datadir}/selinux/modules/pki.pp -######################## -## pki-ca ## -######################## - %files -n pki-ca %defattr(-,root,root,-) %doc base/ca/LICENSE %{_initrddir}/pki-cad -%{_javadir}/pki/ -%dir %{_datadir}/pki +%{_javadir}/pki/pki-ca-%{version}.jar +%{_javadir}/pki/pki-ca.jar %dir %{_datadir}/pki/ca %{_datadir}/pki/ca/conf/ %{_datadir}/pki/ca/emails/ @@ -705,30 +537,34 @@ %{_datadir}/pki/ca/profiles/ca/ %{_datadir}/pki/ca/webapps/ %{_datadir}/pki/ca/setup/ -%dir %{_localstatedir}/lock/pki %dir %{_localstatedir}/lock/pki/ca -%dir %{_localstatedir}/run/pki %dir %{_localstatedir}/run/pki/ca -######################## -## pki-silent ## -######################## - %files -n pki-silent %defattr(-,root,root,-) %doc base/silent/LICENSE %{_bindir}/pkisilent -%{_javadir}/pki/ -%dir %{_datadir}/pki +%{_javadir}/pki/pki-silent-%{version}.jar +%{_javadir}/pki/pki-silent.jar %{_datadir}/pki/silent/ -############################################################################### -### C H A N G E L O G ### -############################################################################### - %changelog +* Thu Jan 13 2011 Matthew Harmsen <mharmsen@xxxxxxxxxx> 9.0.0-3 +- Bugzilla Bug #668839 - Review Request: pki-core +- Removed empty "pre" from "pki-ca" +- Consolidated directory ownership +- Corrected file ownership within subpackages +- Removed all versioning from NSS and NSPR packages + +* Thu Jan 13 2011 Matthew Harmsen <mharmsen@xxxxxxxxxx> 9.0.0-2 +- Bugzilla Bug #668839 - Review Request: pki-core +- Added component versioning comments +- Updated JSS from "4.2.6-10" to "4.2.6-12" +- Modified installation section to preserve timestamps +- Removed sectional comments + * Wed Dec 1 2010 Matthew Harmsen <mharmsen@xxxxxxxxxx> 9.0.0-1 - Initial revision. (kwright@xxxxxxxxxx & mharmsen@xxxxxxxxxx) # svn diff base/common/CMakeLists.txt Index: base/common/CMakeLists.txt =================================================================== --- base/common/CMakeLists.txt (revision 1731) +++ base/common/CMakeLists.txt (working copy) @@ -16,13 +16,12 @@ FILES setup/CertServer.directory setup/menu.xml - setup/web-app_2_3.dtd DESTINATION ${DATA_INSTALL_DIR}/setup/ PERMISSIONS - OWNER_EXECUTE OWNER_WRITE OWNER_READ - GROUP_EXECUTE GROUP_READ - WORLD_EXECUTE WORLD_READ + OWNER_WRITE OWNER_READ + GROUP_READ + WORLD_READ ) add_subdirectory(src) # svn commit base/common/CMakeLists.txt specs/pki-core.spec Sending base/common/CMakeLists.txt Sending specs/pki-core.spec Transmitting file data .. Committed revision 1732. -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review