[Bug 691081] Review Request: erlang-erlzmq2 - Erlang binding for ZeroMQ

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


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

--- Comment #3 from Mikhail Kulemin mihkule...@gmail.com 2011-04-06 02:47:33 
EDT ---
Build new rpm from latest sources.
Now socket is a tuple with index number.

SRPM URL:
http://mkulemin.fedorapeople.org/packages/erlang-erlzmq2-0-1.20110406gitfd99a31.fc14.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network

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


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

--- Comment #26 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 03:10:02 EDT ---
n2n-2.0.1-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/n2n-2.0.1-3.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

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


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

Bug 674008 depends on bug 676159, which changed state.

Bug 676159 Summary: Review Request: crlibm - Correctly Rounded mathematical 
library
https://bugzilla.redhat.com/show_bug.cgi?id=676159

   What|Old Value   |New Value

 Resolution||ERRATA
 Status|ON_QA   |CLOSED

--- Comment #35 from Rosen Diankov rosen.dian...@gmail.com 2011-04-06 
03:09:42 EDT ---
hi guys, anyone know what's happening with the release? any problems left that
i can help in resolving?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693342] Review Request: perl-MooseX-TrackDirty-Attributes - Track dirtied attributes

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
   Fixed In Version||perl-MooseX-TrackDirty-Attr
   ||ibutes-0.02-1.fc16
 Resolution||RAWHIDE
Last Closed||2011-04-06 03:15:40

--- Comment #4 from Petr Pisar ppi...@redhat.com 2011-04-06 03:15:40 EDT ---
Thank you for the review and the repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network

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


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

--- Comment #27 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 03:41:58 EDT ---
n2n-2.0.1-3.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/n2n-2.0.1-3.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network

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


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

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 03:44:37 EDT ---
n2n-2.0.1-3.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/n2n-2.0.1-3.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691027] Review Request: n2n - A layer-two peer-to-peer virtual private network

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


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

Hushan Jia h...@redhat.com changed:

   What|Removed |Added

   Flag||needinfo?(itamar@ispbrasil.
   ||com.br)

--- Comment #29 from Hushan Jia h...@redhat.com 2011-04-06 03:49:35 EDT ---
Hi Itamar, 
The packages are now in el5 and el6, you can test them, also please add a
feedback if it works, so that we can make it into stable release.

Thanks,
Hushan

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 531107] Review Request: perl-SQL-Tokenizer - A Perl package to tokenize SQL, generically

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|NEW
 AssignedTo|mmasl...@redhat.com |nob...@fedoraproject.org
   Flag|fedora-review+  |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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 675963] Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Marcela Mašláňová mmasl...@redhat.com 2011-04-06 04:07:01 
EDT ---
New Package SCM Request
===
Package Name: perl-MDV-Packdrakeng
Short Description: Simple Archive Extractor/Builder
Owners: mmaslano psabata ppisar
Branches: 
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 690359] Review Request: perl-Shipwright - Build and Manage Self-contained Software Bundle

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


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

--- Comment #5 from Marcela Mašláňová mmasl...@redhat.com 2011-04-06 04:13:45 
EDT ---
Please post new koji build with working test suite and we can close this
review.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 691619] Review Request: openvas-manager - Open Vulnerability Assessment (OpenVAS) Manager (edit)

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


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

--- Comment #16 from Xavier Bachelot xav...@bachelot.org 2011-04-06 04:43:25 
EDT ---
This is only nitpicking but the %clean section could be made simpler. you can
remove all of [ %{buildroot} = / ] || 

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693363] Review Request: st - A simple terminal implementation for X

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 226510] Merge Review: udev

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


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

--- Comment #23 from Harald Hoyer har...@redhat.com 2011-04-06 04:53:47 EDT 
---
(In reply to comment #22)
 Ok, once the space/tab thing is fixed, I'm good.  The rest is justified, I
 think, and the source URL was apparently a fluke, because it works for me now
 and the md5sum matches.

already untabifyied in git master.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690359] Review Request: perl-Shipwright - Build and Manage Self-contained Software Bundle

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


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

--- Comment #6 from Robin Lee robinlee.s...@gmail.com 2011-04-06 05:41:31 EDT 
---
Change:
- Update to 2.4.24

Spec URL: http://cheeselee.fedorapeople.org/perl-Shipwright.spec
SRPM URL:
http://cheeselee.fedorapeople.org/perl-Shipwright-2.4.24-1.fc14.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691081] Review Request: erlang-erlzmq2 - Erlang binding for ZeroMQ

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


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

--- Comment #4 from Peter Lemenkov lemen...@gmail.com 2011-04-06 06:24:06 EDT 
---
This is wrong:

%define debug_package %{nil}

This package contains shared library object so it needs a debuginfo.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674008] Review Request: openrave - Open Robotics Automation Virtual Environment

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


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

--- Comment #36 from Tim Niemueller t...@niemueller.de 2011-04-06 06:34:40 
EDT ---
Hi Rosen. We've used OpenRAVE successfully during the RoboCup German Open 2011
last week. It has been integrated with Fawkes to drive a Katana arm. We're
preparing a demo video highlighting some of our work areas, I'll post a link
once it's ready.

I have to talk to my student, I think we've used
openrave-0.2.20-0.2.svn2227.fc14.x86_64 in the end. It still required a little
patch, due to bug #667294 (Boost was missing deprecated iostreams ctors). This
should have been resolved by now and I'm going to test it later today. Please
consider upgrading to non-deprecated calls in a future version.

Rosen, I have currently set to use single-precision floats to speed up OpenRave
a little. Do you think that has overly negative impact, or is that a good
default choice for a distro package? When compiling OpenRave at around 50% it
compiles openravepy_int.o, which takes more than 1.5GB of memory and is pretty
heavy on the system. Any chance of splitting this up in a future release?

We've been seeing some weird slowness on 32-bit systems with slow database
generation, while it ran fine with the identical (besides bitness) 64-bit
version. I'll discuss this with my team members tomorrow if this was an
intermediate problem and if so how it was resolved.

