[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-10 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648

Emmanuel Seyman emman...@seyman.fr changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2015-02-10 05:41:37



--- Comment #8 from Emmanuel Seyman emman...@seyman.fr ---
Built for rawhide and mashed.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648

Emmanuel Seyman emman...@seyman.fr changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #6 from Emmanuel Seyman emman...@seyman.fr ---
(In reply to Petr Šabata from comment #5)

 Ack, everything fixed.  Approving.

Thanks, Petr.

New Package SCM Request
===
Package Name: perl-Palm
Short Description: Palm OS utility functions
Upstream URL: http://search.cpan.org/dist/Palm/
Owners: eseyman
Branches:
InitialCC: perl-sig

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648

Jon Ciesla limburg...@gmail.com changed:

   What|Removed |Added

  Flags|fedora-cvs? |fedora-cvs+



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648



--- Comment #7 from Jon Ciesla limburg...@gmail.com ---
Git done (by process-git-requests).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #5 from Petr Šabata psab...@redhat.com ---
(In reply to Emmanuel Seyman from comment #4)
 (In reply to Petr Šabata from comment #3)
  
  This is not a simple rename.  The distribution got split into two, at least
  -- this one and Palm::PDB.  The latter needs to be packaged before this one
  as it's your (and lbdb's) dependency.
 
 Yup, perl-Palm-PDB was reviewed in bug #1187873 and hit rawhide on
 2015-02-02.

I see.  Totally missed that.

  Wrong lettercase on the Obsoletes line.
  Mark the LICENSE file with %license; this is now mandatory.
  Missing BR: perl
  The minimum required version of EU::MM should be 6.76.  That's when support
  for NO_PACKLIST was introduced.
  You can substitute PERL_INSTALL_ROOT with DESTDIR.
  The URL still points to p5-Palm.
 
 All fixed.
 
 Spec URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm.spec
 SRPM URL:
 http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm-1.014-2.fc21.
 src.rpm

Ack, everything fixed.  Approving.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648

Petr Šabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
   Assignee|nob...@fedoraproject.org|psab...@redhat.com
  Flags||fedora-review?



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648



--- Comment #3 from Petr Šabata psab...@redhat.com ---
(In reply to Emmanuel Seyman from comment #2)
 (In reply to Petr Šabata from comment #1)
  The standalone spec differs from the one included in the SRPM.
 
 Wow, I'm off to a great start on this one. Source RPM rebuilt.

Indeed.

This is not a simple rename.  The distribution got split into two, at least --
this one and Palm::PDB.  The latter needs to be packaged before this one as
it's your (and lbdb's) dependency.

Wrong lettercase on the Obsoletes line.

Mark the LICENSE file with %license; this is now mandatory.

Missing BR: perl

The minimum required version of EU::MM should be 6.76.  That's when support for
NO_PACKLIST was introduced.

You can substitute PERL_INSTALL_ROOT with DESTDIR.

The URL still points to p5-Palm.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648



--- Comment #4 from Emmanuel Seyman emman...@seyman.fr ---
(In reply to Petr Šabata from comment #3)
 
 This is not a simple rename.  The distribution got split into two, at least
 -- this one and Palm::PDB.  The latter needs to be packaged before this one
 as it's your (and lbdb's) dependency.

Yup, perl-Palm-PDB was reviewed in bug #1187873 and hit rawhide on 2015-02-02.

 Wrong lettercase on the Obsoletes line.
 Mark the LICENSE file with %license; this is now mandatory.
 Missing BR: perl
 The minimum required version of EU::MM should be 6.76.  That's when support
 for NO_PACKLIST was introduced.
 You can substitute PERL_INSTALL_ROOT with DESTDIR.
 The URL still points to p5-Palm.

All fixed.

Spec URL: http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm.spec
SRPM URL:
http://people.parinux.org/~seyman/fedora/perl-Palm/perl-Palm-1.014-2.fc21.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648



--- Comment #2 from Emmanuel Seyman emman...@seyman.fr ---
(In reply to Petr Šabata from comment #1)
 The standalone spec differs from the one included in the SRPM.

Wow, I'm off to a great start on this one. Source RPM rebuilt.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1188648] Review Request: perl-Palm - Palm OS utility functions

2015-02-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1188648



--- Comment #1 from Petr Šabata psab...@redhat.com ---
The standalone spec differs from the one included in the SRPM.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review