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

2011-08-11 Thread bugzilla
Please do not reply directly 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

Simon Thompson  changed:

   What|Removed |Added

 CC||s...@roamingzebra.co.uk

--- Comment #38 from Simon Thompson  2011-08-11 
03:41:57 EDT ---
I just tried building this in mock on my machine and it fell over:

DEBUG: In file included from
/builddir/build/BUILD/assimp/code/irrXMLWrapper.h:45In file included from
/builddir/build/BUILD/assimp/code/DXFLoader.cpp:52:
DEBUG: /usr/include/irrlicht/fast_atof.h:8:21: error: irrMath.h: No such file
or directory
DEBUG: In file included from
/builddir/build/BUILD/assimp/code/CSMLoader.cpp:53:
DEBUG: /usr/include/irrlicht/fast_atof.h:8:21: error: irrMath.h: No such file
or directory

Was building against irrlicht-1.7.2-8.fc17 built locally.

Adding:
BuildRequires:  irrlicht-devel
to assimp results in a successful build.

-- 
Configure 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 664980] Review Request: perl-Test-TCP - Testing TCP program

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


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

--- Comment #8 from Jon Ciesla  2011-08-11 04:19:23 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 575491] Review Request: perl-Test-SharedFork - Fork test

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


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

--- Comment #6 from Jon Ciesla  2011-08-11 04:17: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 575493] Review Request: perl-Hash-MultiValue - Store multiple values per key

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


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

--- Comment #7 from Jon Ciesla  2011-08-11 04:18:31 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 575492] Review Request: perl-Filesys-Notify-Simple - Simple and dumb file system watcher

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


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

--- Comment #6 from Jon Ciesla  2011-08-11 04:17:59 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 581321] Review Request: perl-Devel-StackTrace-AsHTML - Displays a stack trace in HTML

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


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

--- Comment #6 from Jon Ciesla  2011-08-11 04:18:53 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 670477] Review Request: perl-HTTP-Server-Simple-PSGI - PSGI handler for HTTP::Server::Simple

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


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

--- Comment #12 from Jon Ciesla  2011-08-11 04:19:51 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 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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


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

--- Comment #11 from Jon Ciesla  2011-08-11 04:16:42 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 the QA contact 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-08-11 Thread bugzilla
Please do not reply directly 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

--- Comment #8 from Jon Ciesla  2011-08-11 04:21:03 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 729669] Review Request: nafees-tehreer-naskh-fonts - Nafees tehreer naskh font for writing Urdu in the naskh script

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2011-08-11 04:32:38 EDT 
---
Review:

+ Koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3265344
+ Source is verified as (sha1sum)
99b05c59a0225bf2970f8221a6eace03e21d30a0  Nafees_Tahreer_Naskh_v1.0.zip
99b05c59a0225bf2970f8221a6eace03e21d30a0 
../SOURCES/Nafees_Tahreer_Naskh_v1.0.zip

+ rpmlint output gave
nafees-tehreer-naskh-fonts.src: I: checking
nafees-tehreer-naskh-fonts.src: I: checking-url http://www.crulp.org/index.htm
(timeout 10 seconds)
nafees-tehreer-naskh-fonts.src: I: checking-url
http://www.crulp.org/Downloads/localization/fonts/NafeesTahreerNaskh/Nafees_Tahreer_Naskh_v1.0.zip
(timeout 10 seconds)
nafees-tehreer-naskh-fonts.noarch: I: checking
nafees-tehreer-naskh-fonts.noarch: I: checking-url
http://www.crulp.org/index.htm (timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ follows packaging guidelines


Suggestions:
1) remove buildroot tag
2) remove cleaning of buildroot in %build
3) remove %clean section


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 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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


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

--- Comment #12 from Fedora Update System  
2011-08-11 04:56:18 EDT ---
perl-JSON-2.17-1.el4 has been submitted as an update for Fedora EPEL 4.
https://admin.fedoraproject.org/updates/perl-JSON-2.17-1.el4

-- 
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 678221] Review Request: perl-EV - Wrapper for the libev high-performance event loop library

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


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

Mathieu Bridon  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?

--- Comment #10 from Mathieu Bridon  2011-08-11 
04:59:50 EDT ---
Package Change Request
==
Package Name: perl-EV
New Branches: el6
Owners: bochecha
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 729674] Review Request: nafees-pakistani-naskh-fonts - Nafees pakistani naskh font for writing Urdu in the Naskh script

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2011-08-11 04:59:08 EDT 
---
Review:

+ Koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3265340
+ Source is verified as (sha1sum)
49c89321b2ce8211becabd819ac91d88fbc79f15  Nafees_Pakistani_Naskh_v2.01.zip
49c89321b2ce8211becabd819ac91d88fbc79f15 
../SOURCES/Nafees_Pakistani_Naskh_v2.01.zip

+ rpmlint output gave
nafees-pakistani-naskh-fonts.src: I: checking
nafees-pakistani-naskh-fonts.src: I: checking-url
http://www.crulp.org/index.htm (timeout 10 seconds)
nafees-pakistani-naskh-fonts.src: I: checking-url
http://www.crulp.org/Downloads/localization/fonts/NafeesPakistaniNaskh/Nafees_Pakistani_Naskh_v2.01.zip
(timeout 10 seconds)
nafees-pakistani-naskh-fonts.noarch: I: checking
nafees-pakistani-naskh-fonts.noarch: I: checking-url
http://www.crulp.org/index.htm (timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ follows packaging guidelines


Suggestions:
1) remove buildroot tag
2) remove cleaning of buildroot in %build
3) remove %clean section

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

[Bug 729666] Review Request: nafees-nastaleeq-fonts - Nafees nastaleeq font for writing Urdu in the Nastaleeq script

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2011-08-11 04:59:26 EDT 
---
Review:

+ Koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3265338
+ Source is verified as (sha1sum)
58636b076c1277037ab7e75bc5370335586feebc  Nafees_Nastaleeq_v1.02.zip
58636b076c1277037ab7e75bc5370335586feebc  ../SOURCES/Nafees_Nastaleeq_v1.02.zip

+ rpmlint output gave
nafees-nastaleeq-fonts.src: I: checking
nafees-nastaleeq-fonts.src: W: spelling-error %description -l en_US leeq ->
liq, lee, lees
The value of this tag appears to be misspelled. Please double-check.

nafees-nastaleeq-fonts.src: W: spelling-error %description -l en_US glyphs ->
glyph, glyph s
The value of this tag appears to be misspelled. Please double-check.

nafees-nastaleeq-fonts.src: I: checking-url http://www.crulp.org/index.htm
(timeout 10 seconds)
nafees-nastaleeq-fonts.src: I: checking-url
http://www.crulp.org/Downloads/localization/fonts/NafeesNastaleeq/Nafees_Nastaleeq_v1.02.zip
(timeout 10 seconds)
nafees-nastaleeq-fonts.noarch: I: checking
nafees-nastaleeq-fonts.noarch: W: spelling-error %description -l en_US leeq ->
liq, lee, lees
The value of this tag appears to be misspelled. Please double-check.

nafees-nastaleeq-fonts.noarch: W: spelling-error %description -l en_US glyphs
-> glyph, glyph s
The value of this tag appears to be misspelled. Please double-check.

