[Bug 506339] Review Request: XZ Utils - LZMA Utils with newer file format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506339 --- Comment #3 from Jindrich Novy jn...@redhat.com 2009-06-17 02:47:09 EDT --- Milos, thanks for taking the review and posting here the mailing WRT the XZ utils packaging. I see I was not the first one to package XZ :) The lzma and xz does conflict so better to IMO obsolete the old lzma by xz. I added the obsoletion and provides to the spec. XZ supports old lzma format as well az XZ so there should be no compatibility issues. Ville, thanks for your spec. It helped me to add the pkgconfig dependency to xz-devel. Another thing is version which is wierd - 4.999.8beta. Should we move the beta to release as Ville did? This seems to be the last release before the XZ-5 so is that worth the trouble? I'll ask upstream about the lzmainfo. BTW. spec and SRPM is now 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 506355] Review Request: munge - Uid 'N' Gid Emporium
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506355 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506425] New: Review Request: lightsquid - report parser and visualyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: lightsquid - report parser and visualyzer https://bugzilla.redhat.com/show_bug.cgi?id=506425 Summary: Review Request: lightsquid - report parser and visualyzer Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: alek...@oscada.org.ua QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://aleksey2005.fedorapeople.org/lightsquid.spec SRPM URL: http://aleksey2005.fedorapeople.org/lightsquid-1.7.1-2.fc10.src.rpm Description: Report parser and visualyzer, generate sort of report light fast no database required no additional perl modules small disk usage template html. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 --- Comment #2 from Chitlesh GOORAH chitl...@gmail.com 2009-06-17 05:28:07 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-veditor.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-veditor-0.6.3-2.fc11.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506425] Review Request: lightsquid - report parser and visualyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506425 --- Comment #1 from Aleksey Popkov alek...@oscada.org.ua 2009-06-17 05:31:09 EDT --- Spec URL: http://aleksey2005.fedorapeople.org/lightsquid.spec SRPM URL: http://aleksey2005.fedorapeople.org/lightsquid-1.7.1-2.fc10.src.rpm Description: Lite, small size and fast log analizer for squid proxy. -- Configure 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 506429] New: Review Request: eclipse-eclox - Eclipse-based doxygen plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: eclipse-eclox - Eclipse-based doxygen plugin https://bugzilla.redhat.com/show_bug.cgi?id=506429 Summary: Review Request: eclipse-eclox - Eclipse-based doxygen plugin Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: chitl...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-eclox.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-eclox-0.8.0-1.20090616svn.fc11.src.rpm Description: Eclox is a doxygen frontend plug-in for eclipse. It aims to provide a slim and sleek integration of the code documentation process into eclipse. -- Configure 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 506431] New: Review Request: eclipse-texlipse - Eclipse-based Latex plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: eclipse-texlipse - Eclipse-based Latex plugin https://bugzilla.redhat.com/show_bug.cgi?id=506431 Summary: Review Request: eclipse-texlipse - Eclipse-based Latex plugin Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: chitl...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-texlipse.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-texlipse-1.3.0-1.20090616cvs.fc11.src.rpm Description: Texlipse is a plugin that adds Latex editing support for the popular Eclipse Java IDE. Key features include: syntax highlight, command completion, bibliography completion, outline navigation and automatic building. -- Configure 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 459548] Review Request: php-gtk - GTK PHP extension
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459548 --- Comment #14 from Patrice FERLET meta...@gmail.com 2009-06-17 05:51:11 EDT --- Since several month I'm full of work and I couldn't work on packages. I will work on my packages soon. For now, I have to use F11 and recompile my packages. Regards -- Configure 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 505928] Review Request: hdapsd - Protects hard drives by parking head when fall is detected
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505928 --- Comment #2 from Tomasz Torcz zdzi...@irc.pl 2009-06-17 05:48:53 EDT --- I've uploaded -2 version: Spec URL: https://pipebreaker.pl/~zdzichu/pkgs/hdapsd.spec SRPM URL: https://pipebreaker.pl/~zdzichu/pkgs/hdapsd-20090401-2.fc11.src.rpm which marks upstart udev files as config files. I've left other as non-marked as doc, because %doc causes them to be installed two times. Thank you for your comments Steve! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506425] Review Request: lightsquid - report parser and visualyzer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506425 --- Comment #2 from Aleksey Popkov alek...@oscada.org.ua 2009-06-17 06:02:43 EDT --- Spec URL: http://aleksey2005.fedorapeople.org/lightsquid.spec SRPM URL: http://aleksey2005.fedorapeople.org/lightsquid-1.7.1-2.fc10.src.rpm Description: Lite, small size and fast log analizer for squid proxy. -- Configure 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 506450] New: Package Review: twitter-glib - A library wrapping the Twitter RESTful API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Package Review: twitter-glib - A library wrapping the Twitter RESTful API https://bugzilla.redhat.com/show_bug.cgi?id=506450 Summary: Package Review: twitter-glib - A library wrapping the Twitter RESTful API Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pbrobin...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Blocks: 506446 Estimated Hours: 0.0 Classification: Fedora SPEC: http://pbrobinson.fedorapeople.org/twitter-glib.spec SRPM: http://pbrobinson.fedorapeople.org/twitter-glib-0.9.8-1.fc11.src.rpm Twitter-GLib is a GObject-based library providing a wrapper around the RESTful API of Twitter. -- Configure 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 505942] Review Request: php-LightweightPicasaAPI - A lightweight API for Picasa in PHP
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=505942
David Nalley da...@gnsa.us changed:
What|Removed |Added
Status|NEW |ASSIGNED
Flag||fedora-review+
--- Comment #2 from David Nalley da...@gnsa.us 2009-06-17 09:17:13 EDT ---
OK MUST: rpmlint must be run on every package. The output should be posted in
the review.
[ke4...@localhost SPECS]$ rpmlint ./php-LightweightPicasaAPI.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@localhost SPECS]$ rpmlint
../SRPMS/php-LightweightPicasaAPI-3.3-1.fc11.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@localhost SPECS]$ rpmlint
../RPMS/noarch/php-LightweightPicasaAPI-3.3-1.fc11.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
OK MUST: The package must be named according to the Package Naming Guidelines
.
OK MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. .
OK MUST: The package must meet the Packaging Guidelines .
OK MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
GPLv3+
OK MUST: The License field in the package spec file must match the actual
license.
header in picasa.php states GPLv3, and refers to license.txt, which has GPLv3
or later.
OK MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
OK MUST: The spec file must be written in American English.
OK MUST: The spec file for the package MUST be legible.
[] MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[ke4...@localhost SOURCES]$ md5sum LightweightPicasaAPIv3.zip*
d7f26f31cb82a8b514ce02394a8a067f LightweightPicasaAPIv3.zip
d7f26f31cb82a8b514ce02394a8a067f LightweightPicasaAPIv3.zip.1
OK MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
NA MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
No real build - so only dos2unix is needed and specified as a build requires
NA MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
NA MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
OK MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
OK MUST: A Fedora package must not list a file more than once in the spec
file's %files listings.
OK MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.
OK MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
OK MUST: Each package must consistently use macros.
OK MUST: The package must contain code, or permissable content.
OK MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity).
OK MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present.
NA MUST: Header files must be in a -devel package.
NA MUST: Static libraries must be in a -static package.
[Bug 506450] Package Review: twitter-glib - A library wrapping the Twitter RESTful 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=506450 Peter Robinson pbrobin...@gmail.com changed: What|Removed |Added Blocks||506476 -- Configure 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 506476] New: Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Package Review: rest - A library for access to RESTful web services https://bugzilla.redhat.com/show_bug.cgi?id=506476 Summary: Package Review: rest - A library for access to RESTful web services Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pbrobin...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Depends on: 506450 Blocks: 506446 Estimated Hours: 0.0 Classification: Fedora SPEC: http://pbrobinson.fedorapeople.org/rest.spec SRPM: http://pbrobinson.fedorapeople.org/rest-0.3-1.fc11.src.rpm This library was designed to make it easier to access web services that claim to be RESTful. A RESTful service should have urls that represent remote objects, which methods can then be called on. The majority of services don't actually adhere to this strict definition. Instead, their RESTful end point usually has an API that is just simpler to use compared to other types of APIs they may support (XML-RPC, for instance). It is this kind of API that this library is attempting to support. -- Configure 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 477683] Review Request: fltk2 - C++ user interface toolkit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477683 --- Comment #17 from Michal Nowak mno...@redhat.com 2009-06-17 09:24:53 EDT --- Thanks for tips Michael, I finally figured it all out: %changelog * Wed Jun 17 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.11.r6786 - disabling the workaroung for fedora 11 * Wed Jun 17 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.10.r6786 - rpath killer - ® sign killer * Wed Jun 17 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.9.r6786 - setting correct soname * Wed Jun 17 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.8.r6786 - rebuild * Wed Jun 17 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.7.r6786 - fltk-2.0.x-r6786-scandir-workaround.patch to workaroung glibc-2.10 non standard behavior * Tue Jun 16 2009 Michal Nowak mno...@redhat.com - 2.0.x-0.6.r6786 - r6786 - fltk-2.0.x-r6671-non-silence-build.patch new...@dhcp-lab-124 SPECS $ rpmlint /home/newman/rpmbuild/SRPMS/fltk2-2.0.x-0.11.r6786.fc11.src.rpm /home/newman/rpmbuild/RPMS/x86_64/fltk2-2.0.x-0.11.r6786.fc11.x86_64.rpm /home/newman/rpmbuild/RPMS/x86_64/fltk2-devel-2.0.x-0.11.r6786.fc11.x86_64.rpm /home/newman/rpmbuild/RPMS/x86_64/fltk2-fluid-2.0.x-0.11.r6786.fc11.x86_64.rpm /home/newman/rpmbuild/RPMS/x86_64/fltk2-doc-2.0.x-0.11.r6786.fc11.x86_64.rpm /home/newman/rpmbuild/RPMS/x86_64/fltk2-debuginfo-2.0.x-0.11.r6786.fc11.x86_64.rpm fltk2.x86_64: W: shared-lib-calls-exit /usr/lib64/libfltk2.so.2.0 e...@glibc_2.2.5 6 packages and 0 specfiles checked; 0 errors, 1 warnings. http://mnowak.fedorapeople.org/fltk2/fltk2.spec http://mnowak.fedorapeople.org/fltk2/fltk2-2.0.x-0.11.r6786.fc11.src.rpm dillo-2.1 (pre-release) compilation run tested against this version. -- Configure 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 502477] Review Request: arista - Easy to use multimedia transcoder for the GNOME Desktop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502477 Marcin marcin_wolyn...@aster.pl changed: What|Removed |Added CC||marcin_wolyn...@aster.pl --- Comment #6 from Marcin marcin_wolyn...@aster.pl 2009-06-17 09:29:40 EDT --- (In reply to comment #3) arista-0.9.1-2.fc11.noarch [fkoo...@localhost ~]$ arista-gtk Traceback (most recent call last): File /usr/bin/arista-gtk, line 1244, in module arista.utils.get_path(locale)) File /usr/lib/python2.6/site-packages/arista/utils.py, line 75, in get_path path: path, IOError: Can't find locale in any known prefix! [fkoo...@localhost ~]$ Maybe a dependency missing? Missing locale subdir from sources in /usr/share/arista -- Configure 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 505358] Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505358 David Nalley da...@gnsa.us changed: What|Removed |Added Status|NEW |ASSIGNED CC||da...@gnsa.us AssignedTo|nob...@fedoraproject.org|da...@gnsa.us Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500082] Review Request: postal - mail server benchmark
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500082 David Nalley da...@gnsa.us changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #9 from David Nalley da...@gnsa.us 2009-06-17 09:34:34 EDT --- This has been built and pushed to stable - sorry for not adding the bug number to bodhi and thus the failure to autoclose -- Configure 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 497310] Review Request: php-simplepie - Simple RSS Library in PHP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497310 David Nalley da...@gnsa.us changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #12 from David Nalley da...@gnsa.us 2009-06-17 09:37:14 EDT --- This has been pushed to stable - apparently forgot to note the bug number in bodhi - apologized for letting it fall through the cracks. -- Configure 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 497140] Review Request: php-IDNA_Convert - Internationalized domain name to UTF-8 converter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=497140 David Nalley da...@gnsa.us changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||CURRENTRELEASE --- Comment #11 from David Nalley da...@gnsa.us 2009-06-17 09:36:42 EDT --- This has been pushed to stable - apparently forgot to note the bug number in bodhi - apologized for letting it fall through the cracks. -- Configure 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 506476] Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506476 Peter Robinson pbrobin...@gmail.com changed: What|Removed |Added Blocks||506486 -- Configure 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 506486] New: Package Review: mojito - A social network data aggregator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Package Review: mojito - A social network data aggregator https://bugzilla.redhat.com/show_bug.cgi?id=506486 Summary: Package Review: mojito - A social network data aggregator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: pbrobin...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Depends on: 506476 Blocks: 506446 Estimated Hours: 0.0 Classification: Fedora SPEC: http://pbrobinson.fedorapeople.org/mojito.spec SRPM: http://pbrobinson.fedorapeople.org/mojito-0.10.3-1.fc11.src.rpm Mojito is a social data server which will fetch data from the social web, such as your friend's blog posts and photos, upcoming events, recently played tracks, and pending eBay* auctions. It also provides a service to update your status on web services which support it, such as MySpace* and Twitter*. -- Configure 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 226210] Merge Review: opal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226210 --- Comment #21 from Peter Robinson pbrobin...@gmail.com 2009-06-17 10:17:18 EDT --- I don't think that in prep will be of any use as its still in the tarball unless its only an issue if we ship/use it in the final rpms (which we don't anyway as the binaries are only used by the windows builds). I don't believe any of the precompiled stuff is used by the linux build at all. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506425] Review Request: lightsquid - a light, small and fast log analyzer for squid proxy
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506425 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Summary|Review Request: lightsquid |Review Request: lightsquid |- report parser and |- a light, small and fast |visualyzer |log analyzer for squid ||proxy -- Configure 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=505017
--- Comment #3 from Alexander Kurtakov akurt...@redhat.com 2009-06-17
10:47:27 EDT ---
Formal review:
[OK] MUST: rpmlint must be run on every package. No warnings/errors.
[OK] MUST: The package must be named according to the Package Naming Guidelines
[OK] MUST: The spec file name match the base package %{name}
[OK] MUST: The package must meet the Packaging Guidelines
[OK] MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
[OK] MUST: The License field in the package spec file must match the actual
license.
[OK] MUST: The spec file must be written in American English.
[OK] MUST: The spec file for the package MUST be legible.
[FAIL] MUST: The sources used to build the package must match the upstream
source, as provided in the spec URL. Source0 URL is not existing.
[OK] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[OK] MUST: All build dependencies must be listed in BuildRequires, except for
any that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional.
[OK] MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
[OK] MUST: A Fedora package must not list a file more than once in the spec
file's %files listings.
[OK] MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.
[OK] MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[OK] MUST: Each package must consistently use macros.
[OK] MUST: The package must contain code, or permissable content.
[OK] MUST: If a package includes something as %doc, it must not affect the
runtime of the application.
[OK] MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here is that the first package to be installed
should own the files or directories that other packages may rely upon.
[OK] MUST: At the beginning of %install, each package MUST run rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
[OK] MUST: All filenames in rpm packages must be valid UTF-8.
[TODO] SHOULD: If the source package does not include license text(s) as a
separate file from upstream, the packager SHOULD query upstream to include it.
The only fix needed is to fix Source0 URL.
And it would be good to ask upstream to include license file, but it's not
blocking this 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 --- Comment #4 from Chitlesh GOORAH chitl...@gmail.com 2009-06-17 10:55:26 EDT --- I have uploaded the SRPM with respect to the source0 url. See release -2 again -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505358] Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=505358
--- Comment #1 from David Nalley da...@gnsa.us 2009-06-17 10:56:45 EDT ---
%{_datadir}/php/*
You really want to own EVERYTHING in /usr/share/php/?
--
Configure 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 505358] Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505358 --- Comment #2 from Patrick Monnerat p...@datasphere.ch 2009-06-17 11:03:17 EDT --- Yes ! That is: everything that is installed by this package. This notation does not concern components outside the current package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 --- Comment #5 from Chitlesh GOORAH chitl...@gmail.com 2009-06-17 11:00:05 EDT --- Updated: Spec URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-veditor.spec SRPM URL: http://chitlesh.fedorapeople.org/RPMS/eclipse-veditor-0.6.3-3.fc11.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493250 --- Comment #42 from Liang Suilong liangsuil...@gmail.com 2009-06-17 11:06:50 EDT --- New Package CVS Request === Package Name: perl-Goo-Canvas Short Description: Perl interface to the GooCanvas Owners: liangsuilong Branches: F-10 F-11 devel InitialCC: liangsuilong 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Flag||fedora-review+ --- Comment #6 from Alexander Kurtakov akurt...@redhat.com 2009-06-17 11:08:58 EDT --- Thanks. 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 506476] Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506476 Yanko Kaneti yan...@declera.com changed: What|Removed |Added CC||yan...@declera.com Flag||fedora-review+ --- Comment #1 from Yanko Kaneti yan...@declera.com 2009-06-17 11:20:12 EDT --- After going through the MUST list: rpmlint: rest.src: W: invalid-license LGPLv2.1 should be License: LGPLv2 Some cosmetic things: BuildRequires: gtk-doc is not really necessary , the tarball has that already built. You can use %configure --disable-static and spare one rm line later. The source tarball matches upstream, the package builds in rawhide i386 and x86_64 mock. After the license is is fixed this package looks fit to be APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505358] Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505358 Ian Weller i...@ianweller.org changed: What|Removed |Added CC||i...@ianweller.org --- Comment #3 from Ian Weller i...@ianweller.org 2009-06-17 11:21:16 EDT --- David, Patrick is correct. Since the only installed files in /usr/share/php are: /usr/share/php/hcard.profile.php /usr/share/php/hkit.class.php those are the only files owned by this package. The key thing is that it does not own /usr/share/php, which is good. -- Configure 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 501381] Review Request: 389-console - A Java based remote management console used for managing 389 Administration Server and 389 Directory Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501381 --- Comment #10 from Rich Megginson rmegg...@redhat.com 2009-06-17 11:23:29 EDT --- ok - the fedora package will be noarch - verified that it works correctly on rawhide x86_64 - correctly pulls and uses jss.x86_64 Spec URL: http://rmeggins.fedorapeople.org/pkgreview/389-console.spec SRPM URL: http://rmeggins.fedorapeople.org/pkgreview/389-console-1.1.3-2.src.rpm Description: A Java based remote management console used for managing 389 Administration Server and 389 Directory Server. This is a rename of fedora-idm-console to 389-console. Source URL: http://port389.org/sources/389-console-1.1.3.tar.bz2 md5sum 389-console-1.1.3.tar.bz2 e9152a2e523a2890767a91cb338e2974 389-console-1.1.3.tar.bz2 sha1sum 389-console-1.1.3.tar.bz2 73068de75b616d8d49ac570d23c007fec882bd89 389-console-1.1.3.tar.bz2 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506476] Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506476 Yanko Kaneti yan...@declera.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|yan...@declera.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 506517] New: Review Request: phpFlickr - PHP client library for Flickr
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: phpFlickr - PHP client library for Flickr https://bugzilla.redhat.com/show_bug.cgi?id=506517 Summary: Review Request: phpFlickr - PHP client library for Flickr Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: da...@gnsa.us QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://ke4qqq.fedorapeople.org/phpFlickr.spec SRPM URL: http://ke4qqq.fedorapeople.org/phpFlickr-2.3.0.1-1.fc11.src.rpm Description: PHP client for the Flickr web service. It has functions that return the responses from Flickr's API in a meaningful way for PHP developers. It will also contain functions that aggregate data from multiple methods. Compatible with PHP4 and PHP5. -- Configure 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 506476] Package Review: rest - A library for access to RESTful web services
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506476 --- Comment #2 from Peter Robinson pbrobin...@gmail.com 2009-06-17 11:51:53 EDT --- I'll update the above. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505358] Review Request: php-hkit - Simple PHP5 API for extracting common microformats from a page
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=505358
David Nalley da...@gnsa.us changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #4 from David Nalley da...@gnsa.us 2009-06-17 12:09:46 EDT ---
OK MUST: rpmlint must be run on every package. The output should be posted in
the review.
[ke4...@localhost SPECS]$ rpmlint ./php-hkit.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@localhost SPECS]$ rpmlint ../SRPMS/php-hkit-0.5-1.fc11.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@localhost SPECS]$ rpmlint ../RPMS/noarch/php-hkit-0.5-1.fc11.noarch.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
OK MUST: The package must be named according to the Package Naming Guidelines
.
OK MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption. .
OK MUST: The package must meet the Packaging Guidelines .
OK MUST: The package must be licensed with a Fedora approved license and meet
the Licensing Guidelines .
LGPLv2+
[] MUST: The License field in the package spec file must match the actual
license.
LGPLv2+ is cited in the class file, which maches what is in the spec
NA MUST: If (and only if) the source package includes the text of the
license(s) in its own file, then that file, containing the text of the
license(s) for the package must be included in %doc.
Not included.
OK MUST: The spec file must be written in American English.
OK MUST: The spec file for the package MUST be legible.
OK MUST: The sources used to build the package must match the upstream source,
as provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[ke4...@localhost SOURCES]$ md5sum hkit-v0.5.tgz*
37f3c192a94a86cd96770d53b3f22095 hkit-v0.5.tgz
37f3c192a94a86cd96770d53b3f22095 hkit-v0.5.tgz.1
OK MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
NA MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
NA MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
[] MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
OK MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory.
OK MUST: A Fedora package must not list a file more than once in the spec
file's %files listings.
OK MUST: Permissions on files must be set properly. Executables should be set
with executable permissions, for example. Every %files section must include a
%defattr(...) line.
OK MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
OK MUST: Each package must consistently use macros.
OK MUST: The package must contain code, or permissable content.
OK MUST: Large documentation files must go in a -doc subpackage. (The
definition of large is left up to the packager's best judgement, but is not
restricted to size. Large can refer to either size or quantity).
OK MUST: If a package includes something as %doc, it must not affect the
runtime of the application. To summarize: If it is in %doc, the program must
run properly if it is not present.
NA 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
[Bug 500506] Review Request: 389-adminutil - renamed from adminutil
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500506 --- Comment #4 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:19:23 EDT --- ping -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500713] Review Request: 389-ds-base - renamed from fedora-ds-base
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500713 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #11 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:17:01 EDT --- tagging fixed - built for f12 in koji -- Configure 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 501383] Review Request: 389-ds-console - 389 Directory Server Management Console
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501383 --- Comment #2 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:20:51 EDT --- ping -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500506] Review Request: 389-adminutil - renamed from adminutil
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500506 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Blocks||501393(389-dsgw) -- Configure 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 501393] Review Request: 389-dsgw - 389 Directory Server Gateway
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501393 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Depends on||500506(389-adminutil) -- Configure 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 501385] Review Request: 389-admin-console - 389 Admin Server Management Console
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501385 --- Comment #2 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:21:17 EDT --- ping -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 500746] Review Request: 389-admin - renamed from fedora-ds-admin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500746 --- Comment #9 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:20:05 EDT --- The tagging issue has been resolved. This this package request 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 500506] Review Request: 389-adminutil - renamed from adminutil
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500506 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Blocks||500746(389-admin) -- Configure 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 500746] Review Request: 389-admin - renamed from fedora-ds-admin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=500746 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Depends on||500506(389-adminutil) -- Configure 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 506339] Review Request: XZ Utils - LZMA Utils with newer file format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506339 --- Comment #4 from Ville Skyttä ville.sky...@iki.fi 2009-06-17 12:25:05 EDT --- I think obsoleting lzma-libs and lzma-devel is definitely premature, and providing them is wrong and should never happen (obsoletion _only_ without provides when the time is right to drop lzma). xz-libs does *not* provide the stuff in lzma-libs (liblzmadec) and xz-devel does not IIRC provide the headers and pkgconfig file that lzma-devel does, and we have packages depending on both in the repo, at least libarchive, never mind people's local builds against lzma-libs/-devel. https://fedoraproject.org/wiki/Packaging/NamingGuidelines#Renaming.2Freplacing_existing_packages Yes, beta should be in the release tag. https://fedoraproject.org/wiki/Packaging/NamingGuidelines#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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 Chitlesh GOORAH chitl...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #7 from Chitlesh GOORAH chitl...@gmail.com 2009-06-17 12:29:21 EDT --- New Package CVS Request === Package Name: eclipse-veditor Short Description: Eclipse-based Verilog/VHDL plugin Owners: chitlesh Branches: F-10 F-11 -- Configure 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 502554] Review Request: mingw32-boost - MinGW Windows Boost C++ 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=502554 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #14 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:34:42 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 506007] Review Request: perl-Sub-WrapPackages - Add wrappers around all the subroutines in packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506007 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:32:57 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 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-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=504709 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #12 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:31:45 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 506176] Review Request: hunspell-ko - Korean hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506176 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:33: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 506187] Review Request: perl-TheSchwartz - Reliable job queue
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506187 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:37: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 506231] Review Request: hunspell-smj - Lule Sami hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506231 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:34:13 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 498413] Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498413 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #17 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:36:54 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 506186] Review Request: perl-p5-Palm - Palm OS utility functions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506186 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:39:11 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 426750] Review Request: ghc-utf8-string - Support reading and writing UTF8 Strings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=426750 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #24 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:38:24 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 506006] Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506006 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:40:23 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Jason Tibbitts ti...@math.uh.edu 2009-06-17 12:41:19 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 461393] Review Request: congruity - Application t o program Logitech® Harmony® universal remote control s
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=461393 --- Comment #7 from Adam Williamson awill...@redhat.com 2009-06-17 12:48:47 EDT --- Whee, I just learned how to do MIME types. :) I'll submit a patch for libconcord which defines appropriate files to be of type application/x-libconcord , and an updated congruity package with a .desktop file that specifies it can handle this MIME type. I believe that would be correct. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure 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 506450] Package Review: twitter-glib - A library wrapping the Twitter RESTful 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=506450 Yanko Kaneti yan...@declera.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||yan...@declera.com AssignedTo|nob...@fedoraproject.org|yan...@declera.com Flag||fedora-review+ --- Comment #1 from Yanko Kaneti yan...@declera.com 2009-06-17 12:45:24 EDT --- MUST: twitter-glib.src: W: invalid-license LGPLv2.1+ Should be License: LGPLv2 Cosmetic: URL: http://live.gnome.org/TwitterGlib makes more sense I think. BuildRequires: gtk-doc is not really necessary , the tarball has that already built. The package has a testsuite that doesn't seem to do anything network related. You could add %check make check With the license change this package is APPROVED. I am not sure why this is a blocker for the rest 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 501393] Review Request: 389-dsgw - 389 Directory Server Gateway
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=501393 Rich Megginson rmegg...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #13 from Rich Megginson rmegg...@redhat.com 2009-06-17 12:55:35 EDT --- tagging issue fixed - built in koji for f12 -- Configure 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 500746] Review Request: 389-admin - renamed from fedora-ds-admin
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=500746
Jochen Schmitt joc...@herr-schmitt.de changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|joc...@herr-schmitt.de
Flag||fedora-review?
--- Comment #10 from Jochen Schmitt joc...@herr-schmitt.de 2009-06-17
13:14:16 EDT ---
Good:
+ Name of the SPEC file matches with package name.
+ Pakcage name fullfill naming guidelines
+ URL tag show on proper project homepage
+ Package contains valid license tag
+ License tag state GPLv2 as a valid OSS license
+ Package contains no subpackages
+ Provides/Obsoletes statement for renaming process are ok.
+ BuildRoot will be cleaned at the beginnung of %clean and %install
+ Could download upstream tar ball via spectool -g
+ Package source matches with upstream
(md5sum: cd3fd64fabc0265e2765101d032d7150)
+ Package contains proper BuildRoot definition
+ Package honour $RPM_OPT_FLAGS
+ Package support SMP-enabled build
+ Local build works fine
+ Rpmlint is silent on debuginfo rpm
+ Local install/uninstall works fine
+ Scratch build works fine
+ Files has proper files permission
+ %files stanza contains no duplicated entries
+ All package files are owned by this package
+ No package files are belong to a other package
+ %doc stanza is mall, no extra sub package is required
+ Package contains proper changelog
Bad:
- I have found source files which state GPLv2+ or AL 2.0 as
license. Please clarify the stated license on the license tag
- Package only contains the LICENSE file, but the COPYING file,
which contains the verbatin text of the GPLv2 is not included
- Usage of the %{_initrddir} macro is obsoleted
- Why do you set the owner/permission of some files explicitly in the
%post scriptlet?
- warning from rpmlint about source rpm.
rpmlint 389-admin-1.1.7-5.fc11.src.rpm
389-admin.src: W: strange-permission 389-admin-git.sh 0775
- Warnings from rpmlint about the binary rpm
$ rpmlint 389-admin-1.1.7-5.fc11.x86_64.rpm
389-admin.x86_64: W: dangerous-command-in-%post chmod
389-admin.x86_64: W: incoherent-init-script-name dirsrv-admin
--
Configure 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2009-06-17 13:40:25 EDT --- eclipse-veditor-0.6.3-3.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/eclipse-veditor-0.6.3-3.fc11 -- Configure 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 505017] Review Request: eclipse-veditor - Eclipse-based Verilog/VHDL plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505017 --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2009-06-17 13:40:20 EDT --- eclipse-veditor-0.6.3-3.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/eclipse-veditor-0.6.3-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 226210] Merge Review: opal
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226210 --- Comment #22 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-17 13:47:02 EDT --- It has been a standard procedure to remove the prebuilt binaries (whether they are used or not). I saw and applied this on so many different packages that I forgot that there is no specific guideline for it except this https://fedoraproject.org/wiki/Packaging:Guidelines#No_inclusion_of_pre-built_binaries_or_libraries Since this guideline does not cover your case, I cannot claim that it is a blocker for the review. But I'd be happy if you reconsider. At the end of the day, it is your responsibility that the prebuilt binaries do not end up in the RPM's. Removing them in %prep may only make your life easier. Working on a clean source tree is a good thing, at least IMHO. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 461393] Review Request: congruity - Application t o program Logitech® Harmony® universal remote control s
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=461393 --- Comment #8 from Adam Williamson awill...@redhat.com 2009-06-17 13:51:45 EDT --- I've submitted the patch to libconcord: https://bugzilla.redhat.com/show_bug.cgi?id=506536 once that gets applied (the maintainer's very quick to respond), I'll put a new congruity package in this bug, with the MIME type added. -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure 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 273701] Review Request: gnome-main-menu - Gnome Main Menu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=273701 --- Comment #76 from Ken Crandall ken.crand...@gmail.com 2009-06-17 13:53:41 EDT --- 0.9.12 does not build on F11: main-menu.c: In function 'main_menu_applet_init': main-menu.c:67: warning: implicit declaration of function 'gnome_program_init' main-menu.c:67: error: 'LIBGNOMEUI_MODULE' undeclared (first use in this function) main-menu.c:67: error: (Each undeclared identifier is reported only once main-menu.c:67: error: for each function it appears in.) make[2]: *** [main-menu.o] Error 1 make[2]: *** Waiting for unfinished jobs main-menu-ui.c: In function 'item_to_recent_doc_tile': main-menu-ui.c:1087: warning: unused variable 'node' main-menu-ui.c:1085: warning: unused variable 'is_local' main-menu-ui.c:1084: warning: unused variable 'is_nfs' main-menu-ui.c:1083: warning: unused variable 'nfs_id' main-menu-ui.c:1082: warning: unused variable 'file' main-menu-ui.c:1081: warning: unused variable 'volume' main-menu-ui.c:1080: warning: unused variable 'mount' main-menu-ui.c: In function 'item_to_system_tile': main-menu-ui.c:1132: warning: unused variable 'basename' main-menu-ui.c: In function 'app_is_in_blacklist': main-menu-ui.c:1248: warning: implicit declaration of function 'geteuid' make[2]: Leaving directory `/home/ken/redhat/BUILD/gnome-main-menu-0.9.12/main-menu/src' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `/home/ken/redhat/BUILD/gnome-main-menu-0.9.12' make: *** [all] Error 2 error: Bad exit status from /home/ken/redhat/TEMP/rpm-tmp.MVyvPe (%build) RPM build errors: Bad exit status from /home/ken/redhat/TEMP/rpm-tmp.MVyvPe (%build) Working on a patch... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 273701] Review Request: gnome-main-menu - Gnome Main Menu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=273701 --- Comment #77 from Hezekiah M. Carty h...@truegeek.net 2009-06-17 14:01:56 EDT --- The Debian patches may be a good places to start: http://patch-tracking.debian.net/package/gnome-main-menu/0.9.12-3 -- Configure 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 273701] Review Request: gnome-main-menu - Gnome Main Menu
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=273701 --- Comment #78 from Ken Crandall ken.crand...@gmail.com 2009-06-17 14:08:09 EDT --- Created an attachment (id=348312) -- (https://bugzilla.redhat.com/attachment.cgi?id=348312) Patch to fix build on Fedora 11 This patch adds libgnomeui include files (which were missing) and causing the build to fail on Fedora 11. It now builds and works for me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487964] Pitivi
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487964 Jonathan Earl Brassow jbras...@redhat.com changed: What|Removed |Added CC||fedora-package-rev...@redha ||t.com, nott...@redhat.com Component|cman-kernel |Package Review AssignedTo|jbras...@redhat.com |nob...@fedoraproject.org QAContact||extras...@fedoraproject.org --- Comment #2 from Jonathan Earl Brassow jbras...@redhat.com 2009-06-17 14:08:33 EDT --- I'm not sure what this is, but I know it isn't a cman-kernel bug. Is package review the correct component? -- Configure 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 505867] Review Request: qelectrotech - Design electric diagrams
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505867 Xavier Guerrin xavier.guer...@gmail.com changed: What|Removed |Added CC||xavier.guer...@gmail.com --- Comment #3 from Xavier Guerrin xavier.guer...@gmail.com 2009-06-17 14:30:57 EDT --- Hello, At least for the moment, the QElectroTech project team prefers to keep the words diagram and element (especially this last one) in the application. You are however free to use the wording of your choice for the RPM package. Cheers, Xavier QElectroTech project developer -- Configure 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 498413] Review Request: vfrnav - Flight Planning software for flights under Visual Flight Rules (VFR)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498413 --- Comment #18 from Fedora Update System upda...@fedoraproject.org 2009-06-17 14:37:32 EDT --- vfrnav-0.3-8.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/vfrnav-0.3-8.fc11 -- Configure 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 506339] Review Request: XZ Utils - LZMA Utils with newer file format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506339 --- Comment #5 from Jindrich Novy jn...@redhat.com 2009-06-17 14:39:51 EDT --- Ok, there is another iteration of packages here: http://people.redhat.com/jnovy/files/xz/ -- Configure 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 506567] New: Review Request: acl2 - Automated reasoning system based on Common Lisp
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: acl2 - Automated reasoning system based on Common Lisp https://bugzilla.redhat.com/show_bug.cgi?id=506567 Summary: Review Request: acl2 - Automated reasoning system based on Common Lisp Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: amd...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: https://www.openproofs.org/packages/acl2/acl2.spec SRPM URL: https://www.openproofs.org/packages/acl2/acl2-3.5-1.fc10.src.rpm Description: ACL2 is an automated reasoning system developed at Computational Logic, Inc., and the University of Texas at Austin. It is valuable if you want to prove properties about a system that can be easily modeled as recursive functions in a Lisp-based language. ACL2 stands for A Computational Logic for Applicative Common Lisp, and is simultaneously a computational logic, formal specification language, programming system, and theorem prover. Its syntax is based on Common Lisp, and many ACL2 specifications are executable (depending on how ACL2 is used). This package is largely complete in that it allows the user to use the vast majority of the functionality of ACL2, the main thing that isn't quite right (that I've been working with upstream to fix) is that some of the (supplementary - program works without them) elisp files don't seem to compile and/or give warnings. I have tested a built version out with some of my own basic usage and have encountered no problems (eg: books load properly despite having moved things around - to comply with packaging policy - compared to their original locations in a usual install). rpmlint output with my comments: acl2.i386: E: devel-dependency acl2-books-source acl2-books-source is not really a devel repository in that it contains the text of the theorem libraries, not of the LISP that runs ACL2 itself. It is critical to using the theorem prover to have this text available. My naming convention matches the Debian package. acl2.i386: E: no-binary acl2.i386: W: only-non-binary-in-usr-lib I believe this is wrong - saved_acl2.core is a binary core file (platform dependent I believe) used by LISP. acl2-books.i386: W: only-non-binary-in-usr-lib I believe the .fasl files are also binary and platform dependent. acl2-books.i386: W: no-documentation acl2-books-certs.noarch: W: no-documentation acl2-books-source.noarch: W: no-documentation acl2-emacs-el.noarch: W: no-documentation acl2-xemacs-el.noarch: W: no-documentation True, but not important IMO (acl2-doc has documentation). acl2-doc.noarch: E: non-standard-dir-perm /usr/share/doc/acl2-3.5/HTML 0775 A standard directory should have permission set to 0755. If you get this message, it means that you have wrong directory permissions in some dirs included in your package. This actually appears to be wrong, though I thought this was actually corrected... (rebuilding now) This package has built in Koji for F11: x86:http://koji.fedoraproject.org/koji/taskinfo?taskID=1406611 x86_64: http://koji.fedoraproject.org/koji/taskinfo?taskID=1406769 and I am rebuilding it now for F12 (builds unfortunately take a long time). Issues that I see: 1) Elisp compilation as above (the compiled elisp packages currently don't work, but downloading the elisp source packages still allows one to use the supplementary LISP). 2) This package includes a LISP core. This core is specific to a given version and implementation of LISP (eg: if built with F10, it would be SBCL v1.0.22 as of now). Is there a good way of dealing with this? (Should I have Fedora version-dependent conditionals that BuildRequire and Require certain versions of LISP?) -- Configure 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Status|NEW |ASSIGNED --- Comment #8 from Rex Dieter rdie...@math.unl.edu 2009-06-17 15:13:51 EDT --- adding FE_NEEDSPONSOR blocker. I'll look things over. If kosher, I can sponsor here. -- Configure 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 503297] Review Request: panoglview - Immersive viewer for spherical panoramas
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=503297
--- Comment #2 from Bruno Postle br...@postle.net 2009-06-17 15:23:59 EDT ---
Thanks I've fixed %defattr and added %{?_smp_mflags}, I've left --vendor= as
this is required for it to build on el4 and el5:
http://bpostle.fedorapeople.org/reviews/panoglview/panoglview.spec
http://bpostle.fedorapeople.org/reviews/panoglview/panoglview-0.2.2-4.fc10.src.rpm
I forgot to add that the way to test this package is to download some typical
equirectangular panoramas and try to view them:
http://www.flickr.com/search/?q=equirectangularl=commderivss=2ct=6w=alladv=1
--
Configure 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 493335] Review Request: perl-RTx-Calendar - Calendar for RT due tasks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493335 --- Comment #6 from Jon Baker jba...@wgm.us 2009-06-17 15:29:27 EDT --- I just successfully installed this on my FC11 RT3 via the download and instructions - it was very easy and intalled without any issue. I'd submit it but I have no idea how ... -- Configure 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 493335] Review Request: perl-RTx-Calendar - Calendar for RT due tasks
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493335 --- Comment #7 from Jon Baker jba...@wgm.us 2009-06-17 15:33:08 EDT --- Never mind, just noticed the comments about /usr/lib and checked, sure enough, that is where it was 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 --- Comment #9 from Rex Dieter rdie...@math.unl.edu 2009-06-17 15:34:44 EDT --- A few minor comments, 1. MUST: remove the ldconfig scriplets. ldconfig is needed only for shlibs, these aren't. 2. SHOULD: Source should include snap or svntime_stamp in the name, to match Release tag. For examples, see https://fedoraproject.org/wiki/Packaging/NamingGuidelines#Snapshot_packages 2a. poke/prod upstream to do a real release. :) 3. Kinda-sorta-should: make a parent virtual package that Requires all subpkgs, so one can do a simple: yum install bespin to get it all at once. (but, this is a matter of taste, up to you whether you want to do it or not). Heck, taking this to an extreme, if you also want to simplify matters further, including everything in a single pkg would be ok with me too. I don't consider any of these issues blockers, and can very well be fixed prior to building (before or after import). Being your first package, I'll add myself as comaintainer, and continue to advise as needed. -- Configure 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 --- Comment #10 from Rex Dieter rdie...@math.unl.edu 2009-06-17 15:37:18 EDT --- Last small suggestion, leave the pkg name and versioning details out of Summary, so use something like: Summary: a style for Qt/KDE s/style/theme/ to taste. -- Configure 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 Rex Dieter rdie...@math.unl.edu changed: What|Removed |Added Blocks|177841(FE-NEEDSPONSOR) | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506231] Review Request: hunspell-smj - Lule Sami hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506231 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504430] Review Request: healpy - A python wrapper of the healpix 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=504430 --- Comment #16 from Jussi Lehtola jussi.leht...@iki.fi 2009-06-17 15:41:03 EDT --- (In reply to comment #14) MUST: The package must be named according to the Package Naming Guidelines. NEEDSWORK - This is a python module, so the correct name is python-healpy. Sorry, I didn't know this is a hard rule since the python modules it depends on, numpy, scipy and pyfits don't follow it. However fixed: Done. *whoops* I though the guideline was that the name must begin with python- if it doesn't start with py, but actually it is If the upstream source has py (or Py) in its name, you can use that name for the package. So, for example, pygtk is acceptable. http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Addon_Packages_.28python_modules.29 So the name should be after all just healpy. Sorry for the mixup :) -- Configure 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 506176] Review Request: hunspell-ko - Korean hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506176 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 491767] Review Request: nss-ldapd - An nsswitch module which uses directory servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491767 --- Comment #25 from Nalin Dahyabhai na...@redhat.com 2009-06-17 15:56:30 EDT --- Created an attachment (id=348333) -- (https://bugzilla.redhat.com/attachment.cgi?id=348333) policy changes -- Configure 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 506578] New: Review Request: rtkit - Realtime Policy and Watchdog Daemon
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rtkit - Realtime Policy and Watchdog Daemon https://bugzilla.redhat.com/show_bug.cgi?id=506578 Summary: Review Request: rtkit - Realtime Policy and Watchdog Daemon Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lpoet...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Target Release: --- Spec URL: http://0pointer.de/public/rtkit.spec SRPM URL: http://0pointer.de/public/rtkit-0.1-1.fc11.src.rpm Description: Realtime Policy and Watchdog Daemon This finally allows us us to securely enable RT scheduling by default for user software such as PulseAudio which really could benefit from it. This is a dependency of upcoming PA 0.9.16. This all depends on a kernel patch that is already in Ingo's tree on the way into mainline and Kyle wants to integrate into the Rawhide kernel very soon. On systems that lack this patch you'll just get a bit of spew in syslog, and clients won't be allowed to get RT scheduling, but nothing should fail beyond 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 491767] Review Request: nss-ldapd - An nsswitch module which uses directory servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=491767 --- Comment #26 from Nalin Dahyabhai na...@redhat.com 2009-06-17 15:57:56 EDT --- I think that with the policy changes (exercising Dan's polgen.py along the way), I think we'll have it sorted soon. Ok to get going on 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 504710] Review Request: rubygem-gettext_rails - Localization support for Ruby on Rails(=2.3) by Ruby-GetText-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=504710 Bug 504710 depends on bug 504709, which changed state. Bug 504709 Summary: Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package https://bugzilla.redhat.com/show_bug.cgi?id=504709 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-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=504709 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-17 16:02:38 EDT --- Rebuilt and push request submitted, closing. Thank you for the review and CVS procedure! -- Configure 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 506581] New: Review Request: xscope - X Window Protocol Viewer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: xscope - X Window Protocol Viewer https://bugzilla.redhat.com/show_bug.cgi?id=506581 Summary: Review Request: xscope - X Window Protocol Viewer Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: yan...@declera.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.declera.com/~yaneti/xscope/xscope.spec SRPM URL: http://www.declera.com/~yaneti/xscope/xscope-1.1-1.gitfccbbd6.fc12.src.rpm Description: xscope sits in-between an X11 client and an X11 server and prints the contents of each request, reply, error, or event that is communicated between them. This information can be useful in debugging and performance tuning of X11 servers and clients. -- Configure 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 --- Comment #11 from Wesley S. Hearn gr33nn1...@gr33nn1nj4.com 2009-06-17 16:18:20 EDT --- New Package CVS Request === Package Name: bespin Short Description: A style for Qt/KDE4 Owners: jknife rdieter Branches: F-10 F-11 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 506185] Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506185 Wesley S. Hearn gr33nn1...@gr33nn1nj4.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 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496635 --- Comment #49 from Mauricio Henriquez buhochil...@gmail.com 2009-06-17 16:19:43 EDT --- (In reply to comment #48) Excellent. Mauricio, please go to the account system and apply for the packager group: https://admin.fedoraproject.org/accounts/ mmm, kind of lost here, I have a fedora account but I supose to join a group?, is that is the case, what is the mono group name? The group is called 'packager' ok buhochileno has applied to packager! , sorry for the late...I'm really bussy right now, so hope that there is not to much step on front!!! Note that once I've sponsored you, you'll be on step #4 here: http://fedoraproject.org/wiki/Package_Review_Process don't know what to do with that :-S ...thanks Toshio... Look at this to http://fedoraproject.org/wiki/PackageMaintainers/Join#Get_Sponsored looking... Paul Mauricio -- Configure 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 500746] Review Request: 389-admin - renamed from fedora-ds-admin
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=500746
--- Comment #11 from Rich Megginson rmegg...@redhat.com 2009-06-17 16:52:14
EDT ---
(In reply to comment #10)
Bad:
- I have found source files which state GPLv2+ or AL 2.0 as
license. Please clarify the stated license on the license tag
- Package only contains the LICENSE file, but the COPYING file,
which contains the verbatin text of the GPLv2 is not included
- Usage of the %{_initrddir} macro is obsoleted
- all of the files in the adminserver (except the Apache modules) should have
been gplv2 - this has been fixed, including the LICENSE file
- there are two Apache modules included with the admin server that are licensed
under the apl 2.0 - mod_admserv and mod_restartd - these must use the apl
because they use apl code - how should I denote this?
- fixed _initrddir
- Why do you set the owner/permission of some files explicitly in the
%post scriptlet?
There is a bug or perhaps it is a feature of rpm that it does not preserve
the ownership/permissions of files/directories marked config noreplace. These
are files and directories which users are accustomed to changing. If we do not
explicitly save and preserve the owner/permission in pre/post, upgrade will
break existing installations. I know it is somewhat of a hack, but this is the
only way I could get it to work. Suggestions are welcome.
- warning from rpmlint about source rpm.
rpmlint 389-admin-1.1.7-5.fc11.src.rpm
389-admin.src: W: strange-permission 389-admin-git.sh 0775
- fixed
- Warnings from rpmlint about the binary rpm
$ rpmlint 389-admin-1.1.7-5.fc11.x86_64.rpm
389-admin.x86_64: W: dangerous-command-in-%post chmod
389-admin.x86_64: W: incoherent-init-script-name dirsrv-admin
- the init script and service name are dirsrv-admin - this was a conscious
decision not to tie it to the package name
new SRPM: http://rmeggins.fedorapeople.org/pkgreview/389-admin-1.1.7-5.src.rpm
new Spec: http://rmeggins.fedorapeople.org/pkgreview/389-admin.spec
new Source URL: http://port389.org/sources/389-admin-1.1.7.tar.bz2
md5sum 389-admin-1.1.7.tar.bz2
1844088cbe44bc9eda371ce606a38405 389-admin-1.1.7.tar.bz2
sha1sum 389-admin-1.1.7.tar.bz2
e6f9beea076220f485aa93c990935dd0134a18da 389-admin-1.1.7.tar.bz2
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506056] Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506056 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||a.bad...@gmail.com AssignedTo|nob...@fedoraproject.org|a.bad...@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 505138] Review Request: trac-peerreview-plugin - Framework for realtime code review within Trac
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505138 Chitlesh GOORAH chitl...@gmail.com changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #5 from Chitlesh GOORAH chitl...@gmail.com 2009-06-17 17:05:42 EDT --- Package Change Request == Package Name: trac-peerreview-plugin New Branches: EL-5 Owners: chitlesh -- Configure 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 489014] Review Request: gnome-do-plugins - Plugins for Gnome Do
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489014 Harry Dance harryrda...@googlemail.com changed: What|Removed |Added CC||harryrda...@googlemail.com --- Comment #23 from Harry Dance harryrda...@googlemail.com 2009-06-17 17:48:34 EDT --- Hi, in the meantime, please could someone explain the best way to install these plugins? -- Configure 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