From the packaging site I need to push updates for collada-dom (which is
basically ready and built, I just didn't have the time). I have to disable
assimp atm, until review bug #635511 is through. Once that is done, we get rid
of the ivcon integrated library, right?

After that, it leaves two libraries in there. We're clear that
convexdecomposition has no standalone upstream and it got patches, therefore
should stay. fparser has critical patches. Rosen, how is upstreaming going on
this? Maybe you can try again and tell them that'll bring fparser to the Fedora
universe as an incentive?

Peter, please have a look again at this review now, I think we've reached a
good state to continue, with the only things to resolve being assimp acceptance
and fparser patch upstreaming and eventually ripping it out of OpenRave. What
do you think?

SPEC has been changed in-place, the SRPM is at
http://fedorapeople.org/~timn/robotics/openrave-0.2.20-0.2.svn2227.fc14.src.rpm.
It contains the patch to work with old boost. It can build with or without
assimp, for the latter just install the review package. Once it has been
accpted I'll add it to the BRs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
06:38:25 EDT ---
collada-dom-2.3-2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/collada-dom-2.3-2.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

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

   What|Removed |Added

 CC||rosen.dian...@gmail.com

--- Comment #34 from Tim Niemueller t...@niemueller.de 2011-04-06 06:34:36 
EDT ---
Seems to work with OpenRave without problems, but I can't tell how to do
specific testing of the assimp integration. Does this still block FE-Legal or
can we continue with the review now?

CC'ing Rosen on this. Rosen, can you give instructions how to test the assimp
integration in OpenRave?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
06:40:25 EDT ---
collada-dom-2.3-2.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/collada-dom-2.3-2.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
06:39:44 EDT ---
collada-dom-2.3-2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/collada-dom-2.3-2.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690953] Review Request: dexter - Address Book that goes hand in hand with Postler

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


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

--- Comment #18 from Michel Alexandre Salim michel+...@sylvestre.me 
2011-04-06 06:45:16 EDT ---
ps we have a patched python-distutils-extra now that no longer fails when no
screen is available, so xvfb-run is no longer needed

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690954] Review Request: postler - An ultra simple desktop mail client

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


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

--- Comment #14 from Michel Alexandre Salim michel+...@sylvestre.me 
2011-04-06 06:43:11 EDT ---
I was waiting for a fix for the Vala issue (which I just backported from Git
yesterday), before finishing my Postler packaging -- sorry for the duplicate
request!

Thomas, let me know if you'd like a comaintainer for this. I have some
differences in my specs:

- CFLAGS needs to be manually exported; when packaging I searched for any
guidelines for packaging with waf (we still really need to write one!) Funnily
all the people involved in that thread on the mailing list are here -- me, you
(waf maintainer) and Christoph who started the thread.

- waf build probably needs the -v option (verbose) -- that's how I discovered
  the wrong compiler flags were being used, and %{?_smp_mflags}

- link against libdb-devel (BerkeleyDB 5) rather than db4

- use system waf rather than the bundled version

Also, I started using the virtual pkgconfig(...) rather than package names, as
they match up with upstream's dependency checking better.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691081] Review Request: erlang-erlzmq2 - Erlang binding for ZeroMQ

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


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

--- Comment #5 from Peter Lemenkov lemen...@gmail.com 2011-04-06 06:46:49 EDT 
---
Failed to build in Koji:

http://koji.fedoraproject.org/koji/taskinfo?taskID=2978125

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690954] Review Request: postler - An ultra simple desktop mail client

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


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

--- Comment #15 from Thomas Moschny thomas.mosc...@gmx.de 2011-04-06 07:26:23 
EDT ---
(In reply to comment #14)
 I was waiting for a fix for the Vala issue (which I just backported from Git
 yesterday), before finishing my Postler packaging -- sorry for the duplicate
 request!

Thanks for applying the Vala fix!

 Thomas, let me know if you'd like a comaintainer for this.

Sure.

 I have some differences in my specs:
 
 - CFLAGS needs to be manually exported; when packaging I searched for any

Right, I will fix that while importing the package (maybe in the evening).

 guidelines for packaging with waf (we still really need to write one!) Funnily
 all the people involved in that thread on the mailing list are here -- me, you
 (waf maintainer) and Christoph who started the thread.

There has been a long thread on the devel mailing list some time ago with no
real outcome. Also see below.

 - waf build probably needs the -v option (verbose) -- that's how I discovered

The -v switch didn't work on my F13 machine, and caused waf itself to crash.

   the wrong compiler flags were being used, and %{?_smp_mflags}

Not really needed, as waf automatically builds in parallel, using as many
threads as it finds cores. Maybe it's nevertheless a good idea to pass -j
%{?_smp_mflags}, so koji has more control.

 - link against libdb-devel (BerkeleyDB 5) rather than db4

Why's that? 

 - use system waf rather than the bundled version

This is something we shouldn't do (in general, for any package, that is). I was
still hoping someone else filed a request for an exception at the packaging
committee. But now that I also own a package using waf, so maybe I have to do
it myself.

Postler bundles waf 1.5.X, but in rawhide we are at 1.6. I didn't thoroughly
test whether there are incompatibilities for postler, but in general, each
package should use its bundled waf copy and not the system one. As waf is a
development tool and not some sort of runtime library, the usual security
argument against bundling doesn't really hold. The bundled copy is what
upstream has released and tested their package with, and deviating from that is
not worth the effort.

 Also, I started using the virtual pkgconfig(...) rather than package names, 
 as they match up with upstream's dependency checking better.

Will consider 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 522821] Review Request: bluetile - A modern tiling window manager with a gentle learning curve

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


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

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

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

--- Comment #27 from Jens Petersen peter...@redhat.com 2011-04-06 07:35:05 
EDT ---
Imported and built.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 522821] Review Request: bluetile - A modern tiling window manager with a gentle learning curve

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


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

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 07:44:19 EDT ---
bluetile-0.5.3-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/bluetile-0.5.3-4.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 610934] Review Request: go - The Google Go Programming Language

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


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

Michel Alexandre Salim michel+...@sylvestre.me changed:

   What|Removed |Added

   Flag|fedora-review?  |

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672395] Review Request: eigen3 - A lightweight C++ template library for vector and matrix math

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


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

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

   What|Removed |Added

 Status|NEW |ASSIGNED
 Depends on||691133
   Flag||fedora-review?

--- Comment #5 from Tim Niemueller t...@niemueller.de 2011-04-06 07:42:57 EDT 
---
I'll review this as far as possible on F-14.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 652987] Review Request: go - The Go programming language

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


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

Michel Alexandre Salim michel+...@sylvestre.me changed:

   What|Removed |Added

 AssignedTo|michel+...@sylvestre.me |nob...@fedoraproject.org
   Flag|fedora-review?  |needinfo?

--- Comment #37 from Michel Alexandre Salim michel+...@sylvestre.me 
2011-04-06 07:46:59 EDT ---
Just a question -- who'll be the primary maintainer of this package? I see
specs from both Renich and Michael.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690359] Review Request: perl-Shipwright - Build and Manage Self-contained Software Bundle

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #7 from Marcela Mašláňová mmasl...@redhat.com 2011-04-06 07:52:01 
EDT ---
http://koji.fedoraproject.org/koji/taskinfo?taskID=2978308

Even on new version rpmlint doesn't complain.

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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 693363] Review Request: st - A simple terminal implementation for X

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


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

