[Bug 551027] Review Request: nicotine+ - A client for the SoulSeek filesharing network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551027 Jussi Lehtola changed: What|Removed |Added CC||jussi.leht...@iki.fi --- Comment #14 from Jussi Lehtola 2010-01-25 02:28:22 EST --- - Nothing owns %{_datadir}/%{alt_name}/ so you need to add %dir {_datadir}/%{alt_name}/ - I personally dislike entries like %{_bindir}/* in spec files, since 1. you have no idea of what files are provided by the package. Please be more verbose and replace it with %{_bindir}/nicotine* -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543940] Review Request: libmpdclient - Library for interfacing Music Player Daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543940 --- Comment #5 from Adrian Reber 2010-01-25 02:19:41 EST --- Ping? Anything else? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558061] Review Request: levmar - Levenberg-Marquardt nonlinear least squares optimization
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558061 --- Comment #9 from Jussi Lehtola 2010-01-25 02:15:03 EST --- (In reply to comment #8) > "-P 0" selects patch 0; earlier I had multiple patches, but the others proved > to be unnecessary. I've removed it, and made the other changes you've > recommended. Here are the updated spec and SPRM: Oh, right. But you can also do it with just %patch0 You're losing the time stamp of README.txt in the conversion. Add the -k switch to the dos2unix command, or use the sed command in http://fedoraproject.org/wiki/Packaging_tricks#Remove_DOS_line_endings -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558061] Review Request: levmar - Levenberg-Marquardt nonlinear least squares optimization
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558061 --- Comment #10 from Jussi Lehtola 2010-01-25 02:18:05 EST --- I'll do the full review once you have made another submission and another informal review. If you're looking for something to package, you could find e.g. PRNG useful, which doesn't seem to be currently in Fedora. http://statmath.wu.ac.at/prng/ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 552898] Merge Review: perl-Authen-SASL - SASL Authentication framework for Perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=552898 Marcela Mašláňová 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553769] Review Request: euca2ools - Tools for Eucalyptus EC2-compatible Computation Service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553769 Garrett Holmstrom changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Garrett Holmstrom 2010-01-25 00:30:26 EST --- New Package CVS Request === Package Name: euca2ools Short Description: Elastic Utility Computing Architecture command line tools Owners: gholms Branches: F-12 F-13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558388] New: Review Request: perl-Net-Whois - Get and parse "whois" domain data from InterNIC
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-Net-Whois - Get and parse "whois" domain data from InterNIC https://bugzilla.redhat.com/show_bug.cgi?id=558388 Summary: Review Request: perl-Net-Whois - Get and parse "whois" domain data from InterNIC Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: colin@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://members.iinet.net.au/~coec/RPMS_for_review/perl-Net-Whois/perl-Net-Whois.spec SRPM URL: http://members.iinet.net.au/~coec/RPMS_for_review/perl-Net-Whois/perl-Net-Whois-1.9-2.el5.src.rpm Description: Net::Whois::Domain new() attempts to retrieve and parse the given domain's "whois" information from the InterNIC (whois.internic.net). If the server could not be contacted, is too busy, or otherwise does not process the query then the constructor does not return a reference and your object is undefined. If the constructor returns a reference, that reference can be used to access the various attributes of the domains' whois entry assuming that there was a match. The member function ok returns 1 if a match 0 if no match. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558389] New: Review Request: perl-Net-Whois-IP - Perl extension for looking up the whois information for ip addresses
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-Net-Whois-IP - Perl extension for looking up the whois information for ip addresses https://bugzilla.redhat.com/show_bug.cgi?id=558389 Summary: Review Request: perl-Net-Whois-IP - Perl extension for looking up the whois information for ip addresses Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: colin@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://members.iinet.net.au/~coec/RPMS_for_review/perl-Net-Whois-IP/perl-Net-Whois-IP.spec SRPM URL: http://members.iinet.net.au/~coec/RPMS_for_review/perl-Net-Whois-IP/perl-Net-Whois-IP-1.04-1.el5.src.rpm Description: Perl module to allow whois lookup of ip addresses. This module should recursively query the various whois providers until it gets the more detailed information including either TechPhone or OrgTechPhone by default; however, this is overrideable. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543154] Review Request: mingw32-SDL_mixer - Simple DirectMedia Layer's Sample Mixer 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=543154 Jason Woofenden changed: What|Removed |Added CC||jason...@herkamire.com --- Comment #4 from Jason Woofenden 2010-01-24 23:52:37 EST --- What is the status of getting this package into Fedora? The SDL game I hack on uses SDL_mixer, so this package would make cross-compiling easy. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543147] Review Request: mingw32-SDL_image - MinGW Windows port of the Image loading library for SDL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543147 Jason Woofenden changed: What|Removed |Added CC||jason...@herkamire.com --- Comment #4 from Jason Woofenden 2010-01-24 23:51:45 EST --- What is the status of getting this package into Fedora? I assume most programs that use SDL also use SDL_image. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558061] Review Request: levmar - Levenberg-Marquardt nonlinear least squares optimization
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558061 --- Comment #8 from Eric Smith 2010-01-24 23:40:20 EST --- Thanks for all of the advice, and the sponsorship offer. "-P 0" selects patch 0; earlier I had multiple patches, but the others proved to be unnecessary. I've removed it, and made the other changes you've recommended. Here are the updated spec and SPRM: Spec URL: http://www.brouhaha.com/~eric/software/fedora/f12/levmar/levmar.spec SRPM URL: http://www.brouhaha.com/~eric/software/fedora/f12/levmar/levmar-2.5-3.fc12.src.rpm Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1942076 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 486757] Review Request: divine-mc - Multi-core model checking system for proving specifications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486757 --- Comment #25 from David A. Wheeler 2010-01-24 22:24:08 EST --- Petr - thanks for reorganizing DiVinE so that the jars are separable. That makes things far more sensible. I've posted an RPM spec of "nips-promela-compiler" here: https://bugzilla.redhat.com/show_bug.cgi?id=558374 I need a reviewer for that - volunteers highly desired. The nips-promela-compiler package is a prerequisite for DiVinE (if you want to handle models in Promela, the language of SPIN, which I do). The idea is to get nips-promela-compiler into the Fedora repository, then update the DiVinE spec to match. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558374] New: Review Request: nips-promela-compiler - Compile Promela into NIPS VM for state-based model checking
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: nips-promela-compiler - Compile Promela into NIPS VM for state-based model checking https://bugzilla.redhat.com/show_bug.cgi?id=558374 Summary: Review Request: nips-promela-compiler - Compile Promela into NIPS VM for state-based model checking Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dwhee...@dwheeler.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.dwheeler.com/nips-promela-compiler.spec SRPM URL: http://www.dwheeler.com/nips-promela-compiler-1.2.5-2.fc12.src.rpm Description: The New Implementation of Promela Semantics (NIPS) compiler accepts models defined in the Promela modeling language (which defines state spaces) and translates them into the NIPS virtual machine (NIPS VM) format. The Promela language was originally defined for the SPIN tool. The installed compiler is named nips_c, not CodeGen ("CodeGen" is a generic name easily confused with other code generators). You can use nips_vm or DiVinE to analyze the compiled models. rpmlint reports 0 errors, 0 warnings, on a Fedora 12 x86_64: $ cd ~/rpmlint/SPECS $ rpmlint nips-promela-compiler.spec ../SRPMS/nips-promela-compiler-1.2.5-2.fc12.src.rpm ../RPMS/x86_64/nips-promela-compiler-1.2.5-2.fc12.x86_64.rpm 2 packages and 1 specfiles checked; 0 errors, 0 warnings. NOTE: This depends on sablecc, which isn't currently in the Fedora repository. I believe there's no license problem with it (LGPL), but I'm trying to get someone else to put that into Fedora. To test it for now, install the JPackage version. You can get that here: http://mirrors.dotsrc.org/jpackage/5.0/generic/free/SRPMS/sablecc-3.2-2.jpp5.src.rpm You can use mock to test its BuildRequires, but it's trickier to do this when there's a package not in the Fedora repository. Here's how I did it: mock -r fedora-12-x86_64 --init mock -r fedora-12-x86_64 --install java-gcj-compat-devel java-gcj-compat java-devel ant jpackage-utils dtdparser jdom junit java-1.6.0-openjdk mock -r fedora-12-x86_64 --copyin ~/rpmbuild/RPMS/noarch/sablecc-3.2-2.fc12.noarch.rpm /tmp mock -r fedora-12-x86_64 --copyin ~/rpmbuild/SRPMS/nips-promela-compiler-1.2.5-2.fc12.src.rpm /tmp mock -r fedora-12-x86_64 --shell # Now in the shell: cd rpm -i /tmp/sablecc* rpmbuild rpmbuild --rebuild /tmp/nips-promela* I'm packaging this because it's a prerequisite for "DiVinE"'s ability to read Promela (SPIN) files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556988] Review Request: ibus-fbterm - IBus front-end for fbterm.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556988 --- Comment #7 from Ding-Yi Chen 2010-01-24 20:40:53 EST --- Thanks for the pa(In reply to comment #5) > (In reply to comment #4) > > 1. Any reason you limit the BuildArch to i586 and x86_64? > > Just because "noarch" as value doesn't work. It's a C program, no doubt it won't work. :-) I've built it with koji under f-12 and f-13 without error. Maybe you can remove the BuildArch line. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557393] Review Request: ghc-haskell-src - Haskell source-code manipulation 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=557393 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Jens Petersen 2010-01-24 19:38:49 EST --- Thanks New Package CVS Request === Package Name: ghc-haskell-src Short Description: Haskell source-code manipulation library Owners: petersen Branches: devel InitialCC: haskell-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557392] Review Request: ghc-parallel - Haskell parallel programming 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=557392 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Jens Petersen 2010-01-24 19:36:55 EST --- Thank you New Package CVS Request === Package Name: ghc-parallel Short Description: Haskell parallel programming library Owners: petersen Branches: devel InitialCC: haskell-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553705] Review Request: rinputd - A server for receiving input events over the network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553705 --- Comment #13 from Gianluca Sforna 2010-01-24 19:36:01 EST --- (In reply to comment #12) > First, thanks for reviewing this. As rinputd logged, it is trying to broadcast > using multicast dns. I am not sure exactly why it's not working for you, but > it's most likely a network configration issue. I'm using the standard Avahi > library for mDNS, so I doubt it's an issue with rinputd. You can also specify > the ip address in Remotux instead of using mDNS. Also tried to enter the IP, no luck. There should be something really wrong in my setup, I will try to diagnose the issue and let you know. Jarod, how are you using this package right now? > > As for rinputd running after ctrl-c, it runs as a daemon by default now. This > means it forks to the background when it starts, relinquishing foreground > control back to the shell. I'm guessing it went back to the shell when you ran > it, and you hit ctrl-c, which is like a noop in the shell. You can run rinputd > in the foreground using the -f switch if that's what you prefer. Ah, you are right. Maybe --verbose should imply -f but anyway, it was a clear PEBCAK case :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557391] Review Request: ghc-QuickCheck - Automatic testing of Haskell programs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557391 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Jens Petersen 2010-01-24 19:35:11 EST --- Thanks! New Package CVS Request === Package Name: ghc-QuickCheck Short Description: Automatic testing of Haskell programs Owners: petersen Branches: devel InitialCC: haskell-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557390] Review Request: ghc-stm - Haskell Software Transactional Memory 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=557390 Jens Petersen changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Jens Petersen 2010-01-24 19:32:24 EST --- New Package CVS Request === Package Name: ghc-stm Short Description: Haskell Software Transactional Memory library Owners: petersen Branches: devel InitialCC: haskell-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557390] Review Request: ghc-stm - Haskell Software Transactional Memory 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=557390 --- Comment #5 from Jens Petersen 2010-01-24 19:29:28 EST --- (In reply to comment #3) > urlgrabber.grabber.URLGrabError: [Errno 14] HTTP Error 404 : > http://petersen.fedorapeople.org/ghc-stm/ghc-stm-2.1.1.2-2.fc12.src.rpm (Sorry had forgotten the srpm into place: anyway the differences are purely cosmetic) Thanks for reviewing. :-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553705] Review Request: rinputd - A server for receiving input events over the network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553705 --- Comment #12 from Chase Douglas 2010-01-24 19:00:38 EST --- Gianluca, First, thanks for reviewing this. As rinputd logged, it is trying to broadcast using multicast dns. I am not sure exactly why it's not working for you, but it's most likely a network configration issue. I'm using the standard Avahi library for mDNS, so I doubt it's an issue with rinputd. You can also specify the ip address in Remotux instead of using mDNS. As for rinputd running after ctrl-c, it runs as a daemon by default now. This means it forks to the background when it starts, relinquishing foreground control back to the shell. I'm guessing it went back to the shell when you ran it, and you hit ctrl-c, which is like a noop in the shell. You can run rinputd in the foreground using the -f switch if that's what you prefer. Thanks, Chase -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553705] Review Request: rinputd - A server for receiving input events over the network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553705 --- Comment #11 from Gianluca Sforna 2010-01-24 18:30:37 EST --- Additionally (not sure this is a bug) hitting CTRL-C gives back the shell but does not actually terminate the daemon -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553705] Review Request: rinputd - A server for receiving input events over the network
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553705 --- Comment #10 from Gianluca Sforna 2010-01-24 18:17:53 EST --- I can review this if Stefan is too busy. BTW, I'm not able to use the app in F-12 with Remotux: even if I disabled the firewall it seems I'm not able to discover my laptop. rinputd --verbose does not show anything other than Listening on port 8771 Avahi service _rinput._tcp with name 'bingo Remote Input' established any idea? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 521716] Review Request: python-zfec - A fast erasure codec with python bindings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=521716 --- Comment #2 from Ruben Kerkhof 2010-01-24 18:09:47 EST --- Hi Thomas, thanks for the review! Upstream released 1.4.6, which includes a few fixes for the issues you mentioned. I asked them to push the new version to pypi, when that's done I'll create a new version. Should be ready in a few days. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551913] Review Request: monodevelop-vala - A vala plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551913 --- Comment #5 from Paul F. Johnson 2010-01-24 17:38:25 EST --- SRC : http://www.all-the-johnsons.co.uk/fedora/monodevelop-vala-2.2-3.fc13.src.rpm SPEC : http://www.all-the-johnsons.co.uk/fedora/monodevelop-vala.spec Fixes the above -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557957] Review Request: perl-Text-vCard - CPAN module for handling vCard files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557957 --- Comment #2 from Philip Prindeville 2010-01-24 17:42:44 EST --- SRPM URL: http://philipp.fedorapeople.org/perl-Text-vCard-2.06-1.fc11.src.rpm Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1941707 $ rpmlint ~/rpmbuild/SRPMS/perl-Text-vCard-2.06-1.fc12.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint ~/rpmbuild/SPECS/perl-Text-vCard.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551915] Review Request: monodevelop-debugger-gdb - A gdb debugger addin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551915 --- Comment #2 from Paul F. Johnson 2010-01-24 17:36:56 EST --- SRPM URL: http://www.all-the-johnsons.co.uk/fedora/monodevelop-debugger-gdb-2.2-2.fc13.src.rpm SPEC : http://www.all-the-johnsons.co.uk/fedora/monodevelop-debugger-gdb.spec Fixes above issues -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551914] Review Request: monodevelop-database - A database plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551914 --- Comment #2 from Paul F. Johnson 2010-01-24 17:35:49 EST --- SRPM URL: http://www.all-the-johnsons.co.uk/fedora/monodevelop-database-2.2-1.fc13.src.rpm SPEC : http://www.all-the-johnsons.co.uk/fedora/monodevelop-database.spec Fixes above issues -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551912] Review Request: monodevelop-java - A java plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551912 --- Comment #4 from Paul F. Johnson 2010-01-24 17:37:51 EST --- SRPM URL: http://www.all-the-johnsons.co.uk/fedora/monodevelop-java-2.2-2.fc13.src.rpm SPEC : http://www.all-the-johnsons.co.uk/fedora/monodevelop-java.spec Fixes above issues -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551911] Review Request: monodevelop-boo - A boo plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551911 --- Comment #2 from Paul F. Johnson 2010-01-24 17:34:35 EST --- SRPM URL: http://www.all-the-johnsons.co.uk/fedora/monodevelop-boo-2.2-1.fc13.src.rpm SPEC http://www.all-the-johnsons.co.uk/fedora/monodevelop-boo.spec Fixes above issues -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556628] Review Request: pymilia - Python wrappers for milia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556628 Thomas Spura changed: What|Removed |Added Summary|Review Request: pymilia - |Review Request: pymilia - |python wrappers for milia |Python wrappers for milia -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 518318] Review Request: vanessa_socket - Simplify TCP/IP socket operations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=518318 --- Comment #10 from Pavel Alexeev (aka Pahan-Hubbitus) 2010-01-24 17:20:34 EST --- (In reply to comment #7) > Here are some more comments that should be useful: > > - libvanessa_socket/vanessa_socket_daemon.c is GPLv2+, not LGPLv2+, so I'm > fairly certain the license field needs to change. You are right. > - You actually do need to run autotools because the stock configure script > adds > an RPATH: > vanessa_socket.x86_64: E: binary-or-shlib-defines-rpath > /usr/bin/vanessa_gethostbyname ['/usr/lib64'] > vanessa_socket-pipe.x86_64: E: binary-or-shlib-defines-rpath > /usr/bin/vanessa_socket_pipe ['/usr/lib64'] Hm, it is strange, because is not appeared in i586 architecture! Thank you. Joshua Roys, also thanks, I follow this guidelines to cut off it. > - The only other complaint rpmlint has relates to a lack of %doc files in the > devel subpackage; I'm not sure how important that is here. > vanessa_socket-devel.x86_64: W: no-documentation It is not important in -devel sub-package. (In reply to comment #9) > > Requires: vanessa_logger >= 0.0.8 > > https://fedoraproject.org/wiki/Packaging:Guidelines#Explicit_Requires > > > > %packagedevel > > ... > > Requires: vanessa_logger-devel >= 0.0.5 > > Questionable version here. Above it's 0.0.8, in the BuildRequires it's 0.0.8, > and here it's 0.0.5. Fixed, thanks. > > > Group: Development/Libraries > > Run-time libraries still belong into group "System Environment/Libraries". Ok, changed. Please, can you provide link where I can read about it more? > > > vanessa_socket_daemon.c > This file is in need of a licence clarification by its author. It's likely he > just forgot to replace the GPLv2+ header, but as long as that one is present, > the following guideline applies: > > https://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Mixed_Source_Licensing_Scenario I ask Simon Horman, wait answer. > > mkdir -p %{buildroot}/{etc,%{prefix}/{lib,bin,doc}} > > This doesn't look right. If creating %prefix/lib is really needed, the build > will fail on 64-bit where %prefix/lib64 would not be created. And %prefix/doc > isn't used by the %files section. It was built on x86_64 successfully. But you are right, it is ambiguous line at all. > > > > %doc README > > The file is duplicated in two packages. In the base library package the file > contents refers to "vanessa_socket_pipe", which is not included in that > package. Nothing explains where to find the missing vanessa_socket_pipe. And > in the separate vanessa_socket-pipe package, the same README file refers to > the > library. => Splitting off the vanessa_socket_pipe program is highly > questionable. Especially, since the base library package contains another > program in /usr/bin already, so nothing is won by splitting off one tool. This splitting and README duplication comes from history based author spec. READMY contains generic information about author, licensing and both subpackages. I think there no error include it in both. http://koji.fedoraproject.org/koji/taskinfo?taskID=1941649 http://hubbitus.net.ru/rpm/Fedora11/vanessa_socket/vanessa_socket-0.0.10-7.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556628] Review Request: pymilia - python wrappers for milia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556628 Sergio Pascual changed: What|Removed |Added Summary|Review Request: Pymilia - |Review Request: pymilia - |Python wrappers for milia |python wrappers for milia --- Comment #12 from Sergio Pascual 2010-01-24 17:07:40 EST --- Lower case is correct, fixed in the summary -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556628] Review Request: Pymilia - Python wrappers for milia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556628 Bug 556628 depends on bug 556625, which changed state. Bug 556625 Summary: Review Request: milia - C++ cosmology library https://bugzilla.redhat.com/show_bug.cgi?id=556625 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556625] Review Request: milia - C++ cosmology 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=556625 Sergio Pascual changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 521166] Review Request: OpenGTL - Graphics Transformation Languages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=521166 --- Comment #22 from Rex Dieter 2010-01-24 16:33:44 EST --- SPEC: http://rdieter.fedorapeople.org/rpms/OpenGTL/OpenGTL.spec %changelog * Sun Jan 24 2010 Rex Dieter - 0.9.12-3 - omit html/installdox -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #16 from Christoph Wickert 2010-01-24 16:31:19 EST --- Although Simons comments sound very uncouth to me, I have to defend him on the merits: (In reply to comment #12) > Simon, your statement "first come, first served" explicitly contradicts our > existing guidenline and policy on the matter. Please read the relevant > guidelines: IMHO Simon has acted IAW the guidelines. Quote: "In the second case, where there is no known package to conflict with at this time, it is up to the packager to make a decision." By the time of the review or when pushing the package to updates, nobody was aware of the other program, so it is Simon's decision. Where we go from here now depends on how constructive the suggestions are. Jason, what do you suggest? Simon, can you talk to upstream and ask him about renaming his package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 521166] Review Request: OpenGTL - Graphics Transformation Languages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=521166 --- Comment #21 from Rex Dieter 2010-01-24 16:26:47 EST --- Looks like installdox doesn't serve much purpose packaging-wise, I'll omit 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 --- Comment #10 from Fedora Update System 2010-01-24 15:54:03 EST --- php-channel-pdepend-1.3-4.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/php-channel-pdepend-1.3-4.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557776] Review Request: libyubikey - Library for parsing / decrypting Yubikey passwords
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557776 --- Comment #4 from Maxim Burgerhout 2010-01-24 15:51:05 EST --- Mamoru, thank you for looking into this package. Timestamps: done. Compilerflags: reverted. Defattr: done. Changelog: done. File placement: reverted. Updated files: - http://dl.dropbox.com/u/284093/update-2/libyubikey.spec - http://dl.dropbox.com/u/284093/update-2/libyubikey-1.5-4.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 --- Comment #9 from Fedora Update System 2010-01-24 15:20:47 EST --- php-channel-phpmd-1.3-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/php-channel-phpmd-1.3-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 506476] Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506476 Jason Tibbitts changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Jason Tibbitts 2010-01-24 14:58:26 EST --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551913] Review Request: monodevelop-vala - A vala plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551913 Paul Lange changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|pala...@gmx.de Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556128] Review Request: ff-utils - Utilities to test force feedback of input device
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556128 --- Comment #5 from Michal Ambroz 2010-01-24 14:38:35 EST --- Spec URL: http://rebus.webz.cz/d/ff-utils.spec SRPM URL: http://rebus.webz.cz/d/ff-utils-2.4.21-2.fc12.src.rpm based on review from Tim (thank you Tim!) I have added the build requirement for the SDL-devel package to be present during the build. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 550143] Review Request: webattery - Command line tool to display battery status
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=550143 --- Comment #12 from Alagunambi Welkin 2010-01-24 14:36:02 EST --- (In reply to comment #11) Thanks for your valuable comment Here is the new spec file and SRPM SPEC URL: http://downloads.sourceforge.net/webattery/webattery-spec-1.2-6 SRPM URL: http://downloads.sourceforge.net/webattery/webattery-1.2-6.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544660] Review Request: php-channel-swift - Adds swift mailer project channel to PEAR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=544660 --- Comment #12 from Fedora Update System 2010-01-24 14:32:56 EST --- php-channel-swift-1.3-2.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/php-channel-swift-1.3-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #15 from Dominik 'Rathann' Mierzejewski 2010-01-24 14:20:54 EST --- Some more information about surf: it was written in 1992 (so it's way older than surf-the-web-browser) and while it is open source, the licence has a non-commercial clause, so it won't end up in Fedora unless the authors relicense it. However, this doesn't stop anyone from packaging it for rpmfusion or some other repository. Also surf is used by VMD, which is a popular and actively developed molecular dynamics software. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556628] Review Request: Pymilia - Python wrappers for milia
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=556628 --- Comment #11 from Jason Tibbitts 2010-01-24 14:21:53 EST --- The requested package name (pymilia) doesn't match the package name in the bug summary (Pymilia). Please verify that the lower case version is correct, and edit the bug summary if so. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557776] Review Request: libyubikey - Library for parsing / decrypting Yubikey passwords
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557776 Mamoru Tasaka changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp --- Comment #3 from Mamoru Tasaka 2010-01-24 14:15:18 EST --- Some notes for 1.5-3 * Compilar flags - "CFLAGS="$RPM_OPT_FLAGS"" in "make %{?_smp_mflags}" is not needed. configure generated by recent autotools acknowledges CFLAGS environment and %configure sets this. Please check what %configure actually does by $ rpm --eval %configure . * Timestamp - Please consider to use %{__make} install DESTDIR=$RPM_BUILD_ROOT INSTALL="%{__install} -p" to keep timestamps on installed files. This method usually works for Makefiles generated by recent autotools. * %defattr - Now we prefer to use %defattr(-,root,root,-) * File placement - As the Summary of the main package already says "low-level C software *development* kit", I don't see strict necessity for putting two binaries into -devel package. - Also "README" should usually be packaged in main package. * %changelog - It is recommended to put one line between each %changelog entry like below because it is useful on Fedora CVS - * Sun Jan 24 2010 - Maxim Burgerhout - 1.5-3 - Took out the dep on libusb1-devel - Moved README doc to -devel: it's mostly about ykdebug and modhex * Sun Jan 24 2010 - Maxim Burgerhout - 1.5-2 - Used macros in Source0 - URL no longer point to redirect - Removed INSTALL from documentation - Moved modhex and ykdebug to -devel - Inserted compilerflags - Inserted INSTALLFLAGS to keep timestamps - Some more macros for make and sed * Wed Jan 20 2010 - Maxim Burgerhout - 1.5-1 - First packaged release - -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551913] Review Request: monodevelop-vala - A vala plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551913 Christian Krause changed: What|Removed |Added CC||c...@plauener.de --- Comment #4 from Christian Krause 2010-01-24 13:12:17 EST --- Just one remark: - URL does not match the link on the download page and so the sources differ from upstream - please use: http://ftp.novell.com/pub/mono/sources/monodevelop-vala/monodevelop-vala-2.2.tar.bz2 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551912] Review Request: monodevelop-java - A java plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551912 Christian Krause changed: What|Removed |Added Status|NEW |ASSIGNED CC||c...@plauener.de AssignedTo|nob...@fedoraproject.org|c...@plauener.de Flag||fedora-review? --- Comment #3 from Christian Krause 2010-01-24 13:06:43 EST --- I've roughly scanned over the package and I've seen a couple of issues. It would be good if they could be fixed first before I do the full complete review: - source files differ from upstream and the source URL does not match the link on the download page: http://ftp.novell.com/pub/mono/sources/monodevelop-java/monodevelop-java-2.2.tar.bz2 - License: it looks like that the source files and the COPYING file points to GPLv2+ instead of MIT - please can you check this? - package does not build (x86_64): Looking for required packages Checking for package 'mono-addins'.. found. Checking for package 'gtk-sharp-2.0'.. ERROR: Package named 'gtk-sharp-2.0' >= 2.12.8 not found. Try adjusting your PKG_CONFIG_PATH environment variable. error: Bad exit status from /var/tmp/rpm-tmp.d20ndr (%build) RPM build errors: - directory %{_libdir}/monodevelop/AddIns/JavaBinding must be packaged -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551915] Review Request: monodevelop-debugger-gdb - A gdb debugger addin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551915 Christian Krause changed: What|Removed |Added Status|NEW |ASSIGNED CC||c...@plauener.de AssignedTo|nob...@fedoraproject.org|c...@plauener.de Flag||fedora-review? --- Comment #1 from Christian Krause 2010-01-24 12:53:52 EST --- I've roughly scanned over the package and I've seen a couple of issues. It would be good if they could be fixed first before I do the full complete review: - source files differ from upstream and the source URL does not match the link on the download page: http://ftp.novell.com/pub/mono/sources/monodevelop-debugger-mdb/monodevelop-debugger-mdb-2.2.tar.bz2 - package does not build (x86_64): Processing files: monodevelop-debugger-gdb-2.2-1.fc13.x86_64 error: File not found by glob: /builddir/build/BUILDROOT/monodevelop-debugger-gdb-2.2-1.fc13.x86_64/usr/lib64/monodevelop/AddIns/MonoDevelop.Debugger/MonoDevelop.Debugger.Gdb* RPM build errors: File not found by glob: /builddir/build/BUILDROOT/monodevelop-debugger-gdb-2.2-1.fc13.x86_64/usr/lib64/monodevelop/AddIns/MonoDevelop.Debugger/MonoDevelop.Debugger.Gdb* -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 504476] Review Request: rubygem-newgem - Bundle Ruby libraries into a RubyGem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504476 Mamoru Tasaka changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp Flag||needinfo?(jan.kle...@hp.com ||) --- Comment #13 from Mamoru Tasaka 2010-01-24 12:55:56 EST --- Jan, ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 549624] Review Request: rubygem-merb-slices - Merb plugin for using and creating application 'slices'
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=549624 Mamoru Tasaka changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp Flag||fedora-review+ Bug 549624 depends on bug 549286, which changed state. Bug 549286 Summary: Review Request: rubygem-merb-core - Lightweight Ruby-based MVC framework for web development https://bugzilla.redhat.com/show_bug.cgi?id=549286 What|Old Value |New Value Status|NEW |ASSIGNED Resolution||ERRATA Status|ASSIGNED|CLOSED --- Comment #1 from Mamoru Tasaka 2010-01-24 12:50:43 EST --- I actually reviewed this ticket but could not find any blockers. - This package (rubygem-merb-slices) is APPROVED by mtasaka - -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557021] Review Request: rubygem-merb-gen - Application and plugin generator scripts for Merb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557021 Mamoru Tasaka changed: What|Removed |Added Depends on||551817 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551817] Review Request: rubygem-templater - Ruby framework for building code generators
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551817 Mamoru Tasaka changed: What|Removed |Added Blocks||557021 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551914] Review Request: monodevelop-database - A database plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551914 Christian Krause changed: What|Removed |Added Status|NEW |ASSIGNED CC||c...@plauener.de AssignedTo|nob...@fedoraproject.org|c...@plauener.de Flag||fedora-review? --- Comment #1 from Christian Krause 2010-01-24 12:46:07 EST --- Please can you always add a link to the spec file as well? Thanks! I've roughly scanned over the package and I've seen a couple of issues. It would be good if they could be fixed first before I do the full complete review: - source files differ from upstream: sources in package: 430571024f88c87d5104121739c6ab7e SOURCES/monodevelop-database-2.2.tar.bz2 sources from upstream (using the URL which was supplied in the spec file) 5bb1b77221066d57ff3ec58435a24bc3 monodevelop-database-2.2.tar.bz2 - the source URL does not match the link on the download page: http://ftp.novell.com/pub/mono/sources/monodevelop-database/monodevelop-database-2.2.tar.bz2 (this should fix the previous issue as well) - package does not build in koji - probably a missing build requirement: checking for GLIB_SHARP... configure: error: Package requirements (glib-sharp-2.0 >= 2.12.8) were not met: No package 'glib-sharp-2.0' found Consider adjusting the PKG_CONFIG_PATH environment variable if you installed software in a non-standard prefix. Alternatively, you may set the environment variables GLIB_SHARP_CFLAGS and GLIB_SHARP_LIBS to avoid the need to call pkg-config. See the pkg-config man page for more details. error: Bad exit status from /var/tmp/rpm-tmp.Z9lbEA (%build) Bad exit status from /var/tmp/rpm-tmp.Z9lbEA (%build) - some rpmlint warnings: SPECS/monodevelop-database.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 4, tab: line 14) - directory %{_libdir}/monodevelop/AddIns/MonoDevelop.Database should be included in the package -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551911] Review Request: monodevelop-boo - A boo plugin for monodevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551911 Christian Krause changed: What|Removed |Added Status|NEW |ASSIGNED CC||c...@plauener.de AssignedTo|nob...@fedoraproject.org|c...@plauener.de Flag||fedora-review? --- Comment #1 from Christian Krause 2010-01-24 12:34:43 EST --- I've roughly scanned over the package and I've seen a couple of issues. It would be good if they could be fixed first before I do the full complete review: - source files differ from upstream: sources in package: 25abd742dc4a0ffcf17c537dea6d791f SOURCES/monodevelop-boo-2.2.tar.bz2 sources from upstream: 56bf610e072030274309c94de1079b8e monodevelop-boo-2.2.tar.bz2 - the source URL does not match the link on the download page: http://ftp.novell.com/pub/mono/sources/monodevelop-boo/monodevelop-boo-2.2.tar.bz2 - some rpmlint warnings: SPECS/monodevelop-boo.spec:41: W: configure-without-libdir-spec SPECS/monodevelop-boo.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 4, tab: line 14) monodevelop-boo-devel.i686: E: description-line-too-long The monodevelop-boo-devel package contains development files for monodevelop-boo. - directory %{_libdir}/monodevelop/AddIns/BooBinding should be included in this package: e.g. use: %files -f %{name}.lang %defattr(-,root,root,-) %{_libdir}/monodevelop/AddIns/BooBinding - License: It looks like that the License should be rather GPLv2+ instead of MIT - please can you have 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557871] Review Request: python-mygpoclient - Python module to connect to the my.gpodder.org webservice
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557871 --- Comment #4 from Jason Tibbitts 2010-01-24 12:35:41 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 555986] Review Request: php-phpunit-PHPUnit - Regression testing framework for unit tests
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=555986 --- Comment #4 from Jason Tibbitts 2010-01-24 12:33:23 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 556625] Review Request: milia - C++ cosmology 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=556625 --- Comment #9 from Jason Tibbitts 2010-01-24 12:33:48 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 542313] Review Request: QTeXEngine - Library enabling Qt based applications to easily export graphics to TeX
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=542313 --- Comment #9 from Jason Tibbitts 2010-01-24 12:31:11 EST --- CVS done (by process-cvs-requests.py). Just a question: where did you find a CVS template that still had the "cvsextras commits" bit in it? That hasn't been used in some time, and if we have documentation out there that still tells you to use that then it needs to be changed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=537587 --- Comment #80 from Jason Tibbitts 2010-01-24 12:28:15 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 --- Comment #9 from Jason Tibbitts 2010-01-24 12:32:10 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544660] Review Request: php-channel-swift - Adds swift mailer project channel to PEAR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=544660 --- Comment #11 from Jason Tibbitts 2010-01-24 12:31:56 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 --- Comment #8 from Jason Tibbitts 2010-01-24 12:32:20 EST --- CVS done (by process-cvs-requests.py). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #14 from Jason Tibbitts 2010-01-24 12:25:43 EST --- Then you have some work to do to rectify this problem. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 550067] Review Request: perl-Template-Tiny - Template Toolkit reimplemented in as little code as possible
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=550067 Mamoru Tasaka changed: What|Removed |Added Flag|fedora-review? | --- Comment #1 from Mamoru Tasaka 2010-01-24 12:14:19 EST --- (The submitter must not set fedora-review flag. This is to be set by the reviewer.) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544531] Review Request: xvkbd - Virtual Keyboard for X Window 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=544531 --- Comment #12 from Mamoru Tasaka 2010-01-24 11:59:23 EST --- (In reply to comment #11) > I reviewed bug 557776. Well, okay. --- This package (xvkbd) is APPROVED by mtasaka --- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from "Get a Fedora Account". After you request for sponsorship a mail will be sent to sponsor members automatically (which is invisible for you) which notifies that you need a sponsor. After that, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 10/11, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544531] Review Request: xvkbd - Virtual Keyboard for X Window 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=544531 Mamoru Tasaka changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557948] Review Request: PyAIML - A Python AIML Interpreter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557948 --- Comment #4 from Sebastian Dziallas 2010-01-24 11:59:27 EST --- D'oh! You're right, indeed. I've pinged the upstream author to see if we've any chances there. Thanks for the review already. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554090] Review Request: sugar-physics - A physical world simulator and playground for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554090 --- Comment #2 from Sebastian Dziallas 2010-01-24 11:46:27 EST --- Thanks for taking on this one! :) (In reply to comment #1) > I'm not yet sponsored as a packager, but here's an unofficial review: > > The Require: tag is no longer necessary, or allowed. Well, since this is a Python package we still need to use the Requires tag, as the dependencies don't get added here automagically. > rpmlint output: > sugar-physics.spec:6: W: non-standard-group Sugar/Activities > sugar-physics.noarch: W: non-standard-group Sugar/Activities > 1 packages and 1 specfiles checked; 0 errors, 2 warnings. > > I'm fairly sure I remember seeing a requirement that the Group: tag come from > the official list in /usr/share/doc/rpm-*/GROUPS, but I can't find that in the > packaging guidellines now, and the Sugar packaging guidelines use > Sugar/Activities just as you have, so I guess that's allowed. Yes, this is common practice among the Sugar Activity packages and in the guidelines here: http://fedoraproject.org/wiki/Packaging:SugarActivityGuidelines#Sample_SPEC > source tarball verified against upstream > compiles and builds on F12 x86_64 and i386 using mock > > I attempted to verify minimal functionality using sugar-emulator. When I > start > the physics activity, I get a pulsating logo, but nothing else seems to happen > for quite a while, then it dumps me back to the activity list. It's entirely > possible that I'm doing something wrong. Oh yeah, I see. Looks like I'm missing another requirement (I had it already installed when giving it some initial testing). After installing python-olpcgames, you should be good to go. I'll add it to the next release bump. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #13 from Simon Wesp 2010-01-24 11:24:44 EST --- (In reply to comment #12) > I urge you to avoid pushing this package until the conflict can be worked > out. Too late. $ yum info surf Available Packages Name : surf Arch : x86_64 Version: 0.3 Release: 2.fc12 Size : 16 k Repo : updates Summary: Simple web browser URL: http://surf.suckless.org/ License: MIT Description: surf is a simple web browser based on WebKit/GTK+. http://admin.fedoraproject.org/updates/F12/FEDORA-2010-0785 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 527986] Review Request: taoframework - Multimedia bindings for Mono
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=527986 --- Comment #15 from Christian Krause 2010-01-24 11:21:33 EST --- (In reply to comment #14) > Sorry it took so long to update. I have to apologize for my long response time as well... > Patched the Makefile and commented the unwanted parts in the specfile out. I've checked the new package and everything looks fine now besides one small issue: In file /usr/lib/mono/gac/Tao.Glfw/2.6.0.0__2bb092b6587e4402/Tao.Glfw.dll.config there is a line: The 2 double quotes one after the other seem to be wrong. Sure, it looks like that it is upstream broken as well, but we should fix it anyway. Once this is fixed, the package can be approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 Christof Damian changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Christof Damian 2010-01-24 11:10:08 EST --- New Package CVS Request === Package Name: php-channel-pdepend Short Description: PHP Depend PEAR channel Owners: cdamian Branches: F-11 F-12 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #12 from Jason Tibbitts 2010-01-24 10:59:36 EST --- Simon, your statement "first come, first served" explicitly contradicts our existing guidenline and policy on the matter. Please read the relevant guidelines: http://fedoraproject.org/wiki/Packaging:Conflicts#Potential_Conflicting_Files "We don't just try to avoid conflicts with existing packages within Fedora but also potential conflicts. This is because the first package to enter Fedora is not always the one that should take on the name. There are several scenarios in which this could come into play:" I urge you to avoid pushing this package until the conflict can be worked out. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 Remi Collet changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #7 from Remi Collet 2010-01-24 10:53:32 EST --- REVIEW: ~ no source files (channel.xml ok, but see notice) * package meets naming * package must meet packaging guidelines * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. * license field matches the actual license (of the packages provided by the channel). * license is open source-compatible (BSD). * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide). * package installs properly * channel (un)registered properly * rpmlint (warnings are ok) : php-channel-pdepend.src: I: checking php-channel-pdepend.noarch: I: checking php-channel-pdepend.noarch: W: no-documentation 2 packages and 1 specfiles checked; 0 errors, 1 warnings. * final provides are sane: php-channel(pear.pdepend.org) = 1.3 php-channel-pdepend = 1.3-3.fc8 * final requires /usr/bin/pear php-pear(PEAR) * %check is not present; no test suite provide. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (pear channel..) * no documentation * %install start with rm * %clean ok Just a minor diff between channel.xml in src.rpm and upstream one (double / in URL) => you could update it during CVS import. * APPROVED * -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 Christof Damian changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Christof Damian 2010-01-24 10:53:19 EST --- New Package CVS Request === Package Name: php-channel-phpmd Short Description: PHP Mess Detector PEAR channel Owners: cdamian Branches: F-11 F-12 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 554101] Review Request: surf - Simple web browser
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554101 --- Comment #11 from Simon Wesp 2010-01-24 10:44:18 EST --- (In reply to comment #10) > but it's still a (potential) conflict. I don't see a conflict! surf is unique in fedora pkgdb there is no review request with another software called surf! first come, first served! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 Remi Collet changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Remi Collet 2010-01-24 10:43:09 EST --- REVIEW * no source files (channel.xml match latest upstream) * package meets naming * package must meet packaging guidelines * specfile is properly named, is cleanly written and uses macros consistently. * dist tag is present. * build root is correct. * license field matches the actual license (of the packages provided by the channel). * license is open source-compatible (BSD). * BuildRequires are proper. * %clean is present. * package builds in mock (rawhide). * package installs properly * channel (un)registered properly * rpmlint (warnings are ok) : php-channel-phpmd.src: I: checking php-channel-phpmd.noarch: I: checking php-channel-phpmd.noarch: W: no-documentation 2 packages and 1 specfiles checked; 0 errors, 1 warnings. * final provides are sane: php-channel(pear.phpmd.org) = 1.3 php-channel-phpmd = 1.3-3.fc8 + final requires /usr/bin/pear php-pear(PEAR) * %check is not present; no test suite provide. * doesn't own any directories it shouldn't. * no duplicates in %files. * file permissions are appropriate. * scriptlets are OK (pear channel..) * no documentation * %install start with rm * %clean ok * APPROVED * -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 547993] Package Name Change: jconvolver - Real-time Convolution Engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=547993 Mamoru Tasaka changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #17 from Mamoru Tasaka 2010-01-24 09:27:24 EST --- Closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557776] Review Request: libyubikey - Library for parsing / decrypting Yubikey passwords
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=557776 --- Comment #2 from Maxim Burgerhout 2010-01-24 09:08:41 EST --- Thanks for looking at this, Akio. I went over your notes: 1. Correct, but as modhex and ykdebug went to -devel, so did README (which is about those two files for the largest part). So -devel now has a %doc section. 2. Fixed. 3. Looked into this. I redid the download so it's timestamp matches upstream. However, the Makefile doesn't seem to handle INSTALLFLAGS or pass it to install-sh, but the timestamps of installed files are intact, afaics. I don't think the install part needs fixing. Please correct me if I'm wrong; I really want to learn this. 4. Fixed. 5. Moved them to -devel. I agree that is a better place for those binaries. 6. Fixed. Missed that. New versions at: http://dl.dropbox.com/u/284093/update-1/libyubikey.spec http://dl.dropbox.com/u/284093/update-1/libyubikey-1.5-3.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 540127] Review Request: qterm - BBS client designed for BBS browsing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=540127 Chen Lei changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544660] Review Request: php-channel-swift - Adds swift mailer project channel to PEAR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=544660 Christof Damian changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Christof Damian 2010-01-24 08:41:15 EST --- New Package CVS Request === Package Name: php-channel-swift Short Description: Adds swift mailer project channel to PEAR Owners: cdamian Branches: F-11 F-12 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 --- Comment #5 from Christof Damian 2010-01-24 08:35:09 EST --- Spec URL: http://rpms.damian.net/SPECS/php-channel-phpmd.spec SRPM URL: http://rpms.damian.net/SRPMS/php-channel-phpmd-1.3-3.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 --- Comment #6 from Christof Damian 2010-01-24 08:34:40 EST --- This is embarrassing :-) Thank you for your patience Spec URL: http://rpms.damian.net/SPECS/php-channel-pdepend.spec SRPM URL: http://rpms.damian.net/SRPMS/php-channel-pdepend-1.3-3.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558228] New: Review Request: perl-Tk-DirSelect - Cross-platform directory selection widget
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-Tk-DirSelect - Cross-platform directory selection widget https://bugzilla.redhat.com/show_bug.cgi?id=558228 Summary: Review Request: perl-Tk-DirSelect - Cross-platform directory selection widget Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: david.hanneq...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://hvad.cfppa-cibeins.com/perl-modules/perl-Tk-DirSelect/perl-Tk-DirSelect.spec SRPM URL: http://hvad.cfppa-cibeins.com/perl-modules/perl-Tk-DirSelect/perl-Tk-DirSelect-1.11-1.fc10.src.rpm Description: This module provides a cross-platform directory selection widget. For systems running Microsoft Windows, this includes selection of local and mapped network drives. A context menu (right-click or ) allows the creation, renaming, and deletion of directories while browsing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543248] Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=543248 Ben Boeckel changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #9 from Ben Boeckel 2010-01-24 07:46:42 EST --- Imported and built. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 --- Comment #5 from Remi Collet 2010-01-24 07:31:12 EST --- in %post: seems %{name}.xml still not used... There some days like this ;) + -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 --- Comment #4 from Remi Collet 2010-01-24 07:29:58 EST --- in %post: missing .xml after %{name} -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544660] Review Request: php-channel-swift - Adds swift mailer project channel to PEAR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=544660 Remi Collet changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #9 from Remi Collet 2010-01-24 07:16:19 EST --- (un)Registration of the channel is now OK. APPROVED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551722] Review Request: php-channel-phpmd - PHP Mess Detector PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551722 --- Comment #3 from Christof Damian 2010-01-24 07:09:49 EST --- Spec URL: http://rpms.damian.net/SPECS/php-channel-phpmd.spec SRPM URL: http://rpms.damian.net/SRPMS/php-channel-phpmd-1.3-2.fc12.src.rpm forgot to change channel xml filename in %post -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 551721] Review Request: php-channel-pdepend - PHP Depend PEAR channel
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=551721 --- Comment #4 from Christof Damian 2010-01-24 07:09:30 EST --- Spec URL: http://rpms.damian.net/SPECS/php-channel-pdepend.spec SRPM URL: http://rpms.damian.net/SRPMS/php-channel-pdepend-1.3-2.fc12.src.rpm forgot to change channel xml filename in %post -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557169] Review Request: bristol - Synthesizer 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=557169 --- Comment #10 from Thomas Spura 2010-01-24 07:04:59 EST --- (In reply to comment #9) > The icon cache scriptlets are outdated, please use the latest version from > https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache If he uses: install -p -m 0644 bitmaps/bicon.svg $RPM_BUILD_ROOT%{_datadir}/pixmaps/bristol.svg this won't be needed anyway. (This suggestion is from: https://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files ) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 544660] Review Request: php-channel-swift - Adds swift mailer project channel to PEAR
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=544660 --- Comment #8 from Christof Damian 2010-01-24 07:07:37 EST --- I obviously wasn't fully awake yet. I fix the other channels too. Spec URL: http://rpms.damian.net/SPECS/php-channel-swift.spec SRPM URL: http://rpms.damian.net/SRPMS/php-channel-swift-1.3-2.fc12.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 456353] Review Request: libffado - Free firewire audio driver 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=456353 --- Comment #12 from John Haxby 2010-01-24 07:01:02 EST --- Created an attachment (id=386419) --> (https://bugzilla.redhat.com/attachment.cgi?id=386419) Updated .spec file In F12, scons can be used to install libffado. Apart from simplifying the .spec file this also means that the ffado mixers are install and I can use the monitor mixer for my FA-101. It would be great to be able to get away from having to install an old firewire module to get this to be usable in F13. This will be all the excuse I need to install the beta, but if you'd like me to install a test rawhide system please let me know. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557169] Review Request: bristol - Synthesizer 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=557169 Christoph Wickert changed: What|Removed |Added CC||cwick...@fedoraproject.org --- Comment #9 from Christoph Wickert 2010-01-24 06:53:18 EST --- The icon cache scriptlets are outdated, please use the latest version from https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 557393] Review Request: ghc-haskell-src - Haskell source-code manipulation 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=557393 Rakesh Pandit changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Rakesh Pandit 2010-01-24 06:43:27 EST --- [+] Build - http://koji.fedoraproject.org/koji/taskinfo?taskID=1941095 - fine [+] source ok from srpm (sha1sum) [rak...@dhcp1-96 SPECS]$ sha1sum haskell-src-1.0.1.3.tar.gz 21b14d8b885e9a2ca6b4226d7619b7ca966097fe haskell-src-1.0.1.3.tar.gz [rak...@dhcp1-96 SPECS]$ sha1sum ../SOURCES/haskell-src-1.0.1.3.tar.gz 21b14d8b885e9a2ca6b4226d7619b7ca966097fe ../SOURCES/haskell-src-1.0.1.3.tar.gz [+] name fine [+] URL fine [+] rpmlint - ok === Koji build rpm /tmp/gachvByqpk/kojiy20BeW/1941097/ghc-haskell-src-prof-1.0.1.3-3.fc13.i686.rpm output: === Rpmlint Messages: ghc-haskell-src-prof.i686: E: devel-dependency ghc-haskell-src-devel ghc-haskell-src-prof.i686: W: no-documentation ghc-haskell-src-prof.i686: W: devel-file-in-non-devel-package /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3/libHShaskell-src-1.0.1.3_p.a 1 packages and 0 specfiles checked; 1 errors, 2 warnings. Ignored. === Koji build rpm /tmp/gachvByqpk/kojiy20BeW/1941097/ghc-haskell-src-doc-1.0.1.3-3.fc13.i686.rpm output: === Rpmlint Messages: 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Clean. === Koji build rpm /tmp/gachvByqpk/kojiy20BeW/1941097/ghc-haskell-src-1.0.1.3-3.fc13.i686.rpm output: === Rpmlint Messages: ghc-haskell-src.i686: W: unstripped-binary-or-object /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3/libHShaskell-src-1.0.1.3-ghc6.12.1.so ghc-haskell-src.i686: W: executable-stack /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3/libHShaskell-src-1.0.1.3-ghc6.12.1.so 1 packages and 0 specfiles checked; 0 errors, 2 warnings. Ignored. === Koji build rpm /tmp/gachvByqpk/kojiy20BeW/1941097/ghc-haskell-src-devel-1.0.1.3-3.fc13.i686.rpm output: === Rpmlint Messages: ghc-haskell-src-devel.i686: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. Ignored. === Log info /tmp/gachvByqpk/kojiy20BeW/1941096/build.log: === Log warnings:warning: Could not canonicalize hostname: xb-01.phx2.fedoraproject.org Log warnings:Warning: Module `Prelude' is deprecated: Log warnings:Warning: Bindings containing unlifted types must use an outermost bang pattern: Ignored. === Log info /tmp/gachvByqpk/kojiy20BeW/1941097/build.log: === Log warnings:warning: Could not canonicalize hostname: xb-01.phx2.fedoraproject.org Log warnings:Warning: Module `Prelude' is deprecated: Log warnings:Warning: Bindings containing unlifted types must use an outermost bang pattern: Log warnings:Warning: The documentation for the following packages are not installed. No Log warnings:warning: Could not canonicalize hostname: xb-01.phx2.fedoraproject.org === Log check ends: === Ignored. May you check with bindings contain unlifted types warnings in log ? They don't seem to be harmful, but still have a look. [+] spec name fine [+] license ok (copy present) and source has marking [+] source tree ok (no binaries present) [+] archs - fine [+] spec file legible and in american english [+] BR's ok [+] %files ok [+] folders owned - ok main /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3 /usr/share/doc/ghc-haskell-src-1.0.1.3 devel /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3 /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3/Language /usr/lib/ghc-6.12.1/haskell-src-1.0.1.3/Language/Haskell doc /usr/share/doc/ghc/html/libraries
[Bug 557392] Review Request: ghc-parallel - Haskell parallel programming 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=557392 Rakesh Pandit changed: What|Removed |Added Status|NEW |ASSIGNED Flag|fedora-review? |fedora-review+ --- Comment #2 from Rakesh Pandit 2010-01-24 06:30:18 EST --- [+] Build - http://koji.fedoraproject.org/koji/taskinfo?taskID=1941092 - fine [+] source ok from srpm (sha1sum) [rak...@dhcp1-96 SPECS]$ sha1sum parallel-1.1.0.1.tar.gz 39bcd5c8a896914c7ddc853049005a71a4eaabb6 parallel-1.1.0.1.tar.gz [rak...@dhcp1-96 SPECS]$ sha1sum ../SOURCES/parallel-1.1.0.1.tar.gz 39bcd5c8a896914c7ddc853049005a71a4eaabb6 ../SOURCES/parallel-1.1.0.1.tar.gz [+] name fine [+] URL fine [+] rpmlint - ok [rak...@dhcp1-96 SPECS]$ rpmlint /home/rakesh/rpmbuild/SRPMS/ghc-parallel-1.1.0.1-2.fc13.src.rpm ghc-parallel.src: W: spelling-error-in-summary en_US Haskell ghc-parallel.src: W: spelling-error-in-description en_US Haskell ghc-parallel.src: W: no-cleaning-of-buildroot %install ghc-parallel.src: W: no-buildroot-tag 1 packages and 0 specfiles checked; 0 errors, 4 warnings. Ignored. [rak...@dhcp1-96 SPECS]$ rpmlint /home/rakesh/rpmbuild/RPMS/i686/ghc-parallel-1.1.0.1-2.fc13.i686.rpm ghc-parallel.i686: W: spelling-error-in-summary en_US Haskell ghc-parallel.i686: W: spelling-error-in-description en_US Haskell ghc-parallel.i686: W: unstripped-binary-or-object /usr/lib/ghc-6.12.1/parallel-1.1.0.1/libHSparallel-1.1.0.1-ghc6.12.1.so ghc-parallel.i686: W: executable-stack /usr/lib/ghc-6.12.1/parallel-1.1.0.1/libHSparallel-1.1.0.1-ghc6.12.1.so 1 packages and 0 specfiles checked; 0 errors, 4 warnings. Ignored. [rak...@dhcp1-96 SPECS]$ rpmlint /home/rakesh/rpmbuild/RPMS/i686/ghc-parallel-devel-1.1.0.1-2.fc13.i686.rpm ghc-parallel-devel.i686: W: spelling-error-in-summary en_US Haskell ghc-parallel-devel.i686: W: spelling-error-in-description en_US Haskell ghc-parallel-devel.i686: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 3 warnings. Ignored. [rak...@dhcp1-96 SPECS]$ rpmlint /home/rakesh/rpmbuild/RPMS/i686/ghc-parallel-doc-1.1.0.1-2.fc13.i686.rpm ghc-parallel-doc.i686: W: spelling-error-in-summary en_US Haskell ghc-parallel-doc.i686: W: spelling-error-in-description en_US Haskell 1 packages and 0 specfiles checked; 0 errors, 2 warnings. Ignored. [rak...@dhcp1-96 SPECS]$ rpmlint /home/rakesh/rpmbuild/RPMS/i686/ghc-parallel-prof-1.1.0.1-2.fc13.i686.rpm ghc-parallel-prof.i686: E: devel-dependency ghc-parallel-devel ghc-parallel-prof.i686: W: spelling-error-in-summary en_US Haskell ghc-parallel-prof.i686: W: spelling-error-in-description en_US Haskell ghc-parallel-prof.i686: W: no-documentation ghc-parallel-prof.i686: W: devel-file-in-non-devel-package /usr/lib/ghc-6.12.1/parallel-1.1.0.1/libHSparallel-1.1.0.1_p.a 1 packages and 0 specfiles checked; 1 errors, 4 warnings. Ignored. [+] spec name fine [+] license ok (copy present) and source has marking [+] source tree ok (no binaries present) [+] archs - fine [+] spec file legible and in american english [+] BR's ok [+] %files ok [+] folders owned - ok main /usr/lib/ghc-6.12.1/parallel-1.1.0.1 /usr/share/doc/ghc-parallel-1.1.0.1 devel /usr/lib/ghc-6.12.1/parallel-1.1.0.1 /usr/lib/ghc-6.12.1/parallel-1.1.0.1/Control doc /usr/share/doc/ghc/html/libraries/parallel-1.1.0.1 [+] source builds fine on fedora ghc archs [+] no bundled dependencies [+] %clean section ok [+] filenames - ok (valid) [+] profiling ok [+] lib goes to %{_libdir}/ghc-%{ghc_version}/%{pkg_name}-%{version} [+] new macros APPROVED 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 530806] Review Request: Pyrit - A GPGPU-driven WPA-PSK key 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=530806 knabberknusperh...@yahoo.de changed: What|Removed |Added Flag|needinfo?(knabberknusperhau | |s...@yahoo.de) | --- Comment #5 from knabberknusperh...@yahoo.de 2010-01-24 06:25:18 EST --- I'm the upstream maintainer. Comments in 2 where implemented. I'll post in this bug when 0.2.5-final becomes available. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review