[Bug 528473] Review Request: php-ezc-DatabaseSchema - eZ Components DatabaseSchema

2009-11-16 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=528473





--- Comment #5 from Guillaume Kulakowski llaum...@gmail.com  2009-11-16 
04:06:19 EDT ---
New Package CVS Request
===
Package Name: php-ezc-DatabaseSchema
Short Description: A set of classes that allow you to use information from a
database schema
Owners: llaumgui
Branches: F-10 F-11 F-12
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 528473] Review Request: php-ezc-DatabaseSchema - eZ Components DatabaseSchema

2009-11-16 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=528473


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

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 528476] Review Request: php-ezc-PersistentObjectDatabaseSchemaTiein - eZ Components PersistentObjectDatabaseSchemaTiein

2009-11-16 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=528476


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

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from Guillaume Kulakowski llaum...@gmail.com  2009-11-16 
04:09:24 EDT ---
New Package CVS Request
===
Package Name: php-ezc-PersistentObjectDatabaseSchemaTiein
Short Description: Automatic generation of PersistentObject from DatabaseSchema
definitions
Owners: llaumgui
Branches: F-10 F-11 F-12
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 529542] Review Request: php-phpunit-phpcpd - Copy/Paste Detector (CPD) for PHP code

2009-11-16 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=529542





--- Comment #4 from Guillaume Kulakowski llaum...@gmail.com  2009-11-16 
04:16:02 EDT ---
@Christof Damian  You approve ? If you approve, please add a line like :

---
This package (php-phpunit-phpcpd) is APPROVED by Christof Damian
---

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

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


[Bug 529544] Review Request: php-phpunit-bytekit - A command-line tool built on the PHP Bytekit extension

2009-11-16 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=529544


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

   What|Removed |Added

   Flag|needinfo?(llaum...@gmail.co |
   |m)  |




--- Comment #2 from Guillaume Kulakowski llaum...@gmail.com  2009-11-16 
04:16:53 EDT ---
OK, I will update my SPEC file.

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

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


[Bug 507083] Review Request: poco - C++ class libraries for network-centric applications

2009-11-16 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=507083





--- Comment #26 from Maxim Udushlivy udushl...@mail.ru  2009-11-16 04:38:54 
EDT ---
SPEC URL: http://crow-designer.sourceforge.net/temp/7/poco.spec
SRPM URL: http://crow-designer.sourceforge.net/temp/7/poco-1.3.5-7.fc11.src.rpm

CHANGES
===
- A removal of the Foundation/src/MSG1.bin binary file was added 
to the %%prep section.
- Values for the top Summary, Group and %%description were 
restored.
- A BuildRoot tag was added.

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

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


[Bug 537631] Review Request: moblin-panel-web - Moblin Panel for Web Browsing

2009-11-16 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=537631


Peter Robinson pbrobin...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898





--- Comment #16 from David Sommerseth dav...@redhat.com  2009-11-16 04:44:17 
EDT ---
I'm ready for some work on this package now, after some über hectic weeks. 
Going to read through a lot of the documentation about sponsorship again, to
refresh what I need to do now.

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

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


[Bug 529542] Review Request: php-phpunit-phpcpd - Copy/Paste Detector (CPD) for PHP code

2009-11-16 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=529542





--- Comment #5 from Christof Damian chris...@damian.net  2009-11-16 05:01:58 
EDT ---
This package (php-phpunit-phpcpd) is APPROVED by Christof Damian

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

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


[Bug 529542] Review Request: php-phpunit-phpcpd - Copy/Paste Detector (CPD) for PHP code

2009-11-16 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=529542


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

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Guillaume Kulakowski llaum...@gmail.com  2009-11-16 
05:22:40 EDT ---
New Package CVS Request
===
Package Name: php-phpunit-phpcpd
Short Description: Copy/Paste Detector (CPD) for PHP code
Owners: llaumgui cdamian
Branches: F-10 F-11 F-12
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 526444] Review Request: MyGUI - Fast, simple and flexible GUI

2009-11-16 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=526444





--- Comment #15 from Thomas Janssen thom...@fedoraproject.org  2009-11-16 
05:54:22 EDT ---
[fix]
Release should be: 0.2.2332svn%{?dist}
So you get a 3.0.0-0.2.2332svn package

http://fedoraproject.org/wiki/Packaging:NamingGuidelines#Package_Release

[fix]
You use a mix of $RPM_BUILD_ROOT and %{buildroot}

[fix]
You use %cmake but no macro for the rest of the commands like install, sed. 

[fix]
No need for VERBOSE=1 in %build

I rebuilt it here and it was running after i installed mygui and mygui-tools (i
tried the LayoutEditor).
Does it make sense to have mygui installed alone? Can one do anything with it,
without the tools?
If not, make the tools installed with the mainpackage, or maybe merge the tool
package into the main package.

As well it couldn't load anything from the list (within the load dialog). Maybe
just me.

Are the code template files needed outside a develpackage? Does the program
work without them installed? You could move them into the devel package and
make that installed by default as well.

[fix]
rpmlint -I dangling-symlink
dangling-symlink:
The target of the symbolic link does not exist within this package or its file
based dependencies.  Verify spelling of the link target and that the target is
included in a package in this package's dependency chain.

[fix]
rpmlint from installed packages:

mygui.x86_64: W: undefined-non-weak-symbol /usr/lib64/libMyGUIEngine.so.3.0.0
dlsym
mygui.x86_64: W: undefined-non-weak-symbol /usr/lib64/libMyGUIEngine.so.3.0.0
uuid_generate
mygui.x86_64: W: undefined-non-weak-symbol /usr/lib64/libMyGUIEngine.so.3.0.0
dlopen
mygui.x86_64: W: undefined-non-weak-symbol /usr/lib64/libMyGUIEngine.so.3.0.0
dlcloserpmlint -I undefined-non-weak-symbol

undefined-non-weak-symbol:
The binary contains undefined non-weak symbols.  This may indicate improper
linkage; check that the binary has been linked as expected.

You might check that linking problem with upstream.

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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898


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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #17 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
07:24:04 EDT ---
Now I am sponsoring you. Please follow Join wiki again.

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

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


[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841





--- Comment #19 from Jon Ciesla l...@jcomserv.net  2009-11-16 08:30:25 EDT ---
I never got an answer to #17.  Are you back?

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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898


David Sommerseth dav...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898





--- Comment #18 from David Sommerseth dav...@redhat.com  2009-11-16 08:36:38 
EDT ---
New Package CVS Request
===
Package Name: imapfilter
Short Description: A flexible client side mail filtering utility for IMAP
servers
Owners: dsommers
Branches: F11 F21
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 537084] Review Request: libmpc - C library for multiple precision complex arithmetic

