[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #8 from Adam Williamson awill...@redhat.com  2009-04-25 02:15:19 
EDT ---
Rahul: my reasoning is that the one from gnome-media in f10 is the one that's
truly 'dead', because it's been turning into the new Pulse-style mixer that
this is intended to supplement. I think it'd be more of a hack to take the old
g-v-c from f10 era and call it something else than it is to just take a
different project.

what we're demonstrating here is that there's likely to be space for a 'full'
mixer for GNOME for the foreseeable future, so to me it seems to make more
sense to pick gnome-alsamixer, which - if the demand turns out to be there -
could easily be resurrected as a fully maintained project upstream; the space
is there for it and it's still a part of GNOME git. You couldn't do that with
the old gnome-volume-control, because that's already become the *new*
gnome-volume-control...

I wouldn't really mind if people felt strongly that we should take the old
g-v-c instead, though. my goal is only to have a decent GUI alsa-level mixer
available out of the box in f11, I'm not that strongly wedded to one or the
other.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497525] Review Request: bubblemon-gnome - Bubbling Load Monitoring Applet for the GNOME Panel

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497525





--- Comment #3 from Edwin ten Brink ed...@tenbrink-bekkers.nl  2009-04-25 
02:17:45 EDT ---
Thank you for your thorough commenting. I have two points where I need some
additional clarification.

1) I've included gettext since it does not appear to be really pulled in by
intltool, see below. Additionally it is suggested by the packaging guidelines:
If the package includes translations, add BuildRequires: gettext. If you
don't, your package could fail to generate translation files in the buildroot.
(https://fedoraproject.org/wiki/Packaging/Guidelines#Handling_Locale_Files).

$ rpm -q gettext
gettext-0.17-8.fc10.i386
$ rpm -q --whatrequires gettext
gettext-devel-0.17-8.fc10.i386
[ed...@localhost ~]$ rpm -q intltool
intltool-0.40.5-1.fc10.i386
[ed...@localhost ~]$ rpm -q --requires intltool
/bin/sh  
/usr/bin/perl  
automake  
gettext-devel  
patch  
perl(Cwd)  
perl(File::Basename)  
perl(File::Copy)  
perl(File::Find)  
perl(Getopt::Long)  
perl(Text::Wrap)  
perl(XML::Parser)  
perl(strict)  
rpmlib(CompressedFileNames) = 3.0.4-1
rpmlib(PayloadFilesHavePrefix) = 4.0-1
rpmlib(VersionedDependencies) = 3.0.3-1

- Do you still feel that gettext is not required? In that case I would like to
suggest a change in the Packaging Guidlines Wiki.

2) The conversion to UTF-8 has been taken care of by the upstream maintainer
during my creation of the rpm spec file since I've found that warning too.
Since it was a 'warning' with a a 'consider' considering the two translated
manpages, I figured this would solve itself in the next upstream release.

W: file-not-utf8 /usr/share/man/sv/man1/bubblemon-gnome2.1.gz
The character encoding of this file is not UTF-8. Consider converting it in
the specfile's %prep section for example using iconv(1).

W: file-not-utf8 /usr/share/man/hu/man1/bubblemon-gnome2.1.gz
The character encoding of this file is not UTF-8.  Consider converting it in
the specfile's %prep section for example using iconv(1).

- Do you think the conversion is a must for this release or a nice to have? It
doesn't really bother to include in this release, but I thought it unnecessary
since I will be removing it the next release anyway.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #9 from Rahul Sundaram sunda...@redhat.com  2009-04-25 02:33:10 
EDT ---

IMO, your understanding seems incorrect. Refer

http://mail.gnome.org/archives/desktop-devel-list/2009-January/msg00330.html

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497339] Review Request: qmforge - Analysis tools for quantum mechanical calculations

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497339


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

   Flag|needinfo?(jussi.leht...@iki |
   |.fi)|




--- Comment #7 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 03:33:50 
EDT ---
Try this one. For me qmforge worked perfectly, but I had PyOpenGL installed. I
added it as a requirement. I don't know whether PyQt4 should pull in PyOpenGL,
it might be a packaging bug in PyQt4.

http://theory.physics.helsinki.fi/~jzlehtol/rpms/qmforge.spec
http://theory.physics.helsinki.fi/~jzlehtol/rpms/qmforge-2.1-4.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 494517] Review Request: python-xkit - A simple, transparent and easy to extend xorg parser

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494517





--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 03:44:13 
EDT ---
Ping, are you going to import?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490576] Review Request: bibtex2html - Collection of tools for translating from BibTeX to HTML

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490576





--- Comment #12 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 03:42:13 
EDT ---
Ping, have you done any reviews and do you still need a sponsor?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 492979] Review Request: python-cliutils - A collection of utilities easing the creation of command line scripts

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=492979





--- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 03:48:52 
EDT ---
Ping, do you still need a sponsor? Do you have other submissions and have you
done reviews of other 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 493335] Review Request: perl-RTx-Calendar - Calendar for RT due tasks

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=493335


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 03:51:43 
EDT ---
It *is* a problem. According to the file system standard, everything that comes
with the distro should be in libdir; /usr/local is for software that hasn't
been packaged and is locally built.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #10 from Adam Williamson awill...@redhat.com  2009-04-25 03:59:01 
EDT ---
Quoting Dave Airlie from the list:

I took a quick look at this, and gnome-media has the code moved to the
gst-mixer subdir, but thats it, the code still believes its called
g-v-c, all the files are called that, the gconf schema keys, the
desktop, etc.

So I suspect the effort to do that vs ship gnome-alsamixer at this point
in the development cycle isn't going to provide any useful advantages to
the advanced sound configuration people are requiring. Like it the
answer to the question is otherwise run a cli app called alsamixer, I'd
ship a kde app quicker :)

To this I'd add that we don't necessarily *want* a fairly heavy gstreamer mixer
applet that supports Pulse as well as ALSA mixer control. We want a simple
standalone application that simply lets you poke the ALSA mixers, nothing else.
gnome-alsamixer seems to fit that description.

But again I will go for the old g-v-c if more people favour it. I'd just ask
that someone else do the work of completing the re-write to identify as
something different from the new g-v-c, and handling the package, as I'm not
familiar with it at 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 494517] Review Request: python-xkit - A simple, transparent and easy to extend xorg parser

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494517





--- Comment #10 from Rahul Sundaram sunda...@redhat.com  2009-04-25 03:57:40 
EDT ---
There has been a build failure. 

http://koji.fedoraproject.org/koji/buildinfo?buildID=97763

I haven't had time to look into it yet.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #11 from Adam Williamson awill...@redhat.com  2009-04-25 04:05:19 
EDT ---
Just uploaded an updated .spec and .src.rpm with the patch applied.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497465] Review Request: perl-Config-INI - Config::INI Perl module

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497465


Jussi Lehtola jussi.leht...@iki.fi changed:

   What|Removed |Added

 CC||jussi.leht...@iki.fi




--- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 04:05:21 
EDT ---
Doesn't build in mock:

