[Bug 1069615] Review Request: jenkins-sshd - Jenkins sshd module
https://bugzilla.redhat.com/show_bug.cgi?id=1069615 Michal Srb m...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks|652183 (FE-JAVASIG) | Fixed In Version||jenkins-sshd-1.6-1.fc21 Resolution|--- |RAWHIDE Last Closed||2014-03-25 02:09:59 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=652183 [Bug 652183] Java SIG tracker 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 1069702] Review Request: jenkins-ssh-credentials-plugin - Jenkins SSH Credentials Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1069702 Bug 1069702 depends on bug 1069615, which changed state. Bug 1069615 Summary: Review Request: jenkins-sshd - Jenkins sshd module https://bugzilla.redhat.com/show_bug.cgi?id=1069615 What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |RAWHIDE -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079890] Review Request: perl-REST-Client - A simple client for interacting with RESTful http/https resources
https://bugzilla.redhat.com/show_bug.cgi?id=1079890 David Dick dd...@cpan.org changed: What|Removed |Added Flags||fedora-cvs? --- Comment #3 from David Dick dd...@cpan.org --- New Package SCM Request === Package Name: perl-REST-Client Short Description: A simple client for interacting with RESTful http/https resources Owners: ddick Branches: f20 el6 epel7 InitialCC: perl-sig -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080054] Review Request: acegisecurity - Java/J2EE application security framework
https://bugzilla.redhat.com/show_bug.cgi?id=1080054 Michal Srb m...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #2 from Michal Srb m...@redhat.com --- Thanks! New Package SCM Request === Package Name: acegisecurity Short Description: Java/J2EE application security framework Owners: msrb sochotni mizdebsk msimacek Branches: InitialCC: java-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 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928
--- Comment #3 from Mohamed El Morabity pikachu.2...@gmail.com ---
Thank you Michael for this review ;).
(In reply to Christopher Meng from comment #2)
Requires: libcurl-devel
Should be
Requires: libcurl-devel%{?_isa}
Thanks you too Christopher for this comment. This will be fixed once the
package imported.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928 Mohamed El Morabity pikachu.2...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Mohamed El Morabity pikachu.2...@gmail.com --- New Package SCM Request === Package Name: dvblinkremote Short Description: Tool for interacting with a DVBLink Connect! Server Owners: melmorabity 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 1079616] Review Request: perl-MooX - Using Moo and MooX:: packages the most lazy way
https://bugzilla.redhat.com/show_bug.cgi?id=1079616 --- Comment #3 from Jean-Marie Renouard jmrenou...@gmail.com --- Hello Raph, ALL my apologies, I obtain similar results with Fedora 20. I will make several tests to check this module installation. Thank you for you' 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 1079616] Review Request: perl-MooX - Using Moo and MooX:: packages the most lazy way
https://bugzilla.redhat.com/show_bug.cgi?id=1079616 --- Comment #4 from Jean-Marie Renouard jmrenou...@gmail.com --- Hello Raph, ALL my apologies, I obtain similar results with Fedora 20. I will make several tests to check this module installation. Thank you for you' 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 1069702] Review Request: jenkins-ssh-credentials-plugin - Jenkins SSH Credentials Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=1069702 Michal Srb m...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks|652183 (FE-JAVASIG) | Fixed In Version||jenkins-ssh-credentials-plu ||gin-1.6.1-1.fc21 Resolution|--- |RAWHIDE Last Closed||2014-03-25 02:51:34 Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=652183 [Bug 652183] Java SIG tracker 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 1055246] Review Request: python-scrapy - A high-level Python Screen Scraping framework
https://bugzilla.redhat.com/show_bug.cgi?id=1055246 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1055246] Review Request: python-scrapy - A high-level Python Screen Scraping framework
https://bugzilla.redhat.com/show_bug.cgi?id=1055246 --- Comment #6 from Fedora Update System upda...@fedoraproject.org --- python-scrapy-0.22.0-1.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/python-scrapy-0.22.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 1079615] Review Request: perl-MooX-HandlesVia - NativeTrait-like behavior for Moo
https://bugzilla.redhat.com/show_bug.cgi?id=1079615 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 1079616] Review Request: perl-MooX - Using Moo and MooX:: packages the most lazy way
https://bugzilla.redhat.com/show_bug.cgi?id=1079616 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 1073508] Review Request: ghc-shelly - Shell-like systems programming in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1073508 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||ghc-shelly-0.15.4.1-1.fc19 Resolution|--- |ERRATA Last Closed||2014-03-25 04:13:06 --- Comment #8 from Fedora Update System upda...@fedoraproject.org --- ghc-shelly-0.15.4.1-1.fc19 has been pushed to the Fedora 19 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1025904] Review Request: solr - Ultra-fast Lucene-based Search Server
https://bugzilla.redhat.com/show_bug.cgi?id=1025904 Bug 1025904 depends on bug 1057670, which changed state. Bug 1057670 Summary: Review Request: jetty8 - jetty compatibility package (libs only) https://bugzilla.redhat.com/show_bug.cgi?id=1057670 What|Removed |Added Status|ON_QA |CLOSED Resolution|--- |ERRATA -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1057670] Review Request: jetty8 - jetty compatibility package (libs only)
https://bugzilla.redhat.com/show_bug.cgi?id=1057670 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||jetty8-8.1.14-4.fc20 Resolution|--- |ERRATA Last Closed||2014-03-25 04:14:21 --- Comment #19 from Fedora Update System upda...@fedoraproject.org --- jetty8-8.1.14-4.fc20 has been pushed to the Fedora 20 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1073508] Review Request: ghc-shelly - Shell-like systems programming in Haskell
https://bugzilla.redhat.com/show_bug.cgi?id=1073508 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|ghc-shelly-0.15.4.1-1.fc19 |ghc-shelly-1.4.4.2-1.fc20 --- Comment #9 from Fedora Update System upda...@fedoraproject.org --- ghc-shelly-1.4.4.2-1.fc20 has been pushed to the Fedora 20 stable repository. If problems still persist, please make note of it in this bug report. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810 --- Comment #5 from David King amigad...@amigadave.com --- Spec URL: http://amigadave.fedorapeople.org/gnome-sound-recorder.spec SRPM URL: http://amigadave.fedorapeople.org/gnome-sound-recorder-3.12.0-1.fc20.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671033 I added a comment above the Obsoletes, as well as a version according to https://fedoraproject.org/wiki/Packaging:Guidelines#Renaming.2FReplacing_Existing_Packages Updated to latest 3.12.0, which would be great to get into Fedora as part of the GNOME 3.12 release. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1022690] Review Request: gela-asis - Platform/compiler independent implementation of AdaSemantic Interface specification.
https://bugzilla.redhat.com/show_bug.cgi?id=1022690 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||gela-asis-0.3.2-2.fc20 Resolution|--- |ERRATA Last Closed||2014-03-25 04:20:35 --- Comment #15 from Fedora Update System upda...@fedoraproject.org --- gela-asis-0.3.2-2.fc20 has been pushed to the Fedora 20 stable repository. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079616] Review Request: perl-MooX - Using Moo and MooX:: packages the most lazy way
https://bugzilla.redhat.com/show_bug.cgi?id=1079616 --- Comment #5 from David Dick dd...@cpan.org --- Dependencies: FIX: BR perl(FindBin) # t/02-importparams.t line 8 FIX: BR perl(lib) # t/02-importparams.t line 9 TODO: add version to BR perl(ExtUtils::MakeMaker) = 6.30 # Makefile.PL line 7 This version is so old that it is irrelevant unless you wanted EL5, which this package is obviously not intended for. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079615] Review Request: perl-MooX-HandlesVia - NativeTrait-like behavior for Moo
https://bugzilla.redhat.com/show_bug.cgi?id=1079615 --- Comment #1 from Petr Pisar ppi...@redhat.com --- URL and Source0 are usable. Ok. Source archive is original (SHA-256: a69de0c201269a4ad2bfce51b200a91bf7f20c0a1d392f130185c8ae533f9360). Ok. Summary verified from README. Ok. TODO: I would speel `NativeTrait-like' as `Native-trait-like' or `NativeTraits-like' because `NativeTrait' is neither word nor Moose term (MouseX::NativeTraits). Description verified from README. Ok. License verified from README, LICENSE and various ./lib files. Ok. No XS code, noarch BuildArch is Ok. TODO: You can replace `PERL_INSTALL_ROOT' with `DESTDIR' argument in the %install section. TODO: Package either README or README.mkdn. They are redundant. TODO: Do not package dist.ini. There is nothing spectacular for the end users besides the git URL. TODO: Constrain `BuildRequire: perl(ExtUtils::MakeMaker)' with `= 6.30' (Makefile.PL:7). namespace::autoclean is optional for tests (t/unimport/namespace_autoclean.t:7). Ok. namespace::clean is optional for tests (t/unimport/namespace_clean.t:7). Ok. TODO: You should build-require both `perl(namespace::clean)' and `perl(namespace::autoclean)', or none for consistency. TODO: Build-require `perl(strict)' (Makefile.PL:2). TODO: Build-require `perl(warnings)' (Makefile.PL:3). FIX: (Build-)require Data::Perl::* modules instead of Data::Perl itself (Data::Perl::Role::Collection::Array, Data::Perl::Role::Collection::Hash, Data::Perl::Role::String, Data::Perl::Role::Number, Data::Perl::Role::Bool). These are loaded via Moo's `with' keyword. FIX: Do not Build-require `perl(Role::Tiny)'. It's not used anywhere. It's loaded via Role::Tiny::With. Data::Dumper is not used (t/from-moose/trait_hash.t:257). Ok. FIX: Build-require `perl(Moo::Role)' (t/handlesvia_in_role.t:7). TODO: Build-require `perl(overload)' (t/from-moose/trait_array.t:98). All tests pass. Ok. $ rpmlint perl-MooX-HandlesVia.spec ../SRPMS/perl-MooX-HandlesVia-0.001005-1.fc21.src.rpm ../RPMS/noarch/perl-MooX-HandlesVia-0.001005-1.fc21.noarch.rpm perl-MooX-HandlesVia.src: W: spelling-error %description -l en_US proxying - propping perl-MooX-HandlesVia.src: W: spelling-error %description -l en_US atttribute - attribute, attributive, attribution perl-MooX-HandlesVia.noarch: W: spelling-error %description -l en_US proxying - propping perl-MooX-HandlesVia.noarch: W: spelling-error %description -l en_US atttribute - attribute, attributive, attribution 2 packages and 1 specfiles checked; 0 errors, 4 warnings. FIX: Correct `atttribute' spelling. $ rpm -q -lv -p ../RPMS/noarch/perl-MooX-HandlesVia-0.001005-1.fc21.noarch.rpm drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/doc/perl-MooX-HandlesVia -rw-r--r--1 rootroot 700 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/Changes -rw-r--r--1 rootroot18410 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/LICENSE -rw-r--r--1 rootroot 1430 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/META.json -rw-r--r--1 rootroot 3231 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/README -rw-r--r--1 rootroot 3485 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/README.mkdn -rw-r--r--1 rootroot 242 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/TODO -rw-r--r--1 rootroot 907 Dec 20 18:24 /usr/share/doc/perl-MooX-HandlesVia/dist.ini -rw-r--r--1 rootroot 2217 Mar 25 09:33 /usr/share/man/man3/Data::Perl::Bool::MooseLike.3pm.gz -rw-r--r--1 rootroot 2453 Mar 25 09:33 /usr/share/man/man3/Data::Perl::Collection::Array::MooseLike.3pm.gz -rw-r--r--1 rootroot 2593 Mar 25 09:33 /usr/share/man/man3/Data::Perl::Collection::Hash::MooseLike.3pm.gz -rw-r--r--1 rootroot 2219 Mar 25 09:33 /usr/share/man/man3/Data::Perl::Number::MooseLike.3pm.gz -rw-r--r--1 rootroot 2600 Mar 25 09:33 /usr/share/man/man3/Data::Perl::String::MooseLike.3pm.gz -rw-r--r--1 rootroot 3213 Mar 25 09:33 /usr/share/man/man3/MooX::HandlesVia.3pm.gz drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/perl5/vendor_perl/Data drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/perl5/vendor_perl/Data/Perl drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/perl5/vendor_perl/Data/Perl/Bool -rw-r--r--1 rootroot 1308 Dec 20 18:24 /usr/share/perl5/vendor_perl/Data/Perl/Bool/MooseLike.pm drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/perl5/vendor_perl/Data/Perl/Collection drwxr-xr-x2 rootroot0 Mar 25 09:33 /usr/share/perl5/vendor_perl/Data/Perl/Collection/Array -rw-r--r--1 rootroot
[Bug 175623] Review Request: yaz - Z39.50/SRW/SRU programs
https://bugzilla.redhat.com/show_bug.cgi?id=175623 Christopher Meng cicku...@gmail.com changed: What|Removed |Added CC||cicku...@gmail.com Flags|fedora-cvs+ |fedora-review+ fedora-cvs? --- Comment #10 from Christopher Meng cicku...@gmail.com --- Package Change Request == Package Name: yaz New Branches: epel7 Owners: cicku -- 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 1033975] Review Request: python-enum34 - Backport of Python 3.4 Enum
https://bugzilla.redhat.com/show_bug.cgi?id=1033975 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Flags||fedora-review+ --- Comment #3 from Sergio Pascual sergio.pa...@gmail.com --- As far as I can see, this OK, 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 1058019] Review Request: utf8proc - Library for processing UTF-8 encoded Unicode strings
https://bugzilla.redhat.com/show_bug.cgi?id=1058019 --- Comment #7 from Milan Bouchet-Valat nalimi...@club.fr --- If you think I shouldn't add a SONAME, then the version above is ready for 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 1079484] Review Request: python-lazy - Lazy attributes for python objects
https://bugzilla.redhat.com/show_bug.cgi?id=1079484
Christopher Meng cicku...@gmail.com changed:
What|Removed |Added
Flags||fedora-review?
--- Comment #1 from Christopher Meng cicku...@gmail.com ---
%{__rm} -- rm
%{__python} -- %{__python2}
%{__make} -- make
%{__cp} -- cp
%{__mkdir_p} -- mkdir -p
%{python_sitelib} -- %{python2_sitelib}
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810 Kalev Lember kalevlem...@gmail.com changed: What|Removed |Added CC||kalevlem...@gmail.com --- Comment #6 from Kalev Lember kalevlem...@gmail.com --- I think it would make sense to obsolete gnome-media as well, in addition to the gnome-media-apps package. gnome-media was just a small package containing translations that gnome-media-apps used; adding the obsoletes would ensure that both packages get removed during upgrades. No need for provides here: these are needed to replace a package that something else might have a dependency on. As an example where provides are needed, if we renamed gnome-desktop3-devel to libgnome-desktop3-devel, we should most definitely add the provides so that packages that buildrequire the old name continue working. Also, obsoletes should be versioned so that if there's ever a need to add gnome-media back, or some other app wants to use the name, it's possible to do so. Obsoletes:gnome-media 3.4.0-7 Obsoletes:gnome-media-apps 3.4.0-7 P.S. What's going on with the review here? I see Christopher has assigned the ticket to himself, while saying he has no time right now? :) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810 --- Comment #7 from David King amigad...@amigadave.com --- Thanks for the information Kalev, I made those changes and edited the spec and SRPM in place. i am not quite sure what is going on with the review, but I reviewed Christopher's package in bug 1065610, which is just waiting on a license clarification. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961 gil cattaneo punto...@libero.it changed: What|Removed |Added CC||punto...@libero.it Assignee|nob...@fedoraproject.org|punto...@libero.it 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 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961
gil cattaneo punto...@libero.it changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flags|fedora-review? |fedora-review+
--- Comment #3 from gil cattaneo punto...@libero.it ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
[ ] = Manual review needed
Issues:
===
- This seems like a Java package, please install fedora-review-plugin-java to
get additional checks
= 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.
[!]: 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]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 10 files have unknown license. Detailed output of
licensecheck in /home/gil/1079961-eclipse-m2e-mavendev/licensecheck.txt
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: Package contains desktop file if it is a GUI application.
[-]: Development files must be in a -devel package
[x]: Package uses nothing in %doc for runtime.
[x]: Package consistently uses macros (instead of hard-coded directory names).
[x]: Package is named according to the Package Naming Guidelines.
[x]: Package does not generate any conflict.
[x]: Package obeys FHS, except libexecdir and /usr/target.
[-]: If the package is a rename of another package, proper Obsoletes and
Provides are present.
[x]: Requires correct, justified where necessary.
[x]: Spec file is legible and written in American English.
[-]: Package contains systemd file(s) if in need.
[x]: Package is not known to require an ExcludeArch tag.
[-]: Large documentation must go in a -doc subpackage. Large could be size
(~1MB) or number of files.
Note: Documentation size is 20480 bytes in 2 files.
[x]: Package complies to the Packaging Guidelines
[x]: Package successfully compiles and builds into binary rpms on at least one
supported primary architecture.
[x]: Package installs properly.
[x]: Rpmlint is run on all rpms the build produces.
Note: There are rpmlint messages (see attachment).
[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
Java:
[x]: Bundled jar/class files should be removed before build
= 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.
[!]: Package does not include license text files separate from upstream.
[x]: Patches link to upstream bugs/comments/lists or are otherwise justified.
[x]: 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 should not be in spec file
[x]: Sources can be downloaded from URI in Source: tag
[x]: Reviewer should test that the package builds in mock.
[x]: Buildroot is not present
[x]:
[Bug 1075774] Review Request: nss-altfiles - NSS module to look up users in /usr/lib/passwd too
https://bugzilla.redhat.com/show_bug.cgi?id=1075774
--- Comment #5 from Christopher Meng cicku...@gmail.com ---
Issues:
Rpmlint (installed packages)
# rpmlint nss-altfiles
nss-altfiles.i686: W: spelling-error Summary(en_US) passwd - passed, password
nss-altfiles.i686: W: unexpanded-macro %description -l C %{prefix}
nss-altfiles.i686: W: unexpanded-macro %description -l C %{prefix}
nss-altfiles.i686: W: no-version-in-last-changelog
nss-altfiles.i686: E: library-not-linked-against-libc
/usr/lib/libnss_altfiles.so.2
1 packages and 0 specfiles checked; 1 errors, 5 warnings.
# echo 'rpmlint-done:'
Suggestions:
1. %setup -q -n nss-altfiles-%{version} -- %setup -q
2. Source0: https://github.com/aperezdc/nss-altfiles/archive/v2.18.1.tar.gz
--
Source0: https://github.com/aperezdc/nss-altfiles/archive/v%{version}.tar.gz
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810
--- Comment #8 from Kalev Lember kalevlem...@gmail.com ---
By the way, the = comparison in 'gnome-media = 3.4.0-7' doesn't do the
expected thing, because 3.4.0-7 != 3.4.0-7%{?dist}
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225810] Merge Review: gnome-applets
https://bugzilla.redhat.com/show_bug.cgi?id=225810 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED CC||panem...@gmail.com Resolution|--- |NOTABUG Last Closed||2014-03-25 06:45:13 --- Comment #1 from Parag AN(पराग) panem...@gmail.com --- The master branch dead.package says Package is retired, replaced by the GNOME Classic session in F19 Let's close this then. -- 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 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810 --- Comment #9 from David King amigad...@amigadave.com --- Thanks again! I just updated the spec to use the '=' syntax instead of =', is that 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 1063040] Review Request: rubygem-em-socksify - Transparent proxy support for any EventMachine protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1063040
Ken Dreyer ktdre...@ktdreyer.com changed:
What|Removed |Added
Flags||fedora-review?
--- Comment #12 from Ken Dreyer ktdre...@ktdreyer.com ---
Hi Nitesh, a couple of additional comments:
- The MIT-LICENSE file in rubygem-em-socksify-0.3.0-5.fc20.src.rpm says
Copyright (c) 2009 - 2014, David Waite and Other Contributors. This is not
correct. The main contributor is Ilya Grigorik, and the first commit in Git is
from 2011 (not 2009).
We need to be very careful that we do not accidentally indicate that copyright
belongs to the wrong person.
(Please see the next item below for a corrected copyright statement in the
MIT-LICENSE file.)
- When you see that the MIT license file is missing from upstream, the first
place to attempt to solve that is upstream, rather than downstream in Fedora.
I've submitted this change here:
https://github.com/igrigorik/em-socksify/pull/15
In the future, if you are packaging software that does not have the full
license text, please submit this sort of change upstream first.
- For Ruby Gem packages, it's also useful for upstream to have the MIT license
metadata in the gemspec. Here's the proposal for that:
https://github.com/igrigorik/em-socksify/pull/14 . Again, in the future, please
do submit this sort of thing upstream for your packages.
- Please update the URL to use https instead of http. I've proposed this
upstream here: https://github.com/igrigorik/em-socksify/pull/16
- Please remove the Generated from ... gem2rpm comment at the top of the
file. This line is not necessary for the package and it will become stale over
time.
- Please remove the trailing spaces at the end of the rubygems-devel line (line
17) and popd line (line 56):
17 BuildRequires: rubygems-devel
56 #popd
- Please remove the following line:
BuildRequires: ruby
This is only necessary on the arch-specific (binary) gems. Since your
em-socksify package is noarch, you only need ruby(release).
- Please adjust the following line:
%{gem_instdir}/spec/*
to be simply
%{gem_instdir}/spec
This will tell RPM to capture the spec directory as well as the contents
within. Please remove the trailing whitespace from this line as well.
- Please add a second percent sign (%) to the macros that are commented in
%check. So %check becomes %%check, and ./%{gem_instdir} becomes
./%%{gem_instdir}. The reason for doing this is that rpmlint will complain
about unescaped macros in the comments. Even if a macro is commented, RPM will
still expand it. Sometimes its innocuous and it's not a problem, but other
times it can cause unexpected behavior, so it's always best to escape all the
rpmlint warnings about this to be on the safe side.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1077810] Review Request: gnome-sound-recorder - Make short recordings from your desktop
https://bugzilla.redhat.com/show_bug.cgi?id=1077810 --- Comment #10 from Kalev Lember kalevlem...@gmail.com --- Yep, looks good to me. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 988102] Review Request: perl-DBD-Sybase - this package provides an interface for perl to connect with Sybase and MSSql databases
https://bugzilla.redhat.com/show_bug.cgi?id=988102 David Dick dd...@cpan.org changed: What|Removed |Added CC||dd...@cpan.org --- Comment #17 from David Dick dd...@cpan.org --- Just a thought. Rather than throw the test suite away completely, you could replace make test with perl -Iblib/lib -Iblib/arch t/base.t which would prove that at least the build has succeeded and the Sybase shared object can be loaded. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1069070] Review Request: ghc-QuickCheck - Automatic testing of Haskell programs
https://bugzilla.redhat.com/show_bug.cgi?id=1069070
Christopher Meng cicku...@gmail.com changed:
What|Removed |Added
Flags|fedora-review? |fedora-review+
--- Comment #3 from Christopher Meng cicku...@gmail.com ---
Package Review
==
Legend:
[x] = Pass, [!] = Fail, [-] = Not applicable, [?] = Not evaluated
= MUST items =
C/C++:
[x]: Package does not contain kernel modules.
[x]: Package contains no static executables.
[-]: Development (unversioned) .so files in -devel subpackage, if present.
Note: Unversioned so-files in private %_libdir subdirectory (see
attachment). Verify they are not in ld path.
[x]: Package does not contain any libtool archives (.la)
[x]: Rpath absent or only used for internal libs.
Generic:
[x]: Package is licensed with an open-source compatible license and meets
other legal requirements as defined in the legal section of Packaging
Guidelines.
[x]: License field in the package spec file matches the actual license.
Note: Checking patched sources after %prep for licenses. Licenses found:
Unknown or generated. 14 files have unknown license. Detailed output of
licensecheck:
Unknown or generated
QuickCheck-2.6/Setup.lhs
QuickCheck-2.6/Test/QuickCheck.hs
QuickCheck-2.6/Test/QuickCheck/All.hs
QuickCheck-2.6/Test/QuickCheck/Arbitrary.hs
QuickCheck-2.6/Test/QuickCheck/Exception.hs
QuickCheck-2.6/Test/QuickCheck/Function.hs
QuickCheck-2.6/Test/QuickCheck/Gen.hs
QuickCheck-2.6/Test/QuickCheck/Modifiers.hs
QuickCheck-2.6/Test/QuickCheck/Monadic.hs
QuickCheck-2.6/Test/QuickCheck/Poly.hs
QuickCheck-2.6/Test/QuickCheck/Property.hs
QuickCheck-2.6/Test/QuickCheck/State.hs
QuickCheck-2.6/Test/QuickCheck/Test.hs
QuickCheck-2.6/Test/QuickCheck/Text.hs
[x]: License file installed when any subpackage combination is installed.
[x]: %build honors applicable compiler flags or justifies otherwise.
[x]: Package contains no bundled libraries without FPC exception.
[x]: Changelog in prescribed format.
[x]: Sources contain only permissible code or content.
[-]: 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.
[-]: 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]: 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 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
Haskell:
[x]: This should never happen
= SHOULD items =
Generic:
[x]: If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[x]: Final provides and requires are sane (see attachments).
[x]: Package functions as described.
[x]: Latest version is packaged.
[x]: Package does not
[Bug 1028206] Review Request: tayga - nat64 implementation for linux
https://bugzilla.redhat.com/show_bug.cgi?id=1028206
--- Comment #4 from Christopher Meng cicku...@gmail.com ---
(In reply to William Brown from comment #3)
Sorry about the delay getting back to you.
Initially, the policy should be in this package, and once it's stable I'll
send to to selinux-policy.
You should open a new bug under selinux-policy and tell the maintainer that a
new package will need some works.
If they deny to adopt, then it's time ship it in this package.
How do you suggest to integrate with systemd? I'm not sure that it supports
network interfaces ... unless you mean to suggest that we run nat64 as a
service over a network interface? (Given this doesn't create an interface
with an IP that is certainly an option to make services such as
tayga@.service).
It should support it well.
%{_mandir}/man5/%{name}.conf.5.gz
%{_mandir}/man8/%{name}.8.gz
--
%{_mandir}/man5/%{name}.conf.5*
%{_mandir}/man8/%{name}.8*
Remove %defattr(-,root,root,0755)
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1072054] Review Request: gnome-code-assistance - Common code assistance services for code editors
https://bugzilla.redhat.com/show_bug.cgi?id=1072054 --- Comment #3 from Kalev Lember kalevlem...@gmail.com --- gnome-code-assistance 0.3.1 is out now; can you update the packaging please? -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 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 #2 from Parag AN(पराग) panem...@gmail.com --- Review: + Package built successfully in mock rawhide + rpmlint on generated rpms gave output python-fedmsg-genacls.noarch: W: spelling-error Summary(en_US) gitosis - mitosis, halitosis python-fedmsg-genacls.noarch: W: spelling-error Summary(en_US) acls - aces, ails, acts python-fedmsg-genacls.noarch: W: spelling-error Summary(en_US) pkgdb - pkg db, pkg-db, pkg python-fedmsg-genacls.noarch: W: spelling-error %description -l en_US pkgdb - pkg db, pkg-db, pkg python-fedmsg-genacls.src: W: spelling-error Summary(en_US) gitosis - mitosis, halitosis python-fedmsg-genacls.src: W: spelling-error Summary(en_US) acls - aces, ails, acts python-fedmsg-genacls.src: W: spelling-error Summary(en_US) pkgdb - pkg db, pkg-db, pkg python-fedmsg-genacls.src: W: spelling-error %description -l en_US pkgdb - pkg db, pkg-db, pkg 2 packages and 0 specfiles checked; 0 errors, 8 warnings. + Source verified with upstream as (sha256sum) srpm tarball:321d70af11cef939a6d665dd32accdec0e0efedf2c1b36723b038ddcf70c5ff4 upstream tarball:321d70af11cef939a6d665dd32accdec0e0efedf2c1b36723b038ddcf70c5ff4 + License is valid and included in LICENSE file + rest looks as per packaging guidelines Suggestions: 1) Either own /etc/fedmsg.d or add Requires: on package that owns that directory but I see fedmsg package should actually own it, please fix fedmsg package. 2) If this is only fedora spec then you don't need Group tag and python conditional macro definition at top of spec. Otherwise follow EPEL specific guideliens like buildroot, %clean, defattr, removal of buildroot in %install 3) cp or install commands should follow -p to preserve timestamp of upstream files being copied in the system. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080411] New: Review Request: trojita - Qt IMAP e-mail client
https://bugzilla.redhat.com/show_bug.cgi?id=1080411 Bug ID: 1080411 Summary: Review Request: trojita - Qt IMAP e-mail client Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: kvo...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://kvolny.fedorapeople.org/trojita.spec SRPM URL: http://kvolny.fedorapeople.org/trojita-0.4.1-1.fc20.src.rpm Description: Trojitá is a Qt IMAP e-mail client which: * Enables you to access your mail anytime, anywhere. * Does not slow you down. If we can improve the productivity of an e-mail user, we better do. * Respects open standards and facilitates modern technologies. We value the vendor-neutrality that IMAP provides and are committed to be as interoperable as possible. * Is efficient — be it at conserving the network bandwidth, keeping memory use at a reasonable level or not hogging the system's CPU. * Can be used on many platforms. One UI is not enough for everyone, but our IMAP core works fine on anything from desktop computers to cell phones and big ERP systems. * Plays well with the rest of the ecosystem. We don't like reinventing wheels, but when the existing wheels quite don't fit the tracks, we're not afraid of making them work. Fedora Account System Username: kvolny Koji scratch builds: F20: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671258 EPEL7: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671502 Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=6671309 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080411] Review Request: trojita - Qt IMAP e-mail client
https://bugzilla.redhat.com/show_bug.cgi?id=1080411 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||cicku...@gmail.com Assignee|nob...@fedoraproject.org|cicku...@gmail.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 175623] Review Request: yaz - Z39.50/SRW/SRU programs
https://bugzilla.redhat.com/show_bug.cgi?id=175623 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 175623] Review Request: yaz - Z39.50/SRW/SRU programs
https://bugzilla.redhat.com/show_bug.cgi?id=175623 --- Comment #11 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 611372] Review Request: perl-Net-Twitter - Perl interface to Twitter
https://bugzilla.redhat.com/show_bug.cgi?id=611372 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 958358] Review Request: darkhttpd - A secure, lightweight, fast, single-threaded HTTP/1.1 server
https://bugzilla.redhat.com/show_bug.cgi?id=958358 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 611372] Review Request: perl-Net-Twitter - Perl interface to Twitter
https://bugzilla.redhat.com/show_bug.cgi?id=611372 --- Comment #30 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 958358] Review Request: darkhttpd - A secure, lightweight, fast, single-threaded HTTP/1.1 server
https://bugzilla.redhat.com/show_bug.cgi?id=958358 --- Comment #14 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 1079890] Review Request: perl-REST-Client - A simple client for interacting with RESTful http/https resources
https://bugzilla.redhat.com/show_bug.cgi?id=1079890 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 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928 --- Comment #5 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 1079890] Review Request: perl-REST-Client - A simple client for interacting with RESTful http/https resources
https://bugzilla.redhat.com/show_bug.cgi?id=1079890 --- 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 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928 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 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961 Mikolaj Izdebski mizde...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Mikolaj Izdebski mizde...@redhat.com --- New Package SCM Request === Package Name: eclipse-m2e-mavendev Short Description: Maven Development Tools Owners: mizdebsk mizdebsk msrb sochotni Branches: f20 InitialCC: java-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 1080411] Review Request: trojita - Qt IMAP e-mail client
https://bugzilla.redhat.com/show_bug.cgi?id=1080411 --- Comment #1 from Karel Volný kvo...@redhat.com --- note that I've filed some bugs upstream for the things I've workarounded in spec: Bug 332578 - superfluous x-test language [https://bugs.kde.org/show_bug.cgi?id=332578] Bug 332579 - unversioned plugins library (rpmlint invalid-soname issue) [https://bugs.kde.org/show_bug.cgi?id=332579] -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080054] Review Request: acegisecurity - Java/J2EE application security framework
https://bugzilla.redhat.com/show_bug.cgi?id=1080054 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 1080054] Review Request: acegisecurity - Java/J2EE application security framework
https://bugzilla.redhat.com/show_bug.cgi?id=1080054 --- 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 1079616] Review Request: perl-MooX - Using Moo and MooX:: packages the most lazy way
https://bugzilla.redhat.com/show_bug.cgi?id=1079616 --- Comment #6 from Ralf Corsepius rc040...@freenet.de --- (In reply to David Dick from comment #5) Good catches - BR:s updated accordingly Updated package: Spec URL: http://corsepiu.fedorapeople.org/packages/perl-MooX.spec SRPM URL: http://corsepiu.fedorapeople.org/packages/perl-MooX-0.101-2.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961 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 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961 --- Comment #5 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 1079484] Review Request: python-lazy - Lazy attributes for python objects
https://bugzilla.redhat.com/show_bug.cgi?id=1079484 --- Comment #2 from David Shea ds...@redhat.com --- Fixed. Spec URL: http://dshea.fedorapeople.org/python-lazy/v2/python-lazy.spec SRPM URL: http://dshea.fedorapeople.org/python-lazy/v2/python-lazy-1.1-1.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1074147] Review Request: apt-cacher-ng - Caching proxy for package files from Debian
https://bugzilla.redhat.com/show_bug.cgi?id=1074147 --- Comment #11 from Kenjiro Nakayama knaka...@redhat.com --- (In reply to Kenjiro Nakayama from comment #10) I got an advice about my ticket. [1] It seems that I should use SHA1 which is included in OpenSSl or some other packages. Altough I will keep waiting for the FPC's comment, I try to use and test SHA1 which is included in Open SSL. [1] https://lists.fedoraproject.org/pipermail/packaging/2014-March/010106.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 1079961] Review Request: eclipse-m2e-mavendev - Maven Development Tools
https://bugzilla.redhat.com/show_bug.cgi?id=1079961 Mikolaj Izdebski mizde...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |NEXTRELEASE Last Closed||2014-03-25 09:14:46 --- Comment #6 from Mikolaj Izdebski mizde...@redhat.com --- buildsys.build.state.change -- mizdebsk's eclipse-m2e-mavendev-0.2.0.201311250404-1.fc21 completed http://koji.fedoraproject.org/koji/buildinfo?buildID=506831 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
[Bug 1080456] Package Change Request: thunderbird EPEL7 branch
https://bugzilla.redhat.com/show_bug.cgi?id=1080456 Jan Horak jho...@redhat.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 1080456] New: Package Change Request: thunderbird EPEL7 branch
https://bugzilla.redhat.com/show_bug.cgi?id=1080456 Bug ID: 1080456 Summary: Package Change Request: thunderbird EPEL7 branch Product: Fedora Version: rawhide Component: Package Review Assignee: nob...@fedoraproject.org Reporter: jho...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Package Change Request == Package Name: thunderbird New Branches: epel7 Owners: gecko-maint xhorak stransky We'd like to introduce Thunderbird package to EPEL7 as long as it's not going to be part of RHEL7. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079928] Review Request: dvblinkremote - Tool for interacting with a DVBLink Connect! Server
https://bugzilla.redhat.com/show_bug.cgi?id=1079928 --- Comment #6 from Fedora Update System upda...@fedoraproject.org --- dvblinkremote-0.2.0-0.1.beta.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/dvblinkremote-0.2.0-0.1.beta.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 1080456] Package Change Request: thunderbird EPEL7 branch
https://bugzilla.redhat.com/show_bug.cgi?id=1080456 --- Comment #1 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 1080456] Package Change Request: thunderbird EPEL7 branch
https://bugzilla.redhat.com/show_bug.cgi?id=1080456 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 225817] Merge Review: gnome-games
https://bugzilla.redhat.com/show_bug.cgi?id=225817 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED CC||panem...@gmail.com Resolution|--- |NOTABUG Last Closed|2010-07-08 00:42:51 |2014-03-25 09:41:16 --- Comment #5 from Parag AN(पराग) panem...@gmail.com --- Package is retired in f20, closing this review. -- 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 533887] Review Request: raw-thumbnailer - thumbnailer for raw images on gnome
https://bugzilla.redhat.com/show_bug.cgi?id=533887 Mohamed El Morabity pikachu.2...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution|--- |CURRENTRELEASE Last Closed|2010-04-19 09:27:24 |2014-03-25 09:59:59 -- 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 226527] Merge Review: vino
https://bugzilla.redhat.com/show_bug.cgi?id=226527 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? -- 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 1070482] Review Request: reaver-wps - Brute force attack against Wifi Protected Setup
https://bugzilla.redhat.com/show_bug.cgi?id=1070482 Jaroslav Škarvada jskar...@redhat.com changed: What|Removed |Added Blocks|177841 (FE-NEEDSPONSOR) | --- Comment #18 from Jaroslav Škarvada jskar...@redhat.com --- I am sponsor now. I will add sponsored status to your FAS account once we progress through this review. Referenced Bugs: https://bugzilla.redhat.com/show_bug.cgi?id=177841 [Bug 177841] Tracker: Review requests from new Fedora packagers who need a sponsor -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080498] Review Request: golang-github-smarterclayton-go-systemd - Go bindings to systemd socket activation, journal and D-BUS APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1080498 Lokesh Mandvekar l...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED 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 1080498] Review Request: golang-github-smarterclayton-go-systemd - Go bindings to systemd socket activation, journal and D-BUS APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1080498 Lokesh Mandvekar l...@redhat.com changed: What|Removed |Added Assignee|nob...@fedoraproject.org|vba...@redhat.com -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1059121] Review Request: js-yui2 - Yahoo User Interface JavaScript library
https://bugzilla.redhat.com/show_bug.cgi?id=1059121 Michael Simacek msima...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||msima...@redhat.com Assignee|nob...@fedoraproject.org|msima...@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 1080498] Review Request: golang-github-smarterclayton-go-systemd - Go bindings to systemd socket activation, journal and D-BUS APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1080498 --- Comment #1 from Lokesh Mandvekar l...@redhat.com --- Spec URL: http://lsm5.fedorapeople.org/golang-github-smarterclayton-go-systemd/golang-github-smarterclayton-go-systemd.spec SRPM URL: http://lsm5.fedorapeople.org/golang-github-smarterclayton-go-systemd/SRPMS/golang-github-smarterclayton-go-systemd-0-0.2.git5cb9e9e.fc21.src.rpm go test commented out for now, but otherwise shows this: + GOPATH=/usr/share/gocode:/home/lsm5/repositories/pkgs/fedora/golang-github-smarterclayton-go-systemd/BUILDROOT/golang-github-smarterclayton-go-systemd-0-0.2.git5cb9e9e.fc21.x86_64//usr/share/gocode + go test github.com/smarterclayton/go-systemd/dbus --- FAIL: TestStartStopUnit (0.01 seconds) methods_test.go:57: Expected one change, got %v [] --- FAIL: TestSubscriptionSetUnit (0.01 seconds) methods_test.go:57: Expected one change, got %v [] --- FAIL: TestSubscribeUnit (0.01 seconds) methods_test.go:57: Expected one change, got %v [] FAIL FAILgithub.com/smarterclayton/go-systemd/dbus0.267s error: Bad exit status from /var/tmp/rpm-tmp.2gcWzP (%check) -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 --- Comment #3 from Ralph Bean rb...@redhat.com --- 1) Either own /etc/fedmsg.d or add Requires: on package that owns that directory but I see fedmsg package should actually own it, please fix fedmsg package. fedmsg fixed in rawhide now. 2) If this is only fedora spec then you don't need Group tag and python conditional macro definition at top of spec. Otherwise follow EPEL specific guideliens like buildroot, %clean, defattr, removal of buildroot in %install Okay -- this will be for el6, but not el5. I think that means that defining the buildroot, %clean, defattr, removal of the buildroot, etc.. all are not required. 3) cp or install commands should follow -p to preserve timestamp of upstream files being copied in the system. Fixed in this release: Spec URL: http://threebean.org/rpm/SPECS/python-fedmsg-genacls.spec SRPM URL: http://threebean.org/rpm/SRPMS/python-fedmsg-genacls-0.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 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #4 from Parag AN(पराग) panem...@gmail.com --- Looks good now. 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 1080498] New: Review Request: golang-github-smarterclayton-go-systemd - Go bindings to systemd socket activation, journal and D-BUS APIs
https://bugzilla.redhat.com/show_bug.cgi?id=1080498 Bug ID: 1080498 Summary: Review Request: golang-github-smarterclayton-go-systemd - Go bindings to systemd socket activation, journal and D-BUS APIs Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: l...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://lsm5.fedorapeople.org/golang-github-smarterclayton-go-systemd/golang-github-smarterclayton-go-systemd.spec SRPM URL: http://lsm5.fedorapeople.org/golang-github-smarterclayton-go-systemd/SRPMS/golang-github-smarterclayton-go-systemd-0-0.1.git5cb9e9e.fc21.src.rpm Description: Go bindings to systemd socket activation, journal and D-BUS APIs Fedora Account System Username: lsm5 Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=6672004 $ rpmlint golang-github-smarterclayton-go-systemd.spec RPMS/noarch/golang-github-smarterclayton-go-systemd-devel-0-0.1.git5cb9e9e.fc21.noarch.rpm SRPMS/golang-github-smarterclayton-go-systemd-0-0.1.git5cb9e9e.fc21.src.rpm golang-github-smarterclayton-go-systemd.spec:38: W: setup-not-quiet golang-github-smarterclayton-go-systemd.src:38: W: setup-not-quiet 2 packages and 1 specfiles checked; 0 errors, 2 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 1070946] Review Request: python-SimpleCV - Open source framework for building computer vision applications
https://bugzilla.redhat.com/show_bug.cgi?id=1070946 --- Comment #9 from Matthias Runge mru...@redhat.com --- Julien, no matter what matplotlib does, please follow the fonts policy. https://fedoraproject.org/wiki/Packaging:FontsPolicy As a side note: that package seems to need some love: https://admin.fedoraproject.org/pkgdb/acls/bugs/python-matplotlib and esp. that font issue is https://bugzilla.redhat.com/show_bug.cgi?id=477445 The issue is now more than 5 years old. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 947071] Review Request: monitorix - A free, open source, lightweight system monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=947071 Christopher Meng cicku...@gmail.com changed: What|Removed |Added Flags|fedora-cvs+ |fedora-cvs? --- Comment #34 from Christopher Meng cicku...@gmail.com --- Package Change Request == Package Name: monitorix New Branches: epel7 Owners: cicku mikaku -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 947071] Review Request: monitorix - A free, open source, lightweight system monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=947071 --- Comment #35 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 947071] Review Request: monitorix - A free, open source, lightweight system monitoring tool
https://bugzilla.redhat.com/show_bug.cgi?id=947071 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 1063040] Review Request: rubygem-em-socksify - Transparent proxy support for any EventMachine protocol
https://bugzilla.redhat.com/show_bug.cgi?id=1063040 --- Comment #13 from Nitesh Narayan Lal niteshnara...@fedoraproject.org --- Hi, Thanks for your comments, here are the changed files: SPEC:http://niteshnarayan.fedorapeople.org/SPECS/rubygem-em-socksify.spec SRPM:http://niteshnarayan.fedorapeople.org/SRPMS/rubygem-em-socksify-0.3.0-6.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 1079615] Review Request: perl-MooX-HandlesVia - NativeTrait-like behavior for Moo
https://bugzilla.redhat.com/show_bug.cgi?id=1079615 --- Comment #2 from Ralf Corsepius rc040...@freenet.de --- (In reply to Petr Pisar from comment #1) TODO: I would speel `NativeTrait-like' as `Native-trait-like' or `NativeTraits-like' because `NativeTrait' is neither word nor Moose term (MouseX::NativeTraits). Not changed: This is upstream's description, they are using inside of the package and in CPAN TODO: You can replace `PERL_INSTALL_ROOT' with `DESTDIR' argument in the %install section. I kept PERL_INSTALL_ROOT. TODO: Package either README or README.mkdn. They are redundant. TODO: Do not package dist.ini. There is nothing spectacular for the end users besides the git URL. Done. This was an oversight on my part. TODO: Constrain `BuildRequire: perl(ExtUtils::MakeMaker)' with `= 6.30' (Makefile.PL:7). Done. TODO: You should build-require both `perl(namespace::clean)' and `perl(namespace::autoclean)', or none for consistency. Done. BR'ing both, now. TODO: Build-require `perl(strict)' (Makefile.PL:2). TODO: Build-require `perl(warnings)' (Makefile.PL:3). Done. FIX: (Build-)require Data::Perl::* modules instead of Data::Perl itself (Data::Perl::Role::Collection::Array, Data::Perl::Role::Collection::Hash, Data::Perl::Role::String, Data::Perl::Role::Number, Data::Perl::Role::Bool). These are loaded via Moo's `with' keyword. Done under protest. All this does it to bloat the dependency tree with redundant deps at no technical benefits. FIX: Do not Build-require `perl(Role::Tiny)'. It's not used anywhere. It's loaded via Role::Tiny::With. Done. FIX: Build-require `perl(Moo::Role)' (t/handlesvia_in_role.t:7). Done. TODO: Build-require `perl(overload)' (t/from-moose/trait_array.t:98). Done. Updated package: Spec URL: http://corsepiu.fedorapeople.org/packages/perl-MooX-HandlesVia.spec SRPM URL: http://corsepiu.fedorapeople.org/packages/perl-MooX-HandlesVia-0.001005-2.fc21.src.rpm -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080583] New: Review Request: compat-qpid-cpp - Compatibility modules for Qpid
https://bugzilla.redhat.com/show_bug.cgi?id=1080583 Bug ID: 1080583 Summary: Review Request: compat-qpid-cpp - Compatibility modules for Qpid Product: Fedora Version: rawhide Component: Package Review Severity: medium Priority: medium Assignee: nob...@fedoraproject.org Reporter: dpie...@redhat.com QA Contact: extras...@fedoraproject.org CC: package-review@lists.fedoraproject.org Spec URL: http://mcpierce.fedorapeople.org/rpms/compat-qpid-cpp.spec SRPM URL: http://mcpierce.fedorapeople.org/rpms/compat-qpid-cpp-0.24-1.fc20.src.rpm Description: Libraries for Qpid C++ client applications Fedora Account System Username: mcpierce -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 Ralph Bean rb...@redhat.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #5 from Ralph Bean rb...@redhat.com --- New Package SCM Request === Package Name: python-fedmsg-genacls Short Description: A fedmsg consumer that sets gitosis acls in response to pkgdb messages Owners: ralph janeznemanic Branches: f20 f19 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 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 --- Comment #6 from Ralph Bean rb...@redhat.com --- Apologies. janeznemanic is not actually in the packagers group. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1080094] Review Request: python-fedmsg-genacls - A fedmsg consumer that sets gitosis acls in response to pkgdb messages
https://bugzilla.redhat.com/show_bug.cgi?id=1080094 --- Comment #7 from Ralph Bean rb...@redhat.com --- New Package SCM Request === Package Name: python-fedmsg-genacls Short Description: A fedmsg consumer that sets gitosis acls in response to pkgdb messages Owners: ralph Branches: f20 f19 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 1078960] Review Request: mino - A user friendly text editor for the Linux console/xterm
https://bugzilla.redhat.com/show_bug.cgi?id=1078960 Jon Bernard jober...@redhat.com changed: What|Removed |Added CC||jober...@redhat.com --- Comment #2 from Jon Bernard jober...@redhat.com --- Package looks good, here are a few things that could be improved: * Groups should be Applications/Editors and not Editors * The Source0 URL gives 404 Not Found * There are two spelling mistakes in the description: - 'syntaxes' should probably be 'syntax' - 'akward' should be 'awkward' -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1075822] Review Request: openstack-marconi - OpenStack Message Queuing Service
https://bugzilla.redhat.com/show_bug.cgi?id=1075822 --- Comment #7 from Jon Bernard jober...@redhat.com --- Here is one for Mino: https://bugzilla.redhat.com/show_bug.cgi?id=1078960#c2 -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1079436] Review Request: jomara - openstack tuskar ui
https://bugzilla.redhat.com/show_bug.cgi?id=1079436 John Eckersberg jecke...@redhat.com changed: What|Removed |Added CC||jecke...@redhat.com Assignee|nob...@fedoraproject.org|jecke...@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 1020839] Review Request: fedora-gooey-karma - GUI tool for adding karma to Bodhi system
https://bugzilla.redhat.com/show_bug.cgi?id=1020839 --- Comment #27 from Fedora Update System upda...@fedoraproject.org --- fedora-gooey-karma-0.1-4.fc19 has been submitted as an update for Fedora 19. https://admin.fedoraproject.org/updates/fedora-gooey-karma-0.1-4.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 1020839] Review Request: fedora-gooey-karma - GUI tool for adding karma to Bodhi system
https://bugzilla.redhat.com/show_bug.cgi?id=1020839 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1020839] Review Request: fedora-gooey-karma - GUI tool for adding karma to Bodhi system
https://bugzilla.redhat.com/show_bug.cgi?id=1020839 --- Comment #28 from Fedora Update System upda...@fedoraproject.org --- fedora-gooey-karma-0.1-4.fc20 has been submitted as an update for Fedora 20. https://admin.fedoraproject.org/updates/fedora-gooey-karma-0.1-4.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 1071507] Review Request python-tuskarclient - Python bindings and CLI to Tuskar
https://bugzilla.redhat.com/show_bug.cgi?id=1071507 John Eckersberg jecke...@redhat.com changed: What|Removed |Added Flags|fedora-review? |fedora-review+ --- Comment #6 from John Eckersberg jecke...@redhat.com --- Looks good. 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 1033975] Review Request: python-enum34 - Backport of Python 3.4 Enum
https://bugzilla.redhat.com/show_bug.cgi?id=1033975 Eric Smith space...@gmail.com changed: What|Removed |Added Flags||fedora-cvs? --- Comment #4 from Eric Smith space...@gmail.com --- New Package SCM Request === Package Name: python-enum34 Short Description: Backport of Python 3.4 Enum Owners: brouhaha 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 988866] Review Request:php-pecl-event - Provides interface to libevent library
https://bugzilla.redhat.com/show_bug.cgi?id=988866 Adam Williamson awill...@redhat.com changed: What|Removed |Added Flags||fedora-review+ --- Comment #15 from Adam Williamson awill...@redhat.com --- [adamw@adam SRPMS]$ rpmlint php-pecl-event-1.9.1-1.remi.src.rpm php-pecl-event.src: W: spelling-error Summary(en_US) libevent - lib event, lib-event, enlivenment php-pecl-event.src: W: spelling-error %description -l en_US libevent - lib event, lib-event, enlivenment php-pecl-event.src: E: unknown-key GPG#00f97f56 1 packages and 0 specfiles checked; 1 errors, 2 warnings. [adamw@adam x86_64]$ rpmlint php-pecl-event-* php-pecl-event.x86_64: W: spelling-error Summary(en_US) libevent - lib event, lib-event, enlivenment php-pecl-event.x86_64: W: spelling-error %description -l en_US libevent - lib event, lib-event, enlivenment 2 packages and 0 specfiles checked; 0 errors, 2 warnings. Those are all fine, the GPG key obviously won't happen in the Fedora build system. All MUST guidelines at https://fedoraproject.org/wiki/Packaging:ReviewGuidelines are fine. SHOULD guidelines look good too. Review is APPROVED. Is it really correct to ship the tests as docs? Maybe a separate subpackage would be better? Just a thought, not blocking review. -- 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 1071490] Review Request: openstack-tuskar - API for managing the deployment of OpenStack
https://bugzilla.redhat.com/show_bug.cgi?id=1071490
John Eckersberg jecke...@redhat.com changed:
What|Removed |Added
Flags||needinfo?(jom...@redhat.com
||)
--- Comment #3 from John Eckersberg jecke...@redhat.com ---
This bit still needs addressed:
[!]: Package meets the Packaging Guidelines::Python
See note %{__python} deprecated at
https://fedoraproject.org/wiki/Packaging:Python#Macros
The latest upstream appears to be 0.2.2 now:
https://pypi.python.org/pypi/tuskar
Assuming that's the right upstream source, you should update the Source0 line
to point to pypi.
Still a lot of rpmlint errors:
Checking: openstack-tuskar-0.1.0-3.fc21.noarch.rpm
openstack-tuskar-0.1.0-3.fc21.src.rpm
openstack-tuskar.noarch: E: explicit-lib-dependency python-amqplib
openstack-tuskar.noarch: W: non-standard-group Application/System
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/cmd/manager.py 0644L /usr/bin/env
openstack-tuskar.noarch: W: non-conffile-in-etc /etc/tuskar/tuskar.wsgi
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/cmd/api.py 0644L /usr/bin/env
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/openstack/common/rootwrap/cmd.py 0644L
/usr/bin/env
openstack-tuskar.noarch: E: non-standard-dir-perm /etc/tuskar 0750L
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/common/service.py 0644L /usr/bin/env
openstack-tuskar.noarch: E: non-readable /etc/tuskar/tuskar.conf 0640L
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/openstack/common/rpc/zmq_receiver.py
0644L /usr/bin/env
openstack-tuskar.noarch: E: non-executable-script
/usr/lib/python2.7/site-packages/tuskar/cmd/dbsync.py 0644L /usr/bin/env
openstack-tuskar.noarch: W: no-manual-page-for-binary tuskar-api
openstack-tuskar.noarch: W: no-manual-page-for-binary tuskar-dbsync
openstack-tuskar.noarch: W: no-manual-page-for-binary tuskar-manager
openstack-tuskar.src: W: non-standard-group Application/System
2 packages and 0 specfiles checked; 9 errors, 6 warnings.
Lemme know if you have any specific questions on those errors. I won't speak
to them all for now.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 1036320] Review Request: nftables - Netfilter Tables userspace utillites
https://bugzilla.redhat.com/show_bug.cgi?id=1036320 --- Comment #21 from Kevin Fenzi ke...@scrye.com --- Thanks for the review! 1. See earlier discussion about the /etc/nftables/ scripts. I think in this case it's valid to have them executable and still marked as config. 2. That isn't something this configure script understands: configure: WARNING: unrecognized options: --disable-silent-rules I could patch the doc building to be more verbose, but how important is it? It shouldn't be needed to see compiler flags or anything... 3. It doesn't currently. It's just the command line utils. Perhaps someday it would grow some functionality around that, but it might never do so and just depend on things that use it at a higher level like firewalld to save persistent rules. -- You are receiving this mail because: You are on the CC list for the bug. You are always notified about changes to this product and component ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
