[Bug 651613] Review Request: haddock - Haskell documentation 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=651613 --- Comment #3 from Lakshmi Narasimhan 2010-11-21 03:06:04 EST --- SPEC file link : https://sites.google.com/site/lakshminaras2002/home/haddock.spec?attredirects=0&d=1 SRPM link https://sites.google.com/site/lakshminaras2002/home/haddock-2.7.2-2.fc14.src.rpm?attredirects=0&d=1 rpmlint output: ghc-haddock.x86_64: W: spelling-error Summary(en_US) Haskell -> Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-haddock-devel.x86_64: W: spelling-error Summary(en_US) Haskell -> Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. ghc-haddock-prof.x86_64: E: devel-dependency ghc-haddock-devel Your package has a dependency on a devel package but it's not a devel package itself. ghc-haddock-prof.x86_64: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. ghc-haddock-prof.x86_64: W: devel-file-in-non-devel-package /usr/lib64/ghc-6.12.3/haddock-2.7.2/libHShaddock-2.7.2_p.a A development file (usually source code) is located in a non-devel package. If you want to include source code in your package, be sure to create a development package. haddock.x86_64: W: spelling-error Summary(en_US) Haskell -> Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. haddock.x86_64: W: no-manual-page-for-binary haddock-2.7.2 Each executable in standard binary directories should have a man page. haddock.src: W: spelling-error Summary(en_US) Haskell -> Gaskell, Gaitskell, Skellum The value of this tag appears to be misspelled. Please double-check. 5 packages and 1 specfiles checked; 1 errors, 7 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 630223] Review Request: ghc-failure - A simple type class for success/failure computations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630223 Lakshmi Narasimhan changed: What|Removed |Added Flag||needinfo?(maths...@gmail.co ||m) --- Comment #1 from Lakshmi Narasimhan 2010-11-21 02:44:51 EST --- SRPM link is not accessible. -- Configure 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 554103] Review Request: fgrun - Graphical frontend for launching FlightGear flight simulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=554103 Hans de Goede changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|hdego...@redhat.com Flag||fedora-review? --- Comment #16 from Hans de Goede 2010-11-21 04:13:35 EST --- Hi, Full review done: Good: - rpmlint checks return: fgrun.x86_64: W: dangling-relative-symlink /usr/bin/fgrun-wrapper opengl-game-wrapper.sh fgrun.x86_64: W: no-manual-page-for-binary fgrun fgrun.x86_64: W: no-manual-page-for-binary fgrun-wrapper 3 packages and 0 specfiles checked; 0 errors, 3 warnings. These can all be ignored. - package meets naming guidelines - license (GPLv2+ and CC-BY-SA) OK, text in %doc, matches source - spec file legible, in am. english - source matches upstream - package compiles on devel (x86) - no missing BR - no unnecessary BR - no locales - not relocatable - no duplicate files - permissions ok - %clean ok - macro use consistent - code, not content - no need for -docs - nothing in %doc affects runtime - .desktop file as needed and correctly installed Needs work: - Is missing a "Requires: hicolor-icon-theme" (for /usr/share/icons/hicolor dir hierarchy ownership). - Icon cache update scripts are of the old variant, see: http://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Icon_Cache For the new style - .desktop file: - contains obsolete: "Encoding=UTF-8" (remove please) - "Icon=fgrun.png" should be "Icon=fgrun", esp. as the package also ships an svg - %files contains: "%defattr(-,root,root)" please use "%defattr(-,root,root.-)" instead Regards, Hans -- Configure 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 527549] Review Request: osm2go - A simple openstreetmap editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=527549 --- Comment #30 from Fabian Affolter 2010-11-21 04:23:10 EST --- There is a patch or patches needed to get rid of the Maemo Mapper because this tool is, like osm2go, initially developed for Maemo and depends indirectly on other parts that are available for the Maemo platform. If you want to help, please provide the needed patches. -- Configure 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 655486] New: Review Request: ghc-recaptcha - A Haskell library to use reCAPTCHA service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: ghc-recaptcha - A Haskell library to use reCAPTCHA service Alias: ghc-recaptcha https://bugzilla.redhat.com/show_bug.cgi?id=655486 Summary: Review Request: ghc-recaptcha - A Haskell library to use reCAPTCHA service Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lakshminaras2...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com, fedora-haskell-l...@redhat.com Estimated Hours: 0.0 Classification: Fedora SPEC URL : https://sites.google.com/site/lakshminaras2002/home/ghc-recaptcha.spec?attredirects=0&d=1 SRPM URL: https://sites.google.com/site/lakshminaras2002/home/ghc-recaptcha-0.1-1.fc14.src.rpm?attredirects=0&d=1 rpmlint output: rpmlint -i ghc-recaptcha-0.1-1.fc14.x86_64.rpm ghc-recaptcha-devel-0.1-1.fc14.x86_64.rpm ghc-recaptcha-prof-0.1-1.fc14.x86_64.rpm ../../SRPMS/ghc-recaptcha-0.1-1.fc14.src.rpm ghc-recaptcha.spec ghc-recaptcha.x86_64: W: spelling-error %description -l en_US hs -> HS, sh, gs The value of this tag appears to be misspelled. Please double-check. ghc-recaptcha-devel.x86_64: W: spelling-error %description -l en_US hs -> HS, sh, gs The value of this tag appears to be misspelled. Please double-check. ghc-recaptcha-prof.x86_64: E: devel-dependency ghc-recaptcha-devel Your package has a dependency on a devel package but it's not a devel package itself. ghc-recaptcha-prof.x86_64: W: spelling-error %description -l en_US hs -> HS, sh, gs The value of this tag appears to be misspelled. Please double-check. ghc-recaptcha-prof.x86_64: W: no-documentation The package contains no documentation (README, doc, etc). You have to include documentation files. ghc-recaptcha-prof.x86_64: W: devel-file-in-non-devel-package /usr/lib64/ghc-6.12.3/recaptcha-0.1/libHSrecaptcha-0.1_p.a A development file (usually source code) is located in a non-devel package. If you want to include source code in your package, be sure to create a development package. ghc-recaptcha.src: W: spelling-error %description -l en_US hs -> HS, sh, gs The value of this tag appears to be misspelled. Please double-check. 4 packages and 1 specfiles checked; 1 errors, 6 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 655486] Review Request: ghc-recaptcha - A Haskell library to use reCAPTCHA service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655486 --- Comment #1 from Lakshmi Narasimhan 2010-11-21 05:13:42 EST --- gitit package is dependent on this 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 630223] Review Request: ghc-failure - A simple type class for success/failure computations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=630223 Lakshmi Narasimhan changed: What|Removed |Added Flag|needinfo?(maths...@gmail.co | |m) | --- Comment #2 from Lakshmi Narasimhan 2010-11-21 05:18:33 EST --- The link is now accessible. Clearing the needinfo flag. Will provide a review asap. -- Configure 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 655379] Review Request: tritonus - Java Sound API Implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=655379
Hans de Goede changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review?
--- Comment #1 from Hans de Goede 2010-11-21 05:31:03 EST
---
Full review done:
Good:
- rpmlint checks return:
tritonus.src: W: invalid-url Source0: tritonus-0.3.7-20101108cvs.tar.bz2
tritonus-esd.x86_64: W: spelling-error Summary(en_US) javax -> java, java x,
Javan
tritonus-esd.x86_64: W: spelling-error Summary(en_US) libesd -> libelous,
libelist, libero
tritonus-esd.x86_64: W: spelling-error %description -l en_US javax -> java,
java x, Javan
tritonus-esd.x86_64: W: spelling-error %description -l en_US libesd ->
libelous, libelist, libero
tritonus-javadoc.x86_64: W: spelling-error Summary(en_US) Javadocs -> Java
docs, Java-docs, Javanese
tritonus-src.x86_64: W: only-non-binary-in-usr-lib
tritonus-src.x86_64: W: no-documentation
8 packages and 0 specfiles checked; 0 errors, 8 warnings.
The first 6 can be ignored, the last 2 go away once the -src package is folded
into the main package (see below).
- package meets naming guidelines
- package meets packaging guidelines
- license (LGPLv2+) OK, text in %doc, matches source
Note the gsm code is an exception license wise, see below.
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- nothing in %doc affects runtime
- no need for .desktop file
Needs work:
- Please remove the following no longer valid comment from the specfile:
# The following circular dependencies are intentional. These classes need each
other.
- Please fix the indentation of the following provides to be like that of the
other provides / tags:
Provides: %{name}-shared = %{version}-%{release}
- Please fold the -src subpackage into the main package (it is tiny and brings
in no external dependencies)
- Please re-introduce the -gsm subpackage, sorry about this, but the gsm java
code is licensed under the GPLv2+, rather then the LGPLv2+, so it needs to be
packaged separately. Do not forget to add a:
License:GPLv2+
line to the subpackage
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639636] Review Request: rubygem-poppler - Ruby binding of poppler-glib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639636 --- Comment #6 from Marek Goldmann 2010-11-21 05:56:10 EST --- Looks good for me, but I'm not an expert. I installed it and used pdf2text.rb example located here: http://ruby-gnome2.cvs.sourceforge.net/viewvc/ruby-gnome2/ruby-gnome2/poppler/sample/pdf2text.rb?view=markup – works! -- Configure 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 655496] Review Request: cambozola - A viewer for multipart jpeg streams
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655496 Cédric OLIVIER changed: What|Removed |Added Blocks||654052 -- Configure 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 527549] Review Request: osm2go - A simple openstreetmap editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=527549 --- Comment #32 from Fabian Affolter 2010-11-21 06:46:04 EST --- (In reply to comment #31) > I managed to compile and run osm2go on Fedora 14 without issues with these > steps: The package is working fine. The issue is that the Wizard has no functionality due the missing Maemo Mapper and missing implementation. This is the blocker. Martin has done a review. So if you are going to review the package and have no problem with the missing 'Wizard', the package will be in Fedora soon. If not, the package is going nowhere without the patches. -- Configure 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 655496] New: Review Request: cambozola - A viewer for multipart jpeg streams
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: cambozola - A viewer for multipart jpeg streams https://bugzilla.redhat.com/show_bug.cgi?id=655496 Summary: Review Request: cambozola - A viewer for multipart jpeg streams Product: Fedora Version: rawhide Platform: Unspecified OS/Version: Unspecified Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cedric.oliv...@free.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora SPEC URL : http://cedric.olivier.free.fr/rpms/cambozola-0.80-1/cambozola.spec SRPM URL : http://cedric.olivier.free.fr/rpms/cambozola-0.80-1/cambozola-0.80-1.fc13.src.rpm Description: Cambozola is a very simple (cheesey!) viewer for multipart jpeg streams that are often pumped out by a streaming webcam server, sending over multiple images per second. rpmlint output : cambozola.src: W: spelling-error Summary(en_US) multipart -> multiparty, multipara, multipartite cambozola.src: W: spelling-error Summary(en_US) jpeg -> peg, j peg, MPEG cambozola.src: W: spelling-error %description -l en_US cheesey -> cheese, cheesy, cheeses cambozola.src: W: spelling-error %description -l en_US multipart -> multiparty, multipara, multipartite cambozola.src: W: spelling-error %description -l en_US jpeg -> peg, j peg, MPEG cambozola.src: W: spelling-error %description -l en_US webcam -> web cam, web-cam, webcast 1 packages and 0 specfiles checked; 0 errors, 6 warnings. mock build : SRPM builds fine in mock. -- Configure 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 527549] Review Request: osm2go - A simple openstreetmap editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=527549 --- Comment #33 from Valent Turkovic 2010-11-21 06:50:16 EST --- Ok, thank you for explaining. -- Configure 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 527549] Review Request: osm2go - A simple openstreetmap editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=527549 --- Comment #31 from Valent Turkovic 2010-11-21 06:26:47 EST --- I managed to compile and run osm2go on Fedora 14 without issues with these steps: yum install goocanvas-devel libgnome-devel libsoup-devel libsoup22-devel svn checkout https://vcs.maemo.org/svn/osm2go/trunk cd trunk ./configure make ./src/osm2go -- Configure 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 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 Cédric OLIVIER 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 632970] Review Request: rec-applet - An audio recording applet for the GNOME-desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=632970 --- Comment #14 from Cédric OLIVIER 2010-11-21 07:59:58 EST --- New Package SCM Request === Package Name: rec-applet Short Description: An audio recording applet Owners: cquad Branches: F-13 F-14 InitialCC: cquad -- Configure 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 655486] Review Request: ghc-recaptcha - A Haskell library to use reCAPTCHA service
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655486 Lakshmi Narasimhan changed: What|Removed |Added Blocks||634048(Haskell-pkg-reviews) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655527] Review Request: pyside-tools - Development tools for PySide
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655527 --- Comment #1 from Kalev Lember 2010-11-21 09:36:22 EST --- Scratch build for rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=2614146 -- Configure 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 638590] Review Request: freemind - a mind-mapping software written in Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=638590 --- Comment #11 from hannes 2010-11-21 09:35:19 EST --- I've found some time today and tried to fix everything. SPEC URL: http://hannes.fedorapeople.org/freemind/freemind.spec SRPM URL: http://hannes.fedorapeople.org/freemind/freemind-0.9.0-0.3.rc10.fc14.src.rpm I also started a koji build but the weird thing is that it won't build on koji but builds just fine in my local mock environment: http://koji.fedoraproject.org/koji/taskinfo?taskID=2614142 rpmlint freemind.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. mock -r fedora-rawhide-x86_64 rebuild /home/johannes/rpmbuild/SRPMS/freemind-0.9.0-0.3.rc10.fc14.src.rpm INFO: mock.py version 1.1.6 starting... State Changed: init plugins INFO: selinux disabled State Changed: start INFO: Start(/home/johannes/rpmbuild/SRPMS/freemind-0.9.0-0.3.rc10.fc14.src.rpm) Config(fedora-rawhide-x86_64) State Changed: lock buildroot State Changed: clean INFO: chroot (/var/lib/mock/fedora-rawhide-x86_64) unlocked and deleted State Changed: init State Changed: lock buildroot Mock Version: 1.1.6 INFO: Mock Version: 1.1.6 INFO: enabled root cache State Changed: unpacking root cache INFO: enabled yum cache State Changed: cleaning yum metadata INFO: enabled ccache State Changed: running yum State Changed: setup State Changed: build INFO: Done(/home/johannes/rpmbuild/SRPMS/freemind-0.9.0-0.3.rc10.fc14.src.rpm) Config(fedora-rawhide-x86_64) 6 minutes 48 seconds INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-x86_64/result -- Configure 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 652544] Review Request: erlang-rpm-macros - Macros for simplifying building of Erlang packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652544 --- Comment #10 from Fedora Update System 2010-11-21 09:56:35 EST --- erlang-rpm-macros-0.1.2-2.fc14,erlang-R14B-0.5.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/erlang-rpm-macros-0.1.2-2.fc14,erlang-R14B-0.5.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 655527] New: Review Request: pyside-tools - Development tools for PySide
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pyside-tools - Development tools for PySide https://bugzilla.redhat.com/show_bug.cgi?id=655527 Summary: Review Request: pyside-tools - Development tools for PySide Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ka...@smartlink.ee QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://kalev.fedorapeople.org/pyside-tools.spec SRPM URL: http://kalev.fedorapeople.org/pyside-tools-0.2.2-1.fc15.src.rpm Description: PySide provides Python bindings for the Qt cross-platform application and UI framework. This package ships the following accompanying tools: * pyside-rcc - PySide resource compiler * pyside-uic - Python User Interface Compiler for PySide * pyside-lupdate - update Qt Linguist translation files for PySide -- Configure 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 549590] Review Request: php-pChart - A PHP class to build charts.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=549590 --- Comment #17 from Eric Christensen 2010-11-21 10:29:03 EST --- Okay, I've added php-common to be required. I believe one of the fonts is already in Fedora but the others are not. I don't feel that I'm qualified to recommend alternate fonts. -- Configure 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 655547] New: Review Request: drupal-advanced_help - Advanced Help module for Drupal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: drupal-advanced_help - Advanced Help module for Drupal https://bugzilla.redhat.com/show_bug.cgi?id=655547 Summary: Review Request: drupal-advanced_help - Advanced Help module for Drupal Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jaredsm...@jaredsmith.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://jsmith.fedorapeople.org/Packaging/Drupal/drupal-advanced_help/drupal-advanced_help.spec SRPM URL: http://jsmith.fedorapeople.org/Packaging/Drupal/drupal-advanced_help/drupal-advanced_help-6.x.1.2-1.fc14.src.rpm Description: The advanced help module for Drupal allows module developers to store their help outside the module system, in pure .html files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655547] Review Request: drupal-advanced_help - Advanced Help module for Drupal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655547 Jared Smith changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630283] Review Request: ghc-neither - Either with monad and applicative 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=630283 --- Comment #4 from Lakshmi Narasimhan 2010-11-21 11:19:56 EST --- Will set the fedora-review flag once ghc-failure is approved. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655547] Review Request: drupal-advanced_help - Advanced Help module for Drupal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655547 Christoph Wickert changed: What|Removed |Added Status|NEW |ASSIGNED CC||cwick...@fedoraproject.org AssignedTo|nob...@fedoraproject.org|cwick...@fedoraproject.org Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655184] Review Request: drupal6-data - Data module helps you model, manage and query related sets of tables.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655184 Eric Christensen changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|e...@christensenplace.us Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 549590] Review Request: php-pChart - A PHP class to build charts.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=549590 --- Comment #18 from Jeffrey Ness 2010-11-21 11:59:46 EST --- Hey Eric, The Development RPM I have been using for pChart requires "liberation-fonts", maybe you can remove the Fonts from you package and rely on it? Just a thought. Jeffrey- -- Configure 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 655547] Review Request: drupal-advanced_help - Advanced Help module for Drupal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655547 Christoph Wickert changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Flag|fedora-review? | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655547] Review Request: drupal-advanced_help - Advanced Help module for Drupal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655547 Christoph Wickert changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||DUPLICATE Last Closed||2010-11-21 12:32:05 --- Comment #1 from Christoph Wickert 2010-11-21 12:32:05 EST --- *** This bug has been marked as a duplicate of bug 642857 *** -- Configure 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 642856] Review Request: drupal6-flexifilter - Allows non-admins to create their own, configurable custom filters
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642856
Christoph Wickert changed:
What|Removed |Added
CC||cwick...@fedoraproject.org
--- Comment #14 from Christoph Wickert 2010-11-21
12:33:36 EST ---
po files are not tagged with %lang but simply included in %files. Better use
%find_lang %{name} --all-name. Also affects the other drupal module packages.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 642858] Review Request: drupal6-footnotes - Allows to easily create automatically numbered footnote references
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=642858
Christoph Wickert changed:
What|Removed |Added
CC||cwick...@fedoraproject.org
--- Comment #5 from Christoph Wickert 2010-11-21
12:40:18 EST ---
(In reply to comment #4)
> [ N/A ] MUST: The spec file MUST handle locales properly. This is done by
> using
> the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
In fact this point does apply here as the package includes
/usr/share/drupal/modules/footnotes/translations/fr.po
/usr/share/drupal/modules/footnotes/translations/ja.po
Better use %find_lang %{name} --all-name.
%define should be %global.
--
Configure 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 653805] Review Request: drupal6-ctools - This suite is primarily a set of APIs and tools to improve the developer experience.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=653805 Nick Bebout changed: What|Removed |Added CC||n...@fedoraproject.org Blocks|177841(FE-NEEDSPONSOR) | --- Comment #5 from Nick Bebout 2010-11-21 14:01:15 EST --- I've sponsored asrob so I'm unblocking FE-NEEDSPONSOR -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 608332] Review Request: rootplot - Plots ROOT data with matplotlib
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=608332
--- Comment #10 from Steve Traylen 2010-11-21 14:18:22
EST ---
Interesting I had always assumed that
> %{python_sitelib}/rootplot
>or
> %{python_sitelib}/rootplot/
were different but you are correct the second one does include the directory
as well.
Have opted for
%{python_sitelib}/rootplot/
for the same reasons as you suggest.
http://cern.ch/straylen/rpms/rootplot/rootplot.spec
http://cern.ch/straylen/rpms/rootplot/rootplot-2.2.1-2.fc13.src.rpm
Will request in a couple of days.
Thanks again.
Steve.
--
Configure 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 639636] Review Request: rubygem-poppler - Ruby binding of poppler-glib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639636 --- Comment #7 from Mamoru Tasaka 2010-11-21 14:24:10 EST --- If this package seems good for you, I would appreciate it if you would assign this bug to yourself and approve this review request. We need more and more reviewers and trying reviewing tickets is highly appreciated. -- Configure 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 653971] Reviews Request: openfetion - A Fetion client written using GTK2+
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=653971
--- Comment #1 from Michael Schwendt 2010-11-21 14:34:08
EST ---
A brief look at the spec file told me the following:
> Summary: A Fetion client written using GTK2+
> %description
> Openfetion is a Fetion client written using GTK2+,
> based on Fetion v4 protocol.
Both %summary and %description could try to explain what "Fetion" is. For
example:
Summary: GTK2+ based client for the Fetion instant messaging protocol
And %description should expand on that. Just repeating a "Name" isn't helpful.
> %setup -q -n %{name}-%{version}
Option -n %{name}-%{version} is the default.
> %files -f %{name}.lang
> %defattr(-,root,root,-)
[...]
%{_datadir}/%{name}/resource/newmessage.wav
%{_datadir}/%{name}/skin/*
[...]
%{_datadir}/%{name}/resource/license.txt
Several directories are not included in the package correctly:
%{_datadir}/%{name}/
%{_datadir}/%{name}/resource/
%{_datadir}/%{name}/skin/
http://fedoraproject.org/wiki/Packaging:UnownedDirectories
As a directory added to a %files section will be included recursively, you
may need to include directories only with the %dir attribute.
> %files -n %{libname}
> %defattr(-,root,root,-)
> %{_datadir}/%{name}/resource/city.xml
> %{_datadir}/%{name}/resource/province.xml
Same here. Plus, you need to decide on directory ownership between this
%libname package and the main %name package.
> %{_libdir}/libofetion.so*
This is wrong. Only the *.so.* files belong into the %libname package. The *.so
symlink belongs into the -devel 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 639636] Review Request: rubygem-poppler - Ruby binding of poppler-glib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639636 Marek Goldmann changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|mgold...@redhat.com Flag||fedora-review+ --- Comment #8 from Marek Goldmann 2010-11-21 14:35:04 EST --- OK. This package (rubygem-poppler) is APPROVED by goldmann -- Configure 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 653407] Review Request: rubygem-hashery - A collection of Hash-like classes 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=653407 --- Comment #10 from Marek Goldmann 2010-11-21 14:44:27 EST --- Done! http://koji.fedoraproject.org/koji/taskinfo?taskID=2613007 -- Configure 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 653407] Review Request: rubygem-hashery - A collection of Hash-like classes 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=653407 Mamoru Tasaka changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||NEXTRELEASE Last Closed||2010-11-21 14:50:56 --- Comment #11 from Mamoru Tasaka 2010-11-21 14:50:56 EST --- Thank you. Now closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655582] New: Review Request: mod_cluster - Apache HTTPD based load balancer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mod_cluster - Apache HTTPD based load balancer https://bugzilla.redhat.com/show_bug.cgi?id=655582 Summary: Review Request: mod_cluster - Apache HTTPD based load balancer 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, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://goldmann.fedorapeople.org/package_review/mod_cluster/mod_cluster.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/mod_cluster/mod_cluster-1.1.0.Final-1.fc14.src.rpm Description: mod_cluster is an httpd-based load balancer. Like mod_jk and mod_proxy, mod_cluster uses a communication channel to forward requests from httpd to one of a set of application server nodes. Unlike mod_jk and mod_proxy, mod_cluster leverages an additional connection between the application server nodes and httpd. The application server nodes use this connection to transmit server-side load balance factors and lifecycle events back to httpd via a custom set of HTTP methods, affectionately called the Mod-Cluster Management Protocol (MCMP). This additional feedback channel allows mod_cluster to offer a level of intelligence and granularity not found in other load balancing solutions. Scratch build for rawhide: https://koji.fedoraproject.org/koji/taskinfo?taskID=2614547 -- Configure 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 654047] Review Request: drupal6-feeds - An import and aggregation framework for Drupal.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654047 Peter Borsa changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Peter Borsa 2010-11-21 15:37:21 EST --- New Package SCM Request === Package Name: drupal6-feeds Short Description: An import and aggregation framework for Drupal. Owners: asrob Branches: f14 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 655184] Review Request: drupal6-data - Data module helps you model, manage and query related sets of tables.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655184 Peter Borsa changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Peter Borsa 2010-11-21 15:39:23 EST --- New Package SCM Request === Package Name: drupal6-data Short Description: Data module helps you model, manage and query related sets of tables. Owners: asrob Branches: f14 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 639636] Review Request: rubygem-poppler - Ruby binding of poppler-glib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=639636 Mamoru Tasaka changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Mamoru Tasaka 2010-11-21 14:44:49 EST --- Thank you! New Package SCM Request === Package Name: rubygem-poppler Short Description: Ruby binding of poppler-glib Owners:mtasaka Branches: f13 f14 -- Configure 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 655581] Review Request: trang - Multi-format schema converter based on RELAX NG
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655581 Ville Skyttä changed: What|Removed |Added CC||mc...@redhat.com --- Comment #1 from Ville Skyttä 2010-11-21 15:02:29 EST --- *** Bug 230282 has been marked as a duplicate of this bug. *** -- Configure 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 654047] Review Request: drupal6-feeds - An import and aggregation framework for Drupal.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654047 Nick Bebout changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Flag|needinfo?(n...@fedoraproject. | |org)| --- Comment #8 from Nick Bebout 2010-11-21 13:59:08 EST --- asrob is sponsored now. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 654047] Review Request: drupal6-feeds - An import and aggregation framework for Drupal.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654047 Nick Bebout changed: What|Removed |Added Status|MODIFIED|ASSIGNED -- Configure 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 652400] Review Request: rubygem-boxgrinder-build - Creates appliances for various virtual environments
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652400 --- Comment #4 from Marek Goldmann 2010-11-21 15:39:34 EST --- Updated files: Spec URL: http://goldmann.fedorapeople.org/package_review/boxgrinder/rubygem-boxgrinder-build.spec SRPM URL: http://goldmann.fedorapeople.org/package_review/boxgrinder/rubygem-boxgrinder-build-0.6.4-2.fc14.src.rpm Koji scratch build for rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=2614586 -- Configure 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 230282] Review Request: trang - Multi-format schema converter based on RELAX NG
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=230282 Ville Skyttä changed: What|Removed |Added CC||ville.sky...@iki.fi Resolution|WONTFIX |DUPLICATE --- Comment #5 from Ville Skyttä 2010-11-21 15:02:29 EST --- *** This bug has been marked as a duplicate of bug 655581 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 655184] Review Request: drupal6-data - Data module helps you model, manage and query related sets of tables.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655184 Nick Bebout changed: What|Removed |Added CC||n...@fedoraproject.org Blocks|177841(FE-NEEDSPONSOR) | --- Comment #1 from Nick Bebout 2010-11-21 14:01:31 EST --- I've sponsored asrob so I'm unblocking FE-NEEDSPONSOR -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566964] Review Request: spooles - sparse matrix 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=566964 Manfred Spraul changed: What|Removed |Added Flag|needinfo?(manf...@colorfull | |ife.com)| --- Comment #5 from Manfred Spraul 2010-11-21 12:38:01 EST --- I'd like to, but I did not have the time to make further steps. My only progress since February is merge of upstream changes :-( Is is possible that you leave the bug reports open until after Christmas? http://sourceforge.net/projects/calculix-rpm/files/rel-2.2/ -- Configure 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 653805] Review Request: drupal6-ctools - This suite is primarily a set of APIs and tools to improve the developer experience.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=653805 Peter Borsa changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Peter Borsa 2010-11-21 15:35:25 EST --- New Package SCM Request === Package Name: drupal6-ctools Short Description: This suite is primarily a set of APIs and tools. Owners: asrob Branches: f14 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 655581] New: Review Request: trang - Multi-format schema converter based on RELAX NG
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: trang - Multi-format schema converter based on RELAX NG https://bugzilla.redhat.com/show_bug.cgi?id=655581 Summary: Review Request: trang - Multi-format schema converter based on RELAX NG Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ville.sky...@iki.fi QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora http://scop.fedorapeople.org/packages/trang.spec http://scop.fedorapeople.org/packages/trang-2009-3.fc14.src.rpm Trang converts between different schema languages for XML. It supports the following languages: RELAX NG (both XML and compact syntax), XML 1.0 DTDs, W3C XML Schema. A schema written in any of the supported schema languages can be converted into any of the other supported schema languages, except that W3C XML Schema is supported for output only, not for input. http://www.thaiopensource.com/relaxng/trang.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 652396] Review Request: rubygem-boxgrinder-core - Core files required by BoxGrinder
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652396 --- Comment #14 from Marek Goldmann 2010-11-21 14:58:15 EST --- Rawhide build: http://koji.fedoraproject.org/koji/taskinfo?taskID=2613414 -- Configure 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 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654909 --- Comment #5 from William Lima 2010-11-21 14:06:00 EST --- (In reply to comment #3) > I'll go ahead and review this and look at sponsoring you. nice, thanks Kevin! > Do you have any other packages to submit at this time? nope > Have you done any pre-reviews of other pending package reviews? > (see http://fedoraproject.org/PackageReviewStatus/NEW.html ). not yet, I'll take a look. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=647510
--- Comment #7 from Dan Horák 2010-11-21 16:05:17 EST ---
It may be also useful to install %{_bindir}/premake as a symlink to the
premake4 binary.
--
Configure 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 588941] Review Request: pypy - Implementation of the Python language, using Python itself
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=588941 --- Comment #10 from Kevin Fenzi 2010-11-21 16:02:51 EST --- Any news on 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 549590] Review Request: php-pChart - A PHP class to build charts.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=549590 --- Comment #20 from Jeffrey Ness 2010-11-21 16:17:23 EST --- Sure. My RPM isn't close to Redhat standard for approval, but it gets the job done for testing: http://flip-edesign.com/source/php-pChart/php-pChart.spec Jeffrey- -- Configure 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 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654909 --- Comment #7 from William Lima 2010-11-21 16:13:52 EST --- latest package files Spec URL: http://poetinha.fedorapeople.org/django-tables.spec SRPM URL: http://poetinha.fedorapeople.org/django-tables-0.2-2.fc13.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 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654909 --- Comment #6 from William Lima 2010-11-21 15:49:10 EST --- o fixed wrong-file-end-of-line-encoding issue o use python2-devel instead of python-devel -- Configure 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 654848] Review Request: apvlv - PDF viewer which behaves like Vim
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654848 Pierre Carrier changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Pierre Carrier 2010-11-21 13:06:40 EST --- New Package SCM Request === Package Name: apvlv Short Description: PDF viewer which behaves like Vim Owners: pcarrier Branches: f14 el6 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 647510] Review Request: premake - A cross-platform build configuration tool
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=647510
Dan Horák changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|d...@danny.cz
Flag||fedora-review?
--- Comment #6 from Dan Horák 2010-11-21 13:05:35 EST ---
formal review is here, see the notes explaining OK* and BAD statuses below:
BAD source files match upstream:
upstream: 1cd4bded80ecbe695ec309352241a00e6ce01c20 premake-4.2.1-src.zip
srpm: a8253af9c17f6e90811b896a2e941492cab4e861 premake-4.2.1-src.zip
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
BAD license field matches the actual license.
OK license is open source-compatible. License text included in package.
BAD latest version is being packaged.
BAD BuildRequires are proper.
OK compiler flags are appropriate.
BAD %clean is present.
BAD package builds in mock (Rawhide/x86_64).
OK debuginfo package looks complete.
BAD rpmlint is silent.
OK final provides and requires look sane.
N/A* %check is present and all tests pass.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK doesn't own any directories it shouldn't.
OK no duplicates in %files.
OK* file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.
OK not a GUI app.
- the freshly downloaded source archive doesn't contain the MacOS/X prebuild
binary
- the resulting license should be GPL+, see point 4 in
https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F
- version 4.3 was released few days ago (includes a switch from GPL to BSD)
- readline-devel must be added as BR
- you shouldn't clean %{_builddir}/%{buildsubdir} -
http://fedoraproject.org/wiki/PackagingGuidelines#.25clean
- rpmlint complains a bit:
premake.x86_64: W: no-version-in-last-changelog
=> the format of changelog is bad (must include version-release pair, also
add an empty line between entries) -
http://fedoraproject.org/wiki/PackagingGuidelines#Changelogs
premake.x86_64: W: no-manual-page-for-binary premake4
=> nice to have, but not a blocker
- a test suite is included in the sources, please check whether it can be run
during the build
- I'd use an explicit "-m 755" when installing the binary, you should also
remove the "%{_builddir}/%{buildsubdir}" part and use "." instead
--
Configure 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 553649] Review Request: rhn-custom-info - Set and list custom values for RHN-enabled machines
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553649 Garrett Holmstrom changed: What|Removed |Added Attachment #461352|0 |1 is obsolete|| --- Comment #11 from Garrett Holmstrom 2010-11-21 16:27:54 EST --- Created attachment 461873 --> https://bugzilla.redhat.com/attachment.cgi?id=461873 Review for new package rhn-custom-info-5.4.5-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 553649] Review Request: rhn-custom-info - Set and list custom values for RHN-enabled machines
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=553649 Garrett Holmstrom changed: What|Removed |Added Status|ASSIGNED|MODIFIED Flag|fedora-review? |fedora-review+ --- Comment #10 from Garrett Holmstrom 2010-11-21 16:26:55 EST --- That's exactly right. Everything looks good to go now. Note that when you make changes to just the spec file you don't need to create another upstream release with its own version number and tarball; you can just bump the Release field in the spec file and rebuild. -- Configure 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 566964] Review Request: spooles - sparse matrix 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=566964 --- Comment #6 from Dan Horák 2010-11-21 13:17:14 EST --- Thanks for the response and have no worries, the review can wait. -- Configure 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 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=654909
--- Comment #4 from Kevin Fenzi 2010-11-21 13:31:18 EST ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines. [1]
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Spec file is legible and written in American English.
[x] Spec file lacks Packager, Vendor, PreReq tags.
[x] Spec uses macros instead of hard-coded directory names.
[x] Package consistently uses macros.
[x] Macros in Summary, %description expandable at SRPM build time.
[x] PreReq is not used.
[x] Requires correct, justified where necessary.
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines. [2]
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)).
[x] Package run rm -rf %{buildroot} (or $RPM_BUILD_ROOT) and the beginning of
%install.
[x] Package use %makeinstall only when ``make install DESTDIR=...'' doesn't
work.
[x] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
[x] The spec file handles locales properly.
[x] Changelog in prescribed format.
[!] Rpmlint output is silent.
[x] License field in the package spec file matches the actual license.
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] License file installed when any subpackage combination is installed.
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[3,4]
[x] Sources contain only permissible code or content.
[!] Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package : 99bf7894c17a3312cdf634bb17b6eb4b
MD5SUM upstream package : 457973f47ac0cf3fc843fbd24ece9846
[?] Compiler flags are appropriate.
[?] %build honors applicable compiler flags or justifies otherwise.
[?] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package does not own files or directories owned by other packages.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Each %files section contains %defattr.
[?] No %config files under /usr.
[?] %config files are marked noreplace or the reason is justified.
[?] Package contains a properly installed %{name}.desktop using
desktop-file-install file if it is a GUI application. [5]
[?] Package contains a valid .desktop file.
[x] Package contains code, or permissable content.
[?] Package contains a SysV-style init script if in need of one.
[!] File names are valid UTF-8.
[?] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[x] Package contains no bundled libraries.
[x] Package does not genrate any conflict.
[x] Package does not contains kernel modules.
[x] Package is not relocatable.
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
[x] Package is not known to require ExcludeArch.
[x] Package installs properly.
[x] Package obeys FHS, except libexecdir and /usr/target.
[x] Package meets the Packaging Guidelines. [6]
=== SUGGESTED ITEMS ===
[?] Package functions as described.
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[x] If the source package does not include license text(s) as a separate file
from upstream, the packager SHOULD query upstream to include it.
[?] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[!] SourceX is a working URL.
[!] SourceX / PatchY prefixed with %{name}.
[x] Final provides and requires are sane (rpm -q --provides and rpm -q
--requires).
[!] %check is present and all tests pass.
[x] Reviewer should test that the package builds in mock.
[x] Package should compile and build into binary rpms on all supported
architectures.
[x] Dist tag is present.
[x] Spec use %global instead of %define.
[x] Scriptlets must be sane, if used.
[x] No file requires outside of /etc, /bin, /sbin, /usr/bin, /usr/sbin.
[x] Packages should try to preserve timestamps of original installed files.
[x] Uses parallel make.
=== Issues ===
1. rpmlint says:
django-tables.noarch: E: zero-length
/usr/lib/python2.7/site-packages/django_tables/app/models.py
I assume that 0 len file is usefull for something?
django-tabl
[Bug 654047] Review Request: drupal6-feeds - An import and aggregation framework for Drupal.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=654047
Eric Christensen changed:
What|Removed |Added
Status|ASSIGNED|MODIFIED
Flag|fedora-review? |fedora-review+,
||needinfo?(n...@fedoraproject.
||org)
--- Comment #7 from Eric Christensen 2010-11-21
11:34:06 EST ---
OK* MUST: rpmlint must be run on the source rpm and all binary rpms the
build produces. The output should be posted in the review.[1]
[u...@mini Desktop]$ rpmlint drupal6-feeds.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[u...@mini Desktop]$ rpmlint drupal6-feeds-1.0-0.1.beta10.fc14.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
OK* MUST: The package must be named according to the Package Naming
Guidelines .
OK* MUST: The spec file name must match the base package %{name}, in the
format %{name}.spec unless your package has an exemption. [2] .
OK* MUST: The package must meet the Packaging Guidelines .
OK* MUST: The package must be licensed with a Fedora approved license and
meet the Licensing Guidelines .
OK* MUST: The License field in the package spec file must match the actual
license. [3]
OK* MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.[4]
OK* MUST: The spec file must be written in American English. [5]
OK* MUST: The spec file for the package MUST be legible. [6]
ok* MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Reviewers should use md5sum for this task.
If no upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
OK* MUST: The package MUST successfully compile and build into binary rpms
on at least one primary architecture. [7]
OK* MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line. [8]
OK* MUST: All build dependencies must be listed in BuildRequires, except
for any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
OK* MUST: The spec file MUST handle locales properly. This is done by using
the %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.[9]
OK* MUST: Every binary RPM package (or subpackage) which stores shared
library files (not just symlinks) in any of the dynamic linker's default paths,
must call ldconfig in %post and %postun. [10]
OK* MUST: Packages must NOT bundle copies of system libraries.[11]
OK* MUST: If the package is designed to be relocatable, the packager must
state this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker. [12]
OK* MUST: A package must own all directories that it creates. If it does
not create a directory that it uses, then it should require a package which
does create that directory. [13]
OK* MUST: A Fedora package must not list a file more than once in the spec
file's %files listings. (Notable exception: license texts in specific
situations)[14]
OK* MUST: Permissions on files must be set properly. Executables should be
set with executable permissions, for example. Every %files section must include
a %defattr(...) line. [15]
OK* MUST: Each package must consistently use macros. [16]
OK* MUST: The package must contain code, or permissable content. [17]
OK* MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity). [18]
OK* MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present. [18]
OK* MUST: Header files must be in a -devel package. [19]
OK* MUST: Static libraries must be in a -static package. [20]
OK* MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package. [19]
OK* MUST: In the vast maj
[Bug 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654909 Kevin Fenzi changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|ke...@tummy.com Flag||fedora-review? --- Comment #3 from Kevin Fenzi 2010-11-21 12:25:41 EST --- I'll go ahead and review this and look at sponsoring you. Do you have any other packages to submit at this time? Have you done any pre-reviews of other pending package reviews? (see http://fedoraproject.org/PackageReviewStatus/NEW.html ). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 630283] Review Request: ghc-neither - Either with monad and applicative 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=630283
Bug 630283 depends on bug 630270, which changed state.
Bug 630270 Summary: Review Request: ghc-transformers - Concrete functor and
monad transformers
https://bugzilla.redhat.com/show_bug.cgi?id=630270
What|Old Value |New Value
Resolution||ERRATA
Status|ON_QA |CLOSED
--- Comment #3 from Lakshmi Narasimhan 2010-11-21
11:18:59 EST ---
[+]MUST: rpmlint must be run on every package. The output should be posted in
the review.
rpmlint -i ghc-neither-* ../../SRPMS/ghc-neither-0.1.0-1.fc14.src.rpm
~/ghc-neither.spec
ghc-neither.x86_64: W: spelling-error %description -l en_US datatype -> data
type, data-type, database
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.x86_64: W: spelling-error %description -l en_US mtl -> mt, ml, mel
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.x86_64: W: spelling-error %description -l en_US datatypes -> data
types, data-types, databases
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.x86_64: W: spelling-error %description -l en_US AEither -> A
Either, Aether, Either
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-devel.x86_64: W: spelling-error %description -l en_US datatype ->
data type, data-type, database
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-devel.x86_64: W: spelling-error %description -l en_US mtl -> mt,
ml, mel
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-devel.x86_64: W: spelling-error %description -l en_US datatypes ->
data types, data-types, databases
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-devel.x86_64: W: spelling-error %description -l en_US AEither -> A
Either, Aether, Either
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-prof.x86_64: E: devel-dependency ghc-neither-devel
Your package has a dependency on a devel package but it's not a devel package
itself.
ghc-neither-prof.x86_64: W: spelling-error %description -l en_US datatype ->
data type, data-type, database
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-prof.x86_64: W: spelling-error %description -l en_US mtl -> mt, ml,
mel
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-prof.x86_64: W: spelling-error %description -l en_US datatypes ->
data types, data-types, databases
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-prof.x86_64: W: spelling-error %description -l en_US AEither -> A
Either, Aether, Either
The value of this tag appears to be misspelled. Please double-check.
ghc-neither-prof.x86_64: W: no-documentation
The package contains no documentation (README, doc, etc). You have to include
documentation files.
ghc-neither-prof.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/ghc-6.12.3/neither-0.1.0/libHSneither-0.1.0_p.a
A development file (usually source code) is located in a non-devel package. If
you want to include source code in your package, be sure to create a
development package.
ghc-neither.src: W: spelling-error %description -l en_US datatype -> data type,
data-type, database
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.src: W: spelling-error %description -l en_US mtl -> mt, ml, mel
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.src: W: spelling-error %description -l en_US datatypes -> data
types, data-types, databases
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.src: W: spelling-error %description -l en_US AEither -> A Either,
Aether, Either
The value of this tag appears to be misspelled. Please double-check.
ghc-neither.src: W: strange-permission ghc-neither.spec 0640L
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.
ghc-neither.src: W: strange-permission neither-0.1.0.tar.gz 0640L
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.
4 packages and 1 specfiles checked; 1 errors, 20 warnings.
[+]MUST: The package must be named according to the Package Naming Guidelines.
[+]MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec
[+]MUST: The package must meet the Packaging Guidelines.
Naming-Yes
Version-release - Matches
License - OK, BSD 2 clause variant
No prebuilt external bits - OK
Spec legibity - OK
Package template -
[Bug 648053] Review Request: libMicro - Portable micro-benchmarks for various system and library calls
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=648053
--- Comment #2 from Michael Schwendt 2010-11-21 14:54:58
EST ---
Run rpmlint on the src.rpm and your built rpms. It's one of the things a
packager ought to do. The ReviewGuidelines are not specific to the reviewer(s).
;)
> %{_libexecdir}/%{name}/*
What package owns %{_libexecdir}/%{name} then?
http://fedoraproject.org/wiki/Packaging:UnownedDirectories
> I don't think we should use %{optflags} on this one, we don't want
> to impact the benchmark. Opinions appreciated.
Not using %optflags and hence also creating a bad -debuginfo package doesn't
fly.
How much overhead do you expect from the code that does the measuring compared
with the actual execution time of the system calls?
--
Configure 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 652396] Review Request: rubygem-boxgrinder-core - Core files required by BoxGrinder
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=652396 Bug 652396 depends on bug 653407, which changed state. Bug 653407 Summary: Review Request: rubygem-hashery - A collection of Hash-like classes for Ruby https://bugzilla.redhat.com/show_bug.cgi?id=653407 What|Old Value |New Value Resolution||NEXTRELEASE Status|ON_QA |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 549590] Review Request: php-pChart - A PHP class to build charts.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=549590 --- Comment #19 from David Nalley 2010-11-21 12:50:35 EST --- (In reply to comment #17) > Okay, I've added php-common to be required. > > I believe one of the fonts is already in Fedora but the others are not. I > don't feel that I'm qualified to recommend alternate fonts. I don't know that you have to recommend alternate fonts, but the package won't work properly has it has fonts coded. Jeffrey: Care to share the spec for the development RPM, perhaps someone else has seen this and come up with a novel solution (such as using liberation fonts) -- Configure 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 642857] Review Request: drupal6-advanced-help - Allows module developers to store their help outside the module system in html
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=642857 Christoph Wickert changed: What|Removed |Added CC||jaredsm...@jaredsmith.net --- Comment #2 from Christoph Wickert 2010-11-21 12:32:05 EST --- *** Bug 655547 has been marked as a duplicate of this bug. *** -- Configure 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 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=470696 --- Comment #77 from Jeroen van Meeuwen 2010-11-21 11:29:47 EST --- I'm working on getting the necessary people to understand this package needs an exception in order to get Real Life Problems fixed. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 654909] Review Request: django-tables - A Django Queryset renderer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654909 Kevin Fenzi changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | Flag|fedora-review? |fedora-review+ --- Comment #8 from Kevin Fenzi 2010-11-21 17:04:47 EST --- Per irc you were going to check with upstream on including the test app as a doc. We could wait to hear from them on that before importing this package. Otherwise I dont see any further blockers, so this package is APPROVED. I'll go ahead and sponsor you. If you have any questions or issues, please don't hesitate to ask me here, email or via irc. -- Configure 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 655599] Review Request: remmina-plugins - Plugins for Remmina Remote Desktop Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655599 Christoph Wickert changed: What|Removed |Added Blocks||617144 -- Configure 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 655599] New: Review Request: remmina-plugins - Plugins for Remmina Remote Desktop Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: remmina-plugins - Plugins for Remmina Remote Desktop Client https://bugzilla.redhat.com/show_bug.cgi?id=655599 Summary: Review Request: remmina-plugins - Plugins for Remmina Remote Desktop Client Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cwick...@fedoraproject.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://cwickert.fedorapeople.org/review/remmina-plugins.spec SRPM URL: http://cwickert.fedorapeople.org/review/remmina-plugins-0.8.4-1.fc14.src.rpm Description: Remmina is a remote desktop client written in GTK+, aiming to be useful for system administrators and travelers, who need to work with lots of remote computers in front of either large monitors or tiny netbooks. Remmina supports multiple network protocols in an integrated and consistent user interface. Currently NX, RDP, Telepathy, VNC, XDMCP and SSH are supported. This package contains the plugins for Remmina. Note: In order to build this package you will also need remmina. -- Configure 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 655599] Review Request: remmina-plugins - Plugins for Remmina Remote Desktop Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=655599 --- Comment #1 from Christoph Wickert 2010-11-21 17:31:21 EST --- (In reply to comment #0) > Note: In order to build this package you will also need remmina. That should read "you also remmina-0.8.3" http://cwickert.fedorapeople.org/review/remmina-0.8.3-1.fc14.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 655601] New: Review Request: jing - RELAX NG validator in Java
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: jing - RELAX NG validator in Java https://bugzilla.redhat.com/show_bug.cgi?id=655601 Summary: Review Request: jing - RELAX NG validator in Java Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ville.sky...@iki.fi QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora http://scop.fedorapeople.org/packages/jing.spec http://scop.fedorapeople.org/packages/jing-2009-3.fc14.src.rpm Jing is a RELAX NG validator written in Java. It implements the RELAX NG 1.0 Specification, RELAX NG Compact Syntax, and parts of RELAX NG DTD Compatibility, specifically checking of ID/IDREF/IDREFS. It also has experimental support for schema languages other than RELAX NG; specifically W3C XML Schema, Schematron 1.5, and Namespace Routing Language. http://www.thaiopensource.com/relaxng/jing.html -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 648053] Review Request: libMicro - Portable micro-benchmarks for various system and library calls
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=648053 --- Comment #3 from Pierre Carrier 2010-11-21 17:49:22 EST --- Thanks Michael. Updated. $ rpmlint -v RPMS/x86_64/libMicro-0.4.0-1.fc14.x86_64.rpm libMicro.x86_64: I: checking libMicro.x86_64: W: spelling-error %description -l en_US awk -> awl, aw, wk libMicro.x86_64: I: checking-url http://hub.opensolaris.org/bin/view/Project+libmicro/ (timeout 10 seconds) libMicro.x86_64: W: no-manual-page-for-binary multiview libMicro.x86_64: W: no-manual-page-for-binary bench 1 packages and 0 specfiles checked; 0 errors, 3 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 506425] Review Request: lightsquid - a light, small and fast log analyzer for squid proxy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506425 Ruediger Landmann changed: What|Removed |Added AssignedTo|alek...@oscada.org.ua |r.landm...@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 515164] Review Request: gnomenu - A consolidated menu for gnome that brings eye candy to the world of the Gnome menu's
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=515164 Henrique "LonelySpooky" Junior changed: What|Removed |Added Flag|needinfo?(henrique...@gmail | |.com) | --- Comment #27 from Henrique "LonelySpooky" Junior 2010-11-21 21:13:17 EST --- Hi, I'll be happy to restart the work here. Thank you Ruediger. I'll be posting updates here ASAP. -- Configure 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 614299] Review Request: python-ordereddict - Py2.7's new collections.OrderedDict that works in Python 2.4-2.6.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=614299 James Ni changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from James Ni 2010-11-21 21:44:55 EST --- New Package SCM Request === Package Name: python-ordereddict Short Description: OrderedDict that works in Python 2.4-2.6 Owners: jamesni Branches: f13 f14 el5 el6 InitialCC: jamesni -- Configure 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 197688] Review Request: inotify-tools - Command line utilities for inotify
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=197688 Mark McKinstry changed: What|Removed |Added CC||mmcki...@nexcess.net Flag||fedora-cvs? --- Comment #9 from Mark McKinstry 2010-11-21 23:08:42 EST --- Package Change Request == Package Name: inotify-tools New Branches: el5 el6 Owners: drago01 mmckinst InitialCC: drago01 I emailed Adel about an EPEL 5 and 6 version of inotify-tools. He's not a RHEL or CentOS user but I am so I'll take care of the EPEL 5 and 6 branch. The kernel for EPEL 4 (2.6.9) does not have inotify support which was added in 2.6.13 so leaving el4 off the branch list is intentional. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 571993] Review Request: non-sequencer - A powerful, real-time, pattern-based MIDI sequencer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=571993
--- Comment #14 from Orcan Ogetbil 2010-11-21 23:15:24
EST ---
Sorry for being late. Here is a more thorough review:
- rpmlint says
non-sequencer.src:61: W: configure-without-libdir-spec
non-sequencer.src: W: invalid-url Source0:
non-sequencer-20100131gitba94d2c354145.tar.bz2
non-sequencer.x86_64: W: no-manual-page-for-binary non-sequencer
These can be ignored. However
non-sequencer.x86_64: W: no-documentation
needs a fix. You need to put at least the COPYING file in this package, also
possibly the TODO file
! Currently, the -doc subpackage does not require the main package. Is this
intentional? If this is what you want, the COPYING file must exist both in the
main package and the -doc subpackage.
* Currently the source tarball contains the .git* stuff that we don't need.
Those can be taken out. The tarball instructions tarball tell us to check out
the latest git version. What we need is the instructions to generate the exact
tarball. In case upstream updates their repo, current instructions will be
incorrect.
* The %doc package should be noarch
* Please explain in the SPEC file where you got the other sources. Do they have
copyrights? Also please explain in the SPEC file what the patch is for, and/or
its upstream status.
* The directory %{_datadir}/%{name}/ must be owned by the main package. See
http://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
* You certainly don't need the BuildRequires: cmake git. I am not sure whether
you really need libpng-devel libjpeg-devel. Somehow -lpng and -ljpeg flags are
passed to the linking, but what adds them?
* Each package must consistently use macros. Please don't use plain /usr. Use
%{_prefix} instead.
! Please span %description to 80 columns as much as possible. Currently it is
set to 70.
* The icon is installed in the wrong place. It should be
%{_datadir}/icons/hicolor/128x128/apps/
Note that you leave the scriptlets as is.
? sed -i "/\.html/d" Makefile
What is the above for? Can't that be moved to %prep?
--
Configure 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 225698] Merge Review: dmidecode
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225698 --- Comment #15 from Orcan Ogetbil 2010-11-21 23:17:47 EST --- Come on Prarit :) you can do it! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 639874] Review Request: python-rocket - Modern, multi-threaded, comet-friendly WSGI web 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=639874 --- Comment #2 from Ilia Cheishvili 2010-11-21 23:56:14 EST --- A very good point. I will see about doing this. Thanks for the wiki link. -- Configure 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 647076] Review Request: jackbeat - audio sequencer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=647076 --- Comment #19 from Orcan Ogetbil 2010-11-21 23:56:32 EST --- As you requested: $ cat ~/.jackdrc /usr/bin/jackd -v -P20 -dalsa -dhw:0 -r48000 -p1024 -n2 -Xseq I tried a few other combinations but it just doesn't work. I see the line Jack: jack_client_open PortAudio in the program output about 20 lines before it shuts jack. This is a similar symptom I observed when I built fluidsynth with portaudio support (Lots of XRUNS leading jackd to shutdown or crash after 2-3 seconds). For this reason, we disabled the fluidsynth's portaudio support in the Fedora package. Over the time I saw a couple other people from other distros report the same issue. Similarly, I disabled the portaudio support of jackbeat. There is no very easy way to do it. I hacked on the source. After that, I compiled and, guess what, jackbeat works properly. This is really strange. With the unhacked jackbeat, even when I use the -c Jack option, it still tries to interact with portaudio. I wonder why this happens. I understand that you don't want this package to be in Fedora yet. Maybe you can ask upstream to make portaudio and pulseaudio support optional for this software, and then we can compile it for jack only. Otherwise the package is in perfect condition, normally I would just go ahead and sponsor you. :( Let me know when you submit another package. I will be busy for the next couple months, but I will definitely make some time. And I appreciate your offer for helping with Audio production packages. I really need it. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 614299] Review Request: python-ordereddict - Py2.7's new collections.OrderedDict that works in Python 2.4-2.6.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=614299 --- Comment #12 from James Ni 2010-11-22 00:05:25 EST --- New Package SCM Request === Package Name: python-ordereddict Short Description: OrderedDict that works in Python 2.4-2.6 Owners: jamesni Branches: f13 el5 el6 InitialCC: jamesni -- Configure 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 513320] Review Request: boxbackup - A fast, secure and automatic online backup system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=513320 --- Comment #18 from Stewart Adam 2010-11-22 00:16:18 EST --- Sorry about the usermgmt stuff, I followed an example from another package without realizing it was non-standard. I've switched to a dynamically created user as well as fixed a minor error in the server package's README file. SPEC: http://firewing.fedorapeople.org/pub/SPECS/boxbackup.spec SRPM: http://firewing.fedorapeople.org/pub/SRPMS/boxbackup-0.11-0.2.rc8.fc14.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 487281] Review Request: eGroupware - A web-based groupware suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487281 Ruediger Landmann changed: What|Removed |Added Attachment #461436|0 |1 is obsolete|| --- Comment #19 from Ruediger Landmann 2010-11-22 00:20:58 EST --- Created attachment 461920 --> https://bugzilla.redhat.com/attachment.cgi?id=461920 Updated patch Updated English patch for release 13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 487281] Review Request: eGroupware - A web-based groupware suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487281 Ruediger Landmann changed: What|Removed |Added Flag||needinfo?(pa...@hubbitus.in ||fo) -- Configure 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 487281] Review Request: eGroupware - A web-based groupware suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487281 --- Comment #20 from Ruediger Landmann 2010-11-22 00:46:24 EST --- I think you linked the wrong spec and srpm in comment #18 ;) * I've attached an updated English language patch -- just place it in the same directory as your spec file and run: patch -p1 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 654443] Review Request: h2 - h2 database engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=654443 --- Comment #7 from Alexander Kurtakov 2010-11-22 02:52:34 EST --- 3. Install license as doc in BOTH main and javadoc subpackage. is still not fixed Please use servlet25 (tomcat6) unless you are going to spend time on tomcat5 maintainership. -- Configure 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
