[Bug 638906] Review Request: erlang-gen_leader - A leader election behavior modeled after gen_server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638906 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:22:59 EDT --- erlang-gen_leader-0-0.2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638582] Review Request: jpanoramamaker -
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638582 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|jpanoramamaker-5-2.fc14 |jpanoramamaker-5-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635515] Review Request: libphidget - Drivers and API for Phidget devices
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635515 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:28:01 EDT --- libphidget-2.1.7.20100621-5.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638582] Review Request: jpanoramamaker -
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638582 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:27:32 EDT --- jpanoramamaker-5-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635515] Review Request: libphidget - Drivers and API for Phidget devices
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635515 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|libphidget-2.1.7.20100621-5 |libphidget-2.1.7.20100621-5 |.fc14 |.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 621015] Review Request: rubygem-authlogic - A simple ruby authentication solution
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=621015 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:29:00 EDT --- rubygem-authlogic-2.1.6-4.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-authlogic'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-authlogic-2.1.6-4.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 640557] Review Request: smartcardpp - Library for accessing smart cards
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=640557 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:29:43 EDT --- smartcardpp-0.2.0-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 577398] Review Request: python-meliae - Python memory usage statistics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=577398 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:31:08 EDT --- python-meliae-0.2.0-3.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 640550] Review Request: libdigidoc - Library for handling digitally signed documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=640550 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:31:39 EDT --- libdigidoc-2.7.0-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630179] Review Request: cloud - Cloud.com CloudStack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630179 --- Comment #10 from Chen Lei supercyp...@gmail.com 2010-10-14 02:29:19 EDT --- Hi Rudd-O, You should submit one package per Review Request. I also suggest you to join Java SIG[1] and introduce cloud there in java-sig maillist to accelerate the review of the whole cloud stack. [1] https://fedoraproject.org/wiki/SIGs/Java -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 577398] Review Request: python-meliae - Python memory usage statistics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=577398 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version||python-meliae-0.2.0-3.fc13 Resolution|NEXTRELEASE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 637489] Review Request: perl-KinoSearch1 - Search engine library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=637489 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:35:45 EDT --- perl-KinoSearch1-1.00-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635788] Review Request: nautilus-terminal - Terminal embedded in Nautilus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635788 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:36:40 EDT --- nautilus-terminal-0.7-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 637489] Review Request: perl-KinoSearch1 - Search engine library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=637489 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-KinoSearch1-1.00-2.fc1 |perl-KinoSearch1-1.00-2.fc1 |4 |3 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638906] Review Request: erlang-gen_leader - A leader election behavior modeled after gen_server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638906 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|erlang-gen_leader-0-0.2.fc1 |erlang-gen_leader-0-0.2.fc1 |2 |3 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635788] Review Request: nautilus-terminal - Terminal embedded in Nautilus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635788 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:38:45 EDT --- nautilus-terminal-0.7-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624471] Review Request: perl-Lingua-EN-Tagger - Part-of-speech tagger for English natural language processing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624471 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:38:04 EDT --- perl-Lingua-EN-Tagger-0.16-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638906] Review Request: erlang-gen_leader - A leader election behavior modeled after gen_server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638906 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:35:14 EDT --- erlang-gen_leader-0-0.2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635788] Review Request: nautilus-terminal - Terminal embedded in Nautilus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635788 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|nautilus-terminal-0.7-1.fc1 |nautilus-terminal-0.7-1.fc1 |4 |3 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635788] Review Request: nautilus-terminal - Terminal embedded in Nautilus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635788 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|nautilus-terminal-0.7-1.fc1 |nautilus-terminal-0.7-1.fc1 |3 |2 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635515] Review Request: libphidget - Drivers and API for Phidget devices
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635515 --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:37:51 EDT --- libphidget-2.1.7.20100621-5.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 579593] Review Request: python3-cherrypy - Pythonic, object-oriented web development framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=579593 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-10-14 02:38:21 EDT --- python3-cherrypy-3.2.0-0.rc1.r2567.1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635515] Review Request: libphidget - Drivers and API for Phidget devices
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635515 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|libphidget-2.1.7.20100621-5 |libphidget-2.1.7.20100621-5 |.fc13 |.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 624471] Review Request: perl-Lingua-EN-Tagger - Part-of-speech tagger for English natural language processing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=624471 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Lingua-EN-Tagger-0.16- |perl-Lingua-EN-Tagger-0.16- |2.fc14 |2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 579593] Review Request: python3-cherrypy - Pythonic, object-oriented web development framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=579593 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version||python3-cherrypy-3.2.0-0.rc ||1.r2567.1.fc13 Resolution|NEXTRELEASE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 464141] Review Request: perl-Text-Trac - Perl extension for formatting text with Trac Wiki Style
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=464141 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Status|ASSIGNED|NEW AssignedTo|mmasl...@redhat.com |nob...@fedoraproject.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531107 --- Comment #15 from Marcela Mašláňová mmasl...@redhat.com 2010-10-14 03:04:35 EDT --- Ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641967] Review Request: aether - Sonatype library to resolve, install and deploy artifacts the Maven way
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641967 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2010-10-14 03:46:21 --- Comment #7 from Stanislav Ochotnicky socho...@redhat.com 2010-10-14 03:46:21 EDT --- Package built. Thanks for fast repos http://koji.fedoraproject.org/koji/taskinfo?taskID=2532655 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 640992] Review Request: google-guice - Lightweight dependency injection framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=640992 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2010-10-14 03:53:40 --- Comment #7 from Stanislav Ochotnicky socho...@redhat.com 2010-10-14 03:53:40 EDT --- Thanks for the fast repos. Package built, closing. http://koji.fedoraproject.org/koji/taskinfo?taskID=2534238 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 570318] Review Request: gnuplot42 - A program for plotting mathematical expressions and data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=570318 --- Comment #8 from Steve Traylen steve.tray...@cern.ch 2010-10-14 04:15:33 EDT --- Thanks again for the review. I've now updated the parralel package to be gnuplot44 http://cern.ch/straylen/rpms/gnuplot42/gnuplot44.spec http://cern.ch/straylen/rpms/gnuplot42/gnuplot44-4.4.2-2.el5.src.rpm other than the upstream update there is not really any significant difference in the package other than being rebased on the current F15 gnuplot. I'm confident that the review of gnuplot42 is applicable and relevant to this new gnuplot44. I'll leave this package here for a week in case there is comment before making the git request. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642547] Review Request: ucd - Property data for the Unicode data set
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642547 --- Comment #4 from Jorge Enrique Barrera jo...@fbarr.net 2010-10-14 04:48:47 EDT --- As an afterthought, would it perhaps be better to rename the project to 'unicode-data', for a more descriptive packagename? Also, perhaps change the path '/usr/share/ucd' to '/usr/share/unicode'? I'd appreciate some feedback. :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 570318] Review Request: gnuplot44 - A program for plotting mathematical expressions and data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=570318 Mark Chappell trem...@tremble.org.uk changed: What|Removed |Added Summary|Review Request: gnuplot42 - |Review Request: gnuplot44 - |A program for plotting |A program for plotting |mathematical expressions|mathematical expressions |and data|and data Alias|gnuplot42 |gnuplot44 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638647] Review Request: mom - Dynamically manage system resources on virtualization hosts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=638647
--- Comment #5 from Xavier Bachelot xav...@bachelot.org 2010-10-14 05:04:16
EDT ---
I'm not a sponsor, so I won't be able to approve the package, but here
are a couple more comments.
- I'd rather see the patch split into 2 patches, one for the initscript
and one for the spec. Actually, the spec part is useless, as the bundled
spec file is not used.
- The initscript part of the patch turns 2 lines into 1 which then will
span over 80 characters, please fix.
- Patches need to have an upstream reference (bug/commit/...).This is
intended to help keep track of the patches. As you're upstream and this
will probably make it in the leext release, this is not a big deal, but
I thought I'd mention it.
- The %build and %install section are not following the template. The build
part should not be done from within the %install section. See
/etc/rpmdevtools/spectemplate-python.spec.
- The install of the initscript could be made shorter :
install -Dp contrib/momd.init $RPM_BUILD_ROOT/%_initddir/momd
- No need to copy the LICENSE and README files to the docdir, just using
%doc LICENSE README in the %files section will be enough.
- The INSTALL file doesn't contain any useful content, it's not needed
to include it currently. This might be changed later if/when its content
evolves.
- Use macros everywhere possible. When moving the examples, replace
/usr/share/doc by %{_defaultdocdir}.
- It might be better to alter setup.py to install the doc to the proper
location rather than move them after install.
- Be consistent with macros. You use both %_initddir and
%{_defaultdocdir}. Choose one style and stick with it. The preferred
style is usually %{_sbindir}.
- No need for %{_defaultdocdir}/%{name}-%{version} in the %files
section, this will automatically be picked up.
- Adding a blank line between changelog entries improve readability.
- BuildRoot and %clean are not needed anymore with recent Fedora
releases. However, this is still needed for EL5, so keep it if you want
to also have EPEL branches. In this case, you'll also want to replace
%{_initddir} by %{_initrddir}.
- There are some useless comments in the spec (first line and in %post).
- Please consider providing a default/dummy conf file and mark it as
%config(noreplace) in the %files section.
- You might want to use %{version} and possibly %{name] macros in the
Source0 tag, so you don't have to change the version in 2 places when
upgrading to a later version.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 591192] Review Request: dh-make - Tool that converts source archives into Debian package source
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=591192
Miroslav Suchý msu...@redhat.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review+
--- Comment #7 from Miroslav Suchý msu...@redhat.com 2010-10-14 06:06:51 EDT
---
install -m 644 -p dh_make.1 %{buildroot}/%{_mandir}/man1/.
That dot at the end of line is not needed. But that is minor.
Anyway - it look good now.
APPROVED
You may request git branch, but please do not build it till debhelper and
dpkg-devel pass the package review or you will end up with broken dependency.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:04:42 EDT --- rubygem-simple-navigation-3.0.0-3.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-simple-navigation-3.0.0-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:09:45 EDT --- rubygem-simple-navigation-3.0.0-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-simple-navigation-3.0.0-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:15:07 EDT --- rubygem-simple-navigation-3.0.0-3.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-simple-navigation-3.0.0-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:23:47 EDT --- rubygem-rb-inotify-0.8.1-2.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:31:26 EDT --- rubygem-rb-inotify-0.8.1-2.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:28:45 EDT --- rubygem-rb-inotify-0.8.1-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:32:03 EDT --- rubygem-rb-inotify-0.8.1-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #12 from Michal Fojtik mfoj...@redhat.com 2010-10-14 06:30:00 EDT --- Dammit, sorry I enter a wrong bugzilla ID in Bodhi for rb-inotify. I'm apologize for that. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:33:44 EDT --- rubygem-rb-inotify-0.8.1-2.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:35:06 EDT --- rubygem-rb-inotify-0.8.1-2.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639617] Review Request: rubygem-cucumber-rails - Cucumber Generators and Runtime for Rails
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639617 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639617] Review Request: rubygem-cucumber-rails - Cucumber Generators and Runtime for Rails
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639617 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:38:26 EDT --- rubygem-cucumber-rails-0.3.2-5.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/rubygem-cucumber-rails-0.3.2-5.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:35:06 EDT --- rubygem-rb-inotify-0.8.1-2.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc12 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2010-10-14 06:35:06 EDT --- rubygem-rb-inotify-0.8.1-2.fc12 has been submitted as an update for Fedora 12. https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641295] Review Request: rubygem-typhoeus - A library for interacting with web services at blinding speed
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=641295
--- Comment #6 from Michal Fojtik mfoj...@redhat.com 2010-10-14 06:49:25 EDT
---
(In reply to comment #5)
For -2:
* Requires
- Requires: libcurl is redundant and should be removed, because
rpmbuild detects and adds this dependency automatically based on
the dependency of the libraries:
https://fedoraproject.org/wiki/Packaging/Guidelines#Requires
Fixed. Thanks for info btw, I didn't know that there is an autodetection.
* Applying patch
- It is better that patches are applied in %prep using %patchX
macro.
Agree. Fixed.
? Tests needing Rails or so to be running
- Well, I am not sure if it is preferable to remove tests
which needs Rails or so to be running.
- When people makes Rails or so running, he/she may want
to check typhoeus using these test scripts.
Maybe it is better that
- we don't remove such tests
- and ignore the failure case for now
- Or:
--
%check
pushd .%{geminstdir}
NEEDSKIP=
for needskip in \
spec/typhoeus/request_spec.rb \
spec/typhoeus/hydra_spec.rb \
spec/typhoeus/remote_spec.rb \
spec/typhoeus/multi_spec.rb \
spec/typhoeus/easy_spec.rb \
spec/typhoeus/remote_proxy_object_spec.rb
do
NEEDSKIP=$NEEDSKIP $needskip
done
for needskip in $NEEDSKIP
do
mv $needskip ${needskip}.save
done
rake spec --trace
for needskip in $NEEDSKIP
do
mv ${needskip}.save $needskip
done
Fixed. Thanks for contributing this. I fully agree that those tests should be
preserved in -doc package for futher testing.
--
! By the way, $ rake spec needs BuildRequires: rubygem(rake).
Fixed.
* Consistent macro usage
- Please use macros consistently. For example:
---
%files
%dir %{ruby_sitearch}/%{gemname}
%{ruby_sitearch}/typhoeus/*.so
---
please use %{gemname} also in the third line.
Fixed.
* Miscs
---
56 %install
57 rm -rf %{buildroot}
58 mkdir -p %{buildroot}%{ruby_sitearch}/%{gemname}
59 mkdir -p %{buildroot}%{gemdir}
60 mkdir -p %{buildroot}%{_prefix}
---
- The line 68 is unneeded.
Fixed.
=== 0.1.31-3
* Thu Oct 14 2010 Michal Fojtik mfoj...@redhat.com - 0.1.31-3
- Preserved failing test files (thx. to mtasaka)
- Fixed macros usage
- Replaced path with macro
- Removed libcurl from requires
Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2534502
Spec URL: http://mifo.sk/RPMS/rubygem-typhoeus.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-typhoeus-0.1.31-3.fc13.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642524] Review Request: rubygem-net-sftp - A pure Ruby implementation of the SFTP client protocol
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642524
--- Comment #3 from Michal Fojtik mfoj...@redhat.com 2010-10-14 07:01:57 EDT
---
(In reply to comment #1)
Taking this one. Overall looks good with some small nits
* rpmlint complains about mixing spaces / tabs, please remove the tab on line
25
Summary: Documentation for %{name}
Fixed.
Now:
rpmlint rubygem-net-sftp-2.0.5-2.fc13.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint rubygem-net-sftp-doc-2.0.5-2.fc13.noarch.rpm
rubygem-net-sftp-doc.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
rpmlint rubygem-net-sftp-2.0.5-2.fc13.src.rpm
rubygem-net-sftp.src: W: no-buildroot-tag
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
* Can you please change %source0 to point to the official rubygems.org gem
Fixed.
* The license of the documentation file should be MIT and LGPLv2 as the
setup.rb is licensed under the LGPL version 2.1
Fixed. I write down a comment in %files to mark setup.rb file as LGPL
These are the only blockers, though I was also wondering why you specify
1.2 for the rubygems dependency (not wrong, just haven't see this before).
You're right. I removed that version dependency.
Also if you have a moment the review guidelines state If the source package
does not include license text(s) as a separate file from upstream, the
packager
SHOULD query upstream to include it, not a biggie though.
Sure, will do :-)
Rpmlint looks fine (with the exception above), the package builds fine on
koji,
and I did a surface functionality test.
Once the three blockers have been resolved about, this package is ready.
APPROVED rubygem-net-sftp [mmorsi]
Thanks for help and review!
* Thu Oct 14 2010 Michal Fojtik mfoj...@redhat.com - 2.0.5-2
- Fixed license
- Fixes source0 URL
Spec URL: http://mifo.sk/RPMS/rubygem-net-sftp.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-net-sftp-2.0.5-2.fc13.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642524] Review Request: rubygem-net-sftp - A pure Ruby implementation of the SFTP client protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642524 Michal Fojtik mfoj...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Michal Fojtik mfoj...@redhat.com 2010-10-14 07:02:50 EDT --- New Package SCM Request === Package Name: rubygem-net-sftp Short Description: A pure Ruby implementation of the SFTP client protocol Owners:mfojtik Branches: f12 f13 f14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 637939] Review Request: rubygem-gdk_pixbuf2 - Ruby binding of GdkPixbuf-2.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=637939 Michal Fojtik mfoj...@redhat.com changed: What|Removed |Added CC||mfoj...@redhat.com QAContact|extras...@fedoraproject.org |mfoj...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 640627] Review Request: rubygem-factory_girl - Framework and DSL for defining and using model instance factories
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=640627
--- Comment #4 from Michal Fojtik mfoj...@redhat.com 2010-10-14 07:30:15 EDT
---
(In reply to comment #3)
Everything looks good save the patch for the Rakefile, the proper way to do it
would be something like this in the %prep section:
%setup -q -c -T
mkdir -p ./%{gemdir}
gem install --local --install-dir ./%{gemdir} \
--force --rdoc %{SOURCE0}
pushd ./%{geminstdir}
%patch0
popd
and then in the %install section
%install
rm -rf %{buildroot}
mkdir -p %{buildroot}%{gemdir}
cp -a .%{gemdir}/* %{buildroot}%{gemdir}
See rubygem-activerecord as an example of how to do this
Thanks for noticing that, it looks definitely better right now.
Fixed.
http://pkgs.fedoraproject.org/gitweb/?p=rubygem-activerecord.git;a=blob;f=rubygem-activerecord.spec
Other than that, everything else looks great. rpmlint is fine, it builds on
koji, and I did a surface functionality test.
1.3.2-2
* Thu Oct 14 2010 Michal Fojtik mfoj...@redhat.com - 1.3.2-3
- Replaced path with path macro
Spec URL: http://mifo.sk/RPMS/rubygem-factory_girl.spec
SRPM URL: http://mifo.sk/RPMS/rubygem-factory_girl-1.3.2-2.fc13.src.rpm
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642985] New: Review Request: rubygem-timecop - Provides a unified method to mock Time.now, Date.today in a single call
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-timecop - Provides a unified method to mock Time.now, Date.today in a single call https://bugzilla.redhat.com/show_bug.cgi?id=642985 Summary: Review Request: rubygem-timecop - Provides a unified method to mock Time.now, Date.today in a single call Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mfoj...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://mifo.sk/RPMS/rubygem-timecop.spec SRPM URL: http://mifo.sk/RPMS/rubygem-timecop-0.3.5-1.fc13.src.rpm Description: A gem providing time travel and time freezing capabilities, making it dead simple to test time-dependent code. It provides a unified method to mock Time.now, Date.today, and DateTime.now in a single call. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642985] Review Request: rubygem-timecop - Provides a unified method to mock Time.now, Date.today in a single call
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642985 --- Comment #1 from Michal Fojtik mfoj...@redhat.com 2010-10-14 07:52:58 EDT --- Some additional notes: Koji build failed for me for the first time I tried to build this package. This seems like Koji issue/Xen issue with time. I tested it on my two machines and it worked fine. I disabled these asserts in tests for now using patch. (Kernel bug reported here: #640608) Loaded suite test_time_stack_item Started FFF. Finished in 0.026321 seconds. 1) Failure: test_compute_dst_adjustment_for_dst_to_dst(TestTimeStackItem) [test_time_stack_item.rb:100]: precondition. false is not true. 2) Failure: test_compute_dst_adjustment_for_dst_to_non_dst(TestTimeStackItem) [test_time_stack_item.rb:118]: precondition. false is not true. 3) Failure: test_compute_dst_adjustment_for_non_dst_to_dst(TestTimeStackItem) [test_time_stack_item.rb:128]: precondition. false is not true. 16 tests, 53 assertions, 3 failures, 0 errors After that: http://koji.fedoraproject.org/koji/taskinfo?taskID=2534587 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642985] Review Request: rubygem-timecop - Provides a unified method to mock Time.now, Date.today in a single call
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642985 Michal Fojtik mfoj...@redhat.com changed: What|Removed |Added See Also||https://bugzilla.redhat.com ||/show_bug.cgi?id=640608 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641390] Review Request: sisu - Sonatype dependency injection framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641390 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||akurt...@redhat.com AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com Flag||fedora-review? --- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 07:55:35 EDT --- I'll do this one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641390] Review Request: sisu - Sonatype dependency injection framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641390 --- Comment #2 from Stanislav Ochotnicky socho...@redhat.com 2010-10-14 08:03:33 EDT --- Re-uploaded the spec/srpm to fix changelog and describe how to get tarball. README is not packaged because it's an empty file and therefore useless (plus rpmlint would complain). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642995] New: Review Request: rubygem-deltacloud-client - Deltacloud REST Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-deltacloud-client - Deltacloud REST Client https://bugzilla.redhat.com/show_bug.cgi?id=642995 Summary: Review Request: rubygem-deltacloud-client - Deltacloud REST Client Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mfoj...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://mifo.sk/RPMS/rubygem-deltacloud-client.spec SRPM URL: http://mifo.sk/RPMS/rubygem-deltacloud-client-0.0.9.8-1.fc13.src.rpm Description: Deltacloud REST Client for API -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641390] Review Request: sisu - Sonatype dependency injection framework
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=641390
Alexander Kurtakov akurt...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2010-10-14
08:17:58 EDT ---
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[X] Rpmlint output:
sisu.noarch: W: spelling-error Summary(en_US) Sonatype - Sonatina, Sonata,
Somatotype
Not a problem.
sisu.noarch: W: no-documentation
Explained by submitter.
sisu.noarch: W: non-conffile-in-etc /etc/maven/fragments/sisu
Not a problem
[X] Package is named according to the Package Naming Guidelines[1].
[X] Spec file name must match the base package name, in the format
%{name}.spec.
[X] Package meets the Packaging Guidelines[2].
[X] Package successfully compiles and builds into binary rpms.
[X] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[X] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[X] License field in the package spec file matches the actual license.
License type: ASL 2.0
[-] 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.
[-] All independent sub-packages have license of their own
[X] Spec file is legible and written in American English.
[-] Sources used to build the package matches the upstream source, as provided
in the spec URL.
[X] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[X] Package must own all directories that it creates.
[X] Package requires other packages for directories it uses.
[X] Package does not contain duplicates in %files.
[X] Permissions on files are set properly.
[X] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[X] Package consistently uses macros.
[X] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[X] Package does not own files or directories owned by other packages.
[X] Javadoc documentation files are generated and included in -javadoc
subpackage
[X] Packages have proper BuildRequires/Requires on jpackage-utils
[X] Javadoc subpackages have Require: jpackage-utils
[X] Package uses %global not %define
[X] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[X] All filenames in rpm packages must be valid UTF-8.
=== Maven ===
[X] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses -Dmaven.test.skip=true explain why it was needed in a
comment
[X] If package uses custom depmap -Dmaven2.jpp.depmap.file=* explain why
it's needed in a comment
[X] Package uses %update_maven_depmap in %post/%postun
[X] Packages have Requires(post) and Requires(postun) on jpackage-utils (for
%update_maven_depmap macro)
=== Other suggestions ===
[X] If possible use upstream build method (maven/ant/javac)
[X] Javadocs are placed in %{_javadocdir}/%{name}-%{version} with
%{_javadocdir}/%{name} symlink
[X] Jar files are installed to %{_javadir}/%{name}-%{version}.jar with
%{_javadir}/%{name}.jar (unversioned) symlink
[X] If package contains pom.xml files install it (including depmaps) even when
building with ant
[X] Avoid having BuildRequires on exact NVR unless necessary
[X] Package has BuildArch: noarch (if possible)
[X] Latest version is packaged.
This package is APPROVED.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226525] Merge Review: velocity
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226525 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||akurt...@redhat.com AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com --- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 08:20:06 EDT --- I'll do this one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226525] Merge Review: velocity
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226525
--- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2010-10-14
08:45:23 EDT ---
Issues:
1. newer version is available
2. File permissions are not using defaults.
3. manual, javadoc and demo subpackages are not installing License
4. included jars are not removed in prep
5. %{_javadocdir}/%{name} is not created/installed
6. both $RPM_BUILD_ROOT and %{buildroot} are used
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641390] Review Request: sisu - Sonatype dependency injection framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641390 Stanislav Ochotnicky socho...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Stanislav Ochotnicky socho...@redhat.com 2010-10-14 08:46:16 EDT --- Thanks for the review. Requesting repos New Package CVS Request === Package Name: sisu Short Description: Sonatype dependency injection framework Owners: sochotni Branches: f14 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 590244] Review Request: pinta - Simple Paint Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=590244 --- Comment #17 from Paul Lange pala...@gmx.de 2010-10-14 08:59:17 EDT --- Doesn't seem like 0.6 is already out. Is there anyone who can finish the review. I'd really appreciate that! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643014] New: Review Request: fetch-crl3 - Downloads Certificate Revocation Lists
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: fetch-crl3 - Downloads Certificate Revocation Lists https://bugzilla.redhat.com/show_bug.cgi?id=643014 Summary: Review Request: fetch-crl3 - Downloads Certificate Revocation Lists Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: steve.tray...@cern.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora This is a parallel package of fetch-crl3 to the existing package fetch-crl present in EPEL4 and EPEL5. It will not be released for Fedora or EPEL6 since these contain fetch-crl at version 3 already. Spec URL: http://cern.ch/straylen/rpms/fetch-crl3/fetch-crl3.spec SRPM URL: http://cern.ch/straylen/rpms/fetch-crl3/fetch-crl3-3.0.3-2.el5.src.rpm Description: This tool and associated cron entry ensure that Certificate Revocation Lists (CRLs) are periodically retrieved from the web sites of the respective Certification Authorities. It assumes that the installed CA files follow the hash.crl_url convention. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 532532] Review request: vafer-dependency - Analyse and modify class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532532 Terje Røsten terje...@phys.ntnu.no changed: What|Removed |Added CC||terje...@phys.ntnu.no --- Comment #1 from Terje Røsten terje...@phys.ntnu.no 2010-10-14 09:17:07 EDT --- Complete rewrite for 0.6 using maven: spec: http://terjeros.fedorapeople.org/java/jdependency.spec srpm: http://terjeros.fedorapeople.org/java/jdependency-0.6-1.fc14.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2534753 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643017] New: Review Request:vafer-jdependency - This project provides an API to analyse class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request:vafer-jdependency - This project provides an API to analyse class dependencies https://bugzilla.redhat.com/show_bug.cgi?id=643017 Summary: Review Request:vafer-jdependency - This project provides an API to analyse class dependencies Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pin...@pingoured.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://pingou.fedorapeople.org/RPMs/vafer-jdependency.spec SRPM URL: http://pingou.fedorapeople.org/RPMs/vafer-jdependency-0.6-1.fc13.src.rpm Description: jdependency is small library that helps you analyze class level dependencies, clashes and missing classes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 636946] Review Request: perl-Astro-FITS-Header - Object Orientated interface to FITS HDUs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636946 Marcela Mašláňová mmasl...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX Last Closed||2010-10-14 09:24:48 --- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2010-10-14 09:24:48 EDT --- After looking into PDL, this package is not necessary. I'm not interested in any more. Although it is not easy to package it, because it have many non-cpan dependencies. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643017] Review Request:vafer-jdependency - This project provides an API to analyse class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643017 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added CC||mefos...@gmail.com --- Comment #1 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 09:28:44 EDT --- *** Bug 532532 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 532532] Review request: vafer-dependency - Analyse and modify class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532532 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Status|NEW |CLOSED CC||akurt...@redhat.com Resolution||DUPLICATE Last Closed||2010-10-14 09:28:44 --- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 09:28:44 EDT --- A proper new package review bug is https://bugzilla.redhat.com/show_bug.cgi?id=643017 *** This bug has been marked as a duplicate of bug 643017 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 532532] Review request: vafer-dependency - Analyse and modify class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532532 --- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 09:42:11 EDT --- (In reply to comment #1) Complete rewrite for 0.6 using maven: spec: http://terjeros.fedorapeople.org/java/jdependency.spec srpm: http://terjeros.fedorapeople.org/java/jdependency-0.6-1.fc14.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2534753 When name changes a new bug report has to be opened and current review should be closed as duplicate. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639991] Review Request: rubygem-rb-inotify - A Ruby wrapper for Linux's inotify, using FFI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639991 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-10-14 09:51:32 EDT --- rubygem-rb-inotify-0.8.1-2.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-rb-inotify'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-rb-inotify-0.8.1-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643017] Review Request:vafer-jdependency - This project provides an API to analyse class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643017 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||akurt...@redhat.com AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com Flag||fedora-review? --- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 09:53:05 EDT --- There is no more vafer in this project. Please name it properly - jdependency. And I'm taking this one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639619] Review Request: rubygem-simple-navigation - Ruby library for creating navigation for your Rails2 or Sinatra application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639619 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-10-14 09:52:25 EDT --- rubygem-simple-navigation-3.0.0-3.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-simple-navigation'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/rubygem-simple-navigation-3.0.0-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 532532] Review request: vafer-dependency - Analyse and modify class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532532 --- Comment #4 from Alexander Kurtakov akurt...@redhat.com 2010-10-14 09:57:20 EDT --- Terje, I'm pretty sure that pingou wouldn't mind if you want to be a co-maintainer. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 359921] Review Request: drupal-cck - Allows you create and customize fields using a web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=359921 Jon Ciesla l...@jcomserv.net changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #16 from Jon Ciesla l...@jcomserv.net 2010-10-14 10:01:32 EDT --- Package Change Request == Package Name: drupal-cck New Branches: el5 el6 Owners: limb -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 359911] Review Request: drupal-views - Provides a method for site designers to control content presentation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=359911 Jon Ciesla l...@jcomserv.net changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #26 from Jon Ciesla l...@jcomserv.net 2010-10-14 10:01:48 EDT --- Package Change Request == Package Name: drupal-views New Branches: el5 el6 Owners: limb -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 532532] Review request: vafer-dependency - Analyse and modify class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=532532 --- Comment #5 from Terje Røsten terje...@phys.ntnu.no 2010-10-14 09:59:59 EDT --- No worry, I have enough packages and some more is coming up any way. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643017] Review Request:vafer-jdependency - This project provides an API to analyse class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643017 --- Comment #3 from Pierre-YvesChibon pin...@pingoured.fr 2010-10-14 10:04:00 EDT --- Spec: http://pingou.fedorapeople.org/RPMs/jdependency.spec SRPM: http://pingou.fedorapeople.org/RPMs/jdependency-0.6-2.fc13.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 638974] Review Request: erlang-protobuffs - A set of Protocol Buffers tools and modules for Erlang applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638974 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-10-14 10:09:34 EDT --- erlang-protobuffs-0-0.2.20100930git58ff962.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545045] Review Request: CQRlib - ANSI C API for quaternion arithmetic and rotation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=545045 --- Comment #27 from Fedora Update System upda...@fedoraproject.org 2010-10-14 10:17:48 EDT --- CQRlib-1.1.1-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/CQRlib-1.1.1-1.el5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545045] Review Request: CQRlib - ANSI C API for quaternion arithmetic and rotation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=545045 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545047] Review Request: NearTree - An API for finding nearest neighbors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=545047 Chen Lei supercyp...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|supercyp...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 636250] Review Request: HotEqn - Java Library to display LateX Code - Dependency for Freemind
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=636250 --- Comment #15 from Orion Poplawski or...@cora.nwra.com 2010-10-14 10:38:39 EDT --- Looks good, approved -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=635511 --- Comment #11 from Tom spot Callaway tcall...@redhat.com 2010-10-14 10:34:19 EDT --- That seems like the best approach to handle the models. As to the IrrXML bundling, since I'm also the Irrlicht maintainer, I think that sounds like a solid approach. Please let me know how they respond. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 643017] Review Request:jdependency - This project provides an API to analyse class dependencies
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=643017 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Summary|Review |Review Request:jdependency |Request:vafer-jdependency - |- This project provides an |This project provides an|API to analyse class |API to analyse class|dependencies |dependencies| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 545047] Review Request: NearTree - An API for finding nearest neighbors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=545047 --- Comment #15 from Chen Lei supercyp...@gmail.com 2010-10-14 10:47:35 EDT --- It seems you should use make all instead of make in %build section. See http://koji.fedoraproject.org/koji/getfile?taskID=2535029name=build.log I also suggest you to add %check section(make tests) to spec. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518546] Review Request: libva - VAAPI video playback acceleration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518546 --- Comment #46 from Tom spot Callaway tcall...@redhat.com 2010-10-14 10:55:24 EDT --- Adam, the spec file in comment 45 doesn't match what falls out of the srpm in comment 44. Can you resolve that inconsistency? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=531107 --- Comment #16 from David Fetter da...@fetter.org 2010-10-14 10:59:21 EDT --- What is it you'd like done about them? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642577] Review Request: rubygem-open4 - Manage child processes and their IO handles easily
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642577
Jozef Zigmund jzigm...@redhat.com changed:
What|Removed |Added
Flag|needinfo?(jzigm...@redhat.c |
|om) |
--- Comment #3 from Jozef Zigmund jzigm...@redhat.com 2010-10-14 11:29:34 EDT
---
MUST: rpmlint must be run on every package. The output should be posted in the
review.
rpmlint rubygem-open4-1.0.1-1.fc13.src.rpm
rubygem-open4.src: W: spelling-error %description -l en_US pid - peed, dip,
pud
rubygem-open4.src: W: spelling-error %description -l en_US stdin - stein,
stain, stdio
rubygem-open4.src: W: spelling-error %description -l en_US stdout - stout, std
out, std-out
rubygem-open4.src: W: spelling-error %description -l en_US stderr - std err,
std-err, stander
rubygem-open4.src: W: no-buildroot-tag
1 packages and 0 specfiles checked; 0 errors, 5 warnings.
[OK]
Each Ruby package MUST indicate the Ruby ABI version it depends on with a line
like:
Requires: ruby(abi) = 1.8
[OK - this MUST condition is written via macro in this spec file]
Packages that contain Ruby Gems MUST be called rubygem-%{gemname} where gemname
is the name from the Gem's specification.
%global gemname open4
Name: rubygem-%{gemname}
[OK]
The Source of the package MUST be the full URL to the released Gem archive
URL: http://github.com/ahoward/open4/tree/master
Source0: http://gems.rubyforge.org/gems/%{gemname}-%{version}.gem
[OK]
The version of the package MUST be the Gem's version
Version: 1.0.1
[OK]
The package MUST have a Requires and a BuildRequires on rubygems
%global rubyabi 1.8
Requires: ruby(abi) = %{rubyabi}
Requires: rubygems
BuildRequires: rubygems
[OK]
The Gem MUST be installed into %{gemdir} defined as:
%global gemdir %(ruby -rubygems -e 'puts Gem::dir' 2/dev/null)
[OK]
If the Gem only contains pure Ruby code, it MUST be marked as BuildArch: noarch
BuildArch: noarch
[OK]
The package MUST own the following files and directories:
%{gemdir}/gems/%{gemname}-%{version}/
%{gemdir}/cache/%{gemname}-%{version}.gem
%{gemdir}/specifications/%{gemname}-%{version}.gemspec
[OK]
The install SHOULD be performed with the command
gem install --local --install-dir %{buildroot}%{gemdir} --force %{SOURCE0}
[OK]
Note: At the end of %description is '.' character and i'm not sure if it's OK.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642524] Review Request: rubygem-net-sftp - A pure Ruby implementation of the SFTP client protocol
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642524 --- Comment #5 from Mohammed Morsi mmo...@redhat.com 2010-10-14 11:48:14 EDT --- Appreciate the update, though regarding the license, its still not fully right. Since setup.rb is shipped with the 'doc' subpackage, it is that package that needs the LGPLv2 in the license. Furthermore it needs to be MIT _and_ LGPLv2 (not _or_) as the provided sources are shipped under both those licenses. 'or' would be appropriate if upstream said 'you may use this project under the terms of the MIT license or the LGPL one, your choice'. So essentially the license for the main package should be 'LICENSE: MIT' and the license for the subpackage (put this under %package doc) should be 'LICENSE: MIT and LGPLv2' Now that being said, looking at setup.rb, it looks like we don't even need to ship it as it simply sets up the upstream environment for a build / install. Unless I'm wrong, you can also just remove setup.rb, and just ship the entire package (both net-sftp and the doc subpackage) under the MIT license. Your choice. Either way, once this license issue is resolved and git is setup, just push as my APPROVED still stands. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642577] Review Request: rubygem-open4 - Manage child processes and their IO handles easily
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642577 --- Comment #4 from Jozef Zigmund jzigm...@redhat.com 2010-10-14 11:57:23 EDT --- Yes i checked this wiki page before, so i've written unformally review to this package, and now i hope that somebody will add me to Fedora Packager Group. And of course i can tell you if i will have sponsoring. (In reply to comment #2) Hello, Jozef: As far as I checked FAS (Fedora Account System), it seems that you are not sponsored and are not in packager group. The person who is going to formally review review requests must be packager group. Would you tell me if you are sponsored? http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641957] Review Request: rubygem-database_cleaner - Strategies for cleaning databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641957 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641962] Review Request: rubygem-couch_potato - Ruby persistence layer for CouchDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641962 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641957] Review Request: rubygem-database_cleaner - Strategies for cleaning databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641957 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp --- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-10-14 12:42:46 EDT --- Hello, Jozef: Just looking at your spec file, it says this package depends on rubygem(bundler), however I don't see rubygem-bundle in Fedora repository or in package review. Would you package rubygem-bundle first if it is needed for this package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641962] Review Request: rubygem-couch_potato - Ruby persistence layer for CouchDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641962 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp --- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-10-14 12:48:52 EDT --- I don't see rubygem(couchrest) and rubygem(activemodel) in Fedora or package review. Would you package them first? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641962] Review Request: rubygem-couch_potato - Ruby persistence layer for CouchDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641962 --- Comment #4 from Jozef Zigmund jzigm...@redhat.com 2010-10-14 13:08:50 EDT --- Yes, i would, but the same as https://bugzilla.redhat.com/show_bug.cgi?id=641957 (In reply to comment #3) I don't see rubygem(couchrest) and rubygem(activemodel) in Fedora or package review. Would you package them first? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641957] Review Request: rubygem-database_cleaner - Strategies for cleaning databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641957 --- Comment #4 from Jozef Zigmund jzigm...@redhat.com 2010-10-14 13:06:53 EDT --- Hi, Mamoru, i'll do package rubygem-bundler. But for now isn't important, because i don't have a sponsor and also i'm not in Fedora Packager Group, so i've focused on this issues. (In reply to comment #3) Hello, Jozef: Just looking at your spec file, it says this package depends on rubygem(bundler), however I don't see rubygem-bundle in Fedora repository or in package review. Would you package rubygem-bundle first if it is needed for this package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 641957] Review Request: rubygem-database_cleaner - Strategies for cleaning databases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=641957 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status Whiteboard||NotReady --- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-10-14 13:27:43 EDT --- (In reply to comment #4) Hi, Mamoru, i'll do package rubygem-bundler. But for now isn't important, because i don't have a sponsor and also i'm not in Fedora Packager Group, so i've focused on this issues. Well, What I mean here is that as Requires: rubygem(bundler) is in the spec file of rubygem-database_cleaner, unless you firstly package rubygem-bundler and rubygem-bundler review request is approved beforehand, this package can never be approved, because unless rubygem-bundler is in Fedora repository this rpm cannot be installed (because of dependency issue). For now setting NotReady. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