ERROR: Bad build req: No Package Found for perl(Mixin::Linewise::Readers) =
0.001. Exiting.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 484567] Review Request: php-ezc-File - eZ Components File

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=484567


Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|fed...@famillecollet.com
   Flag||fedora-review+




--- Comment #3 from Remi Collet fed...@famillecollet.com  2009-04-25 04:09:38 
EDT ---
REVIEW:

+ rpmlint is ok
php-ezc-File.src: I: checking
php-ezc-File.noarch: I: checking
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
+ package name
+ spec file name 
+ package meet the PHP Guidelines (new update)
+ License ok : BSD
+ License is upstream 
+ spec in english and legible
+ license file in sources is provided
+ sources match the upstream sources
36ff7eeb3699b2bf36ccf9b64c224154  File-1.2.tgz
+ Source URL ok
+ build  on F10.x86_64
+ BuildRequires (php-pear = 1:1.4.9-1.2) ok
+ no locale
+ no .so
+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ contain code
+ no documentation
+ no devel
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in mock (fedora-rawhide-x86_64)
+ no test suite
+ scriptlets ok
+ Final Requires ok
php-common = 5.1.1
php-pear(components.ez.no/Base) = 1.4
+ Final Provides ok
php-pear(components.ez.no/File) = 1.2
php-ezc-File = 1.2-2.fc8

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 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593


Andreas Thienemann andr...@bawue.net changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|andr...@bawue.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 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593


Andreas Thienemann andr...@bawue.net changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #12 from Andreas Thienemann andr...@bawue.net  2009-04-25 
04:16:03 EDT ---
Looks good. Approved.

Please be so kind and add all commands needed for tarball generation as a
comment. Not only git clone, but the appropriate tar command. But we don't need
a new review-round for that.

Please follow http://fedoraproject.org/wiki/CVS_admin_requests to have the
existing gnome-alsamixer ACLs changed and a new branch created for F11 and
devel. A Package Change Request would be appropriate for that.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 488538] Review Request: php-ezc-Authentication - eZ Components Authentication

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488538


Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com
 AssignedTo|nob...@fedoraproject.org|fed...@famillecollet.com
   Flag||fedora-review+

Bug 488538 depends on bug 484509, which changed state.

Bug 484509 Summary: Review Request: php-ezc-Base -  eZ Components Base
https://bugzilla.redhat.com/show_bug.cgi?id=484509

   What|Old Value   |New Value

 Status|ASSIGNED|ON_QA
 Resolution||NEXTRELEASE
 Status|ON_QA   |CLOSED



--- Comment #1 from Remi Collet fed...@famillecollet.com  2009-04-25 04:19:12 
EDT ---
REVIEW:

+ rpmlint is ok
php-ezc-Authentication.src: I: checking
php-ezc-Authentication.noarch: I: checking
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
+ package name
+ spec file name 
+ package meet the PHP Guidelines (new update)
+ License ok : BSD
+ License is upstream 
+ spec in english and legible
+ license file in sources is provided
+ sources match the upstream sources
a736fac4611c5d8e495fb7d8831b4fa8  Authentication-1.2.3.tgz
+ Source URL ok
+ build  on F10.x86_64
+ BuildRequires (php-pear = 1:1.4.9-1.2) ok
+ no locale
+ no .so
+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ contain code
+ small documentation not required to run
+ no devel
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in mock (fedora-rawhide-x86_64)
+ no test suite
+ scriptlets ok
+ Final Requires ok
php-bcmath  
php-common = 5.2.1
php-ldap  
php-pear(components.ez.no/Base) = 1.5.2
+ Final Provides ok
php-pear(components.ez.no/Authentication) = 1.2.3
php-ezc-Authentication = 1.2.3-1.fc8


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 497622] New: Review Request: apbs - adaptive poisson boltzmann solver

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: apbs - adaptive poisson boltzmann solver

https://bugzilla.redhat.com/show_bug.cgi?id=497622

   Summary: Review Request: apbs - adaptive poisson boltzmann
solver
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: f...@stanford.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.stanford.edu/~fenn/packs/apbs.spec
SRPM URL: http://www.stanford.edu/~fenn/packs/apbs-1.1.0-1.fc10.src.rpm

Description: APBS is a software package for the numerical solution of the
Poisson-Boltzmann equation (PBE), one of the most popular continuum models for
describing
electrostatic interactions between molecular solutes in salty, aqueous media.
APBS was designed to efficiently evaluate electrostatic properties for such
simulations for a wide range of length scales to enable the investigation of
molecules with tens to millions of atoms. It is also widely used in molecular
visualization (in such applications as PyMOL).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497465] Review Request: perl-Config-INI - Config::INI Perl module

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497465





--- Comment #2 from Iain Arnell iarn...@gmail.com  2009-04-25 04:27:42 EDT ---
does build in mock (see the scratch build url).  The problem is that rawhide
is currently pointing at F11 and perl-Mixin-Linewise doesn't exist there yet
(the update is pending).  You need to tweak your mock configuration to include
the latest F12 stuff from koji.  Something like:

[local]
name=local
baseurl=http://koji.fedoraproject.org/static-repos/dist-f12-build-current/x86_64/
cost=2000
enabled=1

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497465] Review Request: perl-Config-INI - Config::INI Perl module

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497465





--- Comment #3 from Jussi Lehtola jussi.leht...@iki.fi  2009-04-25 04:51:29 
EDT ---
Right, that's why yum --enablerepo=rawhide didn't find anything, either.

Well, in a couple of days the requirements should be in older releases 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 495420] Review Request: rapid-photo-downloader - Images downloader for external devices

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=495420





--- Comment #1 from Fabian Affolter fab...@bernewireless.net  2009-04-25 
04:51:22 EDT ---
notify-python was added as a requirement. 

Spec URL:
http://fab.fedorapeople.org/packages/SRPMS/rapid-photo-downloader.spec
SRPM URL:
http://fab.fedorapeople.org/packages/SRPMS/rapid-photo-downloader-0.0.8-2.b7.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 489795] Review Request: backintime - Simple backup system

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=489795


Simon Wesp cassmod...@fedoraproject.org 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 489795] Review Request: backintime - Simple backup system

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=489795





--- Comment #8 from Simon Wesp cassmod...@fedoraproject.org  2009-04-25 
05:25:54 EDT ---
New Package CVS Request
===
Package Name: backintime
Short Description: Simple backup system
Owners: cassmodiah
Branches: F-10 F-11
InitialCC: 

- Please change %description to Gnome frontend for %{name} and KDE frontend
for %{name}.
DONE (will be in CVS)

- Add TRANSLATIONS to %doc, it's a credits file like AUTHORS
DONE (will be in CVS)

- use cp -p when copying from bindir to sbindir
DONE (will be in CVS)

- mark %{_datadir}/gnome/help/%{name}/ as %doc
DONE (will be in CVS)

