[Bug 253813] Package Review: cjkunifonts-fonts (splitted from fonts-chinese).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Package Review: cjkunifonts-fonts (splitted from fonts-chinese). https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253813 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 01:59 EST --- http://cchance.fedorapeople.org/20070822_cjkuni/cjkuni-fonts.spec http://cchance.fedorapeople.org/20070822_cjkuni/cjkuni-fonts-0.1-1.fc8.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253813] Package Review: cjkunifonts-fonts (splitted from fonts-chinese).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Package Review: cjkunifonts-fonts (splitted from fonts-chinese). https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253813 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEEDINFO|ASSIGNED Flag|needinfo?([EMAIL PROTECTED]| |m) | --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 02:07 EST --- I am still wondering if it is better to split the ukai and uming fonts into separate packages. This might also be better for casual Chinese users who don't need them both. Another way to do that would be to subpackage them from the same srpm - that would be less package maintainence work anyway. Anyway I think the base name should be cjkunifonts like the upstream project. I don't mind if the package is called cjkunifonts or cjkunifonts-fonts say. http://www.freedesktop.org/wiki/Software/CJKUnifonts -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 252104] Review Request: perl-Perlbal-XS-HTTPHeaders - Perlbal extension for processing HTTP headers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Perlbal-XS-HTTPHeaders - Perlbal extension for processing HTTP headers https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=252104 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249524] Review Request: wxdfast - Multi-threaded download manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxdfast - Multi-threaded download manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249524 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 03:34 EST --- some notes from testing before formal review: - update license tag GPL = GPLv2, icons are using Creative Commons Public License - in %build - using %configure is enough, no need to set prefix, ..., no need to explicitly set CXXFLAGS for make - remove execute bit from source files (find + xargs chmod a-x) - debug package does not contain the sources (tested in mock for Rawhide/i386 and natively on FC6/x86_64) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 250240] Review Request: linkage - Lightweight bittorent client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: linkage - Lightweight bittorent client https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250240 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 03:40 EST --- Updated to new upstream version, fixes the rawhide build issues. http://tgmweb.at/gadllah/linkage.spec http://tgmweb.at/gadllah/linkage-0.1.4-1.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 190040] Review Request: hydrogen - Advanced drum machine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: hydrogen - Advanced drum machine https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=190040 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:12 EST --- I'm getting this build error with that patch: src/lib/FLACFile.cpp: In member function 'void FLACFile_real::load(std::string)': src/lib/FLACFile.cpp:167: error: 'set_filename' was not declared in this scope src/lib/FLACFile.cpp:169: error: no matching function for call to 'FLACFile_real::init()' /usr/include/FLAC++/decoder.h:226: note: candidates are: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(FILE*) /usr/include/FLAC++/decoder.h:227: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const char*) /usr/include/FLAC++/decoder.h:228: note: virtual FLAC__StreamDecoderInitStatus FLAC::Decoder::File::init(const std::string) src/lib/FLACFile.cpp:170: error: 'FLAC__FILE_DECODER_OK' was not declared in this scope src/lib/FLACFile.cpp:174: error: 'process_until_end_of_file' was not declared in this scope make[1]: *** [src/FLACFile.o] Error 1 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251680] Review Request: amtterm - Serial-over-lan (sol) client for Intel AMT
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: amtterm - Serial-over-lan (sol) client for Intel AMT https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251680 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:23 EST --- New Package CVS Request === Package Name: amtterm Short Description: Serial-over-lan (sol) client for Intel AMT. Owners: kraxel Branches: F-7 EL-5 Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249524] Review Request: wxdfast - Multi-threaded download manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxdfast - Multi-threaded download manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249524 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:35 EST --- Sorry, overwriting the CXXFLAGS is necessary. Only in the top-level makefile they are OK. The debug package is created correctly too. But I have a solution that can be posted upstream, see attachments. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249524] Review Request: wxdfast - Multi-threaded download manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxdfast - Multi-threaded download manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249524 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:36 EST --- Created an attachment (id=162036) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=162036action=view) patch for Makefile.* -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249524] Review Request: wxdfast - Multi-threaded download manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxdfast - Multi-threaded download manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249524 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:36 EST --- Created an attachment (id=162037) -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=162037action=view) spec file patch -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 04:59 EST --- Thanks for the review! (In reply to comment #6) 1. Does this package need %makeinstall? Yes, the Makefile doesn't support DESTDIR. 2. The URL in the Source line looks wrong... Fixed in -3 at above location. 3. By no means a blocker, but there's no dist tag. This is by intent, I see no need to use the dist tag on the devel branch for any package. (particularly so for a package like this, as you say, which will never exist in any branch) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253232] Review Request: system-config-firewall - graphical interface for basic firewall setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: system-config-firewall - graphical interface for basic firewall setup https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253232 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:00 EST --- Sucessfully built: http://koji.fedoraproject.org/koji/taskinfo?taskID=112438 Closing as NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253579] Review Request: php-pecl-memcache - Extension to work with the Memcached caching daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pecl-memcache - Extension to work with the Memcached caching daemon https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253579 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:07 EST --- Vry nice, clean and portable spec file. This will be a real pleasure to review. Thanks for CC'ing me, feel free to do it again ;-) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249524] Review Request: wxdfast - Multi-threaded download manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: wxdfast - Multi-threaded download manager https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249524 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:08 EST --- hi thx for the patches I applied them, new package here: http://tgmweb.at/gadllah/wxdfast.spec http://tgmweb.at/gadllah/wxdfast-0.6.0-2.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230096] Review Request: iwlwifi-firmwa re - Firmware for Intel® PRO/Wireless 3945 A/B /G network adaptors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: iwlwifi-firmware - Firmware for Intel® PRO/Wireless 3945 A/B/G network adaptors https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230096 [EMAIL PROTECTED] changed: What|Removed |Added Status|CLOSED |ASSIGNED Keywords||Reopened Resolution|NEXTRELEASE | --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:25 EST --- New package, which includes the firmware version still required for the F7 kernels (and current devel?), and also the very latest version, probably required soon by the devel kernels if not already : http://ftp.es6.freshrpms.net/tmp/extras/iwl3945-firmware/iwl3945-firmware.spec http://ftp.es6.freshrpms.net/tmp/extras/iwl3945-firmware/iwl3945-firmware-2.14.1.5-1.src.rpm And I'm reopening the bug to make it easier to track it. I'll be asking for the CVS rename now. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253692] Review Request: compiz-fusion - Collection of Compiz Fusion plugins for Compiz
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compiz-fusion - Collection of Compiz Fusion plugins for Compiz https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253692 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:29 EST --- fixed a gconf bug, new urls: http://tgmweb.at/gadllah/compiz-fusion.spec http://tgmweb.at/gadllah/compiz-fusion-0.5.2-2.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253693] Review Request: compiz-fusion-extras - Additional Compiz Fusion plugins for Compiz
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compiz-fusion-extras - Additional Compiz Fusion plugins for Compiz https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253693 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:30 EST --- fixed a gconf bug, new urls: http://tgmweb.at/gadllah/compiz-fusion-extras.spec http://tgmweb.at/gadllah/compiz-fusion-extras-0.5.2-2.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230096] Review Request: iwl3945-firmwa re - Firmware for Intel® PRO/Wireless 3945 A/B /G network adaptors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: iwl3945-firmware - Firmware for Intel® PRO/Wireless 3945 A/B/G network adaptors https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230096 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: iwlwifi-|Review Request: iwl3945- |firmware - Firmware for |firmware - Firmware for |Intel® PRO/Wireless 3945 |Intel® PRO/Wireless 3945 |A/B/G network adaptors |A/B/G network adaptors Flag|fedora-cvs+ |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:30 EST --- Package Change Request == Package Name: iwlwifi-firmware The package need to be renamed to iwl3945-firmware per upstream split (iwl3945 and iwl4965). Please make the required changes, then I'll commit the new spec files with the changed name and rebuild new packages for the current branches. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253833] New: Review Request: compat-neon025 - library compat package for neon 0.25
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253833 Summary: Review Request: compat-neon025 - library compat package for neon 0.25 Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://jorton.fedorapeople.org/neon/compat-neon025.spec SRPM URL: http://jorton.fedorapeople.org/neon/compat-neon025-0.25.5-1.src.rpm Description: A library compatibility package for neon 0.25 Given that (among other things) rpm is linked against neon 0.25 in F7 it seems prudent to include a compat package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253833] Review Request: compat-neon025 - library compat package for neon 0.25
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-neon025 - library compat package for neon 0.25 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253833 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233596] Review Request: python-Coherence - Python framework to participate in digital living networks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-Coherence - Python framework to participate in digital living networks https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233596 [EMAIL PROTECTED] changed: What|Removed |Added Product|Fedora Extras |Fedora [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 05:33 EST --- Package Change Request == Package Name: python-Coherence New Branches: F-7 For some reason, when devel was branched off for F-7, the F-7 branch of python-Coherence wasn't created (no F-7 directory) : [EMAIL PROTECTED] python-Coherence]$ cvs update -dP cvs update: Updating . cvs update: Updating EL-4 cvs update: Updating EL-5 cvs update: Updating FC-5 cvs update: Updating FC-6 cvs update: Updating devel -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249352] Review Request: popt - C library for parsing command line parameters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: popt - C library for parsing command line parameters https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249352 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn|119782, 205849 | OtherBugsDependingO||119782, 205849 nThis|| --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 06:08 EST --- It seems that the bugs in depends on were intended for blocks, because one is for RHEL and may be fixed in this version and the other is more a feature request and may also be already fixed in the reviewed version. Please change this back in case I am wrong. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249352] Review Request: popt - C library for parsing command line parameters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: popt - C library for parsing command line parameters https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249352 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Flag||fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 07:42 EST --- graphviz (or something else that provides dot) is missing in the BuildRequires, without it errors like this are shown: sh: dot: command not found Problems running dot: exit code=127, command='dot', Arguments='poptint_8h__dep__incl.dot -Tpng -o poptint_8h__dep__incl.png' Review: - rpmlint: ok (this is with BR on graphviz) E: popt-devel zero-length /usr/share/doc/popt-devel-1.12/html/popt_8h__incl.map This is created by doxygen, imho there is no need to remove it manually. W: popt-static no-documentation This is no problem, too. - naming: ok - spec file readability: ok - source: ok f45290e9ac4b1cf209d0042eb6755543 /tmp/popt-1.12.tar.gz f45290e9ac4b1cf209d0042eb6755543 popt-1.12.tar.gz - mockbuild i386 F7: ok - BuildRequires: NEEDSWORK (add graphviz) - ldconfig scriptlets: ok - Directory Ownership: ok - %clean section: ok - macro usage: ok - doc: large api doc is in devel, ok - devel subpackage with headers: ok - static subpackage: ok - .so file in -devel: ok - subpackage requires: ok - .la removed: ok - correct %install - license included: ok - Buildroot: ok Add graphviz to BR before you import the package and it is APPROVED -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251068] Review Request: ocaml-libvirt - OCaml binding for libvirt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocaml-libvirt - OCaml binding for libvirt https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251068 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||253566 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:12 EST --- Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-libvirt.spec SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-libvirt-0.3.2.1-1.fc8.src.rpm Description: OCaml binding for libvirt (http://libvirt.org) and also the virtualization utility 'virt-top'. * Wed Aug 22 2007 Richard W.M. Jones [EMAIL PROTECTED] - 0.3.2.1-1 - Upstream version 0.3.2.1. - Put HTML documentation in -devel package. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253566] Review Request: ocaml-xml-light - Minimal XML parser and printer for OCaml
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocaml-xml-light - Minimal XML parser and printer for OCaml https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253566 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||251068 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? | --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:16 EST --- One question. * Desktop file - This package does not contain any desktop file. However xcircuit is also Tcl/Tk application and xcircuit has desktop file So may it be that netgen should also have desktop file? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251068] Review Request: ocaml-libvirt - OCaml binding for libvirt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ocaml-libvirt - OCaml binding for libvirt https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251068 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:48 EST --- Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-libvirt.spec SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-libvirt-0.3.2.1-2.fc8.src.rpm Description: OCaml binding for libvirt (http://libvirt.org) and also the virtualization utility 'virt-top'. There was an unclosed %if section in the previous spec file - fixed. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Ice - The Internet Communications Engine (Object middleware) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:49 EST --- (In reply to comment #30) I'm working on this. I've made most of the changes, but I'd like some advice on a couple of them. A: ruby-libs dependency - This package has BuildRequires: ruby-libs and -ruby subpackage has Requires: ruby(abi) = 1.8 For consistency (i.e. to avoid that this package is rebuild against ruby 1.9), IMO BuildRequires: ruby-libs should be replaced with ruby(abi) = 1.8. You can't actually BuildRequire: ruby(abi) ... what I've got there right now is consistent with my reading of the Ruby packaging guidelines (http://fedoraproject.org/wiki/Packaging/Ruby). NB: I don't actually use Ruby ... D. naming - Usually foo-devel package should have the corresponding package named foo. IMO -cxx-devel subpackage should just be named as ice-devel. The thing is, the main ice package provides a large number of runtime files and documentation, as well as a set of .so.* libraries (that the runtime files are linked to). The c++-devel package adds two tools and a set of .h files and .so links so that you can build Ice packages using c++ -- it is clearly a c++ development package, not an overall -devel package. I can see the argument for calling it ice-devel, but that's a bit confusing because that sounds like it's *the* Ice development package when in truth it's really just for C++. 4. All source codes check (especially license issue check) - Well, this is extremely hard.. Actually there are (in total) 10448 files (the most number in the packages I have reviewed...) I may skip this check during review, however if I find any issues I will report later. I've done some preliminary looking around. It seems that the majority of source files are under the Ice license, which is the GPLv2 with exceptions for OpenSSL and Orca Robotics components. Here are some numbers (just simple grep for now ...) - 10448 files - 6944 have the Ice license - 13 have GPLv2 or later A lot of the remainder are in the test or demo directories, which I don't build at all -- is that an issue? There are also a lot of .depend files which have no license information at all. And then there are the *.png/*.gif files in the documentation ... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Ice - The Internet Communications Engine (Object middleware) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:52 EST --- - 10448 files - 6944 have the Ice license Okay, whoops, forgot to subtract there: I think something like 3504 have the Ice license. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253191] Exist XML database
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Exist XML database https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253191 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:53 EST --- OK, I see. I would add it to that list, but it seems to be write protected, even after I created an account on the wiki. Maybe it's only for developers, I'm a regular user. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 250924] Review Request: flashrom - Simple program for reading/writing BIOS chips content
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: flashrom - Simple program for reading/writing BIOS chips content https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=250924 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 08:59 EST --- minor update - svn 2744 where added support for EPoX EP-BX3 board. http://peter.fedorapeople.org/flashrom.spec http://peter.fedorapeople.org/flashrom-0.0-1.2744svn.fc7.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253833] Review Request: compat-neon025 - library compat package for neon 0.25
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-neon025 - library compat package for neon 0.25 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253833 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||243638 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253579] Review Request: php-pecl-memcache - Extension to work with the Memcached caching daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: php-pecl-memcache - Extension to work with the Memcached caching daemon https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253579 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 09:21 EST --- I just went through the ReviewGuidelines, double checked the license and tested a Fedora 7 x86_64 build. It all looks fine. Just two minor nitpicks : - Some sections of your spec file are separated by 2 spaces, others only by one. It would be more readable if all were separated by 2 IMHO :-) (before %prep, %build and %changelog). - It's harmless here, but you should always escape macros in comments and the %changelog, so change use %name rather than %pecl_name to use %%name rather than %%pecl_name or even simply use name rather than pecl_name. Again, no big deal, but if some day you put some multi-line macro, it'll do nasty stuff ;-) APPROVED even as-is in any case, great work on PHP packaging, keep it up! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253858] New: Review Request: system-config-vsftpd - graphical utility for administrating vsftpd
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253858 Summary: Review Request: system-config-vsftpd - graphical utility for administrating vsftpd Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://vsftpd-config.svn.sourceforge.net/viewvc/*checkout*/vsftpd-config/trunk/system-config-vsftpd.spec SRPM URL: http://easynews.dl.sourceforge.net/sourceforge/vsftpd-config/system-config-vsftpd-0.4.4-1.fc7.src.rpm Description: System-config-vsftpd is a graphical utility for administrating Very Secure FTP Daemon ( VSFTPD ). This is my first package and I need a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241387] Review Request: codeina - GStreamer Codec Installation Application
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: codeina - GStreamer Codec Installation Application
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241387
[EMAIL PROTECTED] changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review+
--- Additional Comments From [EMAIL PROTECTED] 2007-08-22 09:58 EST ---
Only thing that bothers me a little is
install -D -m 0755 %{SOURCE1}
$RPM_BUILD_ROOT%{_libexecdir}/gst-install-plugins-helper.sh
should that go into %{_libexecdir}/codeina/gst-install-plugins-helper.sh
? Regardless of the answer to that question looks good.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 231814] Review Request: avarice - Program for interfacing the Atmel JTAG ICE to GDB
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: avarice - Program for interfacing the Atmel JTAG ICE to GDB https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231814 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:03 EST --- New Package CVS Request === Package Name: avarice Short Description: Program for interfacing the Atmel JTAG ICE to GDB to allow users to debug their embedded AVR target. Owners: trondd Branches: FC6, F-7, EL-5 InitialCC: Commits by cvsextras: yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 231835] Review Request: uisp - Universal In-System Programmer for Atmel AVR and 8051
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: uisp - Universal In-System Programmer for Atmel AVR and 8051 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=231835 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:04 EST --- New Package CVS Request === Package Name: uisp Short Description: Universal In-System Programmer for Atmel AVR and 8051 Owners: trondd Branches: FC-6, F-7, EL-5 InitialCC: Commits by cvsextras: yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241387] Review Request: codeina - GStreamer Codec Installation Application
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: codeina - GStreamer Codec Installation Application
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241387
--- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:07 EST ---
(In reply to comment #14)
Only thing that bothers me a little is
install -D -m 0755 %{SOURCE1}
$RPM_BUILD_ROOT%{_libexecdir}/gst-install-plugins-helper.sh
should that go into %{_libexecdir}/codeina/gst-install-plugins-helper.sh
No, gst-install-plugins-helper is the hook for gstreamer uses to call the plugin
helper, so this needs to match the path used in GStreamer.
? Regardless of the answer to that question looks good.
The question, or the package? :)
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241387] Review Request: codeina - GStreamer Codec Installation Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: codeina - GStreamer Codec Installation Application https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241387 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:13 EST --- New Package CVS Request === Package Name: codeina Short Description: GStreamer Codec Installation Application Owners: hadess thomasvs Branches: devel InitialCC: [EMAIL PROTECTED] [EMAIL PROTECTED] Commits by cvsextras: yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249352] Review Request: popt - C library for parsing command line parameters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: popt - C library for parsing command line parameters https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249352 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:49 EST --- New Package CVS Request === Package Name: popt Short Description: C library for parsing command line parameters Owners: [EMAIL PROTECTED],[EMAIL PROTECTED] For the guy doing the CVS: /me is maintainer, Panu is co-maintainer. Only development branch (upcoming F8), please. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Ice - The Internet Communications Engine (Object middleware) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 10:56 EST --- (In reply to comment #31) (In reply to comment #30) A: ruby-libs dependency For consistency (i.e. to avoid that this package is rebuild against ruby 1.9), IMO BuildRequires: ruby-libs should be replaced with ruby(abi) = 1.8. You can't actually BuildRequire: ruby(abi) ... what I've got there right now is consistent with my reading of the Ruby packaging guidelines (http://fedoraproject.org/wiki/Packaging/Ruby). Yes. Currently Fedora ruby guideline does not have BuildRequires: ruby(abi) = 1.8, however this is consistent with python, perl, etc and I think this is needed. Also another reviewer (who reviewed ruby module packages I maintain) asked me to do so. D. naming - Usually foo-devel package should have the corresponding package named foo. IMO -cxx-devel subpackage should just be named as ice-devel. The thing is, the main ice package provides a large number of runtime files and documentation, as well as a set of .so.* libraries (that the runtime files are linked to). The c++-devel package adds two tools and a set of .h files and .so links so that you can build Ice packages using c++ -- it is clearly a c++ development package, not an overall -devel package. Please read http://www.redhat.com/archives/rhl-devel-list/2007-August/msg00532.html Actually * Most people expects that the development package of ice is named as ice-devel * And some people even think that splitting development packages are of no means. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 11:20 EST --- (In reply to comment #40) So may it be that netgen should also have desktop file? Yes, true. Netgen should have a desktop file. However, is this the only issue (meanwhile I'm searching for an appropriate icon )? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 11:22 EST --- (In reply to comment #41) However, is this the only issue (meanwhile I'm searching for an appropriate icon )? Yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] New: Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://thias.fedorapeople.org/review/lsdvd/lsdvd.spec SRPM URL: http://thias.fedorapeople.org/review/lsdvd/lsdvd-0.16-2.src.rpm Description: Lsdvd is a c application for reading the contents of a DVD and printing the contents to your terminal. Lsdvd uses libdvdread, the most popular dvd reading library for *nix. Note : Very simple package, very simple codebase. Should be easy to review. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 11:45 EST --- Good review ticket for someone who needs to be sponsored. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Ice - The Internet Communications Engine (Object middleware) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 11:59 EST --- Okay, I've made a new version with most of your concerns addressed. Here's the changelog: - Changed BuildRequires on ruby to ruby(abi) = 1.8 - Fixed all dependencies between subpackages: everything requires the base package, and -devel packages should all require their corresponding non-devel package now - Made ice-csharp require pkgconfig - Modified the user/group creation process based on the wiki - Removed ldconfig for ice-c++-devel subpackage - Made the python_sitelib subdirectory owned by ice-python - Removed executable permission on all files under slice (how did that happen?) - Fixed typo on ice-csharp group - Changed license tag to GPLv2 - Removed macros in changelog - Set CFLAGS as well as CPPFLAGS for make so that building icecpp gets the correct flags too - Renamed ice-c++-devel to ice-devel - Added Provides: for ice-c++-devel and ice-dotnet for people moving from the ZeroC RPMs - Also don't build test or demo for IceCS http://homepages.inf.ed.ac.uk/mef/extras/ice-3.2.1-3.fc7.src.rpm http://homepages.inf.ed.ac.uk/mef/extras/ice.spec -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:06 EST --- ok. Too bad about %makeinstall. ;( I see no further blockers here, so this package is APPROVED. Don't forget to close this once it's been imported and built. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: lsdvd - Small application for displaying the contents
of a DVD
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874
[EMAIL PROTECTED] changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
Flag||fedora-review+
--- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:06 EST ---
I'm afraid I'll ruin that joy for someone as I already did the review :P
- Could use %configure --disable-dependency-tracking but it doesn't make much
difference for a package this small.
- %description contains some info which is not relevant to end users, I would
shorten it to simply lsdvd is an application for reading the contents of a DVD
and printing the list of contents to your terminal.
- Please bump release to 5%{?dist} to upgrade the NEVR of this package already
present in some 3rd party repositories.
All above comments can be addressed after import and before the first build, or
just ignored. Your choice, approved anyway.
--
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253649] Review Request: vinagre - VNC client 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 report. Summary: Review Request: vinagre - VNC client for the GNOME desktop https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253649 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Fixed In Version||0.2-1 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:07 EST --- Uploaded to rawhide. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:08 EST --- Updated: SRPM: http://chitlesh.fedorapeople.org/netgen/netgen-1.3.7-11.fc7.src.rpm SPEC: http://chitlesh.fedorapeople.org/netgen/netgen.spec The png is a screenshot taken on adder4 (an alliance's example) by me. Its license is GPL+. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:13 EST --- New Package CVS Request === Package Name: compat-expat1 Short Description: Library compatibility package for expat 1 Owners: jorton Branches: devel InitialCC: Commits by cvsextras: no -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:15 EST --- New Package CVS Request === Package Name: compat-expat1 Short Description: Library compatibility package for expat 1 Owners: jorton Branches: devel InitialCC: Commits by cvsextras: no [third time lucky with the correct flag waving] -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:25 EST --- Thanks Ville! You can check the updated spec file which I'll be importing in the same location as the previous one, and shout if anything's wrong with it ;-) New Package CVS Request === Package Name: lsdvd Short Description: Small application for displaying the contents of a DVD Owners: [EMAIL PROTECTED] Branches: devel F-7 FC-6 InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:32 EST --- The new specfile looks good, thanks. You missed one tiny detail though - printing the contents of a DVD to your terminal sounds scary to me, which is why I suggested list of contents :) (Applies to Summary too if you like the suggestion, BTW) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 230096] Review Request: iwl3945-firmwa re - Firmware for Intel® PRO/Wireless 3945 A/B /G network adaptors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: iwl3945-firmware - Firmware for Intel® PRO/Wireless 3945 A/B/G network adaptors https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230096 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:49 EST --- By the new procedure, we will no longer rename CVS modules. I have added iwl3945-firmware as a new package owned by you. Please use the dead.package procedure in the old CVS module. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 233596] Review Request: python-Coherence - Python framework to participate in digital living networks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-Coherence - Python framework to participate in digital living networks https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=233596 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241387] Review Request: codeina - GStreamer Codec Installation Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: codeina - GStreamer Codec Installation Application https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241387 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249352] Review Request: popt - C library for parsing command line parameters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: popt - C library for parsing command line parameters https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249352 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs- --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 12:57 EST --- Could you please provide this in the new template format? http://fedoraproject.org/wiki/PackageMaintainers/CVSAdminProcedure Specifically, we need the FAS account names. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 13:22 EST --- Indeed, I had missed that, will fix, thanks! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249352] Review Request: popt - C library for parsing command line parameters
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: popt - C library for parsing command line parameters https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249352 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs- |fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 14:19 EST --- Okay Warren, here we go again: New Package CVS Request === Package Name: popt Short Description: C library for parsing command line parameters Owners: robert,pmatilai Branches: InitialCC: Cvsextras Commits: no -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 15:24 EST --- I think we get a little to speed for this! I little more investigation show that lsdvd isn't used by freevo since 1.7. Instead it uses python-kaa-metadata (which also orphan mmpython). kaa.metadata is already in Fedora... Futhermore lsdvd's sourceforge repository is now empty I think this is good to have it for compat with fc6 f7 And unless others apps are using it (do you you some example ?) I am for orphaning it... Usually apps can access lsdvd output by using kaa.metadata and should uses it instead... -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 15:58 EST --- Yes, there are apps that use lsdvd. kaa-metadata is in Python, this is in plain C, they fit different scenarios. Python is an undesirably big dependency for trimmed down setups, but on the other hand apps written in Python almost certainly want to use a Python lib for this stuff. There's no indication that lsdvd's SF CVS has ever been anything but empty. Apples and oranges, what's the problem? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253874] Review Request: lsdvd - Small application for displaying the contents of a DVD
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: lsdvd - Small application for displaying the contents of a DVD https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253874 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 16:33 EST --- ok sorry for the false positive then... i thought lsdvd was already orphaned So everything is fine...good! -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253910] New: Review Request: ksirk - Turnbased multiplayer board strategy game (conquer the world!)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253910 Summary: Review Request: ksirk - Turnbased multiplayer board strategy game (conquer the world!) Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://people.atrpms.net/~hdegoede/ksirk.spec SRPM URL: http://people.atrpms.net/~hdegoede/ksirk-1.7-1.fc8.src.rpm Description: KsirK is a computerized version of a well known strategy board game. In the current version, KsirK is a usable multi-player mono-machine game with a basic AI. The goal of the game is simply to conquer the World... It is done by attacking your neighbors with your armies. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 251361] Review Request: compat-expat1 - library compat package for expat 1.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-expat1 - library compat package for expat 1.x https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=251361 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 16:49 EST --- Thanks a lot - now who do I bribe to review compat-neon025! :) -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253909] New: Review Request: libpano13 - Library for manipulating panoramic images
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253909 Summary: Review Request: libpano13 - Library for manipulating panoramic images Product: Fedora Version: devel Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://bugbear.blackfish.org.uk/~bruno/apt/SPECS/libpano13.spec SRPM URL: http://bugbear.blackfish.org.uk/~bruno/apt/fedora/linux/7/x86_64/SRPMS.panorama/libpano13-2.9.12-3.fc7.src.rpm Description: Helmut Dersch's Panorama Tools library. Provides very high quality manipulation, correction and stitching of panoramic photographs. Note: I already package libpano12 for fedora, libpano13 is a fork of this libpano12, but is designed to be installed in parallel. Currently they are very similar and the SPEC is almost identical, the differences being that: 1. various command-line tools have been moved from pano12 to pano13 2. pano12 is in the process of being relicensed to LGPLv3+ (from GPLv2+) libpano13 is required for the next hugin release, after that libpano12 can be dropped from fedora. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253469] Review Request: opyum - Offline package installation and update tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: opyum - Offline package installation and update tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253469 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 17:24 EST --- Trying to check for Pirut in the configure script using $ $PYTHON -c import pirut gives the following traceback on Koji. The absence of a running X server causes the issue, since the pirut modules eventually pull in gtk too. Traceback (most recent call last): File string, line 1, in module File /usr/lib/python2.5/site-packages/pirut/__init__.py, line 26, in module import gtk File /usr/lib/python2.5/site-packages/gtk-2.0/gtk/__init__.py, line 76, in module _init() File /usr/lib/python2.5/site-packages/gtk-2.0/gtk/__init__.py, line 64, in _init _gtk.init_check() RuntimeError: could not open display It is important to have the check in the upstream tarball, since Pirut is a critical dependency for Opyum and anybody building from the release tarball should be made aware of that. However, I am going to patch out the check for the RPM package since Pirut is already mentioned as a Requires and BuildRequires. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 252128] Review Request: telepathy-haze - A multi-protocol Libpurple connection manager for Telepathy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: telepathy-haze - A multi-protocol Libpurple connection manager for Telepathy Alias: telepathy-haze https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=252128 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 17:42 EST --- Thanks for the review, Steve! (In reply to comment #2) - Deps seem a bit odd ... for instance [EMAIL PROTECTED] Desktop]$ sudo rpm -ivh ~/rpmbuild/RPMS/i386/telepathy-haze-0.1.1-1.fc7.i386.rpm ~/rpmbuild/RPMS/i386/telepathy-haze-mission-control-0.1.1-1.fc7.i386.rpm error: Failed dependencies: telepathy-mission-control is needed by telepathy-haze-mission-control-0.1.1-1.fc7.i386 Yeah, this is a subpackage which provides a profile for Mission Control, which allows Haze (AIM, at least) to be used by it and applications which use it for their telepathy configuration (such as Empathy). My goal is to make Empathy and other MC apps Just Work with Haze as a backend; there are profiles for Jabber and GTalk in upstream as well but those seem quite redundant as we already have Gabble for XMPP connections. - Application runs but outputs quite a bit of warnings ... ** Message: [error] plugins: /usr/lib/purple-2/liboscar.so is not usable because the 'purple_init_plugin' symbol could not be found. Does the plugin call the PURPLE_INIT_PLUGIN() macro? ** Message: [error] plugins: /usr/lib/purple-2/libsametime.so has a prefs_info, but is a prpl. This is no longer supported. ** Message: [error] plugins: /usr/lib/purple-2/libjabber.so is not usable because the 'purple_init_plugin' symbol could not be found. Does the plugin call the PURPLE_INIT_PLUGIN() macro? These look specific to Pidgin/libpurple; I don't get these messages when running Empathy. :| -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 244894] Review Request: yum-cron - get yum updates via a cron job
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: yum-cron - get yum updates via a cron job https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=244894 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 18:09 EST --- yum-cron has appeared in the devel tree. Thanks to everyone for your help getting this off the ground. Frank's tweaks from comment #18 are not included yet, I wanted to get the part I'd thought about longer out first, and will use including his enhancements as an excuse to go through the patch-build-test-release cycle once the initial package successfully propagates (best way for me to learn the system is to do it). It's also been tagged for pushing into testing for F-7, is awaiting signatures for EPEL-5, and plague refuses to cooperate (plague throws an error even though it builds OK) for FC-6. But I can beat on that weirdness later. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253469] Review Request: opyum - Offline package installation and update tools
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: opyum - Offline package installation and update tools https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253469 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 18:29 EST --- Spec: http://rishi.fedorapeople.org/opyum.spec SRPM: http://rishi.fedorapeople.org/opyum-0.0.2-2.fc8.src.rpm Here is the new pair of Spec SRPM. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 252128] Review Request: telepathy-haze - A multi-protocol Libpurple connection manager for Telepathy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: telepathy-haze - A multi-protocol Libpurple connection manager for Telepathy Alias: telepathy-haze https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=252128 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 18:47 EST --- (In reply to comment #3) Thanks for the review, Steve! No prob. (In reply to comment #2) - Deps seem a bit odd ... for instance [EMAIL PROTECTED] Desktop]$ sudo rpm -ivh ~/rpmbuild/RPMS/i386/telepathy-haze-0.1.1-1.fc7.i386.rpm ~/rpmbuild/RPMS/i386/telepathy-haze-mission-control-0.1.1-1.fc7.i386.rpm error: Failed dependencies: telepathy-mission-control is needed by telepathy-haze-mission-control-0.1.1-1.fc7.i386 Yeah, this is a subpackage which provides a profile for Mission Control, which allows Haze (AIM, at least) to be used by it and applications which use it for their telepathy configuration (such as Empathy). My goal is to make Empathy and other MC apps Just Work with Haze as a backend; there are profiles for Jabber and GTalk in upstream as well but those seem quite redundant as we already have Gabble for XMPP connections. So the telepathy-mission-control dep issue is a literal different package and not suppose to be telepathy-haze-mission-control? - Application runs but outputs quite a bit of warnings ... ** Message: [error] plugins: /usr/lib/purple-2/liboscar.so is not usable because the 'purple_init_plugin' symbol could not be found. Does the plugin call the PURPLE_INIT_PLUGIN() macro? ** Message: [error] plugins: /usr/lib/purple-2/libsametime.so has a prefs_info, but is a prpl. This is no longer supported. ** Message: [error] plugins: /usr/lib/purple-2/libjabber.so is not usable because the 'purple_init_plugin' symbol could not be found. Does the plugin call the PURPLE_INIT_PLUGIN() macro? These look specific to Pidgin/libpurple; I don't get these messages when running Empathy. :| I should have noted that the warnings I saw came running telepathy-haze. Is there a better way I can test it? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 252010] Review Request: rpmrebuild - A tool to build rpm file from rpm database
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rpmrebuild - A tool to build rpm file from rpm database https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=252010 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 19:41 EST --- -/+ rpmlint ran. Errors found but rpm does have the 'dangerous command' note. the hidden file looks as if it is needed. The only-non-binary-in-usr-lib worries me a bit ... + package name is correct + spec name is correct + accepted license - Looks like it's actually GPLv2+ - license not in %doc (commented out) + US English spec + Readable spec + Source is same as upstream + built and installed on i386 + Deps look sane + No locale files + no .so files + not relocateable + no duplicate files + perms look sane + clean is good + macros in use + contains code and OK content -/+ %doc missing ... but doesn't look like it affects the code + doesn't own files/dirs in other packages + install cleans first - the descirption seems a bit odd. Maybe the following (from the rpmrebuild site) would suite it: rpmrebuild is a tool to build an RPM file from a package that has already been installed in a basic use, rpmrebuild use do not require any rpm building knowledge + tested locally and worked -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253941] Review Request: cgi-util - A C library for creating Common Gateway Interface (CGI) programs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cgi-util - A C library for creating Common Gateway Interface (CGI) programs https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253941 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 249522] Review Request: sepostgresql - Security-Enhanced PostgreSQL
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: sepostgresql - Security-Enhanced PostgreSQL https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=249522 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 22:17 EST --- [Spec URL] http://sepgsql.googlecode.com/files/sepostgresql.spec [SRPM URL] http://sepgsql.googlecode.com/files/sepostgresql-8.2.4-0.428.beta.fc8.src.rpm # just a test to avoid splitting, please ignore. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253163] Review Request: sazanami-fonts - Sazanami Japanese TrueType fonts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: sazanami-fonts - Sazanami Japanese TrueType fonts https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253163 --- Additional Comments From [EMAIL PROTECTED] 2007-08-22 23:21 EST --- hmm, I was confused. here is the *correct* description: Spec URL: http://tagoh.fedorapeople.org/sazanami-fonts/sazanami-fonts.spec SRPM URL: http://tagoh.fedorapeople.org/sazanami-fonts/sazanami-fonts-0.20040629-2.20061016.fc8.src.rpm Description: This package contains Japanese TrueType fonts. Sazanami type faces are automatically generated from Wadalab font kit. These also contains some Japanese bitmap fonts -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 234612] Review Request: Ice - The Internet Communications Engine (Object middleware)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: Ice - The Internet Communications Engine (Object middleware) https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=234612 --- Additional Comments From [EMAIL PROTECTED] 2007-08-23 00:06 EST --- Rebuild failed. http://koji.fedoraproject.org/koji/taskinfo?taskID=121392 Not checked in detail, however Requires: ruby(abi) = 1.8 only pulls ruby-libs and ruby package is not pulled. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226725] Review Request: netgen - LVS netlist comparison tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: netgen - LVS netlist comparison tool https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226725 --- Additional Comments From [EMAIL PROTECTED] 2007-08-23 00:07 EST --- At least desktop-file-utils is missing from BR. http://koji.fedoraproject.org/koji/taskinfo?taskID=121359 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241596] Review Request: maildrop - Mail delivery agent with filtering abilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: maildrop - Mail delivery agent with filtering abilities https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=241596 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2007-08-23 00:38 EST --- Well, I retried and this time the rebuild failed... http://koji.fedoraproject.org/koji/taskinfo?taskID=121364 -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 253813] Package Review: cjkunifonts-fonts (splitted from fonts-chinese).
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Package Review: cjkunifonts-fonts (splitted from fonts-chinese). https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=253813 --- Additional Comments From [EMAIL PROTECTED] 2007-08-23 01:42 EST --- I agree to split ukai and uming fonts into separate packages. Generally users will accept the installation of both even most users only need uming for daily use. Okay, let's use cjkunifonts-fonts as package name. -- Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
