[Bug 475322] Review Request: genus2reduction - Computes Reductions of Genus 2 Proper Smooth Curves
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475322 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #12 from Conrad Meyer kon...@tylerc.org 2009-03-20 02:20:51 EDT --- Imported and built in rawhide. Closing. http://koji.fedoraproject.org/koji/taskinfo?taskID=1250744 -- Configure 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 491268] New: Review Request: perl-autodie - Replace functions with equivalents which succeed or die
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-autodie - Replace functions with equivalents which succeed or die Alias: perl-autodie https://bugzilla.redhat.com/show_bug.cgi?id=491268 Summary: Review Request: perl-autodie - Replace functions with equivalents which succeed or die Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/autodie OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cw...@alumni.drew.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~cweyl/review/perl-autodie.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-autodie-1.999-1.fc10.src.rpm Description: bIlujDI' yIchegh()Qo'; yIHegh()! It is better to die() than to return() in failure. -- Klingon programming proverb. The autodie pragma provides a convenient way to replace functions that normally return false on failure with equivalents that throw an exception on failure. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1250698 Additional Comment: Due to current @INC ordering, at the moment this will only work properly in rawhide. perl-autodie is a prereq of perl-Git-CPAN-Patch. *rt-0.05 -- Configure 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 490397] Review Request: gmime22 - Library for creating and parsing MIME messages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490397 Bug 490397 depends on bug 476243, which changed state. Bug 476243 Summary: Create new package gmime22 https://bugzilla.redhat.com/show_bug.cgi?id=476243 What|Old Value |New Value Status|NEW |CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476346] Review Request: python-polybori - Framework for Boolean Rings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476346 --- Comment #5 from Conrad Meyer kon...@tylerc.org 2009-03-20 03:05:54 EDT --- So in mock on F-10 it errors because it can't link against cudd (probably because we haven't built cudd for F-10 (or at least haven't pushed the update)). In F-11 on koji it errors because ... boost is broken? -- Configure 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 490725] Review Request: geanyvc - Version Controler plugin for geany
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490725 --- Comment #2 from Parag AN(पराग) panem...@gmail.com 2009-03-20 03:44:45 EDT --- 1)koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1250821 missing intltool 2) License looks to be GPLv2+ -- Configure 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 490725] Review Request: geanyvc - Version Controler plugin for geany
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490725 --- Comment #3 from Parag AN(पराग) panem...@gmail.com 2009-03-20 03:47:15 EDT --- just found this review already submitted https://bugzilla.redhat.com/show_bug.cgi?id=458773 -- Configure 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 458773] Review Request: geany-vc - Provide uniform access to different version-control systems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=458773 Pierre-YvesChibon pin...@pingoured.fr changed: What|Removed |Added CC||pin...@pingoured.fr --- Comment #7 from Pierre-YvesChibon pin...@pingoured.fr 2009-03-20 03:56:59 EDT --- Are you still interested to get geanyvc in the repo ? -- Configure 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 490725] Review Request: geanyvc - Version Controler plugin for geany
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490725 --- Comment #4 from Pierre-YvesChibon pin...@pingoured.fr 2009-03-20 04:02:09 EDT --- (In reply to comment #2) 1)koji Build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1250821 missing intltool 2) License looks to be GPLv2+ I'll correct these and upload a new spec/srpm just found this review already submitted https://bugzilla.redhat.com/show_bug.cgi?id=458773 Since there has been no change in the bug since October 2008 I left a comment there asking if the reporter was still interested. Let's see if the reporter is still present :) -- Configure 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 490725] Review Request: geanyvc - Version Controler plugin for geany
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490725 --- Comment #5 from Pierre-YvesChibon pin...@pingoured.fr 2009-03-20 04:15:07 EDT --- SRPM: http://pingou.fedorapeople.org/RPMs/geanyvc-0.5-2.fc10.src.rpm SPEC: http://pingou.fedorapeople.org/RPMs/geanyvc.spec KOJI: http://koji.fedoraproject.org/koji/taskinfo?taskID=1250871 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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=476398
--- Comment #28 from Alex Lancaster al...@users.sourceforge.net 2009-03-20
04:18:35 EDT ---
(In reply to comment #27)
The latest spec is an attempt at that -- the SONAME for each library is set to
liblibrary.so.%{version}.
Compiles on koji for rawhide:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1250851
Regarding versioned SONAMES on other distributions, it seems that only Debian
is packaging this at the moment, and they include it the sagemath package
internally:
http://packages.debian.org/sid/amd64/sagemath/filelist
and don't include any version in the sonames:
/usr/lib/sagemath/local/lib/libcurvesntl.so
/usr/lib/sagemath/local/lib/libg0nntl.so
/usr/lib/sagemath/local/lib/libjcntl.so
/usr/lib/sagemath/local/lib/librankntl.so
3. implement make check. disable the tests that fail for the moment and
report them to upstream so they can be fixed in a later release (hopefully
they
will tell us which one's are expected to work and not work).
That was done in the last version also (except all tests are disabled). How do
you want me to disable just the failing tests?
I would just patch out the targets in the Makefile.
--
Configure 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 491240] Review Request: dropwatch - monitor for dropped network packets in the kernel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491240 --- Comment #2 from Parag AN(पराग) panem...@gmail.com 2009-03-20 05:05:22 EDT --- also, I found spec is not following packaging guidelines 1) use valid buidlroot value from http://fedoraproject.org/wiki/PackagingGuidelines#BuildRoot_tag 2) use macros as given at https://fedoraproject.org/wiki/Packaging/RPMMacros 3) compiler flags are not used. http://fedoraproject.org/wiki/PackagingGuidelines#Compiler_flags 4) License should be GPLv2+ 5) rpmlint gave dropwatch.i386: W: non-standard-group Applications/Utilities -- Configure 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 491097] Review Request: hyphen-fa - Farsi hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491097 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 490832] Review Request: hyphen-ia - Interlingua hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490832 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491240] Review Request: dropwatch - monitor for dropped network packets in the kernel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491240 --- Comment #3 from Parag AN(पराग) panem...@gmail.com 2009-03-20 05:16:01 EDT --- Got one more query not related to packaging but I think ifconfig also gives count of dropped packets. Summary of this package is Kernel dropped packet monitor then what is difference between dropped packets counted by ifconfig and this dropwatch utility? -- Configure 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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=476398
--- Comment #29 from Alex Lancaster al...@users.sourceforge.net 2009-03-20
05:52:31 EDT ---
Regarding %check, looks like the export line should be:
export LD_LIBRARY_PATH=$RPM_BUILD_ROOT%{_libdir}/
otherwise it doesn't find the right versioned .so files at run-time.
--
Configure 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 491128] Review Request: photoprint - Utility for printing digital photographs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491128 --- Comment #2 from Zarko zarko.pin...@gmail.com 2009-03-20 06:13:38 EDT --- Spec URL: http://wiki.open.hr/zpintar/fedora10/SPECS/photoprint.spec SRPM URL: http://wiki.open.hr/zpintar/fedora10/SRPMS/photoprint-0.4.0-2.fc10.src.rpm Solved. I added man page too. Please, take a look. -- Configure 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 451582] Review Request: kvirc - themeable qt based irc client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451582 nucleo alekc...@googlemail.com changed: What|Removed |Added CC||alekc...@googlemail.com --- Comment #8 from nucleo alekc...@googlemail.com 2009-03-20 06:12:04 EDT --- (In reply to comment #2) kvirc QT4 port can be build from svn http://nucleo.fedorapeople.org/pkg-reviews/kvirc/kvirc.spec http://nucleo.fedorapeople.org/pkg-reviews/kvirc/kvirc-4.0-0.3.svn3150.fc10.src.rpm http://koji.fedoraproject.org/koji/taskinfo?taskID=1250981 Mostly works fine but I notice strange behavior of tray icon and panel. -- Configure 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 491084] Review Request: medusa - parallel brute forcing pasword cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491084 --- Comment #8 from Miroslav Suchy msu...@redhat.com 2009-03-20 04:54:06 EDT --- It still do not build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1250906 http://koji.fedoraproject.org/koji/getfile?taskID=1250908name=build.log /var/tmp/rpm-tmp.mzjJhi: line 29: aclocal: command not found You may want to add automake to buildrequires... Please run: koji build --scratch dist-f11 medusa-1.5-2.fc11.src.rpm before sending updated src.rpm and make sure that the build succeed. And rpmlint still give 8 errors. Please run rpmlint -i medusa-1.5-2.fc11.src.rpm And make sure you fixed all errors and 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 488359] Review Request: dcbd - daemon and configuration tool for data center bridging
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488359 --- Comment #14 from Jan Zeleny jzel...@redhat.com 2009-03-20 06:32:23 EDT --- Hopefully final version: Updated SRPM: http://jzeleny.fedorapeople.org/packages/dcbd/dcbd-0.9.7-4.fc10.src.rpm Updated SPEC: http://jzeleny.fedorapeople.org/packages/dcbd/dcbd.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 488359] Review Request: dcbd - daemon and configuration tool for data center bridging
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488359 Jan Zeleny jzel...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #16 from Jan Zeleny jzel...@redhat.com 2009-03-20 06:52:29 EDT --- New Package CVS Request === Package Name: dcbd Short Description: Daemon and configuration tool for Intel Enhanced Ethernet for the Data Center software Owners: jzeleny Branches: 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 465858] Package Review: afpfs-ng - Apple Filing Protocol client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=465858
--- Comment #7 from Jan F. Chadima jchad...@redhat.com 2009-03-20 07:04:31
EDT ---
please add ownership of directory /usr/include/afpsa-ng to devel package
%defattr(-,root,root,-)
+%dir %{_includedir}/afpfs-ng
%{_includedir}/afpfs-ng
--
Configure 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 478617] Review Request: zsync - Partial/differential file transfer client over HTTP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478617 --- Comment #6 from Simon Wesp cassmod...@fedoraproject.org 2009-03-20 06:56:57 EDT --- I tried to patch out internal zlib and failed. I'll try it next week, again! -- Configure 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 488359] Review Request: dcbd - daemon and configuration tool for data center bridging
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488359 Dan Horák d...@danny.cz changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #15 from Dan Horák d...@danny.cz 2009-03-20 06:44:18 EDT --- All issues are fixed, this package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491084] Review Request: medusa - parallel brute forcing pasword cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491084 --- Comment #9 from Jan F. Chadima jchad...@redhat.com 2009-03-20 06:45:41 EDT --- tested with koji build Spec URL: http://www.benhur.prf.cuni.cz/medved-7/wydobitki/fedora/medusa.spec SRPM URL: http://www.benhur.prf.cuni.cz/medved-7/wydobitki/fedora/medusa-1.5-3.fc11.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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476398 --- Comment #30 from Michael Schwendt bugs.mich...@gmx.net 2009-03-20 07:49:33 EDT --- My patch was made for pkg Release 2.p7 and was correct for that release. Maybe the versioned soname changes in 3.p7 require changes to %check. [...] Meanwhile, I've contacted the upstream developer. He's interested in it and has responded very quickly with a guess that the test fails because it cannot write to a temporary directory. That's not it. I've forwarded the manually created verbose test output upstream. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491317] New: Review Request: mingw32-gstreamer - MinGW Windows gstreamer library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mingw32-gstreamer - MinGW Windows gstreamer library https://bugzilla.redhat.com/show_bug.cgi?id=491317 Summary: Review Request: mingw32-gstreamer - MinGW Windows gstreamer library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lfar...@lfarkas.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.lfarkas.org/linux/packages/centos/5/SPECS/mingw32-gstreamer.spec SRPM URL: http://www.lfarkas.org/linux/packages/centos/5/SRPMS/mingw32-gstreamer-0.10.22-1.src.rpm Description: MinGW Windows gstreamer library koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1251165 Approved packaging guidelines for MinGW are here: http://fedoraproject.org/wiki/Packaging/MinGW -- Configure 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 489929] Review request: libHBAAPI
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489929 Ronald Pacheco rpach...@redhat.com changed: What|Removed |Added Blocks||488382 -- Configure 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 488359] Review Request: dcbd - daemon and configuration tool for data center bridging
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488359 Ronald Pacheco rpach...@redhat.com changed: What|Removed |Added Blocks||488382 -- Configure 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 489962] Review request: libhbalinux - SNIA HBAAPI vendor 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=489962 Ronald Pacheco rpach...@redhat.com changed: What|Removed |Added Blocks||488382 -- Configure 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 491084] Review Request: medusa - parallel brute forcing pasword cracker
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=491084
--- Comment #10 from Miroslav Suchy msu...@redhat.com 2009-03-20 08:42:34
EDT ---
OK. It build now.
Although there is huge list of compilers warnings. (will be nice to fix, but
not necessery).
There is also warning:
checking for openssl/ssl.h... no
configure: WARNING: *** OpenSSL header files required for SSL support. ***
Many of the Medusa modules depend on the OpenSSL library and header files.
If
multiple modules are unexpectedly disabled, this is likely the cause. Make
sure
to install libssl-dev, openssl-devel or whatever package your distribution
uses
to distribute these files.
not sure if you would like to enable it by specifying additional requirements.
Rpmlint still say a lot of errors:
Do not use
rm -rf ${buildroot}
use either
rm -Rf $RPM_BUILD_ROOT
or
rm -rf %{buildroot}
medusa.src: W: mixed-use-of-spaces-and-tabs (spaces: line 24, tab: line 11)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance. Use either spaces or tabs for indentation, not both.
Your description lines must not exceed 79 characters. If a line is exceeding
this number, cut it to fit in two or more lines.
medusa.src: W: invalid-license GPL
The value of the License tag was not recognized. Known values are: GPL+,
GPL+ or Artistic, GPLv1,
GPLv2 or Artistic, GPLv2+ or Artistic, GPLv2, GPLv2 with exceptions,
GPLv2+, GPLv2+ with exceptions, GPLv3, GPLv3 with exceptions,
GPLv3+, GPLv3+ with exceptions
When you will fix these errors, I will check rest of the requirements.
--
Configure 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 491240] Review Request: dropwatch - monitor for dropped network packets in the kernel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491240 --- Comment #4 from Neil Horman nhor...@redhat.com 2009-03-20 07:17:37 EDT --- Thank you. New package for review Spec URL: https://fedorahosted.org/releases/d/r/dropwatch/dropwatch.spec SRPM URL: https://fedorahosted.org/releases/d/r/dropwatch/dropwatch-1.0-2.fc9.src.rpm Regarding the difference between dropwatch and ifconfig, its all written up on the hosted page: http://fedorahosted.org/dropwatch Ifconfig measures dropped packets in the hardware and driver of a specific interface, while dropwatch monitors them throughout the network stack, disambiguating their location and cause. 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 486977] Review Request: gnu-free-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=486977 --- Comment #15 from Jon Ciesla l...@jcomserv.net 2009-03-20 09:08:39 EDT --- I feel your pain. :) SPEC: http://zanoni.jcomserv.net/fedora/gnu-free-fonts/gnu-free-fonts.spec SRPM: http://zanoni.jcomserv.net/fedora/gnu-free-fonts/gnu-free-fonts-20090104-9.fc10.src.rpm Fixed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added CC||fedora-mi...@lists.fedorapr ||oject.org, ||rjo...@redhat.com Alias||mingw32-gstreamer -- Configure 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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #1 from Richard W.M. Jones rjo...@redhat.com 2009-03-20 09:30:34 EDT --- Koji scratch-build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1251262 -- Configure 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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #2 from Richard W.M. Jones rjo...@redhat.com 2009-03-20 09:32:48 EDT --- Auto-buildrequires says: BuildRequires: bash = 3.2.30.fc10.x86_64 BuildRequires: binutils = 2.18.50.0.9.8.fc10.x86_64 BuildRequires: bison = 2.3.5.fc9.x86_64 BuildRequires: bzip2 = 1.0.5.3.fc10.x86_64 BuildRequires: coreutils = 6.12.20.fc10.x86_64 BuildRequires: cpp = 4.3.2.7.x86_64 BuildRequires: diffutils = 2.8.1.21.fc9.x86_64 BuildRequires: file = 4.26.4.fc10.x86_64 BuildRequires: findutils = 1:4.4.0.1.fc10.x86_64 BuildRequires: flex = 2.5.35.2.fc10.x86_64 BuildRequires: gawk = 3.1.6.2.fc10.x86_64 BuildRequires: gcc = 4.3.2.7.x86_64 BuildRequires: gettext = 0.17.8.fc10.x86_64 BuildRequires: glib2-devel = 2.18.4.1.fc10.x86_64 BuildRequires: glibc-common = 2.9.3.x86_64 BuildRequires: grep = 2.5.1a.61.fc10.x86_64 BuildRequires: gtk-doc = 1.10.2.fc10.noarch BuildRequires: kdelibs3 = 3.5.10.3.fc10.x86_64 BuildRequires: make = 1:3.81.14.fc10.x86_64 BuildRequires: mingw32-binutils = 2.19.2.fc10.x86_64 BuildRequires: mingw32-cpp = 4.3.2.12.fc10.x86_64 BuildRequires: mingw32-dlfcn = 0.0.3.r11.fc10.noarch BuildRequires: mingw32-filesystem = 43.6.fc11.noarch BuildRequires: mingw32-gcc = 4.3.2.12.fc10.x86_64 BuildRequires: mingw32-gettext = 0.17.7.fc10.noarch BuildRequires: mingw32-glib2 = 2.19.5.2.1.fc10.noarch BuildRequires: mingw32-iconv = 1.12.7.fc10.noarch BuildRequires: mingw32-libxml2 = 2.7.2.6.fc10.noarch BuildRequires: mingw32-runtime = 3.15.1.10.fc10.noarch BuildRequires: mingw32-w32api = 3.12.8.fc10.noarch BuildRequires: net-tools = 1.60.91.fc10.x86_64 BuildRequires: patch = 2.5.4.35.fc10.x86_64 BuildRequires: perl = 4:5.10.0.56.fc10.x86_64 BuildRequires: pkgconfig = 1:0.23.6.fc10.x86_64 BuildRequires: sed = 4.1.5.11.fc10.x86_64 BuildRequires: tar = 2:1.20.5.fc10.x86_64 BuildRequires: which = 2.19.3.fc9.x86_64 -- Configure 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 489026] Review Request: auto-buildrequires - Work out BuildRequires for rpmbuild automatically
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489026 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #15 from Richard W.M. Jones rjo...@redhat.com 2009-03-20 09:32:27 EDT --- Package Change Request == Package Name: auto-buildrequires New Branches: EL-5 Owners: oget rjones -- Configure 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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #3 from Richard W.M. Jones rjo...@redhat.com 2009-03-20 09:35:40 EDT --- rpmlint output: mingw32-gstreamer.src:534: W: macro-in-%changelog doc -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #4 from Levente Farkas lfar...@lfarkas.org 2009-03-20 09:40:35 EDT --- it'd be useful to somehow shorten the length of this Auto-buildrequires. eg: don't include default pacakges like patch, sed, etc. don't include pacakges which already required by other BR's eg. mingw32-glib2 already BR all other mingw32* and there are some noise since gcc shouldn't have to be in list. for the rpmlint warning i saw it, but wouldn't like to change native pacakge's changelog. -- Configure 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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 --- Comment #5 from Richard W.M. Jones rjo...@redhat.com 2009-03-20 09:44:08 EDT --- Sure thing! Patches welcome ... -- Configure 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 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||491317(mingw32-gstreamer) -- Configure 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 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||491317(mingw32-gstreamer) -- Configure 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 454410] Review Request: mingw32-gcc - MinGW Windows cross-compiler (GCC) for C and C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=454410 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Blocks||491317(mingw32-gstreamer) -- Configure 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 491317] Review Request: mingw32-gstreamer - MinGW Windows gstreamer 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=491317 Richard W.M. Jones rjo...@redhat.com changed: What|Removed |Added Depends on||454410(mingw32-gcc), ||467402(mingw32-glib2), ||467405(mingw32-libxml2) -- Configure 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 491331] New: Review Request: spacewalk-config - Spacewalk Configuration
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: spacewalk-config - Spacewalk Configuration Alias: spacewalk-config https://bugzilla.redhat.com/show_bug.cgi?id=491331 Summary: Review Request: spacewalk-config - Spacewalk Configuration Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: msu...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Blocks: 452450 Estimated Hours: 0.0 Classification: Fedora Target Release: --- SRPM: http://miroslav.suchy.cz/fedora/spacewalk-config/spacewalk-config-0.5.7-1.src.rpm SPEC: http://miroslav.suchy.cz/fedora/spacewalk-config/spacewalk-config.spec Description: Spacewalk Configuration Templates Rpmlint write out several warning, which I would like to explain dangling-relative-symlink - that is to make apache work with spacewalk ssl certificate non-conffile-in-etc /etc/sysconfig/rhn-satellite-prep - it is not configuration file. It is in fact template, which we want to be replaced with package upgrade. I filled BZ 491301 dangerous-command-in-%pre perl - is ok. although we can use sed, the perl command is much more readible. -- Configure 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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 Miroslav Suchy msu...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||msu...@redhat.com Flag||fedora-review? --- Comment #1 from Miroslav Suchy msu...@redhat.com 2009-03-20 10:11:54 EDT --- Will do this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486584] Review Request: perl-CGI-Application - Framework for building reusable web-applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486584 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-03-20 10:16:57 EDT --- New Package CVS Request === Package Name: perl-CGI-Application Short Description: Framework for building reusable web-applications Owners: eseyman Branches: F-9 F-10 InitialCC: perl-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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 Miroslav Suchy msu...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|msu...@redhat.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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added CC||rc040...@freenet.de --- Comment #2 from Ralf Corsepius rc040...@freenet.de 2009-03-20 10:41:44 EDT --- (In reply to comment #0) Due to current @INC ordering, at the moment this will only work properly in rawhide. Care to explain? Such kind of issues often indicate (include-) file conflicts, i.e. likely indicates a severe design flaw somewhere. -- Configure 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 451582] Review Request: kvirc - themeable qt based irc client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451582 --- Comment #9 from nucleo alekc...@googlemail.com 2009-03-20 10:41:01 EDT --- Updated spec file fixes all rpmlint errors with binary rpm's: 5 packages and 1 specfiles checked; 0 errors, 0 warnings http://nucleo.fedorapeople.org/pkg-reviews/kvirc/kvirc.spec http://nucleo.fedorapeople.org/pkg-reviews/kvirc/kvirc-4.0-0.4.svn3151.fc10.src.rpm Build logs can be found here http://koji.fedoraproject.org/koji/taskinfo?taskID=1251297 -- Configure 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 487665] Review Request: soud - Tools for hardware related services based on udev events
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487665 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||WONTFIX -- Configure 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 465858] Package Review: afpfs-ng - Apple Filing Protocol client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=465858
--- Comment #9 from Stepan Kasal ska...@redhat.com 2009-03-20 11:08:39 EDT
---
(In reply to comment #7)
please add ownership of directory /usr/include/afpsa-ng to devel package
%defattr(-,root,root,-)
+%dir %{_includedir}/afpfs-ng
%{_includedir}/afpfs-ng
Hi Jan,
sorry but this is a misunderstanding. The current spec means that the dir and
all its contents gets packed. With your change, the dir itself would get
packed twice.
An alternate form would be:
%dir %{_includedir}/afpfs-ng
%{_includedir}/afpfs-ng/*.h
This way, rpm would complain if the directory contained anything but the *.h
files.
But I'm perfectly happy with the current form as well.
Stepan
--
Configure 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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 --- Comment #3 from Chris Weyl cw...@alumni.drew.edu 2009-03-20 11:08:27 EDT --- (In reply to comment #2) (In reply to comment #0) Due to current @INC ordering, at the moment this will only work properly in rawhide. Care to explain? Such kind of issues often indicate (include-) file conflicts, i.e. likely indicates a severe design flaw somewhere. Sure -- see bug 489221 for details, but the short version is that this package contains a newer Fatal.pm than core. As in currently released Perl packages core comes before vendor in @INC, it's not a simple matter to override core by delivering anything in vendor. -- Configure 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 490865] Review Request: perl-Date-Leapyear - Is a particular year a leap year?
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490865 --- Comment #5 from Xavier Bachelot xav...@bachelot.org 2009-03-20 11:15:34 EDT --- Imported and built for F-9, F-10, EL-4 and EL-5. -- Configure 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 491201] Review Request: ninvaders - Space Invaders clone written in ncurses for cli gaming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491201 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Status|NEW |ASSIGNED CC||jussi.leht...@iki.fi AssignedTo|nob...@fedoraproject.org|jussi.leht...@iki.fi Flag||fedora-review? --- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 11:16:00 EDT --- I'll do the review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 490576] Review Request: bibtex2html - Collection of tools for translating from BibTeX to HTML
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490576 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 11:14:42 EDT --- True, I seem to have missed that. I have reviewed the package and found it to conform with the Fedora Packaging and Package Review Guidelines. The package is hereby ACCEPTED. Now, you still need to get a sponsor before you can get the package to CVS. You should probably make a couple of unofficial reviews on other Review Requests on Bugzilla, so that sponsors see that you know the guidelines. You might also want to submit other packages for review, but at the moment Bugzilla is quite cramped and we're hoping for more package reviews. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491201] Review Request: ninvaders - Space Invaders clone written in ncurses for cli gaming
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=491201
--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 11:26:29
EDT ---
rpmlint output:
ninvaders.x86_64: W: file-not-utf8 /usr/share/doc/ninvaders-0.1.1/ChangeLog
3 packages and 0 specfiles checked; 0 errors, 1 warnings.
- Fix the rpmlint warning with
iconv -f iso-8859-1 -t utf8 ChangeLog ChangeLog.new \
touch -r ChangeLog ChangeLog.new mv ChangeLog.new ChangeLog
- Change buildroot to the Guideline's recommended version
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
- Remove Requires: ncurses as this is automatically picked up by rpmbuild.
- Add trailing period to %description
- Enable smp make with
make %{?_smp_mflags}
- Is there a reason why you change the name of the binary? I think you should
keep it consistent with upstream...
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=491268
Miroslav Suchy msu...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #4 from Miroslav Suchy msu...@redhat.com 2009-03-20 11:38:12 EDT
---
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: ppc
[x] Rpmlint output: empty
[x] Package is not relocatable.
[x] Buildroot is correct
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic
[-] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 42dd17fa43025b02a9f300dedf95ced3
autodie-1.999.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -fR $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: koji scratch build
[x] Package should compile and build into binary rpms on all supported
architectures.
Tested on:koji scratch build
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] %check is present and the tests pass
*** 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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 --- Comment #5 from Ralf Corsepius rc040...@freenet.de 2009-03-20 12:16:01 EDT --- (In reply to comment #3) (In reply to comment #2) (In reply to comment #0) Due to current @INC ordering, at the moment this will only work properly in rawhide. Care to explain? Such kind of issues often indicate (include-) file conflicts, i.e. likely indicates a severe design flaw somewhere. Sure -- see bug 489221 for details, but the short version is that this package contains a newer Fatal.pm than core. = your package overrides a module from core, which will interfere with any other user of Fatal.pm It's a mistake to approve and add it to 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 490576] Review Request: bibtex2html - Collection of tools for translating from BibTeX to HTML
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490576 --- Comment #11 from Guido Grazioli guido.grazi...@gmail.com 2009-03-20 12:17:32 EDT --- Thank you very much for your help Jussi; i'll look for some small easy package to review to start with. -- Configure 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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added Flag|fedora-review+ |fedora-review? --- Comment #6 from Ralf Corsepius rc040...@freenet.de 2009-03-20 12:19:29 EDT --- Revoking approval. This issue needs further discussion. -- Configure 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 491268] Review Request: perl-autodie - Replace functions with equivalents which succeed or die
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491268 --- Comment #7 from Miroslav Suchy msu...@redhat.com 2009-03-20 12:23:16 EDT --- Well if you do not want to overriden, then you should not install it... Anyway they both provide the same functionality and can be exchanged for each other if I read it correctly. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453848] Review Request: globus-core - Globus Toolkit - Globus Core
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=453848 --- Comment #19 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-03-20 12:30:27 EDT --- For some reason bodhi fails to add these comments automatically. (The web interface says Unable to access one or more bugs.) So I add these myself... globus-core-5.15-1.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/globus-core-5.15-1.fc9 globus-core-5.15-1.fc9 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/globus-core-5.15-1.fc10 -- Configure 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 489550] Review Request: qtscriptgenerator - Qt bindings for Qt Script
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489550 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Alias||qtscriptgenerator --- Comment #1 from Rex Dieter rdie...@math.unl.edu 2009-03-20 12:34:56 EDT --- Spec URL: http://rdieter.fedorapeople.org/pkg-reviews/qtscriptgenerator/qtscriptgenerator.spec SRPM URL: http://rdieter.fedorapeople.org/pkg-reviews/qtscriptgenerator/qtscriptgenerator-0.1.0-2.fc10.src.rpm %changelog Fri Mar 20 2009 Rex Dieter rdie...@fdoraproject.org 0.1.0-2 - qt-4.5.0-7 fixed wrt phonon, drop no_phonon patch Will issue new scratch build as soon as the fixed qt lands in koji buildroot. -- Configure 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 491084] Review Request: medusa - parallel brute forcing pasword cracker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491084 --- Comment #11 from Jan F. Chadima jchad...@redhat.com 2009-03-20 12:40:08 EDT --- Spec URL: http://www.benhur.prf.cuni.cz/medved-7/wydobitki/fedora/medusa.spec SRPM URL: http://www.benhur.prf.cuni.cz/medved-7/wydobitki/fedora/medusa-1.5-4.fc11.src.rpm koji tested, rpmlint tested all modules except afp are included. -- Configure 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 485416] Review Request: msp430-gcc - Cross compiling GNU GCC for the MSP430
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485416 Steve Whitehouse swhit...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Steve Whitehouse swhit...@redhat.com 2009-03-20 12:57:39 EDT --- I've not heard from Ralf yet, but everything looks in order so I'm going to approve this now and we can tidy up any loose ends as we go. -- Configure 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 474549] Review Request: ca-cacert.org - CAcert.org CA root certificates
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474549 Itamar Reis Peixoto ita...@ispbrasil.com.br changed: What|Removed |Added CC||ita...@ispbrasil.com.br --- Comment #8 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2009-03-20 13:08:10 EDT --- http://svn.cacert.org/CAcert/Policies/Agreements/3PVDisclaimerAndLicence.html -- Configure 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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=476398
--- Comment #31 from Conrad Meyer kon...@tylerc.org 2009-03-20 13:16:00 EDT
---
(In reply to comment #29)
Regarding %check, looks like the export line should be:
export LD_LIBRARY_PATH=$RPM_BUILD_ROOT%{_libdir}/
otherwise it doesn't find the right versioned .so files at run-time.
I've updated this:
http://konradm.fedorapeople.org/fedora/SPECS/eclib.spec
http://konradm.fedorapeople.org/fedora/SRPMS/eclib-20080310-3.p7.fc10.src.rpm
(In reply to comment #30)
Meanwhile, I've contacted the upstream developer. He's interested in it and
has
responded very quickly with a guess that the test fails because it cannot
write
to a temporary directory. That's not it. I've forwarded the manually created
verbose test output upstream.
Thank you very much. I guess we're in luck that upstream cares :).
--
Configure 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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=476398
--- Comment #32 from Conrad Meyer kon...@tylerc.org 2009-03-20 13:17:32 EDT
---
(In reply to comment #29)
Regarding %check, looks like the export line should be:
export LD_LIBRARY_PATH=$RPM_BUILD_ROOT%{_libdir}/
otherwise it doesn't find the right versioned .so files at run-time.
I've updated this:
http://konradm.fedorapeople.org/fedora/SPECS/eclib.spec
http://konradm.fedorapeople.org/fedora/SRPMS/eclib-20080310-4.p7.fc10.src.rpm
(In reply to comment #30)
Meanwhile, I've contacted the upstream developer. He's interested in it and
has
responded very quickly with a guess that the test fails because it cannot
write
to a temporary directory. That's not it. I've forwarded the manually created
verbose test output upstream.
Thank you very much. I guess we're in luck that upstream cares :).
--
Configure 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 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476398 --- Comment #33 from Michael Schwendt bugs.mich...@gmx.net 2009-03-20 13:35:17 EDT --- News: Upstream cannot reproduce the test failure ... and still thinks it is somehow related to not finding a file. Doubtful IMO, also after checking strace -- the wrong value found during the test is read from a file which is created from scratch. [...] eclib.spec should BuildRequires: ntl-static instead of ntl-devel, because NTL is a static-only package. Due to a bug in ntl.spec, ntl-devel didn't provide ntl-static so far. ntl-5.4.2-7.fc11 shall fix this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476398] Review Request: eclib - A Library for Doing Computations on Elliptic Curves
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476398 --- Comment #34 from Conrad Meyer kon...@tylerc.org 2009-03-20 13:54:47 EDT --- (In reply to comment #33) News: Upstream cannot reproduce the test failure ... and still thinks it is somehow related to not finding a file. Doubtful IMO, also after checking strace -- the wrong value found during the test is read from a file which is created from scratch. Hopefully they'll be able to figure it out. eclib.spec should BuildRequires: ntl-static instead of ntl-devel, because NTL is a static-only package. Due to a bug in ntl.spec, ntl-devel didn't provide ntl-static so far. ntl-5.4.2-7.fc11 shall fix this. Ah, thanks. http://konradm.fedorapeople.org/fedora/SPECS/eclib.spec http://konradm.fedorapeople.org/fedora/SRPMS/eclib-20080310-5.p7.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 476346] Review Request: python-polybori - Framework for Boolean Rings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476346 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Status Whiteboard||NotReady --- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2009-03-20 14:15:33 EDT --- Please clear the whiteboard when someone gets this building. -- Configure 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 453851] Review Request: globus-common - Globus Toolkit - Common 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=453851
Bug 453851 depends on bug 453847, which changed state.
Bug 453847 Summary: Review Request: grid-packaging-tools - The Grid Packaging
Tools (GPT)
https://bugzilla.redhat.com/show_bug.cgi?id=453847
What|Old Value |New Value
Status|ASSIGNED|CLOSED
Resolution||NEXTRELEASE
--- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se 2009-03-20
14:32:31 EDT ---
(In reply to comment #5)
Small suggestion, instead of having to enumerate all known 64bit arches via
%ifarch alpha ia64 ppc64 s390x sparc64 x86_64
you could consider using a something like:
%if %{_lib} == lib64
Not all 64 bit architectures use /lib64.
--
Configure 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 487901] Review Request: zikula - web-based CMS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487901 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2009-03-20 14:50:46 EDT --- zikula-1.1.1-12.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/zikula-1.1.1-12.fc10 -- Configure 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 487901] Review Request: zikula - web-based CMS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487901 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2009-03-20 14:50:40 EDT --- zikula-1.1.1-12.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/zikula-1.1.1-12.fc9 -- Configure 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 468753] Review Request: nss-myhostname - glibc plugin for local system host name resolution
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=468753 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-03-20 15:01:11 EDT --- It's nice to give a new src.rpm so that I can do another build, but I just updated the old package. It looks good to me. 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 486390] Review Request: simspark - Spark physical simulation 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=486390 Hedayat Vatankhah heda...@grad.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #12 from Hedayat Vatankhah heda...@grad.com 2009-03-20 15:03:18 EDT --- New Package CVS Request === Package Name: simspark Short Description: Spark physical simulation system Owners: hedayat Branches: F-9 F-10 -- Configure 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 491201] Review Request: ninvaders - Space Invaders clone written in ncurses for cli gaming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491201 --- Comment #3 from Adam Miller maxamill...@gmail.com 2009-03-20 15:19:20 EDT --- Spec URL: http://maxamillion.fedorapeople.org/ninvaders.spec SRPM URL: http://maxamillion.fedorapeople.org/ninvaders-0.1.1-2.src.rpm I have fixed all that was mentioned. I also went ahead and changed the name of the binary back to upstream. It was purely a preference choice but I do agree that keeping synchronized with upstream is a better idea. Thank you for your time, looking forward to further review. :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491201] Review Request: ninvaders - Space Invaders clone written in ncurses for cli gaming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491201 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 15:33:00 EDT --- rpmlint output is clean. The package adheres to the Fedora Packaging and Package review Guidelines and is thus 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 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 --- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 15:53:51 EDT --- ping, it's been more than a 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 481717] Review Request: ugene - genome analysis suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481717 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp Flag||fedora-review? --- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-03-20 15:56:09 EDT --- Assigning. -- Configure 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 481717] Review Request: ugene - genome analysis suite
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=481717
--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-03-20
15:57:37 EDT ---
Created an attachment (id=336109)
-- (https://bugzilla.redhat.com/attachment.cgi?id=336109)
Patch to compile with g++44
Some notes
* Latest
- The latest version is 1.4.0 (and it seems 1.4.1 will be
released soon)
* License tag
- src/libs_3rdparty/qtbindings_core/src/qtscriptconcurrent.h is
under GPLv2, which renders libqtbindings_core.so to be
GPLv2.
So the license tag should be GPLv2+ and GPLv2.
Also please write some comments about this on the spec file,
see:
https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#Multiple_Licensing_Scenarios
* Explicit version dependencies
- We now request comments on the spec file when you write
explicit version dependencies for rpms which are automatically
pulled in by the automated detection of library dependencies
by rpmbuild itself, see:
https://fedoraproject.org/wiki/PackagingDrafts/ExplicitRequires
(this is still under Drafts category, but actually this is
already approved by FESCo)
* Multiple Source entry
- There are 2 Source entry. Use full URL one:
https://fedoraproject.org/wiki/Packaging/SourceURL
* 64 bit issue
(In reply to comment #6)
As long as default rpm macros values are identical to preconfigured paths
everything goes OK, but when they are not - rpm build will fail. In fact, the
main problem will be with %{_libdir} since on 64bit platforms it may be
changed
from /usr/lib to /usr/lib64.
Is there any workaround for this problem (e.g. using hardoced libdir path)?
- For this package
-
sed -i -e '/UGENE_INSTALL_DIR/s|/lib/|/%{_lib}/|' src/ugene_globals.pri
-
works.
* g++44 compilation issue
- The attached patch is needed to build this package with g++ 4.4
(please check if this is already fixed in the latest version or not)
* Functionality
- By the way:
-
$ ugene
Translation not found: transl_ja_JP
Translation not found: transl_en
No translations found, exiting
-
ugene won't launch on my system (note that I am using GNOME)
--
Configure 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 226160] Merge Review: mpage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226160 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added CC||jussi.leht...@iki.fi --- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 16:22:01 EDT --- rpmlint output: mpage.src:9: W: hardcoded-path-in-buildroot-tag /var/tmp/mpage-root mpage.src: W: summary-ended-with-dot A tool for printing multiple pages of text on each printed page. mpage.src: W: no-url-tag mpage.x86_64: W: file-not-utf8 /usr/share/doc/mpage-2.5.6/CHANGES mpage.x86_64: W: summary-ended-with-dot A tool for printing multiple pages of text on each printed page. mpage.x86_64: W: no-url-tag mpage-debuginfo.x86_64: W: no-url-tag 3 packages and 0 specfiles checked; 0 errors, 7 warnings. - Fix the rpmlint warnings; url should be http://www.mesa.nl/pub/mpage/ (AFAIK no project home page exists). - Some problem with the license? Program source code seems to be licensed under the following license: * Permission is granted to anyone to make or distribute verbatim * copies of this document as received, in any medium, provided * that this copyright notice is preserved, and that the * distributor grants the recipient permission for further * redistribution as permitted by this notice. On the other hand, the file gpl.in states that mpage is licensed under GPLv2+. Please contact upstream to clarify license. - Include FAQ, COPYING, COPYING.LESSER to copyright. -- 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 491201] Review Request: ninvaders - Space Invaders clone written in ncurses for cli gaming
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491201 Adam Miller maxamill...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Adam Miller maxamill...@gmail.com 2009-03-20 16:36:08 EDT --- New Package CVS Request === Package Name: ninvaders Short Description: Space Invaders clone written in ncurses for cli gaming Owners: maxamillion Branches: F-9 F-10 EL-4 EL-5 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 490724] Review Request: R-RUnit - R Unit test framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490724 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added CC||kon...@tylerc.org AssignedTo|nob...@fedoraproject.org|kon...@tylerc.org Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226146] Merge Review: mkinitrd
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226146
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
CC||jussi.leht...@iki.fi
--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 16:40:48
EDT ---
The package needs some amount of love:
grubby.x86_64: E: executable-marked-as-config-file /sbin/installkernel
grubby.x86_64: E: tag-not-utf8 %changelog
grubby.x86_64: W: invalid-url git://git.fedoraproject.org/git/hosted/mkinitrd
grubby.x86_64: W: non-etc-or-var-file-marked-as-conffile /sbin/installkernel
grubby.x86_64: W: conffile-without-noreplace-flag /sbin/installkernel
libbdevid-python.x86_64: W: no-documentation
libbdevid-python.x86_64: E: non-standard-executable-perm
/usr/lib64/python2.5/site-packages/bdevid.so 0775
libbdevid-python.x86_64: E: explicit-lib-dependency device-mapper-libs
libbdevid-python.x86_64: E: explicit-lib-dependency libselinux
libbdevid-python.x86_64: E: explicit-lib-dependency libsepol
libbdevid-python.x86_64: E: tag-not-utf8 %changelog
libbdevid-python.x86_64: W: invalid-url
git://git.fedoraproject.org/git/hosted/mkinitrd
mkinitrd.src: E: non-utf8-spec-file
/tmp/mkinitrd-6.0.81-1.fc10.src.rpm.30578/mkinitrd.spec
mkinitrd.src:1614: W: macro-in-%changelog config
mkinitrd.src: W: summary-ended-with-dot Creates an initial ramdisk image for
preloading modules.
mkinitrd.src: E: tag-not-utf8 %changelog
mkinitrd.src: W: invalid-url git://git.fedoraproject.org/git/hosted/mkinitrd
mkinitrd.x86_64: E: explicit-lib-dependency libselinux
mkinitrd.x86_64: E: explicit-lib-dependency libsepol
mkinitrd.x86_64: W: summary-ended-with-dot Creates an initial ramdisk image for
preloading modules.
mkinitrd.x86_64: E: tag-not-utf8 %changelog
mkinitrd.x86_64: W: invalid-url git://git.fedoraproject.org/git/hosted/mkinitrd
mkinitrd.x86_64: E: no-binary
mkinitrd-debuginfo.x86_64: E: tag-not-utf8 %changelog
mkinitrd-debuginfo.x86_64: W: invalid-url
git://git.fedoraproject.org/git/hosted/mkinitrd
mkinitrd-devel.x86_64: W: no-documentation
mkinitrd-devel.x86_64: W: no-version-dependency-on
mkinitrd/mkinitrd-libs/libmkinitrd 6.0.81
mkinitrd-devel.x86_64: W: summary-ended-with-dot C header files and library for
functionality exported by libnash.
mkinitrd-devel.x86_64: E: tag-not-utf8 %changelog
mkinitrd-devel.x86_64: W: invalid-url
git://git.fedoraproject.org/git/hosted/mkinitrd
nash.x86_64: E: non-standard-executable-perm /lib64/bdevid/usb.so 0775
nash.x86_64: E: library-without-ldconfig-postin /usr/lib64/libbdevid.so.6.0.81
nash.x86_64: E: library-without-ldconfig-postun /usr/lib64/libbdevid.so.6.0.81
nash.x86_64: E: non-standard-executable-perm /lib64/bdevid/scsi.so 0775
nash.x86_64: E: non-standard-executable-perm /lib64/bdevid/ata.so 0775
nash.x86_64: E: library-without-ldconfig-postin /usr/lib64/libnash.so.6.0.81
nash.x86_64: E: library-without-ldconfig-postun /usr/lib64/libnash.so.6.0.81
nash.x86_64: E: explicit-lib-dependency device-mapper-libs
nash.x86_64: E: explicit-lib-dependency e2fsprogs-libs
nash.x86_64: E: explicit-lib-dependency libselinux
nash.x86_64: E: explicit-lib-dependency libsepol
nash.x86_64: E: explicit-lib-dependency zlib
nash.x86_64: W: summary-not-capitalized nash
nash.x86_64: E: tag-not-utf8 %changelog
nash.x86_64: W: invalid-url git://git.fedoraproject.org/git/hosted/mkinitrd
7 packages and 0 specfiles checked; 28 errors, 17 warnings.
In addition:
- Requires and BuildRequires are a mess. Please: alphabetize and break
requirements one per line. At least fix mixed use of spaces and commas.
- Drop ExclusiveOs
- BuildRoot should be
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
- Description of nash should be extended.
--
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 490724] Review Request: R-RUnit - R Unit test framework
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=490724
--- Comment #1 from Conrad Meyer kon...@tylerc.org 2009-03-20 16:54:05 EDT
---
Created an attachment (id=336118)
-- (https://bugzilla.redhat.com/attachment.cgi?id=336118)
[Review]
Attached is my initial review.
In summary:
- Fix the UTF-8 and one-line-command-in-%post(un) rpmlint warnings.
- Fix the License (GPLv2+)
- Fix the duplicate files
- Why is %{_bindir}/R used in %check? Also, you may want to use %{version} in
your Source0. But that's discretionary.
Everything else looks good.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225978] Merge Review: kudzu
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=225978
Jussi Lehtola jussi.leht...@iki.fi changed:
What|Removed |Added
CC||jussi.leht...@iki.fi
--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-03-20 16:59:42
EDT ---
rpmlint output:
kudzu.src: E: non-utf8-spec-file /tmp/kudzu-1.2.85-3.src.rpm.3418/kudzu.spec
kudzu.src:8: W: unversioned-explicit-obsoletes rhs-hwdiag
kudzu.src:8: W: unversioned-explicit-obsoletes setconsole
kudzu.src:9: E: prereq-use /sbin/chkconfig
kudzu.src:12: E: buildprereq-use pciutils-devel = 2.2.3-4, python-devel python
gettext zlib-devel popt-devel
kudzu.src:30: W: setup-not-quiet
kudzu.src:506: W: macro-in-%changelog d
kudzu.src: W: summary-ended-with-dot The Red Hat Linux hardware probing tool.
kudzu.src: E: tag-not-utf8 %changelog
kudzu.src: W: no-url-tag
kudzu.x86_64: W: summary-ended-with-dot The Red Hat Linux hardware probing
tool.
kudzu.x86_64: E: tag-not-utf8 %changelog
kudzu.x86_64: W: no-url-tag
kudzu.x86_64: W: obsolete-not-provided rhs-hwdiag
kudzu.x86_64: W: obsolete-not-provided setconsole
kudzu-debuginfo.x86_64: E: tag-not-utf8 %changelog
kudzu-debuginfo.x86_64: W: no-url-tag
kudzu-devel.x86_64: W: no-documentation
kudzu-devel.x86_64: W: summary-ended-with-dot Development files needed for
hardware probing using kudzu.
kudzu-devel.x86_64: E: tag-not-utf8 %changelog
kudzu-devel.x86_64: W: no-url-tag
4 packages and 0 specfiles checked; 7 errors, 14 warnings.
- Change source line to use %{version}
- Change buildroot to
%(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
- Requires, obsoletes etc need to be cleaned up
After these have been fixed the package would seem to be good to go.
--
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 490724] Review Request: R-RUnit - R Unit test framework
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=490724
--- Comment #2 from Pierre-YvesChibon pin...@pingoured.fr 2009-03-20
17:15:06 EDT ---
(In reply to comment #1)
Created an attachment (id=336118)
-- (https://bugzilla.redhat.com/attachment.cgi?id=336118) [details]
[Review]
Attached is my initial review.
In summary:
- Fix the UTF-8 and one-line-command-in-%post(un) rpmlint warnings.
I would rather not because if I do so the pdf generated is corrupted. In
addition please see :
https://www.redhat.com/archives/fedora-devel-list/2009-January/msg00741.html
Regarding the one-line-command-in-%post(un) see:
http://fedoraproject.org/wiki/Packaging/R
- Fix the License (GPLv2+)
This actually needs information from upstream since the license version are
different in the DESCRIPTION file, the website and the sources.
- Fix the duplicate files
Fixed
- Why is %{_bindir}/R used in %check?
Fixed (change R CMD INSTALL to %{_bindir}R CMD INSTALL)
Also, you may want to use %{version} in
your Source0. But that's discretionary.
Fixed
I will upload the new version as soon as I have news from upstream concerning
the license used.
Thanks for the review
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 470702] Review Request: L-function - L-function calculator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=470702 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #8 from Jason Tibbitts ti...@math.uh.edu 2009-03-20 17:37:45 EDT --- The new package builds fine and includes the header files in the proper place. Everything looks good to me. 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 491425] New: Review Request: R-RODBC - An ODBC database interface for R
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: R-RODBC - An ODBC database interface for R https://bugzilla.redhat.com/show_bug.cgi?id=491425 Summary: Review Request: R-RODBC - An ODBC database interface for R Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tcall...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://www.auroralinux.org/people/spot/review/new/R-RODBC.spec SRPM URL: http://www.auroralinux.org/people/spot/review/new/R-RODBC-1.2-1.fc11.src.rpm Description: An ODBC database interface for R R-biglm (one of my other packages) grew this as a new dependency in its latest release, so I need this in order to update 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 490724] Review Request: R-RUnit - R Unit test framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490724 --- Comment #3 from Conrad Meyer kon...@tylerc.org 2009-03-20 17:44:17 EDT --- (In reply to comment #2) (In reply to comment #1) - Fix the UTF-8 and one-line-command-in-%post(un) rpmlint warnings. I would rather not because if I do so the pdf generated is corrupted. In addition please see : https://www.redhat.com/archives/fedora-devel-list/2009-January/msg00741.html Regarding the one-line-command-in-%post(un) see: http://fedoraproject.org/wiki/Packaging/R Ok, I accept these warnings. - Fix the License (GPLv2+) This actually needs information from upstream since the license version are different in the DESCRIPTION file, the website and the sources. Either way it is an issue to resolve. I will upload the new version as soon as I have news from upstream concerning the license used. Thanks for the review Thank you for the quick responses. I will approve it when you figure out the license situation. -- Configure 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 470702] Review Request: L-function - L-function calculator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=470702 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Conrad Meyer kon...@tylerc.org 2009-03-20 17:49:23 EDT --- Thanks for the review. New Package CVS Request === Package Name: L-function Short Description: L-function calculator Owners: konradm 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 489728] Review Request: kcheckers - Checkers board 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=489728 --- Comment #1 from Alexey Torkhov atork...@gmail.com 2009-03-20 18:06:00 EDT --- Spec URL: http://atorkhov.fedorapeople.org/kcheckers.spec SRPM URL: http://atorkhov.fedorapeople.org/kcheckers-0.8.1-2.fc10.src.rpm * Fri Mar 20 2009 Alexey Torkhov atork...@gmail.com - 0.8.1-2 - Using find_lang to find qt translations - Use desktop-file-install -- Configure 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 456190] Review Request: dosemu - dos emulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=456190 --- Comment #39 from Justin Zygmont solarflo...@gmail.com 2009-03-20 18:15:35 EDT --- (In reply to comment #38) The -s and -t options are very different, so I'm not sure why you're suggesting I use one vs. the other: -s requires root, which I dont want to require -t tells dosemu not to open a window I'm not sure if it's a problem with slang or with dosemu, but dosemu is the only app I know of that uses slang and exhibits the problem. -s allows certain features that need to access the hardware (ie. direct video i/o, etc) so its ok without it, but its the only option most people would ever need. I think -t can be omitted though, the dosemu script now determines the mode, console size, etc. Just let me know if this works ok for you.. -- Configure 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 489803] Review Request: libserial - C++ library to access serial ports on POSIX systems
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489803 --- Comment #12 from Alex Orlandi nyr...@gmail.com 2009-03-20 18:20:30 EDT --- (In reply to comment #11) Well, we really don't want to patch in licensing info. Yes, of course. Did upstream say when they might do a new releas A new release of 0.6.0 is planned within April '09 (for now just 0.6.0_RC1 is available) Or perhaps you could use a snapshot if they have made changes in their version control system already? I checkout the svn repository but, as far as license statements is concerned, neither tags neither trunk have relevant differences: the only two files containing license statements are just SerailPort.cpp and SerialPort.h. I can try to ask upstream to add a license header to all the source files in the new release of the library 0.6.0. -- Configure 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 469569] Review Request: latex2rtf - LaTeX to RTF converter that handles equations, figures, and cross-references
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469569 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2009-03-20 18:22:42 EDT --- The source URL is correct now, the copying.txt file is included and the tests are run (and seem to complete successfully). Looks good to me. 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 456190] Review Request: dosemu - dos emulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=456190 --- Comment #40 from Derek Atkins warl...@mit.edu 2009-03-20 18:27:02 EDT --- The benefit of using -t is that it uses the current terminal instead of opening up a new window. This is especially useful for batch script processing and build systems where there might not be an X session in place. Yes, I do not see this problem when I remove the -t, but then dosemu opens up a new window which is not the behavior I want. I dont want a new window -- I want it to continue in the existing terminal where the dosemu command was run (in my case from a Makefile) -- Configure 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 490580] Review Request: faust - Compiled language for real-time audio signal processing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=490580 Conrad Meyer kon...@tylerc.org changed: What|Removed |Added CC||kon...@tylerc.org AssignedTo|nob...@fedoraproject.org|kon...@tylerc.org Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491429] New: Review Request: sslogger - A keystroke logging utility for privileged user escalation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: sslogger - A keystroke logging utility for privileged user escalation https://bugzilla.redhat.com/show_bug.cgi?id=491429 Summary: Review Request: sslogger - A keystroke logging utility for privileged user escalation Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: edbr...@brandint.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sourceforge.net/project/platformdownload.php?group_id=253689 SRPM URL: http://sourceforge.net/project/platformdownload.php?group_id=253689 Description: sslogger is session keystroke logging utility that allows permitted users to execute commands, or open interactive shells as the superuser or another user as specified by sudo rules in the sudoers file. When used in interactive session mode, sslogger records keystrokes in a spe- cial file. This file can later be reviewed by running a replay command. sslogger also provides the ability to run in command mode in which a command can be executed as another user. This allows sslogger to be used within a script in non-interactive script. This is my first package and I am seeking a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 490580] Review Request: faust - Compiled language for real-time audio signal processing
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=490580
--- Comment #1 from Conrad Meyer kon...@tylerc.org 2009-03-20 18:37:44 EDT
---
Created an attachment (id=336130)
-- (https://bugzilla.redhat.com/attachment.cgi?id=336130)
[Review]
Attached is my initial review.
Summary:
- rpmlint: only-non-binary-in-usr-lib should be explained or fixed
- should tools/%{name}2pd-* be %doc?
- is the doc subpackage not requiring the base package intentional?
--
Configure 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