Marcela Mašláňová mmasl...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #1 from Marcela Mašláňová mmasl...@redhat.com 2011-04-06 08:05:59 
EDT ---
- rpmlint OK
- package must be named according to Guidelines OK
- spec file name must match the base package %{name} OK
- package must meet the Packaging Guidelines OK
- package must be licensed with Fedora approved license OK
- license field must match actual license OK
- text of the license in its own file must be included in %doc OK
- sources must match the upstream source OK
- package MUST successfully compile and build OK
- architecture listed in ExcludeArch MUST have a bug filed in bugzilla OK
- build dependencies must be listed in BuildRequires OK
- handle locales properly with %find_lang macro OK
- shared library files must call ldconfig in %post(un) OK
- packages must NOT bundle system libraries OK
- package must own all directories that it creates OK
- permissions on files must be set properly OK
- package must consistently use macros OK
- package must contain code, or permissable content OK
- large documentation must go in a -doc OK
- %doc must not affect the runtime of the application OK
- header files must be in a -devel package OK
- static libraries must be in a -static package OK
- library files that end in .so (without suffix) must go in a -devel OK
- devel package usually require base package OK
- packages must NOT contain any .la libtool archives OK
- GUI applications must include a %{name}.desktop file OK
- packages must not own files or directories already owned by other packages OK

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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 532241] Review Request: emacs-identica-mode - Identica mode for emacs.

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


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

Michel Alexandre Salim michel+...@sylvestre.me changed:

   What|Removed |Added

   Flag||needinfo?(saga...@gmail.com
   ||)

--- Comment #6 from Michel Alexandre Salim michel+...@sylvestre.me 2011-04-06 
08:10:29 EDT ---
Hi Arun,

Sorry for the late reply! A couple of problems with the package right now:
- incomplete Build Requirement: you're using makeinfo, which is supplied by
  the texinfo package
- Upstream releases unversioned tarballs on their main download site, which
  makes it hard for someone to verify that the sources are accurate. Suggest
  contacting upstream, and meanwhile downloading the tarballs generated from
the
  Git tags instead (see detailed comments below)

Also, 1.1 is out now -- probably a good time to upgrade.


* TODO Review [72%]
  - [X] Names [2/2]
- [X] Package name
- [X] Spec name
  - [X] Package version [2/2]
 http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Package_Versioning
- [X] Version number
   http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Version_Tag
- [X] Release tag
   http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Release_Tag
  
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#Pre-Release_packages
  - [X] Meets
[[http://fedoraproject.org/wiki/Packaging/Guidelines][guidelines]]
  - [-] Source files match upstream
 Upstream does not put the versioned source tarball in their download
 server; I suggest downloading the tarballs generated from the
 Git tags instead:
 http://git.savannah.gnu.org/cgit/identica-mode.git

 Contacting upstream is probably a good idea too, asking for the
 download tarballs to carry the version number
  - [X] [[http://fedoraproject.org/wiki/Packaging:No_Bundled_Libraries][No
bundled libraries]]
  - [X] License [4/4]
- [X] License is Fedora-approved
- [X] No licensing conflict
- [X] License field accurate
- [X] License included iff packaged by upstream
  - [-] rpmlint [1/2]
- [X] on src.rpm
   emacs-identica-mode.src: W: file-size-mismatch identica-mode.tar.gz = 31463,
http://download.savannah.gnu.org/releases/identica-mode/identica-mode.tar.gz =
33026
  1 packages and 0 specfiles checked; 0 errors, 1 warnings.

   This is due to the source file problem, see above. Otherwise fine
- [ ] on x86_64.rpm
  - [X] Language  locale [2/2]
- [X] Spec in US English
- [X] Spec legible
  - [-] Build [2/3]
- [-] Koji results
   http://koji.fedoraproject.org/koji/taskinfo?taskID=2978357
- [-] BRs complete
   makeinfo not found; needs to BR on texinfo
- [X] Directory ownership
  - [X] Spec inspection [9/9]
- [X] No duplicate files
- [X] File permissions
- [X] Filenames must be UTF-8
- [X] no BuildRoot
([[https://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag][except if
targeting RHEL5]])
- [X] [RHEL]  %clean section
  https://fedoraproject.org/wiki/Packaging/Guidelines#.25clean)
- [X] [RHEL 5] %buildroot cleaned on %install
- [X] Macro usage consistent
- [X] Documentation [1/1]
  - [X] %doc files are non-essential
- [X] Other subpackages
  - [X]
[[http://fedoraproject.org/wiki/Packaging/ScriptletSnippets][Scriptlets]] [1/1]
- [X] info files

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693363] Review Request: st - A simple terminal implementation for X

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #2 from Petr Sabata psab...@redhat.com 2011-04-06 08:13:19 EDT ---
New Package SCM Request
===
Package Name: st
Short Description: A simple terminal implementation for X
Owners: psabata
Branches: f14 f15
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690359] Review Request: perl-Shipwright - Build and Manage Self-contained Software Bundle

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


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

Robin Lee robinlee.s...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Robin Lee robinlee.s...@gmail.com 2011-04-06 08:13:40 EDT 
---
Thank you

New Package SCM Request
===
Package Name: perl-Shipwright
Short Description: Build and Manage Self-contained Software Bundle
Owners: cheeselee
Branches: f13 f14 f15
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

--- Comment #35 from Rich Mattes richmat...@gmail.com 2011-04-06 08:31:46 EDT 
---
The latest updates to the package have addressed the legal concerns by removing
the non-free model files from the tarball.  The remaining model files should
all have bsd-like licenses, and are OK for redistribution.  I do not think I
can lift the FE-LEGAL flag on my own 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 226510] Merge Review: udev

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


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

--- Comment #24 from Jon Ciesla l...@jcomserv.net 2011-04-06 08:55:12 EDT ---
Did you push the changes?

udev.spec:182: W: mixed-use-of-spaces-and-tabs (spaces: line 129, tab: line
182)
The specfile mixes use of spaces and tabs for indentation, which is a cosmetic
annoyance.  Use either spaces or tabs for indentation, not both.


[limb@bamboo udev]$ git log
commit 8df68695ff06d05f52b7187c24fa27969a5036c5
Merge: 0b0304c 58624e4
Author: Harald Hoyer har...@redhat.com
Date:   Thu Mar 31 18:32:08 2011 +0200

Merge branch 'f15/master'

commit 58624e426ed9b5d2b26f4a428cdc5cb8e4e6d4bf
Author: Harald Hoyer har...@redhat.com
Date:   Thu Mar 31 18:30:08 2011 +0200

udev.spec: removed gtk-doc

commit cbf7ac132cb5c8c9c7169c763df6714c5ad45b17
Author: Harald Hoyer har...@redhat.com
Date:   Wed Mar 30 17:36:11 2011 +0200

removed libxslt buildreq and touch man pages, to prevent rebuild

commit 8cf0c7defa5e033369d66ac319ed380c3fe73882
Author: Harald Hoyer har...@redhat.com
Date:   Thu Mar 31 17:58:51 2011 +0200

- fixed udev-trigger.service

commit 0b0304cfe18782b99aaa0814f9610d7ec91fcc2b
Author: Harald Hoyer har...@redhat.com
Date:   Thu Mar 31 17:58:51 2011 +0200

- fixed udev-trigger.service

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Jon Ciesla l...@jcomserv.net 2011-04-06 08:59:59 EDT ---
Fixed, will import.

Thanks so much for the review, and all your help with the rename project. 
Other than the last two modules, whose paths I've at least corrected, it's
finished.

New Package SCM Request
===
Package Name: drupal7
Short Description: An open-source content-management platform
Owners: limb
Branches: f14 f15 el6 el5
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694099] New: Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

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

