[Bug 986641] Review Request: ghc-shakespeare-js - Compile-time variable interpolation for Javascript
https://bugzilla.redhat.com/show_bug.cgi?id=986641 Jens Petersen peter...@redhat.com changed: What|Removed |Added Flags|needinfo?(petersen@redhat.c | |om) | --- Comment #4 from Jens Petersen peter...@redhat.com --- (In reply to Sébastien Willmann from comment #3) alson - also Remove the space before the final dot. Add a dot at the end of the second paragraph. Okay sure thanks - I will fix those. See https://github.com/yesodweb/shakespeare/pull/123 The package seems to contain tests, maybe you can add a check section? Right, but we don't have hspec packaged in Fedora yet. In fact we have been kind of systematically ignoring test-frameworks so it is probably about time we started to package some of them, but I don't think it needs to block this review. :) (I am also planning to update to latest version but that could be done after - it needs an update of shakespeare anyway.) -- You are receiving this mail because: 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 1063171] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||ch...@chriscowley.me.uk --- Comment #1 from Christopher Meng cicku...@gmail.com --- *** Bug 823661 has been marked as a duplicate of this bug. *** -- 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 823661] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=823661 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Resolution|NOTABUG |DUPLICATE --- Comment #11 from Christopher Meng cicku...@gmail.com --- *** This bug has been marked as a duplicate of bug 1063171 *** -- 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 1063171] New: Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 Bug ID: 1063171 Summary: Review Request: lessfs - An inline data deduplicating filesystem Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: cicku...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://cicku.me/lessfs.spec SRPM URL: http://cicku.me/lessfs-1.7.0-1.fc21.src.rpm Description: Lessfs is an userspace fuse-based inline data de-duplicating filesystem that includes support for lzo or QuickLZ compression and encryption. It also supports many databases as backend like hamsterdb, snappy, tokyocabinet and db4. Fedora Account System Username: cicku -- 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 1006116] Review Request: WALinuxAgent - The Windows Azure Linux Agent
https://bugzilla.redhat.com/show_bug.cgi?id=1006116 --- Comment #7 from Christopher Meng cicku...@gmail.com --- Have you tried to find a sponsor? -- You are receiving this mail because: 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 1063171] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 --- Comment #2 from Christopher Meng cicku...@gmail.com --- I added systemd support, but not sure it works or not since I haven't tested it carefully. -- 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 1063171] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 --- Comment #3 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info --- http://cicku.me/lessfs.spec not awailable unfortunately -- 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 1063171] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 --- Comment #4 from Christopher Meng cicku...@gmail.com --- (In reply to Pavel Alexeev (aka Pahan-Hubbitus) from comment #3) http://cicku.me/lessfs.spec not awailable unfortunately Your network problem maybe... Systemd support is not perfect, I'm still exploring... ;) -- 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 1063171] Review Request: lessfs - An inline data deduplicating filesystem
https://bugzilla.redhat.com/show_bug.cgi?id=1063171 --- Comment #5 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info --- Does it ready for review? I could do it. -- 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 1062942] Review Request: csv - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #5 from Fabio Alessandro Locati fabioloc...@gmail.com --- Hi Christopher :), Thank you for the inputs. I've implemented all the changes you mention and I have a question about 6 and 7. 6. I thought that the always-bumping rule was applicable only for versions that have been released. This believe has been confirmed by bug 1018038 which seems to have managed the spec file without bumping the version. 7. I thought I was already doing it. Am I doing it right or wrong? Here you can find the current version of the files: - Spec URL: http://data.fabiolocati.com/fedora/csv/csv.spec - SRPM URL: http://data.fabiolocati.com/fedora/csv/csv-0.07-2.fc20.src.rpm - Koji URL: http://koji.fedoraproject.org/koji/taskinfo?taskID=6511612 -- 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 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API
https://bugzilla.redhat.com/show_bug.cgi?id=1047647
--- Comment #9 from Michael Schwendt bugs.mich...@gmx.net ---
Have you tried adding the suggested options?
--- libchardet.spec.orig2014-02-09 05:47:28.0 +0100
+++ libchardet.spec2014-02-10 10:30:49.024562542 +0100
@@ -34,14 +34,14 @@
%install
make DESTDIR=%{buildroot} install
+%find_lang %{name} --with-man --all-name
%post -p /sbin/ldconfig
%postun -p /sbin/ldconfig
-%files
+%files -f %{name}.lang
%{_libdir}/%{name}.so.*
-%lang(ko) %{_mandir}/ko/man3/*.3*
%exclude %{_libdir}/%{name}.la
%files devel
--
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 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API
https://bugzilla.redhat.com/show_bug.cgi?id=1047647 --- Comment #10 from Michael Schwendt bugs.mich...@gmx.net --- Oh, and the manual pages belong into the -devel package. -- 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 1060465] Review Request: perl-Excel-Writer-XLSX - Create a new file in the Excel 2007+ XLSX format
https://bugzilla.redhat.com/show_bug.cgi?id=1060465 --- Comment #5 from David Dick dd...@cpan.org --- New koji build provided at https://koji.fedoraproject.org/koji/taskinfo?taskID=6511892 -- 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 1062942] Review Request: csv - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 José Matos jama...@fc.up.pt changed: What|Removed |Added CC||jama...@fc.up.pt --- Comment #6 from José Matos jama...@fc.up.pt --- (In reply to Fabio Alessandro Locati from comment #5) 6. I thought that the always-bumping rule was applicable only for versions that have been released. This believe has been confirmed by bug 1018038 which seems to have managed the spec file without bumping the version. It should be done also for packages in the review process. See another example of this discussion in https://bugzilla.redhat.com/show_bug.cgi?id=1041924 The rationale is that it really helps the reviewer that notice what it has changed. It can become really confusing if you have different versions with the same release number. Been there, done that. :-) -- 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 1062942] Review Request: csv - The CSV command line Tool
https://bugzilla.redhat.com/show_bug.cgi?id=1062942 --- Comment #7 from Fabio Alessandro Locati fabioloc...@gmail.com --- (In reply to José Matos from comment #6) It should be done also for packages in the review process. See another example of this discussion in https://bugzilla.redhat.com/show_bug.cgi?id=1041924 The rationale is that it really helps the reviewer that notice what it has changed. It can become really confusing if you have different versions with the same release number. Been there, done that. :-) Thanks a lot for the explanation and the example :) -- 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 1063045] Review Request: gnome-pkg-tools - Tools for the Debian GNOME Packaging Team
https://bugzilla.redhat.com/show_bug.cgi?id=1063045 Sandro Mani manisan...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #7 from Sandro Mani manisan...@gmail.com --- Thanks! I'll tweak the description when importing. Let me know if I can review anything for you. New Package SCM Request === Package Name: gnome-pkg-tools Short Description: Tools for the Debian GNOME Packaging Team Owners: smani Branches: f20 InitialCC: -- 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 1043573] Review Request: openav-luppp - Live performance mixing tool
https://bugzilla.redhat.com/show_bug.cgi?id=1043573 Sandro Mani manisan...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||manisan...@gmail.com Assignee|nob...@fedoraproject.org|manisan...@gmail.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 1024589] Review Request: zlib-js - JavaScript library reimplementing compression
https://bugzilla.redhat.com/show_bug.cgi?id=1024589 Sandro Mani manisan...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||manisan...@gmail.com Assignee|nob...@fedoraproject.org|manisan...@gmail.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 1043573] Review Request: openav-luppp - Live performance mixing tool
https://bugzilla.redhat.com/show_bug.cgi?id=1043573 --- Comment #1 from Sandro Mani manisan...@gmail.com --- The Spec URL is broken. -- 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 1024589] Review Request: zlib-js - JavaScript library reimplementing compression
https://bugzilla.redhat.com/show_bug.cgi?id=1024589 Sandro Mani manisan...@gmail.com changed: What|Removed |Added Status|ASSIGNED|NEW Assignee|manisan...@gmail.com|nob...@fedoraproject.org Flags|fedora-review? | --- Comment #1 from Sandro Mani manisan...@gmail.com --- Uh sorry was a bit too quick, I'd rather leave node.js to someone else :S -- 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 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API
https://bugzilla.redhat.com/show_bug.cgi?id=1047647 --- Comment #11 from Ben Reedy theben...@gmail.com --- Thanks Michael! New SRPM URL: http://breed808.com/rpmfusion-submission/libchardet-1.0.2-5.fc20.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 248552] cfv - utility to test and create .sfv, .csv, .crc,.md5(sfv-like),md5sum, bsd md5, sha1sum, and .torrent files
https://bugzilla.redhat.com/show_bug.cgi?id=248552 --- Comment #4 from Jon Ciesla limburg...@gmail.com --- Any comments from the Fedora maintainer? -- You are receiving this mail because: 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 248552] cfv - utility to test and create .sfv, .csv, .crc,.md5(sfv-like),md5sum, bsd md5, sha1sum, and .torrent files
https://bugzilla.redhat.com/show_bug.cgi?id=248552 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | -- You are receiving this mail because: 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 513307] Review Request: Netdisco - A web-based network management tool
https://bugzilla.redhat.com/show_bug.cgi?id=513307 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 513307] Review Request: Netdisco - A web-based network management tool
https://bugzilla.redhat.com/show_bug.cgi?id=513307 --- Comment #13 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 525929] Review Request: youtube-dl - Small command-line program to download videos from YouTube
https://bugzilla.redhat.com/show_bug.cgi?id=525929 --- Comment #6 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 525929] Review Request: youtube-dl - Small command-line program to download videos from YouTube
https://bugzilla.redhat.com/show_bug.cgi?id=525929 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 620112] Review Request: udpxy - UDP-to-HTTP multicast traffic relay daemon
https://bugzilla.redhat.com/show_bug.cgi?id=620112 --- Comment #38 from Jon Ciesla limburg...@gmail.com --- New packages need a New Package request, not a Package Change request. -- You are receiving this mail because: 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 620112] Review Request: udpxy - UDP-to-HTTP multicast traffic relay daemon
https://bugzilla.redhat.com/show_bug.cgi?id=620112 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | -- You are receiving this mail because: 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 839260] Review Request: Agda-stdlib - Agda standard library
https://bugzilla.redhat.com/show_bug.cgi?id=839260 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 839260] Review Request: Agda-stdlib - Agda standard library
https://bugzilla.redhat.com/show_bug.cgi?id=839260 --- Comment #9 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 858078] Review Request: mingw-qt5-qtsvg - Qt5 for Windows - QtSvg component
https://bugzilla.redhat.com/show_bug.cgi?id=858078 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 858078] Review Request: mingw-qt5-qtsvg - Qt5 for Windows - QtSvg component
https://bugzilla.redhat.com/show_bug.cgi?id=858078 --- Comment #17 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 875299] Review Request: slowhttptest - An Application Layer DoS attack simulator
https://bugzilla.redhat.com/show_bug.cgi?id=875299 --- Comment #15 from Jon Ciesla limburg...@gmail.com --- Any comments from the Fedora maintainer? -- 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 875299] Review Request: slowhttptest - An Application Layer DoS attack simulator
https://bugzilla.redhat.com/show_bug.cgi?id=875299 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|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 989847] Review Request: mingw-plibc - MinGW package for plibc
https://bugzilla.redhat.com/show_bug.cgi?id=989847 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 1054565] Review Request: ghc-securemem - Auto-scrubbing bytestring memory chunks
https://bugzilla.redhat.com/show_bug.cgi?id=1054565 --- Comment #5 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 989850] Review Request: mingw-libmicrohttpd - MinGW package for libmicrohttpd
https://bugzilla.redhat.com/show_bug.cgi?id=989850 --- Comment #12 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 989847] Review Request: mingw-plibc - MinGW package for plibc
https://bugzilla.redhat.com/show_bug.cgi?id=989847 --- Comment #10 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 989850] Review Request: mingw-libmicrohttpd - MinGW package for libmicrohttpd
https://bugzilla.redhat.com/show_bug.cgi?id=989850 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 1054565] Review Request: ghc-securemem - Auto-scrubbing bytestring memory chunks
https://bugzilla.redhat.com/show_bug.cgi?id=1054565 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 1062314] Review Request: apache-commons-collections4 - Extension of the Java Collections Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1062314 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 1062314] Review Request: apache-commons-collections4 - Extension of the Java Collections Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1062314 --- Comment #6 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 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 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 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 --- Comment #6 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 1062843] Review Request: python-rosinstall_generator - Generates rosinstall files
https://bugzilla.redhat.com/show_bug.cgi?id=1062843 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 248552] cfv - utility to test and create .sfv, .csv, .crc,.md5(sfv-like),md5sum, bsd md5, sha1sum, and .torrent files
https://bugzilla.redhat.com/show_bug.cgi?id=248552 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Flags||fedora-cvs? --- Comment #5 from manuel wolfshant wo...@nobugconsulting.ro --- I have no objection. Actually I would have maintained the EPEL branches myself if anyone bothered to ask. -- You are receiving this mail because: 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 1062843] Review Request: python-rosinstall_generator - Generates rosinstall files
https://bugzilla.redhat.com/show_bug.cgi?id=1062843 --- 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 1063045] Review Request: gnome-pkg-tools - Tools for the Debian GNOME Packaging Team
https://bugzilla.redhat.com/show_bug.cgi?id=1063045 --- Comment #8 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 1063045] Review Request: gnome-pkg-tools - Tools for the Debian GNOME Packaging Team
https://bugzilla.redhat.com/show_bug.cgi?id=1063045 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 1032425] Review Request: perl-XML-Parser-Lite - Lightweight regexp-based XML parser
https://bugzilla.redhat.com/show_bug.cgi?id=1032425
--- Comment #6 from Petr Pisar ppi...@redhat.com ---
Spec file changes:
$ diff -u perl-XML-Parser-Lite.spec{.old,}
--- perl-XML-Parser-Lite.spec.old 2014-02-05 16:49:07.0 +0100
+++ perl-XML-Parser-Lite.spec 2014-02-06 10:37:34.0 +0100
@@ -1,11 +1,13 @@
Name: perl-XML-Parser-Lite
Version:0.719
-Release:1%{?dist}
+Release:2%{?dist}
Summary:Lightweight regexp-based XML parser
License:(GPL+ or Artistic) and REX
Group: Development/Libraries
URL:http://search.cpan.org/dist/XML-Parser-Lite/
Source0:
http://www.cpan.org/authors/id/P/PH/PHRED/XML-Parser-Lite-%{version}.tar.gz
+# rt#91434
+Patch0: perl-XML-Parser-Lite-0.719-tests.patch
BuildArch: noarch
# SOAP::Lite is not actually needed
BuildRequires: perl
@@ -18,7 +20,7 @@
BuildRequires: perl(Test::More)
BuildRequires: perl(warnings)
BuildRequires: perl(XMLRPC::Lite)
-BuildRequires: perl(XMLRPC::Test)
+BuildRequires: perl(XMLRPC::Test::Server)
Requires: perl(:MODULE_COMPAT_%(eval $(perl -V:version); echo
$version))
%description
@@ -28,6 +30,7 @@
%prep
%setup -q -n XML-Parser-Lite-%{version}
+%patch0 -p1
%build
perl Makefile.PL INSTALLDIRS=vendor
@@ -47,6 +50,9 @@
%{_mandir}/man3/*
%changelog
+* Thu Feb 06 2014 Petr Šabata con...@redhat.com - 0.719-2
+- Patch the test suite so it actually works
+
* Wed Feb 05 2014 Petr Šabata con...@redhat.com - 0.719-1
- 0.719 bump
Source archive is original (SHA-256:
7c8c8a42bd7e6b64cd92d445132f57a6db23952741796e4764b59cb9c9011b05). Ok.
The patch is Ok.
FIX: Tests do not pass in F21
(http://koji.fedoraproject.org/koji/taskinfo?taskID=6499398). See
https://rt.cpan.org/Public/Bug/Display.html?id=91434 for bug report.
All tests pass. Ok.
rpmlint perl-XML-Parser-Lite.spec
../SRPMS/perl-XML-Parser-Lite-0.719-2.fc21.src.rpm
../RPMS/noarch/perl-XML-Parser-Lite-0.719-2.fc21.noarch.rpm
perl-XML-Parser-Lite.src: W: invalid-license REX
perl-XML-Parser-Lite.noarch: W: invalid-license REX
2 packages and 1 specfiles checked; 0 errors, 2 warnings.
rpmlint is Ok.
$ rpm -q -lv -p ../RPMS/noarch/perl-XML-Parser-Lite-0.719-2.fc21.noarch.rpm
drwxr-xr-x2 rootroot0 Feb 10 14:16
/usr/share/doc/perl-XML-Parser-Lite
-rw-r--r--1 rootroot 295 Dec 2 18:43
/usr/share/doc/perl-XML-Parser-Lite/Changes
-rw-r--r--1 rootroot 1159 Nov 26 19:38
/usr/share/doc/perl-XML-Parser-Lite/README
-rw-r--r--1 rootroot 3351 Feb 10 14:16
/usr/share/man/man3/XML::Parser::Lite.3pm.gz
drwxr-xr-x2 rootroot0 Feb 10 14:16
/usr/share/perl5/vendor_perl/XML
drwxr-xr-x2 rootroot0 Feb 10 14:16
/usr/share/perl5/vendor_perl/XML/Parser
-rw-r--r--1 rootroot10762 Dec 2 18:44
/usr/share/perl5/vendor_perl/XML/Parser/Lite.pm
File layout and permissions are Ok.
$ rpm -q --requires -p
../RPMS/noarch/perl-XML-Parser-Lite-0.719-2.fc21.noarch.rpm | sort | uniq -c
1 perl(:MODULE_COMPAT_5.18.2)
1 perl(re)
1 perl(strict)
1 perl(warnings)
1 rpmlib(CompressedFileNames) = 3.0.4-1
1 rpmlib(FileDigests) = 4.6.0-1
1 rpmlib(PayloadFilesHavePrefix) = 4.0-1
1 rpmlib(PayloadIsXz) = 5.2-1
Binary requires are Ok.
rpm -q --provides -p
../RPMS/noarch/perl-XML-Parser-Lite-0.719-2.fc21.noarch.rpm | sort | uniq -c
1 perl(XML::Parser::Lite) = 0.719
1 perl-XML-Parser-Lite = 0.719-2.fc21
Binary provides are Ok.
$ resolvedeps rawhide
../RPMS/noarch/perl-XML-Parser-Lite-0.719-2.fc21.noarch.rpm
Binary dependencies resolvable. Ok.
FIX: Package does not build in F21
(http://koji.fedoraproject.org/koji/taskinfo?taskID=6512171):
+ make test
PERL_DL_NONLAZY=1 /usr/bin/perl -MExtUtils::Command::MM -MTest::Harness
-e undef *Test::Harness::Switches; test_harness(0, 'blib/lib', 'blib/arch')
t/*.t
t/07-xmlrpc_payload.t .. ok
# Looks like you planned 6 tests but ran 8.
t/26-xmlrpc.t ..
Dubious, test returned 255 (wstat 65280, 0xff00)
All 6 subtests passed
(less 8 skipped subtests: -2 okay)
t/27-xmlparserlite.t ... ok
t/37-mod_xmlrpc.t .. ok
t/Lite.t ... ok
Test Summary Report
---
t/26-xmlrpc.t(Wstat: 65280 Tests: 8 Failed: 2)
Failed tests: 7-8
Non-zero exit status: 255
Parse errors: Bad plan. You planned 6 tests but ran 8.
Files=5, Tests=88, 2 wallclock secs ( 0.13 usr 0.02 sys + 1.51 cusr 0.11
csys = 1.77 CPU)
Result: FAIL
Otherwise the package is good.
Please correct all `FIX' items and provide new spec file.
Resolution: Package NOT approved.
--
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
[Bug 1063119] EPEL7 Branch Request: advancecomp
https://bugzilla.redhat.com/show_bug.cgi?id=1063119 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 1063119] EPEL7 Branch Request: advancecomp
https://bugzilla.redhat.com/show_bug.cgi?id=1063119 --- Comment #2 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 1043573] Review Request: openav-luppp - Live performance mixing tool
https://bugzilla.redhat.com/show_bug.cgi?id=1043573 --- Comment #2 from Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl --- Ooops, fixed. Spec URL: http://in.waw.pl/~zbyszek/fedora/openav-luppp.spec SRPM URL: http://in.waw.pl/~zbyszek/fedora/openav-luppp-1.0-2.fc20.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 1043573] Review Request: openav-luppp - Live performance mixing tool
https://bugzilla.redhat.com/show_bug.cgi?id=1043573 --- Comment #3 from Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl --- In the meanwhile I've also sent a pull request for .desktop and appdata: https://github.com/harryhaaren/openAV-Luppp/pull/71 -- 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 231772] Review Request: Siege - Siege is an http regression testing and benchmarking utility
https://bugzilla.redhat.com/show_bug.cgi?id=231772 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 231772] Review Request: Siege - Siege is an http regression testing and benchmarking utility
https://bugzilla.redhat.com/show_bug.cgi?id=231772 --- Comment #8 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). EL-6 and epel7 already exist. -- You are receiving this mail because: 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 248552] cfv - utility to test and create .sfv, .csv, .crc,.md5(sfv-like),md5sum, bsd md5, sha1sum, and .torrent files
https://bugzilla.redhat.com/show_bug.cgi?id=248552 --- Comment #6 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). Added wolfy as owner. -- You are receiving this mail because: 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 248552] cfv - utility to test and create .sfv, .csv, .crc,.md5(sfv-like),md5sum, bsd md5, sha1sum, and .torrent files
https://bugzilla.redhat.com/show_bug.cgi?id=248552 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 1062911] Review Request: srcpd - Simple Railroad Command Protocol (SRCP) server
https://bugzilla.redhat.com/show_bug.cgi?id=1062911 --- Comment #5 from Denis Fateyev de...@fateyev.com --- (In reply to Volker Fröhlich from comment #3) Ad EL5: You got me the wrong way around: I meant you don't have to make them conditional: clean section instead of if el5; then clean section; endif Achso.. but I don't see what's wrong with the current version with conditionals, anyway. Seems wrapping into conditionals looks more logical for recent Fedora. (In reply to Christopher Meng from comment #4) I just asked this question in packaging about udev rules: https://lists.fedoraproject.org/pipermail/packaging/2014-February/010013.html Yes, thanks, better to clarify this moment. I haven't found any good and recent guideline on that. -- 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 1058812] Review Request: mod_lookup_identity - Apache module to retrieve additional information about the authenticated user.
https://bugzilla.redhat.com/show_bug.cgi?id=1058812 Jan Pazdziora jpazdzi...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #11 from Jan Pazdziora jpazdzi...@redhat.com --- (In reply to Jakub Hrozek from comment #10) This package is APPROVED. Thanks. [*] You should fix two bugs on your website - there is no link to 0.8.3 on http://www.adelton.com/apache/mod_lookup_identity/ - following the link to mod_lookup_identity 0.8.2 takes me to http://www.adelton.com/apache/mod_authnz_pam/; Both fixed. New Package SCM Request === Package Name: mod_lookup_identity Short Description: Apache module to retrieve additional information about the authenticated user Owners: adelton Branches: f20 InitialCC: -- 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 1058812] Review Request: mod_lookup_identity - Apache module to retrieve additional information about the authenticated user.
https://bugzilla.redhat.com/show_bug.cgi?id=1058812 --- Comment #12 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 1058812] Review Request: mod_lookup_identity - Apache module to retrieve additional information about the authenticated user.
https://bugzilla.redhat.com/show_bug.cgi?id=1058812 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 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 Yanko Kaneti yan...@declera.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2014-02-10 09:26:41 --- Comment #7 from Yanko Kaneti yan...@declera.com --- Imported and built. 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 1058812] Review Request: mod_lookup_identity - Apache module to retrieve additional information about the authenticated user.
https://bugzilla.redhat.com/show_bug.cgi?id=1058812 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|NEW |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 1058812] Review Request: mod_lookup_identity - Apache module to retrieve additional information about the authenticated user.
https://bugzilla.redhat.com/show_bug.cgi?id=1058812 --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- mod_lookup_identity-0.8.3-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/mod_lookup_identity-0.8.3-1.fc20 -- 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 192502] Review Request: lilypond-doc - HTML documentation for lilypond
https://bugzilla.redhat.com/show_bug.cgi?id=192502 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added CC||limburg...@gmail.com Flags||fedora-cvs? --- Comment #2 from Jon Ciesla limburg...@gmail.com --- Package Change Request == Package Name: lilypond-doc New Branches: el6 epel7 Owners: cicku limb -- You are receiving this mail because: 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 189656] Review Request: lilypond - A typesetting system for music notation
https://bugzilla.redhat.com/show_bug.cgi?id=189656 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added CC||limburg...@gmail.com Flags||fedora-cvs? --- Comment #23 from Jon Ciesla limburg...@gmail.com --- Package Change Request == Package Name: lilypond New Branches: el6 epel7 Owners: cicku limb -- You are receiving this mail because: 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 189656] Review Request: lilypond - A typesetting system for music notation
https://bugzilla.redhat.com/show_bug.cgi?id=189656 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 189656] Review Request: lilypond - A typesetting system for music notation
https://bugzilla.redhat.com/show_bug.cgi?id=189656 --- Comment #24 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 192502] Review Request: lilypond-doc - HTML documentation for lilypond
https://bugzilla.redhat.com/show_bug.cgi?id=192502 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: 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 192502] Review Request: lilypond-doc - HTML documentation for lilypond
https://bugzilla.redhat.com/show_bug.cgi?id=192502 --- Comment #3 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: 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 1060465] Review Request: perl-Excel-Writer-XLSX - Create a new file in the Excel 2007+ XLSX format
https://bugzilla.redhat.com/show_bug.cgi?id=1060465
--- Comment #6 from Petr Šabata psab...@redhat.com ---
Ok, this looks much better now, just a few more things.
1. You forgot to remove the %clean section.
2. You didn't filter out the underspecified dependencies. You can achieve that
with:
%global __requires_exclude
%{?__requires_exclude:__requires_exclude|}^perl\\(Archive::Zip\\)$
%global __requires_exclude %__requires_exclude|^perl\\(File::Temp\\)$
Although those two were just TODO points, given this is your first package, it
should be perfect :)
3. What's your FAS username?
--
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 1062314] Review Request: apache-commons-collections4 - Extension of the Java Collections Framework
https://bugzilla.redhat.com/show_bug.cgi?id=1062314 Mikolaj Izdebski mizde...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks|652183 (FE-JAVASIG) | Resolution|--- |NEXTRELEASE Last Closed||2014-02-10 10:25:34 --- Comment #7 from Mikolaj Izdebski mizde...@redhat.com --- Rawhide build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6512758 Closing. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=652183 [Bug 652183] Java SIG tracker bug -- 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 1060651] Review Request: rubygem-rspec-support - Common functionality to Rspec series
https://bugzilla.redhat.com/show_bug.cgi?id=1060651 Ken Dreyer ktdre...@ktdreyer.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Ken Dreyer ktdre...@ktdreyer.com --- Looks good to me. Package APPROVED. -- 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 1060443] Review Request: xfdashboard - GNOME shell like dashboard for Xfce
https://bugzilla.redhat.com/show_bug.cgi?id=1060443
--- Comment #9 from Kevin Fenzi ke...@scrye.com ---
(In reply to Mukundan Ragavan from comment #7)
./configure with Fedora default flags throws up error.
http://paste.fedoraproject.org/75748/92002850/
I am looking into this and based on the interactions on #fedora-devel, I
have also contacted the author for his thoughts.
Thats not the way I meant to pass it. Sorry for being unclear...
...
%build
export CFLAGS=%{optflags}
./autogen.sh --prefix=%{_prefix}
make %{?_smp_mflags} V=1
...
works fine here and gets the proper flags I think?
--
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 1063044] Review Request: dh-autoreconf - debhelper add-on to call autoreconf and clean up after the build
https://bugzilla.redhat.com/show_bug.cgi?id=1063044
Ralf Corsepius rc040...@freenet.de changed:
What|Removed |Added
CC||rc040...@freenet.de
--- Comment #2 from Ralf Corsepius rc040...@freenet.de ---
Directory ownerships do not seem OK:
The package must own all directories referenced on this path:
%{perl_vendorlib}/Debian/Debhelper/Sequence/autoreconf.pm
Similar considerations probably also apply to %{_datadir}/cdbs, I don't know
off-head, how directory ownerships are handled there:
%{_datadir}/cdbs/1/rules/
--
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 1063044] Review Request: dh-autoreconf - debhelper add-on to call autoreconf and clean up after the build
https://bugzilla.redhat.com/show_bug.cgi?id=1063044
--- Comment #3 from Sandro Mani manisan...@gmail.com ---
debhelper owns %{perl_vendorlib}/Debian/Debhelper/Sequence/:
$ rpm -ql debhelper
[...]
/usr/share/perl5/vendor_perl/Debian/Debhelper/Sequence
[...]
$ rpm -qf /usr/share/perl5/vendor_perl/Debian/Debhelper/Sequence
debhelper-9.20131227-1.fc21.noarch
cdbs owns {_datadir}/cdbs/1/rules/:
$ rpm -ql cdbs
[...]
/usr/share/cdbs/1
[...]
$ rpm -qf /usr/share/cdbs/1
cdbs-0.4.122-1.fc21.x86_64
I can add some comments in the spec if you feel it would be adequate.
--
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 1060443] Review Request: xfdashboard - GNOME shell like dashboard for Xfce
https://bugzilla.redhat.com/show_bug.cgi?id=1060443 --- Comment #10 from Mukundan Ragavan nonamed...@gmail.com --- Yup! That fixes it. :) I have also addressed Christopher Meng's other comments. I am looking at Christopher Meng's comment 2 now - IceConnectionNumber. Once I fix that, I will upload the new spec and source 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 1058623] Review Request: perl-WebService-Linode - Perl Interface to the Linode.com API
https://bugzilla.redhat.com/show_bug.cgi?id=1058623 Petr Šabata psab...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Petr Šabata psab...@redhat.com --- Alright, 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 1036901] Review Request: rubygem-org-ruby - Ruby routines for parsing org-mode files
https://bugzilla.redhat.com/show_bug.cgi?id=1036901
--- Comment #5 from Ken Dreyer ktdre...@ktdreyer.com ---
Thanks for reviewing. Here's the update to 0.9.0.
* Mon Feb 10 2014 Ken Dreyer ktdre...@ktdreyer.com - 0.9.0-1
- Update to org-mode 0.9.0
- Remove %%{gem_instdir}/bin/org-ruby from -doc sub-package
Exact changes in Git:
http://fedorapeople.org/cgit/ktdreyer/public_git/rubygem-org-ruby.git/commit/?id=a0c19ab0807610358a9a08a7a1770e5b8801b8f5
Spec URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-org-ruby.spec
SRPM URL:
http://ktdreyer.fedorapeople.org/reviews/rubygem-org-ruby-0.9.0-1.fc21.src.rpm
F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6513027
--
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 1056335] Review Request: scalacheck - property-based testing for Scala
https://bugzilla.redhat.com/show_bug.cgi?id=1056335 --- Comment #5 from Fedora Update System upda...@fedoraproject.org --- scalacheck-1.11.3-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/scalacheck-1.11.3-3.fc20 -- 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 1056335] Review Request: scalacheck - property-based testing for Scala
https://bugzilla.redhat.com/show_bug.cgi?id=1056335 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 1063428] New: Review Request: rubygem-bogus - Create fakes to make your isolated unit tests reliable
https://bugzilla.redhat.com/show_bug.cgi?id=1063428 Bug ID: 1063428 Summary: Review Request: rubygem-bogus - Create fakes to make your isolated unit tests reliable Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: ktdre...@ktdreyer.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-bogus.spec SRPM URL: http://ktdreyer.fedorapeople.org/reviews/rubygem-bogus-0.1.4-2.fc21.src.rpm Description: Decreases the need to write integration tests by ensuring that the things you stub or mock actually exist. Fedora Account System Username: ktdreyer F21 scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6513093 -- 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 1043573] Review Request: openav-luppp - Live performance mixing tool
https://bugzilla.redhat.com/show_bug.cgi?id=1043573
--- Comment #4 from Sandro Mani manisan...@gmail.com ---
Full review below. Issues:
- Missing in post and postun:
/usr/bin/update-desktop-database /dev/null || :
- These BR are not needed: gcc-c++
- Must require hicolor-icon-theme
- Sould use -p with install to preserve timestamps
- Install desktop file via
desktop-file-install --dir=%{buildroot}%{_datadir}/applications/ %{SOURCE2}
and remove desktop-file-validate call
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[ ]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
ISC, MIT/X11 (BSD like), *No copyright* GPL (v3 or later), Unknown
or generated. 12 files have unknown license. Detailed output of
licensecheck in /home/sandro/.Data/Desktop/1043573-openav-
luppp/licensecheck.txt
[!]: Package must own all directories that it creates.
Note: Directories without known owners:
/usr/share/icons/hicolor/scalable/apps, /usr/share/icons/hicolor,
/usr/share/icons/hicolor/scalable
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 40960 bytes in 3 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s)
for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: Package does not own files or directories owned by other packages.
[x]: Package uses either %{buildroot} or $RPM_BUILD_ROOT
[x]: Package does not run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) at the
beginning of %install.
[x]: Macros in Summary, %description expandable at SRPM build time.
[x]: Package contains desktop file if it is a GUI application.
[x]: Package installs a %{name}.desktop using desktop-file-install or desktop-
file-validate if there is such a file.
[x]: Package does not contain duplicates in %files.
[x]: Permissions on files are set properly.
[x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package do not use a name that already exist
[x]: Package is not relocatable.
[x]: Sources used to build the package match the upstream source, as provided
in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[?]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[?]: Package should compile and build into binary rpms on all supported
[Bug 1063045] Review Request: gnome-pkg-tools - Tools for the Debian GNOME Packaging Team
https://bugzilla.redhat.com/show_bug.cgi?id=1063045 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- gnome-pkg-tools-0.19.3-3.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/gnome-pkg-tools-0.19.3-3.fc20 -- 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 1063045] Review Request: gnome-pkg-tools - Tools for the Debian GNOME Packaging Team
https://bugzilla.redhat.com/show_bug.cgi?id=1063045 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 1060443] Review Request: xfdashboard - GNOME shell like dashboard for Xfce
https://bugzilla.redhat.com/show_bug.cgi?id=1060443
--- Comment #11 from Mukundan Ragavan nonamed...@gmail.com ---
Changes made -
* Fixed the changelog
* Added BuildRequires: libICE-devel
* Added more detailed description
* Changed to %setup -q
* Added export CFLAGS=%{optflags} in %build
Now, it compiles with the correct flags and generates binary rpms.
mock build log: http://paste.fedoraproject.org/75896/54609139
URLs:
updated SPEC URL:
http://nonamedotc.fedorapeople.org/pkgreview/xfdashboard/xfdashboard.spec
updated SRPM URL:
http://nonamedotc.fedorapeople.org/pkgreview/xfdashboard/xfdashboard-0.1.2-3.fc20.src.rpm
Rpmlint:
$ rpmlint xfdashboard.spec ../SRPMS/xfdashboard-0.1.2-3.fc20.src.rpm
/var/lib/mock/fedora-rawhide-x86_64/result/xfdashboard-*rpm
xfdashboard.x86_64: W: no-manual-page-for-binary xfdashboard
4 packages and 1 specfiles checked; 0 errors, 1 warnings.
--
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 1057766] Review Request: clustal-omega - command line tool for multiple sequence alignment
https://bugzilla.redhat.com/show_bug.cgi?id=1057766 --- Comment #13 from Mukundan Ragavan nonamed...@gmail.com --- I checked with the developers about this one - They informed me that they have used Squid source code with minor modifications directly in the clustal omega source. So, they mention that it would not be possible to unbundle squid and clustal omega. Is there anything I can do here? -- 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 1055395] Review Request: ocaml-xmlm - A streaming XML codec
https://bugzilla.redhat.com/show_bug.cgi?id=1055395 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Jerry James loganje...@gmail.com --- OK, that fixes all of the issues. This package is APPROVED. -- 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 906843] Review Request: rubygem-qpid_proton - Ruby language bindings for the Qpid Proton messaging framework
https://bugzilla.redhat.com/show_bug.cgi?id=906843 Darryl L. Pierce dpie...@redhat.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #15 from Darryl L. Pierce dpie...@redhat.com --- Package Change Request == Package Name: rubygem-qpid_proton New Branches: epel7 Owners: mcpierce -- 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 1055396] Review Request: ocaml-yojson - An optimized parsing and printing library for the JSON format
https://bugzilla.redhat.com/show_bug.cgi?id=1055396 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Jerry James loganje...@gmail.com --- All review issues have been handled. This package is APPROVED. -- 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 1061901] Review Request: saclib - Computer algebra library
https://bugzilla.redhat.com/show_bug.cgi?id=1061901 --- Comment #3 from Jerry James loganje...@gmail.com --- Thanks for the review! Upstream chose today to release version 2.2.6, which is version 2.2.5 plus all of the patches I submitted upstream. If it's okay with you, what I will actually import into git is version 2.2.6, which just means bumping the version number and throwing all of the patches away. -- 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 906843] Review Request: rubygem-qpid_proton - Ruby language bindings for the Qpid Proton messaging framework
https://bugzilla.redhat.com/show_bug.cgi?id=906843 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 906843] Review Request: rubygem-qpid_proton - Ruby language bindings for the Qpid Proton messaging framework
https://bugzilla.redhat.com/show_bug.cgi?id=906843 --- Comment #16 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 1000276] Review Request: python-wstool - A tool for managing a workspace of multiple heterogenous SCM repositories
https://bugzilla.redhat.com/show_bug.cgi?id=1000276 Rich Mattes richmat...@gmail.com changed: What|Removed |Added CC||richmat...@gmail.com Flags|fedora-cvs+ |fedora-cvs? --- Comment #19 from Rich Mattes richmat...@gmail.com --- Package Change Request == Package Name: python-wstool New Branches: el6 epel7 Owners: rmattes ankursinha -- You are receiving this mail because: 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 926034] Review Request: python-catkin_pkg - Library for retrieving information about catkin packages
https://bugzilla.redhat.com/show_bug.cgi?id=926034 Rich Mattes richmat...@gmail.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #14 from Rich Mattes richmat...@gmail.com --- Package Change Request == Package Name: python-catkin_pkg New Branches: el6 epel7 Owners: rmattes ankursinha -- 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 1061901] Review Request: saclib - Computer algebra library
https://bugzilla.redhat.com/show_bug.cgi?id=1061901 Jerry James loganje...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Jerry James loganje...@gmail.com --- New Package SCM Request === Package Name: saclib Short Description: Computer algebra library Owners: jjames Branches: f20 InitialCC: -- 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 1048667] Review Request: python-docker-py - An API client for docker written in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1048667 --- Comment #31 from Fedora Update System upda...@fedoraproject.org --- python-docker-py-0.2.3-8.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-docker-py-0.2.3-8.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 926034] Review Request: python-catkin_pkg - Library for retrieving information about catkin packages
https://bugzilla.redhat.com/show_bug.cgi?id=926034 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
