Bug 589504
Summary: | Review Request: apache-commons-compress - Java API for working with tar, zip and bzip2 files | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Sandro Mathys <sandro> |
Component: | Package Review | Assignee: | Nobody's working on this, feel free to take it <nobody> |
Status: | CLOSED ERRATA | QA Contact: | Fedora Extras Quality Assurance <extras-qa> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | akurtako, fedora-package-review, lemenkov, mat.booth, notting |
Target Milestone: | --- | Flags: | akurtako:
fedora-review+
kevin: fedora-cvs+ |
Target Release: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | apache-commons-compress-1.0-4.fc13 | Doc Type: | Bug Fix |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2010-05-11 19:38:54 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 588764 |
Description
Sandro Mathys
2010-05-06 10:34:10 UTC
Hmm, why are there 2 bugs for this? See https://bugzilla.redhat.com/show_bug.cgi?id=589502 *** Bug 589502 has been marked as a duplicate of this bug. *** Because bugzilla seems to be a bit unstable today (I get proxy and mysql errors all day long) and While retrying to submit this review I accidentally created two bugs. Closed one as duplicate now. Awww, sorry...blocked the wrong bug :/ Bugzilla's errors are confusing me a little :/ I would really like to see this package build with maven2 as it is supposed to be done by upstream developers. All the additional maven integration work should be done too. We need this so any maven project depending on commons-compress can find it without further work from the packager. Additionally building with maven should make the jar a valid OSGi bundle suitable to be used in Eclipse and other OSGi containers. Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec SRPM URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-3.fc12.src.rpm I never used maven2 in a package before but by looking at another spec file I was able to come up with something that seems to work. Not sure if it's normal that pretty some checksums fail, tho. Also, rpmlint now shows a new warning but I think that can be ignored: apache-commons-compress.noarch: W: non-conffile-in-etc /etc/maven/fragments/apache-commons-compress Oh, that OSGi stuff is done automatically, isn't it? Or do I need to cp/install something? Thanks, I'm taking this one. Maven2 part looks good. Yes the OSGi stuff are just additional headers added automatically to the MANIFEST.MF file when you built with maven. Hello All! I'm not a java-expert, but I'm in doubts whether *-javadoc part shold *require* main package. Could someone clarify this - is *-javadoc sub-package useless w/o main package? Hi Peter, thanks for our input. I agree that it's probably not technically necessary. But the template from the guidelines do it as well and so I applied the same: https://fedoraproject.org/wiki/Packaging/Java#maven_2 The guidelines itself don't seem to mention it, so I think I could remove it. Does someone know more about that? (In reply to comment #9) > Hi Peter, thanks for our input. > > I agree that it's probably not technically necessary. But the template from the > guidelines do it as well and so I applied the same: > https://fedoraproject.org/wiki/Packaging/Java#maven_2 > > The guidelines itself don't seem to mention it, so I think I could remove it. > Does someone know more about that? Javadoc packages are perfectly usable without the main package. They are a bunch of html files after all. As this is not mandatory according to the guidelines it should be up to the packager to decide. Review: OK: rpmlint must be run on every package. OUTPUT: apache-commons-compress.noarch: W: spelling-error Summary(en_US) bzip -> zip, blip, b zip apache-commons-compress.noarch: W: spelling-error %description -l en_US bzip -> zip, blip, b zip apache-commons-compress.noarch: W: non-conffile-in-etc /etc/maven/fragments/apache-commons-compress apache-commons-compress-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs -> Java docs, Java-docs, Javanese Not a problem. OK: The package must be named according to the Package Naming Guidelines . OK: The spec file name must match the base package %{name}, in the format %{name}.spec unless your package has an exemption. OK: The package must meet the Packaging Guidelines . OK: The package must be licensed with a Fedora approved license and meet the Licensing Guidelines . OK: The License field in the package spec file must match the actual license. OK: The spec file must be written in American English. OK: The spec file for the package MUST be legible. OK: The sources used to build the package must match the upstream source, as provided in the spec URL. OK: The package MUST successfully compile and build into binary rpms on at least one primary architecture. OK: All build dependencies must be listed in BuildRequires, except for any that are listed in the exceptions section of the Packaging Guidelines ; inclusion of those as BuildRequires is optional. Apply common sense. OK: Packages must NOT bundle copies of system libraries. OK: A package must own all directories that it creates. OK: A Fedora package must not list a file more than once in the spec file's %files listings. OK: Permissions on files must be set properly. OK: Each package must consistently use macros. OK: The package must contain code, or permissable content. OK: Large documentation files must go in a -doc subpackage. Javadocs subpackage. OK: If a package includes something as %doc, it must not affect the runtime of the application. OK: Packages must not own files or directories already owned by other packages. OK: All filenames in rpm packages must be valid UTF-8. Other problems: * You should not install in %build section. Please just remove the "install -d $MAVEN_REPO_LOCAL" line. * All your symlinks are broken "ln -s %{name}-%{version}.jar" should be "ln -s %{shortname}-%{version}.jar * You should not use %define but %global. See http://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define * You can make use of the %{name} macro on a few places (if you want to) - Javadoc subpackage Summary and Description. Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec SRPM URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-4.fc12.src.rpm Thanks for the review Alexander! (In reply to comment #11) > Other problems: > * You should not install in %build section. Please just remove the "install -d > $MAVEN_REPO_LOCAL" line. This line looks important to me. Also, I see no reason not to install something into the unpacked sources if necessary for the build. After all, mvn-jpp will write to that dir as well. Also, the maven template from the guidelines does the same (with mkdir instead of install): http://fedoraproject.org/wiki/Packaging:Java#maven_2 If you have a reason to believe mkdir would fit better I'm willing to replace the install by that, tho. > * All your symlinks are broken "ln -s %{name}-%{version}.jar" should be "ln -s > %{shortname}-%{version}.jar Awww, very good catch! Should have paid more attention as I changed that symlinking part a bit recently :/ FIXED. > * You should not use %define but %global. See > http://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define Right, forgot to check that for the new renamed version - and the jakarta-* thingy is older than this rule :) FIXED. > * You can make use of the %{name} macro on a few places (if you want to) - > Javadoc subpackage Summary and Description. True, FIXED. (In reply to comment #12) > Spec URL: http://red.fedorapeople.org/SRPMS/apache-commons-compress.spec > SRPM URL: > http://red.fedorapeople.org/SRPMS/apache-commons-compress-1.0-4.fc12.src.rpm > > Thanks for the review Alexander! > > (In reply to comment #11) > > Other problems: > > * You should not install in %build section. Please just remove the "install -d > > $MAVEN_REPO_LOCAL" line. > > This line looks important to me. Also, I see no reason not to install something > into the unpacked sources if necessary for the build. After all, mvn-jpp will > write to that dir as well. Also, the maven template from the guidelines does > the same (with mkdir instead of install): > http://fedoraproject.org/wiki/Packaging:Java#maven_2 > > If you have a reason to believe mkdir would fit better I'm willing to replace > the install by that, tho. Well, using install in %build section can make someone believe that you are putting smth in the buildroot, which you are certainly not doing. Using mkdir is preferred to not confuse people. > > > * All your symlinks are broken "ln -s %{name}-%{version}.jar" should be "ln -s > > %{shortname}-%{version}.jar > > Awww, very good catch! Should have paid more attention as I changed that > symlinking part a bit recently :/ FIXED. > > > * You should not use %define but %global. See > > http://fedoraproject.org/wiki/Packaging:Guidelines#.25global_preferred_over_.25define > > Right, forgot to check that for the new renamed version - and the jakarta-* > thingy is older than this rule :) FIXED. > > > * You can make use of the %{name} macro on a few places (if you want to) - > > Javadoc subpackage Summary and Description. > > True, FIXED. Thanks, This package is APPROVED. Okay, will change this to mkdir instead of install before I add it to cvs. Thanks again for the review, Alexander. New Package CVS Request ======================= Package Name: apache-commons-compress Short Description: Java API for working with tar, zip and bzip2 files Owners: red Branches: F-12 F13 InitialCC: CVS done (by process-cvs-requests.py). apache-commons-compress-1.0-4.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/apache-commons-compress-1.0-4.fc12 apache-commons-compress-1.0-4.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/apache-commons-compress-1.0-4.fc13 apache-commons-compress-1.0-4.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. apache-commons-compress-1.0-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. |