2009-11-16 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=537084


Petr Machata pmach...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Comment #12 from Petr Machata pmach...@redhat.com  2009-11-16 08:38:40 
EDT ---
Built and requested an update for F11 and F12.
Thanks for cooperation!

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

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


[Bug 537823] New: Review Request: hunspell-ht - Haitian Creole hunspell dictionaries

2009-11-16 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-ht - Haitian Creole hunspell dictionaries

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

   Summary: Review Request: hunspell-ht - Haitian Creole hunspell
dictionaries
   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/hunspell-ht.spec
SRPM URL: http://people.redhat.com/caolanm/rpms/hunspell-ht-0.02-1.fc11.src.rpm
Description: Haitian Creole hunspell dictionaries

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

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


[Bug 537823] Review Request: hunspell-ht - Haitian Creole hunspell dictionaries

2009-11-16 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=537823


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




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

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


[Bug 537727] Review Request: Slack - A lazy systems administration configuration management toolkit.

2009-11-16 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=537727


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

   What|Removed |Added

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




--- Comment #2 from Jussi Lehtola jussi.leht...@iki.fi  2009-11-16 08:49:30 
EDT ---
Just a few notes:

- the name of the patch should be something of the lines of
 slack-0.15.2-fileplacement.patch
The most important thing is that it begins with slack- so that if one builds
the RPM with rpmbuild the source files of different packages are easily
separated.

- You shouldn't need to define %defattr for the normal files, just use
 %defattr(-,root,root,-)
For the privileged items you can use the special permissions, although I'd
change
 %defattr(0755,root,root,0700)
to
 %defattr(0700,root,root,0700)
to keep things logical..

- I'm wondering whether the use of
 slack_libdir=%{buildroot}/%{_datadir}/%{name}/lib
is reasonable. Wouldn't
 slack_libdir=%{buildroot}/%{_libdir}/%{name}
be more logical? Or, in the case of architecture independent files just
 slack_libdir=%{buildroot}/%{_datadir}/%{name}
Also, please note that currently you don't own the directory
 %{_datadir}/%{name}
that is created by the rpm.

- I wouldn't use compression suffixes in the man files, since they can change
in the future, so change e.g.
 %{_mandir}/man1/slack-diff.1.gz
to
 %{_mandir}/man1/slack-diff.1.*

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

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


[Bug 466331] Review Request: rcov - ruby code coverage tool

2009-11-16 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=466331


Casey Dahlin cdah...@redhat.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||WONTFIX




--- Comment #6 from Casey Dahlin cdah...@redhat.com  2009-11-16 09:47:21 EDT 
---
Its been a week. No reply. I'm thinking this project is stillborn. Closing.

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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898





--- Comment #19 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
10:10:21 EDT ---
F21 is perhaps F12.

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

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


[Bug 528108] Review Request: Vuurmuur - Firewall manager built on top of iptables

2009-11-16 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=528108





--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
10:37:09 EDT ---
Well, sounds good, however I want to check what you actually
want to do anyway. 

Note that I think %_sysconfdir/vuurmuur/config.conf should be
marked (included) as %ghost.

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

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


[Bug 530473] Review Request: lessfs - Lessfs is an inline data deduplicating filesystem.

2009-11-16 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=530473





--- Comment #10 from Adam Miller maxamill...@fedoraproject.org  2009-11-16 
10:42:42 EDT ---
New upstream release yesterday.

Spec URL: http://maxamillion.fedorapeople.org/lessfs.spec
SRPM URL: http://maxamillion.fedorapeople.org/lessfs-0.8.3-1.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 453422] Review Request: songbird - Mozilla based multimedia player

2009-11-16 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=453422


Joel j...@cryregarder.com changed:

   What|Removed |Added

 CC||j...@cryregarder.com




--- Comment #87 from Joel j...@cryregarder.com  2009-11-16 11:03:01 EDT ---
Any progress on songbird?  I tried building the latest src.rpm on fc11 but it
doesn't go through (can post if not common knowledge).

I'm really looking forward to this package in fedora.

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

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


[Bug 515230] Review Request: python-dmidecode - python extension module to access DMI data

2009-11-16 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=515230


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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #4 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
11:14:34 EDT ---
(Removing NEEDSPONSOR)

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

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