Summary: Review Request: perl-Pod-MinimumVersion - Perl version for POD 
directives used

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

   Summary: Review Request: perl-Pod-MinimumVersion - Perl version
for POD directives used
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ppi...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL:
http://ppisar.fedorapeople.org/perl-Pod-MinimumVersion/perl-Pod-MinimumVersion.spec
SRPM URL:
http://ppisar.fedorapeople.org/perl-Pod-MinimumVersion/perl-Pod-MinimumVersion-50-1.fc16.src.rpm
Description:
Pod::MinimumVersion parses the POD in a Perl script, module, or document,
and reports what version of Perl is required to process the directives in
it with pod2man etc.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694099] Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

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


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

Petr Pisar ppi...@redhat.com changed:

   What|Removed |Added

 Blocks||694058

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 680593] Review Request: perl-Browser-Open - Open a browser in a given URL

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


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

--- Comment #1 from Petr Sabata psab...@redhat.com 2011-04-06 09:42:39 EDT ---
Package: perl-Browser-Open
Version: 0.03
Release: 1.fc16
Sources: Browser-Open-0.03.tar.gz
Patches: 
--
Package successfully built locally.
Package successfully built in mock, fedora-rawhide-x86_64.
Package successfully built in mock, fedora-rawhide-i386.
Package successfully built in koji, dist-rawhide.

MUST items:
[  OK  ] Package does NOT include pre-built binaries or libraries
[  OK  ] Spec file is legible and written in American english
[  OK  ] Package successfully builds on at least one supported primary
architecture
[  --  ] All ExcludeArch tags valid, referencing proper bug reports
[  OK  ] Package obeys FHS (with _libexecdir and /srv exceptions)
[  OK  ] No errors reported by rpmlint
[  OK  ] Changelog present and properly formatted
[  OK  ] Package does NOT include Packager, Vendor, Copyright or PreReq tags
[  OK  ] Source tags are working URLs and sources match upstream or justified
otherwise
[  OK  ] Requires correct or justified otherwise
[ NOTE ] BuildRequires correct or justified otherwise
[  OK  ] All file names are in proper UTF-8 encoding
[  OK  ] All plain text failes are in proper UTF-8 encoding
[  --  ] Large documentation files are located in doc subpackage
[  OK  ] All documentation prefixed with %doc
[  OK  ] Documentation is NOT executable
[  OK  ] No files in %doc are needed at run-time
[  --  ] Compiler flags honor Fedora defaults or are justified
[  --  ] Package generates useful debuginfo packages
[  --  ] Header files are placed in devel subpackage
[  --  ] Unversioned shared libraries are placed in devel subpackage
[  --  ] Pkgconfig files are placed in devel subpackage
[  --  ] Full-versioned Requires of the base package in subpackages
[  --  ] Package calls ldconfig in post and postun sections for all
subpackages, if applicable
[  --  ] Static libraries are provided by static subpackage
[  OK  ] Package contains no static executables unless approved by FESCo
[  OK  ] Package does NOT bundle any system libraries
[  --  ] RPath not used for anything besides internal libraries
[  --  ] All config files are marked noreplace or justified otherwise
[  OK  ] No config files are located under /usr
[  --  ] Package contains a SystemV-compatible initscript
[  --  ] A GUI application installs a proper desktop file
[  --  ] All desktop files are installed by desktop-file-install or justified
otherwise
[  OK  ] Package consistently uses macros
[  --  ] makeinstall macro is used only if make install DESTDIR=%{buildroot}
does NOT work
[  --  ] Macros in Summary and description are expandable at build-time
[  --  ] globals used in place of defines
[  --  ] Locales handled correctly -- package requires gettext and uses
find_lang, if applicable
[  --  ] Scriptlets are sane
[  OK  ] Package is not relocatable unless justified
[  OK  ] Package contains only acceptable code or content
[  OK  ] Package owns all the files and directories it creates, installs and/or
uses unless those are already ow
ned by another package
[  OK  ] files sections do NOT contain duplicate files except for licenses
[  OK  ] All files sections use defattr or justify otherwise
[  OK  ] Package does NOT cause any conflicts
[  OK  ] Package does NOT contain kernel modules
[  OK  ] Package does NOT bundle fonts or other general purpose data
[  OK  ] Final Requires and Provides are sane

SHOULD items:
[  OK  ] The Summary does NOT end with a period
[  OK  ] Package does NOT include BuildRoot tag, clean section or buildroot
removal in install section
[  OK  ] Package should preserve files timestamps
[  OK  ] Package does NOT explicitly BuildRequire bash, bzip2, coreutils, cpio,
diffutils, fedora-release, findu
tils, gawk, gcc, gcc-c++, grep, gzip, info, make, patch, redhat-rpm-config,
rpm-build, sed, shadow-utils, tar, u
nzip, util-linux-ng, which or xz
[  OK  ] Description does NOT consist of lines longer than 80 characters
[  OK  ] Package uses parallel make
[  --  ] In case of a web application, package installs date into /usr/share
instead of /var/www
[  --  ] All patches have a comment or an upstream bug link
[  --  ] Package installs manpages for all executables
[  OK  ] Package contains check section and all tests pass
[  ??  ] Package works as expected

NOTES:
--
1. BuildRequire File::Spec::Functions
2. release-pod-syntax.t (optionally) requires Test::Pod 1.00; I'd suggest a
versioned BuildRequires tag

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694099] Review Request: perl-Pod-MinimumVersion - Perl version for POD directives used

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


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

Petr Sabata psab...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||psab...@redhat.com
 AssignedTo|nob...@fedoraproject.org|psab...@redhat.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693493] Review Request: thunderbird-lightning - The calendar extension to Thunderbird

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


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

--- Comment #6 from Matej Cepl mc...@redhat.com 2011-04-06 09:53:04 EDT ---
+ BAD : rpmlint is silent on both source and binary package.
It isn't.
bradford:build $ rpmlint -i thunderbird-lightning-1.0-0.39.b3pre.fc14.src.rpm
RPMS/x86_64/thunderbird-lightning-*
thunderbird-lightning.src: W: strange-permission thunderbird-mozconfig-branded
0755L
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.

thunderbird-lightning.src: W: strange-permission find-external-requires 0755L
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.

thunderbird-lightning.src:19: W: macro-in-comment %define
There is a unescaped macro after a shell style comment in the specfile. Macros
are expanded everywhere, so check if it can cause a problem in this case and
escape the macro with another leading % if appropriate.

