[Bug 1086790] Review Request: gnudos - A GNU library to help new users of the GNU system

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086790



--- Comment #9 from Ralf Corsepius  ---
(In reply to Ralf Corsepius from comment #8)

> * MUSTFIX: Do not ship libtool-archives (*.la):
> %{_libdir}/libgnudos-1.0.la
> You need to *.la's in %install.
Sorry, ugly typo, this should have been: "You need to remove *.la's in
%install"

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1086790] Review Request: gnudos - A GNU library to help new users of the GNU system

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086790



--- Comment #8 from Ralf Corsepius  ---
* MUSTFIX: Remove this
%{_libdir}/debug/*
These files are automatically packaged into *-debuginfo and not supposed to be
packaged into normal packages.

* MUSTFIX: Do not ship static libs
%{_libdir}/libgnudos-1.0.a
Shipping static libs is strongly discouraged in Fedora.
If you really want to ship static libs, these must be packaged into a *-static
package.

As the package seems to support disabling building static libs, consider to
append "--disable-static" to %configure.

* MUSTFIX: Do not ship libtool-archives (*.la):
%{_libdir}/libgnudos-1.0.la
You need to *.la's in %install.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1086790] Review Request: gnudos - A GNU library to help new users of the GNU system

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1086790



--- Comment #7 from Mohammed Isam  ---
Spec URL:
http://sites.google.com/site/mohammedisam2000/home/projects/gnudos.spec
SRPM URL:
http://sites.google.com/site/mohammedisam2000/home/projects/gnudos-1.1-2.fc20.src.rpm

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894608] Review Request: coin-or-SYMPHONY - Solver for mixed-integer linear programs

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894608



--- Comment #7 from Fedora Update System  ---
coin-or-SYMPHONY-5.5.7-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/coin-or-SYMPHONY-5.5.7-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 894608] Review Request: coin-or-SYMPHONY - Solver for mixed-integer linear programs

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894608

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539



--- Comment #13 from Pádraig Brady  ---
BTW this loop looks very inefficient:

for filt in filters/*; do
  sed 's,/usr/bin/env ,/usr/bin/,' $filt > ${filt}.new
  chmod 0755 ${filt}.new
  touch -r $filt ${filt}.new
  mv -f ${filt}.new $filt
done

Could it be replaced with:

sed -i 's,/usr/bin/env ,/usr/bin/,; q' filters/*

or

find filters/ -exec sed -i 's,/usr/bin/env ,/usr/bin/,; q' {} +

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089559] Review Request: tlp - Advanced power management tool for Linux

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089559

Jeremy Newton  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #11 from Jeremy Newton  ---
Thanks!

New Package SCM Request
===
Package Name: tlp
Short Description: Advanced power management tool for Linux
Owners: mystro256
Branches: f19 f20 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 1091361] Review Request: perl-HTML-FormFu-Element-reCAPTCHA - reCAPTCHA component for HTML::FormFu frame work

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091361

David Dick  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #1 from David Dick  ---
License ok.

rpmlint fine (only incorrect failures to recognise spelling)

koji build at http://koji.fedoraproject.org/koji/taskinfo?taskID=6781770

BR: perl(HTML::FormFu) # line 6 of t/recaptcha.t
BR: perl(HTML::FormFu::Constraint) # line 8 of
lib/HTML/FormFu/Element/reCAPTCHA.pm
BR: perl(HTML::FormFu::Element::Multi) # line 8 of
lib/HTML/FormFu/Element/reCAPTCHA.pm

Requires: perl(HTML::FormFu::Constraint) # line 8 of
lib/HTML/FormFu/Element/reCAPTCHA.pm
Requires: perl(HTML::FormFu::Element::Multi) # line 8 of
lib/HTML/FormFu/Element/reCAPTCHA.pm

All these requires are pretty much irrelevant as there is already a BR and
RunTime requirement for HTML::FormFu::Util which is in the same package as all
these other requirements.  However, they may be useful if Carl Franks decides
to split up HTML::FormFu.  I'm fine either way.

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 1091144] Review Request: perl-Parse-DMIDecode - Interface to SMBIOS using dmidecode

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091144



--- Comment #3 from David Dick  ---
removed the unnecessary debug builds.

koji builds at

rawhide http://koji.fedoraproject.org/koji/taskinfo?taskID=6781764

el6 http://koji.fedoraproject.org/koji/taskinfo?taskID=6781767

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091361] Review Request: perl-HTML-FormFu-Element-reCAPTCHA - reCAPTCHA component for HTML::FormFu frame work

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091361

David Dick  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 1079064] Review Request: btbuilder - Role-playing game construction set in the style of the Bard's Tale Construction Set

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1079064



--- Comment #10 from Dennis Payne  ---
(In reply to Dridi Boukelmoune from comment #9)
Spec URL:
https://raw.githubusercontent.com/dulsi/btbuilder/master/btbuilder.spec
SRPM URL: http://identicalsoftware.com/btbuilder/btbuilder-0.4.4-1.fc20.src.rpm

Ok. I didn't know about the fedora-review tool. I should looked at the complete
Package Review Process instead of just the joining page.

> You will need to fix this, and mention all the licenses for the package:
> => for instance "License: GPLv2 and GPLv3+"
GPLv3+ is the only license. It was GPLv2+ but when I replaced the images some
of the new images were GPLv3+. Rather than specify what is GPLv2+ and what is
GPL3+, everything is now GPLv3+.

> I encourage you to bump the Release tag every time you submit a new version
> during the review.
> 
>* date name  - version-3
>- Updated the license for both code and assets

I would but I've been doing new releases of the project. I know I could do
0.4.4-3 even though there is no 0.4.4-1 or 0.4.4-2 I'd rather not.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089634] Review Request: perl-Crypt-SaltedHash - Perl interface to functions that assist in working with salted hashes

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089634

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #10 from Fedora Update System  ---
perl-Crypt-SaltedHash-0.09-1.el6 has been pushed to the Fedora EPEL 6 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 991689] Review Request: dislocker - Utility to access BitLocker encrypted volumes

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=991689

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #27 from Fedora Update System  ---
dislocker-0.3.1-2.20140423git.el6 has been pushed to the Fedora EPEL 6 testing
repository.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1074046] Review Request: telegram-cli - Linux Command-line interface for Telegram

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1074046

Fedora Update System  changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA



--- Comment #20 from Fedora Update System  ---
Package telegram-cli-0-0.5.20140321git1dad2e.el6:
* should fix your issue,
* was pushed to the Fedora EPEL 6 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=epel-testing
telegram-cli-0-0.5.20140321git1dad2e.el6'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-EPEL-2014-1231/telegram-cli-0-0.5.20140321git1dad2e.el6
then log in and leave karma (feedback).

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 462446] Review Request: ttf2pt1 - TrueType to Adobe Type 1 font converter

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=462446

Göran Uddeborg  changed:

   What|Removed |Added

  Flags|fedora-cvs+ |fedora-cvs?



--- Comment #28 from Göran Uddeborg  ---
Package Change Request
==
Package Name: ttf2pt1
New Branches: epel7
Owners: goeran
InitialCC: hubbitus

Pavel Alexeev wants to build chm2pdf for epel7, and this is a dependency.

-- 
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 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232



--- Comment #12 from Ryan O'Hara  ---
(In reply to Jon Ciesla from comment #10)
> No SCM request found.

Sorry about that.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Ryan O'Hara  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Ryan O'Hara  ---
New Package SCM Request
===
Package Name: mariadb-galera
Short Description: A community developed branch of MySQL, with wsrep patches
Owners: rohara
Branches: f20 el6 epel7
InitialCC: rohara

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232

Ryan O'Hara  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 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232



--- Comment #11 from Ryan O'Hara  ---
New Package SCM Request
===
Package Name: galera
Short Description: Synchronous multi-master wsrep provider (replication engine)
Owners: rohara
Branches: f20 el6 epel7
InitialCC: rohara

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076506] Review Request: ambari - Hadoop cluster manager

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076506

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076506] Review Request: ambari - Hadoop cluster manager

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076506



--- Comment #20 from Fedora Update System  ---
ambari-1.5.1-1.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/ambari-1.5.1-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 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234



--- Comment #4 from Jon Ciesla  ---
No SCM request found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232

Jon Ciesla  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 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Jon Ciesla  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 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232



--- Comment #10 from Jon Ciesla  ---
No SCM request found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539



--- Comment #12 from Jerry James  ---
Pádraig and Matthias, subunit has now been built for Rawhide.  Can one of you
retire the existing python-subunit package, in Rawhide only, please?

I will send email about this, too, in case they aren't following this bug.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083232] Review Request: galera - Galera replication engine

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083232

Ryan O'Hara  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 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Ryan O'Hara  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 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539



--- Comment #11 from Jerry James  ---
(In reply to David King from comment #8)
> If it is easy to fix then it would be nice, but as a "should" item I think
> it is only a nice-to-have, not a blocker. Thanks for investigating!

I was able to fix this with a handful of touch commands at the end of %install,
by the way.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #6 from Fedora Update System  ---
trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.el6 has been
submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.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 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052

Fedora Update System  changed:

   What|Removed |Added

 Status|NEW |MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #7 from Fedora Update System  ---
trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.fc19 has been
submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #8 from Fedora Update System  ---
trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.fc20 has been
submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.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 1089463] Review Request: nodejs-fs-extra - A drop-in replacement for Node's native fs module with a few extra methods

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089463

Jamie Nguyen  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-04-25 14:47:25



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 881590] Review Request: leveldbjni - A Java Native Interface to LevelDB

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=881590

gil cattaneo  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |RAWHIDE
Last Closed||2014-04-25 14:36:04



--- Comment #10 from gil cattaneo  ---
not able to build on f20, see logs [1] for more information,
xmvn tool not able to resolve dependencies:

[ERROR] Failed to execute goal on project leveldbjni-linux64: Could not resolve
dependencies for project org.fusesource.leveldbjni:leveldbjni-linux64:jar:1.8:
Cannot access central (http://repo.maven.apache.org/maven2) in offline mode and
the artifact org.fusesource.leveldbjni:leveldbjni:jar:1.8

[1] http://koji.fedoraproject.org/koji/taskinfo?taskID=6777121
http://koji.fedoraproject.org/koji/buildinfo?buildID=513000

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1075601] Review Request: ghc-exceptions - Extensible optionally-pure exceptions

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1075601



--- Comment #3 from Ricky Elrod  ---
Updated.

scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6779898

Spec URL:
http://codeblock.fedorapeople.org/packages/ghc-exceptions/ghc-exceptions.spec
SRPM URL:
http://codeblock.fedorapeople.org/packages/ghc-exceptions/ghc-exceptions-0.5-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 1060989] Review Request: ghc-io-streams - Simple, composable, easy-to-use stream I/O

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1060989

Ricky Elrod  changed:

   What|Removed |Added

  Alias||ghc-io-streams
  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 1091483] New: Review Request: python-pyprintr - Module that allows to emulate the print_r() PHP function

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091483

Bug ID: 1091483
   Summary: Review Request: python-pyprintr - Module that allows
to emulate the print_r() PHP function
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: e...@mayorgalinux.com
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://mayorga.fedorapeople.org/python-pyprintr.spec
SRPM URL: http://mayorga.fedorapeople.org/python-pyprintr-1.0-1.fc20.src.rpm
Description: pyprintr is a module that allows to emulate the print_r() function
of PHP
by printing the objects properties of a class instance and its internal
structure.
Fedora Account System Username: mayorga

rpmlint output:
[makerpm@localhost SPECS]$ rpmlint python-pyprintr.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

[makerpm@localhost SRPMS]$ rpmlint python-pyprintr-1.0-1.fc20.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091479] Review Request: pam-kwallet - PAM module for KWallet

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091479

Rex Dieter  changed:

   What|Removed |Added

 Blocks||656997 (kde-reviews)
  Alias||pam-kwallet




Referenced Bugs:

https://bugzilla.redhat.com/show_bug.cgi?id=656997
[Bug 656997] kde-related package 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 1091479] New: Review Request: pam-kwallet - PAM module for KWallet

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091479

Bug ID: 1091479
   Summary: Review Request: pam-kwallet - PAM module for KWallet
   Product: Fedora
   Version: rawhide
 Component: Package Review
  Severity: medium
  Priority: medium
  Assignee: nob...@fedoraproject.org
  Reporter: rdie...@math.unl.edu
QA Contact: extras...@fedoraproject.org
CC: package-review@lists.fedoraproject.org



Spec URL: http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet.spec
SRPM URL:
http://rdieter.fedorapeople.org/rpms/pam-kwallet/pam-kwallet-0-0.1.20140409git7604d424.fc20.src.rpm
Description: PAM module for kwallet
Fedora Account System Username: rdieter

This is the KDE/kwallet equivalent to gnome-keyring-pam

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 979166] Review Request: tora - Toolkit for Oracle, MySQL and PostreSQL

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=979166



--- Comment #13 from Pavel Alexeev (aka Pahan-Hubbitus)  
---
Great news!
Is it meant you are became on of co-author of antlr3cpp and there no more needs
to have separate copy in tora project?

-- 
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 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #5 from Jon Ciesla  ---
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 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052

Jon Ciesla  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 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392

Jon Ciesla  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 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392



--- Comment #10 from Jon Ciesla  ---
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 894608] Review Request: coin-or-SYMPHONY - Solver for mixed-integer linear programs

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894608



--- Comment #6 from Jon Ciesla  ---
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 894608] Review Request: coin-or-SYMPHONY - Solver for mixed-integer linear programs

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894608

Jon Ciesla  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 979166] Review Request: tora - Toolkit for Oracle, MySQL and PostreSQL

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=979166



--- Comment #12 from ibre5041  ---
At this moment it's OK. All mine patches(pull requests) for antlr3cpp are
applied in trunk. But in the future I'm going to work on this project too and
would like to use mine changes also in Tora project.

-- 
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 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392

Pierre-YvesChibon  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Pierre-YvesChibon  ---
Thanks for the review!

New Package SCM Request
===
Package Name: python-watchdog
Short Description: Filesystem events monitoring
Owners: pingou
Branches: F20, devel
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 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052

Luis Bazan  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 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #4 from Luis Bazan  ---
New Package SCM Request
===
Package Name: trac-blackmagictickettweaks-plugin
Short Description: Various hacks to alter the behavior of the ticket form
Owners: lbazan mayorga
Branches: f19 f20 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 1089213] Review Request: openstack-trove - OpenStack DBaaS

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089213

Pádraig Brady  changed:

   What|Removed |Added

 CC||pbr...@redhat.com



--- Comment #1 from Pádraig Brady  ---
Spec URL: http://fedorapeople.org/~pbrady/openstack-trove.spec
SRPM URL: http://pbrady.fedorapeople.org/openstack-trove-2014.1-1.fc20.src.rpm
Description: OpenStack DBaaS (codename trove) provisioning service.
Fedora Account System Username: pbrady

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052

Eduardo Mayorga  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #3 from Eduardo Mayorga  ---
Good!
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 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392

Antonio Trande  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #8 from Antonio Trande  ---
(In reply to Pierre-YvesChibon from comment #7)
> 
> Hopefully in the future the %check section will be un-commented out and will
> actually be used :)
> 

In the meantime, you may avoid to expand unused macros

#%%{__python2} setup.py test
#%%if 0%%{?with_python3}
#pushd %%{py3dir}
#%%{__python3} setup.py test
#popd
#%%endif

or removing the %check section completely.

> Spec URL: http://pingou.fedorapeople.org//python-watchdog.spec
> SRPM URL:
> http://pingou.fedorapeople.org//python-watchdog-0.7.1-4.fc20.src.rpm

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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #12 from Fedora Update System  ---
darkstat-3.0.718-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/darkstat-3.0.718-3.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091052] Review Request: trac-blackmagictickettweaks-plugin - Various hacks to alter the behavior of the ticket form.

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091052



--- Comment #2 from Luis Bazan  ---
Hi Mayorga

SPEC:
http://lbazan.fedorapeople.org/trac-blackmagictickettweaks-plugin.spec

SRPM:
http://lbazan.fedorapeople.org/trac-blackmagictickettweaks-plugin-0.12.2-2.20140425svn9962.fc20.src.rpm

SOURCE:
http://lbazan.fedorapeople.org/trac-blackmagictickettweaks-plugin-0.12.2.svn9962.tar.gz

Best Regards!

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392



--- Comment #7 from Pierre-YvesChibon  ---
Spec URL: http://pingou.fedorapeople.org//python-watchdog.spec
SRPM URL: http://pingou.fedorapeople.org//python-watchdog-0.7.1-4.fc20.src.rpm

I updated the license tag.

I won't ask for the EL6 branch now and will make sure to coordinate with the
maintainer of python-pytest-cov if I need the el6 branch.

Hopefully in the future the %check section will be un-commented out and will
actually be used :)


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 1091203] Review Request: clpeak - Find peak OpenCL capacities like bandwidth & compute

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091203



--- Comment #9 from Fedora Update System  ---
clpeak-0.1-2.20140425gitc0b94f2.fc19 has been submitted as an update for Fedora
19.
https://admin.fedoraproject.org/updates/clpeak-0.1-2.20140425gitc0b94f2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091203] Review Request: clpeak - Find peak OpenCL capacities like bandwidth & compute

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091203

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1091203] Review Request: clpeak - Find peak OpenCL capacities like bandwidth & compute

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1091203



--- Comment #8 from Fedora Update System  ---
clpeak-0.1-2.20140425gitc0b94f2.fc20 has been submitted as an update for Fedora
20.
https://admin.fedoraproject.org/updates/clpeak-0.1-2.20140425gitc0b94f2.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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #11 from Fedora Update System  ---
darkstat-3.0.718-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/darkstat-3.0.718-3.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362



--- Comment #8 from Fedora Update System  ---
zathura-cb-0.1.2-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/zathura-cb-0.1.2-3.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362



--- Comment #7 from Fedora Update System  ---
zathura-cb-0.1.2-3.fc19 has been submitted as an update for Fedora 19.
https://admin.fedoraproject.org/updates/zathura-cb-0.1.2-3.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 894608] Review Request: coin-or-SYMPHONY - Solver for mixed-integer linear programs

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=894608

Paulo Andrade  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from Paulo Andrade  ---
New Package SCM Request
===
Package Name: coin-or-SYMPHONY
Short Description: Solver for mixed-integer linear programs
Owners: pcpa
Branches: f20
InitialCC:

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539



--- Comment #10 from Jon Ciesla  ---
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 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539

Jon Ciesla  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 1069243] Review Request: ccaudio2 - C++ class framework for telephonic audio applications

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1069243



--- Comment #17 from Ralf Corsepius  ---
(In reply to David Sugar from comment #15)
> And my equally blunt response is that it may never get used in Fedora for
> that very reason, as people will most likely use the supplied and rather
> functional pkg-config and ccaudio2.pc file instead, so it will never "hit"
> anyone anyway. 
Just build your package for an architecture which is using a different set of
multilibs and your *-config will go up in smoke.

> However, to comply with historical gnu policies,
Well, I've never ever heard pkg-config was banned from GNU, nor did I say you
must use pkg-config. It's just that your *-config script lacks the required
amount of generality and that pkg-config is one common approach to fix this
problem.

However, provided how you reacted upon my remark, my proposal would be upstream
to remove the *-config script to avoid misleading people into the traps it set
up.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 415211] Review Request: WebKit - Web content engine library

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=415211

Vincent Danen  changed:

   What|Removed |Added

  Alias|WebKit  |



-- 
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 1065539] Review Request: subunit - C bindings for subunit

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065539

Jerry James  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #9 from Jerry James  ---
New Package SCM Request
===
Package Name: subunit
Short Description: C bindings for subunit
Owners: jjames
Branches: 
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 1088882] Review Request: python-humanize - Turns dates in to human readable format, e.g '3 minutes ago'

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=102



--- Comment #8 from Jon Ciesla  ---
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 1088882] Review Request: python-humanize - Turns dates in to human readable format, e.g '3 minutes ago'

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=102

Jon Ciesla  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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448

Jon Ciesla  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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #10 from Jon Ciesla  ---
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 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362



--- Comment #6 from Jon Ciesla  ---
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 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362

Jon Ciesla  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 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Lukas Bezdicka <255...@mail.muni.cz> changed:

   What|Removed |Added

  Flags||fedora-review+



--- Comment #3 from Lukas Bezdicka <255...@mail.muni.cz> ---
This seems to be fine, please proceed with SCM admin request and so on. Thank
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 1088882] Review Request: python-humanize - Turns dates in to human readable format, e.g '3 minutes ago'

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=102

Steve Traylen  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #7 from Steve Traylen  ---

Thank's for the review.


New Package SCM Request
===
Package Name: python-humanize
Short Description: Turns dates in to human readable format.
Owners: stevetraylen
Branches: f20 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 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362

François Cami  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #5 from François Cami  ---
New Package SCM Request
===
Package Name: zathura-cb
Short Description: Comic book support for zathura
Owners: fcami psabata
Branches: f19 f20 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 1088950] Review Request: perl-MouseX-SimpleConfig - A Mouse role for setting attributes from a simple configfile

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1088950



--- Comment #7 from Fedora Update System  ---
perl-MouseX-SimpleConfig-0.11-2.fc19 has been submitted as an update for Fedora
19.
https://admin.fedoraproject.org/updates/perl-MouseX-SimpleConfig-0.11-2.fc19

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1088950] Review Request: perl-MouseX-SimpleConfig - A Mouse role for setting attributes from a simple configfile

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1088950



--- Comment #8 from Fedora Update System  ---
perl-MouseX-SimpleConfig-0.11-2.fc20 has been submitted as an update for Fedora
20.
https://admin.fedoraproject.org/updates/perl-MouseX-SimpleConfig-0.11-2.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 1088950] Review Request: perl-MouseX-SimpleConfig - A Mouse role for setting attributes from a simple configfile

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1088950

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #9 from Christopher Meng  ---
(In reply to Pavel Alexeev (aka Pahan-Hubbitus) from comment #8)
> Christopher thank you very much for the review. And please say if you
> willing I review some of your packages. I've check several but not found any
> with working URLs.

They have been moved to misc.cicku.me but I haven't enabled the service so they
can't be accessed now.

I will update these bugs when they are ready, don't worry.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #7 from Christopher Meng  ---
(In reply to Pavel Alexeev (aka Pahan-Hubbitus) from comment #6)
> For what --disable-silent-rules needed? I have not seen any differences at
> first glance.

If you don't do that, the compiler/liker flags are hidden, we ca't know if they
are built correctly.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1083234] Review Request: mariadb-galera - MariaDB with wsrep patches

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1083234

Lukas Bezdicka <255...@mail.muni.cz> changed:

   What|Removed |Added

 CC||255...@mail.muni.cz



--- Comment #2 from Lukas Bezdicka <255...@mail.muni.cz> ---
Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed


Issues:
===
- Package installs properly.
  Note: Installation errors (see attachment)
  See: https://fedoraproject.org/wiki/Packaging:Guidelines
- Sources used to build the package match the upstream source, as provided in
  the spec URL.
  Note: Upstream MD5sum check error, diff is in /home/social/Downloads
  /mariadb-galera/diff.txt
  See: http://fedoraproject.org/wiki/Packaging/SourceURL


= MUST items =

C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[x]: Development (unversioned) .so files in -devel subpackage, if present.
 Note: Unversioned so-files in private %_libdir subdirectory (see
 attachment). Verify they are not in ld path.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[x]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "GPL", "LGPL (v2)", "LGPL (v2) (with incorrect FSF address)", "GPL (v2)
 (with incorrect FSF address)", "Public domain GPL (v2)", "Unknown or
 generated", "BSD (4 clause)", "GPL (v2 or later)", "ISC", "zlib/libpng",
 "BSD (3 clause)", "BSD (3 clause) GPL (v2)", "GPL (v2 or later) (with
 incorrect FSF address)", "BSD (2 clause)", "LGPL (v2.1 or later)", "*No
 copyright* Public domain", "GPL (v2)", "GPL (v3 or later)". 1181 files
 have unknown license. Detailed output of licensecheck in
 /home/social/Downloads/mariadb-galera/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: If the package is under multiple licenses, the licensing breakdown must
 be documented in the spec.
[x]: Package must own all directories that it creates.
 Note: Directories without known owners: /etc/logrotate.d
/etc/logrotate.d is shared among packages

[!]: Package does not own files or directories owned by other packages.
 Note: Dirs in package are owned also by: /usr/lib64/mysql/plugin
 (community-mysql-server, mariadb-server), /var/log/mariadb(mariadb-
 server), /var/run/mysqld(community-mysql-server, mariadb-server),
 /var/run/mariadb(mariadb-server), /etc/my.cnf.d(mariadb-libs),
 /var/lib/mysql(community-mysql-server, mariadb-server)
These paths are owned by our conflicts this package(mariadb-galera) is just
alternative mysql server

[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[x]: Each %files section contains %defattr if rpm < 4.4
 Note: %defattr present but not needed
[-]: Package contains desktop file if it is a GUI application.
[x]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[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.
[x]: 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.
[x]: Package contains systemd file(s) if in need.
[x]: Useful -debuginfo package or justification otherwise.
[x]: Package is not known to require an ExcludeArch tag.
[x]: Large documentation must go in a -doc subpackage. Large could be size
 (~1MB) or number of files.
 Note: Documentation size is 245760 bytes in 14 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
 supported primary architecture.
[x]: Rpmlint is run on all rpms the build produces.
 Note: There are rpmlint messages (see attachment).
[x]: If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s)
 for the package is included in %doc.
[x]: Package requires other packages for directories it uses.
[x]: All build dependencies are listed in BuildRequires, except for any that
 are listed in the exce

[Bug 1088882] Review Request: python-humanize - Turns dates in to human readable format, e.g '3 minutes ago'

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=102

Julien Enselme  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #6 from Julien Enselme  ---
>> - Since all currently maintained versions of Fedora have python3, there is 
>> no need for the with_python3 variable.
> I plan EPEL7 and 6 builds also where is is not available.
Point taken.


>> - Since the tests are not available in 0.5, I guess that you should remove 
>> the %check  macro
> I'd rather leave it as a reminder as something to do with next release.
Fine by me.

> fixes up the duplicates also.
Indeed, good job.

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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

  Flags||fedora-cvs?



--- Comment #8 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
Christopher thank you very much for the review. And please say if you willing I
review some of your packages. I've check several but not found any with working
URLs.

New Package SCM Request
===
Package Name: darkstat
Short Description: Network traffic analyzer
Owners: hubbitus
Branches: f19 f21 f21 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 1076448] Review Request: darkstat - Network traffic analyzer

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1076448



--- Comment #6 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
For what --disable-silent-rules needed? I have not seen any differences at
first glance.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1073362] Review Request: zathura-cb - Comic book support for zathura

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1073362

Petr Šabata  changed:

   What|Removed |Added

  Flags|fedora-review?  |fedora-review+



--- Comment #4 from Petr Šabata  ---
Great, approving.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1030154] Review Request: cmdfs - Command Filesystem

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1030154

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

 CC||pa...@hubbitus.info



--- Comment #3 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
$ LANG=C wget -c http://cicku.me/cmdfs.spec
--2014-04-25 17:13:34--  http://cicku.me/cmdfs.spec
Resolving cicku.me (cicku.me)... 37.59.54.136
Connecting to cicku.me (cicku.me)|37.59.54.136|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2014-04-25 17:13:34 ERROR 404: Not Found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1088281] Review Request: python-scales - Stats for Python processes

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1088281

Lubomir Rintel  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution|--- |NEXTRELEASE
Last Closed||2014-04-25 09:11:50



--- Comment #6 from Lubomir Rintel  ---
Imported and built.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1088283] Review Request: python-cassandra-driver - DataStax Python Driver for Apache Cassandra

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1088283
Bug 1088283 depends on bug 1088281, which changed state.

Bug 1088281 Summary: Review Request: python-scales - Stats for Python processes
https://bugzilla.redhat.com/show_bug.cgi?id=1088281

   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 1038593] Review Request: lftpfs - Filesystem with caching based on FUSE and LFTP

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1038593

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

 CC||pa...@hubbitus.info



--- Comment #1 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
$ LANG=C wget -c http://cicku.me/lftpfs.spec
--2014-04-25 17:11:04--  http://cicku.me/lftpfs.spec
Resolving cicku.me (cicku.me)... 37.59.54.136
Connecting to cicku.me (cicku.me)|37.59.54.136|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2014-04-25 17:11:04 ERROR 404: Not Found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1058587] Review Request: drawpile - A collaborative drawing program

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1058587

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

 CC||pa...@hubbitus.info



--- Comment #3 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
$ LANG=C wget -c http://cicku.me/drawpile.spec
--2014-04-25 17:10:22--  http://cicku.me/drawpile.spec
Resolving cicku.me (cicku.me)... 37.59.54.136
Connecting to cicku.me (cicku.me)|37.59.54.136|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2014-04-25 17:10:22 ERROR 404: Not Found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1065281] Review Request: xnbd - Yet another NBD (Network Block Device) server program

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1065281

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

 CC||pa...@hubbitus.info



--- Comment #1 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
$ LANG=C wget -c http://cicku.me/xnbd.spec
--2014-04-25 17:09:29--  http://cicku.me/xnbd.spec
Resolving cicku.me (cicku.me)... 37.59.54.136
Connecting to cicku.me (cicku.me)|37.59.54.136|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2014-04-25 17:09:29 ERROR 404: Not Found.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1077935] Review Request: mingw-taglib - Audio Meta-Data Library

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1077935



--- Comment #12 from Fedora Update System  ---
mingw-taglib-1.9.1-3.fc20 has been submitted as an update for Fedora 20.
https://admin.fedoraproject.org/updates/mingw-taglib-1.9.1-3.fc20

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1077935] Review Request: mingw-taglib - Audio Meta-Data Library

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1077935

Fedora Update System  changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED



-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 1089392] Review Request: python-watchdog - Filesystem events monitoring

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=1089392



--- Comment #6 from Antonio Trande  ---
1- There are files released with other licenses and packaged:

BSD (3 clause)
--
watchdog-0.7.1/%{py3dir}/src/watchdog/observers/winapi.py
watchdog-0.7.1/src/watchdog/observers/winapi.py

MIT/X11 (BSD like)
--
watchdog-0.7.1/%{py3dir}/src/watchdog/utils/importlib2.py
watchdog-0.7.1/%{py3dir}/src/watchdog/utils/unicode_paths.py
watchdog-0.7.1/src/watchdog/utils/importlib2.py
watchdog-0.7.1/src/watchdog/utils/unicode_paths.py

Please, fix License tag

2- Be silent 'macro-in-comment' warnings which come from %check section. 

3- 'python-pytest-cov' is not in EPEL6, so your package can't be built. 


Package Review
==

Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed



= MUST items =

Generic:
[x]: Package is licensed with an open-source compatible license and meets
 other legal requirements as defined in the legal section of Packaging
 Guidelines.
[!]: License field in the package spec file matches the actual license.
 Note: Checking patched sources after %prep for licenses. Licenses found:
 "Apache (v2.0)", "MIT/X11 (BSD like)", "Unknown or generated", "BSD (3
 clause)". 16 files have unknown license. Detailed output of licensecheck
 in /home/sagitter/1089392-python-watchdog/licensecheck.txt
[x]: License file installed when any subpackage combination is installed.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
 Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[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 61440 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

Python:
[x]: Python eggs must not download any dependencies during the build process.
[x]: A package which is used by another package via an egg interface should
 provide egg info.
[ ]: Package meets the Packaging Guidelines::Python
[x]: Package contains BR: python2-devel or python3-devel
[x]: Binary eggs must be removed in %prep

= SHOULD items =

Generic:
[-]: If the source package does not include license text(s) as a separate file
 from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Fully versioned dependency in subpackages if applicable.
 Note: No Requires: %{name}%{?_isa} = %{version}-%{release} in
 python3-watchdog
[ ]: Package functions as described.
[x

[Bug 986863] Review Request: ntopng - High-Speed Web-based Traffic Analysis and Flow Collection

2014-04-25 Thread bugzilla
https://bugzilla.redhat.com/show_bug.cgi?id=986863

Pavel Alexeev (aka Pahan-Hubbitus)  changed:

   What|Removed |Added

 CC||cicku...@gmail.com,
   ||pa...@hubbitus.info
  Flags||needinfo?(cicku...@gmail.co
   ||m)



--- Comment #5 from Pavel Alexeev (aka Pahan-Hubbitus)  ---
Do you interesting continue? Post working file URLs.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified 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   >