[Bug 557546] Review Request: telepathy-sunshine - Gadu-Gadu connection manager for telepathy
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=557546 Patrick Dignan changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|dignan.patr...@gmail.com -- 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 498195] Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook
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=498195 --- Comment #10 from Fedora Update System 2010-02-11 02:21:01 EST --- zarafa-webaccess-6.30.10-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/zarafa-webaccess-6.30.10-1.fc11 -- 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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 --- Comment #6 from Jens Petersen 2010-02-11 02:19:21 EST --- (Anyway just some initial comments and impressions. :) -- 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 498195] Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook
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=498195 --- Comment #12 from Fedora Update System 2010-02-11 02:21:13 EST --- zarafa-webaccess-6.30.10-1.el5 has been submitted as an update for Fedora EPEL 5. http://admin.fedoraproject.org/updates/zarafa-webaccess-6.30.10-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 498195] Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook
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=498195 --- Comment #11 from Fedora Update System 2010-02-11 02:21:10 EST --- zarafa-webaccess-6.30.10-1.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/zarafa-webaccess-6.30.10-1.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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 --- Comment #7 from Jens Petersen 2010-02-11 02:20:52 EST --- (In reply to comment #4) > Ok I will change the name to Anumaan and split AnumaanDefaultLM from it, and > then resubmit. To be clear - you can update the submission and summary here (no need to open a new 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 558685] Review Request: gloobus-preview - A file previewer for
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=558685 Julian Sikorski changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|beleg...@gmail.com -- 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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 --- Comment #5 from Jens Petersen 2010-02-11 02:18:40 EST --- Any chance of fixing the upstream tarball rather than having all the chmod's in the spec file? Similarly better if Anumaan.sh became Anumaan. But I guess these are not blockers. -- 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 563044] Review Request: system-setup-keyboard - Hal keyboard layout callout
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=563044 Peter Hutterer changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #4 from Peter Hutterer 2010-02-11 02:09:07 EST --- imported, closing as NEXTRELEASE. -- 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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 --- Comment #4 from Naveen Kumar 2010-02-11 01:52:53 EST --- Ok I will change the name to Anumaan and split AnumaanDefaultLM from it, and then resubmit. -- 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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 Jens Petersen changed: What|Removed |Added Summary|Review Request: |Review Request: |AnumaanLinux-alpha-0.1 - A |AnumaanLinux-alpha - A |predictive text entry |predictive text entry |system |system --- Comment #2 from Jens Petersen 2010-02-11 01:39:42 EST --- Maybe the package should just be called AnumaanLinux or even Anumaan? if alpha means alpha release? (I think the upstream naming scheme is not ideal but anyway.) -- 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 563444] Review Request: AnumaanLinux-alpha - A predictive text entry system
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=563444 --- Comment #3 from Jens Petersen 2010-02-11 01:41:51 EST --- AnumaanDefaultLM can't be packaged itself? -- 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 561549] Review Request: eclipse-collabnet-merge - CollabNet Merge Client for Subclipse
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=561549 --- Comment #7 from Kevin Fenzi 2010-02-11 01:23:42 EST --- CVS done (by process-cvs-requests.py). -- 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 563044] Review Request: system-setup-keyboard - Hal keyboard layout callout
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=563044 --- Comment #3 from Kevin Fenzi 2010-02-11 01:24:12 EST --- CVS done (by process-cvs-requests.py). -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 --- Comment #2 from Jens Petersen 2010-02-11 01:25:21 EST --- (In reply to comment #0) > I'm not sure the additional Provides: tag is really necessary as no package > were currently depending on abyssinica-fonts. Yeah, your call I think, but it might not hurt for people familiar with the old name. You might add a comment say "# added for f13 can be removed for f15". -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 Jens Petersen changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|peter...@redhat.com Flag||fedora-review? --- Comment #3 from Jens Petersen 2010-02-11 01:26:09 EST --- Taking the review (since I requested the name change;) -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #4 from Jens Petersen 2010-02-11 01:27:35 EST --- Ah also please upload the srpm for reference. :) -- 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 560169] Review Request: bytelist - A java library for lists of bytes
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=560169 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Kevin Fenzi 2010-02-11 01:18:55 EST --- cvs done. Note that you will need to file a bug in the rel-eng track asking for this package to be unblocked now before it will build in rawhide. -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 --- Comment #4 from Kevin Fenzi 2010-02-11 01:23:08 EST --- CVS done (by process-cvs-requests.py). -- 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 560172] Review Request: jvyamlb - YAML processor for JRuby
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=560172 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #7 from Kevin Fenzi 2010-02-11 01:21:59 EST --- CVS done. -- 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 560170] Review Request: jcodings - Java-based codings helper classes for Joni and JRuby
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=560170 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Kevin Fenzi 2010-02-11 01:20:58 EST --- cvs done. Note that you will need to file a ticket in rel-eng track to get this package unblocked before it will build in rawhide. -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 Jens Petersen changed: What|Removed |Added CC||fonts-b...@lists.fedoraproj ||ect.org, ||peter...@redhat.com --- Comment #1 from Jens Petersen 2010-02-11 01:22:47 EST --- How about the "Provides: fonts-sil-abyssinica"? (bug 563395) Also can you reproduce: # note current downloaded filename is lowercase and if so how about renaming the source file? Other than that the new package looks fine to me. -- 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 558685] Review Request: gloobus-preview - A file previewer for
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=558685 --- Comment #12 from Kevin Fenzi 2010-02-11 01:10:14 EST --- CVS done (by process-cvs-requests.py). belegdol: Please remember to assign the review to yourself when you take it on. ;) -- 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 498195] Review Request: zarafa-webaccess - Zarafa Webaccess featuring a 'Look & Feel' similar to Outlook
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=498195 --- Comment #9 from Kevin Fenzi 2010-02-11 01:04:35 EST --- CVS done (by process-cvs-requests.py). -- 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 550067] Review Request: perl-Template-Tiny - Template Toolkit reimplemented in as little code as possible
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=550067 --- Comment #15 from Kevin Fenzi 2010-02-11 01:06:34 EST --- CVS done (by process-cvs-requests.py). -- 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 516524] Review Request: globus-duct-control - Globus Toolkit - Globus Duct Control
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=516524 --- Comment #3 from Kevin Fenzi 2010-02-11 01:05:12 EST --- CVS done (by process-cvs-requests.py). -- 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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 --- Comment #8 from Kevin Fenzi 2010-02-11 01:08:06 EST --- CVS done (by process-cvs-requests.py). -- 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 484597] Review Request: python-argparse - Optparse inspired command line parser for Python
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=484597 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Kevin Fenzi 2010-02-11 01:03:34 EST --- cvs done. -- 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 556556] Review Request: geronimo-parent-poms - Parent POM files for geronimo-specs
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=556556 --- Comment #9 from Kevin Fenzi 2010-02-11 01:07:28 EST --- CVS done (by process-cvs-requests.py). -- 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 493335] Review Request: perl-RTx-Calendar - Calendar for RT due tasks
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=493335 Jeff Fearn changed: What|Removed |Added CC||jfe...@redhat.com --- Comment #13 from Jeff Fearn 2010-02-10 18:37:00 EST --- (In reply to comment #4) > (In reply to comment #2) > > Anyway, even with 3.6 the files installed under > > /usr/local/lib/rt3 are probably a problem > I haven't checked all details yet, but as far as I understand (so far), your > package is facing a design flaw in rt3: Lack of a "system-wide plugins > directory". FYI RT 3.8.7, which is in rawhide, added a second plugin path, which can be set in either RT_Config.pm or RT_SiteConfig.pm e.g. Set($PluginPath,"/var/www/rt3/plugins"); This will then be checked for plugins if the plugin isn't found in /usr/local/lib/rt3/plugins. -- 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 557948] Review Request: PyAIML - A Python AIML Interpreter
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=557948 Sebastian Dziallas changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX --- Comment #5 from Sebastian Dziallas 2010-02-10 17:43:19 EST --- Argh! Upstream is not responsive, so I guess I'll have to close this for now. If he replies, I'll reopen here. -- 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 557969] Review Request: sugar-socialcalc - A spreadsheet activity for the Sugar environment
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=557969 --- Comment #4 from Sebastian Dziallas 2010-02-10 17:42:00 EST --- Thanks for the review! I pinged upstream via e-mail, who replied that he'd fix that and get a new version out soon. -- 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 558685] Review Request: gloobus-preview - A file previewer for
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=558685 Patrick Dignan changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Patrick Dignan 2010-02-10 16:25:05 EST --- New Package CVS Request === Package Name: gloobus-preview Short Description: A Gnome extension to enable previews for any kind of file Owners: dignan Branches: F-12 InitialCC: dignan -- 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 563318] Review Request: ceph - User space components of the CEPH file system
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=563318 Josef Bacik changed: What|Removed |Added Blocks|163776(FE-NEW) | -- 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 563318] Review Request: ceph - User space components of the CEPH file system
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=563318 Josef Bacik changed: What|Removed |Added Blocks||163776(FE-NEW) -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 Mathieu Bridon changed: What|Removed |Added Blocks||563395 -- 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 563673] Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
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=563673 Mathieu Bridon changed: What|Removed |Added Blocks||563399 -- 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 563673] New: Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts https://bugzilla.redhat.com/show_bug.cgi?id=563673 Summary: Review Request: sil-abyssinica-fonts - SIL Abyssinica fonts Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: boche...@fedoraproject.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora This isn't a real review request but the mandatory re-review for renaming a package. Original review can be found here: https://bugzilla.redhat.com/show_bug.cgi?id=291091 Spec URL: http://bochecha.fedorapeople.org/packages/sil-abyssinica-fonts/sil-abyssinica-fonts.spec SRPM URL: http://bochecha.fedorapeople.org/packages/sil-abyssinica-fonts/sil-abyssinica-fonts-1.0-6.fc13.noarch.rpm Description: SIL Abyssinica is a Unicode typeface family containing glyphs for the Ethiopic script. The Ethiopic script is used for writing many of the languages of Ethiopia and Eritrea. Abyssinica SIL supports all Ethiopic characters which are in Unicode including the Unicode 4.1 extensions. Some languages of Ethiopia are not yet able to be fully represented in Unicode and, where necessary, we have included non-Unicode characters in the Private Use Area (see Private-use (PUA) characters supported by Abyssinica SIL). Abyssinica SIL is based on Ethiopic calligraphic traditions. This release is a regular typeface, with no bold or italic version available or planned. For convenience, here is a diff of what changed in the spec file between the current version in CVS and this one: http://bochecha.fedorapeople.org/packages/sil-abyssinica-fonts/0001-Renamed-the-package.patch I'm not sure the additional Provides: tag is really necessary as no package were currently depending on abyssinica-fonts. -- 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 560172] Review Request: jvyamlb - YAML processor for JRuby
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=560172 Victor G. Vasilyev changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Victor G. Vasilyev 2010-02-10 15:34:19 EST --- Package Change Request == Package Name: jvyamlb New Branches: Owners: victorv Updated Description: YAML processor for JRuby The package https://admin.fedoraproject.org/pkgdb/packages/name/jvyamlb exists, but has been orphaned and deprecated. -- 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 563510] Review Request: php-xcache - yet another php cacher
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=563510
--- Comment #6 from Itamar Reis Peixoto 2010-02-10
15:24:53 EST ---
I recommend you to put this out of spec file.
%{__cat} > %{buildroot}%{_sysconfdir}/php.d/xcache.ini << 'EOF'
[xcache-common]
zend_extension = %{php_extdir}/xcache.so
[xcache.admin]
xcache.admin.enable_auth = On
xcache.admin.user = "mOo"
; xcache.admin.pass = md5($your_password)
xcache.admin.pass = ""
[xcache]
xcache.shm_scheme ="mmap"
xcache.size = 60M
xcache.count = 1
xcache.slots =8K
xcache.ttl = 3600
xcache.gc_interval = 300
xcache.var_size =4M
xcache.var_count = 1
xcache.var_slots =8K
xcache.var_ttl = 0
xcache.var_maxttl = 0
xcache.var_gc_interval = 300
EOF
--
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 560172] Review Request: jvyamlb - YAML processor for JRuby
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=560172 Mohammed Morsi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #5 from Mohammed Morsi 2010-02-10 15:21:26 EST --- (In reply to comment #4) > Sorry for my fault. > > The release 4 is prepared for review: > Spec URL: http://victorv.fedorapeople.org/files/jvyamlb.spec > SRPM URL: http://victorv.fedorapeople.org/files/jvyamlb-0.2.5-4.fc13.src.rpm > Everything looks good. Package is fully compliant w/ guidelines and builds fine in mock when jcodings, bytelist dependencies are installed. APPROVED Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import the package. Close this bug as RAWHIDE once it's been successfully imported and built. -- 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 558685] Review Request: gloobus-preview - A file previewer for
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=558685 Julian Sikorski changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Julian Sikorski 2010-02-10 15:16:57 EST --- 1. rpmlint is silent: rpmlint gloobus-preview-0.4.1-5.fc12.src.rpm ../RPMS/x86_64/gloobus-preview-* 3 packages and 0 specfiles checked; 0 errors, 0 warnings. 2. Name is correct. 3. Spec file name is correct. 4. Package meets packaging guidelines 5. Licensing is OK: GPLv3. 6. Spec is written in American English and is legible. 7. Sources match upstream: d41d8cd98f00b204e9800998ecf8427e 8. Builds fine for f13: https://koji.fedoraproject.org/koji/taskinfo?taskID=1975674 9. BuildRequires are correct. 10. There is no locale to handle. 11. Shared libs are private, no need for ldconfig. 12. Nothing is bundled. 14. Not relocatable. 15. Ownerships are correct. 16. No duplicates in %files. 17. Permissions are correct. 18. %clean section is present. 19. Macros are consistent. 20. Package contains code. 21. No large doc. 22. %doc is not required at runtime. 23. No header files static libs nor pkgconfig files. 24. Libtool archives are removed. 25. Desktop file is included and validated. 26. Buildroot is cleaned at the beginning of %install. 27. Filenames are utf-8. 28. Package seems to work correctly. -- 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 563001] Review Request: xfce4-remmina-plugin - Xfce panel plugin for remmina remote desktop client
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=563001
Dominic Hopf changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||dma...@fedoraproject.org
AssignedTo|nob...@fedoraproject.org|dma...@fedoraproject.org
Flag||fedora-review?
--- Comment #2 from Dominic Hopf 2010-02-10 15:13:55
EST ---
Seems you forgot to rename the %{srcname} macro to the new name?
I'll do the review on the weekend.
--
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 560169] Review Request: bytelist - A java library for lists of bytes
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=560169 Victor G. Vasilyev changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Victor G. Vasilyev 2010-02-10 15:16:21 EST --- Package Change Request == Package Name: bytelist New Branches: Owners: mmorsi victorv Updated Description: A java library for lists of bytes The package https://admin.fedoraproject.org/pkgdb/packages/name/bytelist exists, but has been orphaned and deprecated. -- 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 563510] Review Request: php-xcache - yet another php cacher
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=563510
Matthias Runge changed:
What|Removed |Added
CC||mru...@fedoraproject.org
--- Comment #5 from Matthias Runge 2010-02-10
15:09:06 EST ---
As I'm not approved, I could only provide an unofficial review:
rpmlint ../RPMS/i686/php-xcache-1.3.0-1.fc12.i686.rpm
../RPMS/i686/php-xcache-debuginfo-1.3.0-1.fc12.i686.rpm php-xcache.spec
2 packages and 1 specfiles checked; 0 errors, 0 warnings. (ok)
* name meets naming convention (ok)
* can not verify license (BSD given in SPEC, couldn't find this in code, esp.
COPYING does not mention BSD)
* Mixing $RPM_BUILD_ROOT and %{buildroot} should not be done (cf. Packaging
guidelines in Fedora wiki)
* %global preferred over %define (used both)
* some code in SPEC is commented out, decreases readability. Why don't you
remove those lines? Why is code for RHEL 4 and RHEL 5 included?
* Package compiles succesful into binary RPMS
--
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 560172] Review Request: jvyamlb - YAML processor for JRuby
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=560172 --- Comment #4 from Victor G. Vasilyev 2010-02-10 14:56:38 EST --- Sorry for my fault. The release 4 is prepared for review: Spec URL: http://victorv.fedorapeople.org/files/jvyamlb.spec SRPM URL: http://victorv.fedorapeople.org/files/jvyamlb-0.2.5-4.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 560170] Review Request: jcodings - Java-based codings helper classes for Joni and JRuby
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=560170 Victor G. Vasilyev changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Victor G. Vasilyev 2010-02-10 14:41:30 EST --- Package Change Request == Package Name: jcodings New Branches: Owners: mmorsi victorv Updated Description: Java-based codings helper classes for Joni and JRuby The package https://admin.fedoraproject.org/pkgdb/packages/name/jcodings exists, but has been orphaned and deprecated. -- 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 560172] Review Request: jvyamlb - YAML processor for JRuby
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=560172 --- Comment #3 from Mohammed Morsi 2010-02-10 14:31:22 EST --- I think you mistakingly uploaded the built rpm instead of the srpm for the latest release. I'll continue w/ the review once the SRPM is uploaded. -- 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 563336] Review Request: libisds - Library for accessing the Czech Data Boxes
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=563336
--- Comment #1 from Ondrej Vasik 2010-02-10 14:27:00 EST ---
Legend: + = PASSED, - = FAILED, 0 = Not Applicable
+ MUST: rpmlint must be run on every package. The output should be posted in
the review
Sources used when checking:
[re...@localhost koji]$ md5sum libisds.spec
32ac2450de961b8f7dbd5014716f362c libisds.spec
[re...@localhost koji]$ md5sum libisds-0.1-1.el6.src.rpm
4f5082969d4880f4622c4eff93d55c6a libisds-0.1-1.el6.src.rpm
$rpmlint -v libisds-*
libisds-debuginfo.i686: I: checking
libisds-devel.i686: I: checking
libisds-devel.i686: W: no-documentation
libisds.i686: I: checking
libisds.src: I: checking
4 packages and 0 specfiles checked; 0 errors, 1 warnings.
Doc files available in main package, so warning could be ignored.
+ MUST: package named according to the Package Naming Guidelines
+ MUST: The spec file name must match the base package %{name}
+ MUST: The package must meet the Packaging Guidelines .
+ MUST: The package licensed with a Fedora approved license and meets the
Licensing Guidelines
+ MUST: The License field in the package spec file matches the actual
license
+ MUST: 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 must be included in %doc.
+ MUST: The spec file must be written in American English.
+ MUST: The spec file for the package MUST be legible.
+ MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task
>From srpm:
$ md5sum libisds-0.1.tar.bz2
d0b0f647e3d603ca810b639ea703ed82 libisds-0.1.tar.bz2
>From upstream:
$ md5sum libisds-0.1.tar.bz2
d0b0f647e3d603ca810b639ea703ed82 libisds-0.1.tar.bz2
= MATCHES
+ MUST: The package successfully compiles and builds into binary rpms on at
least one primary architecture
- tested on i686, no problems
0 MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch
+ MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines
0 MUST: The spec file handles locales properly. This is done by using the
%find_lang macro
0 MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
0 MUST: Packages must NOT bundle copies of system libraries
+ MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker
+ MUST: Package must own all directories that it creates. If it does not create
a directory that it uses, then it should require a package which does create
that directory
+ MUST: Package must not list a file more than once in the spec file's %files
listings
+ MUST: Permissions on files must be set properly. Every %files section must
include a %defattr(...) line.
+ MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
+ MUST: Each package must consistently use macros
+ MUST: The package must contain code, or permissable content
0 MUST: Large documentation files must go in a -doc subpackage
+ MUST: If a package includes something as %doc, it must not affect the runtime
of the application
0 MUST: Header files must be in a -devel package
0 MUST: Static libraries must be in a -static package
0 MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
+ MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel package
+ MUST: devel packages must require the base package using a fully versioned
dependency: Requires: %{name} = %{version}-%{release}
+ MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built
0 MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section
+ MUST: Packages must not own files or directories already owned by other
packages
+ MUST: At the beginning of %install, each package MUST run rm -rf %{buildroot}
(or $RPM_BUILD_ROOT)
+ MUST: All filenames in rpm packages must be valid UTF-8
All MUST things passed, however I have two comments which should be
addressed/NACKed before I'll set review+ :
1) devel package has %{_includedir}/* although it ships just
%{_includedir}/isds.h . Consider explicitely stating that file to ensure that
possible future changes will be known.
[Bug 560169] Review Request: bytelist - A java library for lists of bytes
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=560169 Mohammed Morsi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Mohammed Morsi 2010-02-10 14:26:32 EST --- > The second release is prepared for review: > Spec URL: http://victorv.fedorapeople.org/files/bytelist.spec > SRPM URL: http://victorv.fedorapeople.org/files/bytelist-1.0.3-2.fc13.src.rpm > Everything looks good. Package is fully compliant w/ guidelines and builds fine in mock when jcodings is installed. APPROVED Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import the package. Close this bug as RAWHIDE once it's been successfully imported and built. -- 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 560170] Review Request: jcodings - Java-based codings helper classes for Joni and JRuby
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=560170
Mohammed Morsi changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #6 from Mohammed Morsi 2010-02-10 13:06:03 EST
---
(In reply to comment #5)
> (In reply to comment #4)
> > Even though this is in the orphaned specfile, the following line doesn't
> > work
> > (try it yourself):
> > find -name '*.jar' -o -name '*.class' -exec rm -f '{}' \;
> >
> > You can replace it w/ this if you want:
> > find \( -name '*.class' -o -name '*.jar' \) -exec rm -rf {} \;
> >
> > or this:
> > find ./ -name '*.jar' -exec rm -f '{}' \;
> > find ./ -name '*.class' -exec rm -f '{}' \;
> fixed as described.
> >
> > As far as the changelog you have a typo, "Jun 2010" should be "Jan 2010".
> fixed.
Great, thanks for both of these.
> >
> > Also why did you remove the previous entries from the changelog
> > http://cvs.fedoraproject.org/viewvc/rpms/jcodings/F-11/jcodings.spec?view=markup
> The clause with "find" was only a line that has not been changed in the spec.
> So, I've considered it as a new package, and a history, from my viewpoint, is
> not interesting for anybody. Nevertheless, if you prefer then I'll save the
> changelog.
Ya, think it's good to leave it in, not only for the specfile but just so we
can see the package maintainer history and that it matches up w/ the cvs commit
history.
> The second release is prepared for review:
> Spec URL: http://victorv.fedorapeople.org/files/jcodings.spec
> SRPM URL: http://victorv.fedorapeople.org/files/jcodings-1.0.2-2.fc13.src.rpm
Excellent, went through all the review guidelines and it fully passes.
APPROVED
Please follow http://fedoraproject.org/wiki/CVSAdminProcedure and import
the package. Close this bug as RAWHIDE once it's been successfully imported
and built.
--
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 452521] Review Request: perl-Coro - Coroutine process abstraction
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=452521 hannes+...@stressinduktion.org changed: What|Removed |Added CC||hannes+...@stressinduktion. ||org Bug 452521 depends on bug 533723, which changed state. Bug 533723 Summary: Review Request: perl-Guard - Safe cleanup blocks https://bugzilla.redhat.com/show_bug.cgi?id=533723 What|Old Value |New Value Status|NEW |ASSIGNED Status|ASSIGNED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED -- 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 561430] Review Request: geronimo-jta - J2EE JTA v1.1 API
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=561430 Alexander Kurtakov changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #5 from Alexander Kurtakov 2010-02-10 12:34:11 EST --- 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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 --- Comment #7 from Mary Ellen Foster 2010-02-10 12:30:54 EST --- FYI: added versioned Obsoletes Spec URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jms.spec SRPM URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jms-1.1.1-4.fc12.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 561430] Review Request: geronimo-jta - J2EE JTA v1.1 API
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=561430 --- Comment #4 from Mary Ellen Foster 2010-02-10 12:31:14 EST --- added versioned Obsoletes Spec URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jta.spec SRPM URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jta-1.1.1-4.fc12.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 561430] Review Request: geronimo-jta - J2EE JTA v1.1 API
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=561430
--- Comment #3 from Alexander Kurtakov 2010-02-10
12:26:20 EST ---
Review:
NOTOK: rpmlint must be run on every package. Warnings:
geronimo-jta.src:27: W: unversioned-explicit-obsoletes geronimo-specs
geronimo-jta.spec:27: W: unversioned-explicit-obsoletes geronimo-specs
Should be fixed. Please put a version for the obsoletes
geronimo-jta.noarch: W: obsolete-not-provided geronimo-specs
geronimo-jta.noarch: W: non-conffile-in-etc /etc/maven/fragments/geronimo-jta
Both are not a problem as we really want to obsolete and the second one is just
how maven works.
OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.
OK: 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 must be included in %doc.
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible.
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. Fetch instructions provided.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates.
OK: A Fedora package must not list a file more than once in the spec file's
%files listings.
OK: Permissions on files must be set properly.
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content.
OK: Large documentation files must go in a -doc subpackage.
OK: If a package includes something as %doc, it must not affect the runtime of
the application.
OK: Packages must not own files or directories already owned by other packages.
OK: At the beginning of %install, each package MUST run rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
OK: All filenames in rpm packages must be valid UTF-8.
Please fix the unversioned obsolete and the package is good.
--
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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 Mary Ellen Foster changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Mary Ellen Foster 2010-02-10 12:21:20 EST --- New Package CVS Request === Package Name: geronimo-jms Short Description: J2EE JMS v1.1 API Owners: mef Branches: F-12 -- 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 555162] Review Request: blast - Berkeley Lazy Abstraction Software Verification Tool
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=555162 Jerry James changed: What|Removed |Added Status|NEW |CLOSED Resolution||CANTFIX Bug 555162 depends on bug 555161, which changed state. Bug 555161 Summary: Review Request: csisat - Tool for LA+EUF Interpolation https://bugzilla.redhat.com/show_bug.cgi?id=555161 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||ERRATA --- Comment #5 from Jerry James 2010-02-10 12:18:15 EST --- Sorry to take so long with this, but you raised some thorny issues. I'm afraid that vampyre is, indeed, required. At least, I can't find a way to untangle it from the main code (grep for "vampyre" in blast/psrc). So I guess we're done: this package cannot be included in Fedora for licensing reasons. We should perhaps look at http://code.google.com/p/cpachecker/ instead. Its author, one of the BLAST maintainers, calls it a "better BLAST". On the other hand, they haven't reached a 1.0 release yet. -- 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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 Alexander Kurtakov changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #5 from Alexander Kurtakov 2010-02-10 12:16:24 EST --- Thanks, 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 563598] Review Request: sugar-settings-manager - Settings manager for the Sugar environment
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=563598 Sebastian Dziallas changed: What|Removed |Added Blocks||558617(SOAS-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 563598] New: Review Request: sugar-settings-manager - Settings manager for the Sugar environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sugar-settings-manager - Settings manager for the Sugar environment https://bugzilla.redhat.com/show_bug.cgi?id=563598 Summary: Review Request: sugar-settings-manager - Settings manager for the Sugar environment Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sebast...@when.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sdz.fedorapeople.org/rpmbuild/sugar-settings-manager.spec SRPM URL: http://sdz.fedorapeople.org/rpmbuild/sugar-settings-manager-0.87.2-1.fc12.src.rpm Koji Scratch Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1974866 Description: This is a dependency from the Sugar packaging effort. -- 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 540617] Review Request: django-lint - lint for (python) django web-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=540617 --- Comment #11 from Matthias Runge 2010-02-10 11:41:13 EST --- Till, as you may have seen (on the mailing list), I'm currently doing some (unofficial) reviews (mostly on django or python packages). I took the package review guidelines https://fedoraproject.org/wiki/Packaging:ReviewGuidelines as guide for the reviews. Do you need a list of reviews? -- 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 560071] Review Request: php-pecl-augeas - PHP bindings to the Augeas API
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=560071 --- Comment #9 from Remi Collet 2010-02-10 11:40:25 EST --- Pedro, Have you test this extension under x86_64 ? Fedora 11 i386 + PHP 5.3.1 => ok Fedora 11 i386 + PHP 5.3.2RC1 => ok EL 5 x86_64 + PHP 5.3.1 => segfault Fedora 12 x86_64 + PḦP 5.3.2RC1 => segfault (build ok and php-cli ok, only apache issue) -- 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 531544] Review Request: python-trml2pdf - Tiny RML2PDF is a tool to easily create PDF documents without programming
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=531544 --- Comment #4 from Matthias Runge 2010-02-10 11:36:49 EST --- OK, using the "raw" version, rpmlint lists no error, no warning in your spec-file. To continue the unofficial review: SPEC-File is in english and legible. (ok) Package compiles into binary RPM Build-Requires are ok. No locale (guess it's ok) No shared library, so no %post and %postun (ok) No system library bundled (ok) %clean removes buildroot (ok) %files section ok, file permissions are sane (ok) No header files, no static libraries (ok) (to be continued) -- 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 562470] Review Request: openvas-client - Client component of Open Vulnerability Assessment (OpenVAS) Scanner
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=562470 Fabian Affolter changed: What|Removed |Added CC||fab...@bernewireless.net Blocks||563471(FE-SECSPIN) -- 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 562469] Review Request: openvas-scanner - Open Vulnerability Assessment (OpenVAS) Scanner
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=562469 Fabian Affolter changed: What|Removed |Added CC||fab...@bernewireless.net Blocks||563471(FE-SECSPIN) -- 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 563471] Tracker: Review Requests for Security Spin related packages
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=563471 Fabian Affolter changed: What|Removed |Added Depends on||562469 -- 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 563471] Tracker: Review Requests for Security Spin related packages
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=563471 Fabian Affolter changed: What|Removed |Added Depends on||562470 -- 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 563471] Tracker: Review Requests for Security Spin related packages
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=563471 Fabian Affolter changed: What|Removed |Added Depends on||562467 -- 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 562467] Review Request: openvas-libraries - Support libraries for Open Vulnerability Assessment (OpenVAS) 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=562467 Fabian Affolter changed: What|Removed |Added CC||fab...@bernewireless.net Blocks||563471(FE-SECSPIN) -- 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 561430] Review Request: geronimo-jta - J2EE JTA v1.1 API
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=561430 --- Comment #2 from Mary Ellen Foster 2010-02-10 10:53:46 EST --- Updated version with better descriptions and cleaned-up Provides and Obsoletes: Spec URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jta.spec SRPM URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jta-1.1.1-2.fc12.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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 --- Comment #4 from Mary Ellen Foster 2010-02-10 10:53:33 EST --- Updated version with better descriptions and cleaned-up Provides and Obsoletes: Spec URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jms.spec SRPM URL: http://www.macs.hw.ac.uk/~mef3/geronimo/geronimo-jms-1.1.1-3.fc12.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 531544] Review Request: python-trml2pdf - Tiny RML2PDF is a tool to easily create PDF documents without programming
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=531544 --- Comment #3 from Matthias Runge 2010-02-10 10:15:47 EST --- I've take the version from http://github.com/ciupicri/rpmbuild/blob/master/SPECS/python-trml2pdf.spec mentioned in the original post. I took the version, wget got me from that URL. Looks like that file is not the intendet spec. I've taken rpmlint-0.93-2.fc12.noarch -- 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 542580] Review Request: statusnet - Open Source microblogging platform
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=542580 --- Comment #9 from Steve Milner 2010-02-10 10:13:45 EST --- RPM: http://www.stevemilner.org/images/statusnet/statusnet-0.8.3-2.fc12.noarch.rpm SRPM: http://www.stevemilner.org/images/statusnet/statusnet-0.8.3-2.fc12.src.rpm SPEC: http://www.stevemilner.org/images/statusnet/statusnet.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 542580] Review Request: statusnet - Open Source microblogging platform
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=542580 --- Comment #10 from Steve Milner 2010-02-10 10:15:37 EST --- Also, currently have instructions of how I set up my test instance temporarily at: http://www.stevemilner.org/images/statusnet/ -- 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 562330] Review Request: libnih - Lightweight application development 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=562330 --- Comment #11 from Casey Dahlin 2010-02-10 09:59:24 EST --- http://sadmac.fedorapeople.org/libnih-1.0.1-4.fc12.src.rpm spec also updated. Both issues fixed. -- 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 532521] Review request: trove4j - High performance collections for Java
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=532521 Alexander Kurtakov changed: What|Removed |Added CC||akurt...@redhat.com --- Comment #9 from Alexander Kurtakov 2010-02-10 09:17:32 EST --- AFAIK, jboss is moving away of repolib and because of this repolib packages should not enter Fedora. -- 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 532521] Review request: trove4j - High performance collections for Java
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=532521 --- Comment #8 from Mary Ellen Foster 2010-02-10 09:15:05 EST --- Growl, dammit, sorry -- will get to this asap -- 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 532520] Review request: derby - An open source database implemented entirely in Java
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=532520 Alexander Kurtakov changed: What|Removed |Added CC||akurt...@redhat.com --- Comment #2 from Alexander Kurtakov 2010-02-10 09:12:48 EST --- Do you plan to install the utilities listed in the description to %_bindir? -- 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 563510] Review Request: php-xcache - yet another php cacher
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=563510 Itamar Reis Peixoto changed: What|Removed |Added CC||ita...@ispbrasil.com.br --- Comment #4 from Itamar Reis Peixoto 2010-02-10 08:41:49 EST --- is this your first package ? Can you add a scratch build here ? -- 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 561549] Review Request: eclipse-collabnet-merge - CollabNet Merge Client for Subclipse
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=561549
Mat Booth changed:
What|Removed |Added
Flag||fedora-cvs?
--- Comment #6 from Mat Booth 2010-02-10 08:40:19 EST
---
(In reply to comment #5)
> Mat, FYI in packages like this where there is only one feature there is no
> need
> to specify the feature to build.
> %{eclipse_base}/buildscripts/pdebuild -d "subclipse svnkit" is just enough to
> build
>
Thanks for the tip, I didn't realise.
> Package is APPROVED.
Thanks Alex.
New Package CVS Request
===
Package Name: eclipse-collabnet-merge
Short Description: CollabNet Merge Client for Subclipse
Owners: mbooth
Branches: F-12
--
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 561430] Review Request: geronimo-jta - J2EE JTA v1.1 API
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=561430 Alexander Kurtakov 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 2010-02-10 08:39:35 EST --- 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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780
--- Comment #3 from Alexander Kurtakov 2010-02-10
08:34:54 EST ---
Review:
OK: rpmlint must be run on every package. Warnings:
geronimo-jms.noarch: W: obsolete-not-provided geronimo-specs-compat
geronimo-jms.noarch: W: non-conffile-in-etc /etc/maven/fragments/geronimo-jms
Both are not a problem as we really want to obsolete and the second one is just
how maven works.
OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.
OK: 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 must be included in %doc.
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible.
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. Fetch instructions provided.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates.
OK: A Fedora package must not list a file more than once in the spec file's
%files listings.
OK: Permissions on files must be set properly.
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content.
OK: Large documentation files must go in a -doc subpackage.
OK: If a package includes something as %doc, it must not affect the runtime of
the application.
OK: Packages must not own files or directories already owned by other packages.
OK: At the beginning of %install, each package MUST run rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
OK: All filenames in rpm packages must be valid UTF-8.
Other comments:
* Please remove
# drop the following asap
Provides: jms = 0:1.1
We should not get in smth that should be removed.
* Also remove
# MEF added these herself -- don't know if they're truly necessary
Provides: %spec_name = %{version}-%{release}
Provides: geronimo-jms-1.1-api = %{version}-%{release}
If smth like this is needed we can add them later but I think that we shouldn't
have Provides unless we are absolutely sure we need them.
* %description is not descriptive
Smth like "The Java Message Service (JMS) API is a messaging standard that
allows application components based on the Java 2 Platform, Enterprise Edition
(J2EE) to create, send, receive, and read messages. It enables distributed
communication that is loosely coupled, reliable, and asynchronous. " is way
better. Shamelessly taken from http://java.sun.com/products/jms/index.jsp
* %description javadoc can be better e.g. "API documentation for %{name}."
Otherwise package is good.
--
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 563510] Review Request: php-xcache - yet another php cacher
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=563510 --- Comment #1 from Timon 2010-02-10 08:34:21 EST --- Created an attachment (id=389992) --> (https://bugzilla.redhat.com/attachment.cgi?id=389992) spec file -- 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 563510] New: Review Request: php-xcache - yet another php cacher
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: php-xcache - yet another php cacher https://bugzilla.redhat.com/show_bug.cgi?id=563510 Summary: Review Request: php-xcache - yet another php cacher Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: timo...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Description: XCache is a fast, stable PHP opcode cacher that has been tested and is now running on production servers under high load. -- 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 563510] Review Request: php-xcache - yet another php cacher
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=563510 --- Comment #3 from Timon 2010-02-10 08:35:49 EST --- spec file: https://bugzilla.redhat.com/attachment.cgi?id=389992 srpm : https://bugzilla.redhat.com/attachment.cgi?id=389994 -- 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 563510] Review Request: php-xcache - yet another php cacher
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=563510 --- Comment #2 from Timon 2010-02-10 08:35:02 EST --- Created an attachment (id=389994) --> (https://bugzilla.redhat.com/attachment.cgi?id=389994) srpm -- 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 556780] Review Request: geronimo-jms - J2EE JMS v1.1 API
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=556780 Alexander Kurtakov 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 2010-02-10 08:14:49 EST --- 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 561549] Review Request: eclipse-collabnet-merge - CollabNet Merge Client for Subclipse
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=561549
Alexander Kurtakov changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #5 from Alexander Kurtakov 2010-02-10
08:09:02 EST ---
Review:
OK: rpmlint must be run on every package. No warnings/errors.
OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines .
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
OK: The License field in the package spec file must match the actual license.
OK: 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 must be included in %doc.
OK: The spec file must be written in American English.
OK: The spec file for the package MUST be legible.
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. Shell script for generating the tarball is provided.
OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
OK: All build dependencies must be listed in BuildRequires
OK: Packages must NOT bundle copies of system libraries.
OK: A package must own all directories that it creates.
OK: A Fedora package must not list a file more than once in the spec file's
%files listings.
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line.
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT).
OK: Each package must consistently use macros.
OK: The package must contain code, or permissable content.
OK: If a package includes something as %doc, it must not affect the runtime of
the application.
OK: Packages must not own files or directories already owned by other packages.
OK: At the beginning of %install, each package MUST run rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
OK: All filenames in rpm packages must be valid UTF-8.
Mat, FYI in packages like this where there is only one feature there is no need
to specify the feature to build.
%{eclipse_base}/buildscripts/pdebuild -d "subclipse svnkit" is just enough to
build
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 560457] Review Request: pyutil - A collection of mature utilities for Python programmers
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=560457 --- Comment #2 from Ruben Kerkhof 2010-02-10 07:35:41 EST --- FYI, python-zbase32 is in Rawhide now. -- 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 563444] Review Request: AnumaanLinux-alpha-0.1 - A predictive text entry system
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=563444 manuel wolfshant changed: What|Removed |Added Summary|Review Request: |Review Request: |AnumaanLinux-alpha-0.1 - A |AnumaanLinux-alpha-0.1 - A |predictive text entry |predictive text entry |system (first package, need |system |sponsor)| -- 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 561549] Review Request: eclipse-collabnet-merge - CollabNet Merge Client for Subclipse
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=561549 --- Comment #4 from Alexander Kurtakov 2010-02-10 06:59:10 EST --- OK, I'll reveiw 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 563481] Review Request: aduna-commons-i18n - Internationalization and localization utilities
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=563481 Mary Ellen Foster changed: What|Removed |Added Depends on||561439 -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 Mary Ellen Foster changed: What|Removed |Added Blocks||563481 -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 Mary Ellen Foster changed: What|Removed |Added Blocks||563483 -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 Mary Ellen Foster changed: What|Removed |Added Blocks||563480 -- 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 563479] Review Request: aduna-commons-text - Manipulate/transform/parse text in various ways
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=563479 Mary Ellen Foster changed: What|Removed |Added Depends on||561439 -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 Mary Ellen Foster changed: What|Removed |Added Blocks||563477 -- 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 563480] Review Request: aduna-commons-concurrent - Extensions to the Java Concurrency package
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=563480 Mary Ellen Foster changed: What|Removed |Added Depends on||561439 -- 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 561439] Review Request: aduna-commons-pom - Aduna Commons parent pom
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=561439 Mary Ellen Foster changed: What|Removed |Added Blocks||563479 -- 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
