[Bug 1120293] Review Request: perl-Inline-C - Write Perl subroutines in C

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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)

2014-07-17 Thread bugzilla
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)

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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)

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

2014-07-17 Thread bugzilla
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

  1   2   >