[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #22 from Fedora Update System  ---
FEDORA-EPEL-2022-72f5f51db3 has been pushed to the Fedora EPEL 9 stable
repository.
If problem still persists, please make note of it in this bug report.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #21 from Fedora Update System  ---
FEDORA-2022-419c129c95 has been pushed to the Fedora 34 stable repository.
If problem still persists, please make note of it in this bug report.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #20 from Fedora Update System  ---
FEDORA-EPEL-2022-8cdac2641e has been pushed to the Fedora EPEL 8 stable
repository.
If problem still persists, please make note of it in this bug report.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #19 from Fedora Update System  ---
FEDORA-2022-133941f2cb has been pushed to the Fedora 35 stable repository.
If problem still persists, please make note of it in this bug report.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-11 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Fedora Update System  changed:

   What|Removed |Added

 Resolution|--- |ERRATA
 Status|ON_QA   |CLOSED
Last Closed||2022-05-12 01:12:31



--- Comment #18 from Fedora Update System  ---
FEDORA-2022-8debd65f1c has been pushed to the Fedora 36 stable repository.
If problem still persists, please make note of it in this bug report.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #17 from Fedora Update System  ---
FEDORA-EPEL-2022-72f5f51db3 has been pushed to the Fedora EPEL 9 testing
repository.

You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-72f5f51db3

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #16 from Fedora Update System  ---
FEDORA-2022-419c129c95 has been pushed to the Fedora 34 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing
--advisory=FEDORA-2022-419c129c95 \*`
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2022-419c129c95

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #15 from Fedora Update System  ---
FEDORA-EPEL-2022-8cdac2641e has been pushed to the Fedora EPEL 8 testing
repository.

You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-8cdac2641e

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #14 from Fedora Update System  ---
FEDORA-2022-133941f2cb has been pushed to the Fedora 35 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing
--advisory=FEDORA-2022-133941f2cb \*`
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2022-133941f2cb

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-04 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #13 from Fedora Update System  ---
FEDORA-2022-8debd65f1c has been pushed to the Fedora 36 testing repository.
Soon you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing
--advisory=FEDORA-2022-8debd65f1c \*`
You can provide feedback for this update here:
https://bodhi.fedoraproject.org/updates/FEDORA-2022-8debd65f1c

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information
on how to test updates.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #11 from Fedora Update System  ---
FEDORA-2022-8debd65f1c has been submitted as an update to Fedora 36.
https://bodhi.fedoraproject.org/updates/FEDORA-2022-8debd65f1c


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #10 from Fedora Update System  ---
FEDORA-EPEL-2022-72f5f51db3 has been submitted as an update to Fedora EPEL 9.
https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2022-72f5f51db3


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Paul Howarth  changed:

   What|Removed |Added

   Fixed In Version||perl-Parse-Distname-0.05-2.
   ||fc37
 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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-03 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #9 from Gwyn Ciesla  ---
(fedscm-admin):  The Pagure repository was created at
https://src.fedoraproject.org/rpms/perl-Parse-Distname


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #8 from Paul Howarth  ---
Thanks, Michal.

$ fedpkg request-branch epel9
https://pagure.io/releng/fedora-scm-requests/issue/44142
$ fedpkg request-repo perl-Parse-Distname 2073377
https://pagure.io/releng/fedora-scm-requests/issue/44143
$ fedpkg request-branch --repo perl-Parse-Distname f36
https://pagure.io/releng/fedora-scm-requests/issue/44144
$ fedpkg request-branch --repo perl-Parse-Distname f35
https://pagure.io/releng/fedora-scm-requests/issue/44145
$ fedpkg request-branch --repo perl-Parse-Distname f34
https://pagure.io/releng/fedora-scm-requests/issue/44146
$ fedpkg request-branch --repo perl-Parse-Distname epel9
https://pagure.io/releng/fedora-scm-requests/issue/44147
$ fedpkg request-branch --repo perl-Parse-Distname epel8
https://pagure.io/releng/fedora-scm-requests/issue/44148


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-05-02 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Michal Josef Spacek  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #7 from Michal Josef Spacek  ---
Source file is ok
Summary is ok
License is ok
Description is ok
URL and Source0 are ok
All tests passed
BuildRequires are ok

$ rpm -qp --requires perl-Parse-Distname-0.05-2.fc37.noarch.rpm | sort | uniq
-c | grep -v rpmlib
  1 perl(Carp)
  1 perl(Exporter) >= 5.57
  1 perl(:MODULE_COMPAT_5.34.1)
  1 perl(strict)
  1 perl(warnings)
Binary requires are Ok.

$ rpm -qp --provides perl-Parse-Distname-0.05-2.fc37.noarch.rpm |sort | uniq -c
  1 perl(Parse::Distname) = 0.05
  1 perl-Parse-Distname = 0.05-2.fc37
Binary provides are Ok.

$ rpmlint ./perl-Parse-Distname*
2 packages and 1 specfiles checked; 0 errors, 0 warnings, 0 badness; has taken
0.3 s
Rpmlint is ok

The package is in line with Fedora and Perl packaging guide lines.

Resolution:
The package can be approved.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-29 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Petr Pisar  changed:

   What|Removed |Added

  Flags||fedora-review?




-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #6 from Paul Howarth  ---
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Parse-Distname/perl-Parse-Distname-0.05-2.fc37.src.rpm

Spec URL is unchanged.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #5 from Michal Josef Spacek  ---
(In reply to Paul Howarth from comment #4)
> Ah, I see what you mean. I did actually have "%doc Changes README t/", which
> is unusual for me. I must have based my spec file on another package that
> included the tests like that. I'll drop the "t/" from the "%doc" line.

yes.

Please, update files.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #4 from Paul Howarth  ---
(In reply to Michal Josef Spacek from comment #3)
> (In reply to Paul Howarth from comment #2)
> > (In reply to Michal Josef Spacek from comment #1)
> > > "%doc Changes README t/"
> > > No contain t/ in doc
> > 
> > I don't usually package the tests. It's a habit from a long time ago when
> > you could get additional dependencies from documentation files.
> > I'm open to persuasion on this.
> 
> That's no about test packaging. We don't package test files in main package.
> I use grep to mine packages and there is no case for it.
> 
> Option is only extra package for tests.

Ah, I see what you mean. I did actually have "%doc Changes README t/", which is
unusual for me. I must have based my spec file on another package that included
the tests like that. I'll drop the "t/" from the "%doc" line.


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377



--- Comment #3 from Michal Josef Spacek  ---
(In reply to Paul Howarth from comment #2)
> (In reply to Michal Josef Spacek from comment #1)
> > @p...@city-fan.org 
> > 
> > "BuildRequires:  coreutils"
> > It is not needed, sure?
> 
> %{_fixperms} in the %install section is implemented using chmod from
> coreutils.

Ah, ok.

> > "%doc Changes README t/"
> > No contain t/ in doc
> 
> I don't usually package the tests. It's a habit from a long time ago when
> you could get additional dependencies from documentation files.
> I'm open to persuasion on this.

That's no about test packaging. We don't package test files in main package.
I use grep to mine packages and there is no case for it.

Option is only extra package for tests.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Paul Howarth  changed:

   What|Removed |Added

  Flags|needinfo?(p...@city-fan.org |
   |)   |



--- Comment #2 from Paul Howarth  ---
(In reply to Michal Josef Spacek from comment #1)
> @p...@city-fan.org 
> 
> "BuildRequires:  coreutils"
> It is not needed, sure?

%{_fixperms} in the %install section is implemented using chmod from coreutils.

> "BuildRequires:  perl(strict)"
> You could move to "Module Runtime" section

Yes, you're right, it should be there.

> "%doc Changes README t/"
> No contain t/ in doc

I don't usually package the tests. It's a habit from a long time ago when you
could get additional dependencies from documentation files.
I'm open to persuasion on this.


-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Michal Josef Spacek  changed:

   What|Removed |Added

  Flags||needinfo?(p...@city-fan.org
   ||)



--- Comment #1 from Michal Josef Spacek  ---
@p...@city-fan.org 

"BuildRequires:  coreutils"
It is not needed, sure?

"BuildRequires:  perl(strict)"
You could move to "Module Runtime" section

"%doc Changes README t/"
No contain t/ in doc


-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-28 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Paul Howarth  changed:

   What|Removed |Added

 Blocks||2079764





Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=2079764
[Bug 2079764] Upgrade perl-Perl-PrereqScanner-NotQuiteLite to 0.9916
-- 
You are receiving this mail because:
You are always notified about changes to this product and component
You are on the CC list for the bug.
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure


[Bug 2073377] Review Request: perl-Parse-Distname - Parse a distribution name

2022-04-27 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=2073377

Michal Josef Spacek  changed:

   What|Removed |Added

   Assignee|nob...@fedoraproject.org|mspa...@redhat.com
   Doc Type|--- |If docs needed, set a value
 Status|NEW |ASSIGNED
 CC||mspa...@redhat.com




-- 
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
https://bugzilla.redhat.com/show_bug.cgi?id=2073377
___
package-review mailing list -- package-review@lists.fedoraproject.org
To unsubscribe send an email to package-review-le...@lists.fedoraproject.org
Fedora Code of Conduct: 
https://docs.fedoraproject.org/en-US/project/code-of-conduct/
List Guidelines: https://fedoraproject.org/wiki/Mailing_list_guidelines
List Archives: 
https://lists.fedoraproject.org/archives/list/package-review@lists.fedoraproject.org
Do not reply to spam on the list, report it: 
https://pagure.io/fedora-infrastructure