[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627





--- Comment #6 from Jakub Hrozek jhro...@redhat.com  2009-01-12 03:54:52 EDT 
---
Done:

Spec URL: http://jhrozek.fedorapeople.org/fsniper.spec
SRPM URL: http://jhrozek.fedorapeople.org/fsniper-1.3.1-3.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 472348] Review Request: hunspell-no - Norwegian hunspell dictionaaries

2009-01-12 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=472348


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #6 from Caolan McNamara caol...@redhat.com  2009-01-12 04:18:28 
EDT ---
Package Change Request
==
Package Name: hunspell-no
New Branches: F-10
Owners: caolanm

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

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


[Bug 472348] Review Request: hunspell-no - Norwegian hunspell dictionaaries

2009-01-12 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=472348


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

 Blocks||479599




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

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


[Bug 479598] Review Request: aopalliance - AOP offers a better solution to many problems than do existing technologies

2009-01-12 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=479598





--- Comment #3 from Sandro Mathys s...@sandro-mathys.ch  2009-01-12 04:32:15 
EDT ---
Spec URL: http://red.fedorapeople.org/SRPMS/aopalliance.spec
SRPM URL:
http://red.fedorapeople.org/SRPMS/aopalliance-1.0-1.20090111cvs.fc11.src.rpm

Thanks for the comments. I tried to address both issues in the new version.
Writing a good summary for this is tough, though ;)

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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|dkay...@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 469474] Review Request: sovix - A website revision system

2009-01-12 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=469474





--- Comment #7 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
04:45:46 EDT ---
- Please preserve the time stamps while copying
  https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps
- It would be nice if you use the same style for commands everywhere in your
spec file.  Relating to %{__rm} vs. rm
- 'Requires: webserver' has to be 'Requires: httpd'
- The URL of this project is http://sovix.org/sovix/Sovix
(http://gnu.org/software/sovix is a redirect)

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

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


[Bug 226148] Merge Review: mlocate

2009-01-12 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=226148





--- Comment #4 from Miloslav Trmac m...@redhat.com  2009-01-12 04:46:40 EDT 
---
Thanks for the patch.  I have applied most of it in mlocate-0.21.1-3, with the
following changes:
- add exit 0 to %triggerpostun as well
- Don't replace /etc/cron.daily by %{_sysconfdir}/cron.daily.
  Somebody might reasonably redefine %{_bindir} etc. to e.g. build a mlocate
  package located under /opt/mlocate, but the cron.daily script must be in
  /etc/cron.daily, where it is expected by cron configuration, not in
  /opt/mlocate/etc/cron.daily, for example.

-- 
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 478941] Review Request: celt - An audio codec for use in low-delay speech and audio communication

2009-01-12 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=478941


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||rjo...@redhat.com
 AssignedTo|nob...@fedoraproject.org|rjo...@redhat.com
   Flag||fedora-review?




--- Comment #1 from Richard W.M. Jones rjo...@redhat.com  2009-01-12 04:49:22 
EDT ---
Taking for review.

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

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


[Bug 448661] Review Request: u-boot-tools - contains mkimage a tool to create kernel bootable images for u-boot

2009-01-12 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=448661


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag||needinfo?(ga...@kernel.cras
   ||hing.org)




--- Comment #14 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
04:51:18 EDT ---
Any progress on this bug?

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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 AssignedTo|dkay...@gmail.com   |nob...@fedoraproject.org
   Flag|fedora-review?  |




--- Comment #7 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
04:48:55 EDT ---
sorry for the spurious assignments, I have not noticed that David is not in the
packager group.
I'll try to do  a review myself later today, if I have the time.

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

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


[Bug 448661] Review Request: u-boot-tools - contains mkimage a tool to create kernel bootable images for u-boot

2009-01-12 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=448661





--- Comment #15 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
05:05:54 EDT ---
Two other small things...

- The URL should be 'http://www.denx.de/wiki/U-Boot/WebHome'
- Please preserve the time stamps while copying
  https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps


Use %defattr(-,root,root,-) instead of %defattr(-,root,root) and leave
%attr(755,root,root) away.  Without further investigations, I think it will
work.  Or is there an issue with the permission of mkimage?
For details
https://fedoraproject.org/wiki/PackageMaintainers/CreatingPackageHowTo#.25files_section

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

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


[Bug 450539] Review Request: service-discovery-applet - Service discovery applet based on Avahi for the Gnome panel

2009-01-12 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=450539


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag||needinfo?(t...@niemueller.de
   ||)




--- Comment #8 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
05:08:09 EDT ---
Is there any progress with this bug?

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

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


[Bug 475897] Review Request: ncmpcpp - Clone of ncmpc with new features and written in C++

2009-01-12 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=475897


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|fab...@bernewireless.net
   Flag||fedora-review?




--- Comment #2 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
05:26:13 EDT ---
Some comment before I do a full review.

- '-n %{name}-%{version}' is not needed in %setup because this is the default.
- ChangeLog is missing in %doc
- I personally like one line per BR ;-)

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

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


