[Bug 1295256] Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295256 Remi Collet changed: What|Removed |Added CC||fed...@famillecollet.com --- Comment #1 from Remi Collet --- Please provide link to .src.rpm (cannot create myself without the getsource.sh) -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Remi Collet changed: What|Removed |Added Flags||fedora-review+ --- Comment #5 from Remi Collet --- No blocker. === 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 --- Comment #4 from Remi Collet --- Created attachment 369 --> https://bugzilla.redhat.com/attachment.cgi?id=369&action=edit review.txt Generated by fedora-review 0.6.0 (3c5c9d7) last change: 2015-05-20 Command line :/usr/bin/fedora-review -r -n /dev/shm/extras/SRPMS/php-container-interop-1.1.0-1.fc21.remi.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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Remi Collet changed: What|Removed |Added Status|NEW |ASSIGNED Assignee|nob...@fedoraproject.org|fed...@famillecollet.com --- Comment #3 from Remi Collet --- Created attachment 368 --> https://bugzilla.redhat.com/attachment.cgi?id=368&action=edit phpci.log phpCompatInfo version 5.0.0 DB built Dec 29 2015 09:34:25 CET static analyze results -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 --- Comment #2 from Upstream Release Monitoring --- remi's scratch build of php-container-interop-1.1.0-1.fc21.remi.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12399866 -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Remi Collet changed: What|Removed |Added CC||fed...@famillecollet.com --- Comment #1 from Remi Collet --- Can you please fix path to 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 1295217] Review Request: msgpuck - a MsgPack serialization library in a self-contained header file
https://bugzilla.redhat.com/show_bug.cgi?id=1295217 --- Comment #1 from Roman Tsisyk --- I'm upstream maintainer. My other packages: https://bugzilla.redhat.com/show_bug.cgi?id=1293100 https://bugzilla.redhat.com/show_bug.cgi?id=1295209 -- 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 1295209] Review Request: lua-fun - functional programming library for Lua and LuaJIT
https://bugzilla.redhat.com/show_bug.cgi?id=1295209 --- Comment #1 from Roman Tsisyk --- I'm upstream maintainer. My other packages: https://bugzilla.redhat.com/show_bug.cgi?id=1293100 https://bugzilla.redhat.com/show_bug.cgi?id=1295217 -- 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 960061] Review Request: rubygem-guard - Guard gives notifications about file modifications
https://bugzilla.redhat.com/show_bug.cgi?id=960061 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 967396] Review Request: rubygem-chosen-rails - Integrate Chosen JavaScript library with Rails asset pipeline
https://bugzilla.redhat.com/show_bug.cgi?id=967396 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 839649] Review Request: rubygem-rails_best_practices - a code metric tool for rails codes, written in Ruby.
https://bugzilla.redhat.com/show_bug.cgi?id=839649 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1218779] Review Request: vagrant-triggers - Vagrant plugin to allow using arbitrary commands on host before/after Vagrant commands
https://bugzilla.redhat.com/show_bug.cgi?id=1218779 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1225406] Review Request: rubygem-jruby-openssl - JRuby's OpenSSL gem
https://bugzilla.redhat.com/show_bug.cgi?id=1225406 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 967332] Review Request: rubygem-chardet - Character encoding auto-detection in Ruby
https://bugzilla.redhat.com/show_bug.cgi?id=967332 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 967333] Review Request: rubygem-code_analyzer - Helps build custom code analyzer tools
https://bugzilla.redhat.com/show_bug.cgi?id=967333 John Skeoch changed: What|Removed |Added Assignee|jstri...@redhat.com |hho...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1295260] Review Request: php-mnapoli-phpunit-easymock - Helpers to build PHPUnit mocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295260 Shawn Iwinski changed: What|Removed |Added Blocks||1295258 (php-di) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295258 [Bug 1295258] Review Request: php-di - The dependency injection container for humans -- 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 1295260] Review Request: php-mnapoli-phpunit-easymock - Helpers to build PHPUnit mocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295260 Shawn Iwinski changed: What|Removed |Added Alias||php-mnapoli-phpunit-easymoc ||k -- 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 1295258] Review Request: php-di - The dependency injection container for humans
https://bugzilla.redhat.com/show_bug.cgi?id=1295258 Shawn Iwinski changed: What|Removed |Added Depends On||1295260 ||(php-mnapoli-phpunit-easymo ||ck) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295260 [Bug 1295260] Review Request: php-mnapoli-phpunit-easymock - Helpers to build PHPUnit mocks -- 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 1295260] New: Review Request: php-mnapoli-phpunit-easymock - Helpers to build PHPUnit mocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295260 Bug ID: 1295260 Summary: Review Request: php-mnapoli-phpunit-easymock - Helpers to build PHPUnit mocks Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/d5ec0ad336c6951a258a108bdd4c1c82515b829e/php-mnapoli-phpunit-easymock/php-mnapoli-phpunit-easymock.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-mnapoli-phpunit-easymock-0.2.0-1.fc23.src.rpm Description: Helpers to build PHPUnit mocks. Autoloader: /usr/share/php/EasyMock/autoload.php Fedora Account System Username: siwinski -- 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 1295258] Review Request: php-di - The dependency injection container for humans
https://bugzilla.redhat.com/show_bug.cgi?id=1295258 Shawn Iwinski changed: What|Removed |Added Blocks||1295259 ||(php-di-symfony2-bridge) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295259 [Bug 1295259] Review Request: php-di-symfony2-bridge - PHP-DI integration with Symfony 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 1295259] Review Request: php-di-symfony2-bridge - PHP-DI integration with Symfony 2
https://bugzilla.redhat.com/show_bug.cgi?id=1295259 Shawn Iwinski changed: What|Removed |Added Depends On||1295258 (php-di) Alias||php-di-symfony2-bridge Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295258 [Bug 1295258] Review Request: php-di - The dependency injection container for humans -- 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 1295259] New: Review Request: php-di-symfony2-bridge - PHP-DI integration with Symfony 2
https://bugzilla.redhat.com/show_bug.cgi?id=1295259 Bug ID: 1295259 Summary: Review Request: php-di-symfony2-bridge - PHP-DI integration with Symfony 2 Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/502f453cbb36c893167ba5041bc8c4d414db13c3/php-di-symfony2-bridge/php-di-symfony2-bridge.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-di-symfony2-bridge-1.1.0-1.fc23.noarch.rpm Description: PHP-DI integration with Symfony 2. Autoloader: /usr/share/php/DI/Bridge/Symfony/autoload.php Fedora Account System Username: siwinski -- 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 1295257] Review Request: php-di-invoker - Generic and extensible callable invoker
https://bugzilla.redhat.com/show_bug.cgi?id=1295257 Shawn Iwinski changed: What|Removed |Added Blocks||1295258 (php-di) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295258 [Bug 1295258] Review Request: php-di - The dependency injection container for humans -- 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 1295256] Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295256 Shawn Iwinski changed: What|Removed |Added Blocks||1295258 (php-di) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295258 [Bug 1295258] Review Request: php-di - The dependency injection container for humans -- 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 1295258] Review Request: php-di - The dependency injection container for humans
https://bugzilla.redhat.com/show_bug.cgi?id=1295258 Shawn Iwinski changed: What|Removed |Added Depends On||1295255 ||(php-container-interop), ||1295256 ||(php-di-phpdoc-reader), ||1295257 (php-di-invoker) Alias||php-di Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295255 [Bug 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.) https://bugzilla.redhat.com/show_bug.cgi?id=1295256 [Bug 1295256] Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks https://bugzilla.redhat.com/show_bug.cgi?id=1295257 [Bug 1295257] Review Request: php-di-invoker - Generic and extensible callable invoker -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Shawn Iwinski changed: What|Removed |Added Blocks||1295258 (php-di) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295258 [Bug 1295258] Review Request: php-di - The dependency injection container for humans -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Shawn Iwinski changed: What|Removed |Added Blocks||1295257 (php-di-invoker) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295257 [Bug 1295257] Review Request: php-di-invoker - Generic and extensible callable invoker -- 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 1295258] New: Review Request: php-di - The dependency injection container for humans
https://bugzilla.redhat.com/show_bug.cgi?id=1295258 Bug ID: 1295258 Summary: Review Request: php-di - The dependency injection container for humans Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/1e8547d8b4f9afaa46143c5beb8de02145a0a50e/php-di/php-di.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-di-5.2.0-1.fc23.noarch.rpm Description: The dependency injection container for humans. Autoloader: /usr/share/php/DI/autoload.php Fedora Account System Username: siwinski -- 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 1295257] Review Request: php-di-invoker - Generic and extensible callable invoker
https://bugzilla.redhat.com/show_bug.cgi?id=1295257 Shawn Iwinski changed: What|Removed |Added Depends On||1295255 ||(php-container-interop) Alias||php-di-invoker Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295255 [Bug 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.) -- 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 1295254] New: Review Request: php-psr-cache - PSR Cache: Common interface for caching libraries
https://bugzilla.redhat.com/show_bug.cgi?id=1295254 Bug ID: 1295254 Summary: Review Request: php-psr-cache - PSR Cache: Common interface for caching libraries Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/4d553ca46cb3d1481190263b43d10537655d80cb/php-psr-cache/php-psr-cache.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-psr-cache-1.0.0-1.fc23.noarch.rpm Description: This package holds all interfaces defined by PSR-6 [1]. Note that this is not a Cache implementation of its own. It is merely an interface that describes a Cache implementation. See the specification for more details. Autoloader: /usr/share/php/Psr/Cache/autoload.php [1] http://www.php-fig.org/psr/psr-6/ Fedora Account System Username: siwinski -- 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 1295257] New: Review Request: php-di-invoker - Generic and extensible callable invoker
https://bugzilla.redhat.com/show_bug.cgi?id=1295257 Bug ID: 1295257 Summary: Review Request: php-di-invoker - Generic and extensible callable invoker Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/d2ec09742470ef3c4acb0b4d37f5792ceb5abfbc/php-di-invoker/php-di-invoker.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-di-invoker-1.2.0-1.fc23.noarch.rpm Description: Generic and extensible callable invoker. Autoloader: /usr/share/php/Invoker/autoload.php Fedora Account System Username: siwinski -- 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 1295256] Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295256 Shawn Iwinski changed: What|Removed |Added Alias||php-di-phpdoc-reader -- 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 1295255] Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Shawn Iwinski changed: What|Removed |Added Alias||php-container-interop -- 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 1295256] New: Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks
https://bugzilla.redhat.com/show_bug.cgi?id=1295256 Bug ID: 1295256 Summary: Review Request: php-di-phpdoc-reader - Parses @var and @param values in PHP docblocks Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/e9e8e1fcce636d6616d9ee4694b02a5b5ccd12db/php-di-phpdoc-reader/php-di-phpdoc-reader.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-di-phpdoc-reader-2.0.1-1.fc23.noarch.rpm Description: Parses @var and @param values in PHP docblocks. Autoloader: /usr/share/php/PhpDocReader/autoload.php Fedora Account System Username: siwinski -- 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 1295254] Review Request: php-psr-cache - PSR Cache: Common interface for caching libraries
https://bugzilla.redhat.com/show_bug.cgi?id=1295254 Shawn Iwinski changed: What|Removed |Added Alias||php-psr-cache -- 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 1295255] New: Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.)
https://bugzilla.redhat.com/show_bug.cgi?id=1295255 Bug ID: 1295255 Summary: Review Request: php-container-interop - Promoting the interoperability of container objects (DIC, SL, etc.) Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: sh...@iwin.ski QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/siwinski/rpms/f4dd4274d09b5c96362c7b71c01169e0d5a9524b/php-container-interop/php-container-interop.spec SRPM URL: https://siwinski.fedorapeople.org/SRPMS/php-container-interop-1.1.0-1.fc23.noarch.rpm Description: container-interop tries to identify and standardize features in container objects (service locators, dependency injection containers, etc.) to achieve interopererability. Through discussions and trials, we try to create a standard, made of common interfaces but also recommendations. If PHP projects that provide container implementations begin to adopt these common standards, then PHP applications and projects that use containers can depend on the common interfaces instead of specific implementations. This facilitates a high-level of interoperability and flexibility that allows users to consume any container implementation that can be adapted to these interfaces. The work done in this project is not officially endorsed by the PHP-FIG [1], but it is being worked on by members of PHP-FIG and other good developers. We adhere to the spirit and ideals of PHP-FIG, and hope this project will pave the way for one or more future PSRs. Autoloader: /usr/share/php/Interop/Container/autoload.php [1] http://www.php-fig.org/ Fedora Account System Username: siwinski -- 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 1263821] Review Request: dput-ng - Next generation Debian package upload tool
https://bugzilla.redhat.com/show_bug.cgi?id=1263821 --- Comment #6 from Michael Kuhn --- Based on the comment at https://bugzilla.redhat.com/show_bug.cgi?id=1278964#c5, I have updated the package according to the new Python packaging guidelines. Spec: https://ikkoku.de/~suraia/dput-ng/dput-ng.spec SRPM: https://ikkoku.de/~suraia/dput-ng/dput-ng-1.10-3.fc23.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 1278964] Review Request: distro-info - Provides information about releases of Debian and Ubuntu
https://bugzilla.redhat.com/show_bug.cgi?id=1278964
--- Comment #6 from Michael Kuhn ---
> You're using unversioned python names and macros assuming that they'll
> remain Python 2. We aren't supposed to assume that anymore, as we move
> toward Python 3. Accordingly, please use %{python2_sitelib}, prefix your
> python- packages as python2- ones, and use python2 packages instead of
> python packages for Python 2 dependencies.
>
> More details on the Python packaging guidelines here:
> https://fedoraproject.org/wiki/Packaging:Python
Thanks for your comment! I have updated the packages based on the new
guidelines.
Spec: https://ikkoku.de/~suraia/distro-info/distro-info.spec
SRPM: https://ikkoku.de/~suraia/distro-info/distro-info-0.14-3.fc23.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 1278963] Review Request: distro-info-data - Information about releases of Debian and Ubuntu (data files)
https://bugzilla.redhat.com/show_bug.cgi?id=1278963 --- Comment #4 from Michael Kuhn --- Based on the comment at https://bugzilla.redhat.com/show_bug.cgi?id=1278964#c5, I have fixed a small Python issue. Spec: https://ikkoku.de/~suraia/distro-info-data/distro-info-data.spec SRPM: https://ikkoku.de/~suraia/distro-info-data/distro-info-data-0.28-3.fc23.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 1295249] Review Request: python-gatspy - General tools for Astronomical Time Series in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1295249 Christian Dersch changed: What|Removed |Added Blocks||115 (Astronomy-SIG) Depends On||1295237 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=115 [Bug 115] Astronomy SIG - review tracker https://bugzilla.redhat.com/show_bug.cgi?id=1295237 [Bug 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother -- 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 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 Christian Dersch changed: What|Removed |Added Blocks||1295249 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1295249 [Bug 1295249] Review Request: python-gatspy - General tools for Astronomical Time Series in Python -- 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 1295249] New: Review Request: python-gatspy - General tools for Astronomical Time Series in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1295249 Bug ID: 1295249 Summary: Review Request: python-gatspy - General tools for Astronomical Time Series in Python Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: lupi...@mailbox.org QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://lupinix.fedorapeople.org/review/python-gatspy.spec SRPM URL: https://lupinix.fedorapeople.org/review/python-gatspy-0.2.1-1.fc23.src.rpm Description: Gatspy contains efficient, well-documented implementations of several common routines for Astronomical time series analysis, including the Lomb-Scargle periodogram, the Supersmoother method, and others. Fedora Account System Username: lupinix Thank you for review in advance! -- 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 1294577] Review Request: jmtpfs - FUSE and libmtp based filesystem for accessing MTP devices
https://bugzilla.redhat.com/show_bug.cgi?id=1294577 --- Comment #4 from Upstream Release Monitoring --- lupinix's scratch build of jmtpfs-0.4-2.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12398438 -- 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 1294577] Review Request: jmtpfs - FUSE and libmtp based filesystem for accessing MTP devices
https://bugzilla.redhat.com/show_bug.cgi?id=1294577 --- Comment #2 from Upstream Release Monitoring --- lupinix's scratch build of jmtpfs-0.4-2.src.rpm for epel7 completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12398449 -- 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 1294577] Review Request: jmtpfs - FUSE and libmtp based filesystem for accessing MTP devices
https://bugzilla.redhat.com/show_bug.cgi?id=1294577
Christian Dersch changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #2 from Upstream Release Monitoring
---
lupinix's scratch build of jmtpfs-0.4-2.src.rpm for epel7 completed
http://koji.fedoraproject.org/koji/taskinfo?taskID=12398449
--- Comment #3 from Christian Dersch ---
Approved!
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- Package installs properly.
Note: Installation errors (see attachment)
See: https://fedoraproject.org/wiki/Packaging:Guidelines
===> Should be a result of bug #1264803 => False positive, not an issue
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Header files in -devel subpackage, if present.
[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: "MIT/X11 (BSD like)", "*No copyright* GPL (v3)", "Unknown or
generated". 18 files have unknown license. Detailed output of
licensecheck in /home/review/1294577-jmtpfs/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[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.
[x]: Each %files section contains %defattr if rpm < 4.4
Note: %defattr present but not needed
[-]: 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.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: 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]: 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]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
[-]: Buildroot is not present
Note: Buildroot: present but not needed
[-]: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
Note: %clean present but not required
===> I think you add these for older EPEL releases
[Bug 1295055] Review Request: python-flake8-import-order - Flake8 plugin for checking order of imports in Python code
https://bugzilla.redhat.com/show_bug.cgi?id=1295055 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #10 from Fedora Update System --- python-flake8-import-order-0.6.1-1.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-1afa7a3d77 -- 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 1293049] Review Request: libpasastro - Pascal interface for standard astronomy libraries
https://bugzilla.redhat.com/show_bug.cgi?id=1293049 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System --- libpasastro-1.0-5.20151222svn.fc23 has been pushed to the Fedora 23 testing repository. If problems still persist, please make note of it in this bug report. See https://fedoraproject.org/wiki/QA:Updates_Testing for instructions on how to install test updates. You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2016-f874adea82 -- 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 1288731] Review Request: os-autoinst - OS-level test automation
https://bugzilla.redhat.com/show_bug.cgi?id=1288731
Neal Gompa changed:
What|Removed |Added
CC||ngomp...@gmail.com
--- Comment #8 from Neal Gompa ---
Quick note, the %license compatibility macro is wrong. It should be
"%{!?_licensedir:%global license %doc}" so that the %doc part is actually
executed instead of being treated as a comment.
--
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 1278964] Review Request: distro-info - Provides information about releases of Debian and Ubuntu
https://bugzilla.redhat.com/show_bug.cgi?id=1278964
Neal Gompa changed:
What|Removed |Added
CC||ngomp...@gmail.com
--- Comment #5 from Neal Gompa ---
Michael,
I see an issue you should fix.
You're using unversioned python names and macros assuming that they'll remain
Python 2. We aren't supposed to assume that anymore, as we move toward Python
3. Accordingly, please use %{python2_sitelib}, prefix your python- packages as
python2- ones, and use python2 packages instead of python packages for Python 2
dependencies.
More details on the Python packaging guidelines here:
https://fedoraproject.org/wiki/Packaging:Python
--
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 1294577] Review Request: jmtpfs - FUSE and libmtp based filesystem for accessing MTP devices
https://bugzilla.redhat.com/show_bug.cgi?id=1294577 Christian Dersch changed: What|Removed |Added Status|NEW |ASSIGNED CC||lupi...@mailbox.org Assignee|nob...@fedoraproject.org|lupi...@mailbox.org Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1157255] Review Request: ufoai - UFO: Alien Invasion strategy game
https://bugzilla.redhat.com/show_bug.cgi?id=1157255
Neal Gompa changed:
What|Removed |Added
CC||ngomp...@gmail.com
Blocks||182235 (FE-Legal)
--- Comment #11 from Neal Gompa ---
Your ufoai-data package spec is missing the %{?dist} tag in the Release field
(like how the ufoai package spec has it).
@Karel, it seems your email got moderated out of Fedora Legal, as I don't see
it in the archives[0].
The Umefont license[1] appears to be an extremely permissive license, so I
don't expect it to be a problem, but that said, this font license question does
need to be answered, so I'm setting FE-Legal to block it.
[0]: https://lists.fedoraproject.org/pipermail/legal/
[1]: https://osdn.jp/projects/ume-font/wiki/FrontPage#h2-Licence
Referenced Bugs:
https://bugzilla.redhat.com/show_bug.cgi?id=182235
[Bug 182235] Fedora Legal Tracker
--
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 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 Robert Scheck changed: What|Removed |Added Assignee|redhat-bugzilla@linuxnetz.d |red...@linuxnetz.de |e | -- 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 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 Robert Scheck changed: What|Removed |Added Status|NEW |ASSIGNED Flags|fedora-review? |fedora-review+ --- Comment #3 from Robert Scheck --- Package looks good to me -> 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 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237
--- Comment #2 from Robert Scheck ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
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.
[x]: License file installed when any subpackage combination is installed.
[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.
[-]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 20480 bytes in 4 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %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]: 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]: 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:
[x]: 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.
[x]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[-]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
python2-supersmoother , python3-supersmoother
[?]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not include license text files separate from upstream.
[x]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[?]: Package should compile and build into binary rpms on all supported
architectures.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a wor
[Bug 1203961] Review Request: rubygem-sensu-em - Ruby/EventMachine library
https://bugzilla.redhat.com/show_bug.cgi?id=1203961 --- Comment #3 from Graeme Gillies --- I believe it's a sensu specific fork of the rubygem-eventmachine gem. This is needed to get sensu packaged and into fedora Regards, Graeme -- 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 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 Robert Scheck changed: What|Removed |Added Assignee|nob...@fedoraproject.org|redhat-bugzilla@linuxnetz.d ||e Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1199693] Review Request: pcp-pmda-cpp - C++ library for PCP PMDAs
https://bugzilla.redhat.com/show_bug.cgi?id=1199693 Nathan Scott changed: What|Removed |Added Flags|fedora-review+ |fedora-review? --- Comment #27 from Nathan Scott --- Hmm, OK, thanks Michael. I've reset that unusual flags setting, hopefully that gets some additional eyeballs on this one. -- 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 1292057] Review Request: perl-DBIx-QueryLog - Logging queries for DBI
https://bugzilla.redhat.com/show_bug.cgi?id=1292057 --- Comment #9 from Fedora Update System --- perl-DBIx-QueryLog-0.41-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1292061] Review Request: perl-POE-Loop-Event - Bridge that allows POE to be driven by Event.pm
https://bugzilla.redhat.com/show_bug.cgi?id=1292061 --- Comment #10 from Fedora Update System --- perl-POE-Loop-Event-1.305-1.el7 has been pushed to the Fedora EPEL 7 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1295237] Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 --- Comment #1 from Upstream Release Monitoring --- lupinix's scratch build of python-supersmoother-0.3.2-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12397829 -- 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 1294568] Review Request: libmtp11 - A software library for MTP media players
https://bugzilla.redhat.com/show_bug.cgi?id=1294568 --- Comment #8 from Fedora Update System --- libmtp11-1.1.10-1.el6 has been submitted as an update to Fedora EPEL 6. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-9e9fcfc8ae -- 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 1294568] Review Request: libmtp11 - A software library for MTP media players
https://bugzilla.redhat.com/show_bug.cgi?id=1294568 Fedora Update System 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 1294568] Review Request: libmtp11 - A software library for MTP media players
https://bugzilla.redhat.com/show_bug.cgi?id=1294568 --- Comment #7 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/libmtp11 -- 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 1295237] New: Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother
https://bugzilla.redhat.com/show_bug.cgi?id=1295237 Bug ID: 1295237 Summary: Review Request: python-supersmoother - Python implementation of Friedman's Supersmoother Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: lupi...@mailbox.org QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://lupinix.fedorapeople.org/review/python-supersmoother.spec SRPM URL: https://lupinix.fedorapeople.org/review/python-supersmoother-0.3.2-1.fc23.src.rpm Description: This is an efficient implementation of Friedman’s SuperSmoother based in Python. It makes use of numpy for fast numerical computation. Fedora Account System Username: lupinix Thank you for review in advance :) -- 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 969723] Review Request: tlsdate - Secure parasitic rdate replacement
https://bugzilla.redhat.com/show_bug.cgi?id=969723 --- Comment #19 from Fedora Update System --- tlsdate-0.0.13-1.fc22 has been pushed to the Fedora 22 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1279175] Review Request: innoextract - Tool to extract installers created by Inno Setup
https://bugzilla.redhat.com/show_bug.cgi?id=1279175 Alexandre Detiste changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2016-01-03 16:11:56 --- Comment #10 from Alexandre Detiste --- https://fedoraproject.org/wiki/Package_Review_Process #15 told me to do 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 1294568] Review Request: libmtp11 - A software library for MTP media players
https://bugzilla.redhat.com/show_bug.cgi?id=1294568 --- Comment #6 from Robert Scheck --- Antonio, thank you very much for the 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 969723] Review Request: tlsdate - Secure parasitic rdate replacement
https://bugzilla.redhat.com/show_bug.cgi?id=969723 --- Comment #18 from Fedora Update System --- tlsdate-0.0.13-1.fc23 has been pushed to the Fedora 23 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 969723] Review Request: tlsdate - Secure parasitic rdate replacement
https://bugzilla.redhat.com/show_bug.cgi?id=969723 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA Last Closed||2016-01-03 15:22:52 -- 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 1295231] New: Review Request: hawaii-system-preferences - Utilities to configure the Hawaii desktop environment
https://bugzilla.redhat.com/show_bug.cgi?id=1295231 Bug ID: 1295231 Summary: Review Request: hawaii-system-preferences - Utilities to configure the Hawaii desktop environment Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: pierluigi.fior...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-system-preferences.spec SRPM URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-system-preferences-0.5.0-1.fc23.src.rpm Description: This package contains configuration utilities for the Hawaii desktop, which allow to configure various settings including desktop fonts, keyboard and mouse properties, sound setup, desktop theme and background, user interface properties, screen resolution, and other settings. Fedora Account System Username: plfiorini -- 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 1295230] New: Review Request: hawaii-workspace - Hawaii workspace, applications and plugins
https://bugzilla.redhat.com/show_bug.cgi?id=1295230 Bug ID: 1295230 Summary: Review Request: hawaii-workspace - Hawaii workspace, applications and plugins Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: pierluigi.fior...@gmail.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-workspace.spec SRPM URL: https://fedorapeople.org/~plfiorini/srpms/hawaii-unstable/hawaii-workspace-0.5.0-1.fc23.src.rpm Description: Hawaii runtime components. Fedora Account System Username: plfiorini -- 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 1295069] Review Request: nodejs-readdir-scoped-modules - Like fs.readdir but handling @org/module dirs as if they were a single entry
https://bugzilla.redhat.com/show_bug.cgi?id=1295069 Tom Hughes changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2016-01-03 14:47:54 -- 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 1295069] Review Request: nodejs-readdir-scoped-modules - Like fs.readdir but handling @org/module dirs as if they were a single entry
https://bugzilla.redhat.com/show_bug.cgi?id=1295069 --- Comment #4 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-readdir-scoped-modules -- 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 1295222] Review Request: kf5-libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map
https://bugzilla.redhat.com/show_bug.cgi?id=1295222 --- Comment #1 from Upstream Release Monitoring --- rdieter's scratch build of kf5-libkgeomap-15.12.0-1.fc23.src.rpm for rawhide completed http://koji.fedoraproject.org/koji/taskinfo?taskID=12396660 -- 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 1295222] Review Request: kf5-libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map
https://bugzilla.redhat.com/show_bug.cgi?id=1295222 Rex Dieter changed: What|Removed |Added Blocks||656997 (kde-reviews), ||1294838 (digikam-5.0) Alias||kf5-libkgeomap Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=656997 [Bug 656997] kde-related package review tracker https://bugzilla.redhat.com/show_bug.cgi?id=1294838 [Bug 1294838] digikam-5.0 tracker -- 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 1295222] New: Review Request: kf5-libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map
https://bugzilla.redhat.com/show_bug.cgi?id=1295222 Bug ID: 1295222 Summary: Review Request: kf5-libkgeomap - A wrapper around different world-map components, to browse and arrange photos over a map Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: rdie...@math.unl.edu QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://rdieter.fedorapeople.org/rpms/kf5/kf5-libkgeomap.spec SRPM URL: https://rdieter.fedorapeople.org/rpms/kf5/kf5-libkgeomap-15.12.0-1.fc23.src.rpm Description: A wrapper around different world-map components, to browse and arrange photos over a map Fedora Account System Username: rdieter -- 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 1294587] Review Request: erlang-p1_utils - Erlang utility modules from ProcessOne
https://bugzilla.redhat.com/show_bug.cgi?id=1294587 Randy Barlow changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE Last Closed||2016-01-03 13:16:20 --- Comment #4 from Randy Barlow --- erlang-p1_utils-1.0.2-1 is now built for Rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=12396373 I also made a follow up release, erlang-p1_utils-1.0.2-2 that addresses the unneeded install -d that you pointed out: https://koji.fedoraproject.org/koji/taskinfo?taskID=12396612 I also filed a pull request to address something I had been fixing in the spec file: https://github.com/processone/p1_utils/pull/4 -- 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 1295075] Review Request: erlang-cache_tab - Erlang cache table application
https://bugzilla.redhat.com/show_bug.cgi?id=1295075 Bug 1295075 depends on bug 1294587, which changed state. Bug 1294587 Summary: Review Request: erlang-p1_utils - Erlang utility modules from ProcessOne https://bugzilla.redhat.com/show_bug.cgi?id=1294587 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- 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 1295069] Review Request: nodejs-readdir-scoped-modules - Like fs.readdir but handling @org/module dirs as if they were a single entry
https://bugzilla.redhat.com/show_bug.cgi?id=1295069 Piotr Popieluch changed: What|Removed |Added Flags||fedora-review+ --- Comment #3 from Piotr Popieluch --- all fine. 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 1295069] Review Request: nodejs-readdir-scoped-modules - Like fs.readdir but handling @org/module dirs as if they were a single entry
https://bugzilla.redhat.com/show_bug.cgi?id=1295069
Piotr Popieluch changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||piotr1...@gmail.com
Assignee|nob...@fedoraproject.org|piotr1...@gmail.com
--- Comment #2 from Piotr Popieluch ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
= MUST items =
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: "ISC", "Unknown or generated". 5 files have unknown license.
Detailed output of licensecheck in /home/piotr/rpmbuild/1295069
-nodejs-readdir-scoped-modules/licensecheck.txt
[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.
[-]: 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]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 10240 bytes in 1 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package is included in %license.
[x]: Package requires other packages for directories it uses.
[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]: 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]: Sources used to build the package match the upstream source, as
provided in the spec URL.
[x]: Spec file name must match the spec package %{name}, in the format
%{name}.spec.
[x]: File names are valid UTF-8.
[x]: Packages must not store files under /srv, /opt or /usr/local
= SHOULD items =
Generic:
[-]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[?]: Package functions as described.
[x]: Latest version is packaged.
[-]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x]: %check is present and all tests pass.
[x]: Packages should try to preserve timestamps of original installed
files.
[x]: Packager, Vendor, PreReq, Copyright tags should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]: Package has no %clean section with rm -rf %{buildroot} (or
$RPM_BUILD_ROOT)
[x]: No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x]: SourceX is a working URL.
[x]: Package should compile and build into binary rpms on all supported
architectures.
[x]: Spec use %g
[Bug 1293058] Review Request: nodejs-fs2 - Node.js fs (file system package) extensions
https://bugzilla.redhat.com/show_bug.cgi?id=1293058 --- Comment #8 from Fedora Update System --- nodejs-fs2-0.2.3-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-47eee51d4e -- 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 1295217] Review Request: msgpuck - a MsgPack serialization library in a self-contained header file
https://bugzilla.redhat.com/show_bug.cgi?id=1295217 Roman Tsisyk 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 1295217] New: Review Request: msgpuck - a MsgPack serialization library in a self-contained header file
https://bugzilla.redhat.com/show_bug.cgi?id=1295217 Bug ID: 1295217 Summary: Review Request: msgpuck - a MsgPack serialization library in a self-contained header file Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: ro...@tsisyk.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://gist.githubusercontent.com/rtsisyk/5bbce59a5a9954df8039/raw/613774f07a5e12093989ebe2ff511d46ffc5ae42/msgpuck-1.0.1.spec SRPM URL: https://gist.github.com/rtsisyk/5bbce59a5a9954df8039/raw/613774f07a5e12093989ebe2ff511d46ffc5ae42/msgpuck-1.0.1-1.fc24.src.rpm Description: MsgPuck is a simple and efficient MsgPack binary serialization library in a self-contained header file. MsgPack is an efficient binary serialization format. It's like JSON. but fast and small. Fedora Account System Username: rtsisyk Buildbot: http://koji.fedoraproject.org/koji/taskinfo?taskID=12396403 This library is header-only. However, I provided a static library for cases when compiler can't inline some of header functions. This package is a spin-off from tarantool (http://tarantool.org). I plan to rebase tarantool package (https://bugzilla.redhat.com/show_bug.cgi?id=1293100) on this library. -- 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 1293058] Review Request: nodejs-fs2 - Node.js fs (file system package) extensions
https://bugzilla.redhat.com/show_bug.cgi?id=1293058 --- Comment #7 from Fedora Update System --- nodejs-fs2-0.2.3-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-96ea5115f4 -- 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 1293058] Review Request: nodejs-fs2 - Node.js fs (file system package) extensions
https://bugzilla.redhat.com/show_bug.cgi?id=1293058 Fedora Update System 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 1293058] Review Request: nodejs-fs2 - Node.js fs (file system package) extensions
https://bugzilla.redhat.com/show_bug.cgi?id=1293058 --- Comment #6 from Fedora Update System --- nodejs-fs2-0.2.3-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-fa251adeb2 -- 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 1294839] Review Request: kf5-libkipi - Common plugin infrastructure for KDE image applications
https://bugzilla.redhat.com/show_bug.cgi?id=1294839 Rex Dieter changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |RAWHIDE Last Closed||2016-01-03 12:21:51 --- Comment #9 from Rex Dieter --- Imported into rawhide, thanks. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1294277] Review Request: python-vxi11 - Python implementation of the VXI-11 protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294277 --- Comment #4 from Fedora Update System --- python-vxi11-0.8-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-c9ca26819f -- 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 1294277] Review Request: python-vxi11 - Python implementation of the VXI-11 protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294277 --- Comment #3 from Fedora Update System --- python-vxi11-0.8-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-5c59b40903 -- 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 1294277] Review Request: python-vxi11 - Python implementation of the VXI-11 protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294277 Fedora Update System changed: What|Removed |Added Status|NEW |MODIFIED --- Comment #3 from Fedora Update System --- python-vxi11-0.8-1.el7 has been submitted as an update to Fedora EPEL 7. https://bodhi.fedoraproject.org/updates/FEDORA-EPEL-2016-5c59b40903 -- 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 1294277] Review Request: python-vxi11 - Python implementation of the VXI-11 protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294277 --- Comment #4 from Fedora Update System --- python-vxi11-0.8-1.fc23 has been submitted as an update to Fedora 23. https://bodhi.fedoraproject.org/updates/FEDORA-2016-c9ca26819f --- Comment #5 from Fedora Update System --- python-vxi11-0.8-1.fc22 has been submitted as an update to Fedora 22. https://bodhi.fedoraproject.org/updates/FEDORA-2016-554031f6a8 -- 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 1294277] Review Request: python-vxi11 - Python implementation of the VXI-11 protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1294277 --- Comment #2 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/python-vxi11 -- 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 1295209] New: Review Request: lua-fun - functional programming library for Lua and LuaJIT
https://bugzilla.redhat.com/show_bug.cgi?id=1295209 Bug ID: 1295209 Summary: Review Request: lua-fun - functional programming library for Lua and LuaJIT Product: Fedora Version: rawhide Component: Package Review Severity: medium Assignee: nob...@fedoraproject.org Reporter: ro...@tsisyk.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://gist.githubusercontent.com/rtsisyk/e3b7d7d5574096aa22e2/raw/8b30fb45845b2e223b3c17676bbdd95f682c0818/luafun-0.1.2.spec SRPM URL: https://gist.github.com/rtsisyk/e3b7d7d5574096aa22e2/raw/8b30fb45845b2e223b3c17676bbdd95f682c0818/lua-fun-0.1.2-1.fc24.src.rpm Description: Lua Fun is a high-performance functional programming library designed for LuaJIT tracing just-in-time compiler. The library provides a set of more than 50 programming primitives typically found in languages like Standard ML, Haskell, Erlang, JavaScript, Python and even Lisp. High-order functions such as map, filter, reduce, zip will help you to write simple and efficient functional code. Fedora Account System Username: rtsisyk Buildbot: http://koji.fedoraproject.org/koji/taskinfo?taskID=12394857 This .src.rpm provides packages both for lua-5.3 and luajit-2.0+. -- 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 1293058] Review Request: nodejs-fs2 - Node.js fs (file system package) extensions
https://bugzilla.redhat.com/show_bug.cgi?id=1293058 --- Comment #5 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/nodejs-fs2 -- 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 1294587] Review Request: erlang-p1_utils - Erlang utility modules from ProcessOne
https://bugzilla.redhat.com/show_bug.cgi?id=1294587 --- Comment #3 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/erlang-p1_utils -- 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 1279175] Review Request: innoextract - Tool to extract installers created by Inno Setup
https://bugzilla.redhat.com/show_bug.cgi?id=1279175 --- Comment #9 from Jon Ciesla --- Package request has been approved: https://admin.fedoraproject.org/pkgdb/package/innoextract -- 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 1295138] Review Request: marsshooter - M.A.R.S. - A ridiculous shooter
https://bugzilla.redhat.com/show_bug.cgi?id=1295138 --- Comment #3 from Antonio Trande --- I have just seen that this package is already in Fedora: https://admin.fedoraproject.org/pkgdb/package/rpms/marsshooter/ -- 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 1294568] Review Request: libmtp11 - A software library for MTP media players
https://bugzilla.redhat.com/show_bug.cgi?id=1294568
Antonio Trande changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #5 from Antonio Trande ---
(In reply to Robert Scheck from comment #4)
> (In reply to Antonio Trande from comment #3)
> > Full RELRO and PIE (http://fedoraproject.org/wiki/Packaging:Guidelines#PIE)
> > issue are part of packaging guidelines, I don't understand why EPEL
> > packagers should choice what they follow or not.
> > Also, we are talking of security issues important for EPEL too, or not?
>
> RHEL 6 doesn't support the %_hardened_build macro. Additionally, even the
> base operating system dependencies of libmtp do not have full relro - why
> would it make sense for a single relatively unimportant library on top? I
> also looked to other EPEL packages...this is usually not manually done. So
> why is this here expected? I get "security", but the impact is relatively
> low from my point of view (libmtp is not used by a network daemon AFAIK).
>
> Check for full relro for libmtp dependencies:
> readelf -l /usr/lib*/libusb-0.1.so.4 | grep -c GNU_RELRO
> readelf -d /usr/lib*/libusb-0.1.so.4 | grep -c BIND_NOW
> readelf -l /lib*/libgcrypt.so.11 | grep -c GNU_RELRO
> readelf -d /lib*/libgcrypt.so.11 | grep -c BIND_NOW
>
> If you still insist on full relro for EPEL 6 for libmtp11, let me know and I
> will add: export LDFLAGS="-Wl,-z,now -Wl,-z,relro %{?__global_ldflags}"
You're right and I don't know if it's (or will be) a topic of discussion sooner
or later.
I have prefered to set manually hardening flags on all my packages in EPEL in
advance; of course I can't force you to do that.
Anyway I drew attention in the package reviews I done.
Package approved.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1295154] Review Request: python-prompt_toolkit - Library for building powerful interactive command lines in Python
https://bugzilla.redhat.com/show_bug.cgi?id=1295154
--- Comment #2 from Terje Røsten ---
Blockers:
1) Needs wcwidth which is not part for Fedora yet.
2) Why is LICENSE set to LICENSE.txt?
Pick the valid and correct license from:
https://fedoraproject.org/wiki/Licensing:Main
3) Fix rpmlint warning:
python-prompt_toolkit.src:60: W: setup-not-quiet
4) Include AUTHORS.rst, CHANGELOG & TODO.rst in packages.
Nice to fix:
1) Include the examples in -example subpackage.
2) Ask upstream to include license info in each source file.
3) No need for empty lines between Summary/ License, BuildArch&BuildReq.
Package Review
==
Legend:[X] = Pass, [!] = Fail, [-] = Not applicable
= MUST items =
Generic:
[X]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[!]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses
found: "BSD (3 clause)", "Unknown or generated". 128 files have
unknown license.
[X]: License file installed when any subpackage combination is installed.
[X]: Package does not own files or directories owned by other packages.
[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]: Package is not known to require an ExcludeArch tag.
[!]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 20480 bytes in 2 files.
[X]: Package complies to the Packaging Guidelines
[X]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[X]: 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]: 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]: Permissions on files are set properly.
[-]: 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:
[X]: Python eggs must not download any dependencies during the build
process.
[X]: A package which is used by another package via an egg interface should
provide egg info.
[X]: Package meets the Packaging Guidelines::Python
[X]: Package contains BR: python2-devel or python3-devel
[X]: Binary eggs must be removed in %prep
= SHOULD items =
Generic:
[X]: If the source package does not include license text(s) as a separate
file from upstream, the packager SHOULD query upstream to include it.
[X]: Final provides and requires are sane (see attachments).
[-]: Fully versioned dependency in subpackages if applicable.
Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
python2-prompt_toolkit , python3-prompt_toolkit
[X]: Package functions as described.
[-]: Latest version is packaged.
[X]: Package does not include license text files separate from upstream.
[-]: Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[X]: P
[Bug 1295126] Review Request: python-guzzle_sphinx_theme - Sphinx theme used by Guzzle
https://bugzilla.redhat.com/show_bug.cgi?id=1295126
--- Comment #1 from Antonio Trande ---
- Move Python3 BuildRequires packages inside python3-%{pypi_name}
- %doc
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/ReadMe.html
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/README.md
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/SourceSerifProReadMe.html
README.rst
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/LICENSE.txt
Better:
%doc
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/*[ReadMe,README].*
%license
guzzle_sphinx_theme/guzzle_sphinx_theme/static/fonts/source-serif-pro/LICENSE.txt
- This package provides fonts released with OFL license.
Please, update License.
- Fix rpmlint errors and warnings.
- Python macro difinitions is necessary only in EPEL6
https://fedoraproject.org/wiki/EPEL:Packaging#Python
Probably in future you will be able to build this package
in EPEL7 too.
https://fedorahosted.org/fpc/ticket/567
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- 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.
Note: License file LICENSE.txt is marked as %doc instead of %license
See:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#License_Text
= MUST items =
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: "MIT/X11 (BSD like)", "Unknown or generated". 104 files have
unknown license. Detailed output of licensecheck in
/home/sagitter/1295126-python-guzzle_sphinx_theme/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package requires other packages for directories it uses.
Note: No known owner of /usr/lib/python3.5/site-packages,
/usr/lib/python3.5
[x]: Package must own all directories that it creates.
Note: Directories without known owners: /usr/lib/python3.5/site-
packages, /usr/lib/python3.5
[!]: Package contains no bundled libraries without FPC exception.
This package provides a little font file already packaged in Fedora
(glyphicons-halflings-fonts): glyphicons-halflings-regular.ttf
[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.
[!]: Requires correct, justified where necessary.
See above.
[!]: Spec file is legible and written in American English.
See my notes.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 81920 bytes in 10 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least
one supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[x]: 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]: 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]: 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 na
