[Bug 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Parse-Debian-Packages- |perl-Parse-Debian-Packages-
   |0.03-2.el7  |0.03-2.el6



--- Comment #22 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.el6 has been pushed to the Fedora EPEL 6
stable repository.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-06-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Parse-Debian-Packages- |perl-Parse-Debian-Packages-
   |0.03-2.fc21 |0.03-2.el7



--- Comment #21 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.el7 has been pushed to the Fedora EPEL 7
stable repository.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Parse-Debian-Packages- |perl-Parse-Debian-Packages-
   |0.03-2.fc22 |0.03-2.fc21



--- Comment #20 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.fc21 has been pushed to the Fedora 21 stable
repository.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||perl-Parse-Debian-Packages-
   ||0.03-2.fc22
 Resolution|--- |ERRATA
Last Closed||2015-05-27 12:10:24



--- Comment #19 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.fc22 has been pushed to the Fedora 22 stable
repository.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #18 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.el7 has been pushed to the Fedora EPEL 7
testing repository.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #12 from Petr Šabata psab...@redhat.com ---
Feel free to push and build the package, Igor.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #13 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
(In reply to Petr Šabata from comment #12)
 Feel free to push and build the package, Igor.

Oh, I really forget about this. Pushed to all branches. Soon will add in bodhi.
Thanks!

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #16 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.fc21 has been submitted as an update for
Fedora 21.
https://admin.fedoraproject.org/updates/perl-Parse-Debian-Packages-0.03-2.fc21

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #15 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.el6 has been submitted as an update for
Fedora EPEL 6.
https://admin.fedoraproject.org/updates/perl-Parse-Debian-Packages-0.03-2.el6

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #14 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.el7 has been submitted as an update for
Fedora EPEL 7.
https://admin.fedoraproject.org/updates/perl-Parse-Debian-Packages-0.03-2.el7

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2015-05-14 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #17 from Fedora Update System upda...@fedoraproject.org ---
perl-Parse-Debian-Packages-0.03-2.fc22 has been submitted as an update for
Fedora 22.
https://admin.fedoraproject.org/updates/perl-Parse-Debian-Packages-0.03-2.fc22

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Igor Gnatenko i.gnatenko.br...@gmail.com changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #10 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
New Package SCM Request
===
Package Name: perl-Parse-Debian-Packages
Short Description: Parse the data from a Debian Packages.gz
Upstream URL: http://search.cpan.org/dist/Parse-Debian-Packages/
Owners: ignatenkobrain
Branches: f20 f21 el6 epel7
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-12-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #11 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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

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

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #9 from Petr Šabata psab...@redhat.com ---
Ack, approving.

Please, add perl-sig to InitialCC when requesting branches.  Thanks.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

Igor Gnatenko i.gnatenko.br...@gmail.com changed:

   What|Removed |Added

 Blocks||1168263




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=1168263
[Bug 1168263] Review Request: multistrap - multiple repository bootstrap
based on apt
-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260

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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #1 from Petr Šabata psab...@redhat.com ---
The standalone SPEC differs from the one the SRPM.
It's just a different e-mail address in the changelog so I'll proceed with the
review.

Wrong deps, both build-time and run-time.
Missing BRs: 
 - perl, called in SPEC
 - perl(strict), used at %check phase
Useless runtime dependency:
 - perl(Test::More), only used by the test suite

`Debian' should be capitalized, in both Summary and %description.

PERL_INSTALL_ROOT can be substituted with a more common DESTDIR.

You don't need to remove empty directories (line 28).

You wouldn't need to remove .packlist files (line 27) if you bumped the
ExtUtils::MakeMaker dependency to 6.76+ and added NO_PACKLIST=1 to the
Makefile.PL call (line 22).  This would only work in F21 and newer, however.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #2 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
(In reply to Petr Šabata from comment #1)
 The standalone SPEC differs from the one the SRPM.
 It's just a different e-mail address in the changelog so I'll proceed with
 the review.
 
 Wrong deps, both build-time and run-time.
 Missing BRs: 
  - perl, called in SPEC
  - perl(strict), used at %check phase
 Useless runtime dependency:
  - perl(Test::More), only used by the test suite
 
 `Debian' should be capitalized, in both Summary and %description.
 
 PERL_INSTALL_ROOT can be substituted with a more common DESTDIR.
 
 You don't need to remove empty directories (line 28).
 
 You wouldn't need to remove .packlist files (line 27) if you bumped the
 ExtUtils::MakeMaker dependency to 6.76+ and added NO_PACKLIST=1 to the
 Makefile.PL call (line 22).  This would only work in F21 and newer, however.

thank you for review.

This is my first perl package, so I didn't know what's right.

I'll fix all at this evening.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #3 from Petr Šabata psab...@redhat.com ---
No worries :)

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #4 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
 You wouldn't need to remove .packlist files (line 27) if you bumped the
 ExtUtils::MakeMaker dependency to 6.76+ and added NO_PACKLIST=1 to the
 Makefile.PL call (line 22)

I want to get this lib for F20, too. What I should do?

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #6 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
Spec URL:
https://raw.githubusercontent.com/ignatenkobrain/fuel-tmp/master/perl-Parse-Debian-Packages.spec
SRPM URL:
https://raw.githubusercontent.com/ignatenkobrain/fuel-tmp/master/perl-Parse-Debian-Packages-0.03-2.fc20.src.rpm

fixed other errors.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #5 from Petr Šabata psab...@redhat.com ---
(In reply to Igor Gnatenko from comment #4)
  You wouldn't need to remove .packlist files (line 27) if you bumped the
  ExtUtils::MakeMaker dependency to 6.76+ and added NO_PACKLIST=1 to the
  Makefile.PL call (line 22)
 
 I want to get this lib for F20, too. What I should do?

Then ignore that point.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #7 from Petr Šabata psab...@redhat.com ---
Still missing the `perl' BR.  The rest is fine.

-- 
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 1168260] Review Request: perl-Parse-Debian-Packages - Parse the data from a debian Packages.gz

2014-11-26 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1168260



--- Comment #8 from Igor Gnatenko i.gnatenko.br...@gmail.com ---
(In reply to Petr Šabata from comment #7)
 Still missing the `perl' BR.  The rest is fine.

ah. forget. now it should be okay. links the same.

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