Please do not reply directly to this email. All additional comments should be made in the comments box of this bug.
Summary: Review Request: cargo-parent - cargo parent pom
https://bugzilla.redhat.com/show_bug.cgi?id=701841
Summary: Review Request: cargo-parent - cargo parent pom Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nobody@fedoraproject.org ReportedBy: orion@cora.nwra.com QAContact: extras-qa@fedoraproject.org CC: notting@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: ---
Spec URL: http://www.cora.nwra.com/~orion/fedora/cargo-parent.spec SRPM URL: http://www.cora.nwra.com/~orion/fedora/cargo-parent-4.7-1.fc15.src.rpm Description: This package contains the cargo parent pom.
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=701841
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks| |652183(FE-JAVASIG) Depends on| |701840
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=701841
Bug 701841 depends on bug 701840, which changed state.
Bug 701840 Summary: Review Request: codehaus-parent - codehaus parent pom https://bugzilla.redhat.com/show_bug.cgi?id=701840
What |Old Value |New Value ---------------------------------------------------------------------------- Status|NEW |ASSIGNED Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE
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=701841
Stanislav Ochotnicky sochotni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |ASSIGNED CC| |sochotni@redhat.com AssignedTo|nobody@fedoraproject.org |sochotni@redhat.com Flag| |fedora-review?
--- Comment #1 from Stanislav Ochotnicky sochotni@redhat.com 2011-05-12 10:52:30 EDT --- I'll review this
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=701841
Stanislav Ochotnicky sochotni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+
--- Comment #2 from Stanislav Ochotnicky sochotni@redhat.com 2011-05-12 11:24:49 EDT --- rpmlint: cargo-parent.noarch: W: summary-not-capitalized C cargo parent pom cargo-parent.noarch: W: invalid-url URL: http://cargo.codehaus.org/ <urlopen error timed out> cargo-parent.noarch: W: no-documentation cargo-parent.noarch: W: non-conffile-in-etc /etc/maven/fragments/cargo-parent cargo-parent.src: W: summary-not-capitalized C cargo parent pom cargo-parent.src: W: invalid-url Source0: cargo-parent-4.7.tar.gz 2 packages and 0 specfiles checked; 0 errors, 6 warnings.
Everything seems to be in order (including license, maven things, BRs/Rs) except for that summary (that needs improving to something a *bit* more descriptive). defattr macros are also not needed anymore (optional).
It would be nice to have above issues fixed but they are minor, package is APPROVED.
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=701841
Stanislav Ochotnicky sochotni@redhat.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Blocks|652183(FE-JAVASIG) |
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=701841
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flag| |fedora-cvs?
--- Comment #3 from Orion Poplawski orion@cora.nwra.com 2011-05-27 16:26:47 EDT --- Updated summary to: Parent pom file for cargo.codehaus.org project
New Package SCM Request ======================= Package Name: cargo-parent Short Description: Parent pom file for cargo.codehaus.org project Owners: orion Branches: f15 f14 el6 InitialCC:
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=701841
--- Comment #4 from Jason Tibbitts tibbs@math.uh.edu 2011-05-28 11:07:54 EDT --- Git done (by process-git-requests).
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=701841
Orion Poplawski orion@cora.nwra.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|ASSIGNED |CLOSED Resolution| |NEXTRELEASE Last Closed| |2011-05-31 16:07:53
--- Comment #5 from Orion Poplawski orion@cora.nwra.com 2011-05-31 16:07:53 EDT --- Checked in and built. Thanks all.
package-review@lists.fedoraproject.org