[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Marcela Mašláňová changed: What|Removed |Added CC||cw...@alumni.drew.edu --- Comment #1 from Marcela Mašláňová 2011-04-18 03:50:21 EDT --- *** Bug 202876 has been marked as a duplicate of this bug. *** -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Lukas Berk changed: What|Removed |Added CC||lb...@redhat.com --- Comment #2 from Lukas Berk 2011-04-19 10:27:08 EDT --- I've tested this on fedora 13 and 14 (x86_64). Both now allow a simple script to include a 'use Data::Alias' statement without breaking. Let me know if further testing is required, (I'm having issue with my rawhide vms currently, I can report back with those once they're up and running again), if you have a more complicated perl script to test with I can test that as well. -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Petr Sabata 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 --- Comment #3 from Petr Sabata 2011-04-20 07:24:06 EDT --- Package doesn't build on F-14 due to systemtap. I haven't looked into this yet. I suppose this package is intented for F-15 and newer. -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 --- Comment #4 from Marcela Mašláňová 2011-04-20 07:49:58 EDT --- I plan it only for F-15+, because we do not have systemtap enabled in F-14. -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 --- Comment #5 from Petr Sabata 2011-04-20 07:55:46 EDT --- Package: perl-Data-Alias Version: 1.12 Release: 1.fc14 Sources: Data-Alias-1.12.tar.gz Patches: Data-Alias-systemtap.patch -- Package failed to build locally! Package successfully built in mock, fedora-rawhide-x86_64. Package successfully built in mock, fedora-rawhide-i386. Package successfully built in mock, fedora-15-x86_64. Package successfully built in mock, fedora-15-i386. Package failed to build in mock, fedora-14-x86_64. Package failed to build in mock, fedora-14-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 [ NOTE ] 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 [ NOTE ] Requires correct or justified otherwise [ FAIL ] 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 [ OK ] Compiler flags honor Fedora defaults or are justified [ OK ] 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 [ OK ] 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 [ OK ] Package uses parallel make [ -- ] In case of a web application, package installs date into /usr/share instead of /var/www [ OK ] 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: -- 1. A cosmetic issue: The second line of the last changelog entry should use two spaces indentation. 2. Duplicate Requires: perl(Exporter), perl(DynaLoader) -- rpm finds them automatically; remove them from Requires. 3. ...on the other hand, they should be BuildRequired. -- Configure bugmail: https:
[Bug 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 --- Comment #6 from Marcela Mašláňová 2011-04-20 08:56:14 EDT --- (In reply to comment #5) > NOTES: > -- > 1. A cosmetic issue: The second line of the last changelog entry should use > two > spaces indentation. Nitpicking but ok. > 2. Duplicate Requires: perl(Exporter), perl(DynaLoader) -- rpm finds them > automatically; remove them from Requires. Ok, will fix. > 3. ...on the other hand, they should be BuildRequired. Ok. Updated SRPM: http://mmaslano.fedorapeople.org/review/perl-Data-Alias-1.12-1.fc14.src.rpm -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Petr Sabata changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #7 from Petr Sabata 2011-04-20 09:13:09 EDT --- --- Data-Alias/perl-Data-Alias.spec 2011-04-14 10:05:19.0 +0200 +++ Data-Alias2/perl-Data-Alias.spec2011-04-20 15:11:17.0 +0200 @@ -9,6 +9,8 @@ # http://rt.cpan.org/Public/Bug/Display.html?id=67430 Patch0: Data-Alias-systemtap.patch +BuildRequires: perl(DynaLoader) +BuildRequires: perl(Exporter) BuildRequires: perl(ExtUtils::MakeMaker) BuildRequires: perl(File::Spec) BuildRequires: perl(Filter::Util::Call) @@ -17,7 +19,6 @@ BuildRequires: perl(strict) BuildRequires: perl(Test::More) BuildRequires: perl(warnings) -Requires: perl(Exporter), perl(DynaLoader) Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo $version)) %{?perl_default_filter} @@ -49,7 +50,7 @@ %files %defattr(-,root,root,-) -%doc Changes README +%doc README Changes %{perl_vendorarch}/auto/* %{perl_vendorarch}/Data* %{_mandir}/man3/* @@ -57,7 +58,7 @@ %changelog * Thu Apr 14 2011 Marcela Mašláňová 1.12-1 - Specfile autogenerated by cpanspec 1.79 for new release. This package - was resurected for 5.12.x + was resurected for 5.12.x * Mon Dec 7 2009 Stepan Kasal - 1.07-6 - rebuild against perl 5.10.1 -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Marcela Mašláňová changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Marcela Mašláňová 2011-04-28 06:50:48 EDT --- New Package SCM Request === Package Name: perl-Data-Alias Short Description: Comprehensive set of aliasing operations Owners: mmaslano psabata ppisar 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs- --- Comment #9 from Jason Tibbitts 2011-04-29 12:34:06 EDT --- This package already appears to exist in the package database, so I cannot process a new package request for it. If there's something special you would like for me to do, please describe it here and re-raise the fedora-cvs flag. -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 --- Comment #10 from Marcela Mašláňová 2011-05-02 07:28:50 EDT --- I re-reviewed this package, because it was marked as dead for a year. Now I created ticket for rel-engs to add it back to collection. I hope that's it and it will be working for rawhide. https://fedorahosted.org/rel-eng/ticket/4684 -- 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 697411] Review Request: perl-Data-Alias - Comprehensive set of aliasing operations
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=697411 Petr Sabata changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Last Closed||2011-05-11 09:01:53 --- Comment #11 from Petr Sabata 2011-05-11 09:01:53 EDT --- Ticket 4684 is fixed, Data::Alias unblocked for 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