nafees-nastaleeq-fonts.noarch: I: checking-url http://www.crulp.org/index.htm
(timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 4 warnings.

+ follows packaging guidelines


Suggestions:
1) remove buildroot tag
2) remove cleaning of buildroot in %build
3) remove %clean section

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

[Bug 729671] Review Request: nafees-riqa-fonts - Nafees riqa font for writing Urdu in the Riqa script

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2011-08-11 04:58:58 EDT 
---
Review:

+ Koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3265342
+ Source is verified as (sha1sum)
fb03dd257ca68fd274f5870b95ace4c46d4c9af6  Nafees_Riqa_v1.0.zip
fb03dd257ca68fd274f5870b95ace4c46d4c9af6  ../SOURCES/Nafees_Riqa_v1.0.zip

+ rpmlint output gave
nafees-riqa-fonts.src: I: checking
nafees-riqa-fonts.src: I: checking-url http://www.crulp.org/index.htm (timeout
10 seconds)
nafees-riqa-fonts.src: I: checking-url
http://www.crulp.org/Downloads/localization/fonts/NafeesRiqa/Nafees_Riqa_v1.0.zip
(timeout 10 seconds)
nafees-riqa-fonts.noarch: I: checking
nafees-riqa-fonts.noarch: I: checking-url http://www.crulp.org/index.htm
(timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 0 warnings.

+ follows packaging guidelines


Suggestions:
1) remove buildroot tag
2) remove cleaning of buildroot in %build
3) remove %clean section

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

[Bug 729651] Review Request: nafees-naskh-fonts - Nafees naskh font for writing Urdu in the Naskh script with full aerab support

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


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

Parag AN(पराग)  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #1 from Parag AN(पराग)  2011-08-11 04:59:17 EDT 
---
Review:

+ Koji build ->http://koji.fedoraproject.org/koji/taskinfo?taskID=3265336
+ Source is verified as (sha1sum)
8039809dc660961f6665c5ef25361d7f6fd7a25f  Nafees_Naskh_v2.01.zip
8039809dc660961f6665c5ef25361d7f6fd7a25f  ../SOURCES/Nafees_Naskh_v2.01.zip

+ rpmlint output gave
nafees-naskh-fonts.src: I: checking
nafees-naskh-fonts.src: W: spelling-error Summary(en_US) aerab -> aerate
The value of this tag appears to be misspelled. Please double-check.

nafees-naskh-fonts.src: W: spelling-error %description -l en_US aerab -> aerate
The value of this tag appears to be misspelled. Please double-check.

nafees-naskh-fonts.src: I: checking-url http://www.crulp.org/index.htm (timeout
10 seconds)
nafees-naskh-fonts.src: I: checking-url
http://www.crulp.org/Downloads/localization/fonts/NafeesNaskh/Nafees_Naskh_v2.01.zip
(timeout 10 seconds)
nafees-naskh-fonts.noarch: I: checking
nafees-naskh-fonts.noarch: W: spelling-error Summary(en_US) aerab -> aerate
The value of this tag appears to be misspelled. Please double-check.

nafees-naskh-fonts.noarch: W: spelling-error %description -l en_US aerab ->
aerate
The value of this tag appears to be misspelled. Please double-check.

nafees-naskh-fonts.noarch: I: checking-url http://www.crulp.org/index.htm
(timeout 10 seconds)
2 packages and 0 specfiles checked; 0 errors, 4 warnings.
+ follows packaging guidelines


Suggestions:
1) remove buildroot tag
2) remove cleaning of buildroot in %build
3) remove %clean section

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

[Bug 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

Steve Traylen  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #9 from Steve Traylen  2011-08-11 05:10:45 
EDT ---

Indeed I was on NVIDEA, does not work from a Mac either but it's fine on an
intel card.

You did not switch from chrpath to a sed of libtool and

http://fedoraproject.org/wiki/Packaging:Guidelines#Removing_Rpath

says to use chrpath as a last resort. Having said that the end result
is good so I leave to you.

APPROVED.

I'd like to see these in EPEL (5 and 6) as well and am happy to maintain them
if you do not want
to do this? Let me know.

Steve.

-- 
Configure 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 630303] Review Request: yesod - Creation of type-safe, RESTful web applications

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


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

Jens Petersen  changed:

   What|Removed |Added

 Blocks|634048(Haskell-pkg-reviews) |

--- Comment #6 from Jens Petersen  2011-08-11 05:41:37 EDT 
---
Latest yesod release is 0.8.2.1 and 0.9 is coming out soon.

Ok, I moving yesod off Haskell-pkg-reviews for now
until is gets updated.  I think it will probably be
better to do it in parts since the whole dependency tree
is really large, maybe starting to yesod-core
or even taking library packages from the bottom of the
dependency tree to Haskell-pkg-reviews.  That will
make it easier eventually to get the whole of yesod reviewed.

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


[Bug 630303] Review Request: yesod - Creation of type-safe, RESTful web applications

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


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

Jens Petersen  changed:

   What|Removed |Added

 Depends on||712272(ghc-unix-compat)

-- 
Configure 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 712272] Review Request: ghc-unix-compat - A portable POSIX-compatibility layer for Haskell

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


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

Jens Petersen  changed:

   What|Removed |Added

 Blocks||630303(yesod)

-- 
Configure 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 712272] Review Request: ghc-unix-compat - A portable POSIX-compatibility layer for Haskell

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


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

Jens Petersen  changed:

   What|Removed |Added

 CC||peter...@redhat.com

--- Comment #1 from Jens Petersen  2011-08-11 05:54:12 EDT 
---
I'd like to review this.
Lakshmi, do you have time to update 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 729926] New: Review Request: httpserver - JBoss httpserver

2011-08-11 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: httpserver - JBoss httpserver

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

   Summary: Review Request: httpserver - JBoss httpserver
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgold...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://goldmann.fedorapeople.org/package_review/httpserver/1/httpserver.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/httpserver/1/httpserver-1.0.0-0.1.Beta1.fc17.src.rpm
Description: This package contains JBoss httpserver

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

$ rpmlint ./httpserver.spec 
./httpserver.spec: W: invalid-url Source0: httpserver-1.0.0.Beta1.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint httpserver-1.0.0-0.1.Beta1.fc17.src.rpm 
httpserver.src: I: enchant-dictionary-not-found en_US
httpserver.src: W: name-repeated-in-summary C httpserver
httpserver.src: W: invalid-url Source0: httpserver-1.0.0.Beta1.tar.xz
1 packages and 0 specfiles checked; 0 errors, 2 warnings.

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


[Bug 729926] Review Request: httpserver - JBoss httpserver

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


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

Marek Goldmann  changed:

   What|Removed |Added

 Blocks||652183(FE-JAVASIG)

-- 
Configure 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 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

Paul Howarth  changed:

   What|Removed |Added

 CC||p...@city-fan.org

--- Comment #1 from Paul Howarth  2011-08-11 06:17:40 EDT ---
spot.fedoraproject.org => NXDOMAIN

-- 
Configure 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 717867] Review Request: ghc-http-types - Generic HTTP types for Haskell

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


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

Jens Petersen  changed:

   What|Removed |Added

 Blocks||634048(Haskell-pkg-reviews)

-- 
Configure 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 729926] Review Request: jboss-httpserver - JBoss httpserver

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


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