thunderbird-lightning.src: W: patch-not-applied Patch0:
thunderbird-version.patch
A patch is included in your package but was not applied. Refer to the patches
documentation to see what's wrong.

thunderbird-lightning.src: W: %ifarch-applied-patch Patch6:
mozilla-build-s390.patch
A patch is applied inside an %ifarch block. Patches must be applied on all
architectures and may contain necessary configure and/or code patch to be
effective only on a given arch.

thunderbird-lightning-debuginfo.x86_64: E: empty-debuginfo-package
This debuginfo package contains no files.  This is often a sign of binaries
being unexpectedly stripped too early during the build, rpmbuild not being
able to strip the binaries, the package actually being a noarch one but
erratically packaged as arch dependent, or something else.  Verify what the
case is, and if there's no way to produce useful debuginfo out of it, disable
creation of the debuginfo package.

thunderbird-lightning.x86_64: E: script-without-shebang
/usr/lib64/mozilla/extensions/{3550f703-e582-4d05-9a08-453d09bdfdc6}/{e2fda1a4-762b-4020-b5ad-a41df1933103}/calendar-js/calFreeBusyService.js
This text file has executable bits set or is located in a path dedicated for
executables, but lacks a shebang and cannot thus be executed.  If the file is
meant to be an executable script, add the shebang, otherwise remove the
executable bits or move the file elsewhere.

thunderbird-lightning.x86_64: E: script-without-shebang
/usr/lib64/mozilla/extensions/{3550f703-e582-4d05-9a08-453d09bdfdc6}/{e2fda1a4-762b-4020-b5ad-a41df1933103}/calendar-js/calAlarmMonitor.js
This text file has executable bits set or is located in a path dedicated for
executables, but lacks a shebang and cannot thus be executed.  If the file is
meant to be an executable script, add the shebang, otherwise remove the
executable bits or move the file elsewhere.

3 packages and 0 specfiles checked; 3 errors, 5 warnings.
bradford:build $ 

Errors must be fixed, warnings should be avoided as well if possible (and I
don't see any which couldn't).

+ GOOD: The package is named according to the Package Naming Guidelines .
+ GOOD: The spec file name matches the base package %{name}, in the format
  %{name}.spec.
+ GOOD: The package meets the Packaging Guidelines .
+ GOOD: The package is licensed with a Fedora approved license and meet the
Licensing Guidelines.
+ GOOD: The License field in the package spec file matches the actual license.
+ GOOD: LICENSE file is in %doc.
+ GOOD: The spec file is written in American English.
+ GOOD: The spec file for the package is legible.
- GOOD: The sources used to build the package matches the upstream source,
as provided in the spec URL.
SHA256: 8b499ec3d81d3242b0cc2de27effb1891a07259adf7e5e4c06150f8c9f5254c2
+ GOOD: The package successfully compiles and build into binary rpms on at
least one supported architecture.
Built on my Fedora15/x86_64
+ GOOD: builds on all architectures
http://koji.fedoraproject.org/koji/taskinfo?taskID=2978202
+ GOOD: All build dependencies are listed in BuildRequires. (builds in koji)
+ GOOD: The spec file MUST handle locales properly.
  No locales provided.
+ GOOD: %post and %postun scripts
no scripts
+ GOOD: not relocatable
+ GOOD: A package owns all directories that it creates.
+ GOOD: A package must not contain any duplicate files in the %files listing.
+ GOOD: Permissions on files must be set properly.
+ GOOD: Each package have a %clean section.
+ GOOD: Each package consistently use macros.
+ GOOD: The package contains code, or permissable content.
+ GOOD: -doc subpackage provided. No documentation
+ GOOD: Files registered in %doc does not affect the runtime of the
application.
+ GOOD: No header files.
+ GOOD: No static libraries.
+ GOOD: No pkgconfig(.pc) files.
+ GOOD: No .so file
+ GOOD: No -devel subpackage.
+ GOOD: No .la libtool archives.
+ GOOD: Packages does not 

[Bug 590355] Review Request: golly - cellular automata simulator

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


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

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

   What|Removed |Added

 Blocks|182235(FE-Legal)|

--- Comment #18 from Tom spot Callaway tcall...@redhat.com 2011-04-06 
10:20:38 EDT ---
Fantastic! Stephen never replied to my email, but this is a solid result.
Lifting FE-Legal.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

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

   What|Removed |Added

 Blocks|182235(FE-Legal)|

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 635511] Review Request: assimp - Library to import various 3D model formats into applications

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


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

--- Comment #36 from Tom spot Callaway tcall...@redhat.com 2011-04-06 
10:29:43 EDT ---
I think at this point, I'm fine to carry the patch in irrlicht, upstream seems
interested in adding it in a future release.

Also, I see no remaining legal issues, so I'm lifting FE-Legal. Go ahead and
finish this review, and let me know if you need any more irrlicht changes.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691096] Review Request: iperf3 - Measurement tool for TCP/UDP bandwidth performance

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


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

--- Comment #2 from Balaji G balaji...@gmail.com 2011-04-06 10:34:24 EDT ---
Kevin, 

Updated with your comments and uploaded to the following location

http://balajig8.fedorapeople.org/packages/iperf3.spec
http://balajig8.fedorapeople.org/packages/iperf3-3.0-0.0.b4.fc14.src.rpm

Thanks for your time.

Thanks,
Cheers,
  - Balaji

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691619] Review Request: openvas-manager - Open Vulnerability Assessment (OpenVAS) Manager (edit)

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


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

--- Comment #17 from Michal Ambroz re...@seznam.cz 2011-04-06 10:39:20 EDT ---
OK. I agree that in situation where most of the related packages
(openvas-libraries, openvas-scanner) doesn't have such safety check it won't
save anybody anyway - I will remove it.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 636819] Review Request: gnome-exe-thumbnailer - gnome thumbnailer for exe files

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


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

--- Comment #28 from elad el...@doom.co.il 2011-04-06 10:47:25 EDT ---
Please note that gnome-exe-thumbnailer would not function in Fedora 15. This is
due to the move of gnome from gconf to gsettings, and gnome-exe-thumbnailer's
upstream doesn't provide a gsettings schema (yet). 

I reported this bug 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694139] New: Review Request: openvas-gsd - Greenbone Security Desktop client application for OpenVAS

2011-04-06 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: openvas-gsd - Greenbone Security Desktop client 
application for OpenVAS

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

   Summary: Review Request: openvas-gsd - Greenbone Security
Desktop client application for OpenVAS
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: re...@seznam.cz
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Dear colleagues,
please I would like to ask you for review of a new openvas package -
openvas-gsd.

Spec URL: http://rebus.fedorapeople.org/SPECS/openvas-gsd.spec
SRPM URL: http://rebus.fedorapeople.org/SRPMS/openvas-gsd-1.1.1-1.fc14.src.rpm

