[Bug 225979] Merge Review: lam
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=225979 Michael Schwendt bugs.mich...@gmx.net changed: What|Removed |Added CC||bugs.mich...@gmx.net Depends on||473593 -- 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 483450] Review Request: ginac - C++ library for symbolic calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483450 Chitlesh GOORAH cgoo...@yahoo.com.au changed: What|Removed |Added CC||cgoo...@yahoo.com.au --- Comment #1 from Chitlesh GOORAH cgoo...@yahoo.com.au 2009-02-01 08:49:52 EDT --- Isn't already on fedora repositories ? -- Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483187
--- Comment #2 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 08:52:02
EDT ---
So shall we call this package kde-plasma-weatherforecast instead?
It's under extragear/base/plasma/applets/weather in KDE SVN, that's where the
name weather comes from.
* please make use of the %{name} macro.
What for? It's not like the name is going to change frequently. That's just
useless use of macros which doesn't help anyone.
Version:0
I thought the recommended version to use for checkouts without a version was
0.0, maybe it changed or maybe I just remembered wrong. (I see they use just 0
in Packaging:NamingGuidelines.) But if I change it from 0.0 to 0 now, this will
break the upgrade path for those who already installed the package from
kde-redhat testing. 0.0 will upgrade just fine to any version number they'll
actually use.
Release:0.2.20090130svn%{?dist}
OK, I can put the %{?dist} at the end, not sure why I put it before, probably
just a typo. (Not that the guidelines clearly say that 0.2%{?dist}.20090130svn
is invalid, but I agree that 0.2.20090130svn%{?dist} makes more sense.)
* Isn't plasma-devel provided by kdelibs-devel which is one of the
dependencies
of kdebase-workspace-devel? So, that BR seems redundant.
What if we make a separate plasma-devel one day?
But those redundant BRs have also been removed from other packages so I can
remove it from this one too.
* Fedora-specific flag -fexceptions is overriden by -fno-exceptions but I
think
this is a problem with the compilation of kde itself. Can this be fixed?
It's a KDE feature. I can bring this up at the next KDE SIG meeting. But does
it really matter for this package? It makes sense for a library
* Do we really need call ldconfig on post{,un} ?
Good question. I think we don't, as there's no public shared library, only a
plugin in %{_kde4_libdir}/kde4. I'll remove it.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 --- Comment #3 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 08:54:39 EDT --- Actually, I'll replace Version: 0.0 with a real version number, I just asked the upstream author what to use. I also asked him whether he prefers kde-plasma-weather or kde-plasma-weatherforecast. -- Configure 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 483438] New: Review Request: libindi - Instrument Neutral Distributed Interface
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libindi - Instrument Neutral Distributed Interface https://bugzilla.redhat.com/show_bug.cgi?id=483438 Summary: Review Request: libindi - Instrument Neutral Distributed Interface Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sergio.pa...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sergiopr.fedorapeople.org/libindi.spec SRPM URL: http://sergiopr.fedorapeople.org/libindi-0.6-1.src.rpm Description: INDI is a distributed control protocol designed to operate astronomical instrumentation. INDI is small, flexible, easy to parse, and scalable. It supports common DCS functions such as remote control, data acquisition, monitoring, and a lot more. -- Configure 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 483450] Review Request: ginac - C++ library for symbolic calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483450 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG --- Comment #2 from Fabian Affolter fab...@bernewireless.net 2009-02-01 09:11:22 EDT --- It is... https://admin.fedoraproject.org/pkgdb/packages/name/ginac -- Configure 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 483453] New: Review Request: perl-MooseX-Method - Method declaration with type checking
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-MooseX-Method - Method declaration with type checking https://bugzilla.redhat.com/show_bug.cgi?id=483453 Summary: Review Request: perl-MooseX-Method - Method declaration with type checking Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: allis...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Method/perl-MooseX-Method.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Method/perl-MooseX-Method-0.43-1.fc10.src.rpm Description: Method declaration with type checking. -- Configure 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 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite 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=470155
--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-02-01
09:35:06 EDT ---
Created an attachment (id=330550)
-- (https://bugzilla.redhat.com/attachment.cgi?id=330550)
python 2.6 patch (proposed)
For 1.0-4
* Dependency
---
$ grep 'import ' xword
import pygtk
import gtk
import gtk.gdk
import gobject
import gnomeprint
import gnomeprint.ui
import pango
import sys
import time
import os, os.path
import md5
import pickle
import ConfigParser
---
- It seems Requires: pygtk2 gnome-python2-gnomeprint is
sufficient.
* Scriptlets
- For desktop-file-install, now --vendor=fedora is not needed
(guidelines changed):
https://fedoraproject.org/wiki/Packaging/Guidelines#desktop-file-install_usage
- Requires: shared-mime-info is not needed:
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#mimeinfo
* Some misc issue
---
%prep
mkdir mime
cp -p %SOURCE2 mime
---
- What are these lines for?
---
gzip $RPM_BUILD_ROOT%{_mandir}/man1/%{name}.1
---
- Not needed. rpmbuild itself gzip's this automatically.
* Macros in %changelog
- In changelog, when you write macros please use %% instead
of % to prevent macros from being expanded.
* Python warning
- By the way now rawhide uses python 2.6 and xword
causes some warnings:
./xword:50: DeprecationWarning: the md5 module is deprecated; use hashlib
instead
import md5
Would you check if the attached patch works for you?
Then I will wait for your another review request or
your pre-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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 --- Comment #4 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 08:59:17 EDT --- Oh, I didn't finish my sentence: I was saying: it makes sense for a library to require exception-safety (and thus -fexceptions) because users of the library may want to use exceptions, but for an application or a plasmoid, it's pretty much irrelevant. And in any case it's not an issue with this package, but with the general KDE default flags. -- Configure 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 472229] Review Request: PyQwt - Python bindings for Qwt
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=472229 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #12 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-02-01 08:00:06 EDT --- Closing. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483450] New: Review Request: ginac - C++ library for symbolic calculations
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: ginac - C++ library for symbolic calculations https://bugzilla.redhat.com/show_bug.cgi?id=483450 Summary: Review Request: ginac - C++ library for symbolic calculations Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/ginac.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/ginac-1.4.4-1.fc10.src.rpm Project URL: http://www.ginac.de/ Description: GiNaC is designed to allow the creation of integrated systems that embed symbolic manipulations together with more established areas of computer science (like computation- intense numeric applications, graphical inter- faces, etc.) under one roof. GiNaC is an iterated and recursive acronym for GiNaC is Not a CAS, where CAS stands for Computer Algebra System. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097245 rpmlint output: [...@laptop24 i386]$ rpmlint ginac* ginac.i386: W: shared-lib-calls-exit /usr/lib/libginac-1.4.so.0.0.4 e...@glibc_2.0 4 packages and 0 specfiles checked; 0 errors, 1 warnings. [...@laptop24 SRPMS]$ rpmlint ginac-1.4.4-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. I reported upstream about the rpmlint issue but so far I have no answer. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483434] Review Request: argtable2 - A library for parsing GNU style command line arguments
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483434 Jess Portnoy kerne...@gmail.com changed: What|Removed |Added URL||http://argtable.sourceforge ||.net/ -- Configure 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 483459] New: Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-MooseX-Iterator - Iterate over collections https://bugzilla.redhat.com/show_bug.cgi?id=483459 Summary: Review Request: perl-MooseX-Iterator - Iterate over collections Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: allis...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator-0.09-1.fc10.src.rpm Description: This is an attempt to add smalltalk-like streams to Moose. It currently works with ArrayRefs and HashRefs. -- Configure 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 483434] New: Review Request: argtable2 - A library for parsing GNU style command line arguments
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: argtable2 - A library for parsing GNU style command line arguments https://bugzilla.redhat.com/show_bug.cgi?id=483434 Summary: Review Request: argtable2 - A library for parsing GNU style command line arguments Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: kerne...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://downloads.sourceforge.net/argtable/argtable2.spec SRPM URL: http://downloads.sourceforge.net/argtable/argtable2-10-1.src.rpm Description: Argtable is an ANSI C library for parsing GNU style command line arguments. It enables a program's command line syntax to be defined in the source code as an array of argtable structs. The command line is then parsed according to that specification and the resulting values are returned in those same structs where they are accessible to the main program. Both tagged (-v, --verbose, --foo=bar) and untagged arguments are supported, as are multiple instances of each argument. Syntax error handling is automatic and the library also provides the means for displaying the command line syntax directly from the array of argument specifications. Argtable can function as a getopt_long replacement, without the user of the program noticing the difference. Unlike getopt_long, argtable is cross platform and works on Windows and Mac as well as Posix systems. -- Configure 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 459948] Review Request: libhildon - Hildon Application Framework - shared libraries
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=459948
Peter Robinson pbrobin...@gmail.com changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #4 from Peter Robinson pbrobin...@gmail.com 2009-02-01 07:11:46
EDT ---
Looks fine but I believe the -devel package also needs to have a Requires:
gtk-doc. Other than that it all looks OK.
APPROVED.
+ rpmlint output
rpmlint -i libhildon-2.0.6-1.fc11.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
+ license matches the actual package license
+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
edad45e30d8388411edf14116dbac2f3 libhildon_2.0.6-1.tar.gz
+ package successfully builds on at least one architecture
tested using koji scratch build
+ BuildRequires list all build dependencies
+ %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun+
does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ %defattr line
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
n/a static libraries should be in -static
+ packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
+ libfoo.so must go in -devel
+ devel must require the fully versioned base
+ packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ %install must start with rm -rf %{buildroot} etc.
+ filenames must be valid UTF-8
Optional:
n/a if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock/koji
n/a the package should build into binary RPMs on all supported architectures
n/a review should test the package functions as described
+ scriptlets should be sane
+ pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin
--
Configure 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 483461] New: Review Request: perl-MooseX-Param - Simple role to provide a standard param method
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-MooseX-Param - Simple role to provide a standard param method https://bugzilla.redhat.com/show_bug.cgi?id=483461 Summary: Review Request: perl-MooseX-Param - Simple role to provide a standard param method Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: allis...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Param/perl-MooseX-Param.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Param/perl-MooseX-Param-0.02-1.fc10.src.rpm Description: This is a very simple Moose role which provides a CGI like param method. -- Configure 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 481527] Review Request: bucardo - asynchronous PostgreSQL replication system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481527 --- Comment #3 from Itamar Reis Peixoto ita...@ispbrasil.com.br 2009-02-01 10:50:28 EDT --- ok, the only reason to strip tarball is to make it smaller, there are no forbidden contents in it. I have mailed upstream about shipping each package in your own tarball https://mail.endcrypt.com/pipermail/bucardo-general/2009-January/000190.html here are a updated version without stripping tarball http://ispbrasil.com.br/bucardo/bucardo.spec http://ispbrasil.com.br/bucardo/bucardo-3.0.9-2.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483438] Review Request: libindi - Instrument Neutral Distributed Interface
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483438 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Blocks||478539 Depends on||482807 -- Configure 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 483451] New: Review Request: k3guitune - Musical instrument tuner
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: k3guitune - Musical instrument tuner https://bugzilla.redhat.com/show_bug.cgi?id=483451 Summary: Review Request: k3guitune - Musical instrument tuner Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dti...@iinet.net.au QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://members.iinet.net.au/~timmsy/k3guitune/k3guitune.spec SRPM URL: http://members.iinet.net.au/~timmsy/k3guitune/k3guitune-1.01-1.fc10.src.rpm Description: 3Guitune is a guitar-and-other-instruments tuner. It takes a signal from the microphone, calculates its frequency, and displays it on a note scale graphic and an oscilloscope. It supports normal, Wien, and physical tuning. === rpmlint: k3guitune.i386: W: dangling-symlink /usr/share/doc/HTML/de/k3guitune/common /usr/share/doc/HTML/de/common k3guitune.i386: W: symlink-should-be-relative /usr/share/doc/HTML/de/k3guitune/common /usr/share/doc/HTML/de/common k3guitune.i386: W: dangling-symlink /usr/share/doc/HTML/nl/k3guitune/common /usr/share/doc/HTML/nl/common k3guitune.i386: W: symlink-should-be-relative /usr/share/doc/HTML/nl/k3guitune/common /usr/share/doc/HTML/nl/common k3guitune.i386: W: dangling-symlink /usr/share/doc/HTML/en/k3guitune/common /usr/share/doc/HTML/en/common k3guitune.i386: W: symlink-should-be-relative /usr/share/doc/HTML/en/k3guitune/common /usr/share/doc/HTML/en/common 3 packages and 0 specfiles checked; 0 errors, 6 warnings. This package is not quite ready: 1. Need to work out what is going on with the locales stuff, and the rpmlint warnings above: /usr/share/doc/HTML/de/k3guitune/common /usr/share/doc/HTML/de/common /usr/share/doc/HTML/de/k3guitune/common /usr/share/doc/HTML/de/common Any tips regarding locales in specs appreciated. 2. In the default audio driver=auto mode, no audio is detected changing this oss works, but also no audio detected. If you switch to alsa mode, the application segfaults: http://fedora.pastebin.com/m4b5602f5 3. The patch for .desktop icon field doesn't match the name of the icon. -- Configure 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 483474] New: Review Request: perl-Test-TempDir - Temporary files support for testing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-Test-TempDir - Temporary files support for testing https://bugzilla.redhat.com/show_bug.cgi?id=483474 Summary: Review Request: perl-Test-TempDir - Temporary files support for testing Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: allis...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://allisson.fedorapeople.org/packages/perl-Test-TempDir/perl-Test-TempDir.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-Test-TempDir/perl-Test-TempDir-0.04-1.fc10.src.rpm Description: Test::TempDir provides temporary directory creation with testing in mind. -- Configure 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 477144] Review Request: eclipse-cmakeed - CMake Editor plug-in for Eclipse
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477144 Alexander Kurtakov akurt...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Alexander Kurtakov akurt...@redhat.com 2009-02-01 06:13:43 EDT --- New Package CVS Request === Package Name: eclipse-cmakeed Short Description: CMake Editor plug-in for Eclipse Owners: akurtakov Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482807] Review Request: libfli - Library for FLI CCD Camera Filter Wheels
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482807 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Blocks||483438 -- Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added CC||shawn.st...@rogers.com Flag||needinfo?(shawn.st...@roger ||s.com) --- Comment #5 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 09:06:15 EDT --- Waiting for feedback from upstream: Kevin_Kofler spstarr_work: What's the official version number of the weather forecast plasmoid? Kevin_Kofler 1.0 prerelease (i.e. 1.0-0.1.2009svn)? Kevin_Kofler And should it be called kde-plasma-weather or kde-plasma-weatherforecast? -- Configure 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 453573] Review Request: cdemu-daemon - userspace daemon for use with cdemu suite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=453573 --- Comment #3 from Jiri Moskovcak jmosk...@redhat.com 2009-02-01 11:33:02 EDT --- Yes, but they want me to convert the kernel modules from dkms to akmod and I don't have time for that now, but I'd like to get cdemu to fedora asap. Jirka -- Configure 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 483016] Package Review: perl-NOCpulse-Debug - Perl debug output 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=483016 Jon Stanley jonstan...@gmail.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|jonstan...@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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483187
--- Comment #6 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-01
12:51:53 EDT ---
(In reply to comment #2)
* please make use of the %{name} macro.
What for? It's not like the name is going to change frequently. That's just
useless use of macros which doesn't help anyone.
I've been told in many reviews that %{name} should be used extensively, except
a couple places like the URL. Even now, since there's a possibility that the
package name may be changed, it might come handy.
(In reply to comment #3)
Actually, I'll replace Version: 0.0 with a real version number, I just asked
the upstream author what to use. I also asked him whether he prefers
kde-plasma-weather or kde-plasma-weatherforecast.
alright. let's wait.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483187
--- Comment #7 from Rex Dieter rdie...@math.unl.edu 2009-02-01 13:01:45 EDT
---
I've been told in many reviews that %{name} should be used extensively
Did this advice have basis in fedora's packaging guidelines? If so, where?
All I'm aware of, is that macros should be used consistently, ie, avoid using
%{name} in some places and hard-code it others.
--
Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483187
--- Comment #8 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-01
13:20:02 EDT ---
(In reply to comment #7)
I've been told in many reviews that %{name} should be used extensively
Did this advice have basis in fedora's packaging guidelines? If so, where?
I'm not 100% sure that it is in the guidelines (I never found it). But I have
been told this so many times by different reviewers, that I took it as an
unwritten convention, or maybe it is so deeply hidden inside the guidelines
that nobody remembers where it is... or it is just a legend.
If it is an unwritten convention, sure, you don't have to obey it. But it would
be nice if you do.
side note: Another thing that I've been told many times to use in a very
specific unique way and that I couldn't find in the guidelines is
%defattr(-,root,root,-)
All I'm aware of, is that macros should be used consistently, ie, avoid using
%{name} in some places and hard-code it others.
Well, strictly speaking, since %{name} has to be used in the BuildRoot, any
other usage will cause an inconsistency.
--
Configure 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 483016] Package Review: perl-NOCpulse-Debug - Perl debug output 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=483016
--- Comment #3 from Jon Stanley jonstan...@gmail.com 2009-02-01 13:21:43 EDT
---
Sorry for the delay, I'd intended to take care of this Friday night. But here
it is.
OK - Package meets naming and packaging guidelines
OK - Spec file matches base package name.
OK - Spec has consistant macro usage.
SEE LATER - Meets Packaging Guidelines.
OK - License
OK - License field in spec matches
Not included upstream - License file included in package
OK - Spec in American English
OK - Spec is legible.
OK - Sources match upstream md5sum:
$ md5sum ../../build/SOURCES/perl-NOCpulse-Debug-1.23.13.tar.gz
perl-NOCpulse-Debug-1.23.13.tar.gz
b3d2897e6c4265561ce728568d2bce3d
../../build/SOURCES/perl-NOCpulse-Debug-1.23.13.tar.gz
b3d2897e6c4265561ce728568d2bce3d perl-NOCpulse-Debug-1.23.13.tar.gz
N/A - Package needs ExcludeArch
OK - BuildRequires correct
N/A - Spec handles locales/find_lang
N/A - Package is relocatable and has a reason to be.
OK - Package has %defattr and permissions on files is good.
OK - Package has a correct %clean section.
OK - Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
OK - Package is code or permissible content.
N/A- Doc subpackage needed/used.
No %doc files - Packages %doc files don't affect runtime.
-- Please include the README from upstream here, and actually the full license
N/A - Headers/static libs in -devel subpackage.
N/A - Spec has needed ldconfig in post and postun
N/A - .pc files in -devel subpackage/requires pkgconfig
N/A - .so files in -devel subpackage.
N/A - -devel package Requires: %{name} = %{version}-%{release}
N/A - .la files are removed.
N/A - Package is a GUI app and has a .desktop file
OK - Package compiles and builds on at least one arch.
OK - Package has no duplicate files in %files.
FAIL - Package doesn't own any directories other packages own.
OK - Package owns all the directories it creates.
OK - No rpmlint output.
- final provides and requires are sane:
(include output of for i in *rpm; do echo $i; rpm -qp --provides $i; echo =;
rpm -qp --requires $i; echo; done
manually indented after checking each line. I also remove the rpmlib junk and
anything provided by glibc.)
perl-NOCpulse-Debug-1.23.13-1.fc11.noarch.rpm
config(perl-NOCpulse-Debug) = 1.23.13-1.fc11
perl(NOCpulse::Debug) = 1.23
perl(NOCpulse::Debug::Stream)
perl(NOCpulse::Debug::Stream::html)
perl(NOCpulse::Debug::Stream::html_comment)
perl(NOCpulse::Debug::Stream::literal)
perl(NOCpulse::Debug::Stream::stdout)
perl(NOCpulse::Debuggable)
perl(NOCpulse::DependencyGraph)
perl(NOCpulse::Log::LogManager)
perl(NOCpulse::Log::Logger)
perl(NOCpulse::Log::test::TestLogger)
perl-NOCpulse-Debug = 1.23.13-1.fc11
=
config(perl-NOCpulse-Debug) = 1.23.13-1.fc11
nocpulse-common
perl(:MODULE_COMPAT_5.10.0)
perl(Carp)
perl(Class::MethodMaker)
perl(Data::Dumper)
perl(IO::Handle)
perl(NOCpulse::Config)
perl(NOCpulse::Debug)
perl(NOCpulse::Log::LogManager)
perl(NOCpulse::Log::Logger)
SHOULD Items:
OK, tested on x86_64 - Should build in mock.
OK - Should build on all supported archs
Didn't test - Should function as described.
No scriptlets - Should have sane scriptlets.
OK - Should have subpackages require base package with fully versioned depend.
OK - Should have dist tag
OK - Should package latest version
N/A - check for outstanding bugs on package. (For core merge reviews)
Please remove ownership of directories %{perl_vendorlib}/NOCpulse and
${_sysconfdir}/nocpulse. These are both owned by nocpulse-common, which you
Require, and two packages can't own the same directories.
--
Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Shawn Starr shawn.st...@rogers.com changed: What|Removed |Added Flag|needinfo?(shawn.st...@roger | |s.com) | --- Comment #9 from Shawn Starr shawn.st...@rogers.com 2009-02-01 13:31:22 EDT --- Well, the plasmoid is Weather Forecast but we didn't change the .desktop file to reflect this. X-KDE-Library=plasma_applet_weather You could call the RPM kde-plasma-weatherforecast but the installed plasmoid is 'plasma_applet_weather' Version is '1.0' for KDE 4.2. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481564] Review Request: bind-to-tinydns - Convert DNS zone files in BIND format to tinydns 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=481564 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #19 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:36:36 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 477818] Review Request: gsql - Integrated database development tool for GNOME
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477818 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:35:21 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 483363] Review Request: yanone-tagesschrift-fonts - Serif decorative latin TTF font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483363 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:37:52 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 482863] Review Request: perl-MooseX-Daemonize - Role for daemonizing your Moose based 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=482863 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #7 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:40:33 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 483400] Review Request: vollkorn-fonts - A serif latin OTF font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483400 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:39: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 482880] Review Request: perl-Sane - Access SANE-compatible scanners with perl
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482880 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:41:51 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 442233] Review Request: oprofileui - user interface for analysing oprofile data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=442233 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #15 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:43:49 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 --- Comment #10 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-01 13:46:11 EDT --- I'm not sure if this is related to packaging. The image on the left-hand side of the applet does not look right. No matter how the weather is, it shows me a large black Ø contained in a larger white rectangle. Is this what this plasmoid supposed to show? -- Configure 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 483074] Review Request: perl-Exception-Base - Lightweight exceptions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483074 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #4 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:49:02 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 483079] Review Request: perl-constant-boolean - Define TRUE and FALSE constants
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483079 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #4 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:51:39 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481667] Review Request: xmlenc - XML output library for java.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481667 Fabian Deutsch fabian.deut...@gmx.de changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NOTABUG --- Comment #9 from Fabian Deutsch fabian.deut...@gmx.de 2009-02-01 13:49:00 EDT --- Included in cvs: http://cvs.fedoraproject.org/viewvc/rpms/xmlenc/ -- Configure 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 481022] Review Request: SPE it is a Python IDE editor released
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481022 --- Comment #2 from MERCIER bioinfornat...@gmail.com 2009-02-01 13:51:05 EDT --- $ koji build --scratch dist-f10 rpmbuild/SRPMS/spe-0.8.4.h-3.fc10.src.rpm Uploading srpm: rpmbuild/SRPMS/spe-0.8.4.h-3.fc10.src.rpm [] 100% 00:00:36 1.16 MiB 32.71 KiB/sec Created task: 1097523 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097523 Watching tasks (this may be safely interrupted)... 1097523 build (dist-f10, spe-0.8.4.h-3.fc10.src.rpm): free 1097523 build (dist-f10, spe-0.8.4.h-3.fc10.src.rpm): free - open (x86-5.fedora.phx.redhat.com) 1097524 buildArch (spe-0.8.4.h-3.fc10.src.rpm, noarch): free 1097524 buildArch (spe-0.8.4.h-3.fc10.src.rpm, noarch): free - open (x86-3.fedora.phx.redhat.com) 1097524 buildArch (spe-0.8.4.h-3.fc10.src.rpm, noarch): open (x86-3.fedora.phx.redhat.com) - closed 0 free 1 open 1 done 0 failed 1097523 build (dist-f10, spe-0.8.4.h-3.fc10.src.rpm): open (x86-5.fedora.phx.redhat.com) - closed 0 free 0 open 2 done 0 failed 1097523 build (dist-f10, spe-0.8.4.h-3.fc10.src.rpm) completed successfully -- Configure 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 481667] Review Request: xmlenc - XML output library for java.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481667 Fabian Deutsch fabian.deut...@gmx.de changed: What|Removed |Added Resolution|NOTABUG |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 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483258 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Kevin Fenzi ke...@tummy.com 2009-02-01 13:53: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 475097] Review Request: gimp-fourier-plugin - A fourier transformation plugin for GIMP.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475097 Fabian Deutsch fabian.deut...@gmx.de changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #14 from Fabian Deutsch fabian.deut...@gmx.de 2009-02-01 13:52:25 EDT --- Included in cvs: http://cvs.fedoraproject.org/viewvc/rpms/gimp-fourier-plugin/ -- Configure 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 475097] Review Request: gimp-fourier-plugin - A fourier transformation plugin for GIMP.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475097 Fabian Deutsch fabian.deut...@gmx.de changed: What|Removed |Added Resolution|RAWHIDE |NEXTRELEASE --- Comment #15 from Fabian Deutsch fabian.deut...@gmx.de 2009-02-01 13:56:14 EDT --- At last found the right way to close this bug. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483360] Review Request: pdfshuffler - PDF file merging, rearranging, and spliting
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483360 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #4 from Kevin Fenzi ke...@tummy.com 2009-02-01 14:01:40 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 483392] Review Request: whohas - Command line tool for query package lists
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483392 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #4 from Kevin Fenzi ke...@tummy.com 2009-02-01 14:00:00 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 481667] Review Request: xmlenc - XML output library for java.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481667 Fabian Deutsch fabian.deut...@gmx.de changed: What|Removed |Added Resolution|RAWHIDE |NEXTRELEASE --- Comment #10 from Fabian Deutsch fabian.deut...@gmx.de 2009-02-01 13:57:10 EDT --- At last found the right way to close this bug. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=475037 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #7 from Kevin Fenzi ke...@tummy.com 2009-02-01 14:04:32 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 --- Comment #11 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 14:05:22 EDT --- side note: Another thing that I've been told many times to use in a very specific unique way and that I couldn't find in the guidelines is %defattr(-,root,root,-) That one is because the files must be owned by root. Without this, they can be owned by some other user depending on how the RPM package is built. The image on the left-hand side of the applet does not look right. No matter how the weather is, it shows me a large black Ø contained in a larger white rectangle. This is because the BBC reports the current weather conditions as N/A for several locations. Apparently they don't get reports of weather conditions, only temperature and wind. So we get a huge N/A icon. :-( -- Configure 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 477144] Review Request: eclipse-cmakeed - CMake Editor plug-in for Eclipse
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=477144 Kevin Fenzi ke...@tummy.com changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #6 from Kevin Fenzi ke...@tummy.com 2009-02-01 14:02:56 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 483403] A collection of quotes in French for gdesklets
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483403 --- Comment #5 from MERCIER bioinfornat...@gmail.com 2009-02-01 14:04:12 EDT --- http://bioinformatiques.free.fr/Citation-1.3-5.fc10.noarch.rpm http://bioinformatiques.free.fr/Citation-1.3-5.fc10.src.rpm http://bioinformatiques.free.fr/Citation.spec _ $ rpmlint rpmbuild/SPECS/Citation.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint rpmbuild/SRPMS/Citation-1.3-5.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint rpmbuild/RPMS/noarch/Citation-1.3-5.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. _ # rpm -ivh Citation-1.3-5.fc10.noarch.rpm Préparation... ### [100%] 1:Citation ### [100%] # ls /usr/share/gdesklets/Displays/Citation/ bg citation.display citation.png citation.script ]# ls /usr/share/doc/Citation-1.3/ COPYING -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483187
--- Comment #12 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 14:12:58
EDT ---
* Sun Feb 01 2009 Kevin Kofler ke...@tigcc.ticalc.org 1.0-0.1.20090130svn
- Set Version to 1.0
- Fix Release (put alphatag before disttag)
- Remove redundant BR plasma-devel
- Don't call ldconfig, not needed
- Use %%{name} macro
Spec URL: http://repo.calcforge.org/f10/kde-plasma-weather.spec
New SRPM URL:
http://repo.calcforge.org/f10/kde-plasma-weather-1.0-0.1.20090130svn.fc9.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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com Flag||fedora-review+ --- Comment #13 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-01 14:25:17 EDT --- (In reply to comment #11) side note: Another thing that I've been told many times to use in a very specific unique way and that I couldn't find in the guidelines is %defattr(-,root,root,-) That one is because the files must be owned by root. Without this, they can be owned by some other user depending on how the RPM package is built. I was more considered about the -s. I don't know why %defattr(-,root,root) or %defattr(0644,root,root,0755) is bad (even for RPMs for which one doesn't need to assign exotic permissions) The image on the left-hand side of the applet does not look right. No matter how the weather is, it shows me a large black Ø contained in a larger white rectangle. This is because the BBC reports the current weather conditions as N/A for several locations. Apparently they don't get reports of weather conditions, only temperature and wind. So we get a huge N/A icon. :-( That's what I suspected. The weather in this s.hole where I reside is so cold that BBC doesn't know how to picture it. Anyways, the package is good to go now. (One last thing, just an idea to clear possible confusions in the future): you can add to the description that this plasmoid is planned to be included in KDE 4.3.) - This package (kde-plasma-weather) is APPROVED by oget - -- Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #14 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 14:46:37 EDT --- New Package CVS Request === Package Name: kde-plasma-weather Short Description: Plasma applet for weather forecasts Owners: kkofler spstarr Branches: InitialCC: tuxbrewr -- Configure 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 483026] Review Request: hatools - Improved shell scripting in High Availability environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483026 Jochen Schmitt joc...@herr-schmitt.de changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #4 from Jochen Schmitt joc...@herr-schmitt.de 2009-02-01 14:45:13 EDT --- Good: + License tag say GPLv2+ as an OSS license wich matches with the copyright notes in the source files. + Adding blank lines betweens the paragraphs. *** 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Flag|fedora-cvs? | --- Comment #15 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 14:48:47 EDT --- Oops, forgot to request branches. -- Configure 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 483187] Review Request: kde-plasma-weather - Plasma applet for weather forecasts
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483187 Kevin Kofler ke...@tigcc.ticalc.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #16 from Kevin Kofler ke...@tigcc.ticalc.org 2009-02-01 14:49:17 EDT --- New Package CVS Request === Package Name: kde-plasma-weather Short Description: Plasma applet for weather forecasts Owners: kkofler spstarr Branches: F-9 F-10 InitialCC: tuxbrewr -- Configure 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 483026] Review Request: hatools - Improved shell scripting in High Availability environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483026 Oliver Falk oli...@linux-kernel.at 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 483026] Review Request: hatools - Improved shell scripting in High Availability environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483026 --- Comment #5 from Oliver Falk oli...@linux-kernel.at 2009-02-01 15:23:28 EDT --- New Package CVS Request === Package Name: hatools Short Description: Improved shell scripting in High Availability environment Owners: oliver Branches: EPEL-4 EPEL-5 F-9 F-10 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 467409] Review Request: mingw32-atk - MinGW Windows Atk 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=467409 --- Comment #10 from Erik van Pienbroek erik-fed...@vanpienbroek.nl 2009-02-01 15:59:41 EDT --- The .src.rpm fails to build on my Rawhide system (local rpmbuild) : libtool: link: i686-pc-mingw32-gcc -shared .libs/libatk-1.0-0.dll.def .libs/atkaction.o .libs/atkcomponent.o .libs/atkdocument.o .libs/atkeditabletext.o .libs/atkgobjectaccessible.o .libs/atkhyperlink.o .libs/atkhyperlinkimpl.o .libs/atkhypertext.o .libs/atkimage.o .libs/atknoopobject.o .libs/atknoopobjectfactory.o .libs/atkobject.o .libs/atkobjectfactory.o .libs/atkregistry.o .libs/atkrelation.o .libs/atkrelationset.o .libs/atkselection.o .libs/atkstate.o .libs/atkstateset.o .libs/atkstreamablecontent.o .libs/atktable.o .libs/atktext.o .libs/atkutil.o .libs/atkmisc.o .libs/atkvalue.o .libs/atk-enum-types.o /usr/i686-pc-mingw32/sys-root/mingw/lib/libgobject-2.0.dll.a -L/usr/i686-pc-mingw32/sys-root/mingw/lib /usr/i686-pc-mingw32/sys-root/mingw/lib/libglib-2.0.dll.a -lws2_32 -lole32 /usr/i686-pc-mingw32/sys-root/mingw/lib/libintl.dll.a /usr/i686-pc-mingw32/sys-root/mingw/lib/libiconv.dll.a -mms-bitfields -Wl,atk-win32-res.o -o .libs/libatk-1.0-0.dll -Wl,--enable-auto-image-base -Xlinker --out-implib -Xlinker .libs/libatk-1.0.dll.a Creating library file: .libs/libatk-1.0.dll.a .libs/atkobject.o: In function `gettext_initialization': /home/erik/rpmbuild/BUILD/atk-1.25.2/atk/atkobject.c:403: undefined reference to `_g_getenv' collect2: ld returned 1 exit status make[3]: *** [libatk-1.0.la] Error 1 make[3]: Leaving directory `/home/erik/rpmbuild/BUILD/atk-1.25.2/atk' make[2]: *** [all] Error 2 $ rpm -qa | grep mingw32 mingw32-iconv-1.12-7.fc11.noarch mingw32-binutils-2.19-1.fc11.x86_64 mingw32-dlfcn-0-0.3.r11.fc11.noarch mingw32-gcc-c++-4.3.2-12.fc11.x86_64 mingw32-pixman-0.13.2-2.fc11.noarch mingw32-cairo-1.8.0-7.fc11.noarch mingw32-glib2-2.19.5-3.fc11.noarch mingw32-termcap-1.3.1-5.fc11.noarch mingw32-zlib-1.2.3-12.fc11.noarch mingw32-libpng-1.2.34-2.fc11.noarch mingw32-gettext-0.17-8.fc11.noarch mingw32-filesystem-45-1.fc11.noarch mingw32-w32api-3.13-1.fc11.noarch mingw32-libxml2-2.7.2-6.fc11.noarch mingw32-cpp-4.3.2-12.fc11.x86_64 mingw32-gcc-4.3.2-12.fc11.x86_64 mingw32-runtime-3.15.1-10.fc11.noarch -- Configure 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 483496] New: Review Request: python-instant - Python module for instant inlining of C and C++ code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-instant - Python module for instant inlining of C and C++ code https://bugzilla.redhat.com/show_bug.cgi?id=483496 Summary: Review Request: python-instant - Python module for instant inlining of C and C++ code Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-instant.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-instant-0.9.5-1.fc10.src.rpm Project URL: http://www.fenics.org/wiki/Instant Description: Instant is a Python module that allows for instant inlining of C and C++ code in Python. It is a small Python module built on top of SWIG and Distutils. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097698 rpmlint output: [...@laptop24 noarch]$ rpmlint python-instant-0.9.5-1.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. [...@laptop24 SRPMS]$ rpmlint python-instant-0.9.5-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 478767] Review Request: spring - Realtime strategy game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478767 Aurelien Bompard gau...@free.fr changed: What|Removed |Added Flag||needinfo?(ianwel...@gmail.c ||om) --- Comment #22 from Aurelien Bompard gau...@free.fr 2009-02-01 16:01:56 EDT --- Well, it was an easy one this time. Fixed, thanks. http://gauret.free.fr/fichiers/rpms/fedora/spring/spring-0.78.2.1-4.fc10.src.rpm Koji builds: F10: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097695 Rawhide: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097706 -- Configure 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 478767] Review Request: spring - Realtime strategy game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=478767 Ian Weller ianwel...@gmail.com changed: What|Removed |Added Flag|needinfo?(ianwel...@gmail.c | |om) | --- Comment #23 from Ian Weller ianwel...@gmail.com 2009-02-01 16:09:11 EDT --- This looks fine to me, I'm holding off on approval until the others are reviewed and I can test them all. -- Configure 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 #13 from Peter Robinson pbrobin...@gmail.com 2009-02-01 16:12:09 EDT --- We already removed iLBC support from Asterisk - can we re-add this support back? Sorry for the delay. Red Hat Legal got this one wrong (it happens to everyone sometimes). The iLBC codec needs to be removed from the opal tarball. Reblocking FE-Legal. I'm reported this upstream on the OPAL bug tracker. I've also had a direct email conversation with the ekiga maintainer and he agrees so is going to discuss this with the OPAL developers so hopefully will have an upstream fix for the ilbc issue shortly. -- 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 459948] Review Request: libhildon - Hildon Application Framework - shared libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459948 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #5 from Michel Alexandre Salim michel.syl...@gmail.com 2009-02-01 16:16:03 EDT --- Thanks, will add the dependency to gtk-doc. New Package CVS Request === Package Name: libhildon Short Description: Hildon Application Framework - shared libraries Owners: salimma Branches: F-9 F-10 EL-5 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 459948] Review Request: libhildon - Hildon Application Framework - shared libraries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=459948 --- Comment #6 from Michel Alexandre Salim michel.syl...@gmail.com 2009-02-01 16:25:14 EDT --- Note: gtk-doc is pulled in by gtk2-devel, so the initial packaging was correct; however, since the bulk of -devel is actually documentation, I'm splitting it to -doc (with explicit gtk-doc dependency this time); example files are moved there as well. -- Configure 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 483498] New: Modern background
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Modern background https://bugzilla.redhat.com/show_bug.cgi?id=483498 Summary: Modern background Product: Fedora Version: 10 Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: bioinfornat...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora http://bioinformatiques.free.fr/earth-and-moon-0.1-1.fc10.noarch.rpm http://bioinformatiques.free.fr/earth-and-moon-0.1-1.fc10.src.rpm http://bioinformatiques.free.fr/earth-and-moon.spec rpmlint issue: _ $ rpmlint rpmbuild/SPECS/earth-and-moon.spec 0 packages and 1 specfiles checked; 0 errors, 0 warnings. $ rpmlint rpmbuild/SRPMS/earth-and-moon-0.1-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. $ rpmlint rpmbuild/RPMS/noarch/earth-and-moon-0.1-1.fc10.noarch.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. _ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483499] New: Review Request: python-ferari - Optimizer for finite element code
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-ferari - Optimizer for finite element code https://bugzilla.redhat.com/show_bug.cgi?id=483499 Summary: Review Request: python-ferari - Optimizer for finite element code Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-ferari.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-ferari-0.0.2-1.fc10.src.rpm Project URL: http://www.fenics.org/wiki/FErari Description: FErari (Finite Element rearrangement to automatically reduce instructions) generates optimized code for evaluation of the element tensor (element stiffness matrix) and functions as an optimizing backend for FFC. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097726 rpmlint output: [...@laptop24 noarch]$ rpmlint python-ferari-0.0.2-1.fc10.noarch.rpm python-ferari.noarch: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [...@laptop24 SRPMS]$ rpmlint python-ferari-0.0.2-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483498] Modern background
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483498 --- Comment #1 from MERCIER bioinfornat...@gmail.com 2009-02-01 16:45:47 EDT --- $ koji build --scratch dist-f10 rpmbuild/SRPMS/earth-and-moon-0.1-1.fc10.src.rpm Uploading srpm: rpmbuild/SRPMS/earth-and-moon-0.1-1.fc10.src.rpm [] 100% 00:07:06 9.91 MiB 23.77 KiB/sec Created task: 1097741 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097741 Watching tasks (this may be safely interrupted)... 1097741 build (dist-f10, earth-and-moon-0.1-1.fc10.src.rpm): open (x86-5.fedora.phx.redhat.com) 1097742 buildArch (earth-and-moon-0.1-1.fc10.src.rpm, noarch): open (ppc10.fedora.phx.redhat.com) 1097742 buildArch (earth-and-moon-0.1-1.fc10.src.rpm, noarch): open (ppc10.fedora.phx.redhat.com) - closed 0 free 1 open 1 done 0 failed 1097741 build (dist-f10, earth-and-moon-0.1-1.fc10.src.rpm): open (x86-5.fedora.phx.redhat.com) - closed 0 free 0 open 2 done 0 failed 1097741 build (dist-f10, earth-and-moon-0.1-1.fc10.src.rpm) completed successfully -- Configure 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 483498] Modern background
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483498 MERCIER bioinfornat...@gmail.com changed: What|Removed |Added Blocks||177841 -- Configure 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 483376] Review Request: fluid-soundfont - Pro-quality GM/GS soundfont
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483376 --- Comment #5 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-01 16:57:45 EDT --- I managed to cut the size of the patches subpackage down to ~250MB by removing some banks (that are also not in the PersonalCopy-Lite) and by extracting only a single layer from each instrument. But 250MB is still too much. I'll work on it more. Here's my working copy if someone wants to help: http://6mata.com:8014/review/fluid-soundfont.spec.with.gus -- Configure 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 483501] New: Review Request: python-fiat - Generation of arbitrary order instances of the Lagrange elements
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: python-fiat - Generation of arbitrary order instances of the Lagrange elements https://bugzilla.redhat.com/show_bug.cgi?id=483501 Summary: Review Request: python-fiat - Generation of arbitrary order instances of the Lagrange elements Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fab...@bernewireless.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fab.fedorapeople.org/packages/SRPMS/python-fiat.spec SRPM URL: http://fab.fedorapeople.org/packages/SRPMS/python-fiat-0.3.4-1.fc10.src.rpm Project URL: http://www.fenics.org/wiki/FIAT Description: The FInite element Automatic Tabulator FIAT supports generation of arbitrary order instances of the Lagrange elements on lines, triangles, and tetrahedra. It is also capable of generating arbitrary order instances of Jacobi-type quadrature rules on the same element shapes. Upcoming versions will also support Hermite and nonconforming elements as well as H(div) and H(curl) elements such as Raviart-Thomas and Nedelec. Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097748 rpmlint output: [...@laptop24 noarch]$ rpmlint python-fiat-0.3.4-1.fc10.noarch.rpm python-fiat.noarch: W: no-documentation 1 packages and 0 specfiles checked; 0 errors, 1 warnings. [...@laptop24 SRPMS]$ rpmlint python-fiat-0.3.4-1.fc10.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483505] New: Review Request: perl-PerlIO-gzip - Perl extension to provide a PerlIO layer to gzip/gunzip
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-PerlIO-gzip - Perl extension to provide a PerlIO layer to gzip/gunzip Alias: perl-PerlIO-gzip https://bugzilla.redhat.com/show_bug.cgi?id=483505 Summary: Review Request: perl-PerlIO-gzip - Perl extension to provide a PerlIO layer to gzip/gunzip Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/PerlIO-gzip/ OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cw...@alumni.drew.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~cweyl/review/perl-PerlIO-gzip.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-PerlIO-gzip-0.18-1.fc10.src.rpm Description: PerlIO::gzip provides a PerlIO layer that manipulates files in the format used by the gzip program. Compression and decompression are implemented. This is akin to Compress::Zlib, except that it operates at the lower PerlIO layer. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1097838 -- Configure 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 483079] Review Request: perl-constant-boolean - Define TRUE and FALSE constants
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483079 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added 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 483453] Review Request: perl-MooseX-Method - Method declaration with type checking
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483453
manuel wolfshant wo...@nobugconsulting.ro changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
Flag||fedora-review+
--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01
19:49:06 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: devel/x86_64
[x] Rpmlint output:
source RPM: empty
binary RPM:empty
[x] Package is not relocatable.
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic (same as perl)
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 70d0f0151c5b9d42019d5eff88d0c4d3738babc6
MooseX-Method-0.43.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: devel/x86_64 koji scratch build
[-] Package should compile and build into binary rpms on all supported
architectures.
Tested on: standard perl module, noarch
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] make check passes
*** 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 483074] Review Request: perl-Exception-Base - Lightweight exceptions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483074 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483459
manuel wolfshant wo...@nobugconsulting.ro changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
Flag||fedora-review?
--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01
20:06:35 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: devel/x86_64
[x] Rpmlint output:
source RPM: empty
binary RPM:empty
[x] Package is not relocatable.
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic (same as perl)
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 1e61a292c89e236b1ef39a086059724ff80b15ff
MooseX-Iterator-0.09.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: devel/x86_64
[-] Package should compile and build into binary rpms on all supported
architectures.
Tested on: standard perl module, noarch
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] make check passes
The package has some odd requires/provides:
[wo...@wolfy tmp]$ rpm -qp --provides
perl-MooseX-Iterator/perl-MooseX-Iterator-0.09-1.fc11.noarch.rpm
perl(Moose::Meta::Attribute::Custom::Iterable)
perl(MooseX::Iterator) = 0.09
perl(MooseX::Iterator::Array) = 0.09
perl(MooseX::Iterator::Hash) = 0.09
perl(MooseX::Iterator::Meta::Iterable) = 0.09
perl(MooseX::Iterator::Role) = 0.09
perl-MooseX-Iterator = 0.09-1.fc11
[wo...@wolfy tmp]$ rpm -qp --requires
perl-MooseX-Iterator/perl-MooseX-Iterator-0.09-1.fc11.noarch.rpm
perl(:MODULE_COMPAT_5.10.0)
perl(Carp)
perl(Moose)
perl(Moose) = 0.26
perl(Moose::Role)
perl(MooseX::Iterator::Array)
perl(MooseX::Iterator::Hash)
perl(MooseX::Iterator::Meta::Iterable)
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(VersionedDependencies) = 3.0.3-1
Is it really intended to require what itself provides?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are
[Bug 483461] Review Request: perl-MooseX-Param - Simple role to provide a standard param method
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483461
manuel wolfshant wo...@nobugconsulting.ro changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
Flag||fedora-review+
--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01
20:12:23 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: devel/x86_64
[x] Rpmlint output:
source RPM: empty
binary RPM:empty
[x] Package is not relocatable.
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic (same as perl)
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 870bc3d595aceb7f2e3ef20ee0cd0917b72e9510
MooseX-Param-0.02.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: devel/x86_64
[-] Package should compile and build into binary rpms on all supported
architectures.
Tested on: standard perl module, noarch
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] make check passes
*** 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 483453] Review Request: perl-MooseX-Method - Method declaration with type checking
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483453 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Flag|fedora-review+ |fedora-review? --- Comment #2 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01 19:58:07 EDT --- Sorry, I take it back for a second. I think that there are some Requires which need to be filtered out, at least: perl(Test::Exception) perl(Test::Moose) perl(Test::More) Also, some provides seem odd, and especially perl(Foo1) perl(Foo2) perl(Foo3) Am I wrong? -- Configure 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 483474] Review Request: perl-Test-TempDir - Temporary files support for testing
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=483474
manuel wolfshant wo...@nobugconsulting.ro changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
Flag||fedora-review?
--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01
20:27:49 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: devel/x86_64
[x] Rpmlint output:
source RPM: empty
binary RPM:empty
[x] Package is not relocatable.
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic (same as perl)
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 7eb8feb31d17ca559c31cb1a692439470d8ad6eb
Test-TempDir-0.04.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package requires other packages for directories it uses.
[!] Package does not contain duplicates in %files.
See issues
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English langu
ages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: devel/x86_64
[-] Package should compile and build into binary rpms on all supported
architectures.
Tested on: standard perl module, noarch
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] make check passes
===Issues===
There are a lot of duplicate requires:
perl(:MODULE_COMPAT_5.10.0)
perl(Carp)
perl(File::NFSLock)
perl(File::Spec)
perl(File::Temp)
perl(Moose)
perl(Moose) = 0.50
perl(Moose::Util::TypeConstraints)
perl(MooseX::Types::Path::Class)
perl(MooseX::Types::Path::Class)
perl(Path::Class)
perl(Path::Class)
perl(Sub::Exporter)
perl(Sub::Exporter)
perl(Test::TempDir::Factory)
perl(Test::TempDir::Handle)
perl(Test::use::ok)
perl(namespace::clean)
perl(namespace::clean) = 0.08
perl(strict)
perl(warnings)
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(VersionedDependencies) = 3.0.3-1
It looks like most of the stuff that you added as Requires is automatically
picked up by rpmbuild, isn't it ?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
[Bug 483474] Review Request: perl-Test-TempDir - Temporary files support for testing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483474 --- Comment #2 from Allisson Azevedo allis...@gmail.com 2009-02-01 20:44:37 EDT --- Update package: Spec URL: http://allisson.fedorapeople.org/packages/perl-Test-TempDir/perl-Test-TempDir.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-Test-TempDir/perl-Test-TempDir-0.04-2.fc10.src.rpm Changes: * Sun Feb 01 2009 Allisson Azevedo allis...@gmail.com 0.04-2 - Fix requires. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #3 from Allisson Azevedo allis...@gmail.com 2009-02-01 20:50:12 EDT --- Hi, Requires and Provides is automatically picked up by rpmbuild. -- Configure 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #2 from Allisson Azevedo allis...@gmail.com 2009-02-01 20:48:37 EDT --- Update package: Spec URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator-0.09-2.fc10.src.rpm Changes: * Sun Feb 01 2009 Allisson Azevedo allis...@gmail.com 0.09-2 - Fix requires. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483461] Review Request: perl-MooseX-Param - Simple role to provide a standard param method
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483461 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Allisson Azevedo allis...@gmail.com 2009-02-01 20:52:34 EDT --- New Package CVS Request === Package Name: perl-MooseX-Param Short Description: Simple role to provide a standard param method Owners: allisson Branches: F-9 F-10 -- Configure 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #4 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01 21:02:56 EDT --- I know it does, but I think (I might be wrong, though) that in this case it is not correct and they should be filtered out. See https://fedoraproject.org/wiki/Packaging/Perl#Filtering_Requires:_and_Provides -- Configure 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 481022] Review Request: SPE - Stani's Python Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481022 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Summary|Review Request: SPE it is a |Review Request: SPE - |Python IDE editor released |Stani's Python Editor -- Configure 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 481022] Review Request: SPE it is a Python IDE editor released
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481022 --- Comment #3 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01 21:08:38 EDT --- Jonathan, if you do not mind, please try to rephrase the %Summary field. SPE Python IDE editor released is not really correct in English. Maybe simply using Python IDE editor or Python IDE with UML,PyChecker,Debugger,GUI design,Blender more -- Configure 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 483474] Review Request: perl-Test-TempDir - Temporary files support for testing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483474 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01 21:12:07 EDT --- OK, no dups any more 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #5 from Allisson Azevedo allis...@gmail.com 2009-02-01 21:16:35 EDT --- I agree, i'll fix this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483474] Review Request: perl-Test-TempDir - Temporary files support for testing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483474 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Allisson Azevedo allis...@gmail.com 2009-02-01 21:18:42 EDT --- New Package CVS Request === Package Name: perl-Test-TempDir Short Description: Temporary files support for testing Owners: allisson Branches: F-9 F-10 -- Configure 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 483505] Review Request: perl-PerlIO-gzip - Perl extension to provide a PerlIO layer to gzip/gunzip
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483505 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-02-01 21:18:50 EDT --- Review: + package builds in mock (rawhide i386). koji build=http://koji.fedoraproject.org/koji/taskinfo?taskID=1097838 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 933fdf283a0d2739f7630420569e3b24 PerlIO-gzip-0.18.tar.gz + package meets naming and packaging guidelines. + specfile is properly named, is cleanly written + Spec file is written in American English. + Spec file is legible. + dist tag is present. + build root is correct. + license is open source-compatible. + License text is included in package. + %doc is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test outputs All tests successful. Files=2, Tests=561, 0 wallclock secs ( 0.07 usr 0.00 sys + 0.70 cusr 0.07 csys = 0.84 CPU) + Package perl-PerlIO-gzip-0.18-1.fc11 - Provides: gzip.so perl(PerlIO::gzip) = 0.18 Requires: libc.so.6 libc.so.6(GLIBC_2.0) libc.so.6(GLIBC_2.1.3) libc.so.6(GLIBC_2.4) libz.so.1 perl = 0:5.008 perl(XSLoader) perl(strict) perl(warnings) rtld(GNU_HASH) 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 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483258 Allisson Azevedo allis...@gmail.com changed: What|Removed |Added 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #6 from Allisson Azevedo allis...@gmail.com 2009-02-01 22:39:31 EDT --- Update package: Spec URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator.spec SRPM URL: http://allisson.fedorapeople.org/packages/perl-MooseX-Iterator/perl-MooseX-Iterator-0.09-3.fc10.src.rpm Changes: * Sun Feb 01 2009 Allisson Azevedo allis...@gmail.com 0.09-3 - Added filter requires. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 476460] Review Request: pymilter - Python interface to sendmail milter 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=476460 --- Comment #18 from Stuart D Gathman stu...@gathman.org 2009-02-01 22:40:23 EDT --- The python-pydns is an optional dependency. The dns module is often handy for milters that need to lookup MX records, for callback, for instance - but is not part of the core functionality. Debian packages would have an optional dependency. In milter applications, missing optional packages result in smoothly degraded function. A user of pymilter, would get a package not found exception if they tried to use Milter/dns. I am torn, I really ought to put the simplified wrapper for pydns (Milter/dns) in pydns - but was reluctant to do so, wanting to minimize changes to pydns. But now that I am the maintainer of pydns, perhaps that is the best way. Milters could also use dnspython - a larger and more capable DNS library for python. In fact, pyspf-2.1 can use either library. So how does Fedora handle optional packages? Just add the dependency, and who cares if a few extra K for pydns is pulled in? I have a feeling it is either that or move the Milter/dns module to pydns. However, that does not solve the larger problem of optional or alternate libraries - which can crop up in any potentially dynamically linked language, including C. %python_sitelib - I am not clear when to use python-sitelib vs python-archlib. I am hearing however that the policy is to remove all unused macros - and resurrect them from CVS if needed. I will fix that and the other issues, and will think about whether to remove Milter/dns or just add the dependency. -- Configure 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 483016] Package Review: perl-NOCpulse-Debug - Perl debug output 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=483016
Ralf Corsepius rc040...@freenet.de changed:
What|Removed |Added
CC||rc040...@freenet.de
--- Comment #4 from Ralf Corsepius rc040...@freenet.de 2009-02-01 22:46:36
EDT ---
(In reply to comment #3)
Please remove ownership of directories %{perl_vendorlib}/NOCpulse and
${_sysconfdir}/nocpulse. These are both owned by nocpulse-common, which you
Require, and two packages can't own the same directories.
Jon, please make yourself familiar with the FPG.
%{perl_vendorlib}/NOCpulse
_must_ be owned by both 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 483406] Review Request: perl-Text-CSV - Comma-separated values manipulator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483406 Ralf Corsepius rc040...@freenet.de changed: What|Removed |Added CC||rc040...@freenet.de Flag||fedora-review+ --- Comment #2 from Ralf Corsepius rc040...@freenet.de 2009-02-01 22:59:21 EDT --- Clean package. 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 481288] Review Request: python-wokkel - Enhancements to the Twisted XMPP protocol implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481288 --- Comment #4 from Parag AN(पराग) panem...@gmail.com 2009-02-01 23:17:36 EDT --- is this built so that we can CLOSE 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 483459] Review Request: perl-MooseX-Iterator - Iterate over collections
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483459 --- Comment #7 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-01 23:35:56 EDT --- Seems sane now. 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