Marek Goldmann  changed:

   What|Removed |Added

Summary|Review Request: httpserver  |Review Request:
   |- JBoss httpserver  |jboss-httpserver - JBoss
   ||httpserver

-- 
Configure 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 729926] Review Request: httpserver - JBoss httpserver

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


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

--- Comment #1 from Marek Goldmann  2011-08-11 06:30:44 
EDT ---
Package renamed to jboss-httpserver:

Spec URL:
http://goldmann.fedorapeople.org/package_review/httpserver/2/jboss-httpserver.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/httpserver/2/jboss-httpserver-1.0.0-0.2.Beta1.fc17.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 635511] Review Request: assimp - Library to import various 3D model formats into applications

2011-08-11 Thread bugzilla
Please do not reply directly 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

Simon Thompson  changed:

   What|Removed |Added

 Blocks||725752

-- 
Configure 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 725752] Review Request: gazebo - 3D multiple robot simulator with dynamics

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


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

Simon Thompson  changed:

   What|Removed |Added

 Depends on||635511

--- Comment #6 from Simon Thompson  2011-08-11 
07:23:27 EDT ---

I found that 0.9.0 wouldn't build against OGRE 1.7.x so have bumped version to
0.10.0.

Spec URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.10.0-1.spec
SRPM URL: http://www.cs.bham.ac.uk/~sjt/rpm/gazebo-0.10.0-1.el6.src.rpm

Its patched to use the external assimp library. There's options for --with
ffmpeg and --with bullet - it currently won't build with the Fedora bullet
package though as it requires changes:
  OPTION(USE_DOUBLE_PRECISION "Use double precision" ON)
  OPTION(INSTALL_LIBS "Set when you want to install libraries" ON)
in bullet's CMakeLists.txt

There's a couple of patches in there to make it build and to install
unversioned .so files into /usr/lib/package style as well a patch to make it
work on x86_64 style machines (based on the architecture detection used in
player) so that lib64 is substituted appropriately.

Adding bug 635511 (assimp) as blocking this.

Any comments on this?

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


[Bug 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

--- Comment #2 from Alexander Kurtakov  2011-08-11 
08:18:28 EDT ---
Package Review
==

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

=== REQUIRED ITEMS ===
[!]  Rpmlint output:
maven-reporting-exec.i686: E: no-binary
maven-reporting-exec-debuginfo.i686: E: empty-debuginfo-package
I see no reason for this package to not be noarch
[x]  Package is named according to the Package Naming Guidelines[1].
[x]  Spec file name must match the base package name, in the format
%{name}.spec.
[x]  Package meets the Packaging Guidelines[2].
[x]  Package successfully compiles and builds into binary rpms.
[x]  Buildroot definition is not present
[x]  Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging
Guidelines[3,4].
[x]  License field in the package spec file matches the actual license.
License type: ASL 2.0
[x]  If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x]  All independent sub-packages have license of their own
[x]  Spec file is legible and written in American English.
[x]  Sources used to build the package matches the upstream source, as provided
in the spec URL.
MD5SUM this package:ddc79b59dcacc6d6d8ff8951b422f1f8 
MD5SUM upstream package:ddc79b59dcacc6d6d8ff8951b422f1f8 
[x]  All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines[5].
[x]  Package must own all directories that it creates.
[x]  Package requires other packages for directories it uses.
[x]  Package does not contain duplicates in %files.
[x]  File sections do not contain %defattr(-,root,root,-) unless changed with
good reason
[x]  Permissions on files are set properly.
[x]  Package does NOT have a %clean section which contains rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). (not needed anymore)
[x]  Package consistently uses macros (no %{buildroot} and $RPM_BUILD_ROOT
mixing)
[x]  Package contains code, or permissable content.
[x]  Fully versioned dependency in subpackages, if present.
[x]  Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x]  Package does not own files or directories owned by other packages.
[x]  Javadoc documentation files are generated and included in -javadoc
subpackage
[x]  Javadocs are placed in %{_javadocdir}/%{name} (no -%{version} symlinks)
[x]  Packages have proper BuildRequires/Requires on jpackage-utils
[x]  Javadoc subpackages have Require: jpackage-utils
[x]  Package uses %global not %define
[-]  If package uses tarball from VCS include comment how to re-create that
tarball (svn export URL, git clone URL, ...)
[-]  If source tarball includes bundled jar/class files these need to be
removed prior to building
[x]  All filenames in rpm packages must be valid UTF-8.
[x]  Jar files are installed to %{_javadir}/%{name}.jar (see [6] for details)
[x]  If package contains pom.xml files install it (including depmaps) even when
building with ant
[x]  pom files has correct add_maven_depmap

=== Maven ===
[x]  Use %{_mavenpomdir} macro for placing pom files instead of
%{_datadir}/maven2/poms
[x]  If package uses "-Dmaven.test.skip=true" explain why it was needed in a
comment
[x]  If package uses custom depmap "-Dmaven.local.depmap.file=*" explain why
it's needed in a comment
[x]  Package DOES NOT use %update_maven_depmap in %post/%postun
[x]  Packages DOES NOT have Requires(post) and Requires(postun) on
jpackage-utils for %update_maven_depmap macro

=== Other suggestions ===
[x]  If possible use upstream build method (maven/ant/javac)
[x]  Avoid having BuildRequires on exact NVR unless necessary
[!]  Package has BuildArch: noarch (if possible)
[x]  Latest version is packaged.
[x]  Reviewer should test that the package builds in mock.

=== Issues ===
1. Is there a reason for this package to not be noarch. I don't see any please
fix it unless you have a reason

-- 
Configure 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 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

Emmanuel Seyman  changed:

   What|Removed |Added

   Keywords||Security
 Status|NEW |ASSIGNED
  Group||security
 CC||emmanuel.seyman@club-intern
   ||et.fr,
   ||security-response-team@redh
   ||at.com
 AssignedTo|nob...@fedoraproject.org|emmanuel.seyman@club-intern
   ||et.fr
   Flag||fedora-review?

--- Comment #2 from Emmanuel Seyman  
2011-08-11 08:30:59 EDT ---
I'm going to pretend that reads :

Spec URL: http://spot.fedorapeople.org/perl-Switch.spec
SRPM URL: http://spot.fedorapeople.org/perl-Switch-2.16-1.fc15.src.rpm

Taking.

-- 
Configure 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 729970] Review Request: jandex - Java Annotation Indexer

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


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

Marek Goldmann  changed:

   What|Removed |Added

 Blocks||652183(FE-JAVASIG)

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

--- Comment #3 from Tomas Radej  2011-08-11 08:33:00 EDT ---
Created attachment 517788
  --> https://bugzilla.redhat.com/attachment.cgi?id=517788
Fixed BuildArch -> noarch

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