[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

2009-01-12 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=478504





--- Comment #12 from Christoph Wickert fed...@christoph-wickert.de  
2009-01-12 05:22:39 EDT ---
(In reply to comment #10)
 Please ask nautilus maintainer to make nautilus own
 %{_libdir}/epiphany/%{ephy_major}/extensions before doing such a thing
 ref:
 https://bugzilla.redhat.com/show_bug.cgi?id=459088#c25
 https://bugzilla.redhat.com/show_bug.cgi?id=469491

Mamoru, at least %{_libdir}/epiphany/ and %{_libdir}/epiphany/%{version} _are_
already owned by epiphany and also owning
%{_libdir}/epiphany/%{version}/extensions will _not_ help us here. Imagine
this:
/usr/lib/epiphany
/usr/lib/epiphany/2.22
/usr/lib/epiphany/2.22/extensions/gget.py*

When epiphany gets updated to 2.23 we have:
/usr/lib/epiphany (unowned)
/usr/lib/epiphany/2.22 (unowned)
/usr/lib/epiphany/2.22/extensions/gget.py* (unowned)
/usr/lib/epiphany
/usr/lib/epiphany/2.23
/usr/lib/epiphany/2.23/extensions

The next day the gget update comes:
/usr/lib/epiphany (unowned)
/usr/lib/epiphany/2.22 (unowned)
/usr/lib/epiphany/2.22/extensions (unowned and empty)
/usr/lib/epiphany
/usr/lib/epiphany/2.23
/usr/lib/epiphany/2.23/extensions/gget.py*

The problem is the version number in there.

 This should only happen on rawhide, no problem.

You are right, there will be no major version update in the stable release. But
the other two problems will remain. We only have the alternatives of
- ether let both packages own the dirs or
- use strictly versioned deps and delay the epiphany update in rawhide because
of them. This is just like the firefox nightmares.

To me the latter is the worst. I'm open to suggestions though, because my gwget
package is also affected. I have been trying to find it's review, but gwget is
one of the former fedora.us packages and I took it over some time ago.

The only solution I can think of: Make epiphany own
%{_libdir}/epiphany/
%{_libdir}/epiphany/plugins
%{_libdir}/epiphany/extensions
%{_libdir}/epiphany/%{version}
%{_libdir}/epiphany/%{version}/plugins - %{_libdir}/epiphany/plugins
%{_libdir}/epiphany/%{version}/extensions - %{_libdir}/epiphany/extensions

What do you think, Mamoru?

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

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


[Bug 456756] Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz

2009-01-12 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=456756


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

   Flag||needinfo?(leigh123li...@goo
   ||glemail.com)




--- Comment #11 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
05:32:08 EDT ---
Any progress on this?

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

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


[Bug 456756] Review Request: alien - A program that converts between the rpm, dpkg, stampede slp, and slackware tgz

2009-01-12 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=456756


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

   What|Removed |Added

   Flag|needinfo?(leigh123li...@goo |
   |glemail.com)|




--- Comment #12 from leigh scott leigh123li...@googlemail.com  2009-01-12 
05:47:49 EDT ---
(In reply to comment #11)
 Any progress on this?

Not yet , I am still working on getting the dep's passed ( dpkg debhelper )

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

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


[Bug 479645] New: Review Request: hunspell-uk - Ukrainian hyphenation rules

2009-01-12 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: hunspell-uk - Ukrainian hyphenation rules

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

   Summary: Review Request: hunspell-uk - Ukrainian hyphenation
rules
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: caol...@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://people.redhat.com/caolanm/rpms/hyphen-uk.spec
SRPM URL:
http://people.redhat.com/caolanm/rpms/hyphen-uk-0.20030903-1.fc10.src.rpm
Description: Ukrainian hyphenation rules

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

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


[Bug 479645] Review Request: hyphen-uk - Ukrainian hyphenation rules

2009-01-12 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=479645


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

Summary|Review Request: hunspell-uk |Review Request: hyphen-uk -
   |- Ukrainian hyphenation |Ukrainian hyphenation rules
   |rules   |




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

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


[Bug 479646] New: Review Request: hunspell-mi - Maori hyphenation rules

2009-01-12 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: hunspell-mi - Maori hyphenation rules

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

   Summary: Review Request: hunspell-mi - Maori hyphenation rules
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: caol...@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://people.redhat.com/caolanm/rpms/hyphen-mi.spec
SRPM URL:
http://people.redhat.com/caolanm/rpms/hyphen-mi-0.20080630-1.fc10.src.rpm
Description: Maori hyphenation rules

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

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


[Bug 479647] New: Review Request: perl-Devel-Dumpvar - Pure-OO reimplementation of dumpvar.pl

2009-01-12 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-Devel-Dumpvar - Pure-OO reimplementation of 
dumpvar.pl

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

   Summary: Review Request:  perl-Devel-Dumpvar - Pure-OO
reimplementation of dumpvar.pl
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@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://mmaslano.fedorapeople.org/perl-Devel-Dumpvar/perl-Devel-Dumpvar.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-Devel-Dumpvar/perl-Devel-Dumpvar-0.04-1.fc10.src.rpm
Description: 
Most perl dumping modules are focused on serializing data structures into
a format that can be rebuilt into the original data structure. They do
this with a variety of different focuses, such as human readability, the
ability to execute the dumped code directly, or to minimize the size of
the dumped data.

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

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


[Bug 479645] Review Request: hyphen-uk - Ukrainian hyphenation rules

2009-01-12 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=479645


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review+




--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
06:10:54 EDT ---
mostly identical to bug 478533. all is fine, sha1sum matches
(63ff1e53152c60044ed8804d54e0d0e8c80e69eb hyph_uk_UA.zip), license (GPLv2+) is
correct.


APPROVED assuming you will use the correct name :)

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

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


[Bug 479646] Review Request: hunspell-mi - Maori hyphenation rules

2009-01-12 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=479646





--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
06:14:11 EDT ---
Is it hyphen or hunspell ? The bug name says hunspell, the spec / SRPM say
hyphen, that source file is hunspell-hyphen...

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

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


[Bug 477190] Review Request: cas - core analysis system

2009-01-12 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=477190





--- Comment #8 from Adam Stokes asto...@redhat.com  2009-01-12 06:20:53 EDT 
---
Under the status is says 'unapproved' could this be why?

Thanks

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

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


[Bug 479646] Review Request: hunspell-mi - Maori hyphenation rules

2009-01-12 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=479646


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review?




--- Comment #2 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
06:21:43 EDT ---
mostly identical to bug 478533. all is fine, sha1sum matches
(53d6bb363d21726aaad7e832bcd252c7ff7baa9f
hunspell-hyphen-mi-0.1.20080630-beta.tar.gz), license (GPLv3+) is
correct.

Waiting for clarifications on the name. I assume hyphen-mi would be the correct
one, but...

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

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


[Bug 477190] Review Request: cas - core analysis system

2009-01-12 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=477190





--- Comment #9 from Adam Stokes asto...@redhat.com  2009-01-12 06:41:00 EDT 
---
Ok I re-applied, may I get approval now?

Thanks!

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

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


[Bug 451771] Review Request: maximus - an addon for metacity

2009-01-12 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=451771





--- Comment #6 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
06:38:09 EDT ---
Any progress on this?

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

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


[Bug 479646] Review Request: hyphen-mi - Maori hyphenation rules

2009-01-12 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=479646


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
06:45:03 EDT ---
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 479646] Review Request: hyphen-mi - Maori hyphenation rules

2009-01-12 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=479646


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Caolan McNamara caol...@redhat.com  2009-01-12 06:49:48 
EDT ---
New Package CVS Request
===
Package Name: hyphen-mi
Short Description: Maori hyphenation rules
Owners: caolanm
Branches: devel
InitialCC:

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

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


[Bug 479645] Review Request: hyphen-uk - Ukrainian hyphenation rules

2009-01-12 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=479645


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Caolan McNamara caol...@redhat.com  2009-01-12 06:33:45 
EDT ---
New Package CVS Request
===
Package Name: hyphen-uk
Short Description: Ukrainian hyphenation rules
Owners: caolanm
Branches: devel
InitialCC:

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

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


[Bug 479646] Review Request: hyphen-mi - Maori hyphenation rules

2009-01-12 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=479646


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

Summary|Review Request: hunspell-mi |Review Request: hyphen-mi -
   |- Maori hyphenation rules   |Maori hyphenation rules




--- Comment #3 from Caolan McNamara caol...@redhat.com  2009-01-12 06:35:21 
EDT ---
hyphen-mi I mean, getting confused.

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

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


[Bug 479666] New: Review Request: perl-Parse-ErrorString-Perl - perl-Parse-ErrorString-Perl

2009-01-12 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-Parse-ErrorString-Perl - 
perl-Parse-ErrorString-Perl

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

   Summary: Review Request: perl-Parse-ErrorString-Perl -
perl-Parse-ErrorString-Perl
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@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://mmaslano.fedorapeople.org/perl-Parse-ErrorString-Perl/perl-Parse-ErrorString-Perl.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-Parse-ErrorString-Perl/perl-Parse-ErrorString-Perl-0.11-1.fc10.src.rpm
Description: description here

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

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


[Bug 479667] New: Review Request: perl-ORLite-Migrate - Extremely light weight SQLite-specific schema migration

2009-01-12 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-ORLite-Migrate - Extremely light weight 
SQLite-specific schema migration

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

   Summary: Review Request: perl-ORLite-Migrate - Extremely light
weight SQLite-specific schema migration
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@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://mmaslano.fedorapeople.org/perl-ORLite-Migrate/perl-ORLite-Migrate.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-ORLite-Migrate/perl-ORLite-Migrate-0.01-1.fc10.src.rpm
Description:
THIS CODE IS EXPERIMENTAL AND SUBJECT TO CHANGE WITHOUT NOTICE
SQLite is a light weight single file SQL database that provides an excellent
platform for embedded storage of structured data. ORLite is a light weight
single class Object-Relational Mapper (ORM) system specifically designed for
(and limited to only) work with SQLite. ORLite::Migrate is a light
weight single class Database Schema Migration enhancement for ORLite.

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

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


[Bug 479669] New: Review Request: perl-File-pushd - Change directory temporarily for a limited scope

2009-01-12 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-File-pushd - Change directory temporarily for a 
limited scope

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

   Summary: Review Request: perl-File-pushd - Change directory
temporarily for a limited scope
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@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://mmaslano.fedorapeople.org/perl-File-pushd/perl-File-pushd.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-File-pushd/perl-File-pushd-1.00-1.fc10.src.rpm
Description: 
File::pushd does a temporary chdir that is easily and automatically
reverted, similar to pushd in some Unix command shells. It works by
creating an object that caches the original working directory. When the
object is destroyed, the destructor calls chdir to revert to the original
working directory. By storing the object in a lexical variable with a
limited scope, this happens automatically at the end of the scope.

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

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


[Bug 226542] Merge Review: words

2009-01-12 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=226542


Karel Zak k...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE
   Flag|needinfo?(k...@redhat.com)  |




--- Comment #6 from Karel Zak k...@redhat.com  2009-01-12 07:41:38 EDT ---
(In reply to comment #4)
 Some suggestions, I really would like to see, but no must:
  - Remove BuildRequires: coreutils, something else anyway pulls that in

 Fixed.

  - Replace Source: mwords.tar.Z by Source:
 http://web.archive.org/web/20060527013227/http://www.dcs.shef.ac.uk/research/ilash/Moby/mwords.tar.Z;
 as it is a valid source URL and everybody can verify it

 OK, good idea. Fixed.

 Karel, olease close the bug report once you've applied the changes or let me 
 know why you won't do this and close this bug report therewith.

 Closing...

-- 
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 479667] Review Request: perl-ORLite-Migrate - Extremely light weight SQLite-specific schema migration

2009-01-12 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=479667


Marcela Maslanova mmasl...@redhat.com changed:

   What|Removed |Added

 Depends on||479669




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

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


[Bug 477732] Review Request: xfconf - Hierarchical configuration system for Xfce

2009-01-12 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=477732





--- Comment #10 from Marcela Maslanova mmasl...@redhat.com  2009-01-12 
07:41:33 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 479669] Review Request: perl-File-pushd - Change directory temporarily for a limited scope