Description:
The GSD is a desktop client that connects to the OpenVAS Manager using
the OMP protocol.

Please note that package name (openvas-gsd) is different from the original name
of the distribution tarball (gsd). I decided this to be like that to keep the
naming convention used for the resto of the openvas modules (openvas-libraries,
openvas-scanner, openvas-cli, openvas-manager, openvas-administrator).

Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=2977490

Best regards
Michal Ambroz

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691619] Review Request: openvas-manager - Open Vulnerability Assessment (OpenVAS) Manager (edit)

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


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

Michal Ambroz re...@seznam.cz changed:

   What|Removed |Added

   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=694139

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694139] Review Request: openvas-gsd - Greenbone Security Desktop client application for OpenVAS

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


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

Michal Ambroz re...@seznam.cz changed:

   What|Removed |Added

 CC||huzai...@redhat.com,
   ||re...@seznam.cz,
   ||stjepan.g...@gmail.com,
   ||xav...@bachelot.org
   See Also||https://bugzilla.redhat.com
   ||/show_bug.cgi?id=691619
  Alias||openvas-gsd

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 692733] Review Request: openvas-cli - Command-line tool to drive OpenVAS Manager

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


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

Michal Ambroz re...@seznam.cz changed:

   What|Removed |Added

  Alias||openvas-cli

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693664] Review Request: supybot-gribble - Cross-platform support bot based on supybot

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


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

Kevin Fenzi ke...@scrye.com changed:

   What|Removed |Added

 CC||ke...@scrye.com

--- Comment #2 from Kevin Fenzi ke...@scrye.com 2011-04-06 11:06:11 EDT ---
I can work on reviewing this in a while unless someone beats me to it. ;)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 687987] Review Request: plymouth-theme-hot-dog - Plymouth Happy Hot Dog Theme

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


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

Will Woods wwo...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(wwo...@redhat.com |
   |)   |

--- Comment #5 from Will Woods wwo...@redhat.com 2011-04-06 11:38:59 EDT ---
Sorry, got held up by other tasks. New spec uploaded to:
http://wwoods.fedorapeople.org/hot-dog/plymouth-theme-hot-dog.spec

No code changes, so there's no new RPMs.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 692257] Review Request: oct2spec - Python script to generate Octave package spec file

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


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

--- Comment #1 from Orion Poplawski or...@cora.nwra.com 2011-04-06 11:46:31 
EDT ---
http://www.cora.nwra.com/~orion/fedora/oct2spec-1.0.1-1.fc14.src.rpm

* Wed Apr 6 2011 Orion Poplawski or...@cora.nwra.com 1.0.1-1
- Update to 1.0.1 - add obsoletes to spec template

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691403] Review Request: perl-IO-Stty - Change and print terminal line settings

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


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

Martin Cermak mcer...@redhat.com changed:

   What|Removed |Added

 CC||mcer...@redhat.com

--- Comment #4 from Martin Cermak mcer...@redhat.com 2011-04-06 12:05:00 EDT 
---
Perfect, thanks for the package addition. It works nicely :-)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 688056] Review Request: sir - A simple application for resizing images

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


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

Arun SAG saga...@gmail.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #12 from Arun SAG saga...@gmail.com 2011-04-06 12:50:25 EDT ---
[+] OK
[X] NOT OKAY
[?] Investigate the issue before committing the package
[-] NA


[+] Package meets naming and packaging guidelines
[+] Spec file matches base package name.
[+] Spec has consistent macro usage.
[+] Meets Packaging Guidelines.
[+] License
[+] License field in spec matches
[+] License file included in package
[+] Spec in American English
[+] Spec is legible.
[+] Sources match upstream 
[zer0c00l@gnubox SPECS]$ md5sum ~/Downloads/sir_2.1.1.tar.gz 
bbb0526a8b828379449468066d166e04  /home/zer0c00l/Downloads/sir_2.1.1.tar.gz
[zer0c00l@gnubox SPECS]$ 


-- done
[-] Package needs ExcludeArch
[+] BuildRequires correct
[+] Spec handles locales/find_lang
[-] Package is relocatable and has a reason to be.
[+] Package has %defattr and permissions on files is good.
[-] Package has a correct %clean section.
[-] Package has correct buildroot
[+] Package is code or permissible content.
[-] Doc subpackage needed/used.
[+] Packages %doc files don't affect runtime.

[-] Headers/static libs in -devel subpackage.
[-] Spec has needed ldconfig in post and postun
[-] .pc files in -devel subpackage/requires pkgconfig
[-] .so files in -devel subpackage.
[-] -devel package Requires: %{name} = %{version}-%{release}
[-] .la files are removed.

[+] Package is a GUI app and has a .desktop file

[+] Package compiles and builds on at least one arch.
http://koji.fedoraproject.org/koji/taskinfo?taskID=2979037

[+] Package has no duplicate files in %files.
[+] Package doesn't own any directories other packages own.
[+] Package owns all the directories it creates.

[?] rpmlint output 
[zer0c00l@gnubox sir]$ rpmlint ~/rpmbuild/RPMS/i686/sir-2.1.1-3.fc14.i686.rpm 
sir.i686: W: no-manual-page-for-binary sir
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

[zer0c00l@gnubox sir]$ rpmlint
~/rpmbuild/RPMS/i686/sir-debuginfo-2.1.1-3.fc14.i686.rpm 
sir-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/sir/.moc
sir-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/sir/.moc
sir-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/sir/.ui
sir-debuginfo.i686: W: hidden-file-or-dir /usr/src/debug/sir/.ui
1 packages and 0 specfiles checked; 0 errors, 4 warnings.

Check the hidden-file-or-dir warning, ask the upstream about the purpose of
their existence before committing the package.

SHOULD Items:

[+] Should build in mock.
[+] Should build on all supported archs
[+] Should function as described.
[-] Should have sane scriptlets.
[-] Should have subpackages require base package with fully versioned depend.
[+] Should have dist tag
[+] Should package latest version
[-] check for outstanding bugs on package. (For core merge reviews)


XXX APPROVED XXX

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

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


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

--- Comment #7 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
13:25:47 EDT ---
imap-upload-1.2-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/imap-upload-1.2-3.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

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


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

--- Comment #8 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
13:32:39 EDT ---
imap-upload-1.2-3.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/imap-upload-1.2-3.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 652987] Review Request: go - The Go programming language

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


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

Renich Bon Ciric ren...@woralelandia.com changed:

   What|Removed |Added

   Flag|needinfo?   |

--- Comment #38 from Renich Bon Ciric ren...@woralelandia.com 2011-04-06 
13:50:20 EDT ---
(In reply to comment #37)
 Just a question -- who'll be the primary maintainer of this package? I see
 specs from both Renich and Michael.

I'm a total newbie on this. I just got sponsored for synapse and libzeitgeist.
I have no objection on being co-maintainer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676175] Review Request: fpaste-server - Fedora Pastebin

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


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

--- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:56:17 EDT 
---
I see no SCM request to process.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #9 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:57:19 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675963] Review Request: perl-MDV-Packdrakeng - Simple Archive Extractor/Builder

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


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

--- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:55:57 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 399721] Review Request: fusion-icon - Compiz Fusion panel applet

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


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

--- Comment #15 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:55:37 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 691972] Review Request: zanata-python-client - library and client for working with Zanata server

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


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

--- Comment #11 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:59:15 EDT 
---
This ticket seems to be assigned to the owner of the package.  It should be
assigned to the reviewer, and you can't review your own packages.

Please fix and re-raise the fedora-cvs flag.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693493] Review Request: thunderbird-lightning - The calendar extension to Thunderbird

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


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

--- Comment #7 from Orion Poplawski or...@cora.nwra.com 2011-04-06 14:05:38 
EDT ---
(In reply to comment #6)
 + BAD : rpmlint is silent on both source and binary package.
 It isn't.
 bradford:build $ rpmlint -i thunderbird-lightning-1.0-0.39.b3pre.fc14.src.rpm
 RPMS/x86_64/thunderbird-lightning-*
 thunderbird-lightning.src: W: strange-permission thunderbird-mozconfig-branded
 0755L

Fixed.  Note that this was copied from the current thunderbird package so is an
issue there.

 thunderbird-lightning.src: W: strange-permission find-external-requires 0755L
 A file that you listed to include in your package has strange permissions.
 Usually, a file should have 0644 permissions.

I think this has to be executable to work.

 thunderbird-lightning.src:19: W: macro-in-comment %define
 There is a unescaped macro after a shell style comment in the specfile. Macros
 are expanded everywhere, so check if it can cause a problem in this case and
 escape the macro with another leading % if appropriate.

Fixed.

 thunderbird-lightning.src: W: patch-not-applied Patch0:
 thunderbird-version.patch
 A patch is included in your package but was not applied. Refer to the patches
 documentation to see what's wrong.

False positive.  Again, same issue in thunderbird package.  We modify the patch
before applying it.

 thunderbird-lightning.src: W: %ifarch-applied-patch Patch6:
 mozilla-build-s390.patch
 A patch is applied inside an %ifarch block. Patches must be applied on all
 architectures and may contain necessary configure and/or code patch to be
 effective only on a given arch.

Fixed.

 thunderbird-lightning-debuginfo.x86_64: E: empty-debuginfo-package

I'm a bit at a loss.  This appears to be because libcalbasecomps.so is getting
stripped by the mozilla build/install process, but I can't figure out where. 
Any help would be appreciated.

 thunderbird-lightning.x86_64: E: script-without-shebang
 /usr/lib64/mozilla/extensions/{3550f703-e582-4d05-9a08-453d09bdfdc6}/{e2fda1a4-762b-4020-b5ad-a41df1933103}/calendar-js/calFreeBusyService.js
 thunderbird-lightning.x86_64: E: script-without-shebang
 /usr/lib64/mozilla/extensions/{3550f703-e582-4d05-9a08-453d09bdfdc6}/{e2fda1a4-762b-4020-b5ad-a41df1933103}/calendar-js/calAlarmMonitor.js

Fixed.

 + BAD : Runs rm -rf $RPM_BUILD_ROOT in %install
 It doesn't.

Not needed in F15, but since we have requests for F13/F14 I added it.

http://www.cora.nwra.com/~orion/fedora/thunderbird-lightning.spec
http://www.cora.nwra.com/~orion/fedora/thunderbird-lightning-1.0-0.40.b3pre.fc14.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693363] Review Request: st - A simple terminal implementation for X

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


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

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

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs-

--- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 14:10:13 EDT 
---
 - packages must not own files or directories already owned by other packages 
 OK

Erm, really?  How did you actually check?  I thought /usr/bin/st might be
conflicting and indeed:
  repoquery --whatprovides /usr/bin/st
shows that it is (openstack-swift provides that file already).

Arguably this is an issue in openstack-swift, but at this point it's no safe
bet that any two-letter command is non-conflicting.

Please work this out and re-raise the fedora-cvs flag when that's been 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676216] Review Request: sugar-calendario - agenda/calendar for sugar

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


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

--- Comment #11 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:57:04 EDT 
---
Do you really not want an F15 branch?

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690359] Review Request: perl-Shipwright - Build and Manage Self-contained Software Bundle

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


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

--- Comment #9 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 13:57:35 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #10 from Jon Ciesla l...@jcomserv.net 2011-04-06 14:03:40 EDT ---
Awesome, 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 652987] Review Request: go - The Go programming language

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


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

--- Comment #39 from W. Michael Petullo m...@flyn.org 2011-04-06 14:19:03 EDT 
---
I would be happy to maintain this package. I maintain several Fedora packages
and anticipate using Go a lot in the near future. I volunteer to be the primary
maintainer with Renich as a co-maintainer.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676216] Review Request: sugar-calendario - agenda/calendar for sugar

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


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

--- Comment #12 from Yader Velásquez yajo...@gmail.com 2011-04-06 14:24:08 
EDT ---
New Package SCM Request
===
Package Name: sugar-calendario
Short Description: agenda/calendar for sugar
Owners: yaderv
Branches: f14 f15
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review

[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 14:48:38 EDT ---
drupal7-7.0-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/drupal7-7.0-4.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 407571] Review Request: tinyxml - A simple, small, C++ XML parser

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


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

--- Comment #28 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 14:48:03 EDT ---
tinyxml-2.6.1-2.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/tinyxml-2.6.1-2.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #14 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 14:48:45 EDT ---
drupal7-7.0-4.el5 has been submitted as an update for Fedora EPEL 5.
https://admin.fedoraproject.org/updates/drupal7-7.0-4.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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #11 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 14:48:23 EDT ---
drupal7-7.0-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/drupal7-7.0-4.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 679820] Review Request: drupal7 - An open-source content-management platform

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


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

--- Comment #12 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 14:48:30 EDT ---
drupal7-7.0-4.el6 has been submitted as an update for Fedora EPEL 6.
https://admin.fedoraproject.org/updates/drupal7-7.0-4.el6

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 652987] Review Request: go - The Go programming language

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


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

--- Comment #40 from Renich Bon Ciric ren...@woralelandia.com 2011-04-06 
15:20:45 EDT ---
(In reply to comment #39)
 I would be happy to maintain this package. I maintain several Fedora packages
 and anticipate using Go a lot in the near future. I volunteer to be the 
 primary
 maintainer with Renich as a co-maintainer.

This would be awesome! ;)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676175] Review Request: fpaste-server - Fedora Pastebin

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


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

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

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Clint Savage her...@gmail.com 2011-04-06 15:54:46 EDT ---
New Package SCM Request
===
Package Name: fpaste-server
Short Description: Fedora Pastebin Server
Owners: herlo, jsteffan
Branches: f14, f15, el6
InitialCC: nb