[Bug 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

Tomas Radej  changed:

   What|Removed |Added

 Attachment #517788|Fixed BuildArch -> noarch   |maven-reporting-exec-1.0.1-
description||1.src.rpm Fixed BuildArch
   ||-> noarch

--- Comment #4 from Tomas Radej  2011-08-11 08:35:32 EDT ---
Comment on attachment 517788
  --> https://bugzilla.redhat.com/attachment.cgi?id=517788
maven-reporting-exec-1.0.1-1.src.rpm Fixed BuildArch -> noarch

Fixed Issue 1 - changed architecture to noarch

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


[Bug 690728] Review Request: Nitrate - A test case management system written in Django

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


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

--- Comment #52 from James Laska  2011-08-11 08:34:30 EDT ---
(In reply to comment #51)
> Ah, thank you James for that information.
> Are javascript frameworks mentioned there somewhere?

You know, I was wondering that too.  I don't see them explicitly mentioned, and
it's hard to determine where the line is between a collection of .js files
(libraries) and a framework.  We can certainly raise this topic on packaging@
for clarification.  Some initial poking I did gave me the same impression you
note later on ... that this is just the *current* policy, and it's subject to
change.

> So then it sounds like the separate packaging I requested
> is not strictly necessary?  Anyway I glad that tinymce
> was packaged up.
> 
> If Fedora later changes the bundling policy for website javascript,
> I think it may be hard work to clean up...

Agreed!  If js frameworks are okay to bundle for now, I think it would still be
wise to at least start package reviews for them ... but not require their
completion for this review?  Thoughts/comments/concerns?

-- 
Configure 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 729970] New: Review Request: jandex - Java Annotation Indexer

2011-08-11 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: jandex - Java Annotation Indexer

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

   Summary: Review Request: jandex - Java Annotation Indexer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mgold...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://goldmann.fedorapeople.org/package_review/jandex/1/jandex.spec
SRPM URL:
http://goldmann.fedorapeople.org/package_review/jandex/1/jandex-1.0.0-1.fc17.src.rpm
Description: This package contains Java Annotation Indexer

$ rpmlint ./jandex.spec 
./jandex.spec: W: invalid-url Source0: jandex-1.0.0.Final.tar.xz
0 packages and 1 specfiles checked; 0 errors, 1 warnings.

$ rpmlint jandex-1.0.0-1.fc17.src.rpm 
jandex.src: I: enchant-dictionary-not-found en_US
jandex.src: W: invalid-url Source0: jandex-1.0.0.Final.tar.xz
1 packages and 0 specfiles checked; 0 errors, 1 warnings.

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

-- 
Configure 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 729972] New: Review Request: perl-Test-Mojibake - Check your source for encoding misbehavior

2011-08-11 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-Test-Mojibake - Check your source for encoding 
misbehavior

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

   Summary: Review Request: perl-Test-Mojibake - Check your source
for encoding misbehavior
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: p...@city-fan.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-Test-Mojibake/branches/fedora/perl-Test-Mojibake.spec
SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Test-Mojibake/perl-Test-Mojibake-0.3-2.fc17.src.rpm

Description:

Many modern text editors automatically save files using UTF-8 codification.
However, the perl interpreter does not expect it by default. Whilst this does
not represent a big deal on (most) backend-oriented programs, Web framework
(Catalyst, Mojolicious) based applications will suffer so-called Mojibake
(literally: "unintelligible sequence of characters"). Even worse: if an editor
saves BOM (Byte Order Mark, U+FEFF character in Unicode) at the start of a
script with the executable bit set (on Unix systems), it won't execute at all,
due to shebang corruption.

Avoiding codification problems is quite simple:

 * Always use utf8/use common::sense when saving source as UTF-8
 * Always specify =encoding utf8 when saving POD as UTF-8
 * Do neither of above when saving as ISO-8859-1
 * Never save BOM (not that it's wrong; just avoid it as you'll barely
   notice its presence when in trouble)

However, if you find yourself upgrading old code to use UTF-8 or trying to
standardize a big project with many developers, each one using a different
platform/editor, reviewing all files manually can be quite painful, especially
in cases where some files have multiple encodings (note: it all started when I
realized that gedit and derivatives are unable to open files with character
conversion tables).

Enter the Test::Mojibake ;)

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

--- Comment #5 from Tomas Radej  2011-08-11 08:47:56 EDT ---
Fixed issue 1 - changed architecture to noarch

SRPM URL: http://tradej.fedorapeople.org/maven-reporting-exec-1.0.1-2.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 678221] Review Request: perl-EV - Wrapper for the libev high-performance event loop library

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


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

--- Comment #11 from Jon Ciesla  2011-08-11 08:54:00 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 724878] Review Request: TexStudio - A feature-rich editor for LaTeX documents

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


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

Martin Gieseking  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|martin.giesek...@uos.de
   Flag||fedora-review?

--- Comment #9 from Martin Gieseking  2011-08-11 
08:54:00 EDT ---
Here's the formal review of the package. I'm sorry, there are still a few
things to fix.

- Add a slash between $RPM_BUILD_ROOT%{_datadir} and %{name} in the rm
  statements. Currently, the files supposed to be removed are still packaged.

- The package provides qm locale files that must be installed with 
  %find_lang %{name} --with-qt and %files -f %{name}.lang. However, as they
  are located in %{_datadir}/texstudio/ instead of a separate subfolder, it's 
  a bit complicated to separate them from the other files in that directory. 
  The qm files must not be added by one of the entries in the %files section.

  Also, texstudio seems to bundle Qt locales (qt_FOO.mq). Please ask upstream
  if they are modified or if they can be removed so that the original locale 
  files that come with Qt are used.

- Please use the original spelling "TeXstudio" in the %description

- Ask upstream to add the GPLv3 license text to the qcodeedit folder. 
  He should also fix the FSF address in COPYING.


$ rpmlint *.rpm
texstudio.src:26: W: unversioned-explicit-provides bundled(qcodeedit)
texstudio.x86_64: E: incorrect-fsf-address /usr/share/doc/texstudio-2.2/COPYING
texstudio.x86_64: E: incorrect-fsf-address /usr/share/texstudio/COPYING
texstudio.x86_64: W: no-manual-page-for-binary texstudio
texstudio-debuginfo.x86_64: W: hidden-file-or-dir
/usr/src/debug/texstudio2.2/.ui
texstudio-debuginfo.x86_64: W: hidden-file-or-dir
/usr/src/debug/texstudio2.2/.ui
texstudio-debuginfo.x86_64: W: hidden-file-or-dir
/usr/src/debug/texstudio2.2/.moc
texstudio-debuginfo.x86_64: W: hidden-file-or-dir
/usr/src/debug/texstudio2.2/.moc
3 packages and 0 specfiles checked; 2 errors, 6 warnings.

-
key:

[+] OK
[.] OK, not applicable
[X] needs work
-

[+] MUST: The package must be named according to the Package Naming Guidelines.
[+] MUST: The spec file name must match the base package %{name}.
[+] MUST: The package must meet the Packaging Guidelines.
[+] MUST: The package must be licensed with a Fedora approved license.
- GPLv2+ and GPLv3 (bundled and statically linked qcodeedit library)

[+] MUST: The License field in the package spec file must match the actual
license.
[+] MUST: The file containing the text of the license(s) for the package must
be included in %doc.
[+] MUST: The spec file must be written in American English.
[+] MUST: The spec file for the package MUST be legible.
[+] MUST: The sources used to build the package must match the upstream source.
$ md5sum texstudio-2.2.tar.gz*
d23cf71c90f3fd950d49bf480285e920  texstudio-2.2.tar.gz
d23cf71c90f3fd950d49bf480285e920  texstudio-2.2.tar.gz.1