- Desktopfile
yeah, it was hard for me to come to a decision. I prefer sed and
desktopinstall, too. But on the other hand i saw in other packages that they
patch the whole desktopfile with categories and so on, because they have to
edit them, so they decided to make the whole with the patch instead of
splitting patch and desktopfileinstall. it's good to hear that you would make
it like i would prefer. this agreement shows me that my line of thought is not
unsolicited.
DONE (will be in 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 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #13 from manuel wolfshant wo...@nobugconsulting.ro  2009-04-25 
05:33:16 EDT ---
Created an attachment (id=341305)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=341305)
thanks for providing a decent alternative to PA

Thanks for bringing this tool in, guys. Despite what PA team thinks, I prefer
the topmost control panel over the one in the bottom. With alsa I can
individually control the left and right output channel. And a couple more
inputs which are doomed in PA as being irrelevant.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497525] Review Request: bubblemon-gnome - Bubbling Load Monitoring Applet for the GNOME Panel

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497525





--- Comment #4 from Christoph Wickert fed...@christoph-wickert.de  2009-04-25 
06:12:43 EDT ---
(In reply to comment #3)

 [ed...@localhost ~]$ rpm -q --requires intltool
 /bin/sh  
 /usr/bin/perl  
 automake  
 gettext-devel  

And gettext-devel requires gettext. On the other hand it does not harm to
mention it, I recall in Fedora  7 intlttol did not pull in gettext-devel. It's
up to you, that's why I said not *really*.

 - Do you think the conversion is a must for this release or a nice to have?

It's a nice to have, I wouldn't call it a blocker. I will do the final review
tomorrow, stay tuned.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 486302] Review Request: parrot - Parrot Virtual Machine

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=486302


Christoph Wickert fed...@christoph-wickert.de changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #45 from Christoph Wickert fed...@christoph-wickert.de  
2009-04-25 06:26:09 EDT ---
(In reply to comment #44)
 (Contrary to what I stated in comment #38, Christoph is probably going to
 sponsor you now.)  

Gerd, I 'm sponsorind you now in order to get parrot into Fedora 11.
Nevertheless I'd like to see some review efforts from you before you submit
other 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 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593


Christoph Wickert fed...@christoph-wickert.de changed:

   What|Removed |Added

 CC||fed...@christoph-wickert.de




--- Comment #14 from Christoph Wickert fed...@christoph-wickert.de  
2009-04-25 06:50:15 EDT ---
Please don't use %makeinstall, see
https://fedoraproject.org/wiki/Packaging/Guidelines#Why_the_.25makeinstall_macro_should_not_be_used

Inline desktop file creation ws forbidden recently, see
https://fedoraproject.org/wiki/Packaging/Guidelines#.desktop_file_creation

rpmbuild --rebuild does not work for F-10, see http://fpaste.org/paste/10125

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593


Dan Horák d...@danny.cz changed:

   What|Removed |Added

 CC||d...@danny.cz




--- Comment #15 from Dan Horák d...@danny.cz  2009-04-25 07:52:56 EDT ---
(In reply to comment #14)
 Inline desktop file creation ws forbidden recently, see
 https://fedoraproject.org/wiki/Packaging/Guidelines#.desktop_file_creation

The guidelines were not updated yet, but inline creating of desktop files is
allowed for few weeks now, see https://fedorahosted.org/fesco/ticket/129 for
details.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #16 from Andreas Thienemann andr...@bawue.net  2009-04-25 
08:02:04 EDT ---
(In reply to comment #14)


 Please don't use %makeinstall, see
 https://fedoraproject.org/wiki/Packaging/Guidelines#Why_the_.25makeinstall_macro_should_not_be_used

As discussed on IRC: make DESTDIR doesn't work, I checked.
Patching the makefile would be an option, but using %makeinstall does work as
expected. If the maintainer wants to patch the Makefile to make DESTDIR work,
that is fine. I'd leave it to his discretion.
It's not a blocker as is.

 Inline desktop file creation ws forbidden recently, see
 https://fedoraproject.org/wiki/Packaging/Guidelines#.desktop_file_creation

The wiki does state You can do this by including a .desktop file you create as
a Source: (e.g. Source3: %{name}.desktop) or generating it in the spec file.

As you mentioned on IRC, there seems to have been a mail somewhere stating
this. So even though it seems not (yet) to be official policy, I've attached a
patch fixing this potential future problem. 

 rpmbuild --rebuild does not work for F-10, see http://fpaste.org/paste/10125  

Old file. That was already fixed.
http://adamwill.fedorapeople.org/gnome-alsamixer/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
was the last submission from packager and does not exhibit that problem.
Not a blocker.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #18 from Andreas Thienemann andr...@bawue.net  2009-04-25 
08:07:11 EDT ---
(In reply to comment #15)

 The guidelines were not updated yet, but inline creating of desktop files is
 allowed for few weeks now, see https://fedorahosted.org/fesco/ticket/129 for
 details.  

Thanks for clearing that up.

Considering that, I'd retract my latest patch. Package is fine as is. If Adam
wants to externalize the .desktop file, it's up to him. I don't consider any of
the raised points blockers for inclusion.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497634] New: Review Request: perl-App-Daemon - Start an Application as a Daemon

2009-04-25 Thread bugzilla
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-App-Daemon - Start an Application as a Daemon
Alias: perl-App-Daemon

https://bugzilla.redhat.com/show_bug.cgi?id=497634

   Summary: Review Request: perl-App-Daemon - Start an Application
as a Daemon
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/App-Daemon/
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~iarnell/review/perl-App-Daemon.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-App-Daemon-0.06-1.fc12.src.rpm

Description:
App::Daemon helps running an application as a daemon.

Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1320265

*rt-0.06

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #17 from Andreas Thienemann andr...@bawue.net  2009-04-25 
08:04:31 EDT ---
Created an attachment (id=341313)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=341313)
changing inline .desktop to external

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #19 from manuel wolfshant wo...@nobugconsulting.ro  2009-04-25 
08:18:33 EDT ---
(In reply to comment #16)

  rpmbuild --rebuild does not work for F-10, see 
  http://fpaste.org/paste/10125  
 
 Old file. That was already fixed.
 http://adamwill.fedorapeople.org/gnome-alsamixer/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm


at least in a mockbuild for F10 the construct
   cat  /%{name}.desktop  EOF
fails because / is not writable by mock.
   cat  %{name}.desktop  EOF
however works just fine.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #20 from Christoph Wickert fed...@christoph-wickert.de  
2009-04-25 08:27:00 EDT ---
(In reply to comment #16)
 Old file. That was already fixed.
 http://adamwill.fedorapeople.org/gnome-alsamixer/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
 was the last submission from packager and does not exhibit that problem.
 Not a blocker. 

This *is* the latest submission and it fails:

$ rpmbuild --rebuild
Desktop/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm 
Installiere Desktop/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Warnung: Benutzer adamw existiert nicht - benutze Root
Warnung: Gruppe adamw existiert nicht - benutze Root
Fehler: Zeile 24: Ungültiges Zeichen '-' in release: Release:
0.1.20090424gitc540b26.gnome-alsamixer-20090424gitc540b26.tar.bz210

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #21 from manuel wolfshant wo...@nobugconsulting.ro  2009-04-25 
08:33:34 EDT ---
Created an attachment (id=341315)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=341315)
successfull build log of alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
(with one small change in the spec)

only change versus
http://adamwill.fedorapeople.org/gnome-alsamixer/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
is removing the leading / in the name of the desktop file when creating it
inline.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #22 from Andreas Thienemann andr...@bawue.net  2009-04-25 
08:38:56 EDT ---
(In reply to comment #20)

 This *is* the latest submission and it fails

Works for me:

[andr...@workstation ~]$ wget -q
http://adamwill.fedorapeople.org/gnome-alsamixer/gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
[andr...@workstation ~]$ rpmbuild --rebuild
gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
Installing gnome-alsamixer-0.9.7-0.1.20090424gitc540b26.fc10.src.rpm
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
warning: user adamw does not exist - using root
warning: group adamw does not exist - using root
Executing(%prep): /bin/sh -e /var/tmp/rpm-tmp.EDaoPb

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497593] Review Request: gnome-alsamixer - advanced mixer for GNOME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497593





--- Comment #23 from Andreas Thienemann andr...@bawue.net  2009-04-25 
08:41:24 EDT ---
(In reply to comment #19)

 at least in a mockbuild for F10 the construct
cat  /%{name}.desktop  EOF
 fails because / is not writable by mock.
cat  %{name}.desktop  EOF
 however works just fine.  

good catch. My fault in the patch when I changed to desktop-file-install which
I corrected when I switched to the external .desktop file so I never noticed
that part during the review...

Should be fixed for the import.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497639] New: Review Request: gmixer - Just a simple audio mixer

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: gmixer - Just a simple audio mixer

https://bugzilla.redhat.com/show_bug.cgi?id=497639

   Summary: Review Request: gmixer - Just a simple audio mixer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: leigh123li...@googlemail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://leigh123linux.fedorapeople.org/review/gmixer/1/gmixer.spec
SRPM URL:
http://leigh123linux.fedorapeople.org/review/gmixer/1/gmixer-0.3.5-1.fc10.src.rpm
Description: Just Another MIXer A GTK+ 1.2 based mixer.
Nothing fancy, just a simple audio mixer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497640] New: Review Request: rubygem-RedCloth - Textile parser for Ruby.

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: rubygem-RedCloth - Textile parser for Ruby.

https://bugzilla.redhat.com/show_bug.cgi?id=497640

   Summary: Review Request: rubygem-RedCloth - Textile parser for
Ruby.
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dpie...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://mcpierce.fedorapeople.org/rpms/rubygem-RedCloth.spec
SRPM URL:
http://mcpierce.fedorapeople.org/rpms/rubygem-RedCloth-4.1.9-1.fc10.src.rpm
Description: Textile parser for Ruby.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 426753] Review Request: xmonad - A tiling window manager

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426753





--- Comment #42 from Jens Petersen peter...@redhat.com  2009-04-25 09:53:22 
EDT ---
http://petersen.fedorapeople.org/xmonad/xmonad.spec
http://petersen.fedorapeople.org/xmonad/xmonad-0.8.1-9.fc10.src.rpm

I have been using this package now recently and it works ok for me -
uses most of Till's latest package but slightly simpler:
just xmonad.desktop for xmonad-start.

I feel this is probably enough for the initial fedora xmonad package.
We can get more sophisticated later if necessary, but for now
I suggest advanced users who want to customize their session startup
can use xorg-x11-xinit-session and ~/.xsession say.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 495412] Review Request: flamerobin - Graphical client for Firebird

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=495412