Sorry, got distracted on the first attempt at this.

Clint

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Client library for zeitgeist

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Client library for zeitgeist

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


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

--- Comment #38 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 16:30:12 EDT ---
libzeitgeist-0.3.6-4.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/libzeitgeist-0.3.6-4.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 674188] Review Request: libzeitgeist - Client library for zeitgeist

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


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

--- Comment #39 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 16:34:42 EDT ---
libzeitgeist-0.3.6-4.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/libzeitgeist-0.3.6-4.fc15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694287] Review Request: openCOLLADA - 3D import and export libraries

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


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

Richard Shaw hobbes1...@gmail.com changed:

   What|Removed |Added

 Blocks||177841(FE-NEEDSPONSOR)

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694287] New: Review Request: openCOLLADA - 3D import and export libraries

2011-04-06 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: openCOLLADA - 3D import and export libraries

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

   Summary: Review Request: openCOLLADA - 3D import and export
libraries
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: hobbes1...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-rev...@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Story Points: ---


Spec URL:
https://docs.google.com/leaf?id=0B4A6of0Rl4nUZWVlOWE3ZGEtNDdiMi00MWZmLWE5NmQtOWQ5YTdmYTY5YWFkhl=en
SRPM URL:
https://docs.google.com/leaf?id=0B4A6of0Rl4nUMzczZmZmOGEtZGRlOC00NDdmLWFhOTUtMDY3MDRiYzlkM2U1hl=en
Description: 
COLLADA is a royalty-free XML schema that enables digital asset
exchange within the interactive 3D industry.
OpenCOLLADA is a Google summer of code opensource project providing
libraries for 3D file interchange between applications like blender.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676175] Review Request: fpaste-server - Fedora Pastebin

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


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

--- Comment #12 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 17:04:35 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 676216] Review Request: sugar-calendario - agenda/calendar for sugar

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


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

--- Comment #13 from Jason Tibbitts ti...@math.uh.edu 2011-04-06 17:05:13 EDT 
---
Git done (by process-git-requests).

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 690954] Review Request: postler - An ultra simple desktop mail client

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


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

Thomas Moschny thomas.mosc...@gmx.de changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #16 from Thomas Moschny thomas.mosc...@gmx.de 2011-04-06 17:22:23 
EDT ---
New Package SCM Request
===
Package Name: postler
Short Description: An ultra simple desktop mail client
Owners: thm
Branches: f14 f15

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 570569] Review Request: drupal-ldap_integration - Drupal ldapauth, ldapgroups, and ldapdata modules

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


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

--- Comment #4 from Orion Poplawski or...@cora.nwra.com 2011-04-06 17:45:25 
EDT ---
http://www.cora.nwra.com/~orion/fedora/drupal6-ldap_integration.spec
http://www.cora.nwra.com/~orion/fedora/drupal6-ldap_integration-1.0-0.2.beta2.fc14.src.rpm

* Wed Apr 6 2011 Orion Poplawski or...@cora.nwra.com - 1.0-0.2.beta2
- Update to latest drupal template
- Update license

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 694287] Review Request: openCOLLADA - 3D import and export libraries

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


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

Nicolas Chauvet (kwizart) kwiz...@gmail.com changed:

   What|Removed |Added

 CC||kwiz...@gmail.com

--- Comment #1 from Nicolas Chauvet (kwizart) kwiz...@gmail.com 2011-04-06 
18:18:08 EDT ---
This is an informal review since I cannot sponsor you.



1/ Can you show the output of rpmlint on the src.rpm and the installed package.
Please fix the empty spaces in the middle of the lines for the %description

2/ Please remove Everything that is not usefull in fedora. This reduce
readability.
(specially everything before the 'Name' field as this is not usefull in
fedora).

3/ The version field is wrong, please follow:
https://fedoraproject.org/wiki/Packaging:Guidelines#Version_and_Release


4/ Uneeded BR: gcc-c++ - It is added by default, please remove.

5/ Buildroot is undeed nowdays:
https://fedoraproject.org/wiki/Packaging:Guidelines#BuildRoot_tag

6/ Remove spurious 'echo %{buildir}'

7/ rpm -E %{?jobsflag} is empty. fedora uses %{?_smp_mflags} instead. Please
remove the previous rpm macro.

8/ Where does CHANGES.TXT come from ? Please add an url if possible
---
# copy CHANGES.txt
cp %{S:1} ./ 
---
It's better to use install -p

9/ About: 
# Add some docs, need to fix eol encoding with dos2unix in some files
- Please move this in %prep section

10/# Manual install ...
Are you sure this cannot be installed with scons ? You should fix it within
scons files then.

Here a preliminary pass for the 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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 624205] Review Request: meego-help - MeeGo documentation and help.

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


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

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

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
Last Closed||2011-04-06 18:17:28

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675426] Review Request: imap-upload - A tool for uploading a local mbox file to IMAP4 server

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
18:29:11 EDT ---
imap-upload-1.2-3.fc14 has been pushed to the Fedora 14 testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 693042] Review Request: perl-MooseX-ChainedAccessors - Accessor class for chained accessors with Moose

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #6 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
18:27:46 EDT ---
perl-MooseX-ChainedAccessors-0.02-1.fc14 has been pushed to the Fedora 14
testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 672418] Review Request: fpdns - Fingerprint DNS

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #13 from Fedora Update System upda...@fedoraproject.org 
2011-04-06 18:29:39 EDT ---
fpdns-0.9.3-1.fc14 has been pushed to the Fedora 14 testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675649] Review Request: perl-Log-Any-Adapter-Dispatch - Log::Any::Adapter::Dispatch Perl module

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


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

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
18:34:28 EDT ---
perl-Log-Any-Adapter-Dispatch-0.06-1.fc14 has been pushed to the Fedora 14
stable repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 689432] Review Request: collada-dom - COLLADA Document Object Model Library

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|MODIFIED|ON_QA

--- Comment #9 from Fedora Update System upda...@fedoraproject.org 2011-04-06 
18:34:53 EDT ---
collada-dom-2.3-2.fc14 has been pushed to the Fedora 14 testing repository.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675649] Review Request: perl-Log-Any-Adapter-Dispatch - Log::Any::Adapter::Dispatch Perl module

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Log-Any-Adapter-Dispat |perl-Log-Any-Adapter-Dispat
   |ch-0.06-1.fc15  |ch-0.06-1.fc14

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


[Bug 675649] Review Request: perl-Log-Any-Adapter-Dispatch - Log::Any::Adapter::Dispatch Perl module

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


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

Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

   Fixed In Version|perl-Log-Any-Adapter-Dispat |perl-Log-Any-Adapter-Dispat
   |ch-0.06-1.fc14  |ch-0.06-1.fc13

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review


  1   2   >