[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Blocks||1118648 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1118648 [Bug 1118648] perl-Inline-0.58 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1104746] Review Request: soscleaner - sosreport data obfuscation
https://bugzilla.redhat.com/show_bug.cgi?id=1104746 Petr Pisar ppi...@redhat.com changed: What|Removed |Added CC||ppi...@redhat.com --- Comment #20 from Petr Pisar ppi...@redhat.com --- Guys, please follow the Fedora review processes https://fedoraproject.org/wiki/Package_Review_Process. This review is neither in ASSIGNED state, neither has fedora-review+. So there is no point in requesting fedora‑cvs? (And I do not comment reassigning the review to cvs component.) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||ppi...@redhat.com Assignee|nob...@fedoraproject.org|ppi...@redhat.com Flags||fedora-review? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 --- Comment #1 from Petr Pisar ppi...@redhat.com --- URL and Source0 are usable. Ok. Source tar ball is original (SHA-256: f8593750d45d1c068195b4f16643fe4b3d50956e79d26c3c8aa7ba887ae75a45). Ok. Summary verified from README. Ok. Description verified from README. Ok. FATAL: lib/Inline/C/Cookbook.pod license is Artistic: =head1 COPYRIGHT Copyright 2000-2014. Ingy döt Net. Copyright 2008, 2010-2014. Sisyphus. See Lhttp://www.perl.com/perl/misc/Artistic.html The URL points to The Artistic License license which is not allowed by Fedora. Please remove the file from source archive, or clarify the license with the upstream and Fedora legal department. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120173] Review Request: perl-Devel-ArgNames - Figure out the names of variables passed into subroutines
https://bugzilla.redhat.com/show_bug.cgi?id=1120173 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Petr Pisar ppi...@redhat.com --- New Package SCM Request === Package Name: perl-Devel-ArgNames Short Description: Figure out the names of variables passed into subroutines Upstream URL: http://search.cpan.org/dist/Devel-ArgNames/ Owners: ppisar jplesnik psabata Branches: f21 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120543] Review Request: php-ocramius-lazy-map - Lazy instantiation logic for a map of objects
https://bugzilla.redhat.com/show_bug.cgi?id=1120543 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Alias||ocramius/lazy-map --- Comment #1 from Remi Collet fed...@famillecollet.com --- As usually, I will clean EL-5 stuff after import. This is a new dependency of PHPUnit. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120543] New: Review Request: php-ocramius-lazy-map - Lazy instantiation logic for a map of objects
https://bugzilla.redhat.com/show_bug.cgi?id=1120543 Bug ID: 1120543 Summary: Review Request: php-ocramius-lazy-map - Lazy instantiation logic for a map of objects Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@famillecollet.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/remicollet/remirepo/bcd0854e4715fdcdf614fde0cae5d16d29023f97/php/php-ocramius-lazy-map/php-ocramius-lazy-map.spec SRPM URL: http://rpms.famillecollet.com/SRPMS/php-ocramius-lazy-map-1.0.0-1.remi.src.rpm Description: This small library aims at providing a very simple and efficient map of lazy-instantiating objects. Fedora Account System Username: remi -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120552] New: Review Request: ghc-websockets - A sensible and clean way to write WebSocket-capable servers in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1120552 Bug ID: 1120552 Summary: Review Request: ghc-websockets - A sensible and clean way to write WebSocket-capable servers in Haskell Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: peter...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://petersen.fedorapeople.org/reviews/ghc-websockets.spec SRPM URL: http://petersen.fedorapeople.org/reviews/ghc-websockets-0.8.2.6-1.fc20.src.rpm Description: This library allows you to write WebSocket-capable servers. See: * The specification of the WebSocket protocol: http://www.whatwg.org/specs/web-socket-protocol/ * The JavaScript API for dealing with WebSockets: http://www.w3.org/TR/websockets/. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120552] Review Request: ghc-websockets - A sensible and clean way to write WebSocket-capable servers in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1120552 --- Comment #1 from Jens Petersen peter...@redhat.com --- This package built on koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=7155713 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120552] Review Request: ghc-websockets - A sensible and clean way to write WebSocket-capable servers in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1120552 --- Comment #2 from Jens Petersen peter...@redhat.com --- http://packdeps.haskellers.com/reverse/websockets -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 --- Comment #3 from David Dick dd...@cpan.org --- (In reply to Petr Pisar from comment #2) TODO: I recommend to package the LICENSE file into duckduckgo sub-package too. Because is does not run-require a specific version of the main package, it can happen an user gets installed different versions of the the sub-packages which can differ in the license. (If upstream decides to change the license). Done. FIX: Do not build-require `perl(Test::Pod)' as the module is never used (t/release-pod-syntax.t:4). Removed. TODO: The /usr/bin/env scripts interpreter is considered harmful by some packageres. It would be nice to replace it with `#!%{_perl}' (or `#!perl' which ExtUtils::MakeMaker fixes automatically). Fixed with a sed script in %setup Spec and SRPM have been uploaded again. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120563] New: Review Request: php-ocramius-instantiator - Instantiate objects in PHP without invoking their constructors
https://bugzilla.redhat.com/show_bug.cgi?id=1120563 Bug ID: 1120563 Summary: Review Request: php-ocramius-instantiator - Instantiate objects in PHP without invoking their constructors Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: fed...@famillecollet.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: https://raw.githubusercontent.com/remicollet/remirepo/569276dfb2a8905e141dd58a00f86346e7b4a4cc/php/php-ocramius-instantiator/php-ocramius-instantiator.spec SRPM URL: http://rpms.famillecollet.com/SRPMS/php-ocramius-instantiator-1.0.0-1.remi.src.rpm Description: This library provides a way of avoiding usage of constructors when instantiating PHP classes. Fedora Account System Username: remi -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120543] Review Request: php-ocramius-lazy-map - Lazy instantiation logic for a map of objects
https://bugzilla.redhat.com/show_bug.cgi?id=1120543 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Blocks||1120563 ||(ocramius/instantiator) Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1120563 [Bug 1120563] Review Request: php-ocramius-instantiator - Instantiate objects in PHP without invoking their constructors -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120563] Review Request: php-ocramius-instantiator - Instantiate objects in PHP without invoking their constructors
https://bugzilla.redhat.com/show_bug.cgi?id=1120563 Remi Collet fed...@famillecollet.com changed: What|Removed |Added Depends On||1120543 (ocramius/lazy-map) Alias||ocramius/instantiator --- Comment #1 from Remi Collet fed...@famillecollet.com --- As usually, I will clean EL-5 stuff after import. This is a new dependency of PHPUnit. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1120543 [Bug 1120543] Review Request: php-ocramius-lazy-map - Lazy instantiation logic for a map of objects -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 --- Comment #4 from David Dick dd...@cpan.org --- s/%setup/%prep/ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 804125] Review Request: rdkit - A toolkit for cheminformatics and machine learning
https://bugzilla.redhat.com/show_bug.cgi?id=804125 Christopher Meng i...@cicku.me changed: What|Removed |Added Blocks||100 (ML-SIG) --- Comment #33 from Christopher Meng i...@cicku.me --- Please patch all /usr/bin/env python to %{__python2}. Please try to fix the W: unused-direct-shlib-dependency issues, you can take a look at the common rpmlint page. Please add the ?_isa bits macro for non-noarch subpackages: http://fedoraproject.org/wiki/Packaging:Guidelines#Requiring_Base_Package Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=100 [Bug 100] Machine Learning SIG - review 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 1118960] Review Request: perl-Clone-PP - Recursively copy Perl data-types
https://bugzilla.redhat.com/show_bug.cgi?id=1118960 David Dick dd...@cpan.org changed: What|Removed |Added Blocks||182235 (FE-Legal) --- Comment #3 from David Dick dd...@cpan.org --- Interesting. I'll block this as FE-Legal to clarify if it is permissible for parts of the test suite to be under the Artistic 1.0 while the rest of the package is under the GPL+ or Artistic, with the Artistic 1.0 files not being distributed in the resulting RPM. 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 1118960] Review Request: perl-Clone-PP - Recursively copy Perl data-types
https://bugzilla.redhat.com/show_bug.cgi?id=1118960 --- Comment #4 from Petr Pisar ppi...@redhat.com --- Don't forget Fedora distributes source RPM too. And the dist-git source cache is publicly available too. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1061056] Review Request: cockpit - A user interface for Linux servers
https://bugzilla.redhat.com/show_bug.cgi?id=1061056 --- Comment #25 from Fedora Update System upda...@fedoraproject.org --- cockpit-0.16-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/cockpit-0.16-1.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Blocks||182235 (FE-Legal) 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 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #5 from Petr Pisar ppi...@redhat.com --- Spec file changes: $ diff -u perl-App-DuckDuckGo.spec{.old,} --- perl-App-DuckDuckGo.spec.old2014-07-10 08:49:22.0 +0200 +++ perl-App-DuckDuckGo.spec2014-07-17 09:18:39.0 +0200 @@ -13,7 +13,6 @@ BuildRequires: perl(MooseX::Getopt) = 0.35 BuildRequires: perl(strict) BuildRequires: perl(Test::More) = 0.90 -BuildRequires: perl(Test::Pod) BuildRequires: perl(WWW::DuckDuckGo) = 0.004 BuildRequires: perl(warnings) Requires: perl(Moose) = 1.24 @@ -31,6 +30,7 @@ %prep %setup -q -n App-DuckDuckGo-%{version} +sed -i 's|#!/usr/bin/env perl|#!%{__perl}|' bin/duckduckgo %build %{__perl} Makefile.PL INSTALLDIRS=vendor @@ -60,7 +60,7 @@ shell script to automatically work with the DuckDuckGo API results. %files -n duckduckgo -%doc README +%doc LICENSE README %{_mandir}/man1/* %{_bindir}/* TODO: I recommend to package the LICENSE file into duckduckgo sub-package too. Because is does not run-require a specific version of the main package, it can happen an user gets installed different versions of the the sub-packages which can differ in the license. (If upstream decides to change the license). %files -n duckduckgo -%doc README +%doc LICENSE README Ok. FIX: Do not build-require `perl(Test::Pod)' as the module is never used (t/release-pod-syntax.t:4). -BuildRequires: perl(Test::Pod) Ok. TODO: The /usr/bin/env scripts interpreter is considered harmful by some packageres. It would be nice to replace it with `#!%{_perl}' (or `#!perl' which ExtUtils::MakeMaker fixes automatically). +sed -i 's|#!/usr/bin/env perl|#!%{__perl}|' bin/duckduckgo Ok. $ rpmlint perl-App-DuckDuckGo.spec ../SRPMS/perl-App-DuckDuckGo-0.008-1.fc22.src.rpm ../RPMS/noarch/perl-App-DuckDuckGo-0.008-1.fc22.noarch.rpm ../RPMS/noarch/duckduckgo-0.008-1.fc22.noarch.rpm duckduckgo.noarch: W: name-repeated-in-summary C DuckDuckGo 3 packages and 1 specfiles checked; 0 errors, 1 warnings. rpmlint is Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-App-DuckDuckGo-0.008-1.fc22.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.18.2) 1 perl(Moose) = 1.24 1 perl(MooseX::Getopt) = 0.35 1 perl(WWW::DuckDuckGo) = 0.004 1 rpmlib(CompressedFileNames) = 3.0.4-1 1 rpmlib(FileDigests) = 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) = 4.0-1 1 rpmlib(PayloadIsXz) = 5.2-1 $ rpm -q --requires -p ../RPMS/noarch/duckduckgo-0.008-1.fc22.noarch.rpm | sort -f | uniq -c 1 /usr/bin/perl 1 perl(App::DuckDuckGo) 1 rpmlib(CompressedFileNames) = 3.0.4-1 1 rpmlib(FileDigests) = 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) = 4.0-1 1 rpmlib(PayloadIsXz) = 5.2-1 Binary requires are Ok. $ resolvedeps rawhide ../RPMS/noarch/perl-App-DuckDuckGo-0.008-1.fc22.noarch.rpm ../RPMS/noarch/duckduckgo-0.008-1.fc22.noarch.rpm Binary dependencies resolvable. Ok. Package builds in F22 (http://koji.fedoraproject.org/koji/taskinfo?taskID=7155754). Ok. Resolution: 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 1120366] Review Request: nodejs-set-immediate - A shim for the setImmediate API
https://bugzilla.redhat.com/show_bug.cgi?id=1120366 --- Comment #2 from Tom Hughes t...@compton.nu --- I've rebuilt the srpm with the updated spec: Spec URL: http://download.compton.nu/nodejs/nodejs-set-immediate.spec SRPM URL: http://download.compton.nu/nodejs/nodejs-set-immediate-0.1.1-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 David Dick dd...@cpan.org changed: What|Removed |Added Flags||fedora-cvs? --- Comment #6 from David Dick dd...@cpan.org --- Thanks for the review Petr! New Package SCM Request === Package Name: perl-App-DuckDuckGo Short Description: Application to query DuckDuckGo Upstream URL: http://search.cpan.org/dist/App-DuckDuckGo/ Owners: ddick Branches: f20, f21 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1114696] Review Request: repo_manager - Manage your RPM repositories easily
https://bugzilla.redhat.com/show_bug.cgi?id=1114696 --- Comment #6 from Pierre-YvesChibon pin...@pingoured.fr --- * Thu Jul 17 2014 Pierre-Yves Chibon pin...@pingoured.fr - 0.1.0-3 - Drop shebang in repo_manager/__init__.py New spec: http://pingou.fedorapeople.org/RPMs/repo_manager.spec New srpm: http://pingou.fedorapeople.org/RPMs/repo_manager-0.1.0-2.fc20.src.rpm 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 1114696] Review Request: repo_manager - Manage your RPM repositories easily
https://bugzilla.redhat.com/show_bug.cgi?id=1114696 --- Comment #7 from Pierre-YvesChibon pin...@pingoured.fr --- I used the wrong url for the srpm (fixed here): New spec: http://pingou.fedorapeople.org/RPMs/repo_manager.spec New srpm: http://pingou.fedorapeople.org/RPMs/repo_manager-0.1.0-3.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Šabata psab...@redhat.com changed: What|Removed |Added Blocks|182235 (FE-Legal) | --- Comment #2 from Petr Šabata psab...@redhat.com --- I removed the offending file from the distribution for now. 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 1115894] Review Request: ghc-hgettext - Bindings to libintl.h (gettext, bindtextdomain)
https://bugzilla.redhat.com/show_bug.cgi?id=1115894 Philip Withnall phi...@tecnocode.co.uk changed: What|Removed |Added Flags||fedora-cvs? --- Comment #7 from Philip Withnall phi...@tecnocode.co.uk --- New Package SCM Request === Package Name: ghc-hgettext Short Description: Bindings to libintl.h (gettext, bindtextdomain). Upstream URL: https://hackage.haskell.org/package/hgettext Owners: pwithnall Branches: f19 f20 f21 InitialCC: haskell-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1115667] Review Request: nodejs-gonzales-pe - Gonzales Preprocessor Edition (fast CSS parser)
https://bugzilla.redhat.com/show_bug.cgi?id=1115667 --- Comment #3 from Parag AN(पराग) panem...@gmail.com --- Review: + mock build is successful for f22 - rpmlint output on rpms gave nodejs-gonzales-pe.src: W: spelling-error %description -l en_US preprocessors - processors, predecessors, microprocessor nodejs-gonzales-pe.x86_64: W: spelling-error %description -l en_US preprocessors - processors, predecessors, microprocessor nodejs-gonzales-pe.x86_64: E: no-binary nodejs-gonzales-pe.x86_64: W: only-non-binary-in-usr-lib nodejs-gonzales-pe-debuginfo.x86_64: E: empty-debuginfo-package 3 packages and 0 specfiles checked; 2 errors, 3 warnings. == See below suggestions + Source verified with upstream as (sha256sum) srpm tarball :4b6bd26612b5dd221696f9a638ed6a8af950c98664400fe3e41b6c040928cf47 upstream tarball :4b6bd26612b5dd221696f9a638ed6a8af950c98664400fe3e41b6c040928cf47 + License MIT is valid - License file is not packaged. + rest looks following packaging guidelines Suggestions: 1) Add following to spec BuildArch: noarch %if 0%{?fedora} = 19 ExclusiveArch: %{nodejs_arches} noarch %else ExclusiveArch: %{ix86} x86_64 %{arm} noarch %endif 2) Add doc directory files and MIT-LICENSE.txt as %doc Please provide updated package -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1115707] Review Request: nodejs-vow-queue - Vow-based task queue
https://bugzilla.redhat.com/show_bug.cgi?id=1115707 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? --- Comment #1 from Parag AN(पराग) panem...@gmail.com --- Review: + mock build is successful for f22 - rpmlint output on rpms gave nodejs-vow-queue.x86_64: E: no-binary nodejs-vow-queue.x86_64: W: only-non-binary-in-usr-lib nodejs-vow-queue.x86_64: W: dangling-symlink /usr/lib/node_modules/vow-queue/node_modules/vow /usr/lib/node_modules/vow nodejs-vow-queue-debuginfo.x86_64: E: empty-debuginfo-package 3 packages and 0 specfiles checked; 2 errors, 2 warnings. == See below suggestion for debuginfo + Source verified with upstream as (sha256sum) srpm tarball :2104d077ef9b98a7f532c7174d931d34c33ad6c33e08858e164891e7cca189cd upstream tarball :2104d077ef9b98a7f532c7174d931d34c33ad6c33e08858e164891e7cca189cd - License tag is not complete. suggestions: 1) As per source file License is dual. That is license tag should be MIT and GPLv3+. Also, please clarify upstream this and ask them to provide which GPL version they really intend this code. 2) Add following to spec BuildArch: noarch %if 0%{?fedora} = 19 ExclusiveArch: %{nodejs_arches} noarch %else ExclusiveArch: %{ix86} x86_64 %{arm} noarch %endif -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 --- Comment #11 from Yanko Kaneti yan...@declera.com --- Do you have a usecase for libmediaart outside tracker? I'd rather not care about libemediaart in f20 for another year. If you happen to be fedora packager you could apply for co-maintainer of the f20 branch in pkgdb. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 --- Comment #12 from Konstantin Svist fry@gmail.com --- latest Rygel requires it (https://wiki.gnome.org/Projects/Rygel) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 --- Comment #3 from Petr Pisar ppi...@redhat.com --- You need to remove the file from Inline-C-0.60.tar.gz archive. Deleting in the SPEC file is too late. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #12 from Jiri Kastner jkast...@redhat.com --- uploaded scratch build to xorg-x11-drv-opentegra repo: http://repos.fedorapeople.org/repos/jkastner/xorg-x11-drv-opentegra/ so wget -P /etc/yum.repos.d/ http://repos.fedorapeople.org/repos/jkastner/xorg-x11-drv-opentegra/fedora-xorg-x11-drv-opentegra.repo; should work -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1062686] Review Request: libmediaart - Manage and cache media art
https://bugzilla.redhat.com/show_bug.cgi?id=1062686 --- Comment #13 from Yanko Kaneti yan...@declera.com --- Oh well.. https://admin.fedoraproject.org/updates/libmediaart-0.4.0-1.fc20 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1118960] Review Request: perl-Clone-PP - Recursively copy Perl data-types
https://bugzilla.redhat.com/show_bug.cgi?id=1118960 --- Comment #5 from David Dick dd...@cpan.org --- Yup. I've also emailed all the authors of the files in question to see if i can get a license change for the files in question. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120366] Review Request: nodejs-set-immediate - A shim for the setImmediate API
https://bugzilla.redhat.com/show_bug.cgi?id=1120366 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #3 from Parag AN(पराग) panem...@gmail.com --- Review: + mock build is successful for f22 + rpmlint output on rpms gave nodejs-set-immediate.noarch: W: spelling-error Summary(en_US) setImmediate - set Immediate, set-immediate, immediateness nodejs-set-immediate.noarch: W: spelling-error %description -l en_US setImmediate - set Immediate, set-immediate, immediateness nodejs-set-immediate.noarch: W: only-non-binary-in-usr-lib nodejs-set-immediate.src: W: spelling-error Summary(en_US) setImmediate - set Immediate, set-immediate, immediateness nodejs-set-immediate.src: W: spelling-error %description -l en_US setImmediate - set Immediate, set-immediate, immediateness 2 packages and 0 specfiles checked; 0 errors, 5 warnings. == Not a issue + Source verified with upstream as (sha256sum) srpm tarball :f30ca71c7b81a9d32163e09cd1aa40d83f7772c6de9f416d3a9d374419dda4d2 upstream tarball :f30ca71c7b81a9d32163e09cd1aa40d83f7772c6de9f416d3a9d374419dda4d2 + License MIT is valid and included in separate text file as patch + rest looks Ok Suggestion: 1) cp command in %prep should preserve timestamp though its not a upstream file. 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 1119197] Review Request: gnushogi - Shogi (Japanese Chess) AI engine
https://bugzilla.redhat.com/show_bug.cgi?id=1119197 Chen Chen aflyho...@hotmail.com changed: What|Removed |Added Attachment #918089|0 |1 is obsolete|| --- Comment #9 from Chen Chen aflyho...@hotmail.com --- Created attachment 918647 -- https://bugzilla.redhat.com/attachment.cgi?id=918647action=edit gnushogi.spec After some struggle I decided to use the released tarball as initial package. So the package is splitted into two: gnushogi and xshogi. http://koji.fedoraproject.org/koji/taskinfo?taskID=7156120 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119197] Review Request: gnushogi - Shogi (Japanese Chess) AI engine
https://bugzilla.redhat.com/show_bug.cgi?id=1119197 --- Comment #10 from Chen Chen aflyho...@hotmail.com --- Created attachment 918648 -- https://bugzilla.redhat.com/attachment.cgi?id=918648action=edit gnushogi-1.4.2-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119197] Review Request: gnushogi - Shogi (Japanese Chess) AI engine
https://bugzilla.redhat.com/show_bug.cgi?id=1119197 --- Comment #11 from Chen Chen aflyho...@hotmail.com --- Created attachment 918649 -- https://bugzilla.redhat.com/attachment.cgi?id=918649action=edit xshogi.spec The xshogi is the X-11 front-end of GNU Shogi. http://koji.fedoraproject.org/koji/taskinfo?taskID=7156126 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119197] Review Request: gnushogi - Shogi (Japanese Chess) AI engine
https://bugzilla.redhat.com/show_bug.cgi?id=1119197 Chen Chen aflyho...@hotmail.com changed: What|Removed |Added Attachment #918361|0 |1 is obsolete|| --- Comment #12 from Chen Chen aflyho...@hotmail.com --- Created attachment 918650 -- https://bugzilla.redhat.com/attachment.cgi?id=918650action=edit xshogi-1.4.2-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1111388] Review Request: gmsh - A three-dimensional finite element mesh generator
https://bugzilla.redhat.com/show_bug.cgi?id=388 --- Comment #10 from Sandro Mani manisan...@gmail.com --- @Christopher: cgnslib BR added fftw: see [1] the FourierModel library is not publicly available. [1] http://geuz.org/pipermail/gmsh/2009/004007.html @Mukundan: All relevant rpmlint (fixable) rpmlint issues fixed. These gmsh.x86_64: W: no-documentation gmsh.x86_64: W: no-manual-page-for-binary gmsh gmsh-common.noarch: W: desktopfile-without-binary /usr/share/applications/gmsh.desktop gmsh gmsh-devel.x86_64: W: only-non-binary-in-usr-lib gmsh-devel.x86_64: W: no-documentation gmsh-libs.x86_64: W: no-documentation [etc for -openmpi, -mpich] are non-issues (documentation gets pulled in via dependencies, desktop file is common for all binaries, only-non-binary-in-usr-lib is probably because %lib contains only a symlink to the so). gmsh-libs.x86_64: W: shared-lib-calls-exit /usr/lib64/libGmsh.so.2.8.4 exit@GLIBC_2.2.5 needs to be considered upstream. Some unused-direct-shlib-dependency remain despite -Wl,--as-needed. Spec URL: http://smani.fedorapeople.org/review/gmsh.spec SRPM URL: http://smani.fedorapeople.org/review/gmsh-2.8.4-3.fc21.src.rpm %changelog * Thu Jul 17 2014 Sandro Mani manisan...@gmail.com - 2.8.4-3 - Fix encoding of file tutorial/t12.geo - Fix line endings of demos/component8.step - Fix incorrect changelog - Add -Wl,--as-needed - Add BRs: cgnslib-devel med-devel - Pass -DENABLE_CGNS=YES -DENABLE_MED=YES to cmake - Install onelab.py in %%{python2_sitelib} - Add missing Requires: gmsh-common -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #13 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- @jiri, do you have a chance to test on trimslice (or else ?) It's working fine on f20 on ac100, but I get a black screen on trimslice despite everything is properly detected according to xrandr. I wonder if this has something to do with my uboot that would toggle some gpio that would prevent. I'm also preparing a new rc of my fedora 20 tegra remix. About this review, I'm supposed to be the reviewer but given that I've made the last update I don't know what the status wrt the review. I would just request a formal ack from your side and approve the package, so we can make more work within the fedora repository. Obviously I volunteer to co-maintain the package. Is it okay for you ? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1115706] Review Request: nodejs-vow-fs - File I/O by Vow
https://bugzilla.redhat.com/show_bug.cgi?id=1115706 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? --- Comment #1 from Parag AN(पराग) panem...@gmail.com --- suggestions: 1) Add following to spec BuildArch: noarch %if 0%{?fedora} = 19 ExclusiveArch: %{nodejs_arches} noarch %else ExclusiveArch: %{ix86} x86_64 %{arm} noarch %endif -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #14 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- Note that I've built the git version that rely on the non-yet-upstream libdrm to enable exa support. The unpatched version was tested along with the exa enabled ddx for tegra. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119095] Review Request: python-djvulibre - Python bindings to DjVuLibre
https://bugzilla.redhat.com/show_bug.cgi?id=1119095 Michel Alexandre Salim mic...@michel-slm.name changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #8 from Michel Alexandre Salim mic...@michel-slm.name --- Hi Brian, Everything looks good -- this package is APPROVED Let me know your FAS username and I'll sponsor it. Feel free to look me up for packaging questions in the future, and welcome again! -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 David Dick dd...@cpan.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||dd...@cpan.org Assignee|nob...@fedoraproject.org|dd...@cpan.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 1120447] Review Request: tiptop - Performance monitoring tool based on hardware counters
https://bugzilla.redhat.com/show_bug.cgi?id=1120447 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Status|NEW |ASSIGNED CC||m...@fabian-affolter.ch Flags||fedora-review+ --- Comment #1 from Fabian Affolter m...@fabian-affolter.ch --- Package Review == Legend: [x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated [ ] = Manual review needed = MUST items = C/C++: [x]: Package does not contain kernel modules. [x]: Package contains no static executables. [x]: Package does not contain any libtool archives (.la) [x]: Rpath absent or only used for internal libs. Generic: [x]: Package is licensed with an open-source compatible license and meets other legal requirements as defined in the legal section of Packaging Guidelines. [!]: License field in the package spec file matches the actual license. Note: Checking patched sources after %prep for licenses. Licenses found: Unknown or generated. 33 files have unknown license. Detailed output of licensecheck in /home/fab/Documents/repos/reviews/1120447-tiptop/licensecheck.txt [x]: %build honors applicable compiler flags or justifies otherwise. [x]: Package contains no bundled libraries without FPC exception. [x]: Changelog in prescribed format. [x]: Sources contain only permissible code or content. [-]: Package contains desktop file if it is a GUI application. [-]: Development files must be in a -devel package [x]: Package uses nothing in %doc for runtime. [x]: Package consistently uses macros (instead of hard-coded directory names). [x]: Package is named according to the Package Naming Guidelines. [x]: Package does not generate any conflict. [x]: Package obeys FHS, except libexecdir and /usr/target. [-]: If the package is a rename of another package, proper Obsoletes and Provides are present. [x]: Requires correct, justified where necessary. [x]: Spec file is legible and written in American English. [-]: Package contains systemd file(s) if in need. [x]: Useful -debuginfo package or justification otherwise. [x]: Package is not known to require an ExcludeArch tag. [-]: Large documentation must go in a -doc subpackage. Large could be size (~1MB) or number of files. Note: Documentation size is 40960 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 %doc. [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]: Package does not contain duplicates in %files. [x]: Permissions on files are set properly. [x]: Package use %makeinstall only when make install' ' DESTDIR=... doesn't work. [x]: Package is named using only allowed ASCII characters. [x]: Package do not use a name that already exist [x]: Package is not relocatable. [x]: Sources used to build the package match the upstream source, as provided in the spec URL. [x]: Spec file name must match the spec package %{name}, in the format %{name}.spec. [x]: File names are valid UTF-8. [x]: Packages must not store files under /srv, /opt or /usr/local = SHOULD items = Generic: [-]: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [x]: Final provides and requires are sane (see attachments). [x]: Package functions as described. [x]: 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]: Package should compile and build into binary rpms on all supported architectures. [-]: %check is present and all tests pass. [x]: Packages should try to preserve timestamps of original installed files. [x]: Packager, Vendor, PreReq, Copyright tags
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 Jiri Kastner jkast...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #15 from Jiri Kastner jkast...@redhat.com --- @nicolas - sure (regarding trimslice and co-maintainer) trimslice works with latest kernel (3.15.5-200.fc20.armv7hl), but needed to use 00-opentegra.conf, see http://ur1.ca/hrl5e idea - there should be check during installation, based on /proc/device-tree: [root@dhcp-26-155 ~]# ls -1 /proc/device-tree/ | grep host1x host1x@5000 i tried on bbb load host1x module and it loaded without problem, but not having host1x@NNN under /proc/device-tree/ so, we may have 00-opentegra.conf in docs and add also some systemd oneshot unit, which checks after boot, that it is running on nvidia :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 David Dick dd...@cpan.org changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #1 from David Dick dd...@cpan.org --- License is correct. Build and Run Time Requires are correct. README.md could be added to %doc Due to Module::CPANfile only being available in f21+, you could use NO_PACKLIST=1 to eliminate the packlist removal. Package builds at http://koji.fedoraproject.org/koji/taskinfo?taskID=7156974 rpmlint is clean No blockers. 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 1118733] Review Request: perl-Retry - Retry Perl module
https://bugzilla.redhat.com/show_bug.cgi?id=1118733 David Dick dd...@cpan.org changed: What|Removed |Added Status|CLOSED |NEW Resolution|DEFERRED|--- Keywords||Reopened --- Comment #3 from David Dick dd...@cpan.org --- Upstream has re-licensed as Artistic 2.0 Spec URL: http://ddick.fedorapeople.org/packages/perl-Retry.spec SRPM URL: http://ddick.fedorapeople.org/packages/perl-Retry-1.01-1.fc20.src.rpm koji builds rawhide at http://koji.fedoraproject.org/koji/taskinfo?taskID=7156941 el6 cannot be built any more due to perl-Moo not being available in el6 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 --- Comment #2 from David Dick dd...@cpan.org --- One last thing, perl(version) = 0.76 could be added to Run Time Requires, which would require an update to the %global __requires_exclude section. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1113915] Review Request: micropython - Implementation of Python 3 with very low memory footprint
https://bugzilla.redhat.com/show_bug.cgi?id=1113915 --- Comment #6 from Matej Stuchlik mstuc...@redhat.com --- I've rebased the package to micropython 1.2, fixed an issue with cflags, added the ldflags and added a patch that stops git from being invoked during build. new SRPM URL: http://mstuchli.fedorapeople.org/micropython-1.2-1.fc20.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #3 from Petr Pisar ppi...@redhat.com --- New Package SCM Request === Package Name: perl-ExtUtils-MakeMaker-CPANfile Short Description: CPANfile support for ExtUtils::MakeMaker Upstream URL: http://search.cpan.org/dist/ExtUtils-MakeMaker-CPANfile/ Owners: ppisar jplesnik psabata Branches: InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1037900] Review Request: rubygem-resque-job-stats - Job-centric stats for Resque
https://bugzilla.redhat.com/show_bug.cgi?id=1037900 František Dvořák val...@civ.zcu.cz changed: What|Removed |Added CC||val...@civ.zcu.cz --- Comment #3 from František Dvořák val...@civ.zcu.cz --- Trying a review... (although unexperienced, I don't have any ruby package yet) Found only cosmetics (nothing needs to be changed): 1) For downloading sources you can consider client that preserves the upstream timestamps: https://fedoraproject.org/wiki/Packaging:Guidelines?rd=PackagingGuidelines#Timestamps 2) Why is the '%exclude %{gem_instdir}/test' in -doc subpackage? But there is probably no reason why not. Remarks: * You're patching out development files in %prep. It is true it may be better than using %exclude in %files, right? (Files won't be installed, they won't be there during testing...). * %check section: It looks like redis-server listens only on IPv4 and communication during testing goes through 127.0.0.1. But it works well in koji and even on IPv6-only machine. Would you agree for review swap with rubygem-json_spec (#1079640)? But I can do formal review here anyway. :-) Thank you, František -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 432646] Review Request: fio - multi-threaded IO generator tool
https://bugzilla.redhat.com/show_bug.cgi?id=432646 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 432646] Review Request: fio - multi-threaded IO generator tool
https://bugzilla.redhat.com/show_bug.cgi?id=432646 --- Comment #13 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1021733] Review Request: python-SecretStorage - Python module for secure storing of passwords and secrets
https://bugzilla.redhat.com/show_bug.cgi?id=1021733 --- Comment #29 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1021733] Review Request: python-SecretStorage - Python module for secure storing of passwords and secrets
https://bugzilla.redhat.com/show_bug.cgi?id=1021733 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1104746] Review Request: soscleaner - sosreport data obfuscation
https://bugzilla.redhat.com/show_bug.cgi?id=1104746 --- Comment #21 from Jon Ciesla limburg...@gmail.com --- Review flag not set. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1104746] Review Request: soscleaner - sosreport data obfuscation
https://bugzilla.redhat.com/show_bug.cgi?id=1104746 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? | -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1115894] Review Request: ghc-hgettext - Bindings to libintl.h (gettext, bindtextdomain)
https://bugzilla.redhat.com/show_bug.cgi?id=1115894 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 --- Comment #7 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120173] Review Request: perl-Devel-ArgNames - Figure out the names of variables passed into subroutines
https://bugzilla.redhat.com/show_bug.cgi?id=1120173 --- Comment #3 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120173] Review Request: perl-Devel-ArgNames - Figure out the names of variables passed into subroutines
https://bugzilla.redhat.com/show_bug.cgi?id=1120173 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1116487] Review Request: perl-App-DuckDuckGo - Application to query DuckDuckGo
https://bugzilla.redhat.com/show_bug.cgi?id=1116487 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 Jon Ciesla limburg...@gmail.com changed: What|Removed |Added Flags|fedora-cvs? |fedora-cvs+ -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 --- Comment #4 from Jon Ciesla limburg...@gmail.com --- Git done (by process-git-requests). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #16 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- (In reply to Jiri Kastner from comment #15) @nicolas - sure (regarding trimslice and co-maintainer) trimslice works with latest kernel (3.15.5-200.fc20.armv7hl), but needed to use 00-opentegra.conf, see http://ur1.ca/hrl5e Which uboot version number are you using ? (is it a vendor provided one or upstream uboot ?) I would handle the 00-opentegra.conf at the remix step for f20. http://fedorapeople.org/cgit/kwizart/public_git/spin-kickstarts.git/log/?h=f20 For F-21 and xorg-server 1.16 there is a dedicated file that will support the outputclass feature: http://lists.x.org/archives/xorg-devel/2014-February/040578.html i tried on bbb load host1x module and it loaded without problem, but not having host1x@NNN under /proc/device-tree/ I'm still not sure but the tegra_drm isn't automatically loaded (host1x is actually handling the modalias), maybe we can use something similar to the way dracut tries to detect hostonly modules ? http://www.spinics.net/lists/linux-initramfs/msg03636.html -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120680] New: Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 Bug ID: 1120680 Summary: Review Request: trace-summary - A script generating break-downs of network traffic Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: m...@fabian-affolter.ch QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://fab.fedorapeople.org/packages/SRPMS/trace-summary.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/trace-summary-0.83-1.fc20.src.rpm Project URL: http://bro.org/sphinx/components/trace-summary/README.html Description: trace-summary is a Python script that generates break-downs of network traffic, including lists of the top hosts, protocols, ports, etc. Optionally, it can generate output separately for incoming vs. outgoing traffic, per subnet, and per time-interval. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=7155750 rpmlint output: [fab@laptop014 SRPMS]$ rpmlint trace-summary-0.83-1.fc20.src.rpm trace-summary.src: W: spelling-error %description -l en_US subnet - subset, sublet, sub net 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [fab@laptop014 noarch]$ rpmlint trace-summary-0.83-1.fc20.noarch.rpm trace-summary.noarch: W: spelling-error %description -l en_US subnet - subset, sublet, sub net trace-summary.noarch: W: no-manual-page-for-binary trace-summary 1 packages and 0 specfiles checked; 0 errors, 2 warnings. Fedora Account System Username: fab -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120680] Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Blocks||979726 (bro) Alias||trace-summary Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=979726 [Bug 979726] bro 2.3 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1113641] Review Request: capstats - A command-line tool collecting packet statistics
https://bugzilla.redhat.com/show_bug.cgi?id=1113641 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Alias|capstats| -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1113640] Review Request: capstats - A command-line tool collecting packet statistics
https://bugzilla.redhat.com/show_bug.cgi?id=1113640 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Alias||capstats -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1104746] Review Request: soscleaner - sosreport data obfuscation
https://bugzilla.redhat.com/show_bug.cgi?id=1104746 Jamie Duncan jdun...@redhat.com changed: What|Removed |Added Flags||fedora-review+ fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1104746] Review Request: soscleaner - sosreport data obfuscation
https://bugzilla.redhat.com/show_bug.cgi?id=1104746 --- Comment #22 from Jamie Duncan jdun...@redhat.com --- setting flags back to what they're supposed to be. fedora-review was ack'd in c#14. resetting fedora-cvs flag to ? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 Nicolas Chauvet (kwizart) kwiz...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #17 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- New Package SCM Request === Package Name: xorg-x11-drv-opentegra Short Description: open source Nvidia Tegra driver Upstream URL: http://cgit.freedesktop.org/xorg/driver/xf86-video-opentegra/ Owners: kwizart jkastner Branches: f20 f21 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120366] Review Request: nodejs-set-immediate - A shim for the setImmediate API
https://bugzilla.redhat.com/show_bug.cgi?id=1120366 Tom Hughes t...@compton.nu changed: What|Removed |Added Flags|fedora-cvs- |fedora-cvs? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120366] Review Request: nodejs-set-immediate - A shim for the setImmediate API
https://bugzilla.redhat.com/show_bug.cgi?id=1120366 Tom Hughes t...@compton.nu changed: What|Removed |Added Flags||fedora-cvs- --- Comment #4 from Tom Hughes t...@compton.nu --- New Package SCM Request === Package Name: nodejs-set-immediate Short Description: A shim for the setImmediate API Owners: tomh jamielinux Branches: f19 f20 f21 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120680] Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com Assignee|nob...@fedoraproject.org|panem...@gmail.com Flags||fedora-review? --- Comment #1 from Parag AN(पराग) panem...@gmail.com --- looks like simple packaging -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120173] Review Request: perl-Devel-ArgNames - Figure out the names of variables passed into subroutines
https://bugzilla.redhat.com/show_bug.cgi?id=1120173 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-Devel-ArgNames-0.03-1. ||fc22 Resolution|--- |NEXTRELEASE Last Closed||2014-07-17 08:58:58 --- Comment #4 from Petr Pisar ppi...@redhat.com --- Fixed as perl-Devel-ArgNames-0.03-1.fc21 in F21. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 --- Comment #4 from Petr Šabata psab...@redhat.com --- Alright, removed. Links updated. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120447] Review Request: tiptop - Performance monitoring tool based on hardware counters
https://bugzilla.redhat.com/show_bug.cgi?id=1120447 Christopher Meng i...@cicku.me changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Christopher Meng i...@cicku.me --- New Package SCM Request === Package Name: tiptop Short Description: Performance monitoring tool based on hardware counters Upstream URL: http://tiptop.gforge.inria.fr/ Owners: cicku Branches: f19 f20 f21 el6 epel7 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120202] Review Request: perl-ExtUtils-MakeMaker-CPANfile - CPANfile support for ExtUtils::MakeMaker
https://bugzilla.redhat.com/show_bug.cgi?id=1120202 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||perl-ExtUtils-MakeMaker-CPA ||Nfile-0.06-1.fc22 Resolution|--- |RAWHIDE Last Closed||2014-07-17 09:32:26 --- Comment #5 from Petr Pisar ppi...@redhat.com --- Thank you for the review and the repository. I added the README.md and the exact version version. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120680] Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #2 from Parag AN(पराग) panem...@gmail.com --- Review: + mock build is successful for f22 + rpmlint output on rpms gave trace-summary.noarch: W: spelling-error %description -l en_US subnet - subset, sublet, sub net trace-summary.noarch: W: no-manual-page-for-binary trace-summary trace-summary.src: W: spelling-error %description -l en_US subnet - subset, sublet, sub net 2 packages and 0 specfiles checked; 0 errors, 3 warnings. = No issues + Source verified with upstream as (sha256sum) srpm tarball :1a452561a8e8b27315ccf85a77eff6c625b974b4cdc9cfd6496c05199fe1993e upstream tarball :1a452561a8e8b27315ccf85a77eff6c625b974b4cdc9cfd6496c05199fe1993e + License BSD is valid and included in its own file as %doc. + rest looks following packaging guidelines 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 1120680] Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 --- Comment #3 from Fabian Affolter m...@fabian-affolter.ch --- Thanks 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 1120680] Review Request: trace-summary - A script generating break-downs of network traffic
https://bugzilla.redhat.com/show_bug.cgi?id=1120680 Fabian Affolter m...@fabian-affolter.ch changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Fabian Affolter m...@fabian-affolter.ch --- New Package SCM Request === Package Name: trace-summary Short Description: A script generating break-downs of network traffic Owners: fab Branches: f20 f21 el6 epel7 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #18 from Jiri Kastner jkast...@redhat.com --- (In reply to Nicolas Chauvet (kwizart) from comment #16) Which uboot version number are you using ? (is it a vendor provided one or upstream uboot ?) U-Boot 2012.04.01-1.02 from compulab -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 --- Comment #19 from Nicolas Chauvet (kwizart) kwiz...@gmail.com --- New Package SCM Request === Package Name: xorg-x11-drv-opentegra Short Description: open source Nvidia Tegra driver Upstream URL: http://cgit.freedesktop.org/xorg/driver/xf86-video-opentegra/ Owners: kwizart jkastner Branches: f19 f20 f21 InitialCC: -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119095] Review Request: python-djvulibre - Python bindings to DjVuLibre
https://bugzilla.redhat.com/show_bug.cgi?id=1119095 --- Comment #9 from Brian Stinson bstin...@ksu.edu --- Great! Thanks for guiding me through this one. I might have a couple of minor questions that I'll send to you out-of-band. My FAS username is bstinson. Is the next step for this package an SCM request? --Brian -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 --- Comment #5 from Petr Pisar ppi...@redhat.com --- URL and Source0 are usable. Ok. Source tar ball is modified (original SHA-256: f8593750d45d1c068195b4f16643fe4b3d50956e79d26c3c8aa7ba887ae75a45). lib/Inline/C/Cookbook.pod has been removed and MANIFEST updated. Ok. Summary verified from README. Ok. Description verified from README. Ok. License verified from lib/Inline/C.pod, lib/Inline/C/ParseRecDescent.pod, lib/Inline/C/ParseRegExp.pod, LICENSE, README. Ok. No XS is code in the binary packages, noarch BuildArch is Ok. TODO: Due to NO_PACKLIST=1 added in recent ExtUtil::MakeMaker, you should increase perl(ExtUtils::MakeMaker) build-time dependency to `= 6.76'. The Concflicts is Ok. TODO: Constrain `perl(File::Spec)' dependency with `= 0.8' (Makefile.PL:23). TODO: Constrain `perl(Parse::RecDescent)' dependency with `= 1.80' (Makefile.PL:25). TODO: Constrain `perl(Test::More)' dependency with `= 0.88' (Makefile.PL:45). TODO: Constrain `perl(Test::Warn)' dependency with `= 0.23' (Makefile.PL:46). TODO: I recommend to conditionalize dependencies on Inline::Filters and Inline::Struct by %perl_bootstrap macro as they are optional as Parse::RecDescent. Test::Pod is not used. Ok. All tests pass. Ok. $ rpmlint perl-Inline-C.spec ../SRPMS/perl-Inline-C-0.60-1.fc22.src.rpm ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm perl-Inline-C.spec:8: W: macro-in-comment %{version} perl-Inline-C.spec:9: W: macro-in-comment %{version} perl-Inline-C.spec:10: W: macro-in-comment %{version} perl-Inline-C.spec:11: W: macro-in-comment %{version} perl-Inline-C.spec:12: W: macro-in-comment %{version} perl-Inline-C.spec:12: W: macro-in-comment %{version} perl-Inline-C.spec: W: invalid-url Source0: Inline-C-0.60.fedora.tar.gz perl-Inline-C.src:8: W: macro-in-comment %{version} perl-Inline-C.src:9: W: macro-in-comment %{version} perl-Inline-C.src:10: W: macro-in-comment %{version} perl-Inline-C.src:11: W: macro-in-comment %{version} perl-Inline-C.src:12: W: macro-in-comment %{version} perl-Inline-C.src:12: W: macro-in-comment %{version} perl-Inline-C.src: W: invalid-url Source0: Inline-C-0.60.fedora.tar.gz 2 packages and 1 specfiles checked; 0 errors, 14 warnings. TODO: Escape % in comments. $ rpm -q -lv -p ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm drwxr-xr-x2 rootroot0 Jul 17 15:50 /usr/share/doc/perl-Inline-C -rw-r--r--1 rootroot 1267 Jul 14 07:04 /usr/share/doc/perl-Inline-C/CONTRIBUTING -rw-r--r--1 rootroot 5435 Jul 14 07:04 /usr/share/doc/perl-Inline-C/Changes -rw-r--r--1 rootroot18350 Jul 14 07:04 /usr/share/doc/perl-Inline-C/LICENSE -rw-r--r--1 rootroot19756 Jul 14 07:04 /usr/share/doc/perl-Inline-C/README -rw-r--r--1 rootroot 9345 Jul 17 15:48 /usr/share/man/man3/Inline::C.3pm.gz -rw-r--r--1 rootroot 2159 Jul 17 15:48 /usr/share/man/man3/Inline::C::ParseRecDescent.3pm.gz -rw-r--r--1 rootroot 2159 Jul 17 15:48 /usr/share/man/man3/Inline::C::ParseRegExp.3pm.gz drwxr-xr-x2 rootroot0 Jul 17 15:48 /usr/share/perl5/vendor_perl/Inline/C -rw-r--r--1 rootroot33308 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C.pm -rw-r--r--1 rootroot19197 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C.pod -rw-r--r--1 rootroot 4122 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C/ParseRecDescent.pm -rw-r--r--1 rootroot 775 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C/ParseRecDescent.pod -rw-r--r--1 rootroot 5576 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C/ParseRegExp.pm -rw-r--r--1 rootroot 743 Jul 14 07:04 /usr/share/perl5/vendor_perl/Inline/C/ParseRegExp.pod File layout and permissions are Ok. $ rpm -q --requires -p ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.18.2) 1 perl(Carp) 1 perl(Config) 1 perl(Cwd) 1 perl(Data::Dumper) 1 perl(ExtUtils::MakeMaker) 1 perl(Fcntl) 1 perl(File::Spec) 1 perl(Inline) = 0.56 1 perl(Inline) = 0.58 1 perl(strict) 1 rpmlib(CompressedFileNames) = 3.0.4-1 1 rpmlib(FileDigests) = 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) = 4.0-1 1 rpmlib(PayloadIsXz) = 5.2-1 TODO: Remove under-specified dependency on `perl(Inline) = 0.56'. FIX: Run-require `perl(FindBin)' (lib/Inline/C.pm:47). TODO: I recommend to run-require Inline::Filters, Inline::Struct, Parse::RecDescent, Time::HiRes. The code croaks without them. This is just a prettier die. $ rpm -q --provides -p ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm | sort -f | uniq -c 1 perl(Inline::C) = 0.60 1
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 --- Comment #6 from Petr Šabata psab...@redhat.com --- (In reply to Petr Pisar from comment #5) TODO: I recommend to conditionalize dependencies on Inline::Filters and Inline::Struct by %perl_bootstrap macro as they are optional as Parse::RecDescent. That's planned once they get into Fedora. TODO: Due to NO_PACKLIST=1 added in recent ExtUtil::MakeMaker, you should increase perl(ExtUtils::MakeMaker) build-time dependency to `= 6.76'. Done. TODO: Escape % in comments. Done. FIX: Run-require `perl(FindBin)' (lib/Inline/C.pm:47). Done. TODO: I recommend to run-require Inline::Filters, Inline::Struct, Parse::RecDescent, Time::HiRes. The code croaks without them. This is just a prettier die. Planned. Will be done once they're available. Links updated. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1119118] Review Request: perl-App-Netdisco - Web-based network management tool
https://bugzilla.redhat.com/show_bug.cgi?id=1119118 Bug 1119118 depends on bug 1119953, which changed state. Bug 1119953 Summary: perl-Data-Dumper-Concise-2.022 is available https://bugzilla.redhat.com/show_bug.cgi?id=1119953 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 956525] Review Request: xorg-x11-drv-opentegra - open source Nvidia Tegra driver
https://bugzilla.redhat.com/show_bug.cgi?id=956525 Dennis Gilmore den...@ausil.us changed: What|Removed |Added Flags|fedora-review+ fedora-cvs? |fedora-review- fedora-cvs- --- Comment #20 from Dennis Gilmore den...@ausil.us --- this is not teh right way to handle package reviews I am naking it all, do it right. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1038238] Review Request: kcm-touchpad - KDE Systemsettings module for touchpads
https://bugzilla.redhat.com/show_bug.cgi?id=1038238 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |NOTABUG Last Closed||2014-07-17 10:48:48 --- Comment #8 from Rex Dieter rdie...@math.unl.edu --- Sorry, I forgot to close this. We'd opted to import this into the existing kcm_touchpad git module, instead of going through a new review largely to rename it to kcm-touchpad instead. Hit me up on irc if you're interested in a scratch build for testing (say, on f20 or whatever). -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Pisar ppi...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #7 from Petr Pisar ppi...@redhat.com --- Spec file changes: $ diff -u perl-Inline-C.spec{.old,} --- perl-Inline-C.spec.old 2014-07-17 15:13:52.0 +0200 +++ perl-Inline-C.spec 2014-07-17 16:24:15.0 +0200 @@ -6,10 +6,10 @@ Group: Development/Libraries URL:http://search.cpan.org/dist/Inline-C/ # Created from http://www.cpan.org/authors/id/I/IN/INGY/Inline-C-%{version}.tar.gz -# tar xzf Inline-C-%{version}.tar.gz \ -# rm -f Inline-C-%{version}/lib/Inline/C/Cookbook.pod \ -# sed -e '/Cookbook.pod$/d' -i Inline-C-%{version}/MANIFEST \ -# tar czf Inline-C-%{version}.fedora.tar.gz Inline-C-%{version} +# tar xzf Inline-C-%%{version}.tar.gz \ +# rm -f Inline-C-%%{version}/lib/Inline/C/Cookbook.pod \ +# sed -e '/Cookbook.pod$/d' -i Inline-C-%%{version}/MANIFEST \ +# tar czf Inline-C-%%{version}.fedora.tar.gz Inline-C-%{version} Source0:Inline-C-%{version}.fedora.tar.gz BuildArch: noarch BuildRequires: perl @@ -19,7 +19,7 @@ BuildRequires: perl(Cwd) BuildRequires: perl(Data::Dumper) BuildRequires: perl(diagnostics) -BuildRequires: perl(ExtUtils::MakeMaker) = 6.30 +BuildRequires: perl(ExtUtils::MakeMaker) = 6.76 BuildRequires: perl(Fcntl) BuildRequires: perl(File::Path) BuildRequires: perl(File::Spec) @@ -34,6 +34,7 @@ BuildRequires: perl(Test::Warn) BuildRequires: perl(Time::HiRes) BuildRequires: perl(warnings) +Requires: perl(FindBin) Requires: perl(Inline) = 0.58 Requires: perl(:MODULE_COMPAT_%(eval $(perl -V:version); echo $version)) # Split from Inline in 0.58 TODO: Due to NO_PACKLIST=1 added in recent ExtUtil::MakeMaker, you should increase perl(ExtUtils::MakeMaker) build-time dependency to `= 6.76'. -BuildRequires: perl(ExtUtils::MakeMaker) = 6.30 +BuildRequires: perl(ExtUtils::MakeMaker) = 6.76 Ok. TODO: Constrain `perl(File::Spec)' dependency with `= 0.8' (Makefile.PL:23). Not addressed. TODO: Constrain `perl(Parse::RecDescent)' dependency with `= 1.80' (Makefile.PL:25). Not addressed. TODO: Constrain `perl(Test::More)' dependency with `= 0.88' (Makefile.PL:45). Not addressed. TODO: Constrain `perl(Test::Warn)' dependency with `= 0.23' (Makefile.PL:46). Not addressed. TODO: I recommend to conditionalize dependencies on Inline::Filters and Inline::Struct by %perl_bootstrap macro as they are optional as Parse::RecDescent. Not addressed. One can add %global perl_bootstrap 1 at the beginning of the spec file now and use the macros now. $ rpmlint perl-Inline-C.spec ../SRPMS/perl-Inline-C-0.60-1.fc22.src.rpm ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm perl-Inline-C.spec:8: W: macro-in-comment %{version} perl-Inline-C.spec:12: W: macro-in-comment %{version} perl-Inline-C.spec: W: invalid-url Source0: Inline-C-0.60.fedora.tar.gz perl-Inline-C.src:8: W: macro-in-comment %{version} perl-Inline-C.src:12: W: macro-in-comment %{version} perl-Inline-C.src: W: invalid-url Source0: Inline-C-0.60.fedora.tar.gz 2 packages and 1 specfiles checked; 0 errors, 6 warnings. TODO: Remove under-specified dependency on `perl(Inline) = 0.56'. Not addressed. $ rpm -q --requires -p ../RPMS/noarch/perl-Inline-C-0.60-1.fc22.noarch.rpm | sort -f | uniq -c 1 perl(:MODULE_COMPAT_5.18.2) 1 perl(Carp) 1 perl(Config) 1 perl(Cwd) 1 perl(Data::Dumper) 1 perl(ExtUtils::MakeMaker) 1 perl(Fcntl) 1 perl(File::Spec) 1 perl(FindBin) 1 perl(Inline) = 0.56 1 perl(Inline) = 0.58 1 perl(strict) 1 rpmlib(CompressedFileNames) = 3.0.4-1 1 rpmlib(FileDigests) = 4.6.0-1 1 rpmlib(PayloadFilesHavePrefix) = 4.0-1 1 rpmlib(PayloadIsXz) = 5.2-1 TODO: Remove under-specified dependency on `perl(Inline) = 0.56'. Not addressed. FIX: Run-require `perl(FindBin)' (lib/Inline/C.pm:47). Ok. TODO: I recommend to run-require Inline::Filters, Inline::Struct, Parse::RecDescent, Time::HiRes. The code croaks without them. This is just a prettier die. Not addressed. Package builds in F22 (http://koji.fedoraproject.org/koji/taskinfo?taskID=7158611). Ok. Please consider fixing the `TODO' items before building the package. Resolution: 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 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C
https://bugzilla.redhat.com/show_bug.cgi?id=1120293 Petr Šabata psab...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #8 from Petr Šabata psab...@redhat.com --- New Package SCM Request === Package Name: perl-Inline-C Short Description: Write Perl subroutines in C Upstream URL: http://search.cpan.org/dist/Inline-C/ Owners: psabata jplesnik ppisar Branches: f21 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1114175] Review Request: golang-github-codegangsta-cli - Package for building command line apps in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1114175 Lokesh Mandvekar l...@fedoraproject.org changed: What|Removed |Added Blocks||1118603 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1118603 [Bug 1118603] docker-io-1.1.1 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1111916] Review Request: golang-github-docker-libcontainer - Configuration options for containers
https://bugzilla.redhat.com/show_bug.cgi?id=916 Lokesh Mandvekar l...@fedoraproject.org changed: What|Removed |Added Blocks||1118603 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1118603 [Bug 1118603] docker-io-1.1.1 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1117562] Review Request: golang-github-tchap-go-patricia - A generic patricia trie implemented in Go
https://bugzilla.redhat.com/show_bug.cgi?id=1117562 Lokesh Mandvekar l...@fedoraproject.org changed: What|Removed |Added Blocks||1118603 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=1118603 [Bug 1118603] docker-io-1.1.1 is available -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1018546] Review Request: musepack-libmpc - Living audio compression
https://bugzilla.redhat.com/show_bug.cgi?id=1018546 --- Comment #4 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com --- I finally got down to this. Here's the new spec: http://ankursinha.fedorapeople.org/musepack-libmpc/musepack-libmpc.spec New srpm: http://ankursinha.fedorapeople.org/musepack-libmpc/musepack-libmpc-1.3.0-0.1.svn484.fc22.src.rpm All dependent packages will need to be rebuilt since this version changes the version of the provided shared object: libmpcdec.so.6()(64bit). The older was libmpcdec.so.5()(64bit) rpmlint errors: [asinha@ankur-laptop SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm ./musepack-libmpc-1.3.0-0.1.svn484.fc20.src.rpm ../SPECS/musepack-libmpc.spec libmpcdec.x86_64: W: spelling-error Summary(en_US) Musepack - Muse pack, Muse-pack, Mudpack libmpcdec.x86_64: W: spelling-error %description -l en_US Musepack - Muse pack, Muse-pack, Mudpack libmpcdec.x86_64: W: no-manual-page-for-binary mpcdec libmpcdec-devel.x86_64: W: spelling-error Summary(en_US) Musepack - Muse pack, Muse-pack, Mudpack libmpcdec-devel.x86_64: W: only-non-binary-in-usr-lib libmpcdec-devel.x86_64: W: no-documentation musepack-libmpc.src: W: spelling-error %description -l en_US lossless - loss less, loss-less, loveless musepack-libmpc.src: W: spelling-error %description -l en_US patentless - patent less, patent-less, pathless musepack-libmpc.src:4: W: macro-in-comment %{actual_name} musepack-libmpc.src: W: invalid-url Source0: libmpc_r484.tar.gz musepack-libmpc.x86_64: W: spelling-error %description -l en_US lossless - loss less, loss-less, loveless musepack-libmpc.x86_64: W: spelling-error %description -l en_US patentless - patent less, patent-less, pathless musepack-libmpc.x86_64: W: no-documentation musepack-libmpc.x86_64: W: no-manual-page-for-binary wavcmp musepack-libmpc.x86_64: W: no-manual-page-for-binary mpccut musepack-libmpc.x86_64: W: no-manual-page-for-binary mpc2sv8 musepack-libmpc.x86_64: W: no-manual-page-for-binary mpcgain musepack-libmpc.x86_64: W: no-manual-page-for-binary mpcenc musepack-libmpc-devel.x86_64: W: no-documentation musepack-libmpc.src: W: spelling-error %description -l en_US lossless - loss less, loss-less, loveless musepack-libmpc.src: W: spelling-error %description -l en_US patentless - patent less, patent-less, pathless musepack-libmpc.src:4: W: macro-in-comment %{actual_name} musepack-libmpc.src: W: invalid-url Source0: libmpc_r484.tar.gz ../SPECS/musepack-libmpc.spec:4: W: macro-in-comment %{actual_name} ../SPECS/musepack-libmpc.spec: W: invalid-url Source0: libmpc_r484.tar.gz 7 packages and 1 specfiles checked; 0 errors, 25 warnings. Thanks, Warm regards, Ankur -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1018544] Review Request: libcuefile - A stripped down version of cuetools
https://bugzilla.redhat.com/show_bug.cgi?id=1018544 Ankur Sinha (FranciscoD) sanjay.an...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution|--- |WONTFIX Last Closed||2014-07-17 11:21:30 --- Comment #3 from Ankur Sinha (FranciscoD) sanjay.an...@gmail.com --- Not required any more. musepack builds against libcue now. Closing -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review