[+] MUST: The package MUST successfully compile and build into binary rpms on
at least one primary architecture.
[.] MUST: If the package does not successfully compile, build or work on an
architecture, ...
[+] MUST: All build dependencies must be listed in BuildRequires.
[+] MUST: When compiling C, C++, or Fortran files, %{optflags} must be applied.
[X] MUST: The spec file MUST handle locales properly.

[+] MUST: If a package installs files below %{_datadir}/icons, the icon cache
must be updated.
[.] MUST: Packages storing shared library files (not just symlinks) must call
ldconfig in %post and %postun.
[+] MUST: Packages must NOT bundle copies of system libraries.
- exception for qcodeedit has been granted

[.] MUST: If the package is designed to be relocatable, ...
[+] MUST: A package must own all directories that it creates. 
[X] MUST: A Fedora package must not list a file more than once in %files.
- COPYING and AUTHORS packaged twice due to missing slash in
  rm -rf $RPM_BUILD_ROOT%{_datadir}%{name}/...

[+] MUST: Permissions on files must be set properly.
[+] MUST: Each package must consistently use macros.
[+] MUST: The package must contain code, or permissable content.
[.] MUST: Large documentation files must go in a -doc subpackage.
[+] MUST: Files in %doc must not affect the runtime of the application.
[.] MUST: Header files must be in a -devel package.
[.] MUST: Static libraries must be in a -static package.
[.] MUST: If a package contains library files with a suffix (e.g.
libfoo.so.1.1), then library files that end in .so (without suffix) must go in
a -devel package.
[.] MUST: devel packages must require the base package using a fully versioned
dependency.
[+] 

[Bug 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

--- Comment #6 from Tomas Radej  2011-08-11 08:56:56 EDT ---
Fixed issue 1 - changed architecture to noarch

Spec URL: http://tradej.fedorapeople.org/maven-reporting-exec.spec
SRPM URL: http://tradej.fedorapeople.org/maven-reporting-exec-1.0.1-2.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 675588] Review Request: pycmd - Tools for managing/searching Python related files

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


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

Fedora Update System  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 675587] Review Request: pytest - Simple powerful testing with Python

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


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

Fedora Update System  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 675587] Review Request: pytest - Simple powerful testing with Python

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


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

--- Comment #13 from Fedora Update System  
2011-08-11 08:59:09 EDT ---
pytest-2.1.0-2.fc16,pycmd-1.0-3.fc16,python-py-1.4.4-2.fc16 has been submitted
as an update for Fedora 16.
https://admin.fedoraproject.org/updates/pytest-2.1.0-2.fc16,pycmd-1.0-3.fc16,python-py-1.4.4-2.fc16

-- 
Configure 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 675588] Review Request: pycmd - Tools for managing/searching Python related files

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


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

--- Comment #14 from Fedora Update System  
2011-08-11 08:59:04 EDT ---
pytest-2.1.0-2.fc16,pycmd-1.0-3.fc16,python-py-1.4.4-2.fc16 has been submitted
as an update for Fedora 16.
https://admin.fedoraproject.org/updates/pytest-2.1.0-2.fc16,pycmd-1.0-3.fc16,python-py-1.4.4-2.fc16

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

Alexander Kurtakov  changed:

   What|Removed |Added

 Attachment #517788|0   |1
is obsolete||

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

Alexander Kurtakov  changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #7 from Alexander Kurtakov  2011-08-11 
09:00:45 EDT ---
Looks good now.

APPROVED.

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


[Bug 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

Emmanuel Seyman  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #3 from Emmanuel Seyman  
2011-08-11 08:58:41 EDT ---

=== KEY ===

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

=== REQUIRED ITEMS ===

 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines including the Perl specific items
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on:  http://koji.fedoraproject.org/koji/taskinfo?taskID=3265813

 [x] Rpmlint output: 2 packages and 1 specfiles checked; 0 errors, 0 warnings.
 [x] Package is not relocatable.
 [x] Buildroot is correct
None specified, default used.

 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic
 [-] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
bf75dc7f171b4718a2118c3d6cbe6013  Switch-2.16.tar.gz

 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [-] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [-] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===

 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: rawhide.x86_64
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: http://koji.fedoraproject.org/koji/taskinfo?taskID=3265813
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.
 [x] %check is present and the tests pass
All tests successful.
Files=3, Tests=590,  1 wallclock secs ( 0.06 usr  0.01 sys +  0.39 cusr  0.01
csys =  0.47 CPU)
Result: PASS

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 700427] Review Request: jopt-simple - A Java command line parser

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


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

--- Comment #11 from Karel Klíč  2011-08-11 09:16:03 EDT ---
Spec URL: http://kklic.fedorapeople.org/jopt-simple.spec
SRPM URL: http://kklic.fedorapeople.org/jopt-simple-3.3-3.fc14.src.rpm

* Thu Aug 11 2011 Karel Klíč  - 3.3-3
- Include the license text in the javadoc package, which is
  independent from the main package
- Use %%add_maven_depmap instead of %%add_to_maven_depmap

-- 
Configure 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 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

Tomas Hoger  changed:

   What|Removed |Added

   Keywords|Security|
  Group|security|
 CC|security-response-team@redh |
   |at.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 608470] Review Request: perl-Test-Script-Run - Test scripts with Perl

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


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

Emmanuel Seyman  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||emmanuel.seyman@club-intern
   ||et.fr
 Resolution||CURRENTRELEASE
Last Closed||2011-08-11 09:21:18

--- Comment #15 from Emmanuel Seyman  
2011-08-11 09:21:18 EDT ---
This looks fixed (the module has been build and released). Closing.

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


[Bug 700427] Review Request: jopt-simple - A Java command line parser

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


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

Alexander Kurtakov  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #12 from Alexander Kurtakov  2011-08-11 
09:23:03 EDT ---
Looks good.

APPROVED

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


[Bug 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

Tomas Radej  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #8 from Tomas Radej  2011-08-11 09:31:27 EDT ---
New Package SCM Request
===
Package Name: maven-reporting-exec
Short Description: Classes to manage report plugin executions with Maven 3
Owners: tradej
Branches: fc15
InitialCC: akurtakov

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

--- Comment #9 from Jon Ciesla  2011-08-11 09:42:22 EDT ---
Git done (by process-git-requests).

Corrected fc15 to f15, added f16.

-- 
Configure 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 700427] Review Request: jopt-simple - A Java command line parser

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


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

Karel Klíč  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #13 from Karel Klíč  2011-08-11 09:42:53 EDT ---
Thank you for the review.

New Package SCM Request
===
Package Name: jopt-simple
Short Description: A Java command line parser
Owners: kklic
Branches: f15 f16
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 729926] Review Request: jboss-httpserver - JBoss httpserver

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


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

Pierre-YvesChibon  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|pin...@pingoured.fr

-- 
Configure 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 700427] Review Request: jopt-simple - A Java command line parser

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


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

--- Comment #14 from Jon Ciesla  2011-08-11 10:03:06 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

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


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

--- Comment #3 from Praveen Kumar  2011-08-11 
10:00:53 EDT ---
Koji Build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3265945

-- 
Configure 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

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


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