[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841





--- Comment #20 from Stjepan Gros stjepan.g...@gmail.com  2009-11-16 11:38:01 
EDT ---
Actually, I'm here for some time already. But I thought that everything was
settled with this entry, i.e. that you'll take it, so I didn't post anything to
this thread?

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

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


[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841





--- Comment #21 from Jon Ciesla l...@jcomserv.net  2009-11-16 11:45:04 EDT ---
Nope, it's a resounding nothing.  I've had a. . .full. . .personal life since
post #17, so I've not touched it.  If you want to do the practice reviews, we
can pick up where we left off.

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

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


[Bug 528108] Review Request: Vuurmuur - Firewall manager built on top of iptables

2009-11-16 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=528108





--- Comment #10 from Stjepan Gros stjepan.g...@gmail.com  2009-11-16 11:48:24 
EDT ---
Ups, sorry, the right links are:

Spec URL: http://www.zemris.fer.hr/~sgros/stuff/fedora/vuurmuur/Vuurmuur.spec
SRPM URL:
http://www.zemris.fer.hr/~sgros/stuff/fedora/vuurmuur/Vuurmuur-0.7-3.fc11.src.rpm

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

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


[Bug 528108] Review Request: Vuurmuur - Firewall manager built on top of iptables

2009-11-16 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=528108





--- Comment #9 from Stjepan Gros stjepan.g...@gmail.com  2009-11-16 11:47:48 
EDT ---
I just noticed that this check was already included in the init.d script but it
wasn't working because it requires some basic configuration to be present,
which now is.

I didn't specify %ghost for config.conf because it is installed.

Here is the new package:

Spec URL: http://www.zemris.fer.hr/~sgros/stuff/fedora/vuurmuur/Vuurmuur.spec
SRPM URL:
http://www.zemris.fer.hr/~sgros/stuff/fedora/vuurmuur/Vuurmuur-0.7-2.fc11.src.rpm

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

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


[Bug 520637] Review Request: mdk - GNU MIX Development Kit

2009-11-16 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=520637





--- Comment #20 from Stjepan Gros stjepan.g...@gmail.com  2009-11-16 12:16:32 
EDT ---
I already have FAS account, the user name is sgros. And I did install client
side certificate.

What puzzles me now is the part After you request for sponsorship... because
I can not find where exactly to request sponsorship?

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

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


[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841





--- Comment #22 from Stjepan Gros stjepan.g...@gmail.com  2009-11-16 12:17:24 
EDT ---
Ok, I'll repackage ndisc6 (there is a new version) and submit it here and then
we'll see what to do next.

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

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


[Bug 509965] Review Request: snmptt - SNMPTT (SNMP Trap Translator) is an SNMP trap handler written in Perl

2009-11-16 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=509965


Ville Skyttä ville.sky...@iki.fi changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|ville.sky...@iki.fi
   Flag||fedora-review?




--- Comment #12 from Ville Skyttä ville.sky...@iki.fi  2009-11-16 12:36:37 
EDT ---
(In reply to comment #10)
 http://giesen.fedorapeople.org/snmptt/snmptt-1.3-0.1.beta2.fc11.src.rpm  

Version 1.3 was released yesterday it seems.  Anyway here's a partial review
from skimming the above beta2 specfile, will complete the review when the
package has been updated to 1.3:

- Summary isn't very helpful wrt. what the package does.  In my package I used
SNMP Trap Translator which isn't perfect but IMO slightly better the current
one.

- %description doesn't actually describe snmptt but snmptrapd.  In my snmptt
package I had this:
SNMPTT (SNMP Trap Translator) is an SNMP trap handler written in Perl
for use with the Net-SNMP / UCD-SNMP snmptrapd program.  It can be
used to translate trap output from snmptrapd to more descriptive and
human friendly form, supports logging, invoking external programs, and
has the ability to accept or reject traps based on a number of
parameters.

- A number of installed files that contain hardcoded paths are installed using
macros.  This is a non-blocker as far as this review is concerned, however I'd
recommend either using those hardcoded paths in the specfile or implementing
something to replace those hardcoded paths in installed files with the
expansions of macros.

- snmptthandler is installed as %{_sbindir}/snmptthandler,
%{_bindir}/snmpttconvert, and %{_bindir}/snmpttconvertmib which doesn't look
right to me.

- %post and %preun are not guarded for non-zero exit status

See my old package in comment #3, it has fixes/improvements for all of the
above.

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

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


[Bug 509965] Review Request: snmptt - SNMPTT (SNMP Trap Translator) is an SNMP trap handler written in Perl

2009-11-16 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=509965





--- Comment #13 from Ville Skyttä ville.sky...@iki.fi  2009-11-16 12:40:42 
EDT ---
One more thing: I shipped snmptt-net-snmp-test in my snmptt package.  I'm not
sure if I ever used it but it is referred to in upstream docs so I'd just like
to confirm if you omitted it on purpose or if it was an oversight.

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

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


[Bug 508518] Review Request: Meiga - Easy tool for file sharing and content publishing

2009-11-16 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=508518


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:42:27 EDT 
---
CVS done.

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

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


[Bug 481528] Review Request: perl-DBIx-Safe - Safer access to your database through a DBI database handle

2009-11-16 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=481528


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #13 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:40:58 
EDT ---
CVS done.

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

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


[Bug 537585] Review Request: picocontainer - Dependency-injection container

2009-11-16 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=537585


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:44:52 EDT 
---
CVS done.

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

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


[Bug 479527] Review Request: synfigstudio - Vector-based 2D animation studio

2009-11-16 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=479527


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:44:04 EDT 
---
CVS done.

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

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


[Bug 531391] Review Request: gwsmhg - hg/mq GUI

2009-11-16 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=531391


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:48:34 EDT 
---
CVS done.

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

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


[Bug 530288] Review Request: stonevpn - easy OpenVPN certificate and configuration management

2009-11-16 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=530288


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #14 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:45:49 
EDT ---
CVS done.

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

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


[Bug 509936] Review Request: tortoisehg - Mercurial gui tools and nautilus plugin

2009-11-16 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=509936


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #16 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:47:42 
EDT ---
CVS done.

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

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


[Bug 520637] Review Request: mdk - GNU MIX Development Kit

2009-11-16 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=520637


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

   What|Removed |Added

 CC||l...@jcomserv.net




--- Comment #21 from Jon Ciesla l...@jcomserv.net  2009-11-16 12:49:15 EDT ---
You have requested sponsorship, on the ndisc6 review.  Mamoru, ndisc6 is also
ready to approve. If you want to sponsor Stjepan, be my guest.

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

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


[Bug 528473] Review Request: php-ezc-DatabaseSchema - eZ Components DatabaseSchema

2009-11-16 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=528473


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:49:13 EDT 
---
CVS done.

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

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


[Bug 528476] Review Request: php-ezc-PersistentObjectDatabaseSchemaTiein - eZ Components PersistentObjectDatabaseSchemaTiein

2009-11-16 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=528476


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #9 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:50:50 EDT 
---
CVS done.

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

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


[Bug 529542] Review Request: php-phpunit-phpcpd - Copy/Paste Detector (CPD) for PHP code

2009-11-16 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=529542


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:52:34 EDT 
---
CVS done.

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

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


[Bug 533725] Review Request: rubygem-linecache - Caches (Ruby source) files

2009-11-16 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=533725





--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
12:55:01 EDT ---
Created an attachment (id=369756)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=369756)
Patch so that linecache gem actually works

For -2:

* %define - %global
  - Still %define is used in the first line.

* BR
  - BuildRequires: rubygem(rake) is needed for
rake test.

* %{geminstdir}/ext
  - I don't think this directory is needed and should completely
be removed.
! Note
  - To avoid trace_nums.so regeneration in %check, rake test should
be done under %_builddir%geminstdir, not under
%buildroot%geminstdir

* Usability
-
[tasa...@localhost ~]$ irb
irb(main):001:0 require 'rubygems'
= true
irb(main):002:0 require 'linecache'
LoadError: no such file to load --
/usr/lib/ruby/gems/1.8/gems/linecache-0.43/lib/../lib/trace_nums
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in
`gem_original_require'
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in
`require'
from /usr/lib/ruby/gems/1.8/gems/linecache-0.43/lib/tracelines.rb:12
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in
`gem_original_require'
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31:in
`require'
from /usr/lib/ruby/gems/1.8/gems/linecache-0.43/lib/linecache.rb:63
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in
`gem_original_require'
from /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:36:in
`require'
from (irb):2
from /usr/lib/ruby/site_ruby/1.8/rubygems/version.rb:33
-
  - i.e. Currently the installed linecache gem does not work.
This is because we move trace_nums.so from %geminstdir/ext to
%ruby_sitearch.
The attached patch (for example) is needed (on Fedora).

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

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


[Bug 533725] Review Request: rubygem-linecache - Caches (Ruby source) files

2009-11-16 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=533725


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   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 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #20 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 12:57:32 
EDT ---
CVS done (with F-11 and F-12 branches instead of F11 F21).

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

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


[Bug 537897] New: Review Request: mingw32-openjpeg - mingw32 package for openjpeg

2009-11-16 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: mingw32-openjpeg - mingw32 package for openjpeg

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

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


Spec URL: http://www.cs.cmu.edu/~agoode/tmp/mingw32-openjpeg.spec
SRPM URL: http://www.cs.cmu.edu/~agoode/tmp/mingw32-openjpeg-1.3-5.fc13.src.rpm
Description: 
mingw32 package for openjpeg

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

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


[Bug 520637] Review Request: mdk - GNU MIX Development Kit

2009-11-16 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=520637


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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #22 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
13:08:47 EDT ---
(In reply to comment #21)
 Mamoru, ndisc6 is also
 ready to approve. If you want to sponsor Stjepan, be my guest.
Thank you. 

(In reply to comment #20)
 What puzzles me now is the part After you request for sponsorship... because
 I can not find where exactly to request sponsorship?  

As I received the mail (on sponsors-mailing-list), I guess you
did the proper procedure.
Now I am sponsoring you. Please follow Join wiki again.

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

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


[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841


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

   What|Removed |Added

 Blocks|177841(FE-NEEDSPONSOR)  |
   Flag|fedora-review?  |fedora-review+




--- Comment #23 from Jon Ciesla l...@jcomserv.net  2009-11-16 13:14:08 EDT ---
Removed NEEDSPONSOR blocker.  I'll look over the new version when you post it,
and approve it nothing new creeps up.

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

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


[Bug 532309] Review Request: rubygem-ruby-net-ldap - A full-featured pure-Ruby LDAP client

2009-11-16 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=532309


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

   What|Removed |Added

Summary|Review Request: |Review Request:
   |rubygem-net-ldap - A|rubygem-ruby-net-ldap - A
   |full-featured pure-Ruby |full-featured pure-Ruby
   |LDAP client |LDAP client




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

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


[Bug 524379] Review Request: gscribble - A desktop blogging client for GNOME

2009-11-16 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=524379





--- Comment #30 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-11-16 
13:31:14 EDT ---
Okay, now:

-
NOTE: Before being sponsored:

This package will be accepted with another few work. 
But before I accept this package, someone (I am a candidate) 
must sponsor you.

Once you are sponsored, you have the right to review other 
submitters' review requests and approve the packages formally. 
For this reason, the person who want to be sponsored (like you) 
are required to show that you have an understanding 
of the process and of the packaging guidelines as is described
on :
http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Usually there are two ways to show this.
A. submit other review requests with enough quality.
B. Do a pre-review of other person's review request
   (at the time you are not sponsored, you cannot do
   a formal review)

When you have submitted a new review request or have pre-reviewed other 
person's review request, please write the bug number on this bug report 
so that I can check your comments or review request.

Fedora package collection review requests which are waiting for someone to
review can be checked on my wiki page:
http://fedoraproject.org/wiki/User:Mtasaka#B._Review_request_tickets
(Check No one is reviewing)

Review guidelines are described mainly on:
http://fedoraproject.org/wiki/Packaging/ReviewGuidelines
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets


You are the upstream of this package so you may not want to
submit another review request, however still I want to see
at least one pre-review done by you.

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

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


[Bug 524379] Review Request: gscribble - A desktop blogging client for GNOME

2009-11-16 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=524379





--- Comment #31 from Roshan Singh singh.rosha...@gmail.com  2009-11-16 
13:46:14 EDT ---
(In reply to comment #30)
 Okay, now:

Its a great day (actually night) for me. 

I will submit 2 more review requests. I maintain one package and the other is
something that I have to compile from source always whenever I format my
laptop. 

I have written the spec files. But now that I know that a lot has to be done in
the spec, I will check it properly for possible errors.

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

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


[Bug 481528] Review Request: perl-DBIx-Safe - Safer access to your database through a DBI database handle

2009-11-16 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=481528





--- Comment #14 from Fedora Update System upda...@fedoraproject.org  
2009-11-16 13:57:51 EDT ---
perl-DBIx-Safe-1.2.5-6.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/perl-DBIx-Safe-1.2.5-6.el5

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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898





--- Comment #22 from Fedora Update System upda...@fedoraproject.org  
2009-11-16 14:02:27 EDT ---
imapfilter-2.0.11-3.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/imapfilter-2.0.11-3.fc12

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

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


[Bug 515898] Review Request: imapfilter - A flexible client side mail filtering utility for IMAP servers

2009-11-16 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=515898





--- Comment #21 from Fedora Update System upda...@fedoraproject.org  
2009-11-16 14:02:21 EDT ---
imapfilter-2.0.11-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/imapfilter-2.0.11-3.fc11

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

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


[Bug 530473] Review Request: lessfs - Lessfs is an inline data deduplicating filesystem.

2009-11-16 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=530473





--- Comment #11 from Pavel Alexeev (aka Pahan-Hubbitus) pa...@hubbitus.info  
2009-11-16 14:17:50 EDT ---
Rpmlint is verbose:
$ rpmlint *
lessfs.src: W: name-repeated-in-summary Lessfs
lessfs.src:97: W: macro-in-%changelog %config
lessfs.src: W: mixed-use-of-spaces-and-tabs (spaces: line 7, tab: line 1)
lessfs.spec:97: W: macro-in-%changelog %config
lessfs.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 7, tab: line 1)
lessfs-debuginfo.i686: W: spurious-executable-perm
/usr/src/debug/lessfs-0.8.2/lessfsck.c
lessfs.i686: W: name-repeated-in-summary Lessfs
lessfs.i686: W: dangling-symlink /usr/share/doc/lessfs-0.8.2/COPYING
/usr/share/automake-1.10/COPYING
lessfs.i686: W: spurious-executable-perm /usr/share/doc/lessfs-0.8.2/ChangeLog
lessfs.i686: E: executable-marked-as-config-file /etc/lessfs.cfg
lessfs.i686: E: script-without-shebang /etc/lessfs.cfg
lessfs.i686: E: no-status-entry /etc/init.d/lessfs
lessfs.i686: W: no-reload-entry /etc/init.d/lessfs
3 packages and 1 specfiles checked; 3 errors, 10 warnings.

Please fix it before formal review starts.

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

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


[Bug 508518] Review Request: meiga - Easy tool for file sharing and content publishing

2009-11-16 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=508518


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

Summary|Review Request: Meiga - |Review Request: meiga -
   |Easy tool for file sharing  |Easy tool for file sharing
   |and content publishing  |and content publishing




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

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


[Bug 515053] Review Request: sugar-getiabooks - Internet Archive Books receiver for Sugar

2009-11-16 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=515053


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 15:05:01 EDT 
---
Note that the spec above doesn't match what's in the srpm above; the latter
elicits an incoherent-version-in-changelog complaint from rpm.  I guess you
fixed that in the separate spec.  Please let me know that you're doing things
like that to avoid confusion.

Anyway, aside from that rpmlint complaint which you've already fixed, this
looks good.  APPROVED

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

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


[Bug 532533] Review request: wsdl4j - Web Services Description Language Toolkit for Java

2009-11-16 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=532533


Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Comment #2 from Mary Ellen Foster mefos...@gmail.com  2009-11-16 16:24:10 
EDT ---
Sorry, it doesn't differ at all -- the JPackage version has a different name
and I didn't realise that this was the same package ultimately.

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

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


[Bug 504891] Review Request: trove - Efficient Java collections

2009-11-16 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=504891





--- Comment #6 from Mary Ellen Foster mefos...@gmail.com  2009-11-16 16:26:54 
EDT ---
I have a package-in-progress of version 2.1 in bug 532521 if anyone's
interested -- Jerry, do you still want to package 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 231305] Review Request: xml-security - Implementation of W3C security standards for XML

2009-11-16 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=231305


Mary Ellen Foster mefos...@gmail.com changed:

   What|Removed |Added

 CC||mefos...@gmail.com




--- Comment #13 from Mary Ellen Foster mefos...@gmail.com  2009-11-16 
16:30:00 EDT ---
Is anyone working on this? Now that OpenJDK is around, building against sun.*
is no longer a showstopper ... I have a newer review request at
https://bugzilla.redhat.com/show_bug.cgi?id=532535 if anyone else is still
interested in it.

-- 
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 504891] Review Request: trove - Efficient Java collections

2009-11-16 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=504891





--- Comment #7 from Jerry James loganje...@gmail.com  2009-11-16 16:39:50 EDT 
---
Not really.  The package I wanted to push that depended on this turned out to
have some severe license problems.  Please go ahead and take it.  I'm not sure
what the proper bugzilla etiquette is, 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 478806] Review Request: simh - A highly portable, multi-system emulator

2009-11-16 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=478806





--- Comment #15 from Jason Tibbitts ti...@math.uh.edu  2009-11-16 17:39:50 
EDT ---
Now I'm getting a broken debuginfo package; it's missing the sources causing
rpmlint to complain:
  simh-debuginfo.x86_64: E: debuginfo-without-sources

Looking at the build log, I see:

+ make -j16 -e 'OPT=-O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2 -fexceptions
-fstack-protector --param=ssp-buffer-size=4 -m64 -mtune=generic' USE_NETWORK=1
make: Nothing to be done for `all'.

and no calls to the compiler, which is pretty odd.  Then I checked your
tarball, and it seems to have the compiled sources already there in a BIN
directory.

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

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


[Bug 537631] Review Request: moblin-panel-web - Moblin Panel for Web Browsing

2009-11-16 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=537631





--- Comment #1 from Christian Krause c...@plauener.de  2009-11-16 18:11:02 
EDT ---
Before I do the full review here are some issues I've seen so far:

1. the Source0 URL seems to be not working, the server returns 404 Not found
(probably we should ask upstream to tag the releases in git ;-) )

IMHO it would be better to use only the file name as Source0 (the complete URL
could be kept in a comment for later when upstream has tagged the releases) -
otherwise it is misleading...

2. it looks like that the BR xulrunner-devel is not needed

3. are you planning to package the moblin web browser as well? In this case I
would recommend that moblin-panel-web requires the moblin-web-browser
(otherwise it just won't do anything... ;-) )

4. regarding the License I'm not sure: the sources in common/ are LGPLv2.1 but
the main application seems to be GPLv2+ - I've asked on the fedora-legal
mailing list for clarification...

I'm really looking forward to get a full moblin UI for Fedora! ;-)

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

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


[Bug 537631] Review Request: moblin-panel-web - Moblin Panel for Web Browsing

2009-11-16 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=537631





--- Comment #2 from Peter Robinson pbrobin...@gmail.com  2009-11-16 18:19:40 
EDT ---
(In reply to comment #1)
 Before I do the full review here are some issues I've seen so far:
 
 1. the Source0 URL seems to be not working, the server returns 404 Not found
 (probably we should ask upstream to tag the releases in git ;-) )

Yes, your correct. See the two lines at the top of the spec file to create the
tar file.

 IMHO it would be better to use only the file name as Source0 (the complete URL
 could be kept in a comment for later when upstream has tagged the releases) -
 otherwise it is misleading...

Yes, I meant to update it as that URL is in my Moblin spec template.

 2. it looks like that the BR xulrunner-devel is not needed

Oh!

 3. are you planning to package the moblin web browser as well? In this case I
 would recommend that moblin-panel-web requires the moblin-web-browser
 (otherwise it just won't do anything... ;-) )

Yes, Its in progress and I believe in the mean time it will use the default
browser instead so in the short term it will still be useful when used with
Firefox.

 4. regarding the License I'm not sure: the sources in common/ are LGPLv2.1 but
 the main application seems to be GPLv2+ - I've asked on the fedora-legal
 mailing list for clarification...

I'll clarify with upstream. I looked in common and what was in the upstream
Moblin specfile. I suspect its a mix of both because of what the other
moblin-panel packages use.

 I'm really looking forward to get a full moblin UI for Fedora! ;-)  

Me too!

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

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


[Bug 527986] Review Request: taoframework - Multimedia bindings for Mono

2009-11-16 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=527986





--- Comment #10 from Paul Lange pala...@gmx.de  2009-11-16 18:24:29 EDT ---
I think it would be wrong to add dependencies for packages only available on
rpmfusion. I'm not sure about it, but I think it would be impossible to install
this package without having enabled rpmfusion.
I think it's better to make the dependencies as good as possible. If there are
any products which use this packages they should care about the required
dependencies.

Fixed config files:
- lua
- ode
- physfs
- SDL_gfx
- glfw

don't fixed because of rpmfusion requirement:
- cg
- sdl: libsmpeg
- ffmpeg

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

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


[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

2009-11-16 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=537587





--- Comment #5 from Nathanael Noblet nathan...@gnat.ca  2009-11-16 18:54:22 
EDT ---
Very few rpmlint messages now...

* Mon Nov 16 2009 Nathanael Noblet nathan...@gnat.ca - 3.9.0-BETA4
- Revise permission on /etc/dspam.conf and /var/lib/dspam
- Remove compression on documentation
- Move driver documentation to subpackages
- Remove sqlite_drv.txt if neither sqlite driver is being built
- Remove .la and .a files
- Disable install-strip for the debuginfo package

* Sun Nov 15 2009 Stevan Bajic ste...@bajic.ch - 3.9.0-BETA4
- Splitting into libdspam package and driver package
- Moving static libs and libtool archives into libdspam-devel package
- Enabling build of single storage driver statically linked into DSPAM
- Enabling build of single storage driver as dynamically loaded module
- Compressing SQL files and documentation
- Added version tag to change log
- Renamed cron and logrotate scripts to follow Fedora packing guidelines
- Added post/postun for libdspam and all libdspam storage drivers
- Replaced all tabs with softtabs (don't know if this is needed but Nathanael
had it that way)
- Enabled install-strip
- Extended package description

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

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


[Bug 532874] Review Request: xorg-x11-drv-wacom - Xorg X11 wacom input driver

2009-11-16 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=532874





--- Comment #6 from Peter Hutterer peter.hutte...@redhat.com  2009-11-16 
19:06:18 EDT ---
Thomas - please file a separate bugreport for this issue, let's not use the
review request for actual bugs on this driver. Please attach the output of
lshal and your Xorg.log to the new bug (if you're using this driver,
temporarily file under linuxwacom and note that you're using xf86-input-wacom
instead). Make sure you assign the bug to me to avoid confusion. 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 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

2009-11-16 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=537587





--- Comment #6 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-11-16 
19:09:37 EDT ---
what are your fedora account user name ?

Can you post a updated spec file + src.rpm file ?

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

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


[Bug 533291] Review Request: ruby-ffi - Foreign Function Interface package for Ruby

2009-11-16 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=533291





--- Comment #7 from Bryan Kearney bkear...@redhat.com  2009-11-16 19:26:19 
EDT ---
Any feedback on the new spec file? Should I kill this bug and re-submit under
the new name?

-- bk

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

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


[Bug 481527] Review Request: bucardo - asynchronous PostgreSQL replication system

2009-11-16 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=481527


David Nalley da...@gnsa.us changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||da...@gnsa.us
 AssignedTo|nob...@fedoraproject.org|da...@gnsa.us
   Flag||fedora-review?




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

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


[Bug 537971] New: Review Request: ghc-mmap - Haskell binding to mmap

2009-11-16 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: ghc-mmap - Haskell binding to mmap

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

   Summary: Review Request: ghc-mmap - Haskell binding to mmap
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: peter...@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://petersen.fedorapeople.org/ghc-mmap/ghc-mmap.spec
SRPM URL:
http://petersen.fedorapeople.org/ghc-mmap/ghc-mmap-0.4.1-1.fc12.src.rpm
Description: 
This library provides a wrapper to mmap(2) or MapViewOfFile, allowing files or
devices to be lazily loaded into memory as strict or lazy ByteStrings,
ForeignPtrs or plain Ptrs, using the virtual memory subsystem to do on-demand
loading. Modifications are also supported.

This required for ghc-hashed-storage which is a dep of darcs 2.3.

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

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


[Bug 481527] Review Request: bucardo - asynchronous PostgreSQL replication system

2009-11-16 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=481527





--- Comment #12 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-11-16 
20:31:41 EDT ---
updated spec file with changes from Greg.

http://itamarjp.fedorapeople.org/bucardo/bucardo.spec
http://itamarjp.fedorapeople.org/bucardo/bucardo-4.4.0-2.fc12.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 481527] Review Request: bucardo - asynchronous PostgreSQL replication system

2009-11-16 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=481527


David Nalley da...@gnsa.us changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #13 from David Nalley da...@gnsa.us  2009-11-16 21:02:37 EDT ---
Itamar: Thanks for updating the SRPM in such a timely manner. 
I just installed and it appears to work well. 


OK: rpmlint must be run on every package. The output should be posted in the
review.
[ke4...@nalleyx60 SPECS]$ rpmlint bucardo.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyx60 SPECS]$ rpmlint ../SRPMS/bucardo-4.4.0-2.fc11.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
[ke4...@nalleyx60 SPECS]$ rpmlint
../RPMS/noarch/bucardo-4.4.0-2.fc11.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

OK: The package must be named according to the Package Naming Guidelines .
OK: The spec file name must match the base package %{name}, in the format
%{name}.spec unless your package has an exemption.
OK: The package must meet the Packaging Guidelines
OK: The package must be licensed with a Fedora approved license and meet the
Licensing Guidelines .
LICENSE file contains BSD
Source code refers to License file

While I don't think it affects the package, the INSTALL file appears to claim a
different license for a library, and upstream may want to take note. 


OK: The License field in the package spec file must match the actual license.
OK: If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package must be included in %doc.
OK: The spec file must be written in American English. 
OK: The spec file for the package MUST be legible. 
OK: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
[ke4...@nalleyx60 SOURCES]$ md5sum Bucardo-4.4.0.tar.gz*
bf32b4dc7302baa1bc269dd34ff65da5  Bucardo-4.4.0.tar.gz
bf32b4dc7302baa1bc269dd34ff65da5  Bucardo-4.4.0.tar.gz.1

OK: The package MUST successfully compile and build into binary rpms on at
least one primary architecture.
NA: If the package does not successfully compile, build or work on an
architecture, then those architectures should be listed in the spec in
ExcludeArch. Each architecture listed in ExcludeArch MUST have a bug filed in
bugzilla, describing the reason that the package does not compile/build/work on
that architecture. The bug number MUST be placed in a comment, next to the
corresponding ExcludeArch line.
OK: All build dependencies must be listed in BuildRequires, except for any that
are listed in the exceptions section of the Packaging Guidelines ; inclusion of
those as BuildRequires is optional. Apply common sense.
NA: The spec file MUST handle locales properly. This is done by using the
%find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.
NA: Every binary RPM package (or subpackage) which stores shared library files
(not just symlinks) in any of the dynamic linker's default paths, must call
ldconfig in %post and %postun. 
OK: Packages must NOT bundle copies of system libraries.
NA: If the package is designed to be relocatable, the packager must state this
fact in the request for review, along with the rationalization for relocation
of that specific package. Without this, use of Prefix: /usr is considered a
blocker. 
OK: A package must own all directories that it creates. If it does not create a
directory that it uses, then it should require a package which does create that
directory. 
OK: A Fedora package must not list a file more than once in the spec file's
%files listings. 
OK: Permissions on files must be set properly. Executables should be set with
executable permissions, for example. Every %files section must include a
%defattr(...) line. 
OK: Each package must have a %clean section, which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). 
OK: Each package must consistently use macros. 
OK: The package must contain code, or permissable content. 
NA: Large documentation files must go in a -doc subpackage. (The definition of
large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity).
OK: If a package includes something as %doc, it must not affect the runtime of
the application. To summarize: If it is in %doc, the program must run properly
if it is not present. 
NA: Header files must be in a -devel package. 
NA: Static libraries must be in a -static package. 
NA: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig' 

[Bug 537971] Review Request: ghc-mmap - Haskell binding to mmap

2009-11-16 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=537971


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 Blocks||537979




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

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


[Bug 537979] Review Request: ghc-hashed-storage - Hashed file storage support

2009-11-16 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=537979


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 CC||fedora-haskell-l...@redhat.
   ||com
 Depends on||537971




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

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


[Bug 537979] New: Review Request: ghc-hashed-storage - Hashed file storage support

2009-11-16 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: ghc-hashed-storage - Hashed file storage support

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

   Summary: Review Request: ghc-hashed-storage - Hashed file
storage support
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: peter...@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://petersen.fedorapeople.org/ghc-hashed-storage/ghc-hashed-storage.spec
SRPM URL:
http://petersen.fedorapeople.org/ghc-hashed-storage/ghc-hashed-storage-0.3.9-1.fc12.src.rpm
Description: 
Support code for reading and manipulating hashed file storage (where each file
and directory is associated with a cryptographic hash, for corruption-resistant
storage and fast comparisons).

The supported storage formats include darcs hashed pristine, a plain filesystem
tree and an indexed plain tree (where the index maintains hashes of the plain
files and directories).

Needed by darcs 2.3.

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

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


[Bug 537979] Review Request: ghc-hashed-storage - Hashed file storage support

2009-11-16 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=537979


Jens Petersen peter...@redhat.com changed:

   What|Removed |Added

 Blocks||522953




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

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


[Bug 481527] Review Request: bucardo - asynchronous PostgreSQL replication system

2009-11-16 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=481527


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

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #14 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-11-16 
21:37:02 EDT ---
New Package CVS Request
===
Package Name: bucardo
Short Description: asynchronous PostgreSQL replication system
Owners: itamarjp
Branches: F-10 F-11 F-12 EL-5
InitialCC: perl-sig

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

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


[Bug 455152] Review Request: visual-4.beta26-1.fc9.i386.rpm - Visual Modelling Library

2009-11-16 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=455152


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||toms...@fedoraproject.org
 Resolution|WONTFIX |DUPLICATE




--- Comment #7 from Thomas Spura toms...@fedoraproject.org  2009-11-16 
21:42:22 EDT ---


*** This bug has been marked as a duplicate of 537983 ***

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

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


[Bug 537983] Review Request: vpython - 3D Programming

2009-11-16 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=537983


Thomas Spura toms...@fedoraproject.org changed:

   What|Removed |Added

 CC||brad.lo...@gmail.com




--- Comment #1 from Thomas Spura toms...@fedoraproject.org  2009-11-16 
21:42:23 EDT ---
*** Bug 455152 has been marked as a duplicate of 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 537983] New: Review Request: vpython - 3D Programming

2009-11-16 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: vpython - 3D Programming

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

   Summary: Review Request: vpython - 3D Programming
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: toms...@fedoraproject.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://tomspur.fedorapeople.org/review/vpython.spec
SRPM URL: http://tomspur.fedorapeople.org/review/vpython-5.13-1.fc12.src.rpm
Description:
VPython makes it easy to create navigable 3D displays and animations, even for
those with limited programming experience. Because it is based on Python, it
also has much to offer for experienced programmers and researchers.

Builds in koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1810799

rpmlint:
$ rpmlint vpython.spec vpython-5.13-1.fc12.src.rpm x86_64/*
vpython.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 12)
vpython.src: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line 12)
vpython.x86_64: E: devel-dependency vpython-devel
vpython-devel.x86_64: W: no-dependency-on vpython/vpython-libs/libvpython
vpython-devel.x86_64: W: no-documentation
vpython-devel.x86_64: W: dangling-relative-symlink
/usr/lib64/python2.6/site-packages/cvisualmodule.so cvisualmodule.so.3.0.0
vpython-doc.x86_64: W: wrong-file-end-of-line-encoding
/usr/share/doc/vpython-doc-5.13/visual/VisualRef.css
5 packages and 1 specfiles checked; 1 errors, 6 warnings.

- I don't see any tabs anymore...
- Don't know what to do with no-dependency-on vpython/vpython-libs/libvpython
- dangling-relative-symlink seems ok to me
- wrong-file-end-of-line-encoding: Tried to fix this with converting the file
  to UTF-8, didn't work. In documentation, is it ignorable?
- devel-file-depency: Should I delete the devel package and ship the *.so file
  with the main package?

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

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


[Bug 537727] Review Request: Slack - A lazy systems administration configuration management toolkit.

2009-11-16 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=537727





--- Comment #3 from Matt T. Proud matt.pr...@gmail.com  2009-11-16 22:19:12 
EDT ---
Hi Jussi,

Thank you for the prompt feedback.  I have gone ahead and incorporated your
suggestions:

- Patch naming,
- Privileges,
- Using %{_datadir}/%{name} due to the item being architecture-independent,
- Ownership of the %{_datadir}/%{name}, and
- The format-agnosticness of the manual pages.

I did make an error in the URL for the SRPM supra.  The correct address is
http://www.nerp.net/~khanreaper/slack-0.15.2-2.fc11.src.rpm.

Let me know if there's anything else I can do!

- M.

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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021


Alex Lancaster al...@users.sourceforge.net changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Comment #14 from Alex Lancaster al...@users.sourceforge.net  2009-11-17 
00:40:42 EDT ---
Taking 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 530021] Review Request: moovida - Media Center

2009-11-16 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=530021





--- Comment #15 from Alex Lancaster al...@users.sourceforge.net  2009-11-17 
00:51:12 EDT ---
As a @redhat.com person have you have already been sponsored?I can't find
any packages you own on PackageDB:

https://admin.fedoraproject.org/pkgdb/users/packages/ggillies

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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021





--- Comment #16 from Graeme Gillies ggill...@redhat.com  2009-11-17 00:55:47 
EDT ---
Sorry no I totally forgot to mention that this is my first package submission
and I have not been sponsored yet. Sorry for the confusion.

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

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


[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

2009-11-16 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=537587





--- Comment #7 from Nathanael Noblet nathan...@gnat.ca  2009-11-17 01:02:02 
EDT ---
account user name is gnat.

The spec file + src.rpm file is at the url provided. I've been updating them
directly each time I make a change.

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

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


[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

2009-11-16 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=537587





--- Comment #8 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-11-17 
01:08:56 EDT ---
please bump the version and post a updated version after each change.

spec file.
src.rpm

also a koji scratch build should be welcome

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

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


[Bug 478806] Review Request: simh - A highly portable, multi-system emulator

2009-11-16 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=478806





--- Comment #16 from Lucian Langa co...@gnome.eu.org  2009-11-17 01:11:38 EDT 
---
I've added the wrong source file.
Hopefully should be ok now:

http://lucilanga.fedorapeople.org/simh.spec
http://lucilanga.fedorapeople.org/simh-3.8.1-4.fc12.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 533899] Review Request: activemq-cpp - C++ implementation of JMS-like messaging client

2009-11-16 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=533899





--- Comment #2 from Steve Traylen steve.tray...@cern.ch  2009-11-17 01:39:40 
EDT ---
Am more happy with these after trying to build something else on top.

http://cern.ch/straylen/rpms/activemq-cpp/activemq-cpp-2.2.6-4.fc12.src.rpm
http://cern.ch/straylen/rpms/activemq-cpp/activemq-cpp.spec

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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021


Alex Lancaster al...@users.sourceforge.net changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 Blocks||177841(FE-NEEDSPONSOR)
 AssignedTo|al...@users.sourceforge.net |nob...@fedoraproject.org
   Flag|fedora-review?  |




--- Comment #17 from Alex Lancaster al...@users.sourceforge.net  2009-11-17 
01:40:09 EDT ---
(In reply to comment #16)
 Sorry no I totally forgot to mention that this is my first package submission
 and I have not been sponsored yet. Sorry for the confusion.  

Ah, so then unfortunately your first review needs to be done by a sponsor,
which I am not, see:

https://fedoraproject.org/wiki/Package_Review_Process#Reviewer

I can do an informal review, but can't do the official one. :(So I'm
unassigning this and setting the blocker bug FE-NEEDSPONSOR. Perhaps you can
pester some redhat colleagues who are also sponsors.  Meanwhile I will make
some quick remarks here and on a followup comment:

Summmary: should not include the package name and should use upstream (US)
spelling as described on http://www.moovida.com/ so:

Summary: Media center

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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021





--- Comment #18 from Alex Lancaster al...@users.sourceforge.net  2009-11-17 
01:46:55 EDT ---
For the moovida-base package though, it would probably make sense to include
the name moovida in the Summary since it isn't really the one that's usually
installed by end users.

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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021


Alex Lancaster al...@users.sourceforge.net changed:

   What|Removed |Added

  Alias||moovida




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

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


[Bug 530021] Review Request: moovida - Media Center

2009-11-16 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=530021





--- Comment #19 from Alex Lancaster al...@users.sourceforge.net  2009-11-17 
02:02:11 EDT ---
OK, to start the ball rolling here's the output for rpmlint for the binary
packages:

# rpmlint *
moovida.noarch: W: name-repeated-in-summary Moovida
moovida-base.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/elisa/core/launcher.py 0644 /usr/bin/python
moovida-base.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/elisa/core/utils/mswin32/structures.py 0644
/usr/bin/python
moovida-base.noarch: E: wrong-script-end-of-line-encoding
/usr/lib/python2.6/site-packages/elisa/core/utils/mswin32/structures.py
moovida-base.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/elisa/core/tests/test_launcher.py 0644
/usr/bin/python
moovida-plugins-bad.noarch: W: no-documentation
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/cs/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/de/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/es/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/fr/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/gl/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/hu/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/it/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/pl/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/database/i18n/pt_BR/LC_MESSAGES/elisa-plugin-database.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/cs/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/de/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/es/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/fr/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/gl/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/hu/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/it/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/pl/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/dvd/i18n/pt_BR/LC_MESSAGES/elisa-plugin-dvd.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/cs/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/de/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/es/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/fr/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/gl/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/hu/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/it/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/pl/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-bad.noarch: W: file-not-in-%lang
/usr/lib/python2.6/site-packages/elisa/plugins/poblesec/i18n/pt_BR/LC_MESSAGES/elisa-plugin-poblesec.mo
moovida-plugins-good.noarch: W: 

[Bug 424841] Review Request: ndisc6 - IPv6 diagnostic tools

2009-11-16 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=424841





--- Comment #24 from Stjepan Gros stjepan.g...@gmail.com  2009-11-17 02:19:44 
EDT ---
Here is the new version:

Spec URL: http://www.zemris.fer.hr/~sgros/stuff/fedora/ndisc6/ndisc6.spec
SRPM URL:
http://www.zemris.fer.hr/~sgros/stuff/fedora/ndisc6/ndisc6-0.9.9-1.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 520637] Review Request: mdk - GNU MIX Development Kit

2009-11-16 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=520637


Stjepan Gros stjepan.g...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #23 from Stjepan Gros stjepan.g...@gmail.com  2009-11-17 02:35:37 
EDT ---
New Package CVS Request
===
Package Name: mdk
Short Description: GNU MIX Development Kit
Owners: sgros
Branches: F-12
InitialCC: sgros

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

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


[Bug 251805] Review Request: orafce - Implementation of some Oracle functions into PostgreSQL

2009-11-16 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=251805





--- Comment #23 from Miroslav Suchý msu...@redhat.com  2009-11-17 02:36:40 
EDT ---
http://yum.pgsqlrpms.org/srpms/8.4/fedora/fedora-11-i386/orafce-3.0.1-1.f11.src.rpm

404 Not Found

Anyway, from quick look on spec itself, I still see, that two of my previous
comments has not been addressed:
 %{_docdir}/pgsql/contrib/*.orafunc
 That should be 
 %doc COPYRIGHT.orafunc INSTALL.orafunc README.orafunc

And:
Do you really want to requires postgresql, I suppose that correct should be be
postgresql-server, but IMHO.

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

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


[Bug 537587] Review Request: dspam - bayesian filtering daemon, client, library and web ui

2009-11-16 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=537587





--- Comment #9 from Nathanael Noblet nathan...@gnat.ca  2009-11-17 02:45:34 
EDT ---
Looks like it completed successfully. 

Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=1811003

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

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