[Bug 701841] New: Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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: [e-mail]
        ReportedBy: [e-mail]
         QAContact: [e-mail]
                CC: [e-mail], [e-mail]
   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.

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|                            |652183(FE-JAVASIG)
         Depends on|                            |701840

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
                 CC|                            |[e-mail]
         AssignedTo|[e-mail]    |[e-mail]
               Flag|                            |fedora-review?

--- Comment #1 from Stanislav Ochotnicky  2011-05-12 10:52:30 EDT ---
I'll review this

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+

--- Comment #2 from Stanislav Ochotnicky  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.

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Blocks|652183(FE-JAVASIG)          |

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|                            |fedora-cvs?

--- Comment #3 from Orion Poplawski  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:

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  2011-05-28 11:07:54 EDT ---
Git done (by process-git-requests).

[Bug 701841] Review Request: cargo-parent - cargo parent pom

[Red Hat Bugzilla]

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  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |CLOSED
         Resolution|                            |NEXTRELEASE
        Last Closed|                            |2011-05-31 16:07:53

--- Comment #5 from Orion Poplawski  2011-05-31 16:07:53 EDT ---
Checked in and built.  Thanks all.