[Bug 1063042] Review Request: rubygem-em-websocket - EventMachine based WebSocket server
https://bugzilla.redhat.com/show_bug.cgi?id=1063042 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com --- Comment #6 from Miroslav Suchý msu...@redhat.com --- Any progress 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 1063039] Review Request: rubygem-em-http-request - EventMachine based, async HTTP Request client
https://bugzilla.redhat.com/show_bug.cgi?id=1063039 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com, ||niteshnarayan@fedoraproject ||.org Flags||needinfo?(niteshnarayan@fed ||oraproject.org) --- Comment #2 from Miroslav Suchý msu...@redhat.com --- Nitash, any progress 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 1245270] New: Review Request: secilc - The SELinux CIL Compiler
https://bugzilla.redhat.com/show_bug.cgi?id=1245270 Bug ID: 1245270 Summary: Review Request: secilc - The SELinux CIL Compiler Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: plaut...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://plautrba.fedorapeople.org/secilc/secilc.spec SRPM URL: http://plautrba.fedorapeople.org/secilc/secilc-2.4-1.fc24.src.rpm Description: The SELinux CIL Compiler is a compiler that converts the CIL language as described on the CIL design wiki into a kernel binary policy file. Please see the CIL Design Wiki at: http://github.com/SELinuxProject/cil/wiki/ for more information about the goals and features on the CIL language. Fedora Account System Username: plautrba -- 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 1231459] Review Request: libosmium - Fast and flexible C++ library for working with OpenStreetMap data
https://bugzilla.redhat.com/show_bug.cgi?id=1231459 --- Comment #17 from Fedora Update System upda...@fedoraproject.org --- libosmium-2.2.0-6.fc21,osmpbf-1.3.3-5.20150712git17fd0cc.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/libosmium-2.2.0-6.fc21,osmpbf-1.3.3-5.20150712git17fd0cc.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1245270] Review Request: secilc - The SELinux CIL Compiler
https://bugzilla.redhat.com/show_bug.cgi?id=1245270 Christopher Meng i...@cicku.me changed: What|Removed |Added CC||i...@cicku.me --- Comment #1 from Christopher Meng i...@cicku.me --- -doc package should be noarch. -- 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 1231458] Review Request: osmpbf - C library to read and write OpenStreetMap PBF files
https://bugzilla.redhat.com/show_bug.cgi?id=1231458 --- Comment #15 from Fedora Update System upda...@fedoraproject.org --- libosmium-2.2.0-6.fc21,osmpbf-1.3.3-5.20150712git17fd0cc.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/libosmium-2.2.0-6.fc21,osmpbf-1.3.3-5.20150712git17fd0cc.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1231458] Review Request: osmpbf - C library to read and write OpenStreetMap PBF files
https://bugzilla.redhat.com/show_bug.cgi?id=1231458 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |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 1231459] Review Request: libosmium - Fast and flexible C++ library for working with OpenStreetMap data
https://bugzilla.redhat.com/show_bug.cgi?id=1231459 --- Comment #16 from Fedora Update System upda...@fedoraproject.org --- osmpbf-1.3.3-5.20150712git17fd0cc.fc22, libosmium-2.2.0-6.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/FEDORA-2015-11804/libosmium-2.2.0-6.fc22,osmpbf-1.3.3-5.20150712git17fd0cc.fc22 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1231459] Review Request: libosmium - Fast and flexible C++ library for working with OpenStreetMap data
https://bugzilla.redhat.com/show_bug.cgi?id=1231459 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 996813] Review Request: SQLCipher - Encrypted SQLite databases
https://bugzilla.redhat.com/show_bug.cgi?id=996813 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED CC||msu...@redhat.com Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed||2015-07-21 10:48:07 --- Comment #8 from Miroslav Suchý msu...@redhat.com --- Closing due long inactivity. Feel free to reopen if you want to continue. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking 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 1004306] Review Request: opvdm - A tool for simulating organic solar cells
https://bugzilla.redhat.com/show_bug.cgi?id=1004306 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED CC||msu...@redhat.com Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed||2015-07-21 10:49:16 --- Comment #11 from Miroslav Suchý msu...@redhat.com --- No SRPM provided for long time. Closing due long inactivity. Feel free to reopen if you want to continue. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking this bug. -- 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 1010557] Review Request: numix-themes - Numix themes for Gnome, Xfce and Openbox
https://bugzilla.redhat.com/show_bug.cgi?id=1010557 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com Flags||needinfo?(satyajit.happy@gm ||ail.com) --- Comment #24 from Miroslav Suchý msu...@redhat.com --- Ping? Any progress 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 1047647] Review Request: libchardet - Mozilla's Universal Charset Detector C/C++ API
https://bugzilla.redhat.com/show_bug.cgi?id=1047647 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED CC||msu...@redhat.com Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed||2015-07-21 10:57:18 --- Comment #15 from Miroslav Suchý msu...@redhat.com --- Closing due long inactivity. Feel free to reopen if you want to continue. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking 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 1245270] Review Request: secilc - The SELinux CIL Compiler
https://bugzilla.redhat.com/show_bug.cgi?id=1245270 --- Comment #2 from Petr Lautrbach plaut...@redhat.com --- Spec URL: http://plautrba.fedorapeople.org/secilc/secilc.spec SRPM URL: http://plautrba.fedorapeople.org/secilc/secilc-2.4-2.fc24.src.rpm * Tue Jul 21 2015 Petr Lautrbach plaut...@redhat.com 2.4-2 - make secilc-doc package noarch -- 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 1048616] Review Request: ocaml-core - Janet Street's OCaml Standard Library
https://bugzilla.redhat.com/show_bug.cgi?id=1048616 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED CC||msu...@redhat.com Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed||2015-07-21 10:57:57 --- Comment #3 from Miroslav Suchý msu...@redhat.com --- Closing due long inactivity. Feel free to reopen if you want to continue. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking 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 1225231] Review Request: light-locker-settings - Just a simple settings dialog for light-locker
https://bugzilla.redhat.com/show_bug.cgi?id=1225231 --- Comment #5 from Raphael Groner projects...@smart.ms --- Spec URL: https://raphgro.fedorapeople.org/review/locker/light-locker-settings.spec SRPM URL: https://raphgro.fedorapeople.org/review/locker/light-locker-settings-1.5.0-2.fc22.src.rpm Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=10426299 * Tue Jul 21 2015 Raphael Groner - 1.5.0-2 - include python3 patch - include usability patch (upstream bug 1463476) - fix execution bits - fix installation of documentation (In reply to Yajo from comment #3) This is an unofficial review. At a first glance, I see the following errors: License:GPLv3+ Reading the source and the project URL, it's only GPLv3. From where do you get that it is GPLv3 only? COPYING file says it's GPLv3+ and that any further version of the GPL license is also possible, I don't see anything to forbid that. %setup -q You might prefer %autosetup unless you target EPEL7. See https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/Guidelines#. 25autosetup My understanding is that both are possible as alternatives. I am not convinced about applying all patches automatically cause they need proper formatting then, at least for the pathes and you won't be able to use individual patch -p options. BuildRequires: python3 Should be python3-devel. See https://fedoraproject.org/wiki/Packaging:Python#BuildRequires Fixed. Found also a python3 patch in upstream tracker. # configure macro does not work IMHO A little explanation would prevent future developers to lose time wondering why. Done. * Tue May 26 2015 Raphael Groner projects...@smart.ms - Please fill this, also with the version stuff. Fixed. You might want to see the rpmlint errors too: Checking: light-locker-settings-1.5.0-1.fc21.x86_64.rpm light-locker-settings-1.5.0-1.fc21.src.rpm light-locker-settings.x86_64: W: spelling-error %description -l en_US screensaver - screen saver, screen-saver, screens aver Fixed. light-locker-settings.x86_64: W: no-version-in-last-changelog Fixed, see above. light-locker-settings.x86_64: E: no-binary ?? light-locker-settings.x86_64: E: script-without-shebang /usr/share/light-locker-settings/light-locker-settings/light-locker-settings. glade Propably due to wrongly set execution bits. Ignore for now. https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions light-locker-settings.x86_64: W: dangling-symlink /usr/share/light-locker-settings/locale /usr/share/locale Wrong path according to FHS. Not sure if patchable in upstream Makefile: cp -rf locale $(DESTDIR)/$(PREFIX)/share ln -sf $(PREFIX)/share/locale $(DESTDIR)/$(PREFIX)/share/$(APPNAME)/locale light-locker-settings.x86_64: W: spurious-executable-perm /usr/share/doc/light-locker-settings/COPYING light-locker-settings.x86_64: W: spurious-executable-perm /usr/share/doc/light-locker-settings/INSTALL Propably due to wrongly set execution bits. Ignore for now. https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions light-locker-settings.x86_64: W: no-manual-page-for-binary light-locker-settings Not sure if we've to provide a manpage. light-locker-settings.x86_64: W: install-file-in-docs /usr/share/doc/light-locker-settings/INSTALL Fixed. light-locker-settings.src: W: spelling-error %description -l en_US screensaver - screen saver, screen-saver, screens aver Fixed. light-locker-settings.src: W: no-version-in-last-changelog Fixed, see above. light-locker-settings.src:36: W: configure-without-libdir-spec 2 packages and 0 specfiles checked; 2 errors, 10 warnings. Also, you might want to create appdata files, or at least notify upstream a bug about it. See https://fedoraproject.org/wiki/Packaging:Guidelines?rd=Packaging/ Guidelines#AppData_files This is no MUST but clearly SHOULD. I'll follow upstream features. https://fedoraproject.org/wiki/Packaging:AppData -- 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 1015868] Review Request: python-qutepart - Source code text editor component based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=1015868 Raphael Groner projects...@smart.ms changed: What|Removed |Added Flags||needinfo?(yajo@gmail.co ||m) --- Comment #28 from Raphael Groner projects...@smart.ms --- Assigned reviewer seems to got lost. Should I take over? -- 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 1245270] Review Request: secilc - The SELinux CIL Compiler
https://bugzilla.redhat.com/show_bug.cgi?id=1245270 Miroslav Grepl mgr...@redhat.com changed: What|Removed |Added CC||mgr...@redhat.com --- Comment #3 from Miroslav Grepl mgr...@redhat.com --- The packages have been built/installed/tested successfully by provided sources. -- 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 1245251] New: Review Request: netspy2ban-1.0-1.fc22.noarch.rpm - GUI Networking Tool and Fail2ban Controller for Fedora 22 Version
https://bugzilla.redhat.com/show_bug.cgi?id=1245251 Bug ID: 1245251 Summary: Review Request: netspy2ban-1.0-1.fc22.noarch.rpm - GUI Networking Tool and Fail2ban Controller for Fedora 22 Version Product: Fedora Version: 22 Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: ftsia...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://github.com/ftsiadimos/netspy2ban/blob/master/rpms/netspy2ban.spec SRPM URL: https://github.com/ftsiadimos/netspy2ban/blob/master/rpms/netspy2ban-1.0-1.fc22.src.rpm Description: NetSpy2Ban is a graphic user interface program for Fedora 22 OS. The program serves three functions. The first function is to view connected network cards and their speed. The second is to allow real time monitoring of your network connections. Lastly, NetSpy2Ban includes a graphic user interface to provide user-friendly functionality for the Fail2Ban service. Fedora Account System 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 635256] Review Request: qtop - tool for monitoring PBS systems
https://bugzilla.redhat.com/show_bug.cgi?id=635256 Fotis Georgatos fo...@cern.ch changed: What|Removed |Added Status|CLOSED |ASSIGNED Resolution|DEFERRED|--- Keywords||Reopened --- Comment #37 from Fotis Georgatos fo...@cern.ch --- Yes, please, let's keep this open! F. -- 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 1221781] Review Request: zipios - C++ library for reading and writing Zip files
https://bugzilla.redhat.com/show_bug.cgi?id=1221781 Raphael Groner projects...@smart.ms changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|projects...@smart.ms Flags||fedora-review? --- Comment #6 from Raphael Groner projects...@smart.ms --- I am still thinking that the name is misleading, see my comment #2. -- 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 1245255] New: Review Request: netspy2ban-1.0-1.fc22.noarch.rpm - GUI Networking Tool and Fail2ban Controller for Fedora 22 Version
https://bugzilla.redhat.com/show_bug.cgi?id=1245255 Bug ID: 1245255 Summary: Review Request: netspy2ban-1.0-1.fc22.noarch.rpm - GUI Networking Tool and Fail2ban Controller for Fedora 22 Version Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: ftsiadi...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://github.com/ftsiadimos/netspy2ban/blob/master/rpms/netspy2ban.spec SRPM URL: https://github.com/ftsiadimos/netspy2ban/blob/master/rpms/netspy2ban-1.0-1.fc22.src.rpm Description: NetSpy2Ban is a graphic user interface program for Fedora 22 OS. The program serves three functions. The first function is to view connected network cards and their speed. The second is to allow real time monitoring of your network connections. Lastly, NetSpy2Ban includes a graphic user interface to provide user-friendly functionality for the Fail2Ban service. Fedora Account System 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 1060386] Review Request: pandorafms-agent - Pandora FMS Linux agent.
https://bugzilla.redhat.com/show_bug.cgi?id=1060386 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com, ||sler...@gmail.com Flags||needinfo?(sler...@gmail.com ||) --- Comment #14 from Miroslav Suchý msu...@redhat.com --- Sancho, any progress here? Are you still interrested in this 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 1242726] Review Request: perl-MooX-Log-Any - Role to add Log::Any
https://bugzilla.redhat.com/show_bug.cgi?id=1242726 --- Comment #5 from Tim Orling ticot...@gmail.com --- Spec URL: https://ttorling.fedorapeople.org/perl-MooX-Log-Any/perl-MooX-Log-Any.spec SRPM URL: https://ttorling.fedorapeople.org/perl-MooX-Log-Any/perl-MooX-Log-Any-0.004002-1.fc23.src.rpm Git URL: https://fedorapeople.org/cgit/ttorling/public_git/perl-MooX-Log-Any.git/ koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=10426934 -- 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 1245251] Review Request: netspy2ban-1.0-1.fc22.noarch.rpm - GUI Networking Tool and Fail2ban Controller for Fedora 22 Version
https://bugzilla.redhat.com/show_bug.cgi?id=1245251 Fotios Tsiadimos ftsia...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |NOTABUG Last Closed||2015-07-21 10:46:43 -- 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 1039323] Review Request: Weatherman
https://bugzilla.redhat.com/show_bug.cgi?id=1039323 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED CC||msu...@redhat.com Blocks|177841 (FE-NEEDSPONSOR) |201449 (FE-DEADREVIEW) Resolution|--- |DEFERRED Last Closed||2015-07-21 10:56:17 --- Comment #8 from Miroslav Suchý msu...@redhat.com --- Closing due long inactivity. Feel free to reopen if you want to continue. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor https://bugzilla.redhat.com/show_bug.cgi?id=201449 [Bug 201449] FE-DEADREVIEW -- Reviews stalled due to lack of submitter response should be blocking 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 1233963] Review Request: python-cachetools - Extensible memoizing collections and decorators
https://bugzilla.redhat.com/show_bug.cgi?id=1233963 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- python-cachetools-1.0.3-1.el7 has been pushed to the Fedora EPEL 7 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077030] Review Request: python-semantic-version - library implementing SemVer
https://bugzilla.redhat.com/show_bug.cgi?id=1077030 Christopher Meng i...@cicku.me changed: What|Removed |Added Status|ASSIGNED|CLOSED CC||i...@cicku.me Blocks|177841 (FE-NEEDSPONSOR) | Resolution|--- |DUPLICATE Flags|fedora-review? | Last Closed||2015-07-21 21:45:37 --- Comment #17 from Christopher Meng i...@cicku.me --- *** This bug has been marked as a duplicate of bug 1207280 *** Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 1207280] Review Request: python-semantic_version - A library implementing the 'SemVer' scheme
https://bugzilla.redhat.com/show_bug.cgi?id=1207280 Christopher Meng i...@cicku.me changed: What|Removed |Added CC||mhriv...@redhat.com --- Comment #17 from Christopher Meng i...@cicku.me --- *** Bug 1077030 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 1219612] Review Request: nodejs-mongodb-core - Core MongoDB driver functionality
https://bugzilla.redhat.com/show_bug.cgi?id=1219612
Parag AN(पराग) panem...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #3 from Parag AN(पराग) panem...@gmail.com ---
Review:
+ Package build successfully in mock F24 x86_64
+ rpmlint on all generated rpms gave output
nodejs-mongodb-core.noarch: W: only-non-binary-in-usr-lib
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
+ Packaged source verified with upstream as (sha256sum)
source in
tarball:6efaf61e6ceaa1fc322a01c480fcf347a1c8e57d911dc6546d4e87b5922d41b5
source from upstream:
6efaf61e6ceaa1fc322a01c480fcf347a1c8e57d911dc6546d4e87b5922d41b5
+ License ASL 2.0 is valid and is included in LICENSE file.
+ follow nodejs packaging guidelines
Suggestions:
1) Group tag is not necessary except you have plans to build this for EPEL5.
2) you should follow
https://fedoraproject.org/wiki/Packaging:Node.js?rd=Node.js/Packagers#ExclusiveArch
and use
ExclusiveArch: %{nodejs_arches} noarch
3) also we have a license tag to mark license files so you should use
%license LICENSE
See https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#License_Text
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 1077030] Review Request: python-semantic-version - library implementing SemVer
https://bugzilla.redhat.com/show_bug.cgi?id=1077030 --- Comment #18 from William Moreno williamjmore...@gmail.com --- Michael Hrivnak you made a great job with the spec, you can request to become a comantainer of semantic version. Please remember than for your firts package you do not need to go for all the suported epel releases -- 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 1242726] Review Request: perl-MooX-Log-Any - Role to add Log::Any
https://bugzilla.redhat.com/show_bug.cgi?id=1242726 Tim Orling ticot...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #7 from Tim Orling ticot...@gmail.com --- New Package SCM Request === Package Name: perl-MooX-Log-Any Short Description: A Moose role to add support for logging via Log::Any Upstream URL: http://search.cpan.org/dist/MooX-Log-Any/ Owners: ttorling psabata group::perl-sig Branches: f21 f22 f23 el6 epel7 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1007619] Review Request: eatmydata - Library and utilities designed to disable fsync
https://bugzilla.redhat.com/show_bug.cgi?id=1007619 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com --- Comment #9 from Miroslav Suchý msu...@redhat.com --- FYI there already exist such SW in Fedora: https://apps.fedoraproject.org/packages/nosync -- 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 1009742] Review Request: perl-Term-Menu - print an interactive menu
https://bugzilla.redhat.com/show_bug.cgi?id=1009742 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added CC||msu...@redhat.com Flags||needinfo?(chjoh...@gmail.co ||m) --- Comment #2 from Miroslav Suchý msu...@redhat.com --- Chris, any progress? Are you still interrested in this 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 1224800] Review Request: xfce-bluetooth - A bluetooth manager for XFCE
https://bugzilla.redhat.com/show_bug.cgi?id=1224800 --- Comment #7 from Raphael Groner projects...@smart.ms --- epel7 build (with buildroot override for xfce4-vala): http://koji.fedoraproject.org/koji/taskinfo?taskID=10427329 -- 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 1243915] Review Request: perl-Pinto - Curate a repository of Perl modules
https://bugzilla.redhat.com/show_bug.cgi?id=1243915 --- Comment #1 from Petr Šabata psab...@redhat.com --- * The pinto and pintod apps could be probably subpackaged; would that be feasible? I'm not sure if there are any other applications using these modules. Perhaps it doesn't make sense. * pintod is a service and should install a systemd service unit file. I see old releases of Pinto used to ship with an initscript (0.83~0.99, at least). You could probably use that for inspiration. * File::Path is only used by the unused bundled cpanm. Drop it from your buildtime dep list. * I can't see Test::Warn mentioned anywhere in the tarball. Unless you have some specific reasons for it, drop it from your dep list too. * Encode and Starman don't appear to be required at buildtime. You may drop them from your buildtime dep list. -- 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 1063039] Review Request: rubygem-em-http-request - EventMachine based, async HTTP Request client
https://bugzilla.redhat.com/show_bug.cgi?id=1063039 Nitesh Narayan Lal niteshnarayanlal...@gmail.com changed: What|Removed |Added CC||niteshnarayanlalleo@gmail.c ||om --- Comment #3 from Nitesh Narayan Lal niteshnarayanlal...@gmail.com --- Nope unfortunately I am surrounded by loads of day job and other responsibilities. I didn't get time to continue this further, sorry for 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 1244764] Review Request: rubygem-diffy - Convenient diffing in ruby
https://bugzilla.redhat.com/show_bug.cgi?id=1244764 --- Comment #4 from Ilya Gradina ilya.grad...@gmail.com --- Spec URL: http://repo.clanwars.org/gitlab/rubygem-diffy.spec SRPM URL: http://repo.clanwars.org/gitlab/rubygem-diffy-3.0.7-3.fc24.src.rpm Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=10428185 -- 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 970972] Review Request: cuisine - Chef-like functionnality for Fabric
https://bugzilla.redhat.com/show_bug.cgi?id=970972 --- Comment #4 from Nils Ratusznik nils.fed...@anotherhomepage.org --- Hi, I'm still interrested. I noticed version 0.7.10 is out, here are the updated files : Spec URL : http://medias.anotherhomepage.org/rpms/cuisine/0.7.10/cuisine.spec SRPM URL : http://medias.anotherhomepage.org/rpms/cuisine/0.7.10/cuisine-0.7.10-1.fc21.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 1063042] Review Request: rubygem-em-websocket - EventMachine based WebSocket server
https://bugzilla.redhat.com/show_bug.cgi?id=1063042 Nitesh Narayan Lal niteshnarayanlal...@gmail.com changed: What|Removed |Added CC||niteshnarayanlalleo@gmail.c ||om --- Comment #7 from Nitesh Narayan Lal niteshnarayanlal...@gmail.com --- Nope unfortunately I am surrounded by loads of day job and other responsibilities. I didn't get time to continue this further, sorry for 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 1242724] Review Request: perl-File-Find-Rule-Age - Rule to match on file age
https://bugzilla.redhat.com/show_bug.cgi?id=1242724 Petr Šabata psab...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 1242727] Review Request: perl-MooX-Roles-Pluggable - Moo eXtension for pluggable roles
https://bugzilla.redhat.com/show_bug.cgi?id=1242727 Petr Šabata psab...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 1242723] Review Request: perl-Alien-Packages - Find information of installed packages
https://bugzilla.redhat.com/show_bug.cgi?id=1242723 Petr Šabata psab...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | --- Comment #2 from Petr Šabata psab...@redhat.com --- Orion, just so you know -- I've just sponsored Tim. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 1221781] Review Request: zipios - C++ library for reading and writing Zip files
https://bugzilla.redhat.com/show_bug.cgi?id=1221781 --- Comment #8 from Raphael Groner projects...@smart.ms --- - W: fsf-wrong-address for a lot of source files, maybe all. Sorry, this should mean reference to those E: incorrect-fsf-address messages. -- 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 1242726] Review Request: perl-MooX-Log-Any - Role to add Log::Any
https://bugzilla.redhat.com/show_bug.cgi?id=1242726
Petr Šabata psab...@redhat.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #6 from Petr Šabata psab...@redhat.com ---
* You should always package the latest version available. In this case,
that would be 0.004002, released in April. The difference[0] is minimal so
I'll just continue reviewing the version you've packaged for now. Updating
will be straightforward.
Ack, updated.
* The package summary could be a little bit more descriptive, A Moose role
to add support for logging via Log::Any, maybe?
Ack, summary changed.
* The BuildRoot tag is only needed in EPEL5. You may drop it.
Ack, dropped.
* The perl version constraint isn't really all that useful in our case.
Ack, version contraint dropped.
Your package calls `make' (lines 35, 40 and 48), `find' (lines 42 and 43),
and `rm' and `rmdir' (lines 38, 42, 43 and 51). You should therefore
buildrequire `make', `findutils' and `coreutils'. If you drop some of these
(see below), drop the added dependency too, of course.
Ack, all three added.
You also execute Makefile.PL, which requires `strict' and `warnings'. Add
`perl(strict)' and `perl(warnings)' to your list.
Ack, both added.
Log::Any::Adapter isn't actually used anywhere in the code, it's only
mentioned in the metadata and, judging from the changelog, I think upstream
meant to remove it. Drop it from your BuildRequires.
Ack, dropped.
* Runtime dependencies -- (...) In your
case, both `Moo::Role' and and `Log::Any' are automatically detected.
Furthermore, as I've already mentioned, Log::Any::Adapter isn't used at all.
You can drop these three `Requires' lines.
Ack, all three dropped.
* Line 38, `rm -rf $RPM_BUILD_ROOT' -- this is no longer needed unless
you're packaging for EPEL5.
Ack, dropped.
* Note: Line 40, `make pure_install PERL_INSTALL_ROOT=$RPM_BUILD_ROOT' --
DESTDIR is supported in both Fedora and EPEL. You can use it instead of
PERL_INSTALL_ROOT here.
Ack, changed to DESTDIR.
* Line 43 (`find $RPM_BUILD_ROOT -depth -type d -exec rmdir {} 2/dev/null
\;') isn't necessary.
* The whole %clean section is only needed in EPEL5. Drop it.
* The standard %defattr definition is no longer needed. Not even in EPEL.
Drop it.
Ack, all dropped.
* Documentation. Don't package `dist.ini', `weaver.ini' or `META.json'.
These have no value for the end user. Also, `README.md' doesn't provide
anything the module perldoc doesn't. In you case, I would keep just the
simple `README file.
The license text requires special handling. Fedora mandates[1] that license
texts are installed with a special macro, %license, which ensures they get
installed even if documentation is disabled. However, EPEL doesn't support
this macro (yet?). There are many ways to work around this. You can, for
example, check whether %_licensedir is defined and if it isn't, define
%license as %doc:
%{!?_licensedir:%global license %%doc}
%license LICENSE
%doc Changes README
...
Ack, looks good.
* Finally, your changelog format isn't valid[2] (missing e-mail address).
Ack, also fixed.
--
That went well! I'm going to approve this package and sponsor you into the
Packager group. You may proceed with an SCM request[0].
Since this is a perl package, please, add `perl-sig' to InitialCC.
Once the package is created in the PkgDB, you may also want to register it in
Anitya[1], our upstream release monitoring service. It automatically checks
for new versions, submits FMN[2] notifications and, optionally, files `please
update' bugs for you.
[0] https://fedoraproject.org/wiki/Package_SCM_admin_requests
[1] https://release-monitoring.org/
[2] https://apps.fedoraproject.org/notifications/
--
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 1221781] Review Request: zipios - C++ library for reading and writing Zip files
https://bugzilla.redhat.com/show_bug.cgi?id=1221781
--- Comment #7 from Raphael Groner projects...@smart.ms ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Dist tag is present.
- Sources used to build the package match the upstream source, as provided
in the spec URL.
Note: Upstream MD5sum check error, diff is in /home/builder/fedora-
review/1221781-zipios/diff.txt
See: http://fedoraproject.org/wiki/Packaging/SourceURL
- License must be 'LGPLv2+ and GPLv2+' as there partly source with not LPGL.
Unknown or generated
zipios-2.1.0/contrib/catch.hpp
zipios-2.1.0/dev/index.php
zipios-2.1.0/src/directorycollection.cpp
- W: fsf-wrong-address for a lot of source files, maybe all.
- some directories are not owned, see down for Notes
- Please verify with FPC that you use the right package name.
I guess zipios++2 is better than zipios++, see in other
comments for reasons.
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: ldconfig called in %post and %postun if required.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
Generic:
[!]: 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: *No copyright* GPL (v2 or later) (with incorrect FSF
address), *No copyright* LGPL (v2 or later) (with incorrect FSF
address), *No copyright* GPL (v2 or later), LGPL (v2 or later)
(with incorrect FSF address), Unknown or generated. 3 files have
unknown license. Detailed output of licensecheck in /home/builder
/fedora-review/1221781-zipios/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[!]: Package requires other packages for directories it uses.
Note: No known owner of /usr/share/doc/zipios
[!]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/share/doc/zipios,
/usr/share/cmake, /usr/share/cmake/Modules
[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.
[-]: Package contains desktop file if it is a GUI application.
[x]: 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).
[?]: Package is named according to the Package Naming Guidelines.
[?]: 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.
[?]: 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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 20480 bytes in 4 files.
[!]: 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 %license.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[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 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 does not use a name that already exists.
[x]: Package is not relocatable.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[Bug 1209054] Review Request: pyqso - logging tool for radio amateurs
https://bugzilla.redhat.com/show_bug.cgi?id=1209054 Michael Schwendt (Fedora Packager Sponsors Group) bugs.mich...@gmx.net changed: What|Removed |Added Blocks||177841 (FE-NEEDSPONSOR) Depends On|177841 (FE-NEEDSPONSOR) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- 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 1228942] Review Request: python-frappe - Meta data driven web framework
https://bugzilla.redhat.com/show_bug.cgi?id=1228942 --- Comment #4 from William Moreno williamjmore...@gmail.com --- Rawhide Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10430922 -- 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 1228943] Review Request: python-frappe-bench - Tool for admin Frappe apps
https://bugzilla.redhat.com/show_bug.cgi?id=1228943 --- Comment #2 from William Moreno williamjmore...@gmail.com --- Rawhide build: http://koji.fedoraproject.org/koji/taskinfo?taskID=10431138 -- 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 1244514] Review Request: python-snappy - Python library for the snappy compression library from Google
https://bugzilla.redhat.com/show_bug.cgi?id=1244514
Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl changed:
What|Removed |Added
CC||zbys...@in.waw.pl
--- Comment #5 from Zbigniew Jędrzejewski-Szmek zbys...@in.waw.pl ---
I think that ship has sailed... and snappy the compressor will always be vastly
more recognizable than snappy the manifold mangler. So I'd vote for
python-snappy for this package, and snappy-geometry for the hypothetical other
package.
Is using %{py3dir} necessary?
--
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 1241919] Review Request: python-line_profiler - Line-by-line profiler for Python.
https://bugzilla.redhat.com/show_bug.cgi?id=1241919
--- Comment #23 from Jackson Isaac jacksonisaac2...@gmail.com ---
I am symlinking the kernprof binary to kernprof-%{python2_version} and
similarly for %{python3_version}.
But I think, we should cp kernprof to kernprof-%{python2_version} for py2 and
for py3 we can keep a symlink.
While building for py3, kernprof binary is built for the py3 version, so
kernprof-py2 will also point to the py3 version as both symlink links to the
same kernprof. Hence, the dangling symlink warning in rpmlint I believe.
Any thoughts on this ?
--
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 1203018] Review Request: baculum - WebGUI tool for Bacula Community program
https://bugzilla.redhat.com/show_bug.cgi?id=1203018 --- Comment #28 from Marcin Haba marcin.h...@bacula.pl --- Spec URL: http://bacula.pl/downloads/baculum/baculum.spec SRPM URL: http://bacula.pl/downloads/baculum/baculum-7.0.6-0.6.b.fc22.src.rpm Hello, @Dominik: You are right. Duplicate files are not desirable in any case. In Makefile from upstream side I splitted Baculum data directory to two directories: - /etc/baculum/Data-apache (provided by baculum-httpd subpackage) - /etc/baculum/Data-lighttpd (provided by baculum-lighttpd subpackage) and next in %post I link one from these directories to framework specific path in /usr/share... By this move, both points are solved: 1) Variable data stored in /etc/baculum directory, not in /usr/share 2) Files are not doubled in particular subpackages Thanks for comments. -- 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 910340] Review Request: boinctui - Fullscreen text mode manager for BOINC-client
https://bugzilla.redhat.com/show_bug.cgi?id=910340 Sergey Suslov suleman1...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |CANTFIX Flags|needinfo?(suleman1971@gmail | |.com) | Last Closed||2015-07-21 15:14:21 --- Comment #6 from Sergey Suslov suleman1...@gmail.com --- Oh, I'm so sorry. I forgot this case. Unfortunately, I migrated to Ubuntu/Debian and can't support Fedora branch now. I'm intent to close 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 1244817] Review Request: python-xvfbwrapper - run headless display inside X virtual framebuffer (Xvfb)
https://bugzilla.redhat.com/show_bug.cgi?id=1244817 Julien Enselme juj...@jujens.eu changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Julien Enselme juj...@jujens.eu --- I didn't contact the maintainer yet (it's a SHOULD item, not a blocker) Indeed. rpmlint catches a summary-not-capitalized C run headless display inside X virtual framebuffer (Xvfb) warning. You should fix this. Not blocking IMHO. 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 1221781] Review Request: zipios - C++ library for reading and writing Zip files
https://bugzilla.redhat.com/show_bug.cgi?id=1221781
--- Comment #9 from Richard Shaw hobbes1...@gmail.com ---
(In reply to Raphael Groner from comment #6)
I am still thinking that the name is misleading, see my comment #2.
The package naming guidelines do not have a strict rule here and even says the
package name could come from the project name or tarball name.
Since the packages are not API/ABI compatible and the changes are extensive, I
think having the name be different is a good thing.
Additionally, appending a number to the package name is not based on the
package version but on the major soversion of the library, but in this case it
woulds still be 2, however the soversion may eventually be 3 and that would
leave this package in a strange state.
(In reply to Raphael Groner from comment #7)
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Dist tag is present.
It was there but I forgot the ? somehow. Fixed.
- Sources used to build the package match the upstream source, as provided
in the spec URL.
Note: Upstream MD5sum check error, diff is in /home/builder/fedora-
review/1221781-zipios/diff.txt
See: http://fedoraproject.org/wiki/Packaging/SourceURL
I think this is the sourceforge issue. I downloaded the tarball directly from
SF without modification.
- License must be 'LGPLv2+ and GPLv2+' as there partly source with not LPGL.
Per the GPL compatibility matrix[1] the combined license is GPLv2+. You don't
generally list both unless it was an OR situation. Fixed.
[!]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/share/doc/zipios,
/usr/share/cmake, /usr/share/cmake/Modules
I think i fixed the doc dir, I incorrectly excluded the whole thing instead of
just the html dir which is reserved for the doc subpackage.
The other two should definitely be owned by the cmake package, not sure what's
going on here.
[!]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in zipios-
doc
f-r is confused here. You can't have an arch specific requires from a noarch
package. I've made that mistake before, bad things happen.
Ok, I'm verifying a new build fixes these problem and I'll pose new links.
--
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 1245330] New: Review Request: python-sphinxcontrib-programoutput - Extension to insert output of commands into documents
https://bugzilla.redhat.com/show_bug.cgi?id=1245330 Bug ID: 1245330 Summary: Review Request: python-sphinxcontrib-programoutput - Extension to insert output of commands into documents Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: zbys...@in.waw.pl QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://in.waw.pl/~zbyszek/fedora/python-sphinxcontrib-programoutput.spec SRPM URL: http://in.waw.pl/~zbyszek/fedora/python-sphinxcontrib-programoutput-0.8-1.fc24.src.rpm Fedora Account System Username: zbyszek Description: A Sphinx extension to literally insert the output of arbitrary commands into documents, helping you to keep your command examples up to date. The name follows the pattern of other sphinxcontrib extensions: python-sphinxcontrib-napoleon.noarch : Sphinx napoleon extension python-sphinxcontrib-adadomain.noarch : Ada domain for the Sphinx documentation generator python3-sphinxcontrib-napoleon.noarch : Sphinx napoleon extension python-sphinxcontrib-cheeseshop.noarch : Sphinx extension cheeseshop python-sphinxcontrib-httpdomain.noarch : Sphinx domain for documenting HTTP APIs python-sphinxcontrib-issuetracker.noarch : Sphinx integration with different issue trackers python3-sphinxcontrib-issuetracker.noarch : Sphinx integration with different issue trackers for Python 3 -- 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 1234905] Review Request: jpype - Full access for Python programs to Java class libraries
https://bugzilla.redhat.com/show_bug.cgi?id=1234905 --- Comment #1 from Raphael Groner projects...@smart.ms --- Spec URL: https://raphgro.fedorapeople.org/review/py/jpype/jpype.spec SRPM URL: https://raphgro.fedorapeople.org/review/py/jpype/jpype-0.6.0-2.src.rpm Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=10428806 * Tue Jul 21 2015 Raphael Groner - 0.6.0-2 - include patch of proxy argument issue Enabled python tests give me: ImportError: dynamic module does not define init function (PyInit__jpype) This error may be related to https://bugs.python.org/issue19615 python3 --version Python 3.4.2 -- 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 1233989] Review Request: gap-pkg-aclib - Almost Crystallographic groups library for GAP
https://bugzilla.redhat.com/show_bug.cgi?id=1233989 --- Comment #3 from Gerald Cox gb...@bzb.us --- Sorry, I missed this one: - Package does not own files or directories owned by other packages. Note: Dirs in package are owned also by: /usr/lib/gap/pkg/aclib/gap(languages, langpacks:, enabled, are, No), /usr/lib/gap/pkg/aclib/htm(languages, langpacks:, enabled, are, No), /usr/lib/gap/pkg/aclib/doc(languages, langpacks:, enabled, are, No), /usr/lib/gap/pkg/aclib(languages, langpacks:, enabled, are, No) -- 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 1229395] Review request: fedora-workstation-repositories - repositories for Fedora Workstation
https://bugzilla.redhat.com/show_bug.cgi?id=1229395 --- Comment #3 from Paul W. Frields pfrie...@redhat.com --- The repositories are meant to have use in the context of a Fedora Workstation installation, and would not necessarily be applicable (or might not work in Cloud case) in other editions. Those editions have the opportunity to add other metadata-enabled repositories that make sense for their use cases. -- 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 1245351] New: Review Request: ps2emu-tools - PS/2 recording/playback tools for ps2emu
https://bugzilla.redhat.com/show_bug.cgi?id=1245351 Bug ID: 1245351 Summary: Review Request: ps2emu-tools - PS/2 recording/playback tools for ps2emu Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: thatsly...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://lyude.net/~lyudess/tmp/ps2emu/ps2emu-tools.spec SRPM URL: https://lyude.net/~lyudess/tmp/ps2emu/ps2emu-tools-0.1.2-0.fc22.src.rpm Description: This contains a set of userspace tools for the ps2emu kernel modules that can record data coming from PS/2 devices, and play them back using the kernel module. The kernel module has not gone upstream yet, however the patches for it have been posted on the LKML[1] pending review, and ps2emu has already been used to fix one of the bugs on the Red Hat bugzilla[2]. You can also find a built version of this package on the copr[3]. [1] https://lkml.org/lkml/2015/7/21/685 [2] https://bugzilla.redhat.com/show_bug.cgi?id=1235175 [3] https://copr.fedoraproject.org/coprs/lyude/ps2emu-tools/ Fedora Account System Username: lyude -- 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 1203018] Review Request: baculum - WebGUI tool for Bacula Community program
https://bugzilla.redhat.com/show_bug.cgi?id=1203018 --- Comment #27 from Marcin Haba marcin.h...@bacula.pl --- Hello, Thanks for your suggestions. (In reply to Dominik 'Rathann' Mierzejewski from comment #26) If this file is modified over time, you should put it somewhere in /var (/var/lib/baculum perhaps) and mark it with %verify macro accordingly so that rpm -V doesn't complain when it's modified. /usr/share is for static content. /var/lib/ is for applications state data. From this reason I do not think that auth data may be stored there. /etc/baculum is better for me, however it is a bit strage for me that web application is allowed to modify /etc/* data. If it is OK for Fedora and there does not exist better candidate for storing these auth data, then I have no choice and I will use /etc/baculum/ baculum-httpd.noarch: E: non-readable /usr/share/baculum/htdocs/protected/Data/baculum.users 0600L $ rpmlint ./baculum-lighttpd-7.0.6-0.5.b.fc22.noarch.rpm [...] baculum-lighttpd.noarch: E: non-readable /usr/share/baculum/htdocs/protected/Data/baculum.users 0600L Please note that you should not duplicate files between two subpackages: https://fedoraproject.org/wiki/Packaging:Guidelines#Duplicate_Files Yes, it is true that these two packages contain the same file. Nevertheless these files are in subpackages that cannot be installed together. I used Conclicts tag for them and I explained it in Spec comment. I know that Conflicts tag is not too desirable in Fedora due to general design requirements. In this case I think that it is reasonable (please see comment in Spec). -- 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 1241808] Review Request: python-oslo-cache - Cache storage for Openstack projects.
https://bugzilla.redhat.com/show_bug.cgi?id=1241808 Ryan Brown rybr...@redhat.com changed: What|Removed |Added CC||rybr...@redhat.com --- Comment #3 from Ryan Brown rybr...@redhat.com --- +1, looks good to me. -- 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 1015868] Review Request: python-qutepart - Source code text editor component based on Qt
https://bugzilla.redhat.com/show_bug.cgi?id=1015868
--- Comment #29 from Raphael Groner projects...@smart.ms ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Permissions on files are set properly.
Note: See rpmlint output
See: http://fedoraproject.org/wiki/Packaging/Guidelines#FilePermissions
- In Fedora we have multiple python runtimes, one for each supported major
release. At this point that's one for python2.x and one for python3.x.
If a piece of software supports python3, it must be packaged for python3.
If it supports python2 as well, it may be packaged for python2. If it
supports only python2 then it must not be packaged for python3.
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[!]: Development (unversioned) .so files in -devel subpackage, if present.
Note: Unversioned so-files in private %_libdir subdirectory (see
attachment). Verify they are not in ld path.
[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.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: Unknown or generated. 72 files have unknown license. Detailed
output of licensecheck in /home/builder/fedora-review/1015868-python-
qutepart/licensecheck.txt
= Okay, LICENSE file states LPGLv2+.
[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.
[-]: Package contains desktop file if it is a GUI application.
[-]: 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.
[x]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 1 files.
[?]: 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 %license.
[x]: Package requires other packages for directories it uses.
[x]: Package must own all directories that it creates.
[x]: Package does not own files or directories owned by other packages.
[x]: All build dependencies are listed in BuildRequires, except for any
that are listed in the exceptions section of Packaging Guidelines.
[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]: Dist tag is present.
[x]: Package does not contain duplicates in %files.
[x]: Package use %makeinstall only when make install DESTDIR=... doesn't
work.
[x]: Package is named using only allowed ASCII characters.
[x]: Package does not use a name that already exists.
[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
Python:
[-]: Python eggs must not download any dependencies during the build
process.
[-]: A package which is used by another package via an egg interface should
provide egg info.
[!]: Package meets the Packaging Guidelines::Python
= Please consider to build for Python3.
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[-]:
[Bug 1245356] New: Review Request: python-ansi - ANSI cursor movement and graphics
https://bugzilla.redhat.com/show_bug.cgi?id=1245356 Bug ID: 1245356 Summary: Review Request: python-ansi - ANSI cursor movement and graphics Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@dyroff.org QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://www.dyroff.net/fedora/python-ansi.spec SRPM URL: http://www.dyroff.net/fedora/python-ansi-0.1.3-1.fc22.src.rpm Description: Various ANSI escape codes, used in moving the cursor in a text console or rendering colored text with Python Fedora Account System Username: sdyroff -- 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 808350] Review Request: racket - Racket development system
https://bugzilla.redhat.com/show_bug.cgi?id=808350 Daniel E. Wilson d...@bureau-13.org changed: What|Removed |Added Flags|needinfo? | --- Comment #63 from Daniel E. Wilson d...@bureau-13.org --- So far, I have heard nothing on this issue. I do have version 6.2 and I will upload those files when I have the time. The only problem will be that as near as I can tell there is no way this package is getting into Fedora. -- 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 1245356] Review Request: python-ansi - ANSI cursor movement and graphics
https://bugzilla.redhat.com/show_bug.cgi?id=1245356 Christopher Meng i...@cicku.me changed: What|Removed |Added Status|NEW |ASSIGNED CC||i...@cicku.me Assignee|nob...@fedoraproject.org|i...@cicku.me Flags||fedora-review? --- Comment #1 from Christopher Meng i...@cicku.me --- Better source url: https://pypi.python.org/packages/source/a/ansi/ansi-0.1.3.tar.gz to avoid dumb folder name like ansi-ansi and github source guideline ... -- 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 1077030] Review Request: python-semantic-version - library implementing SemVer
https://bugzilla.redhat.com/show_bug.cgi?id=1077030 --- Comment #16 from William Moreno williamjmore...@gmail.com --- This app is alredy in Fedora since 2015-05-28 https://admin.fedoraproject.org/pkgdb/package/python-semantic_version/ -- 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 1228943] Review Request: python-frappe-bench - Tool for admin Frappe apps
https://bugzilla.redhat.com/show_bug.cgi?id=1228943 Bug 1228943 depends on bug 1228941, which changed state. Bug 1228941 Summary: Review Request: python-semantic-version - Python2 library implementing the 'SemVer' scheme https://bugzilla.redhat.com/show_bug.cgi?id=1228941 What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX -- 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 1228941] Review Request: python-semantic-version - Python2 library implementing the 'SemVer' scheme
https://bugzilla.redhat.com/show_bug.cgi?id=1228941 William Moreno williamjmore...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX Flags|needinfo?(williamjmorenor@g | |mail.com) | Last Closed||2015-07-21 18:37:23 --- Comment #2 from William Moreno williamjmore...@gmail.com --- This app is already in Fedora https://admin.fedoraproject.org/pkgdb/package/python-semantic_version/ -- 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 1228942] Review Request: python-frappe - Meta data driven web framework
https://bugzilla.redhat.com/show_bug.cgi?id=1228942 Bug 1228942 depends on bug 1228941, which changed state. Bug 1228941 Summary: Review Request: python-semantic-version - Python2 library implementing the 'SemVer' scheme https://bugzilla.redhat.com/show_bug.cgi?id=1228941 What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX -- 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 1228942] Review Request: python-frappe - Meta data driven web framework
https://bugzilla.redhat.com/show_bug.cgi?id=1228942 --- Comment #3 from William Moreno williamjmore...@gmail.com --- Spec URL: https://williamjmorenor.fedorapeople.org/rpmdev/python-frappe.spec SRPM URL: https://williamjmorenor.fedorapeople.org/rpmdev/python-frappe-5.1.3-1.fc22.src.rpm Last upstream release -- 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 1241156] Review Request: go-srpm-macros - RPM macros for building Golang packages for various architectures
https://bugzilla.redhat.com/show_bug.cgi?id=1241156 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|go-srpm-macros-1-1.fc22 |go-srpm-macros-1-1.fc21 --- Comment #11 from Fedora Update System upda...@fedoraproject.org --- go-srpm-macros-1-1.fc21 has been pushed to the Fedora 21 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1239106] Review Request: nodejs-is-redirect - Check if a number is a redirect HTTP status code
https://bugzilla.redhat.com/show_bug.cgi?id=1239106 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||nodejs-is-redirect-1.0.0-1. ||fc22 Resolution|--- |ERRATA Last Closed||2015-07-21 04:22:10 --- Comment #7 from Fedora Update System upda...@fedoraproject.org --- nodejs-is-redirect-1.0.0-1.fc22 has been pushed to the Fedora 22 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1239222] Review Request: php-myclabs-deep-copy - Create deep copies (clones) of your objects
https://bugzilla.redhat.com/show_bug.cgi?id=1239222 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|php-myclabs-deep-copy-1.3.0 |php-myclabs-deep-copy-1.3.0 |-1.fc22 |-1.fc21 --- Comment #14 from Fedora Update System upda...@fedoraproject.org --- php-myclabs-deep-copy-1.3.0-1.fc21 has been pushed to the Fedora 21 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1221459] Review Request: hgsubversion - Mercurial extension for Subversion
https://bugzilla.redhat.com/show_bug.cgi?id=1221459 Bug 1221459 depends on bug 1216264, which changed state. Bug 1216264 Summary: subversion-python broken: causing ViewVC to fail https://bugzilla.redhat.com/show_bug.cgi?id=1216264 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- 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 1239107] Review Request: nodejs-nested-error-stacks - An Error subclass that will chain nested Errors and dump nested stacktraces
https://bugzilla.redhat.com/show_bug.cgi?id=1239107 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||nodejs-nested-error-stacks- ||1.0.0-1.fc22 Resolution|--- |ERRATA Last Closed||2015-07-21 04:16:37 --- Comment #7 from Fedora Update System upda...@fedoraproject.org --- nodejs-nested-error-stacks-1.0.0-1.fc22 has been pushed to the Fedora 22 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1243499] Review Request: python-configparser - Backport of python 3 configparser module to python 2.7 (and 2.6)
https://bugzilla.redhat.com/show_bug.cgi?id=1243499 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- python-configparser-3.5.0b2-0.2.fc22 has been submitted as an update for Fedora 22. https://admin.fedoraproject.org/updates/python-configparser-3.5.0b2-0.2.fc22 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1208616] Review Request: consul - Tool for service discovery, monitoring and configuration http://www.consul.io
https://bugzilla.redhat.com/show_bug.cgi?id=1208616 --- Comment #19 from Boris Ranto bra...@redhat.com --- @Kevin: I just checked my fas account and I am in the packager group and I use the aforementioned e-mail there. I'm not sure what went wrong in the script though. Maybe a network fluke? -- 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 1208911] Review Request: doublecmd - Twin-panel (commander-style) file manager (Qt4 and GTK2)
https://bugzilla.redhat.com/show_bug.cgi?id=1208911 --- Comment #13 from Vít Ondruch vondr...@redhat.com --- (In reply to Raphael Groner from comment #11) Though, I don't see any reason why beta target is needed, do you mean to put in Release? Version 0.6.4 is official with no beta hint at all, the zero in front of version should be enough identification for any beta thought of upstream. I think that the beta builds everything, including the plugins (you do this in separate step) and debug information. Not sure what and if there is any other difference, but I doubt that. (In reply to Raphael Groner from comment #12) FTBFS for arch i686: It builds in Rawhide and F21 at least [1], but the F22 build is broken ATM (it is actually Rawhide build, have to wait till Lazarus lands in updates for F22 :/ ). [1] https://copr.fedoraproject.org/coprs/vondruch/doublecmd/ -- 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 1243499] Review Request: python-configparser - Backport of python 3 configparser module to python 2.7 (and 2.6)
https://bugzilla.redhat.com/show_bug.cgi?id=1243499 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- python-configparser-3.5.0b2-0.2.fc21 has been submitted as an update for Fedora 21. https://admin.fedoraproject.org/updates/python-configparser-3.5.0b2-0.2.fc21 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077030] Review Request: python-semantic-version - library implementing SemVer
https://bugzilla.redhat.com/show_bug.cgi?id=1077030 --- Comment #15 from Miroslav Suchý msu...@redhat.com --- I agree that moving the _licencedir stuff to the top makes sense, but when I do so, the el6 build reports this error: It is weird. I have no idea why it is happening. Ok, you can keep it there. Question 2: After making the requested changes, I get the following errors on el6 from rpmlint. According: https://fedoraproject.org/wiki/Packaging:Guidelines#File_Permissions It is not needed since rpm 4.4 and rhel6 contains rpm-4.8 If rpmlint warns you, then I would say it is rpmlint issue. However if you feel bad about those warning, you can keep it there if you want to. It is really harmless. Can you please post SRPM with remaining issues from #12 fixed. I can then give you approval stamp. -- 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 1244102] Review Request: php-composer-spdx-licenses - SPDX licenses list and validation library
https://bugzilla.redhat.com/show_bug.cgi?id=1244102 Remi Collet fed...@famillecollet.com changed: What|Removed |Added CC||shawn.iwin...@gmail.com --- Comment #11 from Remi Collet fed...@famillecollet.com --- @Shawn, would you have a few minutes for a quick formal review (as the informal already done most of the work). -- 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 1203018] Review Request: baculum - WebGUI tool for Bacula Community program
https://bugzilla.redhat.com/show_bug.cgi?id=1203018 --- Comment #26 from Dominik 'Rathann' Mierzejewski domi...@greysector.net --- (In reply to Marcin Haba from comment #23) For now I would solve this error: baculum-httpd.noarch: E: non-readable /usr/share/baculum/htdocs/protected/Data/baculum.users 0600L In this baculum.users file are stored HTTP Basic auth credentials. From this reason privilages are 0600. I think that I cannot move the file to /etc because the file is modifying via Baculum web interface and it might be strange that web application make modification in /etc file. And of course it is danterous. If this file is modified over time, you should put it somewhere in /var (/var/lib/baculum perhaps) and mark it with %verify macro accordingly so that rpm -V doesn't complain when it's modified. /usr/share is for static content. /etc/baculum could also be used. See https://fedoraproject.org/wiki/Packaging:Guidelines#Filesystem_Layout . [...] $ rpmlint ./baculum-httpd-7.0.6-0.5.b.fc22.noarch.rpm [...] baculum-httpd.noarch: E: non-readable /usr/share/baculum/htdocs/protected/Data/baculum.users 0600L $ rpmlint ./baculum-lighttpd-7.0.6-0.5.b.fc22.noarch.rpm [...] baculum-lighttpd.noarch: E: non-readable /usr/share/baculum/htdocs/protected/Data/baculum.users 0600L Please note that you should not duplicate files between two subpackages: https://fedoraproject.org/wiki/Packaging:Guidelines#Duplicate_Files -- 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 1239222] Review Request: php-myclabs-deep-copy - Create deep copies (clones) of your objects
https://bugzilla.redhat.com/show_bug.cgi?id=1239222 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||php-myclabs-deep-copy-1.3.0 ||-1.fc22 Resolution|--- |ERRATA Last Closed||2015-07-21 04:13:40 --- Comment #13 from Fedora Update System upda...@fedoraproject.org --- php-myclabs-deep-copy-1.3.0-1.fc22 has been pushed to the Fedora 22 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1233988] Review Request: gap-pkg-crystcat - Crystallographic groups catalog
https://bugzilla.redhat.com/show_bug.cgi?id=1233988 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #6 from Fedora Update System upda...@fedoraproject.org --- gap-pkg-crystcat-1.1.6-1.fc22 has been pushed to the Fedora 22 testing repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1241156] Review Request: go-srpm-macros - RPM macros for building Golang packages for various architectures
https://bugzilla.redhat.com/show_bug.cgi?id=1241156 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||go-srpm-macros-1-1.fc22 Resolution|--- |ERRATA Last Closed||2015-07-21 04:19:16 --- Comment #10 from Fedora Update System upda...@fedoraproject.org --- go-srpm-macros-1-1.fc22 has been pushed to the Fedora 22 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1228869] Review Request: gdouros-asea-fonts - an etude on the dominant typeface of Greek typography
https://bugzilla.redhat.com/show_bug.cgi?id=1228869 --- Comment #13 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 1228869] Review Request: gdouros-asea-fonts - an etude on the dominant typeface of Greek typography
https://bugzilla.redhat.com/show_bug.cgi?id=1228869 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 1231459] Review Request: libosmium - Fast and flexible C++ library for working with OpenStreetMap data
https://bugzilla.redhat.com/show_bug.cgi?id=1231459 --- Comment #15 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 751119] Review Request: perl-Judy - Perl wrapper for Judy
https://bugzilla.redhat.com/show_bug.cgi?id=751119 Petr Šabata psab...@redhat.com changed: What|Removed |Added Flags||needinfo?(holca...@gmail.co ||m) --- Comment #11 from Petr Šabata psab...@redhat.com --- Ping. -- 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 1222662] Review Request: notify-sharp3 - C# Desktop Notification with gtk3
https://bugzilla.redhat.com/show_bug.cgi?id=1222662 --- Comment #1 from Claudio Rodrigo Pereyra DIaz claudiorodr...@pereyradiaz.com.ar --- Spec URL: https://elsupergomez.fedorapeople.org/SPECS/notify-sharp3.spec SRPM URL: https://elsupergomez.fedorapeople.org/SRPMS/notify-sharp3-3.0.3-1.fc22.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 1241614] Review Request: perl-Text-Reflow - Perl module for reflowing text files using Knuth's paragraphing algorithm
https://bugzilla.redhat.com/show_bug.cgi?id=1241614 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 679980] Review Request: octopuslb - TCP/IP Load Balancer
https://bugzilla.redhat.com/show_bug.cgi?id=679980 Miroslav Suchý msu...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need 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 1231459] Review Request: libosmium - Fast and flexible C++ library for working with OpenStreetMap data
https://bugzilla.redhat.com/show_bug.cgi?id=1231459 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 1234791] Review Request: perl-Inline-Python - Write Perl subs and classes in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1234791 Petr Šabata psab...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-Inline-Python-0.49-3.e ||l6 ||perl-Inline-Python-0.49-3.f ||c21 Resolution|--- |CURRENTRELEASE Last Closed||2015-07-21 07:28:15 --- Comment #14 from Petr Šabata psab...@redhat.com --- I see you've built the package and it's not in stable. I'll close the bug for you... -- 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 1233963] Review Request: python-cachetools - Extensible memoizing collections and decorators
https://bugzilla.redhat.com/show_bug.cgi?id=1233963 --- 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 1233963] Review Request: python-cachetools - Extensible memoizing collections and decorators
https://bugzilla.redhat.com/show_bug.cgi?id=1233963 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 1243507] Review Request: perl-Algorithm-LUHN - Calculate the Modulus 10 Double Add Double checksum
https://bugzilla.redhat.com/show_bug.cgi?id=1243507 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 1243507] Review Request: perl-Algorithm-LUHN - Calculate the Modulus 10 Double Add Double checksum
https://bugzilla.redhat.com/show_bug.cgi?id=1243507 --- 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 1208911] Review Request: doublecmd - Twin-panel (commander-style) file manager (Qt4 and GTK2)
https://bugzilla.redhat.com/show_bug.cgi?id=1208911 --- Comment #14 from Raphael Groner projects...@smart.ms --- [1] https://copr.fedoraproject.org/coprs/vondruch/doublecmd/ I decided to not like building from subversion, at least not for an official package and if there's no patch needed. FTBFS is magically fixed. But ARM seems to partly build for ages (duration is 1 day), maybe we should consider to use an ExcludeArch here. Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=10424958 -- 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