2009-01-12 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=479669


Marcela Maslanova mmasl...@redhat.com changed:

   What|Removed |Added

 Blocks||479667




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

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


[Bug 478362] Review Request: fmirror - Mirror directories from ftp servers

2009-01-12 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=478362





--- Comment #2 from Fabian Affolter fab...@bernewireless.net  2009-01-12 
08:03:35 EDT ---
Thanks Christoph for your review and your help.  I will incorporate the changes
as soon as possible.

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

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


[Bug 467397] Review Request: mingw32-libpng - MinGW Windows Libpng library

2009-01-12 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=467397


Adam Tkac at...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||at...@redhat.com
 AssignedTo|nob...@fedoraproject.org|at...@redhat.com
   Flag||fedora-review?




--- Comment #2 from Adam Tkac at...@redhat.com  2009-01-12 08:05:49 EDT ---
I will review this package.

current problems:
- the latest version is not packaged (there is 1.2.34 in rawhide)

rpmlint errors:
mingw32-libpng.noarch: W: spurious-executable-perm
/usr/i686-pc-mingw32/sys-root/mingw/lib/libpng12.dll.a
mingw32-libpng.noarch: W: spurious-executable-perm
/usr/i686-pc-mingw32/sys-root/mingw/lib/libpng.dll.a
mingw32-libpng.noarch: E: script-without-shebang
/usr/i686-pc-mingw32/sys-root/mingw/lib/libpng12.la
^^^ I don't think we need libtool *la files installed, do we?

Otherwise I don't see any other problem.

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

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


