[Bug 478767] Review Request: spring - Realtime strategy game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478767 --- Comment #18 from Aurelien Bompard gau...@free.fr 2009-01-25 03:01:29 EDT --- I have no idea what's going on here. Do we have a way to investigate PPC build errors ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476360] Review Request: rubygem-tlsmail - This library enables pop or smtp via ssl/tls
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476360 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #14 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-01-25 02:57:10 EDT --- Closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479546] Review Request: jailkit - Jailkit limits user accounts to specific files and/or commands
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479546 --- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-01-25 03:13:00 EDT --- Okay, as no one else seems to write security related advice on this bug, I will restart to review this bug. To Patrick: Would you check the spec file by Dale and merge it (if you want), and re-upload your srpm/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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459535 --- Comment #41 from Guillaume Kulakowski guillaume.kulakow...@fedoraproject.org 2009-01-25 03:34:03 EDT --- No, I can't. Can you remove guillaume AT llaumgui DOT com account and permit me to change my email by llaumgui AT gmail DOT com for sync my bz account with my fas ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 472683] Review Request: jpcap - Packet capturing library for Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=472683 Patrick Dignan dignan.patr...@gmail.com changed: What|Removed |Added Flag|needinfo?(dignan.patr...@gm | |ail.com)| --- Comment #15 from Patrick Dignan dignan.patr...@gmail.com 2009-01-25 03:35:38 EDT --- Sorry, experiencing some hardware difficulties with my main computer (which runs Fedora), so I've had some downtime. I should hopefully be back up tomorrow, as I hadn't had time to repair it over the week and reinstall. First on my TODO list is getting this built and pushed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478767] Review Request: spring - Realtime strategy game
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=478767
--- Comment #19 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-01-25
03:37:16 EDT ---
(In reply to comment #17)
ppc http://koji.fedoraproject.org/koji/taskinfo?taskID=1080230 (failed)
Please check this:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1080241
For this I modified the lastest spec file in this review request
as:
scons prefix=%{_prefix} \
.
.
configure \
|| { sleep 2 ; find . -name config.log | xargs cat ; exit 1 ; }
Then config.log shows:
scons: Configure: Checking for C++ header file boost/cstdint.hpp...
build/sconf_temp/conftest_1.cpp -
|
|#include boost/cstdint.hpp
|
|
g++ -o build/sconf_temp/conftest_1.o -c .
cc1plus: error: unrecognized command line option -mieee-fp
cc1plus: error: unrecognized command line option -march=i686
cc1plus: error: unrecognized command line option -mfpmath=sse
cc1plus: error: unrecognized command line option -msse
cc1plus: error: unrecognized command line option -mieee-fp
cc1plus: error: unrecognized command line option -march=i686
cc1plus: error: unrecognized command line option -mfpmath=sse
cc1plus: error: unrecognized command line option -msse
scons: Configure: no
I guess CMakeLists.txt rts/build/scons/detect.py rts/build/scons/rts.py
and so on are related to this issue.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479595] Review Request: raddump - RADIUS packets 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=479595 --- Comment #4 from Fabian Affolter fab...@bernewireless.net 2009-01-25 03:43:01 EDT --- Thanks for reviewing and the 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481416] Review Request: viking - GPS data editor and analyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481416 --- Comment #4 from Fabian Affolter fab...@bernewireless.net 2009-01-25 03:43:25 EDT --- Thanks for reviewing and the 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479546] Review Request: jailkit - Jailkit limits user accounts to specific files and/or commands
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=479546
--- Comment #11 from Patrick Dignan dignan.patr...@gmail.com 2009-01-25
04:36:08 EDT ---
Alright, I've looked at his spec file and patches, and mostly the changes look
good to me. One thing I'm going to try is using sed instead of this code:
if [ $1 = 0 ] ; then
if [ -f %{_sysconfdir}/shells ] ; then
TmpFile=`/bin/mktemp %{_tmppath}/.jk_chrootshrpmXX`
grep -v '^%{_sbindir}/jk_chrootsh$' %{_sysconfdir}/shells $TmpFile
cp -f $TmpFile %{_sysconfdir}/shells
rm -f $TmpFile
fi
fi
I'll test something like
sed 's:^%{_sbindir}/jk_chrootsh$::g' %{_sysconfdir}/shells
but I'm not sure if that would make sed a BuildReq, if so I'd just stick with
Dale's implementation.
Other than the %preun %post %postun and the accompanying patches, his spec is
pretty much the same as mine (or at least provides the same functions). So
I'll add those things to mine and test it tomorrow hopefully.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 437419] Review Request: edrip-fonts - Edrip decorative fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=437419 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Blocks||481472 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467408] Review Request: mingw32-jasper - MinGW Windows Jasper 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=467408 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:18:58 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/jasper/mingw32-jasper.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-jasper-1.900.1-6.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 1.900.1-6 - Use _smp_mflags. - Disable static libraries. - Include documentation. - Use the same patches as Fedora native package. - Just run autoconf instead of autoreconf so we don't upgrade libtool. - +BR mingw32-dlfcn. - Don't need the manual pages. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467402] Review Request: mingw32-glib2 - MinGW Windows GLib2 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=467402 Bug 467402 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #7 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:17:33 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/glib2/mingw32-glib2.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-glib2-2.19.5-2.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 2.19.5-2 - Rebase to native Fedora version 2.19.5. - Use _smp_mflags. - Use find_lang. - Don't build static libraries. - +BR dlfcn. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467414] Review Request: mingw32-gnutls - MinGW Windows GnuTLS TLS/SSL encryption 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=467414 Bug 467414 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:17:52 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/gnutls/mingw32-gnutls.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-gnutls-2.6.3-2.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 2.6.3-1 - Rebase to native Fedora version 2.6.3. - Enable C++ library. - Use find_lang macro. - Don't build static library. - Rebase MinGW patch to 2.6.3. - +BR mingw32-dlfcn. - +BR mingw32-readline. - Force rebuild of libtool. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467420] Review Request: mingw32-gtk2 - MinGW Windows Gtk2 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=467420 Bug 467420 depends on bug 467397, which changed state. Bug 467397 Summary: Review Request: mingw32-libpng - MinGW Windows Libpng library https://bugzilla.redhat.com/show_bug.cgi?id=467397 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED Bug 467420 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #5 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:18:14 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/gtk2/mingw32-gtk2.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-gtk2-2.15.0-1.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 2.15.0-1 - Rebase to Fedora native version 2.15.0. - Disable static libraries. - Use _smp_mflags. - Use find_lang macro. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467421] Review Request: mingw32-gtk-vnc - MinGW Windows port of VNC client GTK widget
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467421 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:19:23 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/gtk-vnc/mingw32-gtk-vnc.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-gtk-vnc-0.3.8-0.3.20081030hg.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 0.3.8-0.3.20081030hg - Use _smp_mflags. - Disable static library. - Rebuild libtool. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467409] Review Request: mingw32-atk - MinGW Windows Atk 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=467409 Bug 467409 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #5 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:17:13 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/atk/mingw32-atk.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-atk-1.25.2-1.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 1.25.2-1 - Rebase to latest Fedora native version 1.25.2. - Use find_lang macro. - Use smp_mflags. - Fix URL. - Fix Source URL. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467418] Review Request: mingw32-libvirt - MinGW Windows libvirt virtualization 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=467418 Bug 467418 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED Bug 467418 depends on bug 467399, which changed state. Bug 467399 Summary: Review Request: mingw32-readline - MinGW port of readline for editing typed command lines https://bugzilla.redhat.com/show_bug.cgi?id=467399 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #1 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:18:33 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/libvirt/mingw32-libvirt.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-libvirt-0.5.1-1.fc11.src.rpm * Fri Jan 23 2009 Richard Jones rjo...@redhat.com - 0.5.1-1 - Rebase to Fedora native version 0.5.1. - Use find_lang macro. - Use _smp_mflags. - Disable 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467405] Review Request: mingw32-libxml2 - MinGW Windows libxml2 XML processing 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=467405 --- Comment #11 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:22:22 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/libxml2/mingw32-libxml2.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-libxml2-2.7.2-5.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 2.7.2-5 - Use _smp_mflags. - Disable 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467410] Review Request: mingw32-libgcrypt - MinGW Windows gcrypt encryption 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=467410 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:21:46 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/libgcrypt/mingw32-libgcrypt.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-libgcrypt-1.4.3-3.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 1.4.3-3 - Use _smp_mflags. - Disable 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467405] Review Request: mingw32-libxml2 - MinGW Windows libxml2 XML processing 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=467405 --- Comment #12 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:26:23 EDT --- The package in comment 11 now builds for me in Rawhide without needing autoreconf/libtoolize, 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467401] Review Request: mingw32-libjpeg - MinGW Windows Libjpeg 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=467401 Bug 467401 depends on bug 467397, which changed state. Bug 467397 Summary: Review Request: mingw32-libpng - MinGW Windows Libpng library https://bugzilla.redhat.com/show_bug.cgi?id=467397 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #3 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:22:07 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/libjpeg/mingw32-libjpeg.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-libjpeg-6b-7.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 6b-7 - Disable static libraries. - Use _smp_mflags. - Update for new libtool 2. - +BR mingw32-dlfcn. - Added documentation (README includes the license). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467419] Review Request: mingw32-pango - MinGW Windows Pango 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=467419 Bug 467419 depends on bug 467398, which changed state. Bug 467398 Summary: Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages https://bugzilla.redhat.com/show_bug.cgi?id=467398 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED Bug 467419 depends on bug 467396, which changed state. Bug 467396 Summary: Review Request: mingw32-freetype - Free and portable font rendering engine https://bugzilla.redhat.com/show_bug.cgi?id=467396 What|Old Value |New Value Resolution||NEXTRELEASE Status|ASSIGNED|CLOSED --- Comment #3 from Richard W.M. Jones rjo...@redhat.com 2009-01-25 08:22:37 EDT --- Spec URL: http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/pango/mingw32-pango.spec SRPM URL: http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-pango-1.22.1-2.fc11.src.rpm * Fri Jan 23 2009 Richard W.M. Jones rjo...@redhat.com - 1.22.1-2 - Disable static libraries. - Use _smp_mflags. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480916] Review Request: w_scan - Tool for scanning DVB-C/T transponders
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480916 Felix Kaechele fe...@fetzig.org changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481472] Renaming review: edrip-fonts ⇒ ap anov-edrip-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481472 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added CC||fedora-fonts-bugs-l...@redh ||at.com, ||fedora-package-rev...@redha ||t.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 181993] Review Request: charis-fonts - Charis SIL fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=181993 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Blocks||481476 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481476] New: Renaming review: charis-fonts ⇒ sil-charis-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Renaming review: charis-fonts ⇒ sil-charis-fonts https://bugzilla.redhat.com/show_bug.cgi?id=481476 Summary: Renaming review: charis-fonts ⇒ sil-charis-fonts Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: distribution AssignedTo: nott...@redhat.com ReportedBy: nicolas.mail...@laposte.net QAContact: nott...@redhat.com CC: ka...@redhat.com, jkeat...@redhat.com, fedora-package-review@redhat.com, fedora-fonts-bugs-l...@redhat.com Depends on: 181993 Blocks: 477044 Estimated Hours: 0.0 Classification: Fedora Renaming of charis-fonts to sil-charis-fonts to comply with http://fedoraproject.org/wiki/Packaging:FontsPolicy#Naming New package: http://nim.fedorapeople.org/sil-charis-fonts.spec http://nim.fedorapeople.org/sil-charis-fonts-4.104-5.fc11.src.rpm Blocking request: https://fedorahosted.org/rel-eng/ticket/1248 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 452663] Review Request: andika-fonts - Andika SIL fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=452663 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Blocks||481478 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481478] New: Renaming review: andika-fonts ⇒ sil-andika-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Renaming review: andika-fonts ⇒ sil-andika-fonts https://bugzilla.redhat.com/show_bug.cgi?id=481478 Summary: Renaming review: andika-fonts ⇒ sil-andika-fonts Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: distribution AssignedTo: nott...@redhat.com ReportedBy: nicolas.mail...@laposte.net QAContact: nott...@redhat.com CC: ka...@redhat.com, jkeat...@redhat.com, fedora-package-review@redhat.com, fedora-fonts-bugs-l...@redhat.com Depends on: 452663 Blocks: 477044 Estimated Hours: 0.0 Classification: Fedora Renaming of andika-fonts to sil-andika-fonts to comply with http://fedoraproject.org/wiki/Packaging:FontsPolicy#Naming New package: http://nim.fedorapeople.org/sil-andika-fonts.spec http://nim.fedorapeople.org/sil-andika-fonts-1.0-4.fc11.src.rpm Blocking request: https://fedorahosted.org/rel-eng/ticket/1249 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481482] New: Review Request: perl-Perl6-Junction - Perl6 style Junction operators in Perl5
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-Perl6-Junction - Perl6 style Junction operators in Perl5 https://bugzilla.redhat.com/show_bug.cgi?id=481482 Summary: Review Request: perl-Perl6-Junction - Perl6 style Junction operators in Perl5 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: allis...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://allisson.fedorapeople.org/packages/perl-Perl6-Junction/perl-Perl6-Junction.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-Perl6-Junction/perl-Perl6-Junction-1.4-1.fc10.src.rpm Description: Perl6 style Junction operators in Perl5. This is a lightweight module which provides 'Junction' operators, the most commonly used being 'any' and 'all'. Inspired by the Perl6 design docs, http://dev.perl.org/perl6/doc/design/exe/E06.html. Provides a limited subset of the functionality of LQuantum::Superpositions. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480591] Review Request: ctan-kerkis-fonts - Kerkis type 1 fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480591 Sarantis Paskalis paska...@di.uoa.gr changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #9 from Sarantis Paskalis paska...@di.uoa.gr 2009-01-25 10:16:52 EDT --- Built in rawhide. Added in comps. EOL for tetex-font-kerkis done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480607] Review Request: bluecove - Implementation of JSR-82 Java Bluetooth 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=480607 --- Comment #1 from François Kooman fkoo...@tuxed.net 2009-01-25 11:03:04 EDT --- * Sun Jan 25 2009 François Kooman fkoo...@tuxed.net - 2.1.0-2 - remove javadoc sub package, there isn't any real documentation, only references to JSR-82 API docs at jcp.org - create symlinks without version numbers - remove gpl sub package, bluecove without gpl subpackage is useless - include the interesting docs from gpl subpackage in the package instead of the subpackage - include README.dist Issues remaining: - debug files in i386 package... - it probably won't work immediately on ppc/ppc64 as there is only a check for i386 and x86_64 in the library loader at runtime (no ppc hardware to test on, it does build correctly in koji) Spec URL: http://users.tuxed.net/fkooman/rpmbuild/SPECS/bluecove.spec SRPM URL: http://users.tuxed.net/fkooman/rpmbuild/SRPMS/bluecove-2.1.0-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481484] New: Review Request: sj-fonts - Two fonts by Steve Jordi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sj-fonts - Two fonts by Steve Jordi https://bugzilla.redhat.com/show_bug.cgi?id=481484 Summary: Review Request: sj-fonts - Two fonts by Steve Jordi Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: s...@lank.es QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sven.lank.es/Fedora/SPECS/sj-fonts.spec SRPM URL: http://sven.lank.es/Fedora/SRPM/sj-fonts-2.0.2-1.fc11.src.rpm Description: Two fonts by Steve Jordi released under the GPL -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 451298] Review Request: purple-msn-pecan - Alternative MSN protocol plugin for libpurple
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451298 Edouard Bourguignon ma...@linuxed.net 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481484] Review Request: sj-fonts - Two fonts by Steve Jordi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481484 Sven Lankes s...@lank.es changed: What|Removed |Added Blocks||477406 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478662] Review Request: dustin-dustismo-fonts - font with serif and sans-serif versions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478662 Sven Lankes s...@lank.es changed: What|Removed |Added Blocks|477406 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 451298] Review Request: purple-msn-pecan - Alternative MSN protocol plugin for libpurple
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451298 --- Comment #28 from Edouard Bourguignon ma...@linuxed.net 2009-01-25 11:30:06 EDT --- New Package CVS Request === Package Name: purple-msn-pecan Short Description: msn-pecan is a pidgin plugin adding extra fixtures to enhance the msn protocole. Owners: itamarjp madko Branches: F-10 F-9 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478662] Review Request: dustin-dustismo-fonts - font with serif and sans-serif versions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478662 Sven Lankes s...@lank.es changed: What|Removed |Added Blocks||477406 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478662] Review Request: dustin-dustismo-fonts - font with serif and sans-serif versions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478662 Sven Lankes s...@lank.es changed: What|Removed |Added Blocks|477406 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 477818] Review Request: gsql - Integrated database development tool for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477818 --- Comment #2 from Lucian Langa co...@gnome.eu.org 2009-01-25 11:40:00 EDT --- rpmlint is not silent: gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libxml2.so.2 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libdbus-glib-1.so.2 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /lib64/libdbus-1.so.3 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libatk-1.0.so.0 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /lib64/libgio-2.0.so.0 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libpangoft2-1.0.so.0 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libpangocairo-1.0.so.0 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libcairo.so.2 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libfreetype.so.6 gsql.x86_64: W: unused-direct-shlib-dependency /usr/lib64/libgsql.so.0.0.0 /usr/lib64/libfontconfig.so.1 Please see : https://fedoraproject.org/wiki/PackageMaintainers/Common_Rpmlint_Issues#unused-direct-shlib-dependency Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' (for directory ownership and usability). It is not really necessary to call ldconfig on -plugin and -engine-mysql because no libraries are added on ldconfig search path. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 451298] Review Request: purple-msn-pecan - Alternative MSN protocol plugin for libpurple
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451298 --- Comment #29 from Edouard Bourguignon ma...@linuxed.net 2009-01-25 11:44:43 EDT --- New Package CVS Request === Package Name: purple-msn-pecan Short Description: Alternative MSN protocol plugin for libpurple Owners: itamarjp madko Branches: F-10 F-9 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481173] Review Request: libnss-pgsql - NSS library that interface with PostgreSQL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481173 --- Comment #4 from Edouard Bourguignon ma...@linuxed.net 2009-01-25 11:47:41 EDT --- * SourceURL has been fixed * Your patch has been included. I'm on the way to upstream it Here are the links: SPEC: http://www.linuxed.net/~madko/fedora/libnss-pgsql.spec SRPM: http://www.linuxed.net/~madko/fedora/libnss-pgsql-1.5.0-0.2.beta.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478662] Review Request: dustin-dustismo-fonts - font with serif and sans-serif versions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478662 --- Comment #18 from Sven Lankes s...@lank.es 2009-01-25 11:48:20 EDT --- right - dumbest bz-comment of the week-award accepted - sorry for the noise -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481409] Review Request: 2bc-fonts - A stylized font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481409 --- Comment #6 from Clint Savage her...@gmail.com 2009-01-25 12:00:32 EDT --- 1. Thanks for that catch, I missed the -fonts. 2. Added Decorative to the summary 3. I honestly don't know how that happened. Fixed. 4. Again, I must have been sleepy at some point. I didn't see that change. Fixed. The new font is licensed correctly in the font itself, thus no documentation needed. Because of the license change (a new version of the font) and renaming the spec, two new files exist again. SRPM: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued-fonts-20090124-1.fc10.src.rpm SPEC: http://herlo.fedorapeople.org/rpms/chisholm-to-be-continued-fonts.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481272] Review Request: python-webunit - it test your websites with code that acts like a web browser
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=481272
Tadej Janež tadej.ja...@tadej.hicsalta.si changed:
What|Removed |Added
CC||tadej.ja...@tadej.hicsalta.
||si
--- Comment #2 from Tadej Janež tadej.ja...@tadej.hicsalta.si 2009-01-25
12:03:50 EDT ---
This is my informal review. I cannot sponsor you as I'm not (yet) an approved
packager. However, my review can help your future sponsor when he makes his
own official review.
Good:
+ rpmlint output is clean.
+ The package is named according to the Package Naming Guidelines.
+ The spec file name matches the base package %{name}, in the format
%{name}.spec.
+ The package is licensed with a Fedora approved license and meets the
Licensing Guidelines.
+ The License field in the package spec file matches the actual license.
+ The spec file is written in American English.
+ The spec file for the package is legible.
+ The sources used to build the package match the upstream source, as provided
in the spec URL (md5sum webunit-1.3.8.tar.gz:
97b9e6b5149dadce48b86adbf2db3b0a).
+ The package successfully compiles and builds into binary rpms on at least one
primary architecture.
+ All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of the Packaging Guidelines.
+ The package owns all directories that it creates.
+ The package doesn't contain any duplicate files in the %files listing.
+ Permissions on files are set properly.
+ The package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ The package consistently uses macros.
+ The package contains code, or permissible content.
+ Package doesn't own files or directories already owned by other packages.
+ At the beginning of %install, package runs rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
+ All filenames in rpm packages are valid UTF-8.
+ I tested that the package builds in mock.
+ The package compiles and builds into binary rpms on all supported
architectures.
+ I did a limited test that the package functions as described.
+ egg-info files which are generated by the module's build scripts are included
in the package.
Comments:
- You should use %{python_sitelib} instead of %{python_sitearch}, because your
package is architecture independent (noarch)
- You should remove %{python_sitearch} macro definition as you don't need it.
- You should remove CFLAGS=... in %build section because this a noarch package.
- You should avoid putting demo folder in the top-level %{python_sitelib}
folder and rather put in a subfolder like %{python_sitelib}/%{Project_name}.
- You could shorten the %files section by using:
%{python_sitearch}/%{Project_name}/
instead of:
%{python_sitearch}/%{Project_name}/HTMLParser.py*
%{python_sitearch}/%{Project_name}/IMGSucker.py*
%{python_sitearch}/%{Project_name}/SimpleDOM.py*
%{python_sitearch}/%{Project_name}/__init__.py*
%{python_sitearch}/%{Project_name}/config.py*
%{python_sitearch}/%{Project_name}/cookie.py*
%{python_sitearch}/%{Project_name}/utility.py*
%{python_sitearch}/%{Project_name}/webunittest.py*
And similarly for the demo directory.
- The version in the %changelog section should be on the same line as the date
and your name.
- Because the source package does not include license text(s) as a separate
file from upstream, you should query upstream to include it.
- You should improve the Summary and Description fields. Look at other packages
for examples. Your summary should state something like A python module for
unit testing websites acting like a web browser.
- Optionally, you could shorten and rename %{Project_name} macro to something
like %{project} or %{module}.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481409] Review Request: 2bc-fonts - A stylized font
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=481409
Nicolas Mailhot nicolas.mail...@laposte.net changed:
What|Removed |Added
CC||fedora-fonts-bugs-l...@redh
||at.com
AssignedTo|nicolas.mail...@laposte.net |her...@gmail.com
Flag|fedora-review? |fedora-review+
--- Comment #7 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-01-25
12:15:53 EDT ---
The SPEC still includes a
install -m 0643 -p %{SOURCE1}
I'll approve the package, but please fix this before import in cvs.
✧✧✧ APPROVED ✧✧✧
You can now continue from
https://fedoraproject.org/wiki/Font_package_lifecycle#3.a
Please do not forget the wiki and comps bits.
Thank your for packaging a new font for Fedora.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481484] Review Request: sj-fonts - Two fonts by Steve Jordi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481484 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|nicolas.mail...@laposte.net -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 472229] Review Request: PyQwt - Python bindings for Qwt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=472229 Tadej Janež tadej.ja...@tadej.hicsalta.si changed: What|Removed |Added Flag|needinfo?(tadej.ja...@tadej | |.hicsalta.si) | --- Comment #8 from Tadej Janež tadej.ja...@tadej.hicsalta.si 2009-01-25 12:35:22 EDT --- Thank you for taking time to review my fixed package! And I apologize again for responding so late. (In reply to comment #5) Well, for 5.1.0-2: * Header files dependency: - Well, for example the installed QwtModule.sip contains: 63 #include sipQwtQwtArrayDouble.h 64 #include sipQwtQwtArrayInt.h 65 #include sipQwtQwtArrayQwtDoubleInterval.h 66 #include sipQwtQwtArrayQwtDoublePoint.h - However these header files are not installed. Would you check if these are ignored? I traced what is happening to these files. configure/configure.py in PyQwt-5.1.0.tar.gz contains: - 745 # FIXME: sip-4.7 does not generate those include files anymore 746 for name in [os.path.join(tmp_dir, name) for name in [ 747 'sipQwtQwtArrayDouble.h', 748 'sipQwtQwtArrayInt.h', 749 'sipQwtQwtArrayQwtDoubleInterval.h', 750 'sipQwtQwtArrayQwtDoublePoint.h', 751 ]]: 752 if not os.path.exists(name): 753 open(name, 'w') - So these files get created after running 'python configure.py ...', however, they remain empty. Furthermore 'make install' doesn't install them anywhere, which would indicate that they are obsolete. I think this is an issue that should be handled upstream. I submitted a bug in sourceforge.net bug tracker: https://sourceforge.net/tracker2/?func=detailaid=2534030group_id=82987atid=567933 UPDATE: The author of the package responded that this was a rudimentary hack to maintain backwards compatibility with older SIP versions and he will remove it in future versions of the package. * About grace.py - Well, I checked this file again and actually this file is executable, so it is more correct that this script should simply have 0755 permission (while for qplt.py it seems that this script cannot be executed directly). OK, made grace.py executable again. Then: - NOTE: Before being sponsored: This package will be accepted with another few work. But before I accept this package, someone (I am a candidate) must sponsor you. I would be very glad if you could sponsor me. When you have submitted a new review request or have pre-reviewed other person's review request, please write the bug number on this bug report so that I can check your comments or review request. Ok, I did a pre-review of python-webunit package, which is tracked at bug 481272. Please check its quality and instruct me on my next steps. I also uploaded updated spec file and source package at http://tadej.hicsalta.si/packages/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 438608] Review Request: elisa-plugins-good - Good Plugins for the Elisa Media Center
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=438608 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #28 from Michel Alexandre Salim michel.syl...@gmail.com 2009-01-25 12:51:16 EDT --- ReviewTemplate PASSED • rpmlint: Clean, but no documentation. See licensing comments below • package name • spec file name • package guideline-compliant • license complies with guidelines: Yes • license field accurate • license file not deleted • spec in US English • spec legible • source matches upstream • builds under = 1 archs, others excluded noarch • build dependencies complete • own all directories • no dupes in %files • permission • %clean RPM_BUILD_ROOT • macros used consistently • Package contains code • clean buildroot before install • filenames UTF-8 SHOULD • if license text missing, ask upstream to include it License file seems to come with the base elisa distribution. Could upstream be asked to ship this with the plugins tarball as well? • package build in mock on all architectures • package functioned as described: Works, tested with audio playback on F10/x86_64. Note that video playback locks up the machine, but that is probably an Elisa / Elisa-plugins-bad problem • scriplets are sane • other subpackages should require versioned base • require package not files 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481409] Review Request: 2bc-fonts - A stylized font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481409 Clint Savage her...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Clint Savage her...@gmail.com 2009-01-25 12:48:23 EDT --- New Package CVS Request === Package Name: chisholm-to-be-continued-fonts Short Description: To Be Continued is a decorative/graffiti sans-serif font. Owners: herlo Branches: F-9 F-10 InitialCC: fonts-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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 426751] Review Request: ghc-X11 - A Haskell binding to the X11 graphics 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=426751 --- Comment #47 from Michel Alexandre Salim michel.syl...@gmail.com 2009-01-25 12:54:25 EDT --- Jens, if Yaakov is working on it, do you (or Till) want to take over the review? Thanks. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481494] Renaming review: unikurd-web-font ⇒ kurdit-unikurd-web-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481494 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added AssignedTo|nott...@redhat.com |nicolas.mail...@laposte.net Flag||fedora-review? --- Comment #1 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-01-25 12:57:53 EDT --- ≥ and ≤ are tricky to get right in rpm do yourself a favour, start at Release: 2 and Obsoletes: unikurd-web-font 20020502-2 That will work easily. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475097] Review Request: gimp-fourier-plugin - A fourier transformation plugin for GIMP.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475097 --- Comment #8 from Fabian Deutsch fabian.deut...@gmx.de 2009-01-25 13:12:35 EDT --- I solved those two issues. The included patch resolved - I actually don't know why - the non-standard-dir-perm error. http://www.informatik.uni-bremen.de/~fabiand/fedora/fourier/gimp-fourier-plugin-0.3.2-2.fc9.src.rpm http://www.informatik.uni-bremen.de/~fabiand/fedora/fourier/gimp-fourier-plugin.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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481501] New: Review Request: dustin-domestic-manners - Handwriting font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: dustin-domestic-manners - Handwriting font https://bugzilla.redhat.com/show_bug.cgi?id=481501 Summary: Review Request: dustin-domestic-manners - Handwriting font Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: s...@lank.es QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sven.lank.es/Fedora/SPECS/dustin-domestic-manners-fonts.spec SRPM URL: http://sven.lank.es/Fedora/SRPM/dustin-domestic-manners-fonts-20030527-1.fc11.src.rpm Description: This font mimics the authors handwriting. The name come from the book Domestic Manners of the Americans, by Fanny Trollope. Font contains, letters, numbers, punctuation, accented characters and some special characters (most European Latin characters). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481484] Review Request: sj-fonts - Two fonts by Steve Jordi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481484 --- Comment #3 from Sven Lankes s...@lank.es 2009-01-25 13:29:03 EDT --- (In reply to comment #2) Sorry - typo - SJ, not DJ-Fonts: New Package CVS Request === Package Name: sj-fonts Short Description: Two fonts by Steve Jordi released under the GPL Owners: slankes Branches: F-9 F-10 InitialCC: fonts-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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481484] Review Request: sj-fonts - Two fonts by Steve Jordi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481484 Sven Lankes s...@lank.es changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Sven Lankes s...@lank.es 2009-01-25 13:27:54 EDT --- New Package CVS Request === Package Name: dj-fonts Short Description: Two fonts by Steve Jordi released under the GPL Owners: slankes Branches: F-9 F-10 InitialCC: fonts-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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481501] Review Request: dustin-domestic-manners - Handwriting font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481501 Sven Lankes s...@lank.es changed: What|Removed |Added Blocks||477406 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481494] Renaming review: unikurd-web-font ⇒ kurdit-unikurd-web-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481494 --- Comment #2 from Michal Nowak mno...@redhat.com 2009-01-25 13:33:08 EDT --- Updated: http://mnowak.fedorapeople.org/kurdit-unikurd-web-fonts/kurdit-unikurd-web-fonts.spec http://mnowak.fedorapeople.org/kurdit-unikurd-web-fonts/kurdit-unikurd-web-fonts-20020502-2.fc10.src.rpm * Sun Jan 25 2009 Michal Nowak mno...@redhat.com - 20020502-2 - repackage to save myself a grief -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481494] Renaming review: unikurd-web-font ⇒ kurdit-unikurd-web-fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481494 Nicolas Mailhot nicolas.mail...@laposte.net changed: What|Removed |Added AssignedTo|nicolas.mail...@laposte.net |mno...@redhat.com Flag|fedora-review? |fedora-review+ --- Comment #3 from Nicolas Mailhot nicolas.mail...@laposte.net 2009-01-25 13:53:34 EDT --- upgrade path works ❇❇❇ 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481502] New: Review Request: rubygem-ditz - A command-line issue tracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-ditz - A command-line issue tracker https://bugzilla.redhat.com/show_bug.cgi?id=481502 Summary: Review Request: rubygem-ditz - A command-line issue tracker Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kmcmar...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://kyle.fedorapeople.org/rubygem-ditz/rubygem-ditz.spec SRPM URL: http://kyle.fedorapeople.org/rubygem-ditz/rubygem-ditz-0.5-0.fc10.src.rpm Description: A command-line issue tracker written in ruby. License is GPLv3... Great tool for managing personal todo lists, can generated an html page for publishing. Upstream URL: ditz.rubyforge.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480886] Review Request: tse3 - MIDI Sequencer 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=480886
--- Comment #1 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-01-25
14:46:43 EDT ---
Created an attachment (id=329952)
-- (https://bugzilla.redhat.com/attachment.cgi?id=329952)
some trial patch
For 0.3.1-1:
* About Patch2:
- Well, google shows that awe_voice.h was removed from
kernel headers at 2.6.21 or so (i.e. around Fedora 7)
I don't think it is preferrable to re-introduce macros
which was defined in the header file which is no longer
supported. I guess kernel supports AWE32 in other way
or kernel does not support AWE32 completely any more.
Are macros related to awe_voice.h really needed even
now?
(removing all awe32 related parts like attached does not help ?
I just tried to remove all awe_voice.h related parts
and did not any functionality check as I don't know
how to use this software...)
* Timestamps
- Please consider to use
---
make install DESTDIR=%{buildroot} INSTALL=install -p
---
to keep timestamps on installed files.
* Directory structure about document files
- For HTML files in devel package, I suggest that a subdirectory
should be created under %_defaultdocdir/%name-devel-%version
(named as HTML, for example) and all HTML files should be
moved into the directory.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476973] Review Request: eclipse-rpmstubby - Rpm specfile generator for Eclipse features
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=476973
--- Comment #2 from Alexander Kurtakov akurt...@redhat.com 2009-01-25
14:42:39 EDT ---
Spec URL:
http://akurtakov.fedorapeople.org/eclipse-rpmstubby.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-rpmstubby-0.1.0-1.fc10.src.rpm
(In reply to comment #1)
I made the review. There are a few minor things to go over:
* Two Changelogs need packaged:
./org.eclipse.linuxtools.rpmstubby-feature/ChangeLog
./org.eclipse.linuxtools.rpmstubby/ChangeLog
Fixed
* I think URL needs to be:
http://www.eclipse.org/linuxtools/projectPages/rpmstubby/
Fixed
? Could you provide a download location at the project homepage so we don't
need to use this fetch script?
Not yet because the project needs to be reviewed from eclipse.org before making
official releases.
* Macros should be used consistently. You use %{buildroot} and $RPM_BUILD_ROOT
mixed. This needs corrected. You should also use %{__install} for consistency.
Fixed.
* Summary looks awkward to me, maybe it should be shortened to:
Rpm specfile generator for Eclipse
Current summary describes better what the plugin is doing.
These are all easy to do. I'm approving the package now.
This package (eclipse-rpmstubby) is APPROVED by oget
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476973] Review Request: eclipse-rpmstubby - Rpm specfile generator for Eclipse features
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476973 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2009-01-25 14:48:23 EDT --- New Package CVS Request === Package Name: eclipse-rpmstubby Short Description: Rpm specfile generator for Eclipse features Owners: akurtakov Branches: F-10 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480886] Review Request: tse3 - MIDI Sequencer 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=480886 --- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-01-25 15:12:41 EDT --- Thank you. (In reply to comment #1) * About Patch2: - Well, google shows that awe_voice.h was removed from kernel headers at 2.6.21 or so (i.e. around Fedora 7) I don't think it is preferrable to re-introduce macros which was defined in the header file which is no longer supported. I guess kernel supports AWE32 in other way or kernel does not support AWE32 completely any more. Are macros related to awe_voice.h really needed even now? (removing all awe32 related parts like attached does not help ? I just tried to remove all awe_voice.h related parts and did not any functionality check as I don't know how to use this software...) Thanks for the patch. It seems like you are right. But now I am thinking of dropping the OSS support from tse3 altogether. I see other applications/libraries in Fedora built with --without-oss flag. - It will save us some dependencies, - I am not sure how functional the library will be after this patch, and - OSS is deprecated, so we won't loose much. Does that sound OK? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479835] Review Request: log4c - an application message logging 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=479835 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ke...@tummy.com Flag||fedora-review? --- Comment #2 from Kevin Fenzi ke...@tummy.com 2009-01-25 15:25:52 EDT --- Hey Alex, I would be happy to look at this and bug 479951 and sponsor you. Look for a full review later today. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] New: Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Clanship 1.0.5f rpm https://bugzilla.redhat.com/show_bug.cgi?id=481507 Summary: Clanship 1.0.5f rpm Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: low Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: j...@mikkonen.biz QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Here's my first and initial submission to fedora games rpm's. Packaging request taking from Games sig wishlist: Clanship Spec file: http://www.pcuf.fi/~rasjani/fedora/clanship.spec and SRPMS: http://www.pcuf.fi/~rasjani/fedora/ClanShip-1.0.5f-1.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480886] Review Request: tse3 - MIDI Sequencer 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=480886 --- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-01-25 15:26:53 EDT --- I think removing OSS support should be okay on Fedora. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Depends on||481483 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481507 Itamar Reis Peixoto ita...@ispbrasil.com.br changed: What|Removed |Added Blocks||177841 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481507 Itamar Reis Peixoto ita...@ispbrasil.com.br changed: What|Removed |Added CC||ita...@ispbrasil.com.br --- Comment #1 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2009-01-25 15:28:41 EDT --- is this your first 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479951] Review Request: iniparser - a library for parsing ini-style 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=479951 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ke...@tummy.com Flag|fedora-review+ |fedora-review? --- Comment #7 from Kevin Fenzi ke...@tummy.com 2009-01-25 15:26:55 EDT --- Hey Alex. I would be happy to look at this and bug 479835. Look for more info later today. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] New: Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences https://bugzilla.redhat.com/show_bug.cgi?id=481508 Summary: Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/pipviewer.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/pipviewer-0.3.9-1.fc10.src.rpm Project URL: http://ygingras.net/bioinfo Description: Pipviewer is a visualizer for multiple alignements of genomic sequences. It highlights conserved regions and allows basic anotations. Its main goal is to find conserved probes for the construction of gene order data sets. Selected regions marked as 'probes' can be expoxted to fasta format. It can also retreive gene annotations form the NBCI and display this information along the alignement. Pipviewer is not an aligner. You must compute the alignment with another tool like Clustal or Multi PIP Maker. Koji scratch build: depends on vhybridize rpmlint output: [...@laptop24 noarch]$ rpmlint pipviewer-0.3.9-1.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [...@laptop24 SRPMS]$ rpmlint pipviewer-0.3.9-1.fc10.src.rpm 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481509] New: Review Request: vhybridize - Virtual Hybridization tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: vhybridize - Virtual Hybridization tools https://bugzilla.redhat.com/show_bug.cgi?id=481509 Summary: Review Request: vhybridize - Virtual Hybridization tools Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/vhybridize.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/vhybridize-0.5.9-1.fc10.src.rpm Project URL: http://ygingras.net/bioinfo Description: Virtual Hybridization uses sets of short probes to generate datasets for comparative genomics: given a DNA sequence and a set of probes, the typical output will give a sequence of oriented probe hits along the DNA sequence. Other tools are supplied to allow simple manipulations such as format conversion and extraction of permutations. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1080910 rpmlint output: [...@laptop24 SRPMS]$ rpmlint vhybridize-0.5.9-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [...@laptop24 noarch]$ rpmlint vhybridize-0.5.9-1.fc10.noarch.rpm 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481507 --- Comment #2 from Jani Mikkonen j...@mikkonen.biz 2009-01-25 15:35:28 EDT --- Yes, its my first that i have submitted to fedora. I have had a talk with Hans that he could review and sponsor me in case he finds my packages good enough - but i forgot to add him to cc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481509] Review Request: vhybridize - Virtual Hybridization tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481509 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Blocks||481508 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Depends on||481509 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479835] Review Request: log4c - an application message logging 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=479835
--- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-01-25 15:50:57 EDT ---
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
See below - Spec has consistant macro usage.
OK - Meets Packaging Guidelines.
OK - License (LGPLv2+)
OK - License field in spec matches
OK - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
ca5412b7515d8901714ab7892323adb6 log4c-1.2.1.tar.gz
ca5412b7515d8901714ab7892323adb6 log4c-1.2.1.tar.gz.orig
See below - BuildRequires correct
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
OK - Package is code or permissible content.
OK - Packages %doc files don't affect runtime.
OK - Package has rm -rf RPM_BUILD_ROOT at top of %install
OK - Headers/static libs in -devel subpackage.
OK - Spec has needed ldconfig in post and postun
OK - .so files in -devel subpackage.
OK - -devel package Requires: %{name} = %{version}-%{release}
See below - .la files are removed.
OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
OK - Package doesn't own any directories other packages own.
See below - Package owns all the directories it creates.
OK - Package obey's FHS standard (except for 2 exceptions)
See below - No rpmlint output.
OK - final provides and requires are sane.
SHOULD Items:
OK - Should build in mock.
OK - Should build on all supported archs
OK - Should have dist tag
OK - Should package latest version
OK - Should not use file requires outside of /etc, /bin, /sbin, /usr/bin, or
/usr/sbin
Issues:
1. No need for a %check section if there is nothing in it is there?
2. Please change the Source url to match the sourceurl guidelines for
sourceforge
projects? http://fedoraproject.org/wiki/Packaging/SourceURL
3. Please use a consistent type of macro... either $RPM_BUILD_ROOT or
%{buildroot},
but not both. As a side note, I don't personally think it's worth it to
macroize
things like mv or mkdir, but thats up to you.
4. Please remove .la files, don't %exclude them. excluding has weird effects
sometimes.
5. Looks like you might be missing a BuildRequires... from the build.log:
checking for EXPAT - version = 1.95.1...
no
*** Could not run EXPAT test program, checking why...
*** The test program failed to compile or link. See the file config.log for the
*** exact error that occured. This usually means EXPAT was incorrectly
installed
*** or that you have moved EXPAT since it was installed. In the latter case,
you
*** may want to edit the expat-config script: ''
*** Log4C will still run without EXPAT--it uses some bundled
*** lex/yacc code to parse the configuration file
6. rpmlint says:
log4c.x86_64: W: shared-lib-calls-exit /usr/lib64/liblog4c.so.3.1.0
e...@glibc_2.2.5
log4c-devel.x86_64: W: no-documentation
You might bring up the first to upstream, the second can be ignored.
7. Since your devel subpackage puts a file under /usr/share/aclocal, you
probibly need to add a Requires: automake to the devel subpackage.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480886] Review Request: tse3 - MIDI Sequencer 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=480886 --- Comment #4 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-01-25 15:52:47 EDT --- Update: Spec URL: http://oget.fedorapeople.org/review/tse3.spec SRPM URL: http://oget.fedorapeople.org/review/tse3-0.3.1-2.fc10.src.rpm Changelog: - Move the HTML documentation into an HTML subdirectory - Use INSTALL=install -p - Drop the oss support, and the related patch -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479951] Review Request: iniparser - a library for parsing ini-style 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=479951 --- Comment #8 from Kevin Fenzi ke...@tummy.com 2009-01-25 16:01:58 EDT --- On quick comment here: Is this a pre-release/post-release or snapshot package? If so, see: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Non-Numeric_Version_in_Release Ie, is it a beta of an upcoming 3.0 release? If so the version should probibly be: 3.0-0.1.b -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480050] Review Request: libchamplain - Map view for Clutter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480050 Denis Leroy de...@poolshark.org changed: What|Removed |Added CC||de...@poolshark.org --- Comment #9 from Denis Leroy de...@poolshark.org 2009-01-25 16:15:46 EDT --- Compiles fine on my F-10 system. Jochen, looks like something's borked on your setup. It's also not a compile failure, it's an error from gtkdoc-scan. Anyways, this shouldn't block the review. Jochen, if you want I can take over the review until you figure out what's going on on your end... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480056] Review Request: libchamplain-gtk - Gtk+ widget wrapper for libchamplain
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480056 Denis Leroy de...@poolshark.org changed: What|Removed |Added Status|NEW |ASSIGNED CC||de...@poolshark.org AssignedTo|nob...@fedoraproject.org|de...@poolshark.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480050] Review Request: libchamplain - Map view for Clutter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480050 --- Comment #10 from Jochen Schmitt joc...@herr-schmitt.de 2009-01-25 16:22:22 EDT --- OK, yuo make take over this review, because I don't want to block it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481513] New: Review Request: sugar-pippy - Pippy for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sugar-pippy - Pippy for Sugar https://bugzilla.redhat.com/show_bug.cgi?id=481513 Summary: Review Request: sugar-pippy - Pippy for Sugar Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-pippy.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-pippy-25-1.fc10.src.rpm Project URL: http://wiki.laptop.org/go/Pippy Description: Teaches Python programming by providing access to Python code samples and a fully interactive Python interpreter. The user can type and execute simple Python expressions. For example, it would be possible for a user to write Python statements to calculate expressions, play sounds, or make simple text animation. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1081396 rpmlint output: [...@laptop24 SRPMS]$ rpmlint sugar-pippy-25-1.fc10.src.rpm sugar-pippy.src: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [...@laptop24 noarch]$ rpmlint sugar-pippy-25-1.fc10.noarch.rpm sugar-pippy.noarch: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481513] Review Request: sugar-pippy - Pippy for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481513 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added External Bug ID||OLPC 9213 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481507 --- Comment #3 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2009-01-25 16:44:10 EDT --- please do this. please also reduce the size of your .spec file by merging mkdir -p and cp, you can use something like install -Dp -m 0755 file -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 463035] Review Request: pyroman - Very fast firewall configuration tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=463035 Adam Huffman bl...@verdurin.com changed: What|Removed |Added Flag|needinfo?(bl...@verdurin.co | |m) | --- Comment #8 from Adam Huffman bl...@verdurin.com 2009-01-25 17:33:21 EDT --- I've got an initscript for the package now but it needs more testing. Will upload a new version 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481516] New: Review Request: sugar-stopwatch - Simple stopwatch for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sugar-stopwatch - Simple stopwatch for Sugar https://bugzilla.redhat.com/show_bug.cgi?id=481516 Summary: Review Request: sugar-stopwatch - Simple stopwatch for Sugar Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-stopwatch.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-stopwatch-20090126-1.fc10.src.rpm Project URL: http://wiki.laptop.org/go/Stopwatch Description: This activity provides multiple stopwatches to time events with. Provide a useful timer for races, velocity measurements, etc. Be accessible to innumerate users. Help develop numeracy. Sharing of stopwatch sets, which anyone can manipulate. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1081558 rpmlint output: [...@laptop24 noarch]$ rpmlint sugar-stopwatch-20090126-1.fc10.noarch.rpm sugar-stopwatch.noarch: W: no-documentation sugar-stopwatch.noarch: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 2 warnings. [...@laptop24 SRPMS]$ rpmlint sugar-stopwatch-20090126-1.fc10.src.rpm sugar-stopwatch.src: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481513] Review Request: sugar-pippy - Pippy for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481513 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Blocks||462625 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481516] Review Request: sugar-stopwatch - Simple stopwatch for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481516 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Blocks||462625 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481507] Clanship 1.0.5f rpm
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481507 Jani Mikkonen j...@mikkonen.biz changed: What|Removed |Added CC||hdego...@redhat.com --- Comment #4 from Jani Mikkonen j...@mikkonen.biz 2009-01-25 18:12:10 EDT --- Hans added to cc New spec: http://www.pcuf.fi/~rasjani/fedora/clanship.spec New srpms:http://www.pcuf.fi/~rasjani/fedora/ClanShip-1.0.5f-2.fc10.src.rpm ps. since the clanship seems dead currently, the source can be found in my games repo. git clone git://github.com/rasjani/games.git -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481521] New: Review Request: sugar-clock - Clock activity for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sugar-clock - Clock activity for Sugar https://bugzilla.redhat.com/show_bug.cgi?id=481521 Summary: Review Request: sugar-clock - Clock activity for Sugar Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-clock.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-clock-20090126-1.fc10.src.rpm Project URL: http://wiki.laptop.org/go/Clock Description: This activity displays time in analog, digital, and natural forms. The natural form will be an image of a sun or moon arcing across the sky, rising and setting as the day progresses. This is more than a simple clock; the user will be able to grab any element and readjust it, which will update each of the other elements. In this manner, hopefully the children can explore and understand different methods of telling time. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1081637 rpmlint output: [...@laptop24 noarch]$ rpmlint sugar-clock-20090126-1.fc10.noarch.rpm sugar-clock.noarch: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [...@laptop24 SRPMS]$ rpmlint sugar-clock-20090126-1.fc10.src.rpm sugar-clock.src: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481521] Review Request: sugar-clock - Clock activity for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481521 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added External Bug ID||OLPC 9215 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481521] Review Request: sugar-clock - Clock activity for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481521 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added External Bug ID||OLPC 9216 Blocks||462625 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481523] New: Review Request: rubygem-trollop - A command-line option parsing library for ruby.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-trollop - A command-line option parsing library for ruby. https://bugzilla.redhat.com/show_bug.cgi?id=481523 Summary: Review Request: rubygem-trollop - A command-line option parsing library for ruby. Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kmcmar...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://kyle.fedorapeople.org/rubygem-ditz/rubygem-trollop.spec SRPM URL: http://kyle.fedorapeople.org/rubygem-ditz/rubygem-trollop-1.10.2-0.fc10.src.rpm Description: A command-line option parsing library for ruby. Trollop is designed to provide the maximal amount of GNU-style argument processing in the minimum number of lines of code (for you, the programmer). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481524] Review Request: sugar-help - Help and Dokumentation for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481524 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added External Bug ID||OLPC -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481524] Review Request: sugar-help - Help and Dokumentation for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481524 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added External Bug ID||OLPC 8889 Blocks||462625 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481524] New: Review Request: sugar-help - Help and Dokumentation for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sugar-help - Help and Dokumentation for Sugar https://bugzilla.redhat.com/show_bug.cgi?id=481524 Summary: Review Request: sugar-help - Help and Dokumentation for Sugar Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-help.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-help-7-1.fc10.src.rpm Project URL: http://wiki.laptop.org/go/Help_(activity) Description: The Help activity provides a quick interface to help documentation on the XO. It currently launches a browser and displays html documents describing how to use the XO and the Sugar interface. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1081864 rpmlint output: [...@laptop24 noarch]$ rpmlint sugar-help-7-1.fc10.noarch.rpm sugar-help.noarch: W: hidden-file-or-dir /usr/share/sugar/activities/Help.activity/help/OLPC_simple/TuteBackingUp/.resized_400x300_sdcard.png ... ... sugar-help.noarch: W: hidden-file-or-dir /usr/share/sugar/activities/Help.activity/help/XO/.resized_600x398_xoboxcontents.jpg sugar-help.noarch: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 33 warnings. [...@laptop24 SRPMS]$ rpmlint sugar-help-7-1.fc10.src.rpm sugar-help.src: W: non-standard-group Sugar/Activities 1 packages and 0 specfiles checked; 0 errors, 1 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481527] New: Review Request: bucardo - asynchronous PostgreSQL replication system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: bucardo - asynchronous PostgreSQL replication system https://bugzilla.redhat.com/show_bug.cgi?id=481527 Summary: Review Request: bucardo - asynchronous PostgreSQL replication system Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ita...@ispbrasil.com.br QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Description: Bucardo is an asynchronous PostgreSQL replication system, allowing for both multi-master and multi-slave operations. It was developed at Backcountry.com primarily by Greg Sabino Mullane of End Point Corporation. It's licensed under BSD License. SRPMS and SPEC will coming 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481509] Review Request: vhybridize - Virtual Hybridization tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481509 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-01-25 20:29:40 EDT --- Review: + package builds in mock. Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1080910 + rpmlint is silent for SRPM and for RPM. + source files match upstream. f962b32a9bae82bd67e28f29ad783327 vhybridize-0.5.9.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + %doc files present. + BuildRequires are proper. + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code. + no static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + no translations are available. + Does owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + no scriptlets are used. + Not a GUI app. Suggestions: 1) Fix license in spec which looks GPLv3+ 2) Fix summary. Summary should be Summary : Virtual Hybridization tools. 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. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481527] Review Request: bucardo - asynchronous PostgreSQL replication 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=481527 Itamar Reis Peixoto ita...@ispbrasil.com.br changed: What|Removed |Added Alias||bucardo -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481523] Review Request: rubygem-trollop - A command-line option parsing library 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=481523 --- Comment #1 from Kyle McMartin kmcmar...@redhat.com 2009-01-25 20:41:59 EDT --- (Dependency of rubygem-ditz.) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