--- Comment #4 from Praveen Kumar  2011-08-11 
10:14:07 EDT ---
Spec URL:
http://kumarpraveen.fedorapeople.org/django-pylibmc/django-pylibmc.spec
SRPM URL:
http://kumarpraveen.fedorapeople.org/django-pylibmc/django-pylibmc-0.2.1-2.20110806gitb56e74.fc15.src.rpm

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

-- 
Configure 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 729926] Review Request: jboss-httpserver - JBoss httpserver

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


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

Pierre-YvesChibon  changed:

   What|Removed |Added

   Flag||fedora-review+

--- Comment #2 from Pierre-YvesChibon  2011-08-11 10:34:15 
EDT ---
[!] rpmlint must be run on every package.
jboss-httpserver.src: W: invalid-url Source0:
jboss-httpserver-1.0.0.Beta1.tar.xz
jboss-httpserver.noarch: W: no-documentation
jboss-httpserver-javadoc.noarch: W: spelling-error Summary(en_US) Javadocs
-> Java docs, Java-docs, Avocados
3 packages and 0 specfiles checked; 0 errors, 3 warnings.
-> That is using the jpackage-utils from rawhide

[X] The package must be named according to the Package Naming Guidelines.

[X] The spec file name must match the base package %{name}, in the format
  %{name}.spec unless your package has an exemption.
Since the name httpserver is very generic and this is a jboss related
package the name jboss-httpserver make things clearer.

[X] The package must meet the Packaging Guidelines.

[X] The package must be licensed with a Fedora approved license and meet the
  Licensing Guidelines.

[!] The License field in the package spec file must match the actual license.
   License is a mixture of GPLv2 only, GPLv2+ and GPLv2 only with exception

[NA] If (and only if) the source package includes the text of the license(s)
 in its own file, then that file, containing the text of the license(s) for
 the package must be included in %doc.

[X] The spec file must be written in American English.

[X] The spec file for the package MUST be legible.

[X] The sources used to build the package must match the upstream source, as
  provided in the spec URL.
source from the src.rpm: 
f753544e067b93b0d72166689296212a695b9606 
../rpmbuild/SOURCES/jboss-httpserver-1.0.0.Beta1.tar.xz
source regenerated using the comment in the spec:
f753544e067b93b0d72166689296212a695b9606 
jboss-httpserver-1.0.0.Beta1.tar.xz

[X] The package MUST successfully compile and build into binary rpms on at
 least one primary architecture.
compiled fine on F15 x86_64
koji build : http://koji.fedoraproject.org/koji/taskinfo?taskID=3265967

[NA] If the package does not successfully compile, build or work on an
  architecture, then those architectures should be listed in the spec in
  ExcludeArch.

[X] All build dependencies must be listed in BuildRequires, except for any
 that are listed in the exceptions section of the Packaging Guidelines ;
 inclusion of those as BuildRequires is optional.

[NA] The spec file MUST handle locales properly. This is done by using the
  %find_lang macro. Using %{_datadir}/locale/* is strictly forbidden.

[NA] Every binary RPM package (or subpackage) which stores shared library
  files(not just symlinks) in any of the dynamic linker's default paths,
  must call ldconfig in %post and %postun.

[X] Packages must NOT bundle copies of system libraries.

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

[X] A package must own all directories that it creates. If it does not create
 a directory that it uses, then it should require a package which does
 create that directory.

[X] A Fedora package must not list a file more than once in the spec file's
  %files listings. 

[X] Permissions on files must be set properly. Executables should be set with
 executable permissions, for example. Every %files section must include a
 %defattr(...) line.

[!] Each package must consistently use macros.

[X] The package must contain code, or permissable content.

[X] Large documentation files must go in a -doc subpackage.
That's the -javadoc package here

[NA] If a package includes something as %doc, it must not affect the runtime
 of the application. To summarize: If it is in %doc, the program must run
 properly if it is not present.

[NA] Header files must be in a -devel package.

[NA] Static libraries must be in a -static package.

[NA] If a package contains library files with a suffix (e.g. libfoo.so.1.1),
  then library files that end in .so (without suffix) must go in a -devel
  package.

[NA] In the vast majority of cases, devel packages must require the base
  package using a fully versioned dependency: Requires: %{name} =
  %{version}-%{release}.

[NA] Packages must NOT contain any .la libtool archives, these must be removed
  in the spec if they are built.

[NA] Packages containing GUI applications must include a %{name}.desktop file,
  a

[Bug 729653] Review Request: perl-Pod-Wordlist-hanekomu - Add words for spell checking POD

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


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

Paul Howarth  changed:

   What|Removed |Added

 Blocks||728329(perl-Test-Version)

-- 
Configure 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 728972] Review Request: perl-Test-Vars - Detects unused variables

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


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

Paul Howarth  changed:

   What|Removed |Added

 Blocks||728329(perl-Test-Version)

-- 
Configure 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 729813] Review Request: perl-Test-CPAN-Meta-JSON - Validate a META.json file within a CPAN distribution

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


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

Paul Howarth  changed:

   What|Removed |Added

 Blocks||728329(perl-Test-Version)

-- 
Configure 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 729972] Review Request: perl-Test-Mojibake - Check your source for encoding misbehavior

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


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

Paul Howarth  changed:

   What|Removed |Added

 Blocks||728329(perl-Test-Version)

-- 
Configure 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 728329] Review Request: perl-Test-Version - Check to see that versions in modules are sane

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


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

Paul Howarth  changed:

   What|Removed |Added

 Depends on||729653, 729813, 729972,
   ||728972

--- Comment #2 from Paul Howarth  2011-08-11 10:57:06 EDT ---
Updated for extra release test coverage:

Spec URL:
http://subversion.city-fan.org/repos/cfo-repo/perl-Test-Version/branches/fedora/perl-Test-Version.spec

SRPM URL:
http://www.city-fan.org/~paul/extras/perl-Test-Version/perl-Test-Version-1.0.0-3.fc17.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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

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


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

Ankur Sinha  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+

--- Comment #5 from Ankur Sinha  2011-08-11 11:01:23 
EDT ---
Okay! 

Nice work. Package is good to go.

[ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-*/result/*.rpm
django-pylibmc.noarch: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.noarch: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.noarch: W: spelling-error %description -l en_US backend ->
backed, back end, back-end
django-pylibmc.src: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.src: W: spelling-error %description -l en_US backend -> backed,
back end, back-end
django-pylibmc.src: W: invalid-url Source0:
django-pylibmc-0.2.1.20110806gitb56e74.tar.xz
django-pylibmc.noarch: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.noarch: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.noarch: W: spelling-error %description -l en_US backend ->
backed, back end, back-end
django-pylibmc.src: W: spelling-error Summary(en_US) backend -> backed, back
end, back-end
django-pylibmc.src: W: spelling-error %description -l en_US memcached ->
schemed
django-pylibmc.src: W: spelling-error %description -l en_US backend -> backed,
back end, back-end
django-pylibmc.src: W: invalid-url Source0:
django-pylibmc-0.2.1.20110806gitb56e74.tar.xz
4 packages and 0 specfiles checked; 0 errors, 14 warnings.


XXX APPROVED XXX
Ankur

-- 
Configure 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 730033] New: Review Request: perl-autobox-dump - Human/perl readable strings from the results of an EXPR

2011-08-11 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-autobox-dump - Human/perl readable strings from 
the results of an EXPR
Alias: perl-autobox-dump

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

   Summary: Review Request: perl-autobox-dump - Human/perl
readable strings from the results of an EXPR
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/autobox-dump/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-autobox-dump.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-autobox-dump-20090426.1746-1.fc17.src.rpm

Description:
The autobox::dump pragma adds, via the autobox pragma, a method to normal
expression (such as scalars, arrays, hashes, math, literals, etc.) that
produces a human/perl readable representation of the value of that expression.

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

*rt-0.10_01

-- 
Configure 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 730031] New: Review Request: perl-autobox-Core - Core functions exposed as methods in primitive types

2011-08-11 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-autobox-Core - Core functions exposed as methods 
in primitive types
Alias: perl-autobox-Core

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

   Summary: Review Request: perl-autobox-Core - Core functions
exposed as methods in primitive types
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/autobox-Core/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-autobox-Core.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-autobox-Core-1.2-1.fc17.src.rpm

Description:
The autobox module lets you call methods on primitive datatypes such as
scalars and arrays.

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

*rt-0.10_01

-- 
Configure 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 730034] New: Review Request: perl-autobox-List-Util - Bring the List::Util functions to autobox

2011-08-11 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-autobox-List-Util - Bring the List::Util 
functions to autobox

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

   Summary: Review Request: perl-autobox-List-Util - Bring the
List::Util functions to autobox
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/autobox-List-Util/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-autobox-List-Util.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-autobox-List-Util-20090629-1.fc17.src.rpm

Description:
autobox::List::Util brings all of the functions from List::Util to arrays as
methods.

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

*rt-0.10_01

-- 
Configure 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 730037] New: Review Request: perl-autovivification - Lexically disable autovivification

2011-08-11 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-autovivification - Lexically disable 
autovivification

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

   Summary: Review Request: perl-autovivification - Lexically
disable autovivification
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/autovivification/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-autovivification.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-autovivification-0.09-1.fc17.src.rpm

Description:
When an undefined variable is dereferenced, it gets silently upgraded to an
array or hash reference (depending of the type of the dereferencing). This
behavior is called autovivification and usually does what you mean (e.g.
when you store a value) but it's sometimes unnatural or surprising because
your variables gets populated behind your back. This is especially true
when several levels of dereferencing are involved, in which case all levels
are vivified up to the last, or when it happens in intuitively read-only
constructs like exists.

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

*rt-0.10_01

-- 
Configure 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 730038] New: Review Request: perl-Child - Object oriented simple interface to fork()

2011-08-11 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-Child - Object oriented simple interface to fork()
Alias: perl-Child

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

   Summary: Review Request: perl-Child - Object oriented simple
interface to fork()
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Child/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Child.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Child-0.009-1.fc15.src.rpm

Description:
Fork is too low level, and difficult to manage. Often people forget to exit
at the end, reap their children, and check exit status. The problem is the
low level functions provided to do these things. Throw in pipes for IPC and
you just have a pile of things nobody wants to think about.

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

*rt-0.10_01

-- 
Configure 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 730041] New: Review Request: perl-Date-JD - Conversion between flavors of Julian Date

2011-08-11 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-Date-JD - Conversion between flavors of Julian 
Date
Alias: perl-Date-JD

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

   Summary: Review Request: perl-Date-JD - Conversion between
flavors of Julian Date
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Date-JD/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Date-JD.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Date-JD-0.005-1.fc17.src.rpm

Description:
For date and time calculations it is convenient to represent dates by a
simple linear count of days, rather than in a particular calendar. This is
such a good idea that it has been invented several times. If there were a
single such linear count then it would be the obvious data interchange
format between calendar modules. With several versions, calendar modules
can use such sensible data formats and still have interoperability
problems. This module tackles that problem, by performing conversions
between different flavors of day count. These day count systems are
generically known as "Julian Dates", after the most venerable of them.

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

*rt-0.10_01

-- 
Configure 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 700427] Review Request: jopt-simple - A Java command line parser

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


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

--- Comment #15 from Fedora Update System  
2011-08-11 11:43:33 EDT ---
jopt-simple-3.3-3.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/jopt-simple-3.3-3.fc16

-- 
Configure 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 730040] New: Review Request: perl-Date-ISO8601 - Three ISO 8601 numerical calendars

2011-08-11 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-Date-ISO8601 - Three ISO 8601 numerical calendars
Alias: perl-Date-ISO8601

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

   Summary: Review Request: perl-Date-ISO8601 - Three ISO 8601
numerical calendars
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Date-ISO8601/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Date-ISO8601.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Date-ISO8601-0.004-1.fc17.src.rpm

Description:
The international standard ISO 8601 "Data elements and interchange formats
- Information interchange - Representation of dates and times" defines
three distinct calendars by which days can be labeled. It also defines
textual formats for the representation of dates in these calendars. This
module provides functions to convert dates between these three calendars
and Chronological Julian Day Numbers, which is a suitable format to do
arithmetic with. It also supplies functions that describe the shape of
these calendars, to assist in calendrical calculations. It also supplies
functions to represent dates textually in the ISO 8601 formats. ISO 8601
also covers time of day and time periods, but this module does nothing
relating to those parts of the standard; this is only about labeling days.

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

*rt-0.10_01

-- 
Configure 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 700427] Review Request: jopt-simple - A Java command line parser

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


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

Fedora Update System  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 700427] Review Request: jopt-simple - A Java command line parser

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


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

--- Comment #16 from Fedora Update System  
2011-08-11 11:43:43 EDT ---
jopt-simple-3.3-3.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/jopt-simple-3.3-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 730043] New: Review Request: perl-DateTime-Format-Epoch - Convert DateTimes to/from epoch seconds

2011-08-11 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-DateTime-Format-Epoch - Convert DateTimes to/from 
epoch seconds

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

   Summary: Review Request: perl-DateTime-Format-Epoch - Convert
DateTimes to/from epoch seconds
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/DateTime-Format-Epoch/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL:
http://fedorapeople.org/~iarnell/review/perl-DateTime-Format-Epoch.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-DateTime-Format-Epoch-0.13-1.fc17.src.rpm

Description:
This module can convert a DateTime object (or any object that can be
converted to a DateTime object) to the number of seconds since a given
epoch. It can also do the reverse.

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

*rt-0.10_01

-- 
Configure 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 730047] New: Review Request: perl-Hash-FieldHash - Lightweight field hash implementation

2011-08-11 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-Hash-FieldHash - Lightweight field hash 
implementation
Alias: perl-Hash-FieldHash

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

   Summary: Review Request: perl-Hash-FieldHash - Lightweight
field hash implementation
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/Hash-FieldHash/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-Hash-FieldHash.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-Hash-FieldHash-0.10-1.fc17.src.rpm

Description:
Hash::FieldHash provides the field hash mechanism which supports the inside-
out technique.

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

*rt-0.10_01

-- 
Configure 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 683071] Review Request: php-libvirt - PHP bindings for libvirt virtualization toolkit

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


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

--- Comment #59 from Fedora Update System  
2011-08-11 11:57:07 EDT ---
php-libvirt-0.4.3-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/php-libvirt-0.4.3-1.fc16

-- 
Configure 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 596461] Review Request: lzma-sdk - SDK for lzma compression

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


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

--- Comment #61 from Jason Tibbitts  2011-08-11 11:54:12 EDT 
---
Note that I'm still on to review this, assuming that Jon agrees with what
you've done and wants to proceed.  Just let me know.

-- 
Configure 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 729681] Review Request: maven-reporting-exec - classes managing report plugin execution

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


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

Tomas Radej  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
Last Closed||2011-08-11 12:11:43

-- 
Configure 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 728664] Review Request: django-pylibmc - Django cache backend using pylibmc

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


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

--- Comment #6 from Praveen Kumar  2011-08-11 
12:22:57 EDT ---
New Package SCM Request
===
Package Name: django-pylibmc
Short Description: Django cache backend using pylibmc 
Owners: sundaram pjp kumarpraveen
Branches: f14 f15 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 710203] Review Request: gambas3 - IDE based on a basic interpreter with object extensions

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


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

--- Comment #12 from Tom "spot" Callaway  2011-08-11 
12:29:34 EDT ---
New SRPM: http://spot.fedorapeople.org/gambas3-2.99.2-1.fc15.src.rpm
New SPEC: http://spot.fedorapeople.org/gambas3.spec

I think that all the rpmlint errors are now safe to ignore (hidden-files,
missing docs, one incorrect FSF address)

-- 
Configure 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 730069] New: Review Request: perl-indirect - Lexically warn about using the indirect object syntax

2011-08-11 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-indirect - Lexically warn about using the 
indirect object syntax
Alias: perl-indirect

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

   Summary: Review Request: perl-indirect - Lexically warn about
using the indirect object syntax
   Product: Fedora
   Version: rawhide
  Platform: Unspecified
   URL: http://search.cpan.org/dist/indirect/
OS/Version: Unspecified
Status: NEW
  Severity: unspecified
  Priority: unspecified
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: iarn...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com,
package-review@lists.fedoraproject.org
Classification: Fedora
  Story Points: ---
  Type: ---


Spec URL: http://fedorapeople.org/~iarnell/review/perl-indirect.spec
SRPM URL:
http://fedorapeople.org/~iarnell/review/perl-indirect-0.24-1.fc17.src.rpm

Description:
When enabled (or disabled as some may prefer to say, since you actually
turn it on by calling no indirect), this pragma warns about indirect object
syntax constructs that may have slipped into your code.

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

*rt-0.10_01

-- 
Configure 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 722709] Review Request: mbrowse - GUI SNMP MIB browser

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


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

--- Comment #3 from Nathan Owe  2011-08-11 13:21:17 EDT ---
The group tag is no longer necessary

Also running rpmlint on the RPM gives me there is no man pages for binary
https://fedoraproject.org/wiki/Packaging:Guidelines#Man_pages.

-- 
Configure 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 727030] Review Request: ufw - uncomplicated firewall

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


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

--- Comment #14 from Nathan Owe  2011-08-11 13:22:50 EDT 
---
I am not sure what to do about the following init scripts:
/lib/ufw/ufw-init
/lib/ufw/ufw-init-functions

I know ufw enable will pretty much do /lib/ufw/ufw-init start.
So really do I need to install it to /etc/rc.d/init.d even though ufw looks for
them in /lib and I also tried doing something like it shows in the wiki:
/sbin/chkconfig --add /lib/ufw/ufw-init and it gives a error like "error
reading information on service ufw-init: No such file or directory"

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


[Bug 720864] Review Request: gadmin-httpd - Apache GUI tool - First package

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


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

Nathan Owe  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2011-08-11 13:29:26

-- 
Configure 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 537727] Review Request: Slack - A lazy systems administration configuration management toolkit.

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


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

Nathan Owe  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG
Last Closed||2011-08-11 13:30:08

-- 
Configure 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 537727] Review Request: Slack - A lazy systems administration configuration management toolkit.

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


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

Nathan Owe  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 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

Tom "spot" Callaway  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #4 from Tom "spot" Callaway  2011-08-11 
14:22:21 EDT ---
New Package SCM Request
===
Package Name: perl-Switch
Short Description: A switch statement for Perl
Owners: spot
Branches: f16
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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

Tom "spot" Callaway  changed:

   What|Removed |Added

   Flag||fedora-cvs?

--- Comment #11 from Tom "spot" Callaway  2011-08-11 
14:20:55 EDT ---
New Package SCM Request
===
Package Name: freewrl
Short Description: X3D / VRML visualization program
Owners: spot
Branches: el5 el6 f14 f15 f16
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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

--- Comment #10 from Tom "spot" Callaway  2011-08-11 
14:18:51 EDT ---
I'm not really interested in EPEL for this one, but I'll ask for the branches
and you can maintain them. Thanks 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 729777] Review Request: perl-Switch - A switch statement for Perl

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


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

--- Comment #5 from Jon Ciesla  2011-08-11 14:40:08 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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

--- Comment #12 from Jon Ciesla  2011-08-11 14:39:29 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 722709] Review Request: mbrowse - GUI SNMP MIB browser

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


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

--- Comment #4 from Volker Fröhlich  2011-08-11 15:13:34 EDT 
---
Can you cite the ruling or the section in the guidelines concerning the group
tag?

Well, the guidelines say "should". Since it is a graphical application, I don't
regard it as terribly useful. There actually happens to be an old manpage from
Debian: http://manpages.ubuntu.com/manpages/hardy/man1/mbrowse.1.html.

I'll consider including it, because there is no other help. I'll have to look
over it, if it is still up to date. Upstream seemingly had it with 0.3.1 but I
can't find a trace of it in trunk.

http://mbrowse.git.sourceforge.net/git/gitweb.cgi?p=mbrowse/mbrowse;a=blob;f=debian/changelog;h=94fd1f123a54026003befc0959d04774c39656f9

-- 
Configure 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 726962] Review Request: wmweather - Applet which shows local weather conditions

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


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

--- Comment #3 from Mario Blättermann  2011-08-11 15:21:45 
EDT ---
The "Group" tag was missing.

New files:
Spec URL: http://mariobl.fedorapeople.org/Review/SPECS/wmweather.spec
SRPM URL:
http://mariobl.fedorapeople.org/Review/SRPMS/wmweather-2.4.5-2.fc15.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 722709] Review Request: mbrowse - GUI SNMP MIB browser

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


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

--- Comment #5 from Nathan Owe  2011-08-11 15:27:00 EDT ---
Here is a bit about the Group tag, it is more of a talk:
http://lists.fedoraproject.org/pipermail/packaging/2008-September/005091.html

Also I didn't notice the man page either. Supposedly it said adding a man page
fixed bug number. But I even looked for the bug that it was referencing and I
didn't see that either.

-- 
Configure 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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

Fedora Update System  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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

--- Comment #13 from Fedora Update System  
2011-08-11 15:36:45 EDT ---
freewrl-1.22.12-0.3.pre2.fc15 has been submitted as an update for Fedora 15.
https://admin.fedoraproject.org/updates/freewrl-1.22.12-0.3.pre2.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 726210] Review Request: freewrl - X3D / VRML visualization program

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


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

--- Comment #15 from Fedora Update System  
2011-08-11 15:37:10 EDT ---
freewrl-1.22.12-0.3.pre2.fc14 has been submitted as an update for Fedora 14.
https://admin.fedoraproject.org/updates/freewrl-1.22.12-0.3.pre2.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


  1   2   >