[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Perl-Critic-More-1.000 |perl-Perl-Critic-More-1.000 |-1.fc16 |-1.fc15 Resolution|RAWHIDE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #20 from Fedora Update System upda...@fedoraproject.org 2011-05-19 00:51:49 EDT --- perl-Perl-Critic-More-1.000-1.fc15 has been pushed to the Fedora 15 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2011-05-06 07:30:29 EDT --- perl-Perl-Critic-More-1.000-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-Perl-Critic-More-1.000-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #16 from Jason Tibbitts ti...@math.uh.edu 2011-05-05 14:20:54 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #17 from Petr Pisar ppi...@redhat.com 2011-05-05 15:44:18 EDT --- Jason-san, thank you for processing the request exactly as asked. Unfortunately, Marcela forgot to request InitialCC for perl-sig. Please add that one to f15 branch: Package Name: perl-Perl-Critic-More Branches: f15 Owners: ppisar mmaslano psabata InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #18 from Jason Tibbitts ti...@math.uh.edu 2011-05-05 15:51:40 EDT --- Should be good now. As always with things I have to manually edit, please double check and let me know if there are any issues. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #13 from Marcela Mašláňová mmasl...@redhat.com 2011-05-04 09:07:30 EDT --- (In reply to comment #12) I have no problem in adding perl-Perl-Critic-More into F15, provided it does not imply upgrading other packages. (AFAIK it doesn't. Marcela, verify it, I have no Fedora available now). http://koji.fedoraproject.org/koji/taskinfo?taskID=3049683 At least build in F-15 passed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #14 from Petr Pisar ppi...@redhat.com 2011-05-04 09:17:13 EDT --- (In reply to comment #13) (In reply to comment #12) I have no problem in adding perl-Perl-Critic-More into F15, provided it does not imply upgrading other packages. (AFAIK it doesn't. Marcela, verify it, I have no Fedora available now). http://koji.fedoraproject.org/koji/taskinfo?taskID=3049683 At least build in F-15 passed. And now the run-time part: Are all Requires satisfied in F15? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #15 from Marcela Mašláňová mmasl...@redhat.com 2011-05-04 09:47:56 EDT --- Package Change Request == Package Name: perl-Perl-Critic-More New Branches: f15 Owners: ppisar mmaslano psabata -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Flag|fedora-cvs? | --- Comment #10 from Marcela Mašláňová mmasl...@redhat.com 2011-05-03 03:44:14 EDT --- (In reply to comment #7) Petr, could we have an F-15 branch for this? It's used for the author/developer tests of Perl::Critic itself (as you probably know). I'm taking back my request for branch. What do you think about #9? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #11 from Paul Howarth p...@city-fan.org 2011-05-03 04:16:30 EDT --- As it stands I have the author tests disabled by default but runnable by using --with authortests. I was thinking of changing things so that the author tests were run by default but could be turned off using --without authortests but if there's a consensus against that I could leave things as they are. However, it would still be useful to have the package available for anyone wanting to run the author tests (which would include anyone doing development work on Perl::Critic). Another point is that if the author tests were enabled by default, it would certainly not be the only circular dependency amongst the perl modules in Fedora, and circular dependencies themselves aren't so much of an issue in an incrementally built distribution, as opposed to one that needed bootstrapping. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #12 from Petr Pisar ppi...@redhat.com 2011-05-03 05:05:48 EDT --- I have no problem in adding perl-Perl-Critic-More into F15, provided it does not imply upgrading other packages. (AFAIK it doesn't. Marcela, verify it, I have no Fedora available now). The circular dependencies are problem when upgrading perl interpreter between incompatible versions. Once you push incompatible interpreter, you need to rebuild everything from scratch. I'm preparing a tool for scratch rebuild and I will be pushing to break these superfluous dependencies. Regarding author tests: The one who develops Perl::Critic operates on sources usually and does not use Koji as part of development, thus I cannot see any interjection in these two build systems. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #9 from Petr Pisar ppi...@redhat.com 2011-05-02 14:58:53 EDT --- I'm not sure it's best idea to pull perl-Perl-Critic-More package into perl-Perl-Critic build-time dependencies, as it would create dependency cycle (perl-Perl-Critic-More BuildRequires perl-Perl-Critic for standard tests already). I think it makes more sense to run perl-Perl-Critic-More tests than author tests in perl-Perl-Critic. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #8 from Marcela Mašláňová mmasl...@redhat.com 2011-05-02 01:55:29 EDT --- Package Change Request == Package Name: perl-Perl-Critic-More New Branches: F-15 Owners: ppisar mmaslano psabata -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Paul Howarth p...@city-fan.org changed: What|Removed |Added CC||p...@city-fan.org --- Comment #7 from Paul Howarth p...@city-fan.org 2011-04-30 04:48:02 EDT --- Petr, could we have an F-15 branch for this? It's used for the author/developer tests of Perl::Critic itself (as you probably know). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2011-04-05 11:25:00 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-Perl-Critic-More-1.000 ||-1.fc16 Resolution||RAWHIDE Last Closed||2011-04-05 12:53:34 --- Comment #6 from Petr Pisar ppi...@redhat.com 2011-04-05 12:53:34 EDT --- Thank you for the review and the repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #2 from Petr Pisar ppi...@redhat.com 2011-04-04 09:40:25 EDT --- Fixed package on the same URL. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Sabata psab...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Petr Sabata psab...@redhat.com 2011-04-04 09:44:11 EDT --- --- perl-Perl-Critic-More.spec 2011-03-24 18:23:20.0 +0100 +++ perl-Perl-Critic-More.spec.new 2011-04-04 15:37:30.0 +0200 @@ -26,6 +26,10 @@ Requires: perl(Perl::MinimumVersion) = 0.14 Requires: perl(Readonly) = 1.03 +# Remove underspecified dependencies +%filter_from_requires /^perl(Readonly)\s*$/d +%filter_setup + %description This is a collection of Perl::Critic policies that are not included in the Perl::Critic core for a variety of reasons. -- Approving. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 --- Comment #4 from Petr Pisar ppi...@redhat.com 2011-04-04 10:05:41 EDT --- New Package SCM Request === Package Name: perl-Perl-Critic-More Short Description: Supplemental policies for Perl::Critic Owners: ppisar mmaslano psabata Branches: InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580
--- Comment #1 from Petr Sabata psab...@redhat.com 2011-03-30 05:09:40 EDT ---
Package: perl-Perl-Critic-More
Version: 1.000
Release: 1.fc16
Sources: Perl-Critic-More-1.000.tar.gz
Patches:
--
Package successfully built locally.
Package successfully built in mock, fedora-rawhide-x86_64.
Package successfully built in mock, fedora-rawhide-i386.
Package successfully built in koji, dist-rawhide.
MUST items:
[ OK ] Package does NOT include pre-built binaries or libraries
[ OK ] Spec file is legible and written in American english
[ OK ] Package successfully builds on at least one supported primary
architecture
[ -- ] All ExcludeArch tags valid, referencing proper bug reports
[ OK ] Package obeys FHS (with _libexecdir and /srv exceptions)
[ OK ] No errors reported by rpmlint
[ OK ] Changelog present and properly formatted
[ OK ] Package does NOT include Packager, Vendor, Copyright or PreReq tags
[ OK ] Source tags are working URLs and sources match upstream or justified
otherwise
[ OK ] Requires correct or justified otherwise
[ OK ] BuildRequires correct or justified otherwise
[ OK ] All file names are in proper UTF-8 encoding
[ OK ] All plain text failes are in proper UTF-8 encoding
[ -- ] Large documentation files are located in doc subpackage
[ OK ] All documentation prefixed with %doc
[ OK ] Documentation is NOT executable
[ OK ] No files in %doc are needed at run-time
[ -- ] Compiler flags honor Fedora defaults or are justified
[ -- ] Package generates useful debuginfo packages
[ -- ] Header files are placed in devel subpackage
[ -- ] Unversioned shared libraries are placed in devel subpackage
[ -- ] Pkgconfig files are placed in devel subpackage
[ -- ] Full-versioned Requires of the base package in subpackages
[ -- ] Package calls ldconfig in post and postun sections for all
subpackages, if applicable
[ -- ] Static libraries are provided by static subpackage
[ OK ] Package contains no static executables unless approved by FESCo
[ OK ] Package does NOT bundle any system libraries
[ -- ] RPath not used for anything besides internal libraries
[ -- ] All config files are marked noreplace or justified otherwise
[ OK ] No config files are located under /usr
[ -- ] Package contains a SystemV-compatible initscript
[ -- ] A GUI application installs a proper desktop file
[ -- ] All desktop files are installed by desktop-file-install or justified
otherwise
[ OK ] Package consistently uses macros
[ -- ] makeinstall macro is used only if make install DESTDIR=%{buildroot}
does NOT work
[ -- ] Macros in Summary and description are expandable at build-time
[ -- ] globals used in place of defines
[ -- ] Locales handled correctly -- package requires gettext and uses
find_lang, if applicable
[ -- ] Scriptlets are sane
[ OK ] Package is not relocatable unless justified
[ OK ] Package contains only acceptable code or content
[ OK ] Package owns all the files and directories it creates, installs and/or
uses unless those are already owned by another package
[ OK ] files sections do NOT contain duplicate files except for licenses
[ OK ] All files sections use defattr or justify otherwise
[ OK ] Package does NOT cause any conflicts
[ OK ] Package does NOT contain kernel modules
[ OK ] Package does NOT bundle fonts or other general purpose data
[ NOTE ] Final Requires and Provides are sane
SHOULD items:
[ OK ] The Summary does NOT end with a period
[ OK ] Package does NOT include BuildRoot tag, clean section or buildroot
removal in install section
[ OK ] Package should preserve files timestamps
[ OK ] Package does NOT explicitly BuildRequire bash, bzip2, coreutils, cpio,
diffutils, fedora-release, findutils, gawk, gcc, gcc-c++, grep, gzip, info,
make, patch, redhat-rpm-config, rpm-build, sed, shadow-utils, tar, unzip,
util-linux-ng, which or xz
[ OK ] Description does NOT consist of lines longer than 80 characters
[ -- ] Package uses parallel make
[ -- ] In case of a web application, package installs date into /usr/share
instead of /var/www
[ -- ] All patches have a comment or an upstream bug link
[ -- ] Package installs manpages for all executables
[ OK ] Package contains check section and all tests pass
[ ?? ] Package works as expected
NOTES:
--
Filter out perl(Readonly) underspecified dependency.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Sabata psab...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||psab...@redhat.com AssignedTo|nob...@fedoraproject.org|psab...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 690580] Review Request: perl-Perl-Critic-More - Supplemental policies for Perl::Critic
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=690580 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Blocks||690569 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
