[Bug 719469] Review Request: kbsh - command-line shell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719469 Zack Parsons changed: What|Removed |Added Status|NEW |CLOSED Blocks|177841(FE-NEEDSPONSOR) | Resolution||DEFERRED Last Closed||2011-07-20 03:50:47 -- Configure 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 690038] Review Request: ompl - The Open Motion Planning 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=690038 Martin Gieseking changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de Flag||fedora-review? --- Comment #7 from Martin Gieseking 2011-07-20 03:49:34 EDT --- Sorry for the delay. I've been rather busy in the last few month. You're right, I somehow mixed up the definitions of soname and soversion. I'm sorry for that too. Your latest package currently fails building because of missing BR: python: http://koji.fedoraproject.org/koji/taskinfo?taskID=3213719 Everything else looks fine. $ rpmlint ompl-*.rpm 4 packages and 0 specfiles checked; 0 errors, 0 warnings. - key: [+] OK [.] OK, not applicable [X] needs work - [+] MUST: The package must be named according to the Package Naming Guidelines. [+] MUST: The spec file name must match the base package %{name}. [+] MUST: The package must meet the Packaging Guidelines. [+] MUST: The package must be licensed with a Fedora approved license. - BSD (3-clause variant) [+] MUST: The License field in the package spec file must match the actual license. [+] MUST: The file containing the text of the license(s) for the package must be included in %doc. [+] MUST: The spec file must be written in American English. [+] MUST: The spec file for the package MUST be legible. [+] MUST: The sources used to build the package must match the upstream source. $ md5sum ompl-0.9.3-Source.tar.gz* b95b4b3a6eebc15a668a52a133384cc1 ompl-0.9.3-Source.tar.gz b95b4b3a6eebc15a668a52a133384cc1 ompl-0.9.3-Source.tar.gz.1 [+] MUST: The package MUST successfully compile and build into binary rpms on at least one primary architecture. [.] MUST: If the package does not successfully compile, build or work on an architecture, ... [X] MUST: All build dependencies must be listed in BuildRequires. BR: python is missing [+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied. [.] MUST: The spec file MUST handle locales properly. [+] MUST: Packages storing shared library files (not just symlinks) must call ldconfig in %post and %postun. [+] MUST: Packages must NOT bundle copies of system libraries. [.] MUST: If the package is designed to be relocatable, ... [+] MUST: A package must own all directories that it creates. [+] MUST: A Fedora package must not list a file more than once in %files. [+] MUST: Permissions on files must be set properly. [+] MUST: Each package must consistently use macros. [+] MUST: The package must contain code, or permissable content. [.] MUST: Large documentation files must go in a -doc subpackage. [+] MUST: Files in %doc must not affect the runtime of the application. [+] MUST: Header files must be in a -devel package. [.] MUST: Static libraries must be in a -static package. [+] MUST: library files that end in .so (without suffix) must go in a -devel package. [+] MUST: devel packages must require the base package using a fully versioned dependency. [+] MUST: Packages must NOT contain any .la libtool archives. [.] MUST: Packages containing GUI applications must include a %{name}.desktop file. [+] MUST: Packages must not own files or directories already owned by other packages. [+] MUST: All filenames in rpm packages must be valid UTF-8. [.] SHOULD: If the source package does not include license text(s) as a separate file from upstream, the packager SHOULD query upstream to include it. [+] SHOULD: Patch files should be prefixed with %{name}- [+] SHOULD: All patches should be commented in the spec file [+] SHOULD: The reviewer should test that the package builds in mock. [+] SHOULD: The reviewer should test that the package functions as described. [+] SHOULD: If scriptlets are used, those scriptlets must be sane. [.] SHOULD: Usually, subpackages other than devel should require the base package using a fully versioned dependency. [.] SHOULD: pkgconfig(.pc) files should be placed in a -devel pkg. [.] SHOULD: If the package has file dependencies outside of /etc, /bin, /sbin, /usr/bin, or /usr/sbin ... [.] SHOULD: Your package should contain man pages for binaries/scripts. -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #17 from Jens Petersen 2011-07-20 03:58:32 EDT --- Created attachment 513944 --> https://bugzilla.redhat.com/attachment.cgi?id=513944 leksah.spec-2.patch -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=668820 --- Comment #25 from Mamoru Tasaka 2011-07-20 04:31:28 EDT --- (In reply to comment #23) > Mamoru, what about using rubygem-rdoc as default for rubygems instead of rdoc? This sounds reasonable, but: > We could also think about marking ruby-rdoc and rubygem-rdoc as conflicting > and > rubygems then pulls in rubygem-rdoc instead of ruby-rdoc. This "conflict" usage is _forbidden_ on Fedora's packaging policy. https://fedoraproject.org/wiki/Packaging/Conflicts But for some exceptions, using conflicts needs FPC's approval and I don't think FPC would approve this 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 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=668820 --- Comment #26 from Vít Ondruch 2011-07-20 04:44:05 EDT --- (In reply to comment #25) > > We could also think about marking ruby-rdoc and rubygem-rdoc as conflicting > > and > > rubygems then pulls in rubygem-rdoc instead of ruby-rdoc. > > This "conflict" usage is _forbidden_ on Fedora's packaging policy. > https://fedoraproject.org/wiki/Packaging/Conflicts > But for some exceptions, using conflicts needs FPC's approval and I don't > think > FPC would approve this case. I am aware of this condition. I'll be thinking about it. May be somebody will help to crystallize this 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 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 --- Comment #3 from Mark McLoughlin 2011-07-20 04:46:51 EDT --- (In reply to comment #2) > (In reply to comment #1) > > - I don't think euca2ools is actually required; we run all the euca- > >commands inside EC2 instances not locally > > When we are doing "upload" or "local" style builds, the euca2ools are actually > required to do the bundling. See imagefactory/builders/FedoraBuilder.py, > around lines 1383. Ah, of course - right I was just about to approve this, but then I remembered to run rpmlint on the actual RPM itself: $> rpmlint imagefactory-0.3.1-2.fc14.noarch.rpm imagefactory.noarch: E: explicit-lib-dependency python-httplib2 imagefactory.noarch: W: conffile-without-noreplace-flag /etc/pki/imagefactory/cert-ec2.pem imagefactory.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/imagefactory/ApplicationConfiguration.py 0644L /usr/bin/env imagefactory.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/imagefactory/Template.py 0644L /usr/bin/env imagefactory.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/imagefactory/ImageWarehouse.py 0644L /usr/bin/env imagefactory.noarch: E: non-executable-script /usr/lib/python2.7/site-packages/imagefactory/VMWare.py 0644L /usr/bin/env imagefactory.noarch: W: no-manual-page-for-binary imgfac.py 1 packages and 0 specfiles checked; 5 errors, 2 warnings. Re: explicit-lib-dependency - this looks like a false positive to me. See http://lists.fedoraproject.org/pipermail/devel/2010-March/133955.html Re: conffile-without-noreplace-flag - do we allow users to replace this with their own cert? If so, the error is valid. Re: non-executable-script - do sed -i '/\/usr\/bin\/env python/d' imagefactory/*.py Re: no-manual-page-for-binary - there actually is a man page, but not with the same name as the binary. Do we really want the binary to be installed as imgfac.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 677496] Review Request: nbc - Next Byte Codes (NBC) and Not Exactly C (NXC)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=677496 --- Comment #6 from Jon Ciesla 2011-07-20 06:03:25 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720085] Review Request: perl-Net-FTP-AutoReconnect - FTP client class with automatic reconnect on failure
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720085 --- Comment #9 from Jon Ciesla 2011-07-20 06:03:54 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720086] Review Request: perl-Net-FTP-RetrHandle - Provides a file reading interface for reading files on a remote FTP server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720086 --- Comment #9 from Jon Ciesla 2011-07-20 06:04:17 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 723472] New: Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-aws-sdk - AWS SDK for Ruby https://bugzilla.redhat.com/show_bug.cgi?id=723472 Summary: Review Request: rubygem-aws-sdk - AWS SDK for Ruby Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mgold...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://goldmann.fedorapeople.org/package_review/rubygem-aws-sdk/1/rubygem-aws-sdk.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/rubygem-aws-sdk/1/rubygem-aws-sdk-1.0.1-1.fc15.src.rpm Description: Build Ruby applications with the aws-sdk gem that takes the complexity out of coding directly against the web service interfaces. The gem handles common tasks, such as authentication, request retries, XML processing, error handling, and more. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3214121 $ rpmlint ~/rpmbuild/SRPMS/rubygem-aws-sdk-1.0.1-1.fc15.src.rpm rubygem-aws-sdk.src: I: enchant-dictionary-not-found en_US rubygem-aws-sdk.src: W: no-%build-section 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint ./rubygem-aws-sdk.spec ./rubygem-aws-sdk.spec: W: no-%build-section 0 packages and 1 specfiles checked; 0 errors, 1 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 Michal Fojtik changed: What|Removed |Added CC||mfoj...@redhat.com Flag||fedora-review? --- Comment #1 from Michal Fojtik 2011-07-20 06:13:30 EDT --- Taking this for a 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 706832] Review Request: hibernate-commons-annotations - Hibernate Annotations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=706832 Marek Goldmann changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-07-20 06:19:09 -- Configure 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 Michal Fojtik changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #2 from Michal Fojtik 2011-07-20 06:23:37 EDT --- [ OK ] Packages that contain Ruby Gems must be called rubygem-%{gemname} where gemname [ OK ] The Source of the package must be the full URL to the released Gem archive; the version of the package must be the Gem's version [ OK ] Package has proper license (ASL 2.0) [ OK ] The package must have a Requires and a BuildRequires on rubygems [ OK? ] The %prep and %build sections of the specfile should be empty. %build section is missing, but I though it's OK, since it's not required. [ OK ] The Gem must be installed into %{gemdir} defined as %global gemdir %(ruby -rubygems -e 'puts Gem::dir' 2>/dev/null) [ OK ] The package must own the following files and directories: %{gemdir}/gems/%{gemname}-%{version}/ %{gemdir}/cache/%{gemname}-%{version}.gem %{gemdir}/specifications/%{gemname}-%{version}.gemspec [ OK] Architecture-specific content must not be installed into %{gemdir} There is no architecture specific content. [ OK] If the Gem only contains pure Ruby code, it must be marked as BuildArch: noarch. Marked properly. [ OK ] If the Gem contains binary content (e.g., for a database driver), it must be marked as architecture specific, and all architecture specific content must be moved from the %{gemdir} to the [#ruby_sitearch %{ruby_sitearch} directory] during %install This gem include only SSL certificate, which is not architecture specific. All clear to go so APPROVED. Nice work Marek! -- Configure 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 722914] Review Request: volumeicon - Lightweight volume control for the system tray
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=722914 --- Comment #4 from Veeti Paananen 2011-07-20 06:21:18 EDT --- > Oh? For me it gave GPLv3 straight away. If licensecheck (and the headers) say that the license is GPLv3, isn't the "GPLv3+" in the spec incorrect? > This is nitpicking. But in principle you may be correct. While the spelling error definitely wouldn't block a review, I thought I'd point it 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 Michal Fojtik changed: What|Removed |Added QAContact|extras...@fedoraproject.org |mfoj...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642592] Review Request: rubygem-macaddr - Cross platform mac address determination for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642592 --- Comment #15 from Vít Ondruch 2011-07-20 06:45:31 EDT --- 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 642583] Review Request: rubygem-rr - RR is a framework that features a rich selection of double techniques
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642583 --- Comment #9 from Vít Ondruch 2011-07-20 06:45:07 EDT --- 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 Marek Goldmann changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Marek Goldmann 2011-07-20 06:48:22 EDT --- Michal, thank you for this review! New Package SCM Request === Package Name: rubygem-aws-sdk Short Description: AWS SDK for Ruby Owners: goldmann Branches: f14 f15 el5 el6 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 708670] Review Request: hawtjni - Code generator that produces the JNI code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=708670 Bug 708670 depends on bug 708645, which changed state. Bug 708645 Summary: Review Request: fusesource-pom - Parent POM for FuseSource Maven projects https://bugzilla.redhat.com/show_bug.cgi?id=708645 What|Old Value |New Value Resolution||RAWHIDE Status|ASSIGNED|CLOSED -- Configure 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 708645] Review Request: fusesource-pom - Parent POM for FuseSource Maven projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=708645 Stanislav Ochotnicky changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-07-20 07:02:16 --- Comment #5 from Stanislav Ochotnicky 2011-07-20 07:02:16 EDT --- Package already built: http://koji.fedoraproject.org/koji/buildinfo?buildID=245728 Please close review bugs after building 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 722914] Review Request: volumeicon - Lightweight volume control for the system tray
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=722914 --- Comment #5 from Christoph Wickert 2011-07-20 07:06:29 EDT --- Thanks Veeti, both are valid points. GPLv3+ is definitely wrong, GPLv3 is correct. As for the spelling error: Capitalization can/should be used in the summary but not in the description. (In reply to comment #2) > - As a courtesy to people reading your spec files, please don't use macros in > the URL if they're not absolutely necessary. Why not? we have spectool... > - Patch0 is missing a description in the spec file. Please document what it > does. Has it been sent upstream? It fixes linking problems and I sent it upstream by mail. I'll add a comment on this. > - Where is Source1 from? Has it been sent upstream? It's from gnome-media and has not been sent upstream (yet). (In reply to comment #1) > - Wouldn't it make sense to place another desktop file in the actual menu for > this instead of just having an autostart entry? (?) I don't think so. Icons in the systray are supposed to be persistent throughout the whole session. They should be started on login and not from the menu. -- Configure 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 --- Comment #4 from Jon Ciesla 2011-07-20 07:55:00 EDT --- Git done (by process-git-requests). Michal, in the future, please take ownership of review bugs you're working on. 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 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 Marcela Mašláňová changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 Marcela Mašláňová changed: What|Removed |Added Flag||fedora-review+ --- Comment #1 from Marcela Mašláňová 2011-07-20 08:01:57 EDT --- - rpmlint OK - package must be named according to Guidelines OK - spec file name must match the base package %{name} OK - package must meet the Packaging Guidelines OK - package must be licensed with Fedora approved license OK - license field must match actual license OK - text of the license in its own file must be included in %doc OK - sources must match the upstream source OK - package MUST successfully compile and build OK http://koji.fedoraproject.org/koji/taskinfo?taskID=3214430 - architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK - build dependencies must be listed in BuildRequires OK - handle locales properly with %find_lang macro OK - shared library files must call ldconfig in %post(un) OK - packages must NOT bundle system libraries OK - package must own all directories that it creates OK - permissions on files must be set properly OK - package must consistently use macros OK - package must contain code, or permissable content OK - large documentation must go in a -doc OK - %doc must not affect the runtime of the application OK - header files must be in a -devel package OK - static libraries must be in a -static package OK - library files that end in .so (without suffix) must go in a -devel OK - devel package usually require base package OK - packages must NOT contain any .la libtool archives OK - GUI applications must include a %{name}.desktop file OK - packages must not own files or directories already owned by other packages OK rpm -qp --provides rpmbuild/RPMS/noarch/perl-Module-Runtime-0.008-1.fc16.noarch.rpm perl(Module::Runtime) = 0.008 perl-Module-Runtime = 0.008-1.fc16 rpm -qp --requires rpmbuild/RPMS/noarch/perl-Module-Runtime-0.008-1.fc16.noarch.rpm perl(:MODULE_COMPAT_5.12.3) perl(Exporter) perl(Params::Classify) >= 0.000 perl(parent) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 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 675050] Review Request: cloudfs - Cloud Filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675050 Kaleb KEITHLEY changed: What|Removed |Added Flag||fedora-cvs? --- Comment #25 from Kaleb KEITHLEY 2011-07-20 08:11:33 EDT --- New Package SCM Request === Package Name: cloudfs Short Description: CloudFS is a cloud-capable file system based on GlusterFS (http://gluster.org) Owners: kkeithle Branches: f16 InitialCC: jdarcy -- Configure 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 722914] Review Request: volumeicon - Lightweight volume control for the system tray
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=722914 Christoph Wickert changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Christoph Wickert 2011-07-20 08:16:34 EDT --- New Package SCM Request === Package Name: volumeicon Short Description: Lightweight volume control for the system tray Owners: cwickert Branches: f14 f15 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 Marek Goldmann changed: What|Removed |Added Status|NEW |CLOSED Resolution||RAWHIDE Flag|fedora-cvs+ |fedora-cvs? Last Closed||2011-07-20 08:23:53 --- Comment #5 from Marek Goldmann 2011-07-20 08:23:53 EDT --- Thank you for review and git! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 722914] Review Request: volumeicon - Lightweight volume control for the system tray
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=722914 --- Comment #7 from Jon Ciesla 2011-07-20 08:35:38 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 675050] Review Request: cloudfs - Cloud Filesystem
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=675050 --- Comment #26 from Jon Ciesla 2011-07-20 08:35:13 EDT --- Git done (by process-git-requests). Don't request f16, devel is created by default. -- Configure 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 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720327 Marcela Mašláňová changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 --- Comment #6 from Jon Ciesla 2011-07-20 08:38:11 EDT --- Why was the cvs flag set again? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720327 Marcela Mašláňová changed: What|Removed |Added Flag||fedora-review+ --- Comment #1 from Marcela Mašláňová 2011-07-20 08:40:52 EDT --- - rpmlint OK - package must be named according to Guidelines OK - spec file name must match the base package %{name} OK - package must meet the Packaging Guidelines OK - package must be licensed with Fedora approved license OK - license field must match actual license OK - text of the license in its own file must be included in %doc OK - sources must match the upstream source OK - package MUST successfully compile and build OK - architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK - build dependencies must be listed in BuildRequires OK - handle locales properly with %find_lang macro OK - shared library files must call ldconfig in %post(un) OK - packages must NOT bundle system libraries OK - package must own all directories that it creates OK - permissions on files must be set properly OK - package must consistently use macros OK - package must contain code, or permissable content OK - large documentation must go in a -doc OK - %doc must not affect the runtime of the application OK - header files must be in a -devel package OK - static libraries must be in a -static package OK - library files that end in .so (without suffix) must go in a -devel OK - devel package usually require base package OK - packages must NOT contain any .la libtool archives OK - GUI applications must include a %{name}.desktop file OK - packages must not own files or directories already owned by other packages OK rpm -qp --provides rpmbuild/RPMS/noarch/perl-DynaLoader-Functions-0.000-1.fc16.noarch.rpm perl(DynaLoader::Functions) = 0.000 perl-DynaLoader-Functions = 0.000-1.fc16 rpm -qp --requires rpmbuild/RPMS/noarch/perl-DynaLoader-Functions-0.000-1.fc16.noarch.rpm perl(:MODULE_COMPAT_5.12.3) perl(Carp) perl(DynaLoader) perl(Exporter) perl(Module::Runtime) >= 0.007 perl(Params::Classify) >= 0.008 perl(parent) perl(strict) perl(warnings) rpmlib(CompressedFileNames) <= 3.0.4-1 rpmlib(PayloadFilesHavePrefix) <= 4.0-1 This package is conditionally approved, because build requires is not in buildroot yet. Check if it builds in koji first. It does build for me in local rawhide. 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 723472] Review Request: rubygem-aws-sdk - AWS SDK for Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723472 --- Comment #7 from Marek Goldmann 2011-07-20 08:44:35 EDT --- TBH, I don't know - maybe it was in browser 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 680174] Review Request: herqq - A software library for building UPnP devices and control points
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680174 --- Comment #6 from Jaroslav Reznik 2011-07-20 08:45:39 EDT --- Thanks Rex, fixed. Spec URL: http://rezza.hofyland.cz/fedora/packages/herqq/herqq.spec SRPM URL: http://rezza.hofyland.cz/fedora/packages/herqq/herqq-0.9.1-3.fc14.src.rpm * Wed Jul 20 2011 Jaroslav Reznik - 0.9.1-3 - fix license to LGPLv3+ - qt4 header dir for consistency - shlib soname tracked in %files - -devel should not duplicate COPYING -- Configure 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 680174] Review Request: herqq - A software library for building UPnP devices and control points
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680174 Jaroslav Reznik changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Jaroslav Reznik 2011-07-20 08:47:32 EDT --- New Package SCM Request === Package Name: herqq Short Description: A software library for building UPnP devices and control points Owners: jreznik Branches: f14 f15 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 723523] Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=723523 Marek Goldmann changed: What|Removed |Added Blocks||652183(FE-JAVASIG) -- Configure 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 723523] New: Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams https://bugzilla.redhat.com/show_bug.cgi?id=723523 Summary: Review Request: apiviz - APIviz is a JavaDoc doclet to generate class and package diagrams Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mgold...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://goldmann.fedorapeople.org/package_review/apiviz/1/apiviz.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/apiviz/1/apiviz-1.3.1-1.fc15.src.rpm Description: APIviz is a JavaDoc doclet which extends the Java standard doclet. It generates comprehensive UML-like class and package diagrams for quick understanding of the overall API structure. $ rpmlint ./apiviz-1.3.1-1.fc15.src.rpm apiviz.src: I: enchant-dictionary-not-found en_US apiviz.src: W: name-repeated-in-summary C APIviz apiviz.src: W: invalid-url Source0: apiviz-1.3.1.GA.tar.xz 1 packages and 0 specfiles checked; 0 errors, 2 warnings. $ rpmlint ./apiviz.spec ./apiviz.spec: W: invalid-url Source0: apiviz-1.3.1.GA.tar.xz 0 packages and 1 specfiles checked; 0 errors, 1 warnings. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3214489 -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #18 from Lakshmi Narasimhan 2011-07-20 09:17:12 EDT --- Thanks for the review. Yes. I remember that ghc-leksah, although needed (IMO), was not being picked up automatically. I will check that and update the spec as appropriate. For the icon part, I will go with your suggestion and not resize. -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 Lakshmi Narasimhan changed: What|Removed |Added Flag||fedora-cvs? --- Comment #19 from Lakshmi Narasimhan 2011-07-20 09:18:07 EDT --- New Package SCM Request === Package Name: leksah Short Description: An IDE for Haskell Owners: narasim Branches: f14 f15 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 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=668820 Mo Morsi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #27 from Mo Morsi 2011-07-20 09:25:45 EDT --- OK, thanks for the ACK. Patch updated to address all outstanding feedback Updated SPEC: http://mo.morsi.org/files/rpms/rubygem-rdoc.spec Updated SRPM: http://mo.morsi.org/files/rpms/rubygem-rdoc-3.8-2.fc15.src.rpm Koji Build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3214663 New Package SCM Request === Package Name: rubygem-rdoc Short Description: Produces HTML and command-line documentation for Ruby projects Owners: mmorsi Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719067 Mo Morsi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Mo Morsi 2011-07-20 09:26:46 EDT --- Looks good. 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #20 from Jon Ciesla 2011-07-20 09:28:21 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 680174] Review Request: herqq - A software library for building UPnP devices and control points
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=680174 --- Comment #8 from Jon Ciesla 2011-07-20 09:30:08 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=668820 --- Comment #28 from Jon Ciesla 2011-07-20 09:32:05 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719067 Chris Lalancette changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Chris Lalancette 2011-07-20 09:52:46 EDT --- New Package SCM Request === Package Name: rubygem-hmac Short Description: This module provides common interface to HMAC functionality Owners: clalance mmorsi -- Configure 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 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG 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=719975 Mo Morsi changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Mo Morsi 2011-07-20 09:59:53 EDT --- Looks good. 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 668820] Review Request: rubygem-rdoc - RDoc produces HTML and command-line documentation for Ruby projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=668820 Mo Morsi changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed|2011-03-08 16:08:23 |2011-07-20 09:59:12 --- Comment #29 from Mo Morsi 2011-07-20 09:59:12 EDT --- Pushed to 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 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG 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=719975 Chris Lalancette changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Chris Lalancette 2011-07-20 10:04:40 EDT --- New Package SCM Request === Package Name: rubygem-chunky_png Short Description: Pure ruby library for read/write, chunk-level access to PNG files Owners: clalance mmorsi -- Configure 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 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=706934 Mo Morsi changed: What|Removed |Added CC||mmo...@redhat.com --- Comment #10 from Mo Morsi 2011-07-20 10:13:29 EDT --- Looks like the push to git has been done but the koji build failed and thus this is still not available in rawhide. Any thoughts as to the problem? http://koji.fedoraproject.org/koji/buildinfo?buildID=251113 -- Configure 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 721057] Review Request: rubygem-sass - A powerful but elegant CSS compiler that makes CSS fun again
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721057 Mo Morsi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Mo Morsi 2011-07-20 10:15:56 EDT --- Thank you New Package SCM Request === Package Name: rubygem-sass Short Description: A powerful but elegant CSS compiler Owners: mmorsi Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721063] Review Request: rubygem-factory_girl_rails - factory_girl_rails provides integration between factory_girl and rails 3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721063 Mo Morsi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Mo Morsi 2011-07-20 10:20:54 EDT --- New Package SCM Request === Package Name: rubygem-factory_girl_rails Short Description: provides integration between factory_girl and rails 3 Owners: mmorsi Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721059] Review Request: rubygem-dynamic_form - DynamicForm holds a few helper methods to help you deal with your Rails3 models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721059 Mo Morsi changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Mo Morsi 2011-07-20 10:18:15 EDT --- Thanks New Package SCM Request === Package Name: rubygem-dynamic_form Short Description: DynamicForm holds a few helper methods to help you deal with your Rails3 models Owners: mmorsi Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721066 --- Comment #6 from Mo Morsi 2011-07-20 10:27:09 EDT --- Updated SPEC: http://mo.morsi.org/files/aeolus/rubygem-image_factory_console.spec Updated SRPM: http://mo.morsi.org/files/aeolus/rubygem-image_factory_console-0.4.0-3.fc15.src.rpm (In reply to comment #5) > Also, there's a thread on aeolus-devel: > > https://fedorahosted.org/pipermail/aeolus-devel/2011-July/003247.html > > Looks like we're going to change the name of gem to 'imagefactory-console' Will update this submission when the patches doing so have been ack'd/pushed (In reply to comment #4) > FWIW, I just noticed ruby_sitelib is unused Ah good catch, removed -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #21 from Lakshmi Narasimhan 2011-07-20 10:31:38 EDT --- ghc-leksah is not required and I removed the Requires. -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 Lakshmi Narasimhan changed: What|Removed |Added Status|ASSIGNED|MODIFIED --- Comment #22 from Lakshmi Narasimhan 2011-07-20 10:45:25 EDT --- Built for rawhide http://koji.fedoraproject.org/koji/taskinfo?taskID=3214809 -- Configure 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 721112] Review Request: vmtk - The Vascular Modeling Toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721112 --- Comment #5 from Ankur Sinha 2011-07-20 10:43:12 EDT --- Hi Richard, (In reply to comment #4) > I've made some progress trying to get it to build against the OpenNL package > but I can't really test it because of: > > rpmbuild -bc rpmbuild/vmtk/SPECS/vmtk.spec > error: Failed build dependencies: > InsightToolkit-devel is needed by vmtk-0.9.0-2.fc14.x86_64 > vxl-devel is needed by vmtk-0.9.0-2.fc14.x86_64 > > Because of that I've added the review request of vxl and InsightToolkit as a > blocker for this Review Request. I'm sorry, I forgot to mention that you need to build ITK and vxl for your fedora system before you can test it. I for example, have a local mock repo with these packages. [root@ankur ~]# ls /tmp/mock-repo/InsightToolkit-* /tmp/mock-repo/vxl-* /tmp/mock-repo/InsightToolkit-3.20.0-5.fc16.i686.rpm /tmp/mock-repo/vxl-1.14.0-1.fc16.i686.rpm /tmp/mock-repo/InsightToolkit-3.20.0-5.fc16.src.rpm /tmp/mock-repo/vxl-1.14.0-1.fc16.src.rpm /tmp/mock-repo/InsightToolkit-debuginfo-3.20.0-5.fc16.i686.rpm /tmp/mock-repo/vxl-debuginfo-1.14.0-1.fc16.i686.rpm /tmp/mock-repo/InsightToolkit-devel-3.20.0-5.fc16.i686.rpm /tmp/mock-repo/vxl-devel-1.14.0-1.fc16.i686.rpm /tmp/mock-repo/InsightToolkit-doc-3.20.0-5.fc16.i686.rpm /tmp/mock-repo/vxl-doc-1.14.0-1.fc16.i686.rpm /tmp/mock-repo/InsightToolkit-examples-3.20.0-5.fc16.i686.rpm > > Back to this review :) > > The only part of the project that appears to use OpenNL is the > DifferentialGeometry sub-project. > > I've added a cmake option, USE_EXTERN_OPENNL to the main CMakeLists.txt and > I'm > working on getting the headers and linking working in the CMakeLists.txt for > DifferentialGeometry. The package works with the fedora OpenNL which I packaged recently. You don't need to make any tweaks. The changes I've made in the prep and build sections are enough. At the moment, gdcm is broken in rawhide,(after the latest libpoppler soname bump) so I can't build it as well :/ Thanks, Ankur -- Configure 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 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 --- Comment #2 from Petr Pisar 2011-07-20 10:49:50 EDT --- New Package SCM Request === Package Name: perl-Module-Runtime Short Description: Runtime module handling Owners: ppisar mmaslano psabata Branches: f14 f15 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 Petr Pisar changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 --- Comment #4 from Chris Lalancette 2011-07-20 10:49:51 EDT --- (In reply to comment #3) > (In reply to comment #2) > > (In reply to comment #1) > > > - I don't think euca2ools is actually required; we run all the euca- > > >commands inside EC2 instances not locally > > > > When we are doing "upload" or "local" style builds, the euca2ools are > > actually > > required to do the bundling. See imagefactory/builders/FedoraBuilder.py, > > around lines 1383. > > Ah, of course - right > > I was just about to approve this, but then I remembered to run rpmlint on the > actual RPM itself: > > $> rpmlint imagefactory-0.3.1-2.fc14.noarch.rpm > imagefactory.noarch: E: explicit-lib-dependency python-httplib2 > imagefactory.noarch: W: conffile-without-noreplace-flag > /etc/pki/imagefactory/cert-ec2.pem > imagefactory.noarch: E: non-executable-script > /usr/lib/python2.7/site-packages/imagefactory/ApplicationConfiguration.py > 0644L > /usr/bin/env > imagefactory.noarch: E: non-executable-script > /usr/lib/python2.7/site-packages/imagefactory/Template.py 0644L /usr/bin/env > imagefactory.noarch: E: non-executable-script > /usr/lib/python2.7/site-packages/imagefactory/ImageWarehouse.py 0644L > /usr/bin/env > imagefactory.noarch: E: non-executable-script > /usr/lib/python2.7/site-packages/imagefactory/VMWare.py 0644L /usr/bin/env > imagefactory.noarch: W: no-manual-page-for-binary imgfac.py > 1 packages and 0 specfiles checked; 5 errors, 2 warnings. > > > Re: explicit-lib-dependency - this looks like a false positive to me. See > http://lists.fedoraproject.org/pipermail/devel/2010-March/133955.html Yeah, I figured it was just looking for the lib string, and that post confirms it :). > > Re: conffile-without-noreplace-flag - do we allow users to replace this with > their own cert? If so, the error is valid. No, the user can't replace it. It is the global ec2 certificate, comes from amazon, and is used (in some way) to sign the bundled images. As such, I believe it is correct as-is. > > Re: non-executable-script - do sed -i '/\/usr\/bin\/env python/d' > imagefactory/*.py Fixed. > > Re: no-manual-page-for-binary - there actually is a man page, but not with the > same name as the binary. Do we really want the binary to be installed as > imgfac.py ? This has been a source of annoyance to me for some time. Upstream, I think we should eventually rename the binary to imagefactory and fix all of the references to it, but I don't think I want to make that kind of change right at the moment. I've uploaded revision 3: SRPM: http://people.redhat.com/clalance/imagefactory/imagefactory-0.3.1-3.fc14.src.rpm SPEC: http://people.redhat.com/clalance/imagefactory/imagefactory.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720327 Petr Pisar changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 711462] Review Request: perl-Fedora-Rebuild - Rebuilds Fedora packages from scratch
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711462 --- Comment #17 from Fedora Update System 2011-07-20 10:55:24 EDT --- perl-Fedora-Rebuild-0.8.0-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/perl-Fedora-Rebuild-0.8.0-1.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720327 --- Comment #2 from Petr Pisar 2011-07-20 10:52:36 EDT --- New Package SCM Request === Package Name: perl-DynaLoader-Functions Short Description: Deconstructed dynamic C library loading Owners: ppisar mmaslano psabata Branches: f14 f15 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #24 from Fedora Update System 2011-07-20 10:57:21 EDT --- leksah-0.10.0.4-3.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/leksah-0.10.0.4-3.fc15 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #25 from Lakshmi Narasimhan 2011-07-20 10:58:24 EDT --- Wrong build posted on comment 22 . Rawhide build @ http://koji.fedoraproject.org/koji/buildinfo?buildID=254319 -- Configure 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 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=706934 --- Comment #11 from Chris Lalancette 2011-07-20 10:58:08 EDT --- I think the problem had something to do with an outdated railties. I'll give it another whirl and see what the errors are. -- Configure 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 673776] Review Request: leksah - An IDE for Haskell
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=673776 --- Comment #23 from Fedora Update System 2011-07-20 10:57:12 EDT --- leksah-0.10.0.4-3.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/leksah-0.10.0.4-3.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 Mark McLoughlin changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #5 from Mark McLoughlin 2011-07-20 10:56:31 EDT --- Okay, looks good to 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 711462] Review Request: perl-Fedora-Rebuild - Rebuilds Fedora packages from scratch
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=711462 --- Comment #18 from Fedora Update System 2011-07-20 10:59:55 EDT --- perl-Fedora-Rebuild-0.8.0-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/perl-Fedora-Rebuild-0.8.0-1.fc14 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 706934] Review Request: rubygem-rspec-rails - RSpec-2 for Rails-3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=706934 --- Comment #12 from Chris Lalancette 2011-07-20 11:07:22 EDT --- Right, here's the problem (from root.log): DEBUG util.py:250: --> rubygems-1.8.5-2.fc16.noarch DEBUG util.py:250: --> ruby-libs-1.8.7.352-1.fc16.x86_64 DEBUG util.py:250: --> rubygem-rspec-core-2.6.4-1.fc16.noarch DEBUG util.py:250: --> rubygem-rspec-core-2.6.4-1.fc16.noarch DEBUG util.py:250: --> 1:rubygem-rails-3.0.5-2.fc16.noarch DEBUG util.py:250: --> 1:rubygem-rails-3.0.5-2.fc16.noarch DEBUG util.py:250: --> rubygem-sqlite3-1.3.3-5.fc16.x86_64 DEBUG util.py:250: Error: Package: 1:rubygem-rails-3.0.5-2.fc16.noarch (fedora) DEBUG util.py:250: Requires: rubygem(railties) = 3.0.5 DEBUG util.py:250: Available: rubygem-railties-3.0.9-1.fc16.noarch (fedora) DEBUG util.py:250: rubygem(railties) = 3.0.9 DEBUG util.py:250: You could try using --skip-broken to work around the problem DEBUG util.py:250: You could try running: rpm -Va --nofiles --nodigest The problem is that the updated railties is built, but it needs the updated rails which is not. Now that the new rubygem-rdoc is in, hopefully we can get rails 3.0.9 built, and then we can build 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 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 --- Comment #6 from Chris Lalancette 2011-07-20 11:13:02 EDT --- 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 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721066 --- Comment #7 from Chris Lalancette 2011-07-20 11:12:23 EDT --- These patches are now in the repository, so we should be good to rename the RPM and get this package in. -- Configure 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 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 Chris Lalancette changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Chris Lalancette 2011-07-20 11:13:57 EDT --- New Package SCM Request === Package Name: imagefactory Short Description: System image generation tool Owners: clalance -- Configure 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 721066] Review Request: rubygem-image_factory_console - QMF Console for Aeolus Image Factory
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721066 --- Comment #8 from Mark McLoughlin 2011-07-20 11:17:18 EDT --- Well, there hasn't been an official upstream release to package yet - I'd wait until tomorrow for that -- Configure 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 722427] Review Request: python-importlib - Backport of importlib.import_module() from Python 2.7
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=722427 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2011-07-20 11:30:49 EDT --- python-importlib-1.0.2-1.el5 has been pushed to the Fedora EPEL 5 testing repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 --- Comment #18 from Fedora Update System 2011-07-20 11:31:17 EDT --- python26-argparse-1.2.1-3.el5, packagedb-cli-1.0.0-6.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 Fedora Update System changed: What|Removed |Added Fixed In Version|python26-argparse-1.2.1-3.e |packagedb-cli-1.0.0-6.el6 |l5 | -- Configure 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 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||python26-argparse-1.2.1-3.e ||l5 Resolution||ERRATA Last Closed||2011-07-20 11:31:24 -- Configure 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 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 --- Comment #19 from Fedora Update System 2011-07-20 11:32:13 EDT --- packagedb-cli-1.0.0-6.el6 has been pushed to the Fedora EPEL 6 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617340] Review Request: throttle - copy stdin to stdout at the specified speed (or lower)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617340 --- Comment #26 from Fedora Update System 2011-07-20 11:32:57 EDT --- throttle-1.2-4.el6 has been pushed to the Fedora EPEL 6 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 717555] Review Request: packagedb-cli - A CLI for pkgdb
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717555 Fedora Update System changed: What|Removed |Added Fixed In Version|packagedb-cli-1.0.0-6.fc14 |python26-argparse-1.2.1-3.e ||l5 -- Configure 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 717681] Review Request: python26-argparse - Optparse inspired command line parser for Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=717681 --- Comment #7 from Fedora Update System 2011-07-20 11:31:12 EDT --- python26-argparse-1.2.1-3.el5, packagedb-cli-1.0.0-6.el5 has been pushed to the Fedora EPEL 5 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617340] Review Request: throttle - copy stdin to stdout at the specified speed (or lower)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617340 --- Comment #27 from Fedora Update System 2011-07-20 11:34:10 EDT --- throttle-1.2-4.el5 has been pushed to the Fedora EPEL 5 stable repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719854] Review Request: rubygem-xmlparser-0.6.81-1 - Ruby bindings to the Expat XML parsing 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=719854 --- Comment #25 from Shawn Starr 2011-07-20 11:34:43 EDT --- Additional comments: the DSO library should be in %{ruby_sitearch}/%{gemname}/*.so Same as rubygem-cairo, although rubygem-json uses %{ruby_sitearch}/%{gemname}/etc/ for the DSO location. -- Configure 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 721078] Review Request: aeolus-conductor - A web UI for managing cloud instances.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721078 Chris Lalancette changed: What|Removed |Added CC||clala...@redhat.com --- Comment #3 from Chris Lalancette 2011-07-20 11:35:57 EDT --- I've fixed most of the issues you've identified here. I still have to figure out what to do about the Source URL; I'll post again once I have that licked. -- Configure 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 723575] New: Review Request: libpki - Easy-to-use PKI library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libpki - Easy-to-use PKI library https://bugzilla.redhat.com/show_bug.cgi?id=723575 Summary: Review Request: libpki - Easy-to-use PKI library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: p...@datasphere.ch QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec URL: http://monnerat.fedorapeople.org/libpki.spec SRPM URL: http://monnerat.fedorapeople.org/libpki-0.6.5-1.fc14.src.rpm Description: This library provides the developer with all the needed features to manage certificates, from generation to validation. The LibPKI Project enables developers with the possibility to implement complex cryptographic operations with a few simple function calls by implementing an high-level cryptographic API. The last version of the OCSP responder (ocspd) uses this library: thus I need libpki to upgrade ocspd. rpmlint output: $ rpmlint SPECS/libpki.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint SRPMS/libpki-0.6.5-1.fc14.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint RPMS/x86_64/libpki-0.6.5-1.fc14.x86_64.rpm libpki.x86_64: E: incorrect-fsf-address /usr/share/doc/libpki-0.6.5/COPYING libpki.x86_64: W: no-manual-page-for-binary pki-xpair libpki.x86_64: W: no-manual-page-for-binary pki-crl libpki.x86_64: W: no-manual-page-for-binary pki-request libpki.x86_64: W: no-manual-page-for-binary pki-query libpki.x86_64: W: no-manual-page-for-binary pki-cert libpki.x86_64: W: no-manual-page-for-binary pki-derenc libpki.x86_64: W: no-manual-page-for-binary url-tool libpki.x86_64: W: no-manual-page-for-binary pki-tool libpki.x86_64: W: no-manual-page-for-binary pki-siginfo 1 packages and 0 specfiles checked; 1 errors, 9 warnings. $ rpmlint RPMS/x86_64/libpki-devel-0.6.5-1.fc14.x86_64.rpm libpki-devel.x86_64: W: no-documentation libpki-devel.x86_64: W: no-manual-page-for-binary libpki-config 1 packages and 0 specfiles checked; 0 errors, 2 warnings. $ rpmlint RPMS/x86_64/libpki-static-0.6.5-1.fc14.x86_64.rpm libpki-static.x86_64: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. $ rpmlint RPMS/x86_64/libpki-doc-0.6.5-1.fc14.x86_64.rpm libpki-doc.x86_64: E: incorrect-fsf-address /usr/share/doc/libpki-doc-0.6.5/COPYING 1 packages and 0 specfiles checked; 1 errors, 0 warnings. $ rpmlint RPMS/x86_64/libpki-debuginfo-0.6.5-1.fc14.x86_64.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. - Upstream has been requested by e-mail to change the license file and to provide man1 pages. - Documentation that can be provided in the -devel and -static subpackages is already present in the (required) main package. - Although "make test" is recognized, no %check spec section is provided: the upstream-provided tests need a tty input and can only be executed by themselves without failing :-( Thanks in advance 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 719067] Review Request: rubygem-hmac - This module provides common interface to HMAC functionality
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=719067 --- Comment #6 from Jon Ciesla 2011-07-20 11:46:28 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721112] Review Request: vmtk - The Vascular Modeling Toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721112 --- Comment #6 from Richard Shaw 2011-07-20 11:43:55 EDT --- (In reply to comment #5) > (In reply to comment #4) > > I've made some progress trying to get it to build against the OpenNL package > > but I can't really test it because of: > > > > rpmbuild -bc rpmbuild/vmtk/SPECS/vmtk.spec > > error: Failed build dependencies: > > InsightToolkit-devel is needed by vmtk-0.9.0-2.fc14.x86_64 > > vxl-devel is needed by vmtk-0.9.0-2.fc14.x86_64 > > > > Because of that I've added the review request of vxl and InsightToolkit as a > > blocker for this Review Request. > > I'm sorry, I forgot to mention that you need to build ITK and vxl for your > fedora system before you can test it. I for example, have a local mock repo > with these packages. Yup, I run a local repo as well. I'll try that. > > The only part of the project that appears to use OpenNL is the > > DifferentialGeometry sub-project. > > > > I've added a cmake option, USE_EXTERN_OPENNL to the main CMakeLists.txt and > > I'm > > working on getting the headers and linking working in the CMakeLists.txt for > > DifferentialGeometry. > > > The package works with the fedora OpenNL which I packaged recently. You don't > need to make any tweaks. The changes I've made in the prep and build sections > are enough. Well, I was trying for a few reasons. One, I'm trying to improve my cmake-fu :) Also, hacking the link.txt is not ideal. If I can come up with a good cmake patch then maybe upstream will accept it (which is the best case). Also, in retrospect, I'm wondering if we should have renamed the header file for OpenNL to be consistent with the library name, i.e.: nl.h -> opennl.h What do you think? Richard -- Configure 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 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 --- Comment #3 from Jon Ciesla 2011-07-20 11:47:15 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721057] Review Request: rubygem-sass - A powerful but elegant CSS compiler that makes CSS fun again
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721057 --- Comment #6 from Jon Ciesla 2011-07-20 11:48:13 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 719975] Review Request: rubygem-chunky_png - Pure ruby library for read/write, chunk-level access to PNG 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=719975 --- Comment #6 from Jon Ciesla 2011-07-20 11:46:52 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 720327] Review Request: perl-DynaLoader-Functions - Deconstructed dynamic C library loading
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720327 --- Comment #3 from Jon Ciesla 2011-07-20 11:47:45 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721059] Review Request: rubygem-dynamic_form - DynamicForm holds a few helper methods to help you deal with your Rails3 models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721059 --- Comment #5 from Jon Ciesla 2011-07-20 11:48:44 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721063] Review Request: rubygem-factory_girl_rails - factory_girl_rails provides integration between factory_girl and rails 3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721063 --- Comment #5 from Jon Ciesla 2011-07-20 11:49:14 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721066] Review Request: rubygem-imagefactory-console - QMF Console for Aeolus Image Factory
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721066 Mo Morsi changed: What|Removed |Added Summary|Review Request: |Review Request: |rubygem-image_factory_conso |rubygem-imagefactory-consol |le - QMF Console for Aeolus |e - QMF Console for Aeolus |Image Factory |Image Factory --- Comment #9 from Mo Morsi 2011-07-20 11:52:51 EDT --- Renamed the package Updated SPEC: http://mo.morsi.org/files/aeolus/rubygem-imagefactory-console.spec Updated SRPM: http://mo.morsi.org/files/aeolus/rubygem-imagefactory-console-0.4.0-4.fc15.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 723581] New: aeolus-configure - Aeolus Configure Puppet Recipe
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: aeolus-configure - Aeolus Configure Puppet Recipe https://bugzilla.redhat.com/show_bug.cgi?id=723581 Summary: aeolus-configure - Aeolus Configure Puppet Recipe Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: unspecified Priority: unspecified Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mmo...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, package-review@lists.fedoraproject.org Classification: Fedora Story Points: --- Spec: http://mo.morsi.org/files/aeolus/aeolus-configure.spec SRPM: http://mo.morsi.org/files/aeolus/aeolus-configure-2.0.1-2.src.rpm Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=3215056 Description: Aeolus Configure Puppet Recipe -- Configure 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 720323] Review Request: perl-Module-Runtime - Runtime module handling
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=720323 Petr Pisar changed: What|Removed |Added Status|ASSIGNED|MODIFIED Fixed In Version||perl-Module-Runtime-0.008-1 ||.fc16 --- Comment #4 from Petr Pisar 2011-07-20 12:01:30 EDT --- Thank you for the review and the repository. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 721059] Review Request: rubygem-dynamic_form - DynamicForm holds a few helper methods to help you deal with your Rails3 models
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721059 Mo Morsi changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-07-20 12:06:36 --- Comment #6 from Mo Morsi 2011-07-20 12:06:36 EDT --- Pushed and built against 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 721063] Review Request: rubygem-factory_girl_rails - factory_girl_rails provides integration between factory_girl and rails 3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721063 Mo Morsi changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-07-20 12:08:05 --- Comment #6 from Mo Morsi 2011-07-20 12:08:05 EDT --- Pushed and built against 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 721057] Review Request: rubygem-sass - A powerful but elegant CSS compiler that makes CSS fun again
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721057 Mo Morsi changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Last Closed||2011-07-20 12:08:14 --- Comment #7 from Mo Morsi 2011-07-20 12:08:14 EDT --- Pushed and built against 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 721144] Review Request: imagefactory - System image generation tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=721144 --- Comment #8 from Jon Ciesla 2011-07-20 11:49:40 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review