[Bug 488108] Review Request: django-tagging - A generic tagging application for Django projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488108 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-03-03 03:05:06 EDT --- Review: + package builds in mock. Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1216130 + rpmlint is silent for SRPM and for RPM. + files in source tarball match source packaged in srpm but no way to verify md5sum as source is svn checkout. + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc files present. + BuildRequires are proper. + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code. + no static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + no translations are available. + Does owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + no scriptlets are used. + egg file generated from source. Suggestions: 1) I will suggest to use %doc CONTRIBUTORS.txt LICENSE.txt docs/* will make not to create docs named directory 2) Change %define to %global APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=487392
--- Comment #9 from Dan Horák d...@danny.cz 2009-03-03 03:09:02 EDT ---
another round
- desktop-file-utils is missing as BR
- the main package must use %dir %{_libdir}/%{name} in the %files section
instead of %{_libdir}/%{name} only
- you should replace all occurrences of the string crash-catcher with
%{name}, it will make it easier to change the name of the package
- the package must use a released source archive or follow the guideline for
pre-release packages
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488101] Review Request: django-contact-form - An extensible contact-form application for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488101 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-03-03 03:12:14 EDT --- Review: + package builds in mock. Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1216155 + rpmlint is silent for SRPM and for RPM. + source files match upstream. 4192a2d364814f503dde030985208c37 contact_form-0.3.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc files present. + BuildRequires are proper. + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code. + no static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + no translations are available. + Does owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + no scriptlets are used. + egg file generated from source. Suggestions: 1) I will suggest to use %doc CONTRIBUTORS.txt LICENSE.txt docs/* will make not to create docs named directory 2) Change %define to %global APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=478852
--- Comment #1 from Alexey Torkhov atork...@gmail.com 2009-03-03 03:20:21
EDT ---
- Avoid use of %makeinstall macros if make install DESTDIR=%{buildroot}
working:
https://fedoraproject.org/wiki/Packaging/Guidelines#Why_the_.25makeinstall_macro_should_not_be_used
- Add desktop-file-utils to BuildRequires.
- Remove --vendor fedora from desktop-file-install call as per guidelines.
- Running update-desktop-database in %post and %postun is not needed as desktop
entry doesn't has 'MimeType' key:
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488100] Firebird SQL database management system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488100 --- Comment #8 from MERCIER bioinfornat...@gmail.com 2009-03-03 03:49:05 EDT --- i have already an account name: bioinfornatics and i have already submit 4 package: 2 wait the review and 2 in stable repo thanks for your help -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488197] Review Request: ibus-rawcode - The Rawcode engine for IBus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488197 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Parag AN(पराग) panem...@gmail.com 2009-03-03 03:46:43 EDT --- Review: + package builds in mock. Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1216185 + rpmlint is silent for SRPM and for RPM. + source files match upstream. 26e7aeb1e54ebbab4de222f67ffe174f ibus-rawcode-1.0.0.20090303.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc files present. + BuildRequires are proper. + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code. + no static libraries. + no .pc file present. + no -devel subpackage exists. + no .la files. + translations are available. + Does owns the directories it creates. + no duplicates in %files. + file permissions are appropriate. + no scriptlets are used. + Package ibus-rawcode-1.0.0.20090303-2.fc11-i586 = Requires: Requires: libc.so.6 libc.so.6(GLIBC_2.0) libdbus-1.so.3 libglib-2.0.so.0 libgobject-2.0.so.0 libibus.so.0 rtld(GNU_HASH) Provides: ibus-rawcode = 1.0.0.20090303-2.fc11 APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488197] Review Request: ibus-rawcode - The Rawcode engine for IBus
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488197 Pravin Satpute psatp...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Pravin Satpute psatp...@redhat.com 2009-03-03 04:20:06 EDT --- New Package CVS Request === Package Name: ibus-rawcode Short Description: The Rawcode input engine for IBus platform Owners: pravins Branches: devel InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488099] Review Request: python-polib - A library to parse and manage gettext catalogs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488099 Ignacio Vazquez-Abrams ivazquez...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ignacio Vazquez-Abrams ivazquez...@gmail.com 2009-03-03 04:32:16 EDT --- New Package CVS Request === Package Name: python-polib Short Description: A library to parse and manage gettext catalogs Owners: ivazquez Branches: F-9 F-10 EL-5 InitialCC: i18n-team -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488108] Review Request: django-tagging - A generic tagging application for Django projects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488108 Ignacio Vazquez-Abrams ivazquez...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ignacio Vazquez-Abrams ivazquez...@gmail.com 2009-03-03 04:37:03 EDT --- Package Name: django-tagging Short Description: A generic tagging application for Django projects Owners: ivazquez Branches: F-9 F-10 EL-5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488106] Review Request: django-notification - User notification management for the Django web framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488106 Ignacio Vazquez-Abrams ivazquez...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ignacio Vazquez-Abrams ivazquez...@gmail.com 2009-03-03 04:34:51 EDT --- New Package CVS Request === Package Name: django-notification Short Description: User notification management for the Django web framework Owners: ivazquez Branches: F-9 F-10 EL-5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486757] Review Request: divine-mc - Multi-core model checking system for proving specifications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486757 Peter Rockai proc...@redhat.com changed: What|Removed |Added CC||proc...@redhat.com --- Comment #4 from Peter Rockai proc...@redhat.com 2009-03-03 04:41:16 EDT --- Hi, I suppose it is generally fine, however please note I don't have a fedora installation handy where I could test the packages. You might also want to consider checking that the outcome of the tests you put in %check is correct (you can use --report and some simple grepping for that). Actually, DiVinE 2.0 will come with a bunch of unit and functional tests bundled, so running those will be probably preferable. First release on the 2.0 branch is planned in a course of few weeks. However, looking at http://fedoraproject.org/wiki/Releases/11/Schedule I guess we will miss Fedora 11 with that release, so adding a few checks to the spectime in the meantime would be nice. Cf. https://anna.fi.muni.cz/divine/trac/browser/test/reachability.sh https://anna.fi.muni.cz/divine/trac/browser/test/owcty.sh Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488101] Review Request: django-contact-form - An extensible contact-form application for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488101 Ignacio Vazquez-Abrams ivazquez...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Ignacio Vazquez-Abrams ivazquez...@gmail.com 2009-03-03 04:39:43 EDT --- New Package CVS Request === Package Name: django-contact-form Short Description: An extensible contact-form application for Django Owners: ivazquez Branches: F-9 F-10 EL-5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487312] Review Request: tuned - A dynamic adaptive system tuning daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487312 Phil Knirsch pknir...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Phil Knirsch pknir...@redhat.com 2009-03-03 05:03:45 EDT --- Package successfully built in dist-f11. Closing bug as NEXTRELEASE as per Package Review Guideline. Thanks everyone! Regards, Phil -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487535] Review Request: bltk - Baterry Life Tool Kit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487535 Jiri Skala jsk...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Jiri Skala jsk...@redhat.com 2009-03-03 05:27:09 EDT --- New Package CVS Request === Package Name: bltk Short Description: The Battery Life Tool Kit Owners: jskala Branches: devel InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 477570] Review Request: couchdb - A document database server, accessible via a RESTful JSON API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477570 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #14 from Allisson Azevedo allis...@gmail.com 2009-03-03 06:01:50 EDT --- Hi, i can't see EL-5 branch for CouchDB. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483620] Review Request: libbind - ISC's standard resolver library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483620 --- Comment #4 from Adam Tkac at...@redhat.com 2009-03-03 06:15:39 EDT --- Fixed package + spec: http://atkac.fedorapeople.org/libbind-6.0-0.3.b1.fc11.src.rpm http://atkac.fedorapeople.org/libbind.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488010] Review Request: ibus-table-cangjie - Cang Jie input method for ibus-table.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488010 Caius kaio Chance ccha...@redhat.com changed: What|Removed |Added AssignedTo|peter...@redhat.com |nob...@fedoraproject.org -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=478852
--- Comment #2 from Michael Schwendt bugs.mich...@gmx.net 2009-03-03
06:16:12 EDT ---
Requires: SDL = 1.0
Ancient. Really should be removed in favour of the automatic libsdl soname
dependency.
%configure
[...]
%makeinstall inst_dir=%{buildroot}%{_datadir}/%{name}
Rather:
%configure inst_dir=%{_datadir}/%{name}
[...]
make DESTDIR=%{buildroot} inst_dir=%{_datadir}/%{name} install
%defattr(0644,root,root,0755)
%defattr(-,root,root,-) won't work? Why?
* desktop-file-install warns about the Icon= entry
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 --- Comment #10 from Jiri Moskovcak jmosk...@redhat.com 2009-03-03 06:21:04 EDT --- Fixed spec and srpm: http://jmoskovc.fedorapeople.org/crash-catcher-0.0.1-12.fc10.src.rpm http://jmoskovc.fedorapeople.org/crash-catcher.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 426751] Review Request: ghc-X11 - A Haskell binding to the X11 graphics library.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=426751
Jens Petersen peter...@redhat.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #58 from Jens Petersen peter...@redhat.com 2009-03-03 06:26:04
EDT ---
Okay according to spot this is okay as BSD:
https://www.redhat.com/archives/fedora-legal-list/2009-March/msg2.html
I am sponsoring Yaakov.
Here is the review:
+:ok, =:needs attention, -:needs fixing, NA: not applicable
MUST Items:
[+] MUST: rpmlint output
clean
[+] MUST: Package Naming Guidelines
[+] MUST: spec file name must match base package %{name}
[+] MUST: Packaging Guidelines.
[+] MUST: Licensing Guidelines
[+] MUST: License field in the package spec file must match actual license.
[+] MUST: include license files in %doc if available in source
[+] MUST: The spec file must be written in American English and be legible.
[+] MUST: source md5sum matches upstream release
73a4ba56b8cef69ce3659ab452e4530b X11-1.4.5.tar.gz
[+] MUST: must successfully compile and build into binary rpms on one main arch
[+] MUST: if necessary use ExcludeArch for other archs
[+] MUST: All build dependencies must be listed in BuildRequires
[NA] MUST: use %find_lang macro for .po translations
[NA] MUST: packages which store shared library files in the dynamic linker's
default paths, must call ldconfig in %post and %postun.
[+] MUST: A package must own all directories that it creates.
[+] MUST: A package must not contain any duplicate files in the %files listing.
[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[+] MUST: Each package must consistently use macros, as described in the macros
section of Packaging Guidelines.
[+] MUST: The package must contain code, or permissable content. This is
described in detail in the code vs. content section of Packaging Guidelines.
[+] MUST: Large documentation files should go in a doc subpackage.
[+] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[+] MUST: Header files must be in a -devel package.
[NA] MUST: Static libraries must be in a -static package.
[NA] MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
(for directory ownership and usability).
[NA] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.
[NA] MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency
[+] MUST: Packages must NOT contain any .la libtool archives, these should be
removed in the spec.
[NA] MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section.
[+] MUST: Packages must not own files or directories already owned by other
packages.
[+] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[+] MUST: All filenames in rpm packages must be valid UTF-8.
SHOULD Items:
[+] SHOULD: The reviewer should test that the package builds in mock.
[+] SHOULD: The package should compile and build into binary rpms on all
supported architectures.
http://koji.fedoraproject.org/koji/taskinfo?taskID=1216598
[+] SHOULD: The reviewer should test that the package functions as described.
[+] SHOULD: If scriptlets are used, those scriptlets must be sane.
This package is APPROVED for inclusion in Fedora.
Thanks for your patience as this was also one of the early packages
used as a testcase during revising the Haskell Packaging Guidelines.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 470696] Review Request: rubygem-passenger - Passenger Ruby on Rails deployment system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=470696 --- Comment #20 from Johan Kok johan-fed...@deds.nl 2009-03-03 06:41:54 EDT --- (In reply to comment #18) Phusion Passenger's upstream told me they would list the changes anyway. Jeroen, do you have any news on licensing and/or listing changes from upstream? After a quick look at the Phusion Passenger 2.1.1 (beta, released this week), I can't find a new license and/or a listing of those changes to the mod_scgi code in there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481732] Review Request: stardict-dic-cs_CZ - czech dictionary for stardict
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481732 --- Comment #15 from Petr Sklenar pskle...@redhat.com 2009-03-03 07:55:18 EDT --- yes , you are right, I am sorry, my mistake. again: New Package CVS Request === Package Name: stardict-dic-cs_CZ Short Description: Czech dictionaries for StarDict. Owners: psklenar Branches: F-10 F-11 InitialCC: psklenar -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 473348] Review Request: drraw - Web based presentation front-end for RRDtool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=473348 --- Comment #4 from Xavier Bachelot xav...@bachelot.org 2009-03-03 08:44:12 EDT --- New version, drraw 2.2 beta 2 : http://www.bachelot.org/fedora/SPECS/drraw.spec http://www.bachelot.org/fedora/SRPMS/drraw-2.2-0.4.b2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=487392
--- Comment #11 from Dan Horák d...@danny.cz 2009-03-03 09:10:04 EDT ---
resolved issue
- source archive available to download and is the same as included in the srpm
d026802acb81aa2ec26154fd09361b1c30eb70c9 crash-catcher-0.0.1.tar.gz
remaining issues
- the crash-catcher service is on by default
- don't use --vendor fedora when installing the desktop file and use only
%{_datadir}/applications/%{name}.desktop in %files section for the gui
subpackage
- remove %{_libdir}/%{name} from %files section of the libs subpackage,
because then the directory and all plugins are incorrectly owned by the libs
subpackage
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 428568] Review Request: synfig - Synfig is a vector based 2D animation package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=428568 Kamil Pawlowski kami...@gmail.com changed: What|Removed |Added CC||kami...@gmail.com --- Comment #32 from Kamil Pawlowski kami...@gmail.com 2009-03-03 09:14:20 EDT --- in synfig.spec: sed -i.rpath 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=|g' '\n' libtool should be: sed -i.rpath 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=|g' libtool because sed return error -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 428568] Review Request: synfig - Synfig is a vector based 2D animation package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=428568 --- Comment #33 from Kamil Pawlowski kami...@gmail.com 2009-03-03 09:27:22 EDT --- I have the same problem if I try build synfig from spec or by source, this problem stop me from building synfigstudio. $ gdb synfig (...) warning: /usr/lib/debug/usr/bin/synfig.debug: The separate debug info file has no debug info (no debugging symbols found) (no debugging symbols found) (gdb) r synfig -q synfig_icon.sif -o 128x128/synfig_icon.png --time 0 -w 128 -h 128 Starting program: /usr/bin/synfig synfig -q synfig_icon.sif -o 128x128/synfig_icon.png --time 0 -w 128 -h 128 (no debugging symbols found) (no debugging symbols found) [Thread debugging using libthread_db enabled] [New Thread 0x77fcf700 (LWP 3843)] warning: /usr/lib/debug/usr/lib64/synfig/modules/libmod_gif.so.debug: The separate debug info file has no debug info (...) and all other modules give same error too (...) I/O warning : failed to load external entity synfig synfig(3843) [15:21:09] error: Standard Exception: Couldn't create parsing context Unable to open synfig. Throwing out job... synfig(3843) [15:21:10] info: target name not specified - using png Program received signal SIGSEGV, Segmentation fault. ofcourse i have synfig-debug installed -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487094] Review Request: stp - Constraint solver/decision procedure
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487094 --- Comment #2 from David A. Wheeler dwhee...@dwheeler.com 2009-03-03 09:35:18 EDT --- Okay, posted updated .spec/SRPM. Moved stuff around, used a macro (using %global not %define as suggested by some draft guidance), and removed the dup document file. Hope it's all okay now. I did scratch koji builds for both dist-f10 and dist-f11, and it works for both versions on all architectures: Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1216922 0 free 0 open 5 done 0 failed Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1216974 0 free 0 open 5 done 0 failed rpmlint is clean for the .spec file, SRPM, and a binary RPM (stp-0.1-2.fc11.i586.rpm). Here are the new versions: http://www.dwheeler.com/stp.spec http://www.dwheeler.com/stp-0.1-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487052] Review Request: dc3dd - Patched version of GNU dd for use in computer forensics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487052 --- Comment #2 from Adam Miller maxamill...@gmail.com 2009-03-03 09:48:19 EDT --- Spec URL: http://maxamillion.fedorapeople.org/dc3dd.spec SRPM URL: http://maxamillion.fedorapeople.org/dc3dd-6.12.2-2.src.rpm I've cleaned up everything listed above. I fixed the licensing, removed the .gmo binaries, fixed source0 (sorry for that one, I based this spec off another spec of mine and mixed that up), added the doc items you listed, and fixed EOF encoding for the current and two added files to the package. Thank you very much for your feedback and I look forward to further review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485417 Glauber de Oliveira Costa gco...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #23 from Glauber de Oliveira Costa gco...@redhat.com 2009-03-03 10:00:26 EDT --- Can CC to fedora-virt-ma...@redhat.com be late added? (since I got it wrong in the initial review) I don't see a way to do it myself in packagedb. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487094] Review Request: stp - Constraint solver/decision procedure
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487094 Marcela Maslanova mmasl...@redhat.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Marcela Maslanova mmasl...@redhat.com 2009-03-03 10:04:31 EDT --- ACCEPTED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 457060] Review Request: wordpress-mu - Multi-user variant of WordPress
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=457060 --- Comment #18 from Bret McMillan br...@redhat.com 2009-03-03 10:21:51 EDT --- Package Change Request == Package Name: wordpress-mu New Branches: EL-5 Owners: bretm Description: WordPress-MU is a variant of WordPress that supports more than one blog per instance. It is the basis for the wordpress.com hosted service, and scales to large numbers of users and blogs. I'd like to add an EPEL branch for RHEL 5 support; we've been using it for several months now on that platform. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 457060] Review Request: wordpress-mu - Multi-user variant of WordPress
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=457060 Bret McMillan br...@redhat.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486961] Review Request: libservicelog - Servicelog Database and Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486961 --- Comment #3 from Ondrej Vasik ova...@redhat.com 2009-03-03 10:24:06 EDT --- Additionally groupdel in postun would mean deleting that group even on update (as postun is run after post). Other checks: + package builds in mock (development i386). + rpmlint is silent for SRPM + rpmlint is not silent for RPM. - mostly could be ignored: - .so is in libpackage - servicelog.db file strange permissions are recommended in usptream installation procedure, so I guess required - dangerous groupdel in postun mentioned earlier + source files match upstream. e3280d47a16a29beef586d5b6068a562 libservicelog-1.0.1.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc files present. + BuildRequires are proper. + defattr usage is correct. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code. + no static libraries. - .pc file present (in devel subpackage). - I guess it is necessary for that package - Requires: pkgconfig is missing for devel subpackage + no .la files. + no translations are available. - Does NOT owns the directories it creates. /var/lib/servicelog has to be owned + no duplicates in %files. + file permissions are appropriate. + Not a GUI app. Summary of things to fix: 1) groupdel in post (consider complete removal) 2) var/lib/servicelog has to be owned 3) Requires: pkgconfig is missing for devel subpackage -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486961] Review Request: libservicelog - Servicelog Database and Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486961 --- Comment #4 from Roman Rakus rra...@redhat.com 2009-03-03 11:16:35 EDT --- https://fedoraproject.org/w/uploads/a/ac/Libservicelog.spec spec file updated. Asked upstream about group service. Waiting to answer... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487637] Review Request: rtaudio - Real-time Audio I/O Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487637 --- Comment #14 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-03-03 11:12:49 EDT --- Thanks, and congratulations from me, too! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486961] Review Request: libservicelog - Servicelog Database and Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486961 --- Comment #5 from Ondrej Vasik ova...@redhat.com 2009-03-03 11:33:36 EDT --- I recommend to also add - otherwise pre scriptlet will fail for no shadow-utils or for existing service group: Requires(pre): shadow-utils %pre getent group news /dev/null || /usr/sbin/groupadd service -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486961] Review Request: libservicelog - Servicelog Database and Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486961 --- Comment #6 from Ondrej Vasik ova...@redhat.com 2009-03-03 11:34:46 EDT --- Sorry, I meant: %post getent group service /dev/null || /usr/sbin/groupadd service cut paste is evil ;) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488285] New: Review Request: txt2man - Convert flat ASCII text to man page format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: txt2man - Convert flat ASCII text to man page format https://bugzilla.redhat.com/show_bug.cgi?id=488285 Summary: Review Request: txt2man - Convert flat ASCII text to man page format Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sindr...@fedoraproject.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sindrepb.fedorapeople.org/packages/txt2man.spec SRPM URL: http://sindrepb.fedorapeople.org/packages/txt2man-1.5.5-1.fc10.src.rpm Description: tx2man is a shell script using gnu awk, that should run on any Unix-like system. The syntax of the ASCII text is very straightforward and looks very much like the output of the man(1) program. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 469686] Review Request: ogre-meshmagick - Command line manipulation tool for Ogre meshes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469686 --- Comment #2 from Alexey Torkhov atork...@gmail.com 2009-03-03 11:50:12 EDT --- Spec URL: http://atorkhov.fedorapeople.org/meshmagick.spec SRPM URL: http://atorkhov.fedorapeople.org/meshmagick-0.5.2-2.20090124svn2618.fc10.src.rpm Rpmlint output: meshmagick.src:13: W: configure-without-libdir-spec meshmagick-devel.x86_64: W: no-documentation * Tue Mar 03 2009 Alexey Torkhov atork...@gmail.com - 0.5.2-2.20090124svn2618 - Renamed package to simply meshmagick - Clarified commands to get source - Minor fixes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 469686] Review Request: ogre-meshmagick - Command line manipulation tool for Ogre meshes
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469686 Dan Horák d...@danny.cz changed: What|Removed |Added Status|NEW |ASSIGNED CC||d...@danny.cz AssignedTo|nob...@fedoraproject.org|d...@danny.cz Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484535] Review Request: kde-plasma-networkmanagement - Plasmoid to control Network Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484535 Ben Boeckel maths...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #21 from Ben Boeckel maths...@gmail.com 2009-03-03 12:01:21 EDT --- Built for rawhide, closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488103] Review Request: django-evolution - Schema evolution for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488103 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Michel Alexandre Salim michel.syl...@gmail.com 2009-03-03 12:04:51 EDT --- Change looks good. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475852] Review Request: gnustep-base - GNUstep Base library package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475852 --- Comment #8 from Michel Alexandre Salim michel.syl...@gmail.com 2009-03-03 12:09:40 EDT --- GNUstep's page lists libffi as required and ffcall as optional. Should we block on 475112, or switch requirement to libffi? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483868] Review Request: iscan-firmware - Firmware for Epson flatbed scanners
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483868 --- Comment #8 from Fedora Update System upda...@fedoraproject.org 2009-03-03 12:15:50 EDT --- iscan-firmware-2.1.0-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/iscan-firmware-2.1.0-3.fc10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483868] Review Request: iscan-firmware - Firmware for Epson flatbed scanners
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483868 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2009-03-03 12:16:31 EDT --- iscan-firmware-2.1.0-3.fc9 has been submitted as an update for Fedora 9. http://admin.fedoraproject.org/updates/iscan-firmware-2.1.0-3.fc9 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487296] Review Request: sssd - System Security Services Daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487296 --- Comment #2 from Jakub Hrozek jhro...@redhat.com 2009-03-03 12:15:08 EDT --- The packages required to build are no longer in-tree. They're not in Fedora yet, but they can be obtained by rebuilding the samba-base SRPM that is placed at: http://simo.fedorapeople.org/samba-base/SRPMS/ I've updated the packages linked in the opening comment. I'll also update this bug again once the packages are in Fedora. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486961] Review Request: libservicelog - Servicelog Database and Library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486961 --- Comment #7 from Roman Rakus rra...@redhat.com 2009-03-03 12:15:30 EDT --- updated -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475852] Review Request: gnustep-base - GNUstep Base library package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475852 Jochen Schmitt joc...@herr-schmitt.de changed: What|Removed |Added Blocks|459211 |475112 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459211] Review Request: oolite - A space sim game, inspired by Elite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459211 Jochen Schmitt joc...@herr-schmitt.de changed: What|Removed |Added Depends on|475852 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487296] Review Request: sssd - System Security Services Daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487296 Martin Nagy mn...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||mn...@redhat.com AssignedTo|nob...@fedoraproject.org|mn...@redhat.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481594] Review Request: python-pygooglechart - A complete Python wrapper for the Google Chart API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481594 --- Comment #10 from Jochen Schmitt joc...@herr-schmitt.de 2009-03-03 12:25:06 EDT --- Please close this bug, if you have push this new package on bodhi. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475852] Review Request: gnustep-base - GNUstep Base library package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475852 --- Comment #9 from Jochen Schmitt joc...@herr-schmitt.de 2009-03-03 12:23:02 EDT --- I have blocked and ping the maintainer of libffi. If they may not reply in the next 7 days, I may make the fix by mayslf, because it's trivial and open a AWOL processs. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485621] Review Request: perl-Gtk2-MozEmbed - Interface to the Mozilla embedding widget
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=485621
--- Comment #3 from Chris Weyl cw...@alumni.drew.edu 2009-03-03 12:24:25 EDT
---
Basically, Perl packages should own everything they provide (including
directories) under %{perl_vendorarch} or %{perl_vendorlib} except
%{perl_vendorarch}/auto Even if they're already owned by another Perl
package. (Rationale being that as we move to 5.10.1, 5.10.2, etc, we will see
directory ownership start to be a bit fuzzy.)
https://fedoraproject.org/wiki/Packaging:Perl#Directory_Ownership
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480567] Review Request: dnssec-conf - DNSSEC and DLV configuration and priming tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480567 --- Comment #8 from Jochen Schmitt joc...@herr-schmitt.de 2009-03-03 12:26:26 EDT --- Please close this bug after you have push your package on bodhi -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 --- Comment #12 from Jiri Moskovcak jmosk...@redhat.com 2009-03-03 12:41:29 EDT --- Updated files: http://jmoskovc.fedorapeople.org/crash-catcher-0.0.1-12.fc10.src.rpm http://jmoskovc.fedorapeople.org/crash-catcher.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485621] Review Request: perl-Gtk2-MozEmbed - Interface to the Mozilla embedding widget
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485621 --- Comment #4 from Remi Collet fed...@famillecollet.com 2009-03-03 12:56:30 EDT --- Thanks for pointing me to this Guidelines section I haven't noticed... It seems clear now :) http://remi.fedorapeople.org/perl-Gtk2-MozEmbed.spec http://remi.fedorapeople.org/perl-XML-Entities-0.02-3.fc8.src.rpm %changelog * Tue Mar 03 2009 Remi Collet fed...@famillecollet.com - 0.08-3 - Fix from review : Gtk2 dir ownership -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 Dan Horák d...@danny.cz changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #13 from Dan Horák d...@danny.cz 2009-03-03 13:31:41 EDT --- All issues are fixed, this package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488198] Review Request: sugar-update-control - Activity update control panel for Sugar
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=488198
Suravee Suthikulpanit suravee.suthikulpa...@amd.com changed:
What|Removed |Added
CC||suravee.suthikulpa...@amd.c
||om
--- Comment #1 from Suravee Suthikulpanit suravee.suthikulpa...@amd.com
2009-03-03 13:36:40 EDT ---
This is an unofficial review:
Please check the following:
---
- rpmlint -iv sugar-update-control.spec
sugar-update-control.spec:42: E: files-attr-not-set
A file or a directory entry in a %files section does not have attributes set
which may result in security issues in the resulting binary package depending
on the system where the package is built. Add default attributes using
%defattr before it in the %files section, or use per line %attr's.
sugar-update-control.spec:44: E: files-attr-not-set
A file or a directory entry in a %files section does not have attributes set
which may result in security issues in the resulting binary package depending
on the system where the package is built. Add default attributes using
%defattr before it in the %files section, or use per line %attr's.
sugar-update-control.spec:45: E: files-attr-not-set
A file or a directory entry in a %files section does not have attributes set
which may result in security issues in the resulting binary package depending
on the system where the package is built. Add default attributes using
%defattr before it in the %files section, or use per line %attr's.
0 packages and 1 specfiles checked; 3 errors, 0 warnings.
---
- rpmlint -iv
~/rpmbuild/RPMS/noarch/sugar-update-control-0.20-1.fc10.noarch.rpm
sugar-update-control.noarch: I: checking
sugar-update-control.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/bitfrost/util/urlrange.py 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed. If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.
sugar-update-control.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/bitfrost/update/microformat.py 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed. If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.
sugar-update-control.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/bitfrost/update/actutils.py 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed. If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.
sugar-update-control.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/bitfrost/update/actinfo.py 0644
This text file contains a shebang or is located in a path dedicated for
executables, but lacks the executable bits and cannot thus be executed. If
the file is meant to be an executable script, add the executable bits,
otherwise remove the shebang or move the file elsewhere.
sugar-update-control.noarch: E: incorrect-locale-subdir
/usr/share/locale/pseudo/LC_MESSAGES/sugar-update-control.mo
1 packages and 0 specfiles checked; 5 errors, 0 warnings.
---
Please see http://fedoraproject.org/wiki/PackagingGuidelines#File_Permissions.
Also, rerun rpmlint -iv on the src.rpm and rpm before submission. The
above errors seems to be fixed by adding %defattr(-,root,root,-) in the file
section. However, rpmlint -iv gives many other warnings.
- It seems like the /usr/share/sugar directory is own by other package.
Please see
http://fedoraproject.org/wiki/PackagingGuidelines#File_and_Directory_Ownership
SUGGESTIONS:
- Use %{_datadir} instead of /usr/share
- Please provide instruction on how to get the latest upstream package (tar.gz)
for checksum comparison.
- Use rm -rf instead of rm
OKAYS:
- Lincensing (GPLv2+), license file included in the package.
- Buildroot is correct
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488198] Review Request: sugar-update-control - Activity update control panel for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488198 --- Comment #2 from Suravee Suthikulpanit suravee.suthikulpa...@amd.com 2009-03-03 13:37:44 EDT --- Also check out http://fedoraproject.org/wiki/Packaging/Python -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483615] Review Request: CodeAnalyst - Performance Analysis Suite for AMD-based System (based on Oprofile)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483615 --- Comment #16 from Suravee Suthikulpanit suravee.suthikulpa...@amd.com 2009-03-03 13:36:29 EDT --- Help review BUG 488198 (https://bugzilla.redhat.com/show_bug.cgi?id=488198) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 --- Comment #14 from Jiri Moskovcak jmosk...@redhat.com 2009-03-03 13:45:38 EDT --- New Package CVS Request === Package Name: abrt Short Description: Automatic bug detection and reporting tool Owners: jmosk...@redhat.com zprik...@redhat.com Branches: F-10 F-11 #abrt is a new name for crash-catcher, since it's already a trademark.. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488333] New: Review Request: nmh - the MH mail handling system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: nmh - the MH mail handling system https://bugzilla.redhat.com/show_bug.cgi?id=488333 Summary: Review Request: nmh - the MH mail handling system Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rals...@pobox.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Package Change Request == Package Name: nmh New Branches: EL-4 EL-5 Owners: ralston I and another person both asked the current Fedora maintainer of this package to maintain it for EPEL, but the maintainer has not responded to the requests: https://bugzilla.redhat.com/show_bug.cgi?id=475479 Furthermore, the Fedora maintainer has not indicated his desire (or lack thereof) to participate in EPEL via the Contributors Status document: http://fedoraproject.org/wiki/EPEL/ContributorStatus Therefore, I am requesting EL-4 and EL-5 branches for this package, and to become a co-maintainer. (There was no existing Review Request for this package, which is why I am creating a new one.) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: crash-catcher - apps crash detecting daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 Jiri Moskovcak jmosk...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488333] Review Request: nmh - the MH mail handling system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488333 James Ralston rals...@pobox.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488333] Review Request: nmh - the MH mail handling system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488333 James Ralston rals...@pobox.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|rals...@pobox.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480567] Review Request: dnssec-conf - DNSSEC and DLV configuration and priming tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480567 Paul Wouters p...@xelerance.com changed: What|Removed |Added Status|RELEASE_PENDING |CLOSED Resolution||CURRENTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 250804] Review Request: perl-Encode-Detect - Detects the encoding of data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=250804 James Ralston rals...@pobox.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #26 from James Ralston rals...@pobox.com 2009-03-03 13:56:36 EDT --- Yes, I think we can close this. :-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487114] Review Request: gvrpcd - A program for announcing VLANs using GVRP.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487114 --- Comment #5 from Jasper Capel ca...@stone-it.com 2009-03-03 13:58:39 EDT --- I created a new init-script from scratch, this one should be compliant. I'll advise upstream about the license, and the broken link in the README (I think this should be http://wiki.wireshark.org/GVRP). I added some comments to the sysconfig-file, and commented the option-lines, so they should now only be set if a user wants to override defaults set in the init script. rpmlint does still complain: gvrpcd.x86_64: W: service-default-enabled /etc/rc.d/init.d/gvrpcd It made sense to me to enable gvrpcd by default when installed - it is not a network daemon listening on any sockets, so it shouldn't be a security risk. On the other hand, if it's not started the server might not even be able to communicate. If you don't share this view, I'll disable it as it's not a big deal to enable the daemon by hand either. :) gvrpcd.x86_64: W: incoherent-subsys /etc/rc.d/init.d/gvrpcd $prog I think this is because of the $prog, instead of gvrpcd in the init-script and can be ignored. New files: SRPM: http://bender.newnewyork.nl/review/gvrpcd-1.1-3.fc11.src.rpm SPEC: http://bender.newnewyork.nl/review/gvrpcd.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487421] Review Request: libyaml - YAML 1.1 parser and emitter written in C
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487421 --- Comment #12 from John Eckersberg jecke...@redhat.com 2009-03-03 14:00:48 EDT --- I have pulled the .a file out of the -devel package and rebuilt. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 486657] Review Request: blahtexml - Converts TeX equations to MathML
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=486657 --- Comment #11 from Jasper Capel ca...@stone-it.com 2009-03-03 14:12:09 EDT --- Thanks! :) I applied to the packager group in FAS (account name: jasper). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475861] Review Request: gnustep-gui - The GNUstep gui library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475861 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Depends on||475852 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475861] Review Request: gnustep-gui - The GNUstep gui library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475861 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Blocks||476056 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459211] Review Request: oolite - A space sim game, inspired by Elite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459211 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Depends on||475852 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476056] Review Request: gnustep-back - The GNUstep backend library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476056 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added CC||michel.syl...@gmail.com Depends on||475861 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475852] Review Request: gnustep-base - GNUstep Base library package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475852 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Blocks|475112 |459211, 475861 Depends on||475112 --- Comment #10 from Michel Alexandre Salim michel.syl...@gmail.com 2009-03-03 14:29:23 EDT --- Jochen: ffcall, not libffi. We're depending on 475112, not blocking 475112 -- we're still blocking oolite, which I've added back. Making this block gnustep-gui as well, and gnustep-gui block gnustep-backend. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478852 --- Comment #3 from Marcin Zajaczkowski msz...@wp.pl 2009-03-03 14:30:16 EDT --- Thanks guys for your suggestion. I made new SPEC with following changes: - replaced %makeinstall - added desktop-file-utils to BuildRequires - removed --vendor fedora - removed update-desktop-database - removed SDL from Requires http://timeoff.wsisiz.edu.pl/rpms/lpairs/lpairs.spec http://timeoff.wsisiz.edu.pl/rpms/lpairs/lpairs-1.0.4-3.src.rpm %defattr(-,root,root,-) won't work? Why? Probably works. I have never used them. I started using RPMs when original packages were made only by RedHat and there were many user pages with packages. I was safer in my opinion to force mentioned rights (especially for suided files). Are there situations where (-,root,root,-) is better? Automatic builds uses (0644,root,root,0755) anyway. * desktop-file-install warns about the Icon= entry Probably I have too old desktop-file-install (Fedora 8), but I didn't see any warning. Is it about .png extension? If yes I would have to patch original file or create a new one. Is it worth to do that? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478852 --- Comment #4 from Alexey Torkhov atork...@gmail.com 2009-03-03 14:49:12 EDT --- (In reply to comment #3) %defattr(-,root,root,-) won't work? Why? Probably works. I have never used them. I started using RPMs when original packages were made only by RedHat and there were many user pages with packages. I was safer in my opinion to force mentioned rights (especially for suided files). Are there situations where (-,root,root,-) is better? Automatic builds uses (0644,root,root,0755) anyway. Well, by guidelines you should use (-,root,root,-) unless you have a very good reason to deviate from that: https://fedoraproject.org/wiki/Packaging/Guidelines#File_Permissions * desktop-file-install warns about the Icon= entry Probably I have too old desktop-file-install (Fedora 8), but I didn't see any warning. Is it about .png extension? If yes I would have to patch original file or create a new one. Is it worth to do that? Yes, short name should be without extension. Add CC-BY-SA to License tag as package includes some of tango icons. License status of fonts and sounds must be clarified. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 446097] Review Request: pytraffic - sliding block puzzle board game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=446097 Alexey Torkhov atork...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED Flag|needinfo?(sindr...@fedorapr |fedora-review? |oject.org) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478852 Alexey Torkhov atork...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485418 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added CC||a.bad...@gmail.com --- Comment #22 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-03-03 15:01:29 EDT --- (In reply to comment #21) (In reply to comment #20) I don't see virt-maint in the list of cc lists we can add. It's fedora-virt-ma...@redhat.com. Can it be added? I can do that. How is this different from xen-ma...@redhat.com? Should I just replace that with this one or do they exist for separate purposes? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 446097] Review Request: pytraffic - sliding block puzzle board game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=446097 Alexey Torkhov atork...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|atork...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485418 --- Comment #23 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-03-03 15:06:19 EDT --- hmm... and looking at the mailing list, it looks like there is an account for this already -- Someone seems to have set up virtmaint to go there. This is a tiny bit dangerous as it's a real account, not one of our pseudo accounts. So it will be subject to account expiration, someone could compromise its password, etc -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478852] Review Request: lpairs - Classical memory game with cards
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478852 Alexey Torkhov atork...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|atork...@gmail.com -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485418 --- Comment #24 from Eduardo Habkost ehabk...@redhat.com 2009-03-03 15:11:11 EDT --- (In reply to comment #22) It's fedora-virt-ma...@redhat.com. Can it be added? I can do that. How is this different from xen-ma...@redhat.com? Should I just replace that with this one or do they exist for separate purposes? xen-maint is used for different purposes. Please keep xen-maint as is. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488347] New: Review Request: kdepim3 - Compatibility support for kdepim3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kdepim3 - Compatibility support for kdepim3 https://bugzilla.redhat.com/show_bug.cgi?id=488347 Summary: Review Request: kdepim3 - Compatibility support for kdepim3 Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: rdie...@math.unl.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://rdieter.fedorapeople.org/pkg-reviews/kdepim3/kdepim3.spec SRPM URL: http://rdieter.fedorapeople.org/pkg-reviews/kdepim3/kdepim3-3.5.10-1.src.rpm Description: Compatibility support for kdepim3, including libkcal. Needed for some legacy kde3 applications, including taskjuggler. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488035] Review Request: tigervnc - A TigerVNC remote display system - rename from TightVNC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488035 --- Comment #1 from Dan Horák d...@danny.cz 2009-03-03 15:15:20 EDT --- formal review is here, see the notes below: OK source files match upstream (no difference between the included archive and fresh svn checkout): 7957167797dd10c5f3542dab4aaa25834aab41ec tigervnc-0.0.90-20090302svn3621.tar.bz2 OK package meets naming and versioning guidelines. OK specfile is properly named, is cleanly written and uses macros consistently. OK dist tag is present. OK license field matches the actual license. OK license is open source-compatible (GPLv2+). License text included in package. OK latest version is being packaged. OK BuildRequires are proper. OK compiler flags are appropriate. OK %clean is present. OK package builds in mock (Rawhide/x86_64). OK debuginfo package looks complete. OK* rpmlint is silent. OK final provides and requires look sane. N/A %check is present and all tests pass. OK no shared libraries are added to the regular linker search paths. OK owns the directories it creates. OK doesn't own any directories it shouldn't. OK no duplicates in %files. OK file permissions are appropriate. OK* correct scriptlets present. OK code, not content. OK documentation is small, so no -docs subpackage is necessary. OK %docs are not necessary for the proper functioning of the package. OK no headers. OK no pkgconfig files. OK no libtool .la droppings. OK* is GUI app, desktop file is present - rpmlint complains a bit tigervnc-server-module.x86_64: W: no-documentation = OK tigervnc-server-module.x86_64: W: filename-too-long-for-joliet tigervnc-server-module-0.0.90-0.1.20090302svn3621.fc11.x86_64.rpm = OK tigervnc-server.x86_64: W: incoherent-init-script-name vncserver = OK tigervnc.src: W: mixed-use-of-spaces-and-tabs (spaces: line 18, tab: line 1) = please fix - there is new version of icon-cache scriptlets approved (IIRC), but not yet moved from Drafts, so keep the current one - drop --vendor fedora when install the desktop file - you can include unix/README as %doc, it contains some acknowledgments that would be nice to distribute -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488347] Review Request: kdepim3 - Compatibility support for kdepim3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488347 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks||467136 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488347] Review Request: kdepim3 - Compatibility support for kdepim3
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488347 --- Comment #1 from Rex Dieter rdie...@math.unl.edu 2009-03-03 15:23:23 EDT --- better SRPM URL: SRPM URL: http://rdieter.fedorapeople.org/pkg-reviews/kdepim3/kdepim3-3.5.10-1.fc11.src.rpm Scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1218022 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488035] Review Request: tigervnc - A TigerVNC remote display system - rename from TightVNC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488035 --- Comment #2 from Adam Tkac at...@redhat.com 2009-03-03 15:26:21 EDT --- Updated package: http://atkac.fedorapeople.org/tigervnc-0.0.90-0.2.20090302svn3621.fc11.src.rpm http://atkac.fedorapeople.org/tigervnc.spec -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488333] Review Request: nmh - the MH mail handling system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488333 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #1 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:28:44 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488035] Review Request: tigervnc - A TigerVNC remote display system - rename from TightVNC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488035 Adam Tkac at...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Adam Tkac at...@redhat.com 2009-03-03 15:31:14 EDT --- New Package CVS Request === Package Name: tigervnc Short Description: A TigerVNC remote display system Owners: atkac Branches: devel InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488035] Review Request: tigervnc - A TigerVNC remote display system - rename from TightVNC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488035 Dan Horák d...@danny.cz changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Dan Horák d...@danny.cz 2009-03-03 15:29:40 EDT --- All issues fixed, package is APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487392] Review Request: abrt - Automatic bug detection and reporting tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487392 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Summary|Review Request: |Review Request: abrt - |crash-catcher - apps crash |Automatic bug detection and |detecting daemon|reporting tool Flag|fedora-cvs? |fedora-cvs+ --- Comment #15 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:32:19 EDT --- Please use FAS names in Owners field. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481732] Review Request: stardict-dic-cs_CZ - czech dictionary for stardict
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481732 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #16 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:36:06 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485418 --- Comment #25 from Toshio Ernie Kuratomi a.bad...@gmail.com 2009-03-03 15:35:31 EDT --- Okay. I've added virtmaint on the CC. Do you know who created/is responsible for virtmaint? I need to get it straightened out if it should be a pseudo account or we're likely to have problems down the line. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487535] Review Request: bltk - Baterry Life Tool Kit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487535 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #4 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:37:39 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488106] Review Request: django-notification - User notification management for the Django web framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488106 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:43:28 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488035] Review Request: tigervnc - A TigerVNC remote display system - rename from TightVNC
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488035 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #5 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:40:15 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488099] Review Request: python-polib - A library to parse and manage gettext catalogs
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488099 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:41:37 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 488101] Review Request: django-contact-form - An extensible contact-form application for Django
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=488101 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-03-03 15:42:34 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
