[Bug 770986] Review Request: rubygem-kgio - Kinder, gentler I/O for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770986 Mamoru Tasaka mtas...@fedoraproject.org changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@fedoraproject.org Flag||fedora-review? --- Comment #1 from Mamoru Tasaka mtas...@fedoraproject.org 2011-12-31 03:18:42 EST --- Some notes: * Build failure - Build fails on F-17: http://koji.fedoraproject.org/koji/taskinfo?taskID=3611721 -doc subpackage is marked as %doc, however generated rpms differ between i686 and x86_64. It can be said that this is a bug in ruby(-rdoc) side, however for now please consider rdoc-generated files arch-dependent. * Arch dependent binary module - kgio_ext.so is arch-dependent and must be placed under %ruby_sitearch . Please check ruby packaging guideline. * Redundant items - The following items are now redundant on Fedora (may not be under EPEL) - BuildRoot: line - %clean section (completely) - %defattr at the beginning of %files * rpmlint issue - Please suppress (fix) the following rpmlint issues rubygem-kgio-doc.noarch: E: non-readable /usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/NEWS 0600L rubygem-kgio-doc.noarch: E: non-readable /usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/ChangeLog 0600L * Documentation files placement - README file should be in main package, not -doc, because it says README. - NEWS, ChangeLog can be in main package (and I recommend so), however not a blocker. * Enabling test suite - As this gem contains test/ directory, please add %check section and execute some tests there Note: you may see some errors for missing strace module on ./test/test_autopush.rb:3, however I think this can be patched out. By the way, I appreciate it if you would review my review request (bug 770796) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 756491] Review Request: python-libcloud - Python library that abstracts away differences among multiple cloud provider APIs.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=756491 Haïkel Guémar karlthe...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Haïkel Guémar karlthe...@gmail.com 2011-12-31 04:11:52 EST --- All valid points above remains ok and the summary has been fixed so there's no more blocker. I approve python-libcloud to join Fedora Package Collection. Have a pleasant happy new year eve ! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770174] Review Request: kde-partitionmanager - GUI for managing disk partitions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770174 --- Comment #6 from Mattia mattia.ve...@tiscali.it 2011-12-31 04:18:56 EST --- Thank you Hans Robin. I've uploaded the new .spec and .src http://www.messafuoco.com/fedora/kde-partitionmanager.spec http://www.messafuoco.com/fedora/kde-partitionmanager-1.0.3-5.20111223svn.fc16.src.rpm (same version as previous) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 757866] Review Request: kde-printer-applet - KDE printer applet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=757866 Mattia mattia.ve...@tiscali.it changed: What|Removed |Added CC||mattia.ve...@tiscali.it --- Comment #2 from Mattia mattia.ve...@tiscali.it 2011-12-31 05:50:44 EST --- I think the .desktop file should be renamed to respect the new package name for consistency (from 'printer-applet.desktop' to 'kde-printer-applet.desktop'). Also, the .spec file is missing the desktop-file-validate step. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770174] Review Request: kde-partitionmanager - GUI for managing disk partitions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770174 Mattia mattia.ve...@tiscali.it changed: What|Removed |Added Blocks||656997(kde-reviews) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770837] Review Request: sphinx 2.0.3 - Sphinx full-text search 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=770837 Jakub Jedelsky jakub.jedel...@gmail.com changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG Last Closed||2011-12-31 05:57:30 --- Comment #3 from Jakub Jedelsky jakub.jedel...@gmail.com 2011-12-31 05:57:30 EST --- I contacted author of current package and closing this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770837] Review Request: sphinx 2.0.3 - Sphinx full-text search 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=770837 Ivan Romanov dr...@land.ru changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771051] Review Request: kde-runtime - KDE Runtime
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771051 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks||765955(kde-4.8), ||656997(kde-reviews) Alias||kde-runtime -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771052] New: Review Request: kde-workspace - KDE Workspace
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kde-workspace - KDE Workspace https://bugzilla.redhat.com/show_bug.cgi?id=771052 Summary: Review Request: kde-workspace - KDE Workspace Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rdie...@math.unl.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://rdieter.fedorapeople.org/rpms/kde-workspace/kde-workspace.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/kde-workspace/kde-workspace-4.7.95-10.fc16.src.rpm Description: The KDE Workspace consists of what is the desktop of the KDE Desktop Environment. This package contains: * khotkeys (a hotkey daemon) * klipper (a cut paste history utility) * kmenuedit (the menu editor) * krandrtray (resize and rotate X screens) * krunner (a command run interface) * kwin (the window manager of KDE) * plasma (the KDE desktop, panels and widgets workspace application) * systemsettings (the configuration editor) Package rename, kdebase-workspace - kde-runtime , to match newer upstream module naming. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771052] Review Request: kde-workspace - KDE Workspace
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=771052 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks||765955(kde-4.8), ||656997(kde-reviews) Alias||kde-workspace -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771051] New: Review Request: kde-runtime - KDE Runtime
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kde-runtime - KDE Runtime https://bugzilla.redhat.com/show_bug.cgi?id=771051 Summary: Review Request: kde-runtime - KDE Runtime Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rdie...@math.unl.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://rdieter.fedorapeople.org/rpms/kde-runtime/kde-runtime.spec SRPM URL: http://rdieter.fedorapeople.org/rpms/kde-runtime/kde-runtime-4.7.95-10.fc16.src.rpm Description: Core runtime for KDE 4. Package rename kdebase-runtime - kde-runtime to match newer upstream naming. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770283] Review Request: uptimed - A daemon to record and keep track of system up times
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=770283
Ivan Romanov dr...@land.ru changed:
What|Removed |Added
CC||dr...@land.ru
--- Comment #3 from Ivan Romanov dr...@land.ru 2011-12-31 11:25:05 EST ---
Hi Tomaz.
%clean stage can be dropped.
https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
use %defattr(-,root,root,-) instead of %defattr(-,root,root)
BuildRoot can be dropped.
https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag
Your package doesn't provide any .h files. But it provides -devel subpackage. I
think you should use libuptimed/*.h
Your package doesn't provide /etc/init.d/uptimed file. But it works with
systemctl.
I think rm -rf %{buildroot} can be dropped. Really this command always are
doing when you use rpmbuild -bb some.spec
Can you provide -static subpackage? It's not necessarily but would be more
prefarable.
For directories use %dir in %files.
%dir %{_localstatedir}/spool/uptimed/
You must drop .la file
https://fedoraproject.org/wiki/Packaging/Guidelines#Packaging_Static_Libraries
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 757866] Review Request: kde-printer-applet - KDE printer applet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=757866 --- Comment #3 from Kevin Kofler ke...@tigcc.ticalc.org 2011-12-31 11:42:13 EST --- I don't think renaming the .desktop file makes sense. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770174] Review Request: kde-partitionmanager - GUI for managing disk partitions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770174 --- Comment #7 from Kevin Kofler ke...@tigcc.ticalc.org 2011-12-31 11:44:15 EST --- So does this support parted 3 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 608852] Review Request: epris - a dbus service to listen to music
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608852 --- Comment #30 from Fedora Update System upda...@fedoraproject.org 2011-12-31 12:16:41 EST --- epris-0.2-4.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/epris-0.2-4.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 608852] Review Request: epris - a dbus service to listen to music
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608852 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608852] Review Request: epris - a dbus service to listen to music
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608852 --- Comment #31 from Fedora Update System upda...@fedoraproject.org 2011-12-31 12:16:53 EST --- epris-0.2-4.fc16 has been submitted as an update for Fedora 16. https://admin.fedoraproject.org/updates/epris-0.2-4.fc16 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 757866] Review Request: kde-printer-applet - KDE printer applet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=757866 --- Comment #5 from Rex Dieter rdie...@math.unl.edu 2011-12-31 12:23:50 EST --- make that being generic... *upstream* -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 757866] Review Request: kde-printer-applet - KDE printer applet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=757866 --- Comment #4 from Rex Dieter rdie...@math.unl.edu 2011-12-31 12:23:26 EST --- I'll advocate renaming stuff away from being generic, but I'd also argue it's not something that should be considered a review blocker. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 757866] Review Request: kde-printer-applet - KDE printer applet
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=757866 --- Comment #6 from Kevin Kofler ke...@tigcc.ticalc.org 2011-12-31 13:18:12 EST --- Agreed, renaming things upstream would make a lot of sense, it's renaming the .desktop file downstream which I don't think is helpful. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 771070] New: Review Request: nwipe - Securely erase disks using a variety of recognized methods
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: nwipe - Securely erase disks using a variety of recognized methods https://bugzilla.redhat.com/show_bug.cgi?id=771070 Summary: Review Request: nwipe - Securely erase disks using a variety of recognized methods Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: low Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: re...@seznam.cz QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Type: --- Regression: --- Mount Type: --- Documentation: --- Spec URL: http://rebus.fedorapeople.org/SPECS/nwipe.spec SRPM URL: http://rebus.fedorapeople.org/SRPMS/nwipe-0.05-1.fc16.src.rpm PF 2012 , To the new year 2012 with a clean disk :). Please would you be so kind and do a formal review for this package? The nwipe is a command that will securely erase disks using a variety of recognized methods. It is a fork of the dwipe command used by Darik's Boot and Nuke (dban). Nwipe was created out of need to run the DBAN dwipe command outside of DBAN. This allows it to use any host distribution which gives better hardware support. It is essentially the same as dwipe, with a few changes: - pthreads is used instead of fork - The parted library is used to detect drives - The code is designed to be compiled with gcc Koji build: rawhide - http://koji.fedoraproject.org/koji/taskinfo?taskID=3612336 f16 - http://koji.fedoraproject.org/koji/taskinfo?taskID=3612342 Best regards Michal Ambroz -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 770986] Review Request: rubygem-kgio - Kinder, gentler I/O for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=770986 --- Comment #2 from Guillermo Gómez guillermo.go...@gmail.com 2011-12-31 14:52:21 EST --- (In reply to comment #1) * Build failure - Build fails on F-17: http://koji.fedoraproject.org/koji/taskinfo?taskID=3611721 * For now rdoc-generated files arch-dependent * It builds fine now in rawhide * Arch dependent binary module - kgio_ext.so is arch-dependent and must be placed under %ruby_sitearch . Please check ruby packaging guideline. Oops, fixed * Redundant items - The following items are now redundant on Fedora (may not be under EPEL) - BuildRoot: line - %clean section (completely) - %defattr at the beginning of %files I reviewed the items against epel guidelines because i plan to include epel version releases. - BuildRoot line will keep if not a blocker (MUST in EPEL) - %clean section (MUST in EPEL) - %defattr at the beginning of %files, (removed) * rpmlint issue - Please suppress (fix) the following rpmlint issues rubygem-kgio-doc.noarch: E: non-readable /usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/NEWS 0600L rubygem-kgio-doc.noarch: E: non-readable /usr/lib/ruby/gems/1.8/gems/kgio-2.7.0/ChangeLog 0600L Hmmm, this issues disappeared after removing noarch for doc subpackage. fixed * Documentation files placement - README file should be in main package, not -doc, because it says README. - NEWS, ChangeLog can be in main package (and I recommend so), however not a blocker. * README, NEWS, ChangeLog location fixed * Enabling test suite - As this gem contains test/ directory, please add %check section and execute some tests there Note: you may see some errors for missing strace module on ./test/test_autopush.rb:3, however I think this can be patched out. yes.. did it.. dont know if the best way...included patch0 just for test_autopush.rb. %check section added By the way, I appreciate it if you would review my review request (bug 770796) I'll take a look asap $ rpmlint -v SRPMS/rubygem-kgio-2.7.0-2.fc16.src.rpm rubygem-kgio.src: I: checking rubygem-kgio.src: I: checking-url http://bogomips.org/kgio (timeout 10 seconds) rubygem-kgio.src: I: checking-url http://rubygems.org/downloads/kgio-2.7.0.gem (timeout 10 seconds) 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint -v RPMS/x86_64/rubygem-kgio-2.7.0-2.fc16.x86_64.rpm rubygem-kgio.x86_64: I: checking rubygem-kgio.x86_64: I: checking-url http://bogomips.org/kgio (timeout 10 seconds) 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint -v RPMS/x86_64/rubygem-kgio-doc-2.7.0-2.fc16.x86_64.rpm rubygem-kgio-doc.x86_64: I: checking rubygem-kgio-doc.x86_64: I: checking-url http://bogomips.org/kgio (timeout 10 seconds) 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Not sure about my method for patching the test file (not really good at patching so far). New spec:http://gomix.fedorapeople.org/rubygem-kgio/rubygem-kgio.spec New srpm:http://gomix.fedorapeople.org/rubygem-kgio/rubygem-kgio-2.7.0-2.fc16.src.rpm thanks ! _- Guillermo -_ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608852] Review Request: epris - a dbus service to listen to music
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=608852 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #32 from Fedora Update System upda...@fedoraproject.org 2011-12-31 15:24:27 EST --- epris-0.2-4.fc16 has been pushed to the Fedora 16 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 676129] Review Request: qconf - Allows you to have a nice configure script for your qmake-based project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=676129 --- Comment #41 from Fedora Update System upda...@fedoraproject.org 2011-12-31 21:45:54 EST --- qconf-1.4-3.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/qconf-1.4-3.el6 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 709328] Review Request: psi-plus - Jabber client based on Qt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=709328 --- Comment #32 from Ivan Romanov dr...@land.ru 2011-12-31 23:01:13 EST --- $ rpmlint psi-plus-0.15-0.23.20111220git5157.fc15.src.rpm psi-plus.src: W: invalid-license LGPLv2.1+ psi-plus.src: W: invalid-license MIT/X11 psi-plus.src: W: invalid-license zlib/libpng psi-plus.src: W: strange-permission generate-tarball.sh 0755L psi-plus.src:207: W: macro-in-comment %doc psi-plus.src:207: W: macro-in-comment %files psi-plus.src: W: invalid-url Source1: language_ru.tar.bz2 1 packages and 0 specfiles checked; 0 errors, 7 warnings. $ rpmlint psi-plus-0.15-0.23.20111220git5157.fc15.i686.rpm psi-plus.i686: W: incoherent-version-in-changelog 0.15-0.23.20111220git5157 ['1:0.15-0.23.20111220git5157.fc15', '1:0.15-0.23.20111220git5157'] psi-plus.i686: W: invalid-license LGPLv2.1+ psi-plus.i686: W: invalid-license MIT/X11 psi-plus.i686: W: invalid-license zlib/libpng psi-plus.i686: E: incorrect-fsf-address /usr/share/doc/psi-plus-0.15/COPYING psi-plus.i686: W: no-manual-page-for-binary psi-plus 1 packages and 0 specfiles checked; 1 errors, 5 warnings. $ rpmlint psi-plus-plugins-0.15-0.23.20111220git5157.fc15.i686.rpm psi-plus-plugins.i686: W: spelling-error %description -l en_US Autoreply - Auto reply, Auto-reply, Reputably psi-plus-plugins.i686: W: spelling-error %description -l en_US Captcha - Catchall psi-plus-plugins.i686: W: spelling-error %description -l en_US captcha - catchall psi-plus-plugins.i686: W: spelling-error %description -l en_US vCards - v Cards, cards, canards psi-plus-plugins.i686: W: spelling-error %description -l en_US iconsets - icon sets, icon-sets, consents psi-plus-plugins.i686: W: spelling-error %description -l en_US submenu - sub menu, sub-menu, submerse psi-plus-plugins.i686: W: spelling-error %description -l en_US Gmail - Gail, Mail, Email psi-plus-plugins.i686: W: spelling-error %description -l en_US Juick - Buick, Juice, Juicy psi-plus-plugins.i686: W: spelling-error %description -l en_US microblogging - micro blogging, micro-blogging, microbiological psi-plus-plugins.i686: W: spelling-error %description -l en_US popup - pop up, pop-up, popular psi-plus-plugins.i686: W: spelling-error %description -l en_US Qip - Sip, Nip, Rip psi-plus-plugins.i686: W: spelling-error %description -l en_US Screenshot - Screen shot, Screen-shot, Screens hot psi-plus-plugins.i686: W: spelling-error %description -l en_US screenshot - screen shot, screen-shot, screens hot psi-plus-plugins.i686: W: spelling-error %description -l en_US Yandex - Spandex psi-plus-plugins.i686: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 15 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 771070] Review Request: nwipe - Securely erase disks using a variety of recognized methods
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=771070
Ivan Romanov dr...@land.ru changed:
What|Removed |Added
CC||dr...@land.ru
--- Comment #1 from Ivan Romanov dr...@land.ru 2011-12-31 23:08:06 EST ---
Hi Michal.
I can review your package.
Add %defattr(-,root,root,-) to %files stage
rm -rf %{buildroot} can be dropped.
Really rpmbuild always clean %{buildroot}. So you don't need to remove it
mannualy.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767583] Review Request: python-poppler-qt4
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=767583 --- Comment #13 from Rex Dieter rdie...@math.unl.edu 2012-01-01 01:32:35 EST --- Heck, I can add these myself (to rawhide), if there are no objection or otherwise good reason not to do so. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 767583] Review Request: python-poppler-qt4
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=767583
Rex Dieter rdie...@math.unl.edu changed:
What|Removed |Added
CC||rdie...@math.unl.edu
--- Comment #12 from Rex Dieter rdie...@math.unl.edu 2012-01-01 01:31:31 EST
---
Good stuff, might I suggest adding some runtime dependencies:
Requires: PyQt4
%{?_sip_api:Requires: sip-api(%{_sip_api_major}) = %{_sip_api}}
the latter item is a trick we use elsewhere in the distro to track sip binary
compatibility.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 709328] Review Request: psi-plus - Jabber client based on Qt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=709328 Ivan Romanov dr...@land.ru changed: What|Removed |Added Attachment #503103|0 |1 is obsolete|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
