[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

URL||http://www.xs4all.nl/~mmzeem
   ||an/ocaml/




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

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


[Bug 445067] New: Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: ocaml-ounit - Unit test framework for
OCaml
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-ounit.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-ounit-1.0.2-1.fc9.src.rpm
Description: Unit test framework for OCaml

rpmlint reports:

  ocaml-ounit.i386: E: no-binary
  ocaml-ounit.i386: E: only-non-binary-in-usr-lib

Both can be ignored for OCaml packages.

Koji scratch build:

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

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

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


[Bug 444721] Review Request: gtksourceview2-sharp - gtksourceview2 C# binding

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: gtksourceview2-sharp - gtksourceview2 C# binding


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 02:40 EST ---
Paul?

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

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


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||445068
  nThis||




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

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


[Bug 445068] Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-bin-prot - Read and write OCaml values in a 
type-safe binary protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||445067
URL||http://ocaml.janestcapital.c
   ||om/?q=node/13




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

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


[Bug 445068] New: Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: ocaml-bin-prot - Read and write OCaml
values in a type-safe binary protocol
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-bin-prot.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-bin-prot-1.0.5-1.fc9.src.rpm
Description: Read and write OCaml values in a type-safe binary protocol

rpmlint says:

ocaml-bin-prot.i386: W: devel-file-in-non-devel-package 
/usr/lib/ocaml/bin_prot/map_to_safe.ml

This is OK because of the exception for *.ml files detailed here:
https://fedoraproject.org/wiki/Packaging/OCaml#head-f1fbee40338b95c780f241fa3a86f6a82e5830dd
(ie. there is no corresponding map_to_safe.mli file).

Note that this package is only available on little-endian
architectures, and is only tested upstream on x86, hence the
ExclusiveArch.

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

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


[Bug 445070] New: Review Request: ocaml-res - OCaml library for resizing arrays and strings

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: ocaml-res - OCaml library for resizing
arrays and strings
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-res.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-res-2.2.5-1.fc9.src.rpm
Description: OCaml library for resizing arrays and strings

rpmlint says:
  ocaml-res.i386: E: no-binary
  ocaml-res.i386: E: only-non-binary-in-usr-lib
which is OK for OCaml packages.

Koji scratch build is here:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=593964

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

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


[Bug 439141] Review Request: python-demjson - Python JSON module and lint checker

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-demjson -  Python JSON module and lint checker


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 03:30 EST ---
(In reply to comment #12)
 is this pushed to rawhide?

Not sure I understand this question. Afaict there's a python-demjson package
available in f9 as well as in devel.

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

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


[Bug 434707] Review Request: ocaml-sexplib - OCaml library for converting OCaml values to S-expressions

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-sexplib - OCaml library for converting OCaml 
values to S-expressions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 03:31 EST ---
Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-sexplib.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-sexplib-3.7.4-1.fc9.src.rpm

* Sat May  3 2008 Richard W.M. Jones [EMAIL PROTECTED] - 3.7.4-1
- New upstream version 3.7.4.


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

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


[Bug 445067] Review Request: ocaml-ounit - Unit test framework for OCaml

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-ounit - Unit test framework for OCaml


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||445074
  nThis||




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

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


[Bug 445070] Review Request: ocaml-res - OCaml library for resizing arrays and strings

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-res - OCaml library for resizing arrays and 
strings


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||445074
  nThis||




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

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


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||434707, 445067, 445068,
   ||445070




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

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


[Bug 434707] Review Request: ocaml-sexplib - OCaml library for converting OCaml values to S-expressions

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-sexplib - OCaml library for converting OCaml 
values to S-expressions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||445074
  nThis||




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

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


[Bug 445068] Review Request: ocaml-bin-prot - Read and write OCaml values in a type-safe binary protocol

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-bin-prot - Read and write OCaml values in a 
type-safe binary protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||445074
  nThis||




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

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


[Bug 445074] New: Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: ocaml-core - Many and various
enhancements to the OCaml standard library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.annexia.org/tmp/ocaml/ocaml-core.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/ocaml-core-0.5.0-2.fc9.src.rpm
Description: Many and various enhancements to the OCaml standard library

This is rpmlint clean.

Please pay careful attention to the dependencies!

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

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


[Bug 445074] Review Request: ocaml-core - Many and various enhancements to the OCaml standard library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ocaml-core - Many and various enhancements to the 
OCaml standard library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

URL||http://ocaml.janestcapital.c
   ||om/?q=node/13




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

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


[Bug 441411] Review Request: adonthell - A 2D graphical RPG game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: adonthell - A 2D graphical RPG game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 04:42 EST ---
Hi Mathieu,

Short intro: I'm a long time Fedora contributer and a sponsor. I haven't been
sponsoring anyone in a while as I've been rather busy with other stuff. But
getting new people into Fedora is important so I've decided to make some time
for sponsering again. Most of the things I do within Fedora are gaming related
so I'm very happy to see adonthell getting packaged.

As such I would like to sponsor you. The first step here would be to finish up
the packaging of adonthell and wastesedge so that they are in a state where they
can be approved. The next step then is for you to either package one or two more
packages, for example from these lists:
http://fedoraproject.org/wiki/SIGs/Games/WishList
http://fedoraproject.org/wiki/PackageMaintainers/WishList

Or you can do some reviews of other people packages, when you do this please add
a note that it is not an official review as you aren't a contributer yet.

The purpose here is for you to show a good understanding of the packaging
guidelines. Once you've done a couple of good reviews and / or submitted one or
two more good packages, then you can apply for cvsextras membership in the
account system and I'll sponsor you.

As for the review of this package, Mamoru has done a fine job so far, please fix
all the items he has mentioned in his last comment and then I'll do a full
review pointing out any last issues.


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

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


[Bug 441415] Review Request: wastesedge - Official game package for Adonthell

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wastesedge - Official game package for Adonthell


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||441411
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 04:42 EST ---
Hi Mathieu,

Short intro: I'm a long time Fedora contributer and a sponsor. I haven't been
sponsoring anyone in a while as I've been rather busy with other stuff. But
getting new people into Fedora is important so I've decided to make some time
for sponsering again. Most of the things I do within Fedora are gaming related
so I'm very happy to see adonthell getting packaged.

As such I would like to sponsor you. The first step here would be to finish up
the packaging of adonthell and wastesedge so that they are in a state where they
can be approved. The next step then is for you to either package one or two more
packages, for example from these lists:
http://fedoraproject.org/wiki/SIGs/Games/WishList
http://fedoraproject.org/wiki/PackageMaintainers/WishList

Or you can do some reviews of other people packages, when you do this please add
a note that it is not an official review as you aren't a contributer yet.

The purpose here is for you to show a good understanding of the packaging
guidelines. Once you've done a couple of good reviews and / or submitted one or
two more good packages, then you can apply for cvsextras membership in the
account system and I'll sponsor you.

I'll look into reviewing this once adonthell is in an approvable state.


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

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


[Bug 445077] Review Request: lsnipes - A text-mode maze game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: lsnipes - A text-mode maze game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 05:15 EST ---
Scratch build results:
  http://koji.fedoraproject.org/koji/taskinfo?taskID=594007

rpmlint output:
  None.


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

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


[Bug 441411] Review Request: adonthell - A 2D graphical RPG game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: adonthell - A 2D graphical RPG game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||441415
  nThis||




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

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


[Bug 445077] New: Review Request: lsnipes - A text-mode maze game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: lsnipes - A text-mode maze game
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://ndim.fedorapeople.org/lsnipes/0.9.4-2.fc9/lsnipes.spec
SRPM URL: 
http://ndim.fedorapeople.org/lsnipes/0.9.4-2.fc9/lsnipes-0.9.4-2.fc9.src.rpm
Description: A text-mode maze game

Linux Snipes is a reimplementation of an old text-mode DOS game. You
are in a maze with a number of enemies (the snipes) and a few
hives which create more of the enemies. Your job is to kill the
snipes and their hives before they get the best of you.  26 option
levels let you change characteristics of the game such as whether or
not diagonal shots bounce off the walls.  10 levels of difficulty (only
partially implemented) let you build your skills gradually.

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

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


[Bug 434827] Review Request: jcommon - java jfreereport base utility library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: jcommon - java jfreereport base utility library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 06:14 EST ---
imported, can't actually build it yet. But I assume it'll all come together
after devel auto moves to F-10

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

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


[Bug 429221] Review Request: dzcomm - Dzcomm a RS-232 API/lib

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: dzcomm - Dzcomm a RS-232 API/lib


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 05:58 EST ---
Hi Ian,

Short intro: I'm a long time Fedora contributer and a sponsor. I haven't been
sponsoring anyone in a while as I've been rather busy with other stuff. But
getting new people into Fedora is important so I've decided to make some time
for sponsering again. As such I would like to sponsor you. The first step here
would be to finish up the packaging of dzcomm and scantool_net so that they are
in a state where they can be approved. The next step then is for you to either
package one or two more packages, for example from these lists:
http://fedoraproject.org/wiki/PackageMaintainers/WishList
http://fedoraproject.org/wiki/SIGs/Games/WishList

Or you can do some reviews of other people packages, when you do this please add
a note that it is not an official review as you aren't a contributer yet.

The purpose here is for you to show a good understanding of the packaging
guidelines. Once you've done a couple of good reviews and / or submitted one or
two more good packages, then you can apply for cvsextras membership in the
account system and I'll sponsor you.

---

For starters I would like to give you a hand with fixing some of the issues with
this package (dzcomm), as you're running into some issues which can be somewhat
hard to fix for someone new to packaging, and on top of that this package should
really be building a shared library not a static library as static libraries are
evil, which will require some further patching of the Makefiles.

However before I start spending time on this I would first like to get a firm
commitment from your side, are you willing to make time for this to make this
work, if there will be a silence of 2 weeks between comments all the time, then
this is not going to work.


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

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


[Bug 444830] Review Request: fpm2 - Password manager with GTK2 GUI

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: fpm2 - Password manager with GTK2 GUI


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 06:03 EST ---
Hi Aleš,

Short intro: I'm a long time Fedora contributer and a sponsor. I haven't been
sponsoring anyone in a while as I've been rather busy with other stuff. But
getting new people into Fedora is important so I've decided to make some time
for sponsering again.  As such I would like to sponsor you. The first step here
would be to finish up the packaging of fpm2 so that it is in a state where it
can be approved (I haven't looked yet, it might already be).

Normally the next step then is for you to either package one or two more
packages, for example from these lists:
http://fedoraproject.org/wiki/SIGs/Games/WishList
http://fedoraproject.org/wiki/PackageMaintainers/WishList

Or to do some reviews of other people packages, when you do this please add
a note that it is not an official review as you aren't a contributer yet.

The purpose here is for you to show a good understanding of the packaging
guidelines. Once you've done a couple of good reviews and / or submitted one or
two more good packages, then you can apply for cvsextras membership in the
account system and I'll sponsor you.

---

As I already said this is the normal (and prefered) procedure, but seeing that
you're also upstream here, I wonder: are you planning on packaging other
software for Fedora, or do you just want to maintain fpm2 in Fedora?


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

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


[Bug 444356] Review Request: bpython - Fancy Curses Interface To The Python Interactive Interpreter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: bpython -  Fancy Curses Interface To The Python 
Interactive Interpreter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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


[Bug 443177] Review Request: html2ps - HTML to PostScript converter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: html2ps -  HTML to PostScript converter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 07:26 EST ---
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, or are watching someone who is.

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


[Bug 443177] Review Request: html2ps - HTML to PostScript converter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: html2ps -  HTML to PostScript converter


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 07:25 EST ---
New Package CVS Request
===
Package Name: html2ps
Short Description: HTML to PostScript converter
Owners: pertusus
Branches: F-8 EL-5
InitialCC: 
Cvsextras Commits: yes


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

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


[Bug 444356] Review Request: bpython - Fancy Curses Interface To The Python Interactive Interpreter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: bpython -  Fancy Curses Interface To The Python 
Interactive Interpreter


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 07:35 EST ---
Going to re-do the review later (first review eaten by crashing Firefox).

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 07:45 EST ---
Thanks for the review. Here is a new release based on the latest upstream which
should fix the crashes you've been seeing:

Spec URL: http://people.atrpms.net/~hdegoede/wordwarvi.spec
SRPM URL: http://people.atrpms.net/~hdegoede/wordwarvi-0.08-1.fc9.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, or are watching someone who is.

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


[Bug 443177] Review Request: html2ps - HTML to PostScript converter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: html2ps -  HTML to PostScript converter


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 07:59 EST ---
(In reply to comment #8)
 Thanks for the review!
No problem!

(In reply to comment #7)
 Branches: F-8 EL-5
Don't forget F-9 :)

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

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


[Bug 445085] New: Review Request: jcommon-serializer - JFree Java General Serialization Framework

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: jcommon-serializer - JFree Java General
Serialization Framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.redhat.com/caolanm/jfreereport/jcommon-serializer.spec
SRPM URL: 
http://people.redhat.com/caolanm/jfreereport/jcommon-serializer-0.2.0-1.fc9.src.rpm
Description: JFree Java General Serialization Framework

A build requirement for jfreereport and so OOo 3.0

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

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


[Bug 443177] Review Request: html2ps - HTML to PostScript converter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: html2ps -  HTML to PostScript converter


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 08:42 EST ---
OOps. Added F-9 

New Package CVS Request
===
Package Name: html2ps
Short Description: HTML to PostScript converter
Owners: pertusus
Branches: F-8 F-9 EL-5
InitialCC: 
Cvsextras Commits: yes


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

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


[Bug 188445] Review Request: bootconf

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: bootconf


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 08:41 EST ---
http://www.courier-mta.org/bootconf/download/bootconf-1.2-2.src.rpm

This should address everything except the permissions on the spec file. I don't
know why rpmbuild does this. The spec file is packaged inside the tarball, the
srpm gets created by rpmbuild -ta, and that's what you get.

The spec file has normal permissions inside the tarball. My shell umask is 002,
and that's what happens when my build script runs rpmbuild -ta. I don't know
what's going on.


-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the QA contact for the bug, or are watching the QA contact.

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


[Bug 438750] Review Request: qtoctave - fronted for octave written using qt4 widgets

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: qtoctave - fronted for octave written using qt4 widgets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|ASSIGNED
   Flag|needinfo?([EMAIL PROTECTED]|
   |asoni.it)   |




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 09:47 EST ---
Sorry for the big delay (still very annoying troubles at work :-( ).
Here is a new SRPM and a new SPEC file:

Spec URL: http://www.claudiotomasoni.it/files/RPMS/qtoctave.spec
SRPM URL: http://www.claudiotomasoni.it/files/RPMS/qtoctave-0.7.4-1.fc8.src.rpm

and these are the news:
- updated to 0.7.4
- removed the qt4 explicit requirement
- added a chmod command in the %prep section to fix file permissions oddity
(many .cpp and .h executable files)

I'm looking for a package to pre-review. In the meantime you can take a look to
my other submission:
https://bugzilla.redhat.com/show_bug.cgi?id=439100

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

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


[Bug 435576] Review Request: ncdu - Text-based disk usage viewer

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: ncdu - Text-based disk usage viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 11:01 EST ---
Okay, the segfaults are gone now, and everything else looks good.  I'm going to
change the fedora-review flag to +.

One concern I just noticed is that the rpm is 48M in size (vs 5M for the SRPM,
because of the sound conversion from ogg to wav in building).  It's not a review
blocker at all, but I do think it would be better for the game to play the ogg
files directly.

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 11:13 EST ---
(In reply to comment #18)
 One concern I just noticed is that the rpm is 48M in size (vs 5M for the SRPM,
 because of the sound conversion from ogg to wav in building).  It's not a 
 review
 blocker at all, but I do think it would be better for the game to play the ogg
 files directly.

Agreed, Stephen any chance you could do this for one of the next releases? Maybe
switch to SDL_mixer for the sound, that supports ogg out of the box.



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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 11:17 EST ---
New Package CVS Request
===
Package Name:  wordwarvi
Short Description: Side-scrolling shoot 'em up '80s style arcade game
Owners:jwrdegoede
Branches:  F-8 F-9
InitialCC: 
Cvsextras Commits: Yes




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

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


[Bug 445094] New: Review Request: pommed - Apple laptop hotkey event handler

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: pommed - Apple laptop hotkey event
handler
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.diffingo.com/downloads/diffingo-repo/review/pommed.spec
SRPM URL: 
http://www.diffingo.com/downloads/diffingo-repo/review/pommed-1.18-0.1.20080502svn.fc9.src.rpm
Description:
pommed handles the hotkeys found on the Apple PowerBook, MacBook, MacBook Pro,
and MacBook Air laptops.

pommed can adjust the LCD backlight, sound volume, keyboard backlight and eject
the CD-ROM drive. pommed also monitors the ambient light sensors to
automatically light up the keyboard backlight on the MacBook Pro, MacBook Air
and PowerBook when applicable. pommed also supports the Apple Remote.

rpmlint is not silent, however I think it is safe to ignore these warnings:

pommed.x86_64: W: dangerous-command-in-%pre cp
-- This is done to move the now mbpeventd configuration file to the pommed 
one, as the pommed and mbpeventd projects have merged
pommed.x86_64: W: incoherent-subsys /etc/init.d/pommed $prog
-- rpmlint doesn't realize $prog = pommed
pommed.x86_64: W: service-default-enabled /etc/init.d/pommed
-- This is the desired behaviour
pommed-gpomme.x86_64: W: no-documentation
-- Doesn't matter
pommed-wmpomme.x86_64: W: no-documentation
-- Doesn't matter

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

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


[Bug 444952] Review Request: tlock - terminal lock

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tlock - terminal lock


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 12:44 EST ---
  Hello Ralf, thank you so much for the comments.

  I've removed the first four glitches, and the new SRPM  Spec files are
accessible at the above links, please have a look.

  In 5 MUSTFIX: When you say package %{name}, are you referring to the
tlock source or the RPM? I'm not very sure how to go about this devel
subpackage business.

  Is it just that, I create a new spec file for new rpm which will install
the librpass.so  readpass.h files. And in the above RPM  spec file I only
install the tlock binary along with its manual?

  Also could you please explain a bit more about 6 MUSTFIX? I didn't get what
special treatment is required for info manuals.

Thank you!

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

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


[Bug 444952] Review Request: tlock - terminal lock

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: tlock - terminal lock


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 13:23 EST ---
6 MUSTFIX is also fixed. Please see

SPEC URL: http://pjp.dgplug.org/tools/tlock.spec
SRPM URL: http://pjp.dgplug.org/tools/tlock-1.1-0.2.fc8.src.rpm

Thank you!


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

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


[Bug 439744] Review Request: rubberband - Audio time-stretching and pitch-shifting library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: rubberband - Audio time-stretching and pitch-shifting 
library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 13:21 EST ---
Ping again,

Can this please be pushed to stable updates?


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

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


[Bug 433926] Review Request: freehoo - Freehoo is a free console based messenger for Yahoo IM Service

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: freehoo - Freehoo is a free console based messenger 
for Yahoo IM Service


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 14:26 EST ---
Built against new freehoo 3.5.2:

http://www.bludgeon.org/~rayvd/rpms/freehoo/freehoo.spec
http://www.bludgeon.org/~rayvd/rpms/freehoo/freehoo-3.5.2-1.src.rpm

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 15:09 EST ---
I'll look into it, but so far as I can tell, libsndfile doesn't do ogg yet.
http://www.mega-nerd.com/libsndfile/#Features


Though, there is this:
http://www.metadecks.org/software/libsndfile/

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

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


[Bug 438043] Review Request: GMT - Generic Mapping Tools

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: GMT - Generic Mapping Tools


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 15:22 EST ---
There is a new upstream release. 

Also I will be a week off for vacations.

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 15:29 EST ---
(In reply to comment #21)
 I'll look into it, but so far as I can tell, libsndfile doesn't do ogg yet.
 http://www.mega-nerd.com/libsndfile/#Features
 
 
 Though, there is this:
 http://www.metadecks.org/software/libsndfile/

Well, judging from its last release date (Aug 31 2006) libsndfile is somewhat
dead. So perhaps using SDL_mixer (which will replace both portaudio and
libsndfile in one go) is a better idea? You can use SDL to manage just the
sound, or just the sound and joystick, gaining joystick support in one go.


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

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


[Bug 444165] Review Request: mkrdns - automatic reverse DNS zone generator

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mkrdns - automatic reverse DNS zone generator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 15:44 EST ---
NEEDSWORK: rpmlint
mkrdns.spec: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 11)
- after Buildarch: and Requires: you inserted tabs but after all other tags you
only used spaces, therefore you should change this. You should remove the
Requires anyhow, because they are already added automatically by rpm for perl
requirements.

Naming: ok
License: ok (GPLv2+), not included
Building: ok, builds in koji
Clean section: ok
Macro Usage: ok
Source Package: ok, matches upstream, latest version:
474ffb39d1eb615d0e9d8a7d52a27953  mkrdns-3_3.gz
474ffb39d1eb615d0e9d8a7d52a27953  mkrdns-3.3-1.fc8/mkrdns-3_3.gz

Buildroot: ok
%files: ok

Besides the mixed whitespace, there is another minor issue:
You write /usr/bin/pod2man instead of just pod2man, which does not make sense to
me, because on my machine install is also in /usr/bin and you rely on /usr/bin
being in the path there.

Also the timestamp of the executable is lost, maybe you can change the spec to
preserve it, but I do not know a good way to do this.

In conclusion: Remove the tab after Buildarch, remove the Requires: and remove
the /usr/bin from the pod2man invocation, then I will approve this package.

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 16:04 EST ---

Hmm, sounds like a non-trivial change.  

And I kind of actually *like* that I can (in fact, must) write my own mixer code
with portaudio.  (for instance, I suspect you'd have a hard time writing a game
like this: http://bethewumpus.sourceforge.net with SDL_mixer.)

BTW, there's already joystick support (in fact, to play the game, a joystick is
practically mandatory. Unless you're doing cross platform stuff, the linux
joystick driver is easy enough to program against that a library is not really
necessary.

I don't think libsndfile is dead, ardour and audacity both use it, for 
instance. 

Hmm, come to think of it, audacity supports ogg as well, maybe that's a place I
should look.

What I need is a snippet of code that can read ogg files and give me an array of
mono, 16-bit, 44.1khz PCM data.









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

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


[Bug 445105] New: Review Request: milkytracker - SDL clone of the Fasttracker II module tracker

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: milkytracker - SDL clone of the
Fasttracker II module tracker
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.ee.oulu.fi/~muep/rpms/milky/milkytracker.spec
SRPM URL: 
http://www.ee.oulu.fi/~muep/rpms/milky/milkytracker-0.90.80-1.fc9.src.rpm
Description: 
MilkyTracker is an application for creating music in the .MOD and .XM formats.
Its goal is to be free replacement for the popular Fasttracker II software.

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

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


[Bug 438105] Review Request: libconcord - L ibrary to talk to Logitech® Harmony® univ ersal remote controls

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libconcord - Library to talk to Logitech® Harmony® 
universal remote controls


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 17:39 EST ---
I plan on adding the perl tests at an additional time.  I'd like to get the 
package through review before Stephen goes on vacation.

Spec 
URL:https://rpm.silfreed.net:8002/index.cgi/file/37414f878d6a/libconcord/libconcord.spec
SRPM URL:
http://www.silfreed.net/download/repo/packages/libconcord/libconcord-0.20-4.src.rpm

%changelog
* Sat May 03 2008 Douglas E. Warner [EMAIL PROTECTED] 0.20-4
- installing python packages to sitearch again
- adding additional docs


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

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


[Bug 328161] Review Request: concordance - Software to program the Logitech Harmony remote control

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: concordance - Software to program the Logitech Harmony 
remote control


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 17:42 EST ---
I'll try to push as much of the other items you posted from the review 
upstream in the near future.

Spec URL: 
https://rpm.silfreed.net:8002/index.cgi/file/37414f878d6a/concordance/concordance.spec
SRPM URL: 
http://www.silfreed.net/download/repo/packages/concordance/concordance-0.20-4.src.rpm

%changelog
* Sat May 03 2008 Douglas E. Warner [EMAIL PROTECTED] 0.20-4
- adding additional docs
- removed harmony provides/obsoletes
- removing private getopt sources
- removing udev/pam_console rules
- running generated xml file through xmllint at build time

* Tue Apr 22 2008 Douglas E. Warner [EMAIL PROTECTED] 0.20-3
- fixed Source0 url to downloads.sourceforge.net instead of dl.sourceforge.net


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

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


[Bug 445028] Review Request: python-pylons - A python web application toolkit

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: python-pylons - A python web application toolkit


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 17:48 EST ---
Note, I just smoke tested this on F9 and things seem to work correctly.  F9 SRPM
packages for this and all dependencies are in http://www.kylev.com/pylons-rpm/ ,
no modifications made to spec files.


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

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


[Bug 445094] Review Request: pommed - Apple laptop hotkey event handler

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: pommed - Apple laptop hotkey event handler


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 18:14 EST ---
Probably should, but I have no idea how to do that... I'm not sure if it has
anything to do with HAL, but I know it has it's own policy and dbus 
configuration.

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

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


[Bug 426867] Review Request: scala - Hybrid functional/object-oriented language

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: scala - Hybrid functional/object-oriented language


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 18:37 EST ---
Also, see
http://www.nabble.com/-scala--licensing-of-scala-collection-immutable-Tree.scala-td16932952.html
for the discussion of the licensing issue. Upstream has contacted the original
author and got the code relicensed under the Scala license. 

Does this mean I can revert the license tag immediately or do I have to wait for
the next release with the LGPL notice removed?

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

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


[Bug 439100] Review Request: octaviz - 3D visualization system for Octave

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: octaviz - 3D visualization system for Octave


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 19:26 EST ---
(In reply to comment #7)
 Also, I'm willing to sponsor you if you'll demonstrate an ability to review 
 packages by doing at least one pre-review as described above.

I guess I can sponsor him (see bug 438750)

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

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


[Bug 438105] Review Request: libconcord - L ibrary to talk to Logitech® Harmony® univ ersal remote controls

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: libconcord - Library to talk to Logitech® Harmony® 
universal remote controls


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 19:37 EST ---
Package Review
==

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

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format 
%{name}.spec.
 [x] Package meets the Packaging Guidelines.
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: i386 (F8 system, F8 mock, devel mock)
 [x] Rpmlint output:
 Minor warnings:
 libconcord-perl.i386: W: no-documentation
 libconcord-python.i386: W: no-documentation
 [x] Package is not relocatable.
 [x] Buildroot is correct
 %(mktemp -ud %{_tmppath}/%{name}-%{version}-%{release}-XX)
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPLv3+
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package: c2487e851864f38c4da4fe02093652a5
 MD5SUM upstream package: c2487e851864f38c4da4fe02093652a5
 [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.
 [x] 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.
 [!] Package does not contain duplicates in %files.
 TODO is packaged in both libconcord and libconcord-devel.
 I would suggest simply removing it from libconcord-devel, judging by other
packages.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [x] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [x] Development .so files in -devel subpackage, if present.
 [x] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== 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: i386 (F8 and devel)
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested all arches via koji scratch build for F8.
 [x] Package functions as described.
 [x] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files are correct.
 [x] File based requires are sane.

=== SUMMARY ===

APPROVED.

Before actually building/pushing this, you should resolve the duplicate TODO 
file.

Minor notes: There is a README for the Perl bindings in the source package. You
might want to package that with libconcord-perl. Latest CVS has a similar README
for the Python bindings. If you feel really energetic, you could add that file
as an extra source, and package that in libconcord-python.


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

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


[Bug 442522] Review Request: audit-viewer - Audit event viewer

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: audit-viewer - Audit event viewer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 442640] Review Request: epeg - Immensely fast JPEG thumbnailer

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: epeg - Immensely fast JPEG thumbnailer


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 20:01 EST ---
* Sun May 04 2008 Pavel Stalwart Shevchuk [EMAIL PROTECTED] - 0.9.1.042-2
- Fixed timestamp of source tarball
- Preserve timestamps of installed files

New spec: http://rpm.scwlab.com/epeg-goes-rawhide/epeg.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/epeg-0.9.1.042-2.x86_64/
epeg-0.9.1.042-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 20:04 EST ---
Oh, and Hans, about packaging dependencies, now libsndfile isn't needed, but
libvorbisfile is needed.

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

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


[Bug 442437] Review Request: edje - A graphical layout and animation library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: edje - A graphical layout and animation library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 20:34 EST ---
* Sun May 04 2008 Pavel Stalwart Shevchuk [EMAIL PROTECTED] - 0.5.0.042-2
- Added missing dependencies for edje-devel
- Fixed timestamp of source tarball
- Preserve timestamps of installed files
- Added html docs


New spec: http://rpm.scwlab.com/edje-goes-rawhide/edje.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/edje-0.5.0.042-2.x86_64/
edje-0.5.0.042-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

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

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


[Bug 445105] Review Request: milkytracker - SDL clone of the Fasttracker II module tracker

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: milkytracker - SDL clone of the Fasttracker II module 
tracker


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 442808] Review Request: epsilon - Small, display independent, and quick thumbnailing library

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: epsilon - Small, display independent, and quick 
thumbnailing library


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 22:00 EST ---
* Sun May 04 2008 Pavel Stalwart Shevchuk [EMAIL PROTECTED] - 0.3.0.012-2
- Added missing dependencies for epsilon-devel
- Fixed timestamp of source tarball
- Preserve timestamps of installed files

New spec: http://rpm.scwlab.com/epsilon-goes-rawhide/epsilon.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/epsilon-0.3.0.012-2.x86_64/
epsilon-0.3.0.012-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

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

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


[Bug 444165] Review Request: mkrdns - automatic reverse DNS zone generator

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: mkrdns - automatic reverse DNS zone generator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 22:04 EST ---
WRT the timestamp: try
  zcat %{SOURCE0}  $RPM_BUILD_ROOT%{_bindir}/mkrdns
  chmod 0755 $RPM_BUILD_ROOT%{_bindir}/mkrdns
  touch -r ${SOURCE0}  $RPM_BUILD_ROOT%{_bindir}/mkrdns

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

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


[Bug 439100] Review Request: octaviz - 3D visualization system for Octave

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: octaviz - 3D visualization system for Octave


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 22:57 EST ---
Hi Mamoru, if you'd like to sponsor Claudio then that's fine with me.

And Claudio -- can you please respond to comment #7?  If you fix the 
license and improve the VTK_DATA_ROOT situation then I'll approve 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, or are watching someone who is.

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


[Bug 443449] Review Request: udpcast - UDP broadcast file distribution and installation

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: udpcast - UDP broadcast file distribution and 
installation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 23:23 EST ---
The --prefix, --mandir, and --libdir options are set by the %configure macro, so
you should delete those.  See the definition in /usr/lib/rpm/redhat/macros. 
Otherwise, this package looks ready to go.

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, or are watching someone who is.

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


[Bug 442404] Review Request: efreet - An implementation of several specifications from freedesktop.org

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: efreet - An implementation of several specifications 
from freedesktop.org


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 23:28 EST ---
* Sun May 04 2008 Pavel Stalwart Shevchuk [EMAIL PROTECTED] - 0.0.3.042-2
- Fixed missing dependencies for efreet-devel
- Fixed timestamp of source tarball
- Preserve timestamps of installed files

New spec: http://rpm.scwlab.com/efreet-goes-rawhide/efreet.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/efreet-0.0.3.042-2.fc9.x86_64/
efreet-0.0.3.042-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

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

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


[Bug 442022] Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style arcade game

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: wordwarvi - Side-scrolling shoot 'em up '80s style 
arcade game


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 23:31 EST ---
cvs done.

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

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


[Bug 443177] Review Request: html2ps - HTML to PostScript converter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: html2ps -  HTML to PostScript converter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 23:34 EST ---
cvs done. 

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

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


[Bug 442462] Review Request: e_dbus - Wrappers around dbus for EFL based applications

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: e_dbus - Wrappers around dbus for EFL based 
applications


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-03 23:58 EST ---
* Sun May 04 2008 Pavel Stalwart Shevchuk [EMAIL PROTECTED] - 0.1.0.042-2
- Fixed typo in E_Nm.h, fix committed upstream
- Added missing dependencies to e_dbus-devel
- Fixed timestamp of source tarball
- Preserve timestamps of installed files

New spec: http://rpm.scwlab.com/e_dbus-goes-rawhide/e_dbus.spec
New SRPM: http://rpm.scwlab.com/fedora/e/9/e_dbus-0.1.0.042-2.fc9.x86_64/
e_dbus-0.1.0.042-2.fc9.src.rpm
Built RPMs: http://rpm.scwlab.com/fedora/e/9/

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

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


[Bug 444792] Review Request: Augeas - library for changing configuration files

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Augeas - library for changing configuration files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-05-04 01:23 EST ---
New Package CVS Request
===
Package Name: augeas
Short Description: a configuration API
Owners: lutter
Branches: F-9
InitialCC: 
Cvsextras Commits: yes

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

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


[Bug 444792] Review Request: Augeas - library for changing configuration files

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: Augeas - library for changing configuration files


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-04 01:21 EST ---
Thanks for spotting this - I'll check it and address it before I import the 
package

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

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


[Bug 445126] New: Review Request: report-mirror - Fedora MirrorManager client

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.




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

   Summary: Review Request: report-mirror - Fedora MirrorManager
client
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://jstanley.fedorapeople.org/report-mirror.spec
SRPM URL: 
http://jstanley.fedorapeople.org/report-mirror-0.1-2.git20080504.fc8.src.rpm
Description:

This is the client for MirrorManager, the Fedora mirror management system.
The configuration file should be edited prior to using this package.  For
more information, For further information, please see:
http://fedoraproject.org/wiki/Infrastructure/Mirroring

[EMAIL PROTECTED] ~]$ rpmlint 
/var/lib/mock/fedora-8-i386/result/report-mirror-0.1-2.git20080504.fc8.noarch.rpm
[EMAIL PROTECTED] ~]$ 

I need a sponsor :).  I'm also copying Matt Domsch on this request, since he is 
the upstream author.

A little about me:  I'm currently heading up the bug triage team within Fedora, 
and noticed that this wasn't packaged, but probably should be.  Let me know if 
further information is required.

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

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


[Bug 445126] Review Request: report-mirror - Fedora MirrorManager client

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: report-mirror - Fedora MirrorManager client


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





--- Additional Comments From [EMAIL PROTECTED]  2008-05-04 01:35 EST ---
One thing that I'm not sure about in this package is the version number.  Matt,
can you help me out here?  Is there some canonical released version of MM that
I can base the version number off of?

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

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


[Bug 445126] Review Request: report-mirror - Fedora MirrorManager client

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: report-mirror - Fedora MirrorManager client


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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


[Bug 444356] Review Request: bpython - Fancy Curses Interface To The Python Interactive Interpreter

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: bpython -  Fancy Curses Interface To The Python 
Interactive Interpreter


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-05-04 01:41 EST ---
Review Guidelines MUST items:
- [OK] rpmlint output: none
- [OK] package name
- [OK] %{name}.spec
- [OK] Packaging Guidelines
- [OK] Licensing Guidelines (MIT)
- [OK] License Field in spec
- [OK] License text in %doc
- [OK] Spec file in en_US
- [OK] legible spec file
- [OK] source matches upstream
- [OK] compiles successfully as noarch
- [OK] BuildRequires
- N/A  %find_lang
- N/A  shared libs
- N/A  not relocatable
- [OK] directory ownership
- [OK] no duplicate files in %files
- [OK] proper permissions on files, %defattr present
- [OK] %clean section cleans %{buildroot}
- [OK] consistently uses macros
- [OK] package contains code
- N/A  large docs
- [OK] %doc files do not affect runtime behaviour
- N/A  header files in -devel
- N/A  static libs in -static
- N/A  foo.pc files
- N/A  libfoo.so.1.1
- N/A  no devel package
- N/A  no .la archives
- N/A  not a GUI app
- [OK] Does not own files/dirs owned by other packages
- [OK] %install cleans out %{buildroot} first
- [OK] all filenames are valid ASCII and thus UTF-8

Review Guidelines SHOULD items:
- N/A  license text
- N/A  no translated descriptions available
- [OK] builds in mock: http://koji.fedoraproject.org/koji/taskinfo?taskID=594044
- N/A  noarch package
- [OK] appears to work
- N/A  no scriptlets
- N/A  no sub-packages
- N/A  no foo.pc
- N/A  no file dependencies

Remarks on Python Packaging Guidelines:
- [OK] good python_sitelib macro definition
- [OK] use %{python_sitelib}
- N/A  Not an egg, only builds .egg-info
- [OK] Includes all byte compiled files

Remarks on Packaging Guidelines:
- package uses second allowed choice for BuildRoot:
- CFLAGS=%{optflags}... what for in a noarch package?
- desktop file:
  - as non-GUI app bpython does not need one
  - it is non-functional
  - ergo: why not leave it out?
  If you do decide to add a desktop file starting bpython in a terminal:
  Please use %{__install}'s -p parameter.
I'd change those three, but they are OK as-is.

ACCEPTED.


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

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


[Bug 444257] Review Request: nted - Musical score editor

2008-05-03 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: nted - Musical score editor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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