[Bug 657405] Review Request: lbzip2 - fast, multi-threaded bzip2 utility
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=657405 Lukáš Zapletal changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #5 from Lukáš Zapletal 2010-12-07 02:49:04 EST --- Correcting the fedora-cvs flag. -- 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 604331] Review Request: redshift - Adjusts the color temperature of your screen according to time of day
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=604331 --- Comment #34 from Mamoru Tasaka 2010-12-07 02:31:01 EST --- Alexander, please file a bug report against redshift and not write such comment here (on review request) any longer. This review request finished about 3 months before. -- 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 604331] Review Request: redshift - Adjusts the color temperature of your screen according to time of day
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=604331 Alexander Hunt changed: What|Removed |Added CC||alexander.hunt2...@gmail.co ||m --- Comment #33 from Alexander Hunt 2010-12-07 01:40:39 EST --- redshift-1.5-1.fc14 - (on x86_64 platform) Accessories/Redshift command menu property has to use "sudo gtk-redshift" in order for it to work from the menu, or sudo redshift to work from terminal; without sudo it does not find the clock applet, and not being able to find another provider, errors out. -- 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 641690] Review Request: k4dirstat - Graphical Directory Statistics for Used Disk Space
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=641690 Dmitrij S. Kryzhevich changed: What|Removed |Added Flag||fedora-cvs? --- Comment #13 from Dmitrij S. Kryzhevich 2010-12-07 01:37:11 EST --- New Package SCM Request === Package Name: k4dirstat Short Description: Graphical Directory Statistics for Used Disk Space Owners: krege Branches: f13 f14 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 654374] Review Request: navit - Car navigation system with routing engine
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=654374 --- Comment #25 from viji 2010-12-06 23:43:27 EST --- We need this package in the distro, as there is a huge amount of work is going on to have a "Geo" spin for Fedora 15. There are many people interested to see this package in Fedora, just check the "CC" list of this bug, OSGeo mailing list, Fedora mail thread - "Fedora 15 exciting plans". The upstream guys are very responsive and fixing the bugs quickly, so far there is no "instability" reported regarding the "svn" package. Regarding the GPL violations, I will remove these "violating" stuffs and repack them properly. Let me check the comments from upstream guys also. The maps can be on a separate sub-package, will do it on the next review-update Can I go ahead with the changes and 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 654374] Review Request: navit - Car navigation system with routing engine
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=654374 --- Comment #24 from Ralf Corsepius 2010-12-06 23:23:18 EST --- (In reply to comment #23) > Ralf, > > Thank you so much for your observations. The tarball is just an svn checkout, > no repackaging. Then let me rephrase it: Your tarballs violate the GPLv2. > The files you have mentioned are included by the upstream. Right, but it's you who is tarring up their sources. > Anyway I will check it and discuss with them, also will compare with the > latest SVN. There actually are several issues at once: - Them shipping Apache-v2-licensed works (android) bundled with GPLv2 licenced works violates the GPLv2. - Them bundling fonts and android code is a mistake. > Hope you have gone through my previous comments and change-logs, the reason > behind the SVN version: > > - The stable branch is not that much "stable" with all supported Fedora > versions like 13/14/15, specifically qt breaks. > - The bug fixes are been made to SVN and not porting properly back to the > stable release. > - Navit upstream also recommend to use the SVN version > - Since this project is very rapidly evolving users are very keen to test the > latest features and enhancements. > > Conclusion: Will Switch back to stable after 1.0 release. My conclusion: ATM, this package is too immature for inclusion into a distro. > Maps are not on a separate package now, because: > > - For quick testing, navit is very rapidly evolving. Seems as if we are facing a miscommunication: I want you to package them as a noarch _subpackage_ of navit. (With my FPC head on) You currently are shipping a big, optional data file as part of an application package. > - People expect to see "some" maps quickly after the installation Yes, the way upstream currently is shipping maps is not really useful. They need to do something about it. > - The maps directory contains the POI icons as well, in different formats - > xpm, svg, png etc, need to differentiate and repack all of them properly Again, I am talking about the actual map file. >Conclusion: Up to 1.0 we will keep this in the same package Under these circumstances, you can consider this review failed. -- 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 637877] Rename request: meego-panel-status - MeeGo Panel for Social Network Status
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=637877 Adam Williamson changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|awill...@redhat.com --- Comment #8 from Adam Williamson 2010-12-06 22:56:30 EST --- Some build issues here. Trying to build in mock in Rawhide gives: configure: error: Package requirements (dbus-glib-1 clutter-x11-1.0 >= 1.0.0clutter-gtk-0.10 libsocialweb-client >= 0.25.3 mx-1.0 >= 0.9.0 champlain-0.6 geoclue gconf-2.0) were not met: No package 'clutter-gtk-0.10' found No package 'champlain-0.6' found Rawhide clutter-gtk provides clutter-gtk-1.0 (not 0.10) and Rawhide champlain provides champlain-0.8 . I expect it'd build on F14 so this doesn't technically block the review, but since we're targeting F15 with the Meego stuff, it'd probably be best to grab some newer upstream code or something which can work with the newer clutter-gtk and champlain in Rawhide, if that's possible. Aside from the above, this looks 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 592500] Review Request: libeatmydata - Disable fsync() and friends
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=592500 --- Comment #2 from Zing 2010-12-06 22:54:22 EST --- Although I submitted this, if anyone wants to pick this up and maintain this for fedora, they are welcome to. -- 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 659856] Review Request:eclipse-vrapper - Vim-like editing in Eclipse
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=659856 --- Comment #6 from Jesse Keating 2010-12-06 21:24:16 EST --- Spec URL: http://jkeating.fedorapeople.org/review/eclipse-vrapper/eclipse-vrapper.spec SRPM URL: http://jkeating.fedorapeople.org/review/eclipse-vrapper/eclipse-vrapper-0.15.0-0.2.svn351.fc14.src.rpm New stuff posted. -- 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 609295] Review Request: cement - CLI Application Framework 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=609295 --- Comment #1 from BJ Dierkes 2010-12-06 21:09:06 EST --- Updated: SPEC: http://5dollarwhitebox.org/tmp/cement.spec SRPM: http://5dollarwhitebox.org/tmp/cement-0.8.12-1.fc14.src.rpm $ rpmlint -i SPECS/cement.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint -i RPMS/noarch/cement*0.8.12*.rpm 2 packages and 0 specfiles checked; 0 errors, 0 warnings. -- 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 659856] Review Request:eclipse-vrapper - Vim-like editing in Eclipse
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=659856 --- Comment #5 from Jesse Keating 2010-12-06 21:01:30 EST --- Erm, getting some errors on the jdt build: generateScript: [eclipse.buildScript] Some inter-plug-in dependencies have not been satisfied. [eclipse.buildScript] Bundle net.sourceforge.vrapper.aptana.ide: [eclipse.buildScript] Missing required plug-in com.aptana.ide.editor.html_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.editors_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.core_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.core.ui_1.5.0. [eclipse.buildScript] Bundle net.sourceforge.vrapper.eclipse.cdt: [eclipse.buildScript] Missing required plug-in org.eclipse.cdt.ui_5.1.1. And more for cdt: generateScript: [eclipse.buildScript] Some inter-plug-in dependencies have not been satisfied. [eclipse.buildScript] Bundle net.sourceforge.vrapper.aptana.ide: [eclipse.buildScript] Missing required plug-in com.aptana.ide.editor.html_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.editors_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.core_1.5.0. [eclipse.buildScript] Missing required plug-in com.aptana.ide.core.ui_1.5.0. [eclipse.buildScript] Bundle org.eclipse.cdt.launch.remote: [eclipse.buildScript] Missing required plug-in org.eclipse.rse.ui_[3.0.0,4.0.0). [eclipse.buildScript] Missing required plug-in org.eclipse.rse.subsystems.shells.core_[3.0.0,4.0.0). [eclipse.buildScript] Missing required plug-in org.eclipse.rse.subsystems.files.core_[3.0.0,4.0.0). [eclipse.buildScript] Missing required plug-in org.eclipse.rse.core_[3.0.0,4.0.0). [eclipse.buildScript] Missing required plug-in org.eclipse.rse.services_[3.0.0,4.0.0). [eclipse.buildScript] Missing required plug-in org.eclipse.rse.files.ui_[3.0.0,4.0.0). [eclipse.buildScript] Bundle org.eclipse.tm.tcf.rse: [eclipse.buildScript] Missing required plug-in org.eclipse.rse.core_0.0.0. [eclipse.buildScript] Missing required plug-in org.eclipse.rse.ui_0.0.0. [eclipse.buildScript] Missing required plug-in org.eclipse.rse.services_0.0.0. [eclipse.buildScript] Missing required plug-in org.eclipse.rse.subsystems.files.core_0.0.0. [eclipse.buildScript] Missing required plug-in org.eclipse.rse.subsystems.processes.core_0.0.0. [eclipse.buildScript] Missing required plug-in org.eclipse.rse.processes.ui_0.0.0. I think I'll leave those disabled and just do the core and surround stuff. -- 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 659896] Review Request: cp2k - A molecular dynamics engine capable of classical and Car-Parrinello simulations
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=659896 Jussi Lehtola changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Jussi Lehtola 2010-12-06 20:59:21 EST --- (In reply to comment #5) > All done. Why did you write: > > > SHOULD: If the package does not include license text(s) as separate files > > from > > upstream, the packager should query upstream to include it. NEEDSWORK > > ? Because of the item in http://fedoraproject.org/wiki/Packaging/ReviewGuidelines and because the GPL license is not included (only the COPYRIGHT) which is a different thing. Okay, compilation is now against ATLAS and libint support seems to be OK. I haven't run a test calculation, however; maybe I'll get the time to do that in the near future. The package has been 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 592487] Review Request: ffgtk - A solution for controlling Fritz!Box or compatible routers
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=592487 --- Comment #8 from Jason Tibbitts 2010-12-06 20:45:05 EST --- Sure, just clear the whiteboard when you've updated the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 580480] Review Request: ghc-type-level - A type-level library for Haskell
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=580480 --- Comment #23 from Jens Petersen 2010-12-06 19:38:04 EST --- > You forgot to request "InitialCC: haskell-sig". added -- 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 612076] Review Request: ghc-feldspar-language - Functional Embedded Language for DSP and PARallelism
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=612076 --- Comment #22 from Jens Petersen 2010-12-06 19:35:53 EST --- > "InitialCC: haskell-sig" would be good. added -- 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 659812] Review Request: libindicator - Shared functions for Ayatana indicators
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=659812 Jef Spaleta changed: What|Removed |Added CC||jspal...@gmail.com AssignedTo|nob...@fedoraproject.org|jspal...@gmail.com Flag||fedora-review+ --- Comment #1 from Jef Spaleta 2010-12-06 19:10:13 EST --- APPROVED for rawhide (f15+) checksum of source url matches checksum of included source tarball. sha512sum: c94ad89a5afc980e50f8ab83784b052c3ba1c66fb2d2ddf4201d225c577be78422d3ae3f7bf66cbf47745abc7b2e1263b24a9cfa625f72e4c5b2a2c7769319d7 license tag agrees with project homepage licensing and copying file in upstream source tree. package and subpackage naming is good. local mock builds against 64bit rawhide succeed. F-13 and F-14 fail due to the gtk3 requirement. I get the same rpmlint warning messages and concur with the explanations given. The main binary package looks sane with regard to ownership and payload. The devel and gtk3-devel packages look sane with regard to ownership and payload tool subpackages look sane. rpm scriptlets look good for ldconfig for gtk2 and gtk3 library payloads. The only executable from the tools and gtk3-tools subpackages are located in libexecdir has correct permissions. No end-user executables that need a desktopfile. gtk2 and gtk3 subpackages install side by side with no conflicts -- 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 652571] Review Request: ghc-digest - cryptographic hashes of bytestrings
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=652571 Jens Petersen changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-cvs? --- Comment #3 from Jens Petersen 2010-12-06 19:08:52 EST --- Thanks for the review. :) New Package SCM Request === Package Name: ghc-digest Short Description: Haskell hash digest library Owners: petersen Branches: f13 f14 InitialCC: haskell-sig -- 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 659856] Review Request:eclipse-vrapper - Vim-like editing in Eclipse
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=659856 --- Comment #4 from Jesse Keating 2010-12-06 19:07:07 EST --- I didn't consider those because they weren't built in the upstream vrapper packages. I don't know what their stability status is, but I'll try 'em. -- 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 427121] Review Request: grib_api - ECMWF encoding/decoding GRIB software
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=427121 --- Comment #18 from Orion Poplawski 2010-12-06 18:53:13 EST --- Sorry - Package Change Request == Package Name: grib_api New Branches: el6 el5 Owners: orion -- 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 427121] Review Request: grib_api - ECMWF encoding/decoding GRIB software
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=427121 Orion Poplawski changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #17 from Orion Poplawski 2010-12-06 18:52:00 EST --- New Package SCM Request === Package Name: grib_api Short Description: ECMWF encoding/decoding GRIB software Owners: orion Branches: el6 el5 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543718] Review Request: wxmacmolplt - A graphics program for plotting 3-D molecular structures and normal modes
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=543718 Dominik 'Rathann' Mierzejewski changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Dominik 'Rathann' Mierzejewski 2010-12-06 18:43:52 EST --- Excellent! Big thanks for reviewing this. I promise I'll review a package by the end of the week. New Package SCM Request === Package Name: wxmacmolplt Short Description: A graphics program for plotting 3-D molecular structures and normal modes Owners: rathann Branches: f13 f14 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659895] Review Request: makedepf90 - Create Makefile dependency list for Fortran source files
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=659895 Dominik 'Rathann' Mierzejewski changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Dominik 'Rathann' Mierzejewski 2010-12-06 18:40:08 EST --- Excellent! Thanks for the speedy review. I'll fix the timestamp issue on import. New Package SCM Request === Package Name: makedepf90 Short Description: Create Makefile dependency list for Fortran source files Owners: rathann Branches: f13 f14 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659896] Review Request: cp2k - A molecular dynamics engine capable of classical and Car-Parrinello simulations
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=659896 Dominik 'Rathann' Mierzejewski changed: What|Removed |Added Summary|Review Request: cp2k - |Review Request: cp2k - A |Molecular simulations |molecular dynamics engine |software|capable of classical and ||Car-Parrinello simulations --- Comment #5 from Dominik 'Rathann' Mierzejewski 2010-12-06 18:33:12 EST --- Spec URL: http://rathann.fedorapeople.org/review/cp2k.spec SRPM URL: http://rathann.fedorapeople.org/review/cp2k-2.1-2.20101006.fc13.src.rpm All done. Why did you write: > SHOULD: If the package does not include license text(s) as separate files from > upstream, the packager should query upstream to include it. NEEDSWORK ? -- 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 647602] Review Request: ant-antunit - Provide antunit ant task
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=647602 --- Comment #9 from Orion Poplawski 2010-12-06 18:00:09 EST --- Good catch on the Provides. 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 647602] Review Request: ant-antunit - Provide antunit ant task
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=647602
Alexander Kurtakov changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #7 from Alexander Kurtakov 2010-12-06
17:54:21 EST ---
Please drop the following line when you commit:
Provides: %{name} = %{version}-%{release}
It is not needed.
The 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 647602] Review Request: ant-antunit - Provide antunit ant task
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=647602 Orion Poplawski changed: What|Removed |Added Flag||fedora-cvs? -- 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 647602] Review Request: ant-antunit - Provide antunit ant task
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=647602 --- Comment #8 from Orion Poplawski 2010-12-06 17:55:59 EST --- New Package SCM Request === Package Name: ant-antunit Short Description: Provide antunit ant task Owners: orion Branches: f14 f13 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647602] Review Request: ant-antunit - Provide antunit ant task
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=647602 Alexander Kurtakov changed: What|Removed |Added Summary|Review Request: |Review Request: ant-antunit |apache-ant-antunit -|- Provide antunit ant task |Provide antunit ant task| -- 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 647602] Review Request: apache-ant-antunit - Provide antunit ant task
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=647602 --- Comment #6 from Orion Poplawski 2010-12-06 17:47:25 EST --- http://www.cora.nwra.com/~orion/fedora/ant-antunit.spec http://www.cora.nwra.com/~orion/fedora/ant-antunit-1.1-3.fc14.src.rpm * Mon Dec 6 2010 Orion Poplawski 1.1-3 - Rename to ant-antunit - Drop BuildRoot and %%clean -- 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 647602] Review Request: apache-ant-antunit - Provide antunit ant task
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=647602
--- Comment #5 from Alexander Kurtakov 2010-12-06
17:37:39 EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Rpmlint output:
apache-ant-antunit.noarch: W: spelling-error %description -l en_US junit ->
unit, j unit, jun it
apache-ant-antunit.noarch: W: spelling-error %description -l en_US setUp -> set
Up, setup, setups
apache-ant-antunit.noarch: W: spelling-error %description -l en_US tearDown ->
tear Down, tear-Down, tear-down
apache-ant-antunit.noarch: W: non-conffile-in-etc
/etc/maven/fragments/apache-ant-antunit
Not a problem
[!] Package is named according to the Package Naming Guidelines. I still think
it's better to name it just ant-antunit for the sake of consistency with the
rest of the ant packages.
[x] Spec file name must match the base package name, in the format
%{name}.spec.
[!] Package meets the Packaging Guidelines[2]. Remove versioned jar and
javadoc.
[x] Package successfully compiles and builds into binary rpms.
[!] Buildroot definition is not present
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type: ASL 2.0
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package:6a9b3db4333be1f2b45b11917811da5e
MD5SUM upstream package:6a9b3db4333be1f2b45b11917811da5e
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[!] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[!] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[x] Javadoc subpackages have Require: jpackage-utils
[x] Package uses %global not %define
[-] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[x] All filenames in rpm packages must be valid UTF-8.
[!] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x] If package contains pom.xml files install it (including depmaps) even when
building with ant
[x] pom files has correct add_to_maven_depmap call which resolves to the pom
file (use "JPP." and "JPP-" correctly)
=== Maven ===
[x] Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[-] If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[-] If package uses custom depmap "-Dmaven2.jpp.depmap.file=*" explain why
it's needed in a comment
[x] Package uses %update_maven_depmap in %post/%postun
[x] Packages have Requires(post) and Requires(postun) on jpackage-utils (for
%update_maven_depmap macro)
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[x] Latest version is packaged.
=== Issues ===
1. Naming. Can we drop apache to match the rest of the ant packages?
2. Don't install versioned jar and javadoc.
3. Remove clean section and buildroot tag.
--
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 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 Jef Spaleta changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|jspal...@gmail.com Flag||fedora-review+ --- Comment #7 from Jef Spaleta 2010-12-06 17:33:56 EST --- Looks good. Builds without error on 64bit rawhide under mock. Binary packages appear well formed with correct ownership for files and directories. No executables or associated desktop files to worry about. ldconfig called in scriptlets for install libraries. Minimal docs in the main package with COPYING file -devel subpackage looks good. rpmlint runs have ignorable warnings. Passes review for rawhide and fc15+ -- 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 647602] Review Request: apache-ant-antunit - Provide antunit ant task
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=647602 Alexander Kurtakov changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com Flag||fedora-review? --- Comment #4 from Alexander Kurtakov 2010-12-06 17:17:39 EST --- I'll do this one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615428] Review Request: olfs - OPeNDAP Lightweight Frontend Servlet - client interface for Hyrax
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=615428 Orion Poplawski changed: What|Removed |Added Status Whiteboard|BuildFails | -- 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 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 --- Comment #6 from Adam Williamson 2010-12-06 16:24:34 EST --- that's fine, I wasn't really planning on targeting F14. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 --- Comment #5 from Jef Spaleta 2010-12-06 16:10:40 EST --- This is lovely. When trying to build dee against F14 I get an error concerning DBus-1.0.gir not available. On rawhide this is available in gobject-introspection-devel On F13 this is availabe in gir-repository-devel On F14 it appears to be unavailable from any package. Not a review blocker. Just be aware that once dee goes in it is going to be F15+ only. -jef -- 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 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash 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=656363 --- Comment #6 from Fedora Update System 2010-12-06 16:05:45 EST --- perl-Package-Stash-XS-0.17-2.el4 has been submitted as an update for Fedora EPEL 4. https://admin.fedoraproject.org/updates/perl-Package-Stash-XS-0.17-2.el4 -- 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 629664] Review Request: jtnef - Java TNEF 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=629664 Bug 629664 depends on bug 629660, which changed state. Bug 629660 Summary: Review Request: apache-poi - The Java API for Microsoft Documents https://bugzilla.redhat.com/show_bug.cgi?id=629660 What|Old Value |New Value Resolution|NEXTRELEASE | Status|CLOSED |ASSIGNED -- 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 629660] Review Request: apache-poi - The Java API for Microsoft Documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=629660 Orion Poplawski changed: What|Removed |Added Status|CLOSED |ASSIGNED Resolution|NEXTRELEASE | Keywords||Reopened --- Comment #17 from Orion Poplawski 2010-12-06 15:54:36 EST --- Oh, right, I'm waiting for the build system to be able to handle files with spaces in it. According to https://fedorahosted.org/fedora-packager/ticket/85, this should be 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 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 --- Comment #4 from Jef Spaleta 2010-12-06 15:36:04 EST --- Indeed. i missed that. Yippie for expansive guidance! spec file looks good. I just need finish the local mock build and confirm binary build stuff on a primary arch. -jef -- 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 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 Adam Williamson changed: What|Removed |Added CC||tcall...@redhat.com --- Comment #3 from Adam Williamson 2010-12-06 15:24:49 EST --- "Just FYI, I don't think I can pass this through review until there is at least a comment from upstream concerning the missing license headers." I think we're generally okay to ship source which has no specific header but is clearly marked with an acceptable license in other ways by upstream (dee is on the project page, and by the inclusion of the license files in the tarball); having a header on each specific source file is a 'nice-to-have', not a must. But I'll CC spot to check this. "It's not just the binary..we do distribute the srpm's as well and the licensing tag has to make sense for both the srpm and the binary rpm." No, it doesn't. http://fedoraproject.org/wiki/Packaging/LicensingGuidelines "The License: field refers to the licenses of the contents of the *binary* rpm. When in doubt, ask." -- 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 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies
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=612581 --- Comment #27 from Fedora Update System 2010-12-06 15:04:44 EST --- spacewalk-backend-1.2.74-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 549590] Review Request: php-pChart - A PHP class to build charts.
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=549590 --- Comment #21 from Jeffrey Ness 2010-12-06 15:04:20 EST --- Any progress on this package? My package is at a standstill waiting on it.. Jeffrey- -- 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 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies
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=612581 Fedora Update System changed: What|Removed |Added Fixed In Version|spacewalk-backend-1.2.74-2. |spacewalk-backend-1.2.74-2. |fc14|fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226650] Merge Review: xorg-x11-xdm
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=226650 Bug 226650 depends on bug 237621, which changed state. Bug 237621 Summary: xdm needs to use consolekit https://bugzilla.redhat.com/show_bug.cgi?id=237621 What|Old Value |New Value Resolution||WONTFIX Status|ASSIGNED|CLOSED Resolution|WONTFIX | Status|CLOSED |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491331] Review Request: spacewalk-config - Spacewalk Configuration
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=491331 Fedora Update System changed: What|Removed |Added Fixed In Version|spacewalk-config-1.2.7-1.fc |spacewalk-config-1.2.7-1.fc |13 |14 -- 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 596593] Review Request: ibus-table-chinese - Chinese tables for IBus-Table
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=596593 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #19 from Fedora Update System 2010-12-06 15:02:02 EST --- ibus-table-chinese-1.3.0.20101206-1.fc14 has been pushed to the Fedora 14 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update ibus-table-chinese'. You can provide feedback for this update here: https://admin.fedoraproject.org/updates/ibus-table-chinese-1.3.0.20101206-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies
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=612581 Fedora Update System changed: What|Removed |Added Fixed In Version||spacewalk-backend-1.2.74-2. ||fc14 Resolution|NEXTRELEASE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 583105] Review Request: djmount - Mount MediaServers content as a Linux 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=583105 Fedora Update System changed: What|Removed |Added Fixed In Version|djmount-0.71-2.fc14 |djmount-0.71-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 612581] Review Request: spacewalk-backend - Common programs needed to be installed on the Spacewalk servers/proxies
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=612581 --- Comment #26 from Fedora Update System 2010-12-06 14:57:34 EST --- spacewalk-backend-1.2.74-2.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 583105] Review Request: djmount - Mount MediaServers content as a Linux 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=583105 --- Comment #9 from Fedora Update System 2010-12-06 15:01:44 EST --- djmount-0.71-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491331] Review Request: spacewalk-config - Spacewalk Configuration
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=491331 --- Comment #19 from Fedora Update System 2010-12-06 15:00:56 EST --- spacewalk-config-1.2.7-1.fc14 has been pushed to the Fedora 14 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491331] Review Request: spacewalk-config - Spacewalk Configuration
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=491331 Fedora Update System changed: What|Removed |Added Fixed In Version||spacewalk-config-1.2.7-1.fc ||13 Resolution|NEXTRELEASE |ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 491331] Review Request: spacewalk-config - Spacewalk Configuration
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=491331 --- Comment #18 from Fedora Update System 2010-12-06 14:57:20 EST --- spacewalk-config-1.2.7-1.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 629660] Review Request: apache-poi - The Java API for Microsoft Documents
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=629660 --- Comment #16 from Terje Røsten 2010-12-06 14:53:03 EST --- git seems fine, however I can't find any useful builds in koji? -- 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 659746] Review Request: dee - Model to synchronize multiple instances over DBus
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=659746 Jef Spaleta changed: What|Removed |Added CC||jspal...@gmail.com --- Comment #2 from Jef Spaleta 2010-12-06 14:44:44 EST --- Just FYI, I don't think I can pass this through review until there is at least a comment from upstream concerning the missing license headers. At least a clarification of intention in the upstream report would be enough for me as a archived statement on record. That being said.. I think you have to leave the GPLv3 in the license field because the srpm does ship with the example and test code. It's not just the binary..we do distribute the srpm's as well and the licensing tag has to make sense for both the srpm and the binary rpm. -jef -- 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 654374] Review Request: navit - Car navigation system with routing engine
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=654374 --- Comment #23 from viji 2010-12-06 14:39:14 EST --- Ralf, Thank you so much for your observations. The tarball is just an svn checkout, no repackaging. The files you have mentioned are included by the upstream. Anyway I will check it and discuss with them, also will compare with the latest SVN. Hope you have gone through my previous comments and change-logs, the reason behind the SVN version: - The stable branch is not that much "stable" with all supported Fedora versions like 13/14/15, specifically qt breaks. - The bug fixes are been made to SVN and not porting properly back to the stable release. - Navit upstream also recommend to use the SVN version - Since this project is very rapidly evolving users are very keen to test the latest features and enhancements. Conclusion: Will Switch back to stable after 1.0 release. Maps are not on a separate package now, because: - For quick testing, navit is very rapidly evolving. - People expect to see "some" maps quickly after the installation - The maps directory contains the POI icons as well, in different formats - xpm, svg, png etc, need to differentiate and repack all of them properly Conclusion: Up to 1.0 we will keep this in the same package -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 654374] Review Request: navit - Car navigation system with routing engine
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=654374 --- Comment #22 from viji 2010-12-06 14:34:21 EST --- Michel, Thank you so much for your review. Will update the ticket with the new spec and srpm after fixing all the issues mentioned. -- 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 526034] Review Request: ocaml-xmlm - OCaml library for streaming XML input and output
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=526034 Jason Tibbitts changed: What|Removed |Added Blocks||201449(FE-DEADREVIEW) Flag|fedora-review? | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561484] Review Request: jruby - Pure Java implementation of the Ruby 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=561484 --- Comment #8 from Mohammed Morsi 2010-12-06 13:51:41 EST --- Thanks for the review. Updated the package SPEC URL: http://mo.morsi.org/files/jruby/jruby.spec SRPM URL: http://mo.morsi.org/files/jruby/jruby-1.5.6-1.fc13.src.rpm Issues 1-7 have been taken care of. As far as issue #8, I don't think we will be able to do this. Even though JRuby is a standard ruby interpreter there are enough differences between it and MRI (the main / official ruby interpreter) that each gem and ruby library/application needs to explicitly support one or the other or both. I don't think we can just use the stock ruby libraries shipped for MRI with JRuby. -- 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 526034] Review Request: ocaml-xmlm - OCaml library for streaming XML input and output
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=526034 Richard W.M. Jones changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||DEFERRED Last Closed||2010-12-06 13:48:34 --- Comment #6 from Richard W.M. Jones 2010-12-06 13:48:34 EST --- I'm going to close this one .. I don't have a use for it any more since I'm trying to get rid of XML from my life. -- 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 629664] Review Request: jtnef - Java TNEF 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=629664 Alexander Kurtakov changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Alexander Kurtakov 2010-12-06 12:49:34 EST --- Thanks. This package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 656010] Review Request: libsrtp - An implementation of the Secure Real-time Transport Protocol (SRTP)
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=656010 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 629664] Review Request: jtnef - Java TNEF 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=629664 Orion Poplawski changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Orion Poplawski 2010-12-06 13:01:55 EST --- Thanks! New Package SCM Request === Package Name: jtnef Short Description: Java TNEF package Owners: orion Branches: f14 f13 el6 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 656010] Review Request: libsrtp - An implementation of the Secure Real-time Transport Protocol (SRTP)
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=656010 --- Comment #8 from Fedora Update System 2010-12-06 12:48:21 EST --- libsrtp-1.4.4-1.20101004cvs.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/libsrtp-1.4.4-1.20101004cvs.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 427483] Review Request: publican-jboss - JBoss Theme
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=427483 --- Comment #21 from Jason Tibbitts 2010-12-06 13:01:47 EST --- Is there any reason why this ticket is still open? -- 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 600517] Review Request: R-coda - coda: Output analysis and diagnostics for MCMC
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=600517 --- Comment #11 from Jason Tibbitts 2010-12-06 13:29:22 EST --- I happen to have a need for this package, and an also a sponsor. I see you've submitted several packages, but at least jags seems to be held up in review. Are you still interested in submitting these packages to Fedora? If so I'll try to work out what the hold up is. -- 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 529387] Review Request: rcrpanel - Create a front panel for an electronics device
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=529387 --- Comment #22 from Jason Tibbitts 2010-12-06 13:31:36 EST --- It will, but only if you provide the ticket number when you make your update. -- 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 656010] Review Request: libsrtp - An implementation of the Secure Real-time Transport Protocol (SRTP)
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=656010 --- Comment #7 from Fedora Update System 2010-12-06 12:48:13 EST --- libsrtp-1.4.4-1.20101004cvs.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/libsrtp-1.4.4-1.20101004cvs.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash 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=656363 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash 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=656363 --- Comment #4 from Fedora Update System 2010-12-06 13:19:09 EST --- perl-Package-Stash-XS-0.17-2.fc13 has been submitted as an update for Fedora 13. https://admin.fedoraproject.org/updates/perl-Package-Stash-XS-0.17-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash 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=656363 --- Comment #5 from Fedora Update System 2010-12-06 13:19:15 EST --- perl-Package-Stash-XS-0.17-2.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/perl-Package-Stash-XS-0.17-2.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 529387] Review Request: rcrpanel - Create a front panel for an electronics device
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=529387 John J. McDonough changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE Last Closed||2010-12-06 13:15:12 --- Comment #21 from John J. McDonough 2010-12-06 13:15:12 EST --- No, I was sort of expecting Bodhi to do it for 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 660393] New: Review Request: netxen-firmware - QLogic Linux Intelligent Ethernet (3000 and 3100 Series) Adapter Firmware
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: netxen-firmware - QLogic Linux Intelligent Ethernet (3000 and 3100 Series) Adapter Firmware https://bugzilla.redhat.com/show_bug.cgi?id=660393 Summary: Review Request: netxen-firmware - QLogic Linux Intelligent Ethernet (3000 and 3100 Series) Adapter Firmware Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://www.auroralinux.org/people/spot/review/new/netxen-firmware.spec SRPM URL: http://www.auroralinux.org/people/spot/review/new/netxen-firmware-4.0.534-1.fc14.src.rpm Description: QLogic Linux Intelligent Ethernet (3000 and 3100 Series) Adapter Firmware -- 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 526034] Review Request: ocaml-xmlm - OCaml library for streaming XML input and output
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=526034 --- Comment #5 from Jason Tibbitts 2010-12-06 13:01:01 EST --- Did you still want to continue with this package submission, or should we just close this ticket? -- 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 529387] Review Request: rcrpanel - Create a front panel for an electronics device
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=529387 --- Comment #20 from Jason Tibbitts 2010-12-06 13:02:49 EST --- Is there any reason why this ticket is still open? -- 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 629664] Review Request: jtnef - Java TNEF 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=629664 --- Comment #5 from Orion Poplawski 2010-12-06 12:35:09 EST --- * Mon Dec 6 2010 Orion Poplawski - 1.6.0-3 - Drop versioned jar and javadoc - Remove BuildRoot and %%clean http://www.cora.nwra.com/~orion/fedora/jtnef-1.6.0-2.fc13.src.rpm http://www.cora.nwra.com/~orion/fedora/jtnef.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 629664] Review Request: jtnef - Java TNEF 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=629664
--- Comment #4 from Alexander Kurtakov 2010-12-06
12:22:41 EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Rpmlint output:
jtnef.noarch: W: spelling-error %description -l en_US tnef -> tuneful, TEFL,
tine
jtnef-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs -> Java docs,
Java-docs, Javanese
Not a problem.
[x] Package is named according to the Package Naming Guidelines[1].
[x] Spec file name must match the base package name, in the format
%{name}.spec.
[!] Package meets the Packaging Guidelines[2].
Remove the versioned jar file and javadoc folder.
[x] Package successfully compiles and builds into binary rpms.
[!] Buildroot definition is not present
We don't need buildroot nowadays (it's even ignored).
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x] License field in the package spec file matches the actual license.
License type:GPLv2+
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] All independent sub-packages have license of their own
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package:790cf94af9d5ab859fddee09f4cd884c
MD5SUM upstream package:790cf94af9d5ab859fddee09f4cd884c
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[!] Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x] Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x] Package contains code, or permissable content.
[-] Fully versioned dependency in subpackages, if present.
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
[x] Javadoc documentation files are generated and included in -javadoc
subpackage
[!] Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x] Packages have proper BuildRequires/Requires on jpackage-utils
[x] Javadoc subpackages have Require: jpackage-utils
[-] Package uses %global not %define
[-] If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-] If source tarball includes bundled jar/class files these need to be
removed prior to building
[x] All filenames in rpm packages must be valid UTF-8.
[x] Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[-] If package contains pom.xml files install it (including depmaps) even when
building with ant
[-] pom files has correct add_to_maven_depmap call which resolves to the pom
file (use "JPP." and "JPP-" correctly)
=== Other suggestions ===
[x] If possible use upstream build method (maven/ant/javac)
[x] Avoid having BuildRequires on exact NVR unless necessary
[x] Package has BuildArch: noarch (if possible)
[x] Latest version is packaged.
[x] Reviewer should test that the package builds in mock.
=== Issues ===
1. Remove versioned jar and javadoc.
2. Remove BuildRoot.
3. Remove clean section.
Overall the package is quite 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 650717] Review Request: PolicyKit-olpc - OLPC-specific PolicyKit overrides
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=650717 Daniel Drake changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Daniel Drake 2010-12-06 12:05:53 EST --- Thanks Peter! Admins: this once-broken package is now finally fixed and ready for use once again. Please add a F14 branch, and unblock it from being built in rawhide. Package Change Request == Package Name: PolicyKit-olpc New Branches: F14 Unblock Branches: 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 634037] Review Request: ghc-MissingH - Large utility 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=634037 Ben Boeckel changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Ben Boeckel 2010-12-06 12:07:07 EST --- Thanks. Will apply the patch when importing. New Package CVS Request === Package Name: ghc-MissingH Short Description: Large utility library Owners: mathstuf Branches: F-13 F-14 InitialCC: haskell-sig -- 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 629664] Review Request: jtnef - Java TNEF 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=629664 Alexander Kurtakov changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|akurt...@redhat.com Flag||fedora-review? --- Comment #3 from Alexander Kurtakov 2010-12-06 11:58:59 EST --- I'll take 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 660095] Review Request: impressive - A program that displays presentation slides
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=660095 --- Comment #12 from Michael J Gruber 2010-12-06 11:57:33 EST --- (In reply to comment #11) > "unretiring" being some mix of the "new submission" (e.g. new bug) and > "update" > (e.g. existing repo) processes, I'm afraid I'll trip over the next wire > shortly. such as this one: Building impressive-0.10.3-3.fc15 for dist-rawhide Created task: 2647367 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=2647367 Watching tasks (this may be safely interrupted)... 2647367 build (dist-rawhide, /impressive:3209089b8a5eec9ff3720599e9353223d5312348): free 2647367 build (dist-rawhide, /impressive:3209089b8a5eec9ff3720599e9353223d5312348): free -> open (x86-14.phx2.fedoraproject.org) 2647368 buildSRPMFromSCM (/impressive:3209089b8a5eec9ff3720599e9353223d5312348): free 2647368 buildSRPMFromSCM (/impressive:3209089b8a5eec9ff3720599e9353223d5312348): free -> open (x86-05.phx2.fedoraproject.org) 2647368 buildSRPMFromSCM (/impressive:3209089b8a5eec9ff3720599e9353223d5312348): open (x86-05.phx2.fedoraproject.org) -> closed 0 free 1 open 1 done 0 failed 2647367 build (dist-rawhide, /impressive:3209089b8a5eec9ff3720599e9353223d5312348): open (x86-14.phx2.fedoraproject.org) -> FAILED: BuildError: package impressive is blocked for tag dist-f15 0 free 0 open 1 done 1 failed 2647367 build (dist-rawhide, /impressive:3209089b8a5eec9ff3720599e9353223d5312348) failed Maybe some part of the infrastructure is updated (allowing me to push and build) and other isn't (denying me to push the build). I'll try again later. -- 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 592487] Review Request: ffgtk - A solution for controlling Fritz!Box or compatible routers
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=592487 --- Comment #7 from Louis Lagendijk 2010-12-06 11:52:12 EST --- Sorry for the delay: I am traveling a lot over the last weeks. I have already made most for the updates and had one additional question outstanding with the author. I plan to get back to packaging ffgtk next week or the week after (depending on whether I will be on business trip next week) -- 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 572515] Review Request: jogl - Java bindings for OpenGL
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=572515 --- Comment #34 from Sylvestre Ledru 2010-12-06 11:51:18 EST --- "Reference antlr.classpath not found." That means that antlr cannot be found from ant. I guess the dependency is explicit, it is probably that the ant declaration of usage of antlr is wrong. -- 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 660095] Review Request: impressive - A program that displays presentation slides
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=660095 --- Comment #11 from Michael J Gruber 2010-12-06 11:49:17 EST --- (In reply to comment #9) > This package already has an f14 branch, so I can't process that request > as-is. > I'm guessing that you were asking for the package to be unretired, but if > that's the case then it would have been helpful for you to say that. Well, that's why I said "Taking over an orphaned+retired F13 package". Sorry for avoiding the trigger word "unretire". > I've unretired the f14 and master branches; you should log into pkgdb and > claim > them. I will create the el6 branch for you now. Thanks. Claimed them, pushing and building. "unretiring" being some mix of the "new submission" (e.g. new bug) and "update" (e.g. existing repo) processes, I'm afraid I'll trip over the next wire shortly. -- 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 660024] Review Request: libepsilon - Powerful wavelet image compressor
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=660024 --- Comment #2 from viji 2010-12-06 11:51:25 EST --- Looks fine, few things: 1. Can we have the following support as well, I am sure that there will be a bug report if don not build the libraries with these, better to handle this time. --enable-pthreads Enable POSIX threads support [default=no] --enable-clusterEnable cluster mode [default=no] --enable-mpiEnable MPI support [default=no] 2. Include the licence file also, COPYING, in doc -- 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 629443] Review Request: jackson - A JSON-processor package 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=629443 Stanislav Ochotnicky changed: What|Removed |Added Flag|fedora-review? | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 629443] Review Request: jackson - A JSON-processor package 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=629443 Stanislav Ochotnicky changed: What|Removed |Added Status|ASSIGNED|CLOSED Blocks|652183(FE-JAVASIG) |201449(FE-DEADREVIEW) Resolution||NOTABUG Last Closed||2010-12-06 11:14:31 --- Comment #4 from Stanislav Ochotnicky 2010-12-06 11:14:31 EST --- OK since there was no response I am closing this bug. Feel free to re-open if you find time to work on this later. -- 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 660095] Review Request: impressive - A program that displays presentation slides
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=660095 --- Comment #9 from Jason Tibbitts 2010-12-06 11:12:30 EST --- This package already has an f14 branch, so I can't process that request as-is. I'm guessing that you were asking for the package to be unretired, but if that's the case then it would have been helpful for you to say that. I've unretired the f14 and master branches; you should log into pkgdb and claim them. I will create the el6 branch for you 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 660095] Review Request: impressive - A program that displays presentation slides
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=660095 --- Comment #10 from Jason Tibbitts 2010-12-06 11:12:46 EST --- Git done (by process-git-requests). -- 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 655379] Review Request: tritonus - Java Sound API Implementation
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=655379 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #15 from Jason Tibbitts 2010-12-06 11:07:51 EST --- Not only is that change request not properly formatted, but this package already has an f14 branch (and of course it always has a 'master' branch which you never need to request). I'm going to take a wild guess and say that you're requesting for those branches to be unretired, which I've done. However, if that's what you wanted, why not just say that? You should now log into pkgdb and claim the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 572515] Review Request: jogl - Java bindings for OpenGL
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=572515 --- Comment #33 from Jon Ciesla 2010-12-06 11:06:45 EST --- Hey, to anyone who knows Java more than I do: gluegen is now FTBFS in rawhide. I fixed a packaging issue which then revealed a Java/antlr issue: http://koji.fedoraproject.org/koji/getfile?taskID=2647172&name=build.log -- 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 639369] Review Request: gfs2-utils - Fileysystem utilities for the GFS2 Filesystem
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=639369 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? | --- Comment #31 from Jason Tibbitts 2010-12-06 11:04:36 EST --- I'm not sure why the fedora-cvs flag is still set, but I don't see unprocessed request, so I'll clear the flag. If there is something left to be done, please explain. -- 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 436481] Review Request: perl-Net-Amazon-S3 - Use the Amazon Simple Storage Service (S3)
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=436481 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs- --- Comment #7 from Jason Tibbitts 2010-12-06 11:03:15 EST --- Where did the Fedora maintainer agree to this? I don't see anything in this ticket. -- 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 656010] Review Request: libsrtp - An implementation of the Secure Real-time Transport Protocol (SRTP)
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=656010 --- Comment #6 from Jason Tibbitts 2010-12-06 11:00:31 EST --- Git done (by process-git-requests). -- 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 659368] Review Request: luabind - A library that helps create bindings between C++ and Lua
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=659368 --- Comment #6 from Jason Tibbitts 2010-12-06 11:01:29 EST --- Git done (by process-git-requests). -- 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 657795] Review Request: libspatialite - Enables SQLite to support spatial data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=657795 --- Comment #12 from Jason Tibbitts 2010-12-06 11:01:14 EST --- Git done (by process-git-requests). -- 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 656363] Review Request: perl-Package-Stash-XS - Faster and more correct implementation of the Package::Stash 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=656363 --- Comment #3 from Jason Tibbitts 2010-12-06 11:00:51 EST --- Git done (by process-git-requests). -- 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 630283] Review Request: ghc-neither - Either with monad and applicative instances
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=630283 --- Comment #6 from Jason Tibbitts 2010-12-06 10:57:45 EST --- Git done (by process-git-requests). -- 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 630223] Review Request: ghc-failure - A simple type class for success/failure computations
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=630223 --- Comment #8 from Jason Tibbitts 2010-12-06 10:56:52 EST --- Git done (by process-git-requests). -- 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 597725] Review Request: mozilla-svg-edit - A complete vector graphics editor in the browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=597725 --- Comment #8 from Jason Tibbitts 2010-12-06 10:38:14 EST --- It doesn't look like the original submitter has participated in any of the discussion in this ticket. Raghunath, did you still wish to submit this package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