--- Comment #9 from Philippe Makowski makowski.fed...@gmail.com  2009-04-25 
10:47:54 EDT ---
ok, upgraded
Spec URL: http://ibphoenix.fr//fedora/flamerobin.spec
SRPM URL: http://ibphoenix.fr//fedora/flamerobin-0.9.2-1.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 488539] Review Request: php-ezc-Configuration - eZ Components Configuration

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488539


Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com
 AssignedTo|nob...@fedoraproject.org|fed...@famillecollet.com
   Flag||fedora-review+

Bug 488539 depends on bug 484509, which changed state.

Bug 484509 Summary: Review Request: php-ezc-Base -  eZ Components Base
https://bugzilla.redhat.com/show_bug.cgi?id=484509

   What|Old Value   |New Value

 Status|ASSIGNED|ON_QA
 Resolution||NEXTRELEASE
 Status|ON_QA   |CLOSED



--- Comment #1 from Remi Collet fed...@famillecollet.com  2009-04-25 11:30:17 
EDT ---
REVIEW:

+ rpmlint is ok
php-ezc-Configuration.src: I: checking
php-ezc-Configuration.noarch: I: checking
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
+ package name
+ spec file name 
+ package meet the PHP Guidelines (new update)
+ License ok : BSD
+ License is upstream 
+ spec in english and legible
+ license file in sources is provided
+ sources match the upstream sources
e588fc7317e45674998d42d064ae731a  Configuration-1.3.2.tgz
+ Source URL ok
+ build  on F10.x86_64
+ BuildRequires (php-pear = 1:1.4.9-1.2, php-channel(components.ez.no)) ok
+ no locale
+ no .so
+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ contain code
+ small documentation not required to run
+ no devel
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in mock (fedora-rawhide-x86_64)
+ no test suite
+ scriptlets ok
+ Final Requires ok
/usr/bin/pear  
php-common = 5.2.1
php-pear(components.ez.no/Base) = 1.5.2
+ Final Provides ok
php-pear(components.ez.no/Configuration) = 1.3.2
php-ezc-Configuration = 1.3.2-1.fc8




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 488542] Review Request: php-ezc-Database - eZ Components Database

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488542


Remi Collet fed...@famillecollet.com changed:

   What|Removed |Added

 CC||fed...@famillecollet.com
 AssignedTo|nob...@fedoraproject.org|fed...@famillecollet.com
   Flag||fedora-review+




--- Comment #2 from Remi Collet fed...@famillecollet.com  2009-04-25 11:39:12 
EDT ---
REVIEW:

+ rpmlint is ok
php-ezc-Database.src: I: checking
php-ezc-Database.noarch: I: checking
2 packages and 1 specfiles checked; 0 errors, 0 warnings.
+ package name ok
+ spec file name ok
+ package meet the PHP Guidelines (new update)
+ License ok : BSD
+ License is upstream 
+ spec in english and legible
+ license file in sources is provided
+ sources match the upstream sources
b48377b4153f49da304337a7e768bf38  Database-1.4.4.tgz
+ Source URL ok
+ build  on F10.x86_64
+ BuildRequires (php-pear = 1:1.4.9-1.2, php-channel(components.ez.no)) ok
+ no locale
+ no .so
+ own all directories that it creates
+ no duplicate file
+ %defattr ok
+ %clean section
+ use macros consistently
+ contain code
+ small documentation not required to run
+ no devel
+ no pkgconfig
+ no sub-package
+ no GUI
+ don't own files or directories already owned by other packages
+ %install start with rm -rf 
+ valid UTF-8
+ build in mock (fedora-rawhide-x86_64)
+ no test suite
+ scriptlets ok
+ Final Requires ok
/usr/bin/pear  
php-common = 5.2.1
php-pdo  
php-pear(components.ez.no/Base) = 1.6.1
+ Final Provides ok
php-pear(components.ez.no/Database) = 1.4.4
php-ezc-Database = 1.4.4-1.fc8


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 453855] Review Request: globus-openssl-module - Globus Toolkit - Globus OpenSSL Module Wrapper

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=453855


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review+  |fedora-review?