[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

2009-01-12 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=478504





--- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-12 
08:30:33 EDT ---
(In reply to comment #12)
 (In reply to comment #10)
 Imagine this:
 /usr/lib/epiphany
 /usr/lib/epiphany/2.22
 /usr/lib/epiphany/2.22/extensions/gget.py*
 
 When epiphany gets updated to 2.23 we have:
 /usr/lib/epiphany (unowned)
 /usr/lib/epiphany/2.22 (unowned)
 /usr/lib/epiphany/2.22/extensions/gget.py* (unowned)
 /usr/lib/epiphany
 /usr/lib/epiphany/2.23
 /usr/lib/epiphany/2.23/extensions

Well,
- As when epiphany is upgrade from 2.22 to 2.23, then
  I guess gget-epiphany-extension will no longer work (although
  I don't know this package well) unless gget is rebuilt
  against new epiphany.

  i.e. if epiphany can be upgraded without gget-epiphany-extension
   is rebuilt, it is _already_ wrong. Not-rebuilt gget-epiphany-extension
   should prevent epiphany from being upgraded in such a case
   (theoretically).

  Some idea:
  - Add Conflicts: epiphany = 2.23
Conflicts: epiphany  2.22
  - Ask epiphany maintainer to support Provides: epiphany(abi) = 2.22,
for example.

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

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


[Bug 479596] Review Request: serafettin-fonts - Comic Sans fonts

2009-01-12 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=479596





--- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-01-12 
08:37:41 EDT ---
Spec URL: http://oget.fedorapeople.org/review/serafettin-fonts.spec
SRPM URL:
http://oget.fedorapeople.org/review/serafettin-fonts-0.4-1.fc10.src.rpm

Changelog:
- New version with a name change (comic - cartoon)

I also made a wiki page for this font:
https://fedoraproject.org/wiki/Serafettin_fonts

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

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


[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

2009-01-12 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=478504





--- Comment #14 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-12 
08:44:29 EDT ---
Also, as some packages already use %_libdir/epiphany//extensions,
making every package use this directory own this directory cannot be
accepted and the owner of this directory must be unified.

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

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


[Bug 451298] Review Request: pidgin-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-12 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=451298





--- Comment #9 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-12 
08:46:51 EDT ---
(In reply to comment #5)
 I already do have an account, my login is madko. I'm just trying to log in
 but it says that my password has expired so i'm waiting for the reset password
 email... I guess you don't find my account because i haven't filled any 
 details
 on it?

well

to get sponsored first you need to join the Fedora Packager CVS Commit Group
(packager)

at the flowing link.

https://admin.fedoraproject.org/accounts/group/view/packager

Can you do this as soon as possible ?

after you join the packager's group then I will get your account sponsored and
your package reviewed :-)

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

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


[Bug 226380] Merge Review: rsync

2009-01-12 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=226380





--- Comment #10 from Simo Sorce sso...@redhat.com  2009-01-12 09:10:14 EDT ---
No special reason, I did a mistake in rsync-3.0.5-0.fc10 so I had to bump up to
re-tag and build :/

-- 
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 450539] Review Request: service-discovery-applet - Service discovery applet based on Avahi for the Gnome panel

2009-01-12 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=450539





--- Comment #9 from Noel J. Bergman n...@devtech.com  2009-01-12 09:34:18 EDT 
---
For what it is worth, I have a replacement for the VNC plugin.  Trying to get
it into upstream, and will post here for your convenience.  It handles three of
the common VNC viewers, so it is more distro and version independent, and
provides an easy means to plugin any other VNC viewer that someone might want.

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

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


[Bug 450539] Review Request: service-discovery-applet - Service discovery applet based on Avahi for the Gnome panel

2009-01-12 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=450539


Noel J. Bergman n...@devtech.com changed:

   What|Removed |Added

External Bug ID||Ubuntu Launchpad 130334




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

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


[Bug 450539] Review Request: service-discovery-applet - Service discovery applet based on Avahi for the Gnome panel

2009-01-12 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=450539





--- Comment #10 from Noel J. Bergman n...@devtech.com  2009-01-12 09:37:12 
EDT ---
Created an attachment (id=328742)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=328742)
Replacement VNC plugin for Service Discovery Applet

This replaces vncviewer.py.in.  It autodetects which vnc viewer is available,
and handles the difference in command line parameters including the port /
screen difference between XVNC and Vinagre.

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

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


[Bug 451298] Review Request: pidgin-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-12 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=451298





--- Comment #10 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-12 
09:46:29 EDT ---
(In reply to comment #6)
 I can't change my password:
 
 
 500 Internal error
 
 The server encountered an unexpected condition which prevented it from
 fulfilling the request.
 
 
 from https://admin.fedoraproject.org/accounts/user/setnewpass/


your password has been mailed to you.


https://fedorahosted.org/fas/ticket/84

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

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


[Bug 475000] Review Request: cmconvert - CacheMate import file converter

2009-01-12 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=475000





--- Comment #9 from Patrick C. F. Ernzer p...@redhat.com  2009-01-12 10:05:49 
EDT ---
(In reply to comment #7)
 Any progress with the seeking of a sponsor?

I have been sponsored through another package but am drowning in work things
(the usual heap after being 2 weeks on vacation), I hope to get this (and
jpilot-backup) pushed in the next 2 weeks.

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

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


[Bug 473348] Review Request: drraw - Web based presentation front-end for RRDtool

2009-01-12 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=473348





--- Comment #2 from Xavier Bachelot xav...@bachelot.org  2009-01-12 10:04:38 
EDT ---
(In reply to comment #1)
 This is not an official review.
 
 1.  When applying patch #2, patch returns:
 
 patch unexpectedly ends in middle of line
 
I've removed this patch and replaced it with a patch that update to latest
trunk. There's only a handful of changes since the last beta.

 While this doesn't abort the build, the patch file should be fixed to not
 generate this error.
 
 2.  This would fail to run with SELinux activated. You should fix the context
 of the CGI script, per http://fedoraproject.org/wiki/PackagingDrafts/SELinux. 
 Creating a drraw-selinux subpackage would probably be ideal in this case.
 
I'm not SELinux proficient, but I'll try to take care of that. Thanks for the
pointer. In your opinion, does such a requirement block the review or can we
save that for later ?

 3.  Patch #2 appears to be a bug fix.  If you have filed a bug with the
 upstream source, you should reference this in your spec file; if not, you
 should file one and reference it.  See
 http://fedoraproject.org/wiki/Packaging/Guidelines#All_patches_should_have_an_upstream_bug_link_or_comment

There's no bug tracker for this project. This was reported on their mailing
list by a colleague of mine, as well as some patch for new features. Actually,
even the svn repository is not public, but after a few mails with upstream, I
now have read-only access.

I'll post new spec and srpms when the selinux issue is addressed. Thanks for
the pre-review.

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

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


[Bug 225700] Merge Review: docbook-dtds

2009-01-12 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=225700


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

 CC||mmarc...@redhat.com
 AssignedTo|nob...@fedoraproject.org|mmarc...@redhat.com
   Flag||fedora-review?




-- 
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 225703] Merge Review: docbook-style-dsssl

2009-01-12 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=225703


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

 CC||mmarc...@redhat.com
 AssignedTo|nob...@fedoraproject.org|mmarc...@redhat.com
   Flag||fedora-review?




-- 
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 225700] Merge Review: docbook-dtds

2009-01-12 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=225700


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

 AssignedTo|mmarc...@redhat.com |nob...@fedoraproject.org
   Flag|fedora-review?  |




-- 
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 225703] Merge Review: docbook-style-dsssl

2009-01-12 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=225703


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

 AssignedTo|mmarc...@redhat.com |nob...@fedoraproject.org
   Flag|fedora-review?  |




-- 
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 470250] Review Request: docbook5-style-xsl - Norman Walsh's XSL stylesheets for DocBook 5

2009-01-12 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=470250





--- Comment #5 from Michal Marciniszyn mmarc...@redhat.com  2009-01-12 
10:42:58 EDT ---
Spec file uses mixed spaces and tabs, please fix this issue

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

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


[Bug 470250] Review Request: docbook5-style-xsl - Norman Walsh's XSL stylesheets for DocBook 5

2009-01-12 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=470250


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 471243] Review Request: docbook5-schemas - Norman Walsh's schemas (DTD, Relax NG, W3C schema) for Docbook 5.X

2009-01-12 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=471243


Michal Marciniszyn mmarc...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   Flag||fedora-review+




--- Comment #8 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
10:56:47 EDT ---
Package Review
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines.
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: devel/x86_64
 [x] Rpmlint output:
source RPM: empty
binary RPM:
fsniper.x86_64: W: non-standard-uid /var/lib/fsniper fsniper
fsniper.x86_64: W: non-standard-gid /var/lib/fsniper fsniper
1 packages and 0 specfiles checked; 0 errors, 2 warnings.
= acceptable, the daemon needs it's own user
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPLv3+
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package match the upstream source, as provided
in the spec URL.
 SHA1SUM of package: 7f4587d83e8c1bbfb142c645f5d40a4bee50bdb7
fsniper-1.3.1.tar.gz
 [x] Package is not known to require ExcludeArch.
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf or $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.
 [x] Final provides and requires are sane.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: devel/x86_64
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: koji scratch build
(http://koji.fedoraproject.org/koji/taskinfo?taskID=1047004)
 [x] Package functions as described.
 [x] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [x] File based requires are sane.
 [x] %check is present and the test passes.


=== Issues ===
1. Nice to have: upstart style initscript



*** APPROVED ***


David, thank you for pre-review.

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

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


[Bug 474603] Review Request: irrlicht - A high performance realtime 3D engine

2009-01-12 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=474603


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #10 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
11:18:31 EDT ---
New Package CVS Request
===
Package Name: irrlicht
Short Description: A high performance real-time 3D engine
Owners: spot
Branches: F-9 F-10 devel
InitialCC: 

... and it's done.

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

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


[Bug 454020] Review Request: tcl-zlib - Tcl extension for zlib support

2009-01-12 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=454020


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #4 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
11:18:38 EDT ---
New Package CVS Request
===
Package Name: tcl-zlib
Short Description: Tcl extension for zlib support
Owners: spot
Branches: F-9 F-10 devel
InitialCC: 

... and it's done.

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

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


[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation

2009-01-12 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=474606


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #4 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
11:18:27 EDT ---
New Package CVS Request
===
Package Name: SolarModel
Short Description: Real-time 3D Solar System simulation
Owners: spot
Branches: F-9 F-10 devel
InitialCC: 

... and it's done.

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

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


[Bug 471243] Review Request: docbook5-schemas - Norman Walsh's schemas (DTD, Relax NG, W3C schema) for Docbook 5.X

2009-01-12 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=471243





--- Comment #2 from Michal Marciniszyn mmarc...@redhat.com  2009-01-12 
11:25:41 EDT ---
Does not rebuild in mock, please check buildrequires and add all that are
needed.

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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627





--- Comment #9 from Jakub Hrozek jhro...@redhat.com  2009-01-12 11:30:26 EDT 
---
David, Manuel, thank you both for the review.

Manuel, is there any documentation on the upstart-style initscripts? A quick
search of fedora wiki for the word upstart didn't reveal much..

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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627


Jakub Hrozek jhro...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #10 from Jakub Hrozek jhro...@redhat.com  2009-01-12 11:34:18 EDT 
---
New Package CVS Request
===
Package Name: fsniper 
Short Description: A tool that monitors directories for new files and invokes
scripts on them
Owners: jhrozek
Branches: F-10 F-9
InitialCC: none

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

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


[Bug 477190] Review Request: cas - core analysis system

2009-01-12 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=477190





--- Comment #10 from Adam Stokes asto...@redhat.com  2009-01-12 11:37:12 EDT 
---
Ok thanks to spot I am now sponsored.

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

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


[Bug 479711] New: Review Request: perl-Time-Duration-Parse - Parse string that represents time duration

2009-01-12 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-Time-Duration-Parse - Parse string that 
represents time duration

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

   Summary: Review Request: perl-Time-Duration-Parse - Parse
string that represents time duration
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/Time-Duration-Parse
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cw...@alumni.drew.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~cweyl/review/perl-Time-Duration-Parse.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-Time-Duration-Parse-0.06-1.fc10.src.rpm

Description:
Time::Duration::Parse is a module to parse human readable duration strings
like _2 minutes and 3 seconds_ to seconds.

It does the opposite of duration_exact() in Time::Duration and is roundtrip
safe. So, the following is always true.

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

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

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


[Bug 467627] Review Request: fsniper - A tool that monitors directories for new files and invokes scripts on them

2009-01-12 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=467627





--- Comment #11 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-12 
11:42:42 EDT ---
https://fedoraproject.org/wiki/Packaging/SysVInitScript

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

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


[Bug 471243] Review Request: docbook5-schemas - Norman Walsh's schemas (DTD, Relax NG, W3C schema) for Docbook 5.X

2009-01-12 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=471243





--- Comment #3 from Ondrej Vasik ova...@redhat.com  2009-01-12 11:47:38 EDT 
---
Thanks... BuildRequires: libxml2 = 2.4.8 was missing, as xmlcatalog used in
build section as well. Updated files provided to reviewer via nfs share...

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

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


[Bug 454022] Review Request: coccinella - Chat client with whiteboard

2009-01-12 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=454022


Bug 454022 depends on bug 454020, which changed state.

Bug 454020 Summary: Review Request: tcl-zlib - Tcl extension for zlib support
https://bugzilla.redhat.com/show_bug.cgi?id=454020

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 454020] Review Request: tcl-zlib - Tcl extension for zlib support

2009-01-12 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=454020


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #5 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
11:50:08 EDT ---
Built for F-9, F-10, devel. Thanks for your patience on this one.

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

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


[Bug 471243] Review Request: docbook5-schemas - Norman Walsh's schemas (DTD, Relax NG, W3C schema) for Docbook 5.X

2009-01-12 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=471243


Gianluca Sforna gia...@gmail.com changed:

   What|Removed |Added

 CC||gia...@gmail.com




--- Comment #4 from Gianluca Sforna gia...@gmail.com  2009-01-12 12:03:03 EDT 
---

(In reply to comment #3)
 Updated files provided to reviewer via nfs share...

Not that there is anything wrong with this, but maybe you're just not aware of
the very handy storage service available to all Fedora contributors:
http://fedoraproject.org/wiki/Infrastructure/fedorapeople.org

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

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


[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

2009-01-12 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=478504





--- Comment #15 from Christoph Wickert fed...@christoph-wickert.de  
2009-01-12 12:16:33 EDT ---
(In reply to comment #13)
 - As when epiphany is upgrade from 2.22 to 2.23, then
   I guess gget-epiphany-extension will no longer work (although
   I don't know this package well) unless gget is rebuilt
   against new epiphany.

Correct. It will no longer work because epiphany won't find it, because it's
looking in a different location.

   i.e. if epiphany can be upgraded without gget-epiphany-extension
is rebuilt, it is _already_ wrong. Not-rebuilt gget-epiphany-extension
should prevent epiphany from being upgraded in such a case
(theoretically).

I disagree. It's better to temporarily loose a certain functionality than
prevent epiphany from being updated. People who are using rawhide are used to
thinks breaking from time to time, but still they are running rawhide because
they want to follow the latest development.
What if the epiphany update is part of a larger Gnome update? The older
epiphany might no longer work with the updated Gnome stuff and then we make the
whole app useless instead of a single extension.

   Some idea:
   - Add Conflicts: epiphany = 2.23
 Conflicts: epiphany  2.22

Congratulations, you have just made it conflict with _every_ epiphany release!
:)

   - Ask epiphany maintainer to support Provides: epiphany(abi) = 2.22,
 for example.

And them make the extension Requires: epiphany(abi) = 2.22? What's the
advantage over Requires: epiphany = 2.22?


(In reply to comment #14)
 Also, as some packages already use %_libdir/epiphany//extensions,
 making every package use this directory own this directory cannot be
 accepted and the owner of this directory must be unified.

Why? We have a couple of these situations whenever one package does not
necessarily depend on the other. Duplicate ownership of a dir is bad, but
unowned dirs are even worse.

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

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


[Bug 474603] Review Request: irrlicht - A high performance realtime 3D engine

2009-01-12 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=474603


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||RAWHIDE




--- Comment #11 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
13:00:06 EDT ---
Built for F-9, F-10, devel.

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

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


[Bug 474606] Review Request: SolarModel - Realtime 3D Solar System simulation

2009-01-12 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=474606


Bug 474606 depends on bug 474603, which changed state.

Bug 474603 Summary: Review Request: irrlicht - A high performance realtime 3D 
engine
https://bugzilla.redhat.com/show_bug.cgi?id=474603

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||RAWHIDE



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

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


[Bug 479723] New: Review Request: cgit - A fast webinterface for git

2009-01-12 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: cgit - A fast webinterface for git

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

   Summary: Review Request: cgit - A fast webinterface for git
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: t...@pobox.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://tmz.fedorapeople.org/reviews/pending/cgit/cgit.spec
SRPM URL:
http://tmz.fedorapeople.org/reviews/pending/cgit/cgit-0.8.1-1.fc10.src.rpm
Description: This is an attempt to create a fast web interface for the git scm,
using a builtin cache to decrease server io-pressure.

In addition to the spec and srpm above, the output of a mock build for F-10
i386 is at: http://tmz.fedorapeople.org/reviews/pending/cgit

Potential concerns are that cgit currently requires some git sources to build. 
Eventually, this will be fixed when libgit2 is ready.  Near term, Lars is
working upstream to include the needed git files in the cgit tarballs.

Another minor issue is package cleanup.  The package owns the cache dir,
/var/cache/cgit, but it won't know about the files created within that dir. 
Would a %preun script to delete these files be appropriate?

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

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


[Bug 477732] Review Request: xfconf - Hierarchical configuration system for Xfce

2009-01-12 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=477732





--- Comment #11 from Kevin Fenzi ke...@tummy.com  2009-01-12 13:10:21 EDT ---
Yes? I was at fudcon this weekend, and intend to import and build this sometime
soon this week time permitting. 

I need to first build the new libxfce4util package. 
Sorry for any delay.

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

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


[Bug 479596] Review Request: serafettin-fonts - Comic Sans fonts

2009-01-12 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=479596





--- Comment #3 from Tom spot Callaway tcall...@redhat.com  2009-01-12 
13:12:18 EDT ---
Licensing history looks clean, I will assume it is unless evidence to the
contrary appears.

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

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


[Bug 478504] Review Request: gget - Download Manager for the GNOME desktop.

2009-01-12 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=478504





--- Comment #16 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-12 
13:27:00 EDT ---
(In reply to comment #15)
 (In reply to comment #13)
i.e. if epiphany can be upgraded without gget-epiphany-extension
 is rebuilt, it is _already_ wrong. Not-rebuilt 
  gget-epiphany-extension
 should prevent epiphany from being upgraded in such a case
 (theoretically).
 
 I disagree. It's better to temporarily loose a certain functionality than
 prevent epiphany from being updated. People who are using rawhide are used to
 thinks breaking from time to time, but still they are running rawhide because
 they want to follow the latest development.
 What if the epiphany update is part of a larger Gnome update? The older
 epiphany might no longer work with the updated Gnome stuff and then we make 
 the
 whole app useless instead of a single extension.

A larger Gnome update in your sense had been already disturbed before
many times (largely because some new gnome components won't build against
new gnome libraries, gnome-desktop for example...).
Other packages failing to rebuild have also prevented major packages in 
your sense from being upgraded. I can see no gain to loose dependency
on this package for the reason you raised (and for this package
you can simply remove this, while for gnome-desktop (for example)
we actually have to wait until (almost) all package are rebuilt)


Some idea:
- Add Conflicts: epiphany = 2.23
  Conflicts: epiphany  2.22
 
 Congratulations, you have just made it conflict with _every_ epiphany release!
 :)

First of all:
- Please don't use congratulations :) or ! carelessly, please.
  While I don't know how commonly these words or emoticons are used
  in discussions like this bug report in your country, 
  these words can be taken very differently than what you expect
  in other countries.

Then:
if epiphany has 2.22{,X} version, the epiphany won't conflict
with these two.
Note that this method is sometimes used when
- A srpm creates 2 (or more) subpackages
- The two packages don't depend on each other, and can be installed
  seperately or together
- However if the versions of the two package differ, something
  nasty could occur

- Ask epiphany maintainer to support Provides: epiphany(abi) = 2.22,
  for example.
 
 And them make the extension Requires: epiphany(abi) = 2.22? What's the
 advantage over Requires: epiphany = 2.22?

It won't work for ephiphany minor release bump (i.e. 2.22.3, for example)
For example ruby has Provides: ruby(abi) = 1.8 while the current
ruby version (on Fedora) is 1.8.6(.287)
(and actually I think if epiphany changes the extension every time
 its major/middle version changes, epiphany should provide such
 information to rpm, like ruby, python or so actually do)


 (In reply to comment #14)
  Also, as some packages already use %_libdir/epiphany//extensions,
  making every package use this directory own this directory cannot be
  accepted and the owner of this directory must be unified.
 
 Why? We have a couple of these situations whenever one package does not
 necessarily depend on the other. Duplicate ownership of a dir is bad, but
 unowned dirs are even worse.

But they all depend on epiphany.

So I am asking you to ask epiphany maintainer first (as I did to 
vim maintainer). It is not desired that no one using that
directory tries to argue with epiphany maintainer and gets satisfied with
his/her local hack.
KDE has kde-filesystem, font packages got to use fontpackages-filesystem
and so on. Please contact with epiphany maintainer first.

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

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


[Bug 450539] Review Request: service-discovery-applet - Service discovery applet based on Avahi for the Gnome panel

2009-01-12 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=450539


Tim Niemueller t...@niemueller.de changed:

   What|Removed |Added

   Flag|needinfo?(t...@niemueller.de |
   |)   |




--- Comment #11 from Tim Niemueller t...@niemueller.de  2009-01-12 13:34:18 
EDT ---
I'm currently short in time. I still think the service-discovery-applet should
be in Fedora. My time constraints (thesis) will not allow me to do this before
March though. Either we wait or someone else takes over.

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

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


[Bug 451298] Review Request: pidgin-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-12 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=451298


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

   What|Removed |Added

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




--- Comment #11 from Jussi Lehtola jussi.leht...@iki.fi  2009-01-12 13:32:08 
EDT ---
I see you have updated the SPEC  SRPM.

This plugin should be named purple-msn-pecan instead of pidgin-msn-pecan, since
it's a plugin for libpurple (not pidgin); also other existing libpurple plugins
have been named in a similar fashion:

purple-facebookchat.x86_64 : Libpurple plug-in supporting facebook IM
purple-microblog.x86_64 : Libpurple plug-in supporting microblog services like
Twitter

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

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


[Bug 473723] Review Request: perl-Eval-Context - Evaluate perl code in context wrapper

2009-01-12 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=473723





--- Comment #8 from Chris Weyl cw...@alumni.drew.edu  2009-01-12 13:44:10 EDT 
---
That's not picky at all :)  The failing test is one using Test::Perl::Critic --
it tests _how_ you code, not if the code works or not, so it's OK to exclude
along the lines of Test::Kwalitee, Test::Pod::Coverage, etc.

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

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


[Bug 479730] New: Review Request: perl-DateTimeX-Easy - Parse a date/time string using the best method available

2009-01-12 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-DateTimeX-Easy - Parse a date/time string using 
the best method available
Alias: perl-DateTimeX-Easy

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

   Summary: Review Request: perl-DateTimeX-Easy - Parse a
date/time string using the best method available
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/DateTimeX-Easy
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cw...@alumni.drew.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~cweyl/review/perl-DateTimeX-Easy.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-DateTimeX-Easy-0.085-1.fc10.src.rpm

Description:
DateTimeX::Easy makes DateTime object creation quick and easy. It uses a
variety of DateTime::Format packages to do the bulk of the parsing, with
some custom tweaks to smooth out the rough edges (mainly concerning
timezone detection and selection).

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

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

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


[Bug 474149] Review Request: chipmunk - A rigid body physics library

2009-01-12 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=474149


Jon Ciesla l...@jcomserv.net changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #18 from Jon Ciesla l...@jcomserv.net  2009-01-12 13:57:48 EDT ---
I got it to build, but it's the ugly mv in %install method.  Bleah.

Imported and build in rawhide.

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

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


[Bug 474149] Review Request: chipmunk - A rigid body physics library

2009-01-12 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=474149





--- Comment #19 from Fedora Update System upda...@fedoraproject.org  
2009-01-12 14:05:14 EDT ---
chipmunk-4.1.0-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/chipmunk-4.1.0-4.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 479732] New: Review Request: perl-MooseX-Role-Parameterized - parameterized roles

2009-01-12 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-MooseX-Role-Parameterized - parameterized roles

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

   Summary: Review Request: perl-MooseX-Role-Parameterized -
parameterized roles
   Product: Fedora
   Version: rawhide
  Platform: All
   URL: http://search.cpan.org/dist/MooseX-Role-Parameterized
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: cw...@alumni.drew.edu
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://fedorapeople.org/~cweyl/review/perl-MooseX-Role-Parameterized.spec
SRPM URL:
http://fedorapeople.org/~cweyl/review/perl-MooseX-Role-Parameterized-0.02-1.fc10.src.rpm

Description:
Roles are composable units of behavior. They are useful for factoring out
functionality common to many classes from any part of your class hierarchy.
(See Moose::Cookbook::Roles::Recipe1 for an introduction to Moose::Role.)

While combining roles affords you a great deal of flexibility, individual
roles have very little in the way of configurability. Core Moose provides
alias for renaming methods to avoid conflicts, and excludes for ignoring
methods you don't want or need (see Moose::Cookbook::Roles::Recipe2 for more
about alias and excludes).

Because roles serve many different masters, they usually provide only the
least common denominator of functionality. To empower roles further, more
configurability than alias and excludes is required. Perhaps your role needs
to know which method to call when it is done. Or what default value to use for
its url attribute.

Parameterized roles offer exactly this solution.

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

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

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


[Bug 479723] Review Request: cgit - A fast webinterface for git

2009-01-12 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=479723





--- Comment #2 from Todd Zullinger t...@pobox.com  2009-01-12 14:16:18 EDT ---
httpd is required for ownership of /etc/httpd/conf.d.  If that can be made more
generic, I'd be all for it.

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

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


[Bug 478941] Review Request: celt - An audio codec for use in low-delay speech and audio communication

2009-01-12 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=478941





--- Comment #2 from Richard W.M. Jones rjo...@redhat.com  2009-01-12 14:11:33 
EDT ---
- rpmlint output

celt-devel.x86_64: W: no-documentation

Just move (or duplicate) the README and TODO files into the
-devel subpackage.

+ package name satisfies the packaging naming guidelines
+ specfile name matches the package base name
+ package should satisfy packaging guidelines
+ license meets guidelines and is acceptable to Fedora
? license matches the actual package license

There's a number of source files (both *.c and *.h) that
don't contain explicit license information.  Are we sure
about the origin of these files?

+ %doc includes license file
+ spec file written in American English
+ spec file is legible
+ upstream sources match sources in the srpm
  dd7cda953e4df448083b69af9afc15e1 401008
+ package successfully builds on at least one architecture
  x86_64
n/a ExcludeArch bugs filed
+ BuildRequires list all build dependencies
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1047518
n/a %find_lang instead of %{_datadir}/locale/*
+ binary RPM with shared library files must call ldconfig in %post and %postun
+ does not use Prefix: /usr
+ package owns all directories it creates
+ no duplicate files in %files
+ %defattr line
+ %clean contains rm -rf $RPM_BUILD_ROOT
+ consistent use of macros
+ package must contain code or permissible content
n/a large documentation files should go in -doc subpackage
+ files marked %doc should not affect package
+ header files should be in -devel
n/a static libraries should be in -static
+ packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
+ libfoo.so must go in -devel
+ -devel must require the fully versioned base
+ packages should not contain libtool .la files
n/a packages containing GUI apps must include %{name}.desktop file
+ packages must not own files or directories owned by other packages
+ %install must start with rm -rf %{buildroot} etc.
+ filenames must be valid UTF-8

Optional:

? if there is no license file, packager should query upstream
n/a translations of description and summary for non-English languages, if
available
+ reviewer should build the package in mock
  http://koji.fedoraproject.org/koji/taskinfo?taskID=1047518
+ the package should build into binary RPMs on all supported architectures
- review should test the package functions as described
+ scriptlets should be sane
+ pkgconfig files should go in -devel
+ shouldn't have file dependencies outside /etc /bin /sbin /usr/bin or
/usr/sbin

--

So there's that %doc thing to fix, and can you query upstream
about the libcelt/*.[ch] files which don't have license information
to find out who wrote them and/or where they came from.

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

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


[Bug 479723] Review Request: cgit - A fast webinterface for git

2009-01-12 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=479723


seth vidal svi...@redhat.com changed:

   What|Removed |Added

 CC||svi...@redhat.com




--- Comment #1 from seth vidal svi...@redhat.com  2009-01-12 14:11:42 EDT ---
rpmlint checks out
package passes all MUST  requirements from review checks

Does it REALLY require httpd - or does it only require 'webserver'?

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

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


[Bug 479055] Review Request: perl-Class-Throwable - A minimal lightweight exception class

2009-01-12 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=479055


Gabriel Somlo so...@cmu.edu changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Gabriel Somlo so...@cmu.edu  2009-01-12 14:14:02 EDT ---
successfully built and submitted for stable (forgot to add -b 479055 to bodhi)

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

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


[Bug 479056] Review Request: perl-Catalyst-Plugin-Authorization-ACL - ACL Support for Catalyst Applications

2009-01-12 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=479056


Bug 479056 depends on bug 479055, which changed state.

Bug 479055 Summary: Review Request: perl-Class-Throwable - A minimal 
lightweight exception class
https://bugzilla.redhat.com/show_bug.cgi?id=479055

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 478942] Review Request: perl-Catalyst-Authentication-Store-DBIx-Class - A storage class for Catalyst Authentication

2009-01-12 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=478942





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-01-12 14:33:58 EDT ---
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc9 has been submitted
as an update for Fedora 9.
http://admin.fedoraproject.org/updates/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.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 478942] Review Request: perl-Catalyst-Authentication-Store-DBIx-Class - A storage class for Catalyst Authentication

2009-01-12 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=478942





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-01-12 14:34:07 EDT ---
perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.fc10 has been submitted
as an update for Fedora 10.
http://admin.fedoraproject.org/updates/perl-Catalyst-Authentication-Store-DBIx-Class-0.1082-2.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 479723] Review Request: cgit - A fast webinterface for git

2009-01-12 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=479723


seth vidal svi...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|svi...@redhat.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 451298] Review Request: pidgin-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-12 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=451298





--- Comment #12 from Edouard Bourguignon edouard.bourguig...@oxalya.com  
2009-01-12 14:41:46 EDT ---
Ok I will change that. thanks 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 479585] Review Request: megaupload-dl - Megaupload automatic downloader

2009-01-12 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=479585





--- Comment #14 from Marek Mahut mma...@redhat.com  2009-01-12 14:51:11 EDT 
---
Created an attachment (id=328778)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=328778)
mail from legal

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

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


[Bug 474149] Review Request: chipmunk - A rigid body physics library

2009-01-12 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=474149





--- Comment #20 from Conrad Meyer kon...@tylerc.org  2009-01-12 14:48:28 EDT 
---
Any chance you want to work with the maintainer of xmoto to get xmoto 0.5 to
use the system chipmunk?

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

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


[Bug 479723] Review Request: cgit - A fast webinterface for git

2009-01-12 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=479723





--- Comment #3 from Jeremy Katz ka...@redhat.com  2009-01-12 14:50:54 EDT ---
The other comment I have from a quick look is that the %define at the top for
the make really probably should just be explicitly in the %build section.

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

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


[Bug 479585] Review Request: megaupload-dl - Megaupload automatic downloader

2009-01-12 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=479585





--- Comment #13 from Marek Mahut mma...@redhat.com  2009-01-12 14:49:28 EDT 
---
Jochen,

Cleared issue with legal team, we are good to go, I'll attach mail from legal
later.

http://mmahut.fedorapeople.org/reviews/megaupload-dl/megaupload-dl-0.2-2.fc8.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 474149] Review Request: chipmunk - A rigid body physics library

2009-01-12 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=474149





--- Comment #21 from Conrad Meyer kon...@tylerc.org  2009-01-12 14:52:29 EDT 
---
Just kidding, it appears you are the xmoto maintainer. Good job :D.

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

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


[Bug 451298] Review Request: pidgin-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-12 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=451298





--- Comment #13 from Edouard Bourguignon edouard.bourguig...@oxalya.com  
2009-01-12 14:52:46 EDT ---
So here are the new files:
http://www.linuxed.net/~madko/fedora/purple-msn-pecan.spec
http://www.linuxed.net/~madko/fedora/purple-msn-pecan-0.0.17-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


  1   2   >