--- Comment #10 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-25 
12:33:39 EDT ---
Hold on, I said no further questions but I need to take a step back and ask
another:

* The installed package gives the rpmlint:

$ rpmlint globus-openssl-module
globus-openssl-module.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libglobus_openssl.so.0.0.6 /lib64/libdl.so.2
globus-openssl-module.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libglobus_openssl.so.0.0.6 /lib64/libz.so.1
globus-openssl-module.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libglobus_openssl.so.0.0.6 /usr/lib64/libltdl.so.3
globus-openssl-module.x86_64: W: unused-direct-shlib-dependency
/usr/lib64/libglobus_openssl.so.0.0.6 /lib64/libpthread.so.0

Also, some of the other globus packages that we approved have these kind of
warnings. Are these warnings given because the .pc files report more links than
they actually need to? Or how can we fix these?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490588] Review Request: minicomputer - Software Synthesizer

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490588





--- Comment #11 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-25 
12:46:35 EDT ---
Alright, I updated the flags one more time:

Spec URL: http://oget.fedorapeople.org/review/minicomputer.spec
SRPM URL: http://oget.fedorapeople.org/review/minicomputer-1.3-3.fc10.src.rpm

Changelog: 1.3-3
- Cleanup the compiler flags

Rawhide build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1320465

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 484229] Review Request: qbittorrent - A bittorrent Client

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=484229


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #49 from leigh scott leigh123li...@googlemail.com  2009-04-25 
12:49:50 EDT ---
Package Change Request
==
Package Name: qbittorrent
New Branches: F-11
Owners leigh123linux

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485580] Review Request: netactview - Graphical network connections viewer for Linux

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485580


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #10 from leigh scott leigh123li...@googlemail.com  2009-04-25 
12:57:42 EDT ---
Package Change Request
==
Package Name: netactview
New Branches: F11
Owners: leigh123linux

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485580] Review Request: netactview - Graphical network connections viewer for Linux

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485580





--- Comment #11 from leigh scott leigh123li...@googlemail.com  2009-04-25 
12:58:13 EDT ---
Package Change Request
==
Package Name: netactview
New Branches: F-11
Owners: leigh123linux

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457213] Review Request: html2text - HTML-to-text converter

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457213


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #45 from leigh scott leigh123li...@googlemail.com  2009-04-25 
12:59:54 EDT ---
Package Change Request
==
Package Name: html2text
New Branches: F-11
Owners: leigh123linux

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 488542] Review Request: php-ezc-Database - eZ Components Database

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488542


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 488542] Review Request: php-ezc-Database - eZ Components Database

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488542





--- Comment #3 from Guillaume Kulakowski llaum...@gmail.com  2009-04-25 
13:12:21 EDT ---
New Package CVS Request
===
Package Name: php-ezc-Database
Short Description: A lightweight database layer on top of PHP's PDO
Owners: llaumgui
Branches: 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 468597] Review Request: rubygem-ferret - Full-featured text search engine library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=468597


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
   Flag|needinfo?(kana...@kanarip.c |
   |om) |




--- Comment #45 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-04-25 
13:15:23 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 488539] Review Request: php-ezc-Configuration - eZ Components Configuration

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488539


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite file format

2009-04-25 Thread bugzilla
Please do not reply directly 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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

   Flag||needinfo?(alexes...@yahoo.c
   ||om)




--- Comment #21 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-04-25 
13:14:30 EDT ---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 488538] Review Request: php-ezc-Authentication - eZ Components Authentication

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488538


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Guillaume Kulakowski llaum...@gmail.com  2009-04-25 
13:18:56 EDT ---
New Package CVS Request
===
Package Name: php-ezc-Authentication
Short Description: Support for different means of identification 
authentication
Owners: llaumgui
Branches: 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 488539] Review Request: php-ezc-Configuration - eZ Components Configuration

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=488539





--- Comment #2 from Guillaume Kulakowski llaum...@gmail.com  2009-04-25 
13:15:10 EDT ---
New Package CVS Request
===
Package Name: php-ezc-Configuration
Short Description: Allows you to use configuration files in different formats
Owners: llaumgui
Branches: 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 493246] Review Request: Shutter -- a feature-rich screenshot program.

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=493246


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-04-25 
13:19:57 EDT ---
(Removing NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 484567] Review Request: php-ezc-File - eZ Components File

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=484567


Guillaume Kulakowski llaum...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Guillaume Kulakowski llaum...@gmail.com  2009-04-25 
13:20:16 EDT ---
New Package CVS Request
===
Package Name: php-ezc-File
Short Description: Provides support for file operations which are not covered
by PHP
Owners: llaumgui
Branches: 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 490588] Review Request: minicomputer - Software Synthesizer

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490588


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #12 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-04-25 13:29:35 EDT ---
Ok it seems fine for me 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


[Bug 226146] Merge Review: mkinitrd

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=226146


Hans de Goede hdego...@redhat.com changed:

   What|Removed |Added

 AssignedTo|jussi.leht...@iki.fi|hdego...@redhat.com




--- Comment #3 from Hans de Goede hdego...@redhat.com  2009-04-25 13:27:37 
EDT ---
Sorry, -ENOTIME, atleast not for the coming month. I'm assigning this to me
(even though I'm not the reviewer) so that this does not drop of my radar.

-- 
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 484229] Review Request: qbittorrent - A bittorrent Client

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=484229


Rex Dieter rdie...@math.unl.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #50 from Rex Dieter rdie...@math.unl.edu  2009-04-25 13:32:33 EDT 
---
fyi, F-11 branches have already been made for all packages.  (you may need a
cvs -d up)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 485580] Review Request: netactview - Graphical network connections viewer for Linux

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=485580


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 484229] Review Request: qbittorrent - A bittorrent Client

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=484229





--- Comment #51 from leigh scott leigh123li...@googlemail.com  2009-04-25 
13:55:47 EDT ---
(In reply to comment #50)
 fyi, F-11 branches have already been made for all packages.  (you may need a
 cvs -d up)  

Thank you for the info.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 457213] Review Request: html2text - HTML-to-text converter

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=457213


leigh scott leigh123li...@googlemail.com changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490588] Review Request: minicomputer - Software Synthesizer

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490588


Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #13 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-25 
13:57:24 EDT ---
Thank you Pavel!

New Package CVS Request
===
Package Name: minicomputer
Short Description: Software Synthesizer
Owners: oget
Branches: F-9 F-10 F-11
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=447368





--- Comment #9 from Zach Carter z.car...@f5.com  2009-04-25 15:06:01 EDT ---
New upstream version, built for f11:

http://www.zachcarter.com/rpms/schroot.spec
http://www.zachcarter.com/rpms/schroot-1.2.2-1.fc11.src.rpm

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490588] Review Request: minicomputer - Software Synthesizer

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490588





--- Comment #14 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-04-25 
15:31:04 EDT ---
I think you need to assign yourself to the bug and set the bug to ASSIGNED.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497441] Review Request: mumble - Voice chat application

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497441





--- Comment #14 from Igor Jurišković juriskovic.i...@gmail.com  2009-04-25 
15:50:09 EDT ---
Nah, don't like to wait. So I did almost everything as I think it should be.

MUMBLE:
[make...@localhost SPECS]$ rpmls ../RPMS/i386/mumble-1.1.8-2.fc9.i386.rpm
-rwxr-xr-x  /usr/bin/mumble
-rwxr-xr-x  /usr/lib/libmumble.so
-rwxr-xr-x  /usr/lib/libmumble.so.1
-rwxr-xr-x  /usr/lib/libmumble.so.1.1
-rwxr-xr-x  /usr/lib/libmumble.so.1.1.8
-rwxr-xr-x  /usr/lib/mumble/liblink.so
drwxr-xr-x  /usr/share/doc/mumble-1.1.8
-rw-r--r--  /usr/share/doc/mumble-1.1.8/CHANGES
-rw-r--r--  /usr/share/doc/mumble-1.1.8/LICENSE
-rw-r--r--  /usr/share/doc/mumble-1.1.8/README
-rw-r--r--  /usr/share/doc/mumble-1.1.8/README.Linux
[make...@localhost SPECS]$

[make...@localhost SPECS]$ rpmlint ../RPMS/i386/mumble-1.1.8-2.fc9.i386.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[make...@localhost SPECS]$
-
MURMUR:
[make...@localhost SPECS]$ rpmls ../RPMS/i386/murmur-1.1.8-2.fc9.i386.rpm
lrw-rw-r--  /etc/mumble-server.ini
-rw-rw-r--  /etc/murmur/murmur.ini
-rwxr-xr-x  /etc/rc.d/init.d/murmur
lrwxrwxrwx  /usr/bin/mumble-server
lrwxrwxrwx  /usr/sbin/murmur
-rwxr-xr-x  /usr/sbin/murmurd
drwxr-xr-x  /usr/share/doc/murmur-1.1.8
-rw-r--r--  /usr/share/doc/murmur-1.1.8/CHANGES
-rw-r--r--  /usr/share/doc/murmur-1.1.8/LICENSE
-rw-r--r--  /usr/share/doc/murmur-1.1.8/README
-rw-r--r--  /usr/share/doc/murmur-1.1.8/README.Linux
[make...@localhost SPECS]$

[make...@localhost SPECS]$ rpmlint ../RPMS/i386/murmur-1.1.8-2.fc9.i386.rpm
murmur.i386: W: non-standard-uid /etc/rc.d/init.d/murmur murmur
murmur.i386: W: non-standard-gid /etc/rc.d/init.d/murmur murmur
murmur.i386: W: non-standard-uid /etc/murmur/murmur.ini murmur
murmur.i386: W: non-standard-gid /etc/murmur/murmur.ini murmur
murmur.i386: W: non-standard-uid /usr/bin/mumble-server murmur
murmur.i386: W: non-standard-gid /usr/bin/mumble-server murmur
murmur.i386: W: non-standard-uid /usr/sbin/murmur murmur
murmur.i386: W: non-standard-gid /usr/sbin/murmur murmur
murmur.i386: W: non-standard-uid /etc/mumble-server.ini murmur
murmur.i386: W: non-standard-gid /etc/mumble-server.ini murmur
murmur.i386: W: non-standard-uid /usr/sbin/murmurd murmur
murmur.i386: W: non-standard-gid /usr/sbin/murmurd murmur
murmur.i386: W: spurious-bracket-in-%preun
1 packages and 0 specfiles checked; 0 errors, 13 warnings.
[make...@localhost SPECS]$

All those warnings are related to new user and group murmur:murmur. They
shouldn't make any problems? But W13 is related to if[] in %preun. Tried to fix
it but no success. Its something related to space before ]?
-
SRPM:
[make...@localhost SPECS]$ rpmlint ../SRPMS/mumble-1.1.8-2.fc9.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[make...@localhost SPECS]$
-
SPEC:
[make...@localhost SPECS]$ rpmlint mumble.spec
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[make...@localhost SPECS]$
-
INIT:
When I do /etc/init.d/murmur it throws me an error. Should I remove this line
or fix it somehow?
root /home/makerpm/rpmbuild/SPECS  #  /etc/init.d/murmur
/etc/init.d/murmur: line 16: /lib/lsb/init-functions: No such file or directory
Usage: servicename {start|stop|status|restart|reload|force-reload|condrestart
root /home/makerpm/rpmbuild/SPECS  #


- fixed murmur.init. lockfile pointed to /var/log/subsys/murmur correct one is
/var/lock/subsys/murmur
-

SPEC: http://78.46.84.75:81/tj/etmain/mumble/mumble.spec
SRPM: http://78.46.84.75:81/tj/etmain/mumble/mumble-1.1.8-2.fc9.src.rpm
NEW INIT: http://78.46.84.75:81/tj/etmain/mumble/murmur.init


TODO:
Install:
icons - where, how...?
scripts - should I add them as %doc or /usr/share/mumble/?
desktop-file for mumble
subpackage for mumble-overlay and desktop file - can somebody explain this a
little bit more. As far as I see mumble-overlay is just bash script that
preloads library before starting chosen program. Why would I create desktop
file for script?


Hmm, that's all? 

Review wanted. :)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490380] Review Request: qt-creator - Lightweight and cross-platform IDE for Qt

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490380





--- Comment #12 from Muayyad Alsadi als...@ojuba.org  2009-04-25 16:32:48 EDT 
---
Created an attachment (id=341325)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=341325)
spec file, fixing the icon

I noticed that the icons are put in %{_datadir}/pixmaps/qtcreator_logo_*.png

so I added links to them in
%{_datadir}/icons/hicolor/*/apps/Nokia-QtCreator.png

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497660] Review Request: perl-MIME-Charset - Charset Informations for MIME

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497660


Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Blocks||497661




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497661] Review Request: perl-MIME-EncWords - Deal with RFC 2047 encoded words

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497661


Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Depends on||497660




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497661] New: Review Request: perl-MIME-EncWords - Deal with RFC 2047 encoded words

2009-04-25 Thread bugzilla
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-MIME-EncWords - Deal with RFC 2047 encoded words

https://bugzilla.redhat.com/show_bug.cgi?id=497661

   Summary: Review Request: perl-MIME-EncWords - Deal with RFC
2047 encoded words
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xav...@bachelot.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.bachelot.org/fedora/SPECS/perl-MIME-EncWords.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-MIME-EncWords-1.010.101-1.fc10.src.rpm
Description: perl module to deal with RFC 2047 encoded words.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497660] New: Review Request: perl-MIME-Charset - Charset Informations for MIME

2009-04-25 Thread bugzilla
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-MIME-Charset - Charset Informations for MIME

https://bugzilla.redhat.com/show_bug.cgi?id=497660

   Summary: Review Request: perl-MIME-Charset - Charset
Informations for MIME
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xav...@bachelot.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.bachelot.org/fedora/SPECS/perl-MIME-Charset.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-MIME-Charset-1.006.2-1.fc10.src.rpm
Description: MIME::Charset provides informations about character sets used for
MIME messages on Internet.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497663] Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497663


Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Depends on||497662




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497662] Review Request: perl-HTML-StripScripts - Strip scripting constructs out of HTML

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497662


Xavier Bachelot xav...@bachelot.org changed:

   What|Removed |Added

 Blocks||497663




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497525] Review Request: bubblemon-gnome - Bubbling Load Monitoring Applet for the GNOME Panel

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497525





--- Comment #5 from Edwin ten Brink ed...@tenbrink-bekkers.nl  2009-04-25 
17:02:37 EDT ---
(In reply to comment #4)

I've uploaded the changed files for your final review. (The previous locations
were URL's of a different mirror.)

Spec URL:
http://savannah.inetbridge.net/bubblemon/fedora-stage/gnome-applet-bubblemon.spec
SRPM URL:
http://savannah.inetbridge.net/bubblemon/fedora-stage/gnome-applet-bubblemon-2.0.13-3.fc10.src.rpm

In these files all your remarks are incorporated with the exception of:
- gettext (where I decided to follow the suggestions in the Packaging
Guidelines), and
- the conversion of the two manpages (languages sv and hu) since they are
already included by the upstream maintainer in the upcoming next release and
these files have a small target audience which should have the used character
encoding anyway.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497665] New: Review Request: perl-Crypt-CipherSaber - Perl module implementing CipherSaber encryption

2009-04-25 Thread bugzilla
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-Crypt-CipherSaber - Perl module implementing 
CipherSaber encryption

https://bugzilla.redhat.com/show_bug.cgi?id=497665

   Summary: Review Request: perl-Crypt-CipherSaber - Perl module
implementing CipherSaber encryption
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xav...@bachelot.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.bachelot.org/fedora/SPECS/perl-Crypt-CipherSaber.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-Crypt-CipherSaber-1.00-1.fc10.src.rpm
Description: The Crypt::CipherSaber module implements CipherSaber encryption,
described at http://ciphersaber.gurus.com/. It is simple, fairly speedy, and
relatively secure algorithm based on RC4.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497663] New: Review Request: perl-HTML-StripScripts-Parser - XSS filter using HTML::Parser

2009-04-25 Thread bugzilla
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-HTML-StripScripts-Parser - XSS filter using 
HTML::Parser

https://bugzilla.redhat.com/show_bug.cgi?id=497663

   Summary: Review Request: perl-HTML-StripScripts-Parser - XSS
filter using HTML::Parser
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xav...@bachelot.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://www.bachelot.org/fedora/SPECS/perl-HTML-StripScripts-Parser.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-HTML-StripScripts-Parser-1.02-1.fc10.src.rpm
Description: This class provides an easy interface to HTML::StripScripts, using
HTML::Parser to parse the HTML.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 497662] New: Review Request: perl-HTML-StripScripts - Strip scripting constructs out of HTML

2009-04-25 Thread bugzilla
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-HTML-StripScripts - Strip scripting constructs 
out of HTML

https://bugzilla.redhat.com/show_bug.cgi?id=497662

   Summary: Review Request: perl-HTML-StripScripts - Strip
scripting constructs out of HTML
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: xav...@bachelot.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.bachelot.org/fedora/SPECS/perl-HTML-StripScripts.spec
SRPM URL:
http://www.bachelot.org/fedora/SRPMS/perl-HTML-StripScripts-1.04-1.fc10.src.rpm
Description: This module strips scripting constructs out of HTML, leaving as
much non-scripting markup in place as possible. This allows web applications to
display HTML originating from an untrusted source without introducing XSS
(cross site scripting) vulnerabilities.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 495001] Review Request: bareftp - File transfer client supporting the FTP, FTP over SSL/TLS (FTPS) and SSH File Transfer Protocol (SFTP)

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=495001


Simon Wesp cassmod...@fedoraproject.org changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #10 from Simon Wesp cassmod...@fedoraproject.org  2009-04-25 
17:04:37 EDT ---
---REVIEW PACKAGENAME---

template: https://fedoraproject.org/wiki/Packaging/ReviewGuidelines

--
MUST: rpmlint must be run on every package. The output should be posted in the
review.
O.K.

bareftp.i386: E: no-binary
The package should be of the noarch architecture because it doesn't contain
any binaries.

bareftp.i386: W: only-non-binary-in-usr-lib
There are only non binary files in /usr/lib so they should be in /usr/share.

2 packages and 0 specfiles checked; 1 errors, 1 warnings.

StartWrapper - okay
dll=lib - okay

--
MUST: The package must be named according to the Package Naming Guidelines .
O.K.

--
MUST: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
O.K.

--
MUST: The package must meet the Packaging Guidelines .
O.K.

--
MUST: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
O.K.

--
MUST: The License field in the package spec file must match the actual license.
O.K.

--
MUST: If (and only if) the source package includes the text of the license(s)
in its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
O.K.

--
MUST: The spec file must be written in American English.
O.K.

--
MUST: The spec file for the package MUST be legible.
O.K.

--
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
Package: d19c2cb8ea26703e4d9314772a0acc94
my DL: d19c2cb8ea26703e4d9314772a0acc94

--
MUST: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
O.K.

--
MUST: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
O.K.

--
MUST: All build dependencies must be listed in BuildRequires, except for any
that are listed in the exceptions section of the Packaging Guidelines ;
inclusion of those as BuildRequires is optional. Apply common sense.
O.K.

--
MUST: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
O.K.

--
MUST: Every binary RPM package (or subpackage) which stores shared library
files (not just symlinks) in any of the dynamic linker's default paths, must
call ldconfig in %post and %postun.
N/A

--
MUST: If the package is designed to be relocatable, the packager must state
this fact in the request for review, along with the rationalization for
relocation of that specific package. Without this, use of Prefix: /usr is
considered a blocker.
O.K.

--
MUST: A package must own all directories that it creates. If it does not create
a directory that it uses, then it should require a package which does create
that directory.
O.K.

--
MUST: A Fedora package must not list a file more than once in the spec file's
%files listings.
O.K.

--
MUST: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line.
O.K.

--
MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT).
O.K.

--
MUST: Each package must consistently use macros.
O.K.

--
MUST: The package must contain code, or permissable content.
O.K.

--
MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity).
N/A

--
MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present.
N/A

--

[Bug 497525] Review Request: bubblemon-gnome - Bubbling Load Monitoring Applet for the GNOME Panel

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=497525





--- Comment #6 from Edwin ten Brink ed...@tenbrink-bekkers.nl  2009-04-25 
17:13:07 EDT ---
(In reply to comment #2)
 Preserve timestamps during install:
 
 make install DESTDIR=%{buildroot} INSTALL=install -p
 
 See: https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps  

This is a valid comment. I'm only wondering why the generated template spec
file does not include the part INSTALL=install -p.

Isn't this something which should be changed in rpmdev-newspec, which I used to
generate the 'offending' line? Future newcomers might be making this same
mistake again by following the generated spec.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 495001] Review Request: bareftp - File transfer client supporting the FTP, FTP over SSL/TLS (FTPS) and SSH File Transfer Protocol (SFTP)

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=495001


Itamar Reis Peixoto ita...@ispbrasil.com.br changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #11 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-04-25 
17:50:19 EDT ---
New Package CVS Request
===
Package Name: bareftp
Short Description:  File transfer client supporting ftp, ftps and sftp
protocols
Owners: itamarjp cassmodiah
Branches: F-9 F-10 F-11
InitialCC:

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 490380] Review Request: qt-creator - Lightweight and cross-platform IDE for Qt

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=490380


Itamar Reis Peixoto ita...@ispbrasil.com.br changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #13 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-04-25 
18:15:52 EDT ---
(In reply to comment #12)
thanks for the patch, applyed

-

qt 4.5.1 has been pushed for F-9 and F-10, I like to make qt-creator available
for these branches too


Package Change Request
==
Package Name: qt-creator
New Branches: F-9 F-10 
Owners: itamarjp

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 426752] Review Request: ghc-X11-xft - Haskell binding to Xft

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=426752





--- Comment #6 from Jens Petersen peter...@redhat.com  2009-04-25 18:54:34 
EDT ---
ping?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 447368] Review Request: schroot - Execute commands in a chroot environment

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=447368


Clint Savage her...@gmail.com changed:

   What|Removed |Added

 CC||her...@gmail.com




--- Comment #10 from Clint Savage her...@gmail.com  2009-04-25 19:14:27 EDT 
---
The pam security patch seems fine to me.  It's a straight over move from a
non-fedora pam to our system-auth model.  

Looks pretty good.  I'd say a cursory review from someone a bit more
experienced than I is a good idea, but it sure seems like a nice applcation.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 494148] Review Request: soci - The database access library for C++ programmers

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494148





--- Comment #17 from Denis Arnaud denis.arnaud_fed...@m4x.org  2009-04-25 
20:33:54 EDT ---
The files for the new Extras-SOCI project are as following:
Spec URL: http://denisarnaud.fedorapeople.org/extras-soci/extras-soci.spec
SRPM URL:
http://denisarnaud.fedorapeople.org/extras-soci/extras-soci-3.1.0-1.fc10.src.rpm

And the corresponding Koji builds:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1320781 (i586) and
http://koji.fedoraproject.org/koji/taskinfo?taskID=1320788 (x86_64)

Those files correspond to the CVS trunk version of SOCI, i.e., when installing
those RPM packages, we install the latest version of the SOCI library (and
header files).

-
Note: for the (pristine) version 3.0.0 of SOCI, the files have been moved to
the following location, and are still valid:
Spec URL: http://denisarnaud.fedorapeople.org/pristine300/soci.spec
SRPM URL:
http://denisarnaud.fedorapeople.org/pristine300/soci-3.0.0-4.fc10.src.rpm

[Patch URLs:
http://denisarnaud.fedorapeople.org/pristine300/soci-3.0.0-fix-gcc43-compatibility.patch
and
http://denisarnaud.fedorapeople.org/pristine300/soci-3.0.0-fix-gnu-autotools-compatibility.patch
has been updated
]  

[As that release (pristine 3.0.0 version) is fully based on the official
release of the SOCI library, they are not hosted by the Extras-SOCI project]
-

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 494693] Review Request: gloox - A rock-solid, full-featured Jabber/XMPP client library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494693





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-04-26 00:44:59 EDT ---
gloox-1.0-0.5.SVNr4003.fc9 has been submitted as an update for Fedora 9.
http://admin.fedoraproject.org/updates/gloox-1.0-0.5.SVNr4003.fc9

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 494693] Review Request: gloox - A rock-solid, full-featured Jabber/XMPP client library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494693





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-04-26 00:46:00 EDT ---
gloox-1.0-0.5.SVNr4003.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/gloox-1.0-0.5.SVNr4003.fc11

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 494693] Review Request: gloox - A rock-solid, full-featured Jabber/XMPP client library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494693





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-04-26 00:45:29 EDT ---
gloox-1.0-0.5.SVNr4003.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/gloox-1.0-0.5.SVNr4003.fc10

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 494693] Review Request: gloox - A rock-solid, full-featured Jabber/XMPP client library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494693


Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info 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 494695] Review Request: qutIM - Multiplatform Instant Messenger on Qt4

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=494695


Bug 494695 depends on bug 494693, which changed state.

Bug 494693 Summary: Review Request: gloox - A rock-solid, full-featured 
Jabber/XMPP client library
https://bugzilla.redhat.com/show_bug.cgi?id=494693

   What|Old Value   |New Value

 Status|NEW |ASSIGNED
 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 496718] Review Request: mingw32-plotmm - MinGW GTKmm plot widget for scientific applications

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=496718


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se
   Flag||fedora-review?




--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-04-26 
01:06:23 EDT ---
Build fails:

+ libtoolize --force --copy
/var/tmp/rpm-tmp.sWKsRh: line 44: libtoolize: command not found

Missing BuildRequires on: libtool, automake, autoconf.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 491618] Review Request: mingw32-libsqlite3x - MinGW Windows C++ wrapper for the sqlite database library

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=491618


Mattias Ellert mattias.ell...@fysast.uu.se changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mattias.ell...@fysast.uu.se
   Flag||fedora-review?




--- Comment #1 from Mattias Ellert mattias.ell...@fysast.uu.se  2009-04-26 
01:06:16 EDT ---
Build fails:

 i686-pc-mingw32-g++ -DHAVE_CONFIG_H -I. -O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOUR\
CE=2 -fexceptions --param=ssp-buffer-size=4 -mms-bitfields -O2 -Wall -c
sqlite3\
x_command.cpp  -DPIC
env: i686-pc-mingw32-g++: Permission denied
make[1]: *** [sqlite3x_command.lo] Error 1
make[1]: Leaving directory `/builddir/build/BUILD/libsqlite3x-2007.10.18'
make: *** [all] Error 2

Probable cause: missing BuildRequires: mingw32-gcc-c++

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review


[Bug 489564] Review Request: Blueman - Bluetooth Manager

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=489564


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lemen...@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 481536] Review Request: enano - Enano CMS, a php-based modular content management system

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=481536


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lemen...@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 495420] Review Request: rapid-photo-downloader - Images downloader for external devices

2009-04-25 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=495420


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||lemen...@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