[Bug 444760] Review Request: obm - Open Business Management

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #18 from Remi Collet   2009-02-14 
02:30:27 EDT ---
Grr... .src.cpm seems broken :(
And version 2.1.14 available...

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

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


[Bug 473537] Review Request: jcodings - Java Libraries for Ruby String Encodings

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Conrad Meyer  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #8 from Conrad Meyer   2009-02-13 22:54:57 EDT 
---
Imported and built in rawhide. Closing.

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

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

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


[Bug 478506] Review Request: trac-customfieldsadmin-plugin - expose ticket custom fields via web admin interface

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Ian Weller   2009-02-13 22:03:06 EDT 
---
I'll hopefully be able to get to this tomorrow evening.

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

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


[Bug 485007] Review Request: rhnpush - Package uploader for the RHN Satellite/Spacewalk Server

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Michael Stahnke   2009-02-13 21:48:42 
EDT ---
Spec URL: http://stahnma.fedorapeople.org/reviews/rhnpush.spec
SRPM URL: http://stahnma.fedorapeople.org/reviews/rhnpush-0.3.1-4.fc10.src.rpm


* Found a better Source URL.

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

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


[Bug 485007] Review Request: rhnpush - Package uploader for the RHN Satellite/Spacewalk Server

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Michael Stahnke   2009-02-13 21:31:37 
EDT ---


Spec URL: http://stahnma.fedorapeople.org/reviews/rhnpush.spec
SRPM URL: http://stahnma.fedorapeople.org/reviews/rhnpush-0.3.1-3.fc10.src.rpm

* I have updated the package slightly.  I added a source description, as per 
 http://fedoraproject.org/wiki/Packaging/SourceURL .
* I tried to play with %check and tests, but I couldn't them to work.  It seems
like I am missing a harness setup of some type.  I will ping the sw-dev list
and see if I can find out anything.  Either way, that shouldn't be a blocker
for package review.

* Man page:  It's a little different than many, but certainly not wrong.  See
(man vim), it looks similar.  I think it's because the man page is generated
using doc2book on the SGML file.  I'd rather just write it in groff, but I am
not upstream.

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

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


[Bug 478506] Review Request: trac-customfieldsadmin-plugin - expose ticket custom fields via web admin interface

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Jon Stanley   2009-02-13 21:00:45 EDT 
---
ping?

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

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


[Bug 484379] Review Request: hartke-aurulent-sans-fonts - A sans-serif font

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Toshio Ernie Kuratomi  changed:

   What|Removed |Added

 CC||a.bad...@gmail.com
   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Toshio Ernie Kuratomi   2009-02-13 
20:30:11 EDT ---
cvs done.

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

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


[Bug 485542] Review Request: breip-font - The Breip handwriting-style cursive font

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Stephen Carter   2009-02-13 
20:12:56 EDT ---
I also have another font awaiting review, which you can find here: 
https://bugzilla.redhat.com/show_bug.cgi?id=484057 . I need two clean font
package submissions to be sponsored, and this is my second package. Comments
and reviews are welcome!

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

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


[Bug 471520] Review Request: rubygem-simple-rss - A simple, flexible, extensible, and liberal RSS and Atom reader for Ruby

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #16 from Kevin Fenzi   2009-02-13 20:13:08 EDT ---
cvs done.

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

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


[Bug 485542] Review Request: breip-font - The Breip handwriting-style cursive font

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Stephen Carter  changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 484057] Review Request: epigrafica-fonts - The Epigrafica font family.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Stephen Carter   2009-02-13 
20:11:42 EDT ---
My second font submission can be found here:
https://bugzilla.redhat.com/show_bug.cgi?id=485542 . I will be cleaning up this
one and uploading fresh packages very soon.

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

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


[Bug 485542] New: Review Request: breip-font - The Breip handwriting-style cursive font

2009-02-13 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: breip-font - The Breip handwriting-style cursive font

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

   Summary: Review Request: breip-font - The Breip
handwriting-style cursive font
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: scart...@learn.senecac.on.ca
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://matrix.senecac.on.ca/~scarter4/breip-font.spec
SRPM URL: http://matrix.senecac.on.ca/~scarter4/breip-font-0.5.1-2.fc10.src.rpm
Description: Breip is a handwriting font made by Alan Hussey using the
handwriting of his friend Adam Breipohl, hence the name 'breip'.

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

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


[Bug 471520] Review Request: rubygem-simple-rss - A simple, flexible, extensible, and liberal RSS and Atom reader for Ruby

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Michael Stahnke  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #15 from Michael Stahnke   2009-02-13 20:11:25 
EDT ---
New Package CVS Request
===
Package Name: rubygem-simple-rss
Short Description: A simple, flexible, extensible, and liberal RSS and Atom
reader for Ruby  
Owners: stahnma kanarip
Branches: EL-5
InitialCC:

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

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


[Bug 485424] Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Kevin Fenzi   2009-02-13 19:48:29 EDT ---
cvs done.

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

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


[Bug 472525] Review Request: kio_sysinfo - KIO slave which shows basic system information

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Kevin Fenzi   2009-02-13 19:43:26 EDT ---
cvs done.

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

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


[Bug 466953] Review Request: perl-NOCpulse-Utils - NOCpulse utility packages

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #9 from Kevin Fenzi   2009-02-13 19:44:57 EDT ---
cvs done.

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

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


[Bug 480944] Review Request: perl-Test-Harness-Straps - Detailed analysis of test results

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Kevin Fenzi   2009-02-13 19:46:06 EDT ---
cvs done.

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

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


[Bug 484704] Review Request: libapogee - Library for Apogee CCD Cameras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Kevin Fenzi  changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Kevin Fenzi   2009-02-13 19:40:36 EDT ---
cvs done.

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

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


[Bug 476460] Review Request: pymilter - Python interface to sendmail milter API

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Stuart D Gathman  changed:

   What|Removed |Added

   Flag|needinfo?(stu...@gathman.or |
   |g)  |




--- Comment #27 from Stuart D Gathman   2009-02-13 19:27:04 
EDT ---
Swamped at my paying job.  Debating whether to try and submit milter/milter-spf
[http://bmsi.com/python/milter.html] as  my test (it would be nice to have a
package that actually *uses* pymilter as part of Fedora).  The problem is, I
already know they don't currently meet Fedora packaging guidelines.  I suppose
if I "flunk" I can keep trying.  I would also need to upgrade my home server to
Fedora 10 to test running everything under Fedora.  (Currently runs in
production on EL3, EL4, EL5).

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

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


[Bug 485001] Review Request: perl-MooseX-Emulate-Class-Accessor-Fast - Emulate Class::Accessor::Fast behavior using Moose attributes

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Chris Weyl   2009-02-13 18:19:26 EDT 
---
(In reply to comment #3)
> Urgh...
> Please filter out perl(Test::More) and perl(Test::Exception) from the 
> automatic
> Requires. A quick glance shows no use for them except in the tests.

Where are you seeing these?  The srpm depends on them, but I can't find them
generated anywhere else...  (on my F-10 system, at least)

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

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


[Bug 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Milos Jakubicek   2009-02-13 18:10:11 
EDT ---
Yep, I'm quoting the answer to my email:

"
Hey, Milos,

I fixed compilation with recent Audacious versions in 3.1.0-rc7.
[http://www.luminal.org/files/imms/imms-3.1.0-rc7.tar.bz2]

I am fairly sure all the code in question is GPL. I made sure of that
before including it.
I do not have the time to follow up with the authors of the code right
now, but I'm certainly willing to apply any patches to clarify
licenses.
Perhaps, Artur, the Debian IMMS maintainer (cc'ed) can help you there.
He already did some work on getting explicit releases from the
authors.

Have fun,
Michael. 
"

It seems that all the thins are GPL indeed...looking into debian(lenny) imms
packages, I found following:

"
+Author: Michael Grigoriev 
+
+Copyright (C) 2003-2005 Michael Grigoriev
+
+With the following exceptions:
+
+md5.{h,c} (added ability to restrict maximum size to process)
+from GNU textutils
+levenshtein.{h,c} (stripped down)
+python-Levenshtein library
+by David Necas (Yeti) 
+regexx.{h,cc} (stripped down)
+Regexx - Regular Expressions C++ solution
+by Gustavo Niemeyer 
+xidle.{h,cc}
+Borrows from xautolock 2.1
+by Michel Eyckmans (MCE) 
+normal.h
+by by Agner Fog
"

I'll prepare patches and request adding a LICENSE file into the source tarball
and license information to all the source files (and merging the gcc43 patch of
course too). I'd also include the above information as %doc. Would it be ok
then?

Regarding the review: the audacious plugin is working now, updated link are
here:

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.3.rc7.fc10.src.rpm
Koji scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1125850

As there are now both xmms and audacious plugins available, I'm considering
splitting the package into a main one and two subpackages (-xmms, -audacious),
I'm just not sure whether this wouldn't hit the usability from the user point
of view too much.

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

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


[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Andreas Thienemann  changed:

   What|Removed |Added

 CC||andr...@bawue.net




--- Comment #27 from Andreas Thienemann   2009-02-13 
17:38:09 EDT ---
Hello  Gratien,

Robert was kind enough to ask me whether I'd be willing to sponsor this.

A cursory glance at the spec looks good, but please add some linebreaks and \
to the long lines in the %install part of the .spec file.
This makes things much easier to read.

The review looks okay so far, seems the .spec has come a long way. :)

As soon as Robert indicates he's finished I'll do a final test of the software
and recheck the review. After that, I'd be happy to sponsor you.

That okay with you?

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

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


[Bug 485159] Review Request: anki - Flashcard program for using space repetition learning

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Christian Krause   2009-02-13 17:13:32 
EDT ---
I've updated the packages:
- fixed an installation problem
- corrected the license field with the help of Jason Tibbitts (fedora-legal
mailing list)

Spec URL: http://www-user.tu-chemnitz.de/~tiwi/anki.spec
SRPM URL: http://www-user.tu-chemnitz.de/~tiwi/anki-0.9.9.6-2.fc10.src.rpm

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

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


[Bug 453422] Review Request: songbird - Mozilla based multimedia player

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #28 from David Halik   2009-02-13 16:46:24 
EDT ---
Based off of the method used in the Thunderbird package, the internal provides
are now filtered out properly. This avoids any potential conflicts between
system libraries, such as libxul, and those packaged with Songbird.

http://rpm.rutgers.edu/fedora/songbird.spec
http://rpm.rutgers.edu/fedora/songbird-1.0.0-4.fc10.src.rpm

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

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


[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #26 from Robert Scheck   2009-02-13 
16:39:21 EDT ---
Right, I just wanted to make sure, that /usr/share/rear usually doesn't get
touched at all and dynamic stuff ends in /var or the config stuff in /etc.

I didn't get the SVN/CVS suggestion.

I think, the "rear mkrpm" thing can easily solved with some RPM macros inside
of the spec file which do just the right thing, if it is executed on Fedora
and/or RHEL. Would that be acceptable for you? E.g.

%if 0%{?fedora}%{?rhel}
rm -f unwanted-file-at-fedora
%endif

Or

%if 0%{?fedora}%{?rhel}
# Fedora specific stuff
%else
# Other distribution stuff
%endif

If that's okay, I would try to provide you a working example. Hopefully I got
your point correct, but I think so. 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.

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


[Bug 480103] Review Request: bnIRC - An ncurses based IRC client and modular IRC framework.(Need Sponsorship. First time Packager)

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Tom Wisniewski   2009-02-13 
15:57:14 EDT ---
Alright.  Finally found some free time to work on this again.  I have split the
rpm into two packages, the rpm and a devel rpm.  The links are below. Please
have a look and let me know what else needs to be changed.  Thanks again for
all suggestions/feedback.

Spec URL:  http://dev.zerogin.com/bnIRC.spec
RPM URL:   http://dev.zerogin.com/bnIRC-1.1.1-5.fc10.i386.rpm
SRPM URL:  http://dev.zerogin.com/bnIRC-1.1.1-5.fc10.src.rpm
DEVEL RPM URL: http://dev.zerogin.com/bnIRC-devel-1.1.1-5.fc10.i386.rpm

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

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


[Bug 485186] Review Request: perl-MouseX-Types - Organize your Mouse types in libraries

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Chris Weyl   2009-02-13 15:38:23 EDT 
---
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.

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


[Bug 484975] Review Request: perl-Scope-Upper - Act on upper scopes

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Chris Weyl   2009-02-13 15:38:28 EDT 
---
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.

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


[Bug 485015] Review Request: perl-Class-C3-Adopt-NEXT - Reduce one's dependency on NEXT

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Chris Weyl  changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #4 from Chris Weyl   2009-02-13 15:38:19 EDT 
---
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.

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


[Bug 463767] Review Request: cloog - The Chunky Loop Generator

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #28 from Dodji Seketeli   2009-02-13 14:40:33 EDT 
---
For the info problem, I will submit a bug report to upstream.
As for the second review request, I have just filed a new one at
https://bugzilla.redhat.com/show_bug.cgi?id=485496.

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

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


[Bug 485424] Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Alexander Kurtakov  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Alexander Kurtakov   2009-02-13 
14:13:02 EDT ---
New Package CVS Request
===
Package Name: eclipse-birt
Short Description:  Eclipse-based reporting system
Owners: akurtakov
Branches: 
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.

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


[Bug 485496] New: Review request of geglmm - the C++ Binding to the GEGL library

2009-02-13 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 of geglmm - the C++ Binding to the GEGL library

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

   Summary: Review request of geglmm - the C++ Binding to the GEGL
library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: do...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


spec URL: http://people.redhat.com/dseketel/rpms/geglmm/geglmm.spec-0.0.22-1
SRPM URL: http://people.redhat.com/dseketel/rpms/geglmm/geglmm-0.0.22-1.src.rpm

Description:

This is the C++ binding of the GEGL graph based image processing framework.
This bindings adhere to the APIs of the gtkmm2 library.

I have put the result of rpmlint at
http://people.redhat.com/dseketel/rpms/geglmm/rpmlint-output-geglmm.spec-0.0.22-1.txt.

It's spitting an only-non-binary-in-usr-lib error, and I think it's because of
the things that are being install in %{_libdir}, like the m4 files, the
pkgconfig files and the configuration header files. Those files are necessary,
I think. Isn't rpmlint being a bit too pedantic ?

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

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


[Bug 485496] Review request of geglmm - the C++ Binding to the GEGL library

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Dodji Seketeli  changed:

   What|Removed |Added

 Depends on||177841




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

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


[Bug 485424] Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Andrew Overholt  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Andrew Overholt   2009-02-13 13:36:12 
EDT ---
Thanks.  This package is approved.

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

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


[Bug 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite file format

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Mamoru Tasaka   2009-02-13 
12:57:39 EDT ---
(In reply to comment #11)
> Also I do not have a rawhide system handy. Did you test it
> on rawhide already?

Like other packages I reviewed I don't know how to use this 
software so I did just some basic test for this software 
(like if the software lauches)

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

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


[Bug 480538] Review Request: iptux -- a tool for sharing and transporting files and directories in Lan

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #21 from Mamoru Tasaka   2009-02-13 
12:54:37 EDT ---
Then:
-
NOTE: Before being sponsored:

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

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

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

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

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

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


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

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


[Bug 480538] Review Request: iptux -- a tool for sharing and transporting files and directories in Lan

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp




--- Comment #20 from Mamoru Tasaka   2009-02-13 
12:54:06 EDT ---
For 0.4.4-2:

* License
  - Although the project URL says this is under GPLv2,
judging from the source tarball the license tag should
be "GPLv2+"
* Version information is specified only in
  src/iptux.cpp, which is under GPLv2+
* Also src/AboutIptux.cpp shows this is under GPLv2+
  (this can be seen by Help->About on iptux)
* ref:
 
https://fedoraproject.org/wiki/Licensing/FAQ#How_do_I_figure_out_what_version_of_the_GPL.2FLGPL_my_package_is_under.3F

* BuildRequires
  - This srpm won't build without "BR: gettext"
http://koji.fedoraproject.org/koji/taskinfo?taskID=1125112

* Timestamps
  - Please consider to use
--
make install DESTDIR=$RPM_BUILD_ROOT INSTALL="install -p"
--
to keep timestamps on installed files as much as possible.
This method usually works for Makefiles generated from recent
autotools.

* %{?name}
  - I guess using %{name} is enough.

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

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


[Bug 225797] Review Request: gimp-data-extras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jason Tibbitts  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #18 from Jason Tibbitts   2009-02-13 12:34:29 
EDT ---
Looks good, thanks.

APPROVED; just fix up the release to an integer >= 1 when you check in.

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

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


[Bug 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite file format

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Alex Eskin  changed:

   What|Removed |Added

   Flag|needinfo?(alexes...@yahoo.c |
   |om) |




--- Comment #11 from Alex Eskin   2009-02-13 12:28:09 EDT 
---
Sorry, I was away for a bit. I will test your patch and
respond to the other comments this weekend. 

Also I do not have a rawhide system handy. Did you test it
on rawhide already?

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

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


[Bug 485424] Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Alexander Kurtakov   2009-02-13 
12:15:06 EDT ---
Spec URL:
http://akurtakov.fedorapeople.org/eclipse-birt.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-birt-2.3.1-2.fc10.src.rpm


(In reply to comment #1)
> - please add a comment stating why we're only building the feature we are
Fixed. It was already done but more details added.
> - please add information about steps to get us to a full BIRT here: 
> http://fedoraproject.org/wiki/Eclipse
Fixed. Same as previous.
> - please add a comment above the one patch stating why we're doing this
Fixed.
> - let's change the variable "qualifier" to "rhinoqualifier" to avoid confusion
Fixed.
> - should we be using the same qualifier as BIRT like we do with Mylyn?
No. Different BIRT plugins hava different qualifiers so it's not applicable.
> 
> Lines beginning with X need attention; those beginning with * are okay:
> 
> * verify the final provides and requires of the binary RPMs
> X make sure lines are <= 80 characters
>   - please add some line continuations to fix this where possible
Fixed.

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

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


[Bug 485420] Review Request: openbios - Open Source implementation of IEEE 1275-1994

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Glauber de Oliveira Costa   2009-02-13 
12:07:01 EDT ---
Updated:

http://glommer.fedorapeople.org/openbios-1.0-0.2.svn450.fc11.src.rpm
http://glommer.fedorapeople.org/openbios.spec

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Glauber de Oliveira Costa   2009-02-13 
11:55:54 EDT ---
Updated:

http://glommer.fedorapeople.org/bochs-bios-2.3.8-0.2.git36989b0d2.fc11.src.rpm
http://glommer.fedorapeople.org/bochs-bios.spec

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

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


[Bug 484933] Review Request: libwps - Library for reading and converting Microsoft Works word processor documents

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Orcan 'oget' Ogetbil  changed:

   What|Removed |Added

 CC||oget.fed...@gmail.com




--- Comment #2 from Orcan 'oget' Ogetbil   2009-02-13 
11:46:25 EDT ---
Apart from what Michael said:

* The group for the -doc subpackage should probably be Documentation instead of
Applications/Publishing.

* The source tarball does not have the proper timestamp. You can use, for
example
   wget -N http://URLtoSOURCEfile
to get it with the right timestamp.

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Eduardo Habkost   2009-02-13 11:21:33 
EDT ---
%changelog seems to be in reverse order.  :)

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Glauber de Oliveira Costa   2009-02-13 
11:14:57 EDT ---
wrt setup: Maybe we could use it, but what would it buy us?
the rpm directives seem to be quite arcane, I believe this is much cleaner.

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Glauber de Oliveira Costa   2009-02-13 
11:04:22 EDT ---
Ok, Updated it.

You can grab:


http://glommer.fedorapeople.org/vgabios-0.6-0.2beta.fc11.src.rpm
http://glommer.fedorapeople.org/vgabios.spec

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

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


[Bug 484379] Review Request: hartke-aurulent-sans-fonts - A sans-serif font

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Paul Lange  changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #4 from Paul Lange   2009-02-13 11:05:59 EDT ---
Package Change Request
==
Package Name: hartke-aurulent-fonts (change to hartke-aurulent-sans-fonts)
New Branches: none
Owners: palango

I did a mistake with the package name when filling it into the Request. Sorry
for that!
Could you please rename the package to hartke-aurulent-sans-fonts . 
Thank you very much and sorry for the circumstances.

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

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


[Bug 485424] Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Andrew Overholt  changed:

   What|Removed |Added

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




--- Comment #1 from Andrew Overholt   2009-02-13 11:00:31 
EDT ---
- please add a comment stating why we're only building the feature we are
- please add information about steps to get us to a full BIRT here: 
http://fedoraproject.org/wiki/Eclipse
- please add a comment above the one patch stating why we're doing this
- let's change the variable "qualifier" to "rhinoqualifier" to avoid confusion
- should we be using the same qualifier as BIRT like we do with Mylyn?

Lines beginning with X need attention; those beginning with * are okay:

* verify the final provides and requires of the binary RPMs
X make sure lines are <= 80 characters
  - please add some line continuations to fix this where possible
* package successfully compiles and builds
* BuildRequires are proper
* macros fine
* package is named appropriately
* it is legal for Fedora to distribute this
* license field matches the actual license.
* license is open source-compatible.
* specfile name matches %{name}
* md5sum matches upstream
* skim the summary and description for typos, etc.
* summary and description good
* correct buildroot
* %{?dist} used correctly
* license text included in package and marked with %doc
* packages meets FHS (http://www.pathname.com/fhs/)
* rpmlint on .srpm gives no output
* changelog format okay
* Summary tag does not end in a period
* no PreReq
* specfile is legible
* specfile written in American English
* no -doc sub-package necessary
* not native, so no rpath, static linking, etc.
* no config files
* not a GUI app
* no -devel necessary
* install section begins with rm -rf $RPM_BUILD_ROOT or %{buildroot}
* no translations so no locale handling
* no Requires(pre,post)
* package not relocatable
* package contains code
* package owns all directories and files
* no %files duplicates
* file permissions fine
* %clean present
* %doc files do not affect runtime
* not a web app
* package includes license text in the package and marks it with %doc
* run rpmlint on the binary RPMs => no output

$ rpmlint ../RPMS/noarch/eclipse-birt-2.3.1-1.fc10.noarch.rpm
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-xml.jar
../../../../../java/batik/batik-xml.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-parser.jar
../../../../../java/batik/batik-parser.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/fop.jar
../../../../../java/fop.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-svg-dom.jar
../../../../../java/batik/batik-svg-dom.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.xml.resolver_1.2.0.jar
../../../../../java/xml-commons-resolver.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/javax.xml_1.3.4.jar
../../../../../java/xml-commons-apis.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/xml-commons-apis-ext.jar
../../../../../java/xml-commons-apis-ext.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-dom.jar
../../../../../java/batik/batik-dom.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.xerces_2.9.0.jar
../../../../../java/xerces-j2.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-svggen.jar
../../../../../java/batik/batik-svggen.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-awt-util.jar
../../../../../java/batik/batik-awt-util.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/rhino/eclipse/plugins/org.mozilla.rhino_1.6.7.v20080214/lib/js.jar
../../../../../../../java/js.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-swing.jar
../../../../../java/batik/batik-swing.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/batik-css.jar
../../../../../java/batik/batik-css.jar
eclipse-birt.noarch: W: dangling-relative-symlink
/usr/share/eclipse/dropins/birt/eclipse/plugins/org.apache.c

[Bug 468189] Review Request: rear - Relax and Recovery (disaster recovery framework)

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Gratien D'haese  changed:

   What|Removed |Added

   Flag|needinfo?(gratien.dha...@it |
   |3.be)   |




--- Comment #25 from Gratien D'haese   2009-02-13 
10:58:46 EDT ---
Hi all,
thank you (Robert) for taking a closer look into rear - much appreciated - the
goal is indeed to make the software cleaner (and better).

As Schlomo (in #24) already mentioned the doc files are kept double for the
simple reason that it is very easy to make a new 'in-house' rpm for example
(with the command 'rear mkrpm'). However, if we must exclude them, we can of
course. In that case the customers who want to build their own in-house rpms
should work with a tar-ball of rear instead. Less convenient I think, but not
impossible.
We could also provide SVN/CVS to accomplish this.
What does fedora recommend?

In the rpmls output all directories have mode 755. I think Robert question was
to check if the /usr/share/rear directory was meant to be in read-only mode.
The short answer is yes. During a normal run (mkrescue/mkbackup workflow) rear
will not write nor modify anything under /usr/share/rear. We have /var/rear
which will be used to create the recovery information under.

Do not hesitate to ask if you need more clarification. We're happy to answer
and to comply as much as possible to fedora's rules.

best regards,
Gratien

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

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


[Bug 476404] Review Request: bullet - 3D Collision Detection and Rigid Body Dynamics Library

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka  changed:

   What|Removed |Added

   Flag||needinfo?(br...@gnoll.org)




--- Comment #17 from Mamoru Tasaka   2009-02-13 
10:49:19 EDT ---
ping?

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

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


[Bug 478470] Review Request: mrpt - The Mobile Robot Programming Toolkit (MRPT)

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka  changed:

   What|Removed |Added

   Flag|needinfo?(joseluisblan...@g |
   |mail.com)   |




--- Comment #22 from Mamoru Tasaka   2009-02-13 
10:48:36 EDT ---
(The submitter replied to me that update will come in a few
 days)

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

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


[Bug 463767] Review Request: cloog - The Chunky Loop Generator

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #27 from Mamoru Tasaka   2009-02-13 
10:42:01 EDT ---
Well:

* Not a blocker, however "rpm -ivv cloog-ppl" bears some warning on
  info file:
-
D: adding 4 entries to Filedigests index.
D:   install: %post(cloog-ppl-0.15-0.3.gitad322.fc11.i386) synchronous
scriptlet start
D:   install: %post(cloog-ppl-0.15-0.3.gitad322.fc11.i386)  execv(/bin/sh)
pid 5361
+ /sbin/ldconfig
+ /sbin/install-info /usr/share/info/cloog.info /usr/share/info/dir
install-info: warning: no info dir entry in `/usr/share/info/cloog.info'
-

* Usually for a packager needing sponsorship, I request him/her to
  either
  - submit another review request
  - or do a (at least one) pre-review of other person's review
request
  as written on
  http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored .

  Do you have another review request of yours or review requests
  on which you did some pre-review?

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

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

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


[Bug 470155] Review Request: xword - Reads and writes crossword puzzles in the Across Lite file format

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka  changed:

   What|Removed |Added

   Flag||needinfo?(alexes...@yahoo.c
   ||om)




--- Comment #10 from Mamoru Tasaka   2009-02-13 
10:44:10 EDT ---
ping?

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

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


[Bug 476460] Review Request: pymilter - Python interface to sendmail milter API

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka  changed:

   What|Removed |Added

   Flag||needinfo?(stu...@gathman.or
   ||g)




--- Comment #26 from Mamoru Tasaka   2009-02-13 
10:46:36 EDT ---
ping?

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

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


[Bug 485420] Review Request: openbios - Open Source implementation of IEEE 1275-1994

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mark McLoughlin  changed:

   What|Removed |Added

 CC||mar...@redhat.com




--- Comment #2 from Mark McLoughlin   2009-02-13 10:25:26 
EDT ---
Reviewers: see bug #464621 for a review of similar package (etherboot)

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mark McLoughlin  changed:

   What|Removed |Added

 CC||mar...@redhat.com




--- Comment #4 from Mark McLoughlin   2009-02-13 10:24:55 
EDT ---
Reviews: see bug #464621 for a review of similar package (etherboot)

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mark McLoughlin  changed:

   What|Removed |Added

 CC||mar...@redhat.com




--- Comment #5 from Mark McLoughlin   2009-02-13 10:25:10 
EDT ---
Reviewers: see bug #464621 for a review of similar package (etherboot)

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

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


[Bug 485420] Review Request: openbios - Open Source implementation of IEEE 1275-1994

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Eduardo Habkost  changed:

   What|Removed |Added

 CC||ehabk...@redhat.com




--- Comment #1 from Eduardo Habkost   2009-02-13 10:21:43 
EDT ---
> %files
> %defattr(-,root,root,-)
> %dir %{_datadir}/openbios/openbios-sparc32
> %dir %{_datadir}/openbios/openbios-sparc64
> #%dir %{_datadir}/openbios/openbios-ppc

Shouldn't the package own the %{_datadir}/openbios directory?

http://fedoraproject.org/wiki/Packaging/UnownedDirectories

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #4 from Glauber de Oliveira Costa   2009-02-13 
10:21:47 EDT ---
1) yeah, we do own vgabios directory. I now remember having this problem with
etherboot too, but forgot about it. 

2) Forgot again

3) Got from a template, slipped through my fingers.

I'll update the packages.

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Eduardo Habkost   2009-02-13 10:18:44 
EDT ---
> %doc

Uh?

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Eduardo Habkost   2009-02-13 10:17:09 
EDT ---
> %install

rm -rf $RPM_BUILD_ROOT

http://fedoraproject.org/wiki/Packaging/Guidelines#Prepping_BuildRoot_For_.25install

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

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


[Bug 475017] Review Request: l2fprod-common - In JavaSE missing Swing components, inspired from modern user interfaces

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Mamoru Tasaka   2009-02-13 
10:14:30 EDT ---
I will recheck this package once the srpm is renewed.

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Eduardo Habkost   2009-02-13 10:16:45 
EDT ---
> %install

rm -rf $RPM_BUILD_ROOT

http://fedoraproject.org/wiki/Packaging/Guidelines#Prepping_BuildRoot_For_.25install

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

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


[Bug 485418] Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Eduardo Habkost  changed:

   What|Removed |Added

 CC||ehabk...@redhat.com




--- Comment #1 from Eduardo Habkost   2009-02-13 10:13:44 
EDT ---
> %files
> %defattr(-,root,root,-)
> %{_datadir}/vgabios/VGABIOS-lgpl-latest.bin
> %{_datadir}/vgabios/VGABIOS-lgpl-latest.cirrus.bin
> %{_datadir}/vgabios/VGABIOS-lgpl-latest.cirrus.debug.bin
> %{_datadir}/vgabios/VGABIOS-lgpl-latest.debug.bin

Your package also owns %{_datadir}/vgabios, doesn't it?

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

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


[Bug 225797] Review Request: gimp-data-extras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #17 from Nils Philippsen   2009-02-13 10:11:50 
EDT ---
(In reply to comment #16)
> > I've checked with upstream and they told me I should consider it as GPLv2+.
> [...]
> > I can attach an IRC log snippet if necessary.
> 
> Can you include that in the package?  I checked with spot and he indicated 
> that
> this is OK as long as you're reasonably certain that the person you were
> communicating with is the copyright holder.  Obviously it will be academic 
> once
> a version is out, but until then we have to clarify the license.

I've included it as "license-clarification.txt" in the package and it gets
installed along with the other documentation.

New files:
Spec file: http://nphilipp.fedorapeople.org/review/gimp-data-extras.spec
SRPM file:
http://nphilipp.fedorapeople.org/review/gimp-data-extras-2.0.2-0.3.fc10.src.rpm

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Eduardo Habkost   2009-02-13 10:09:37 
EDT ---
> mkdir -p prebuilt
> cd prebuilt
> tar -xzf %{SOURCE1}

You can probably use %setup here, with magic parameters to make it create
prebuilt/ and unpack only %{SOURCE1} inside it.

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

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


[Bug 485417] Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Eduardo Habkost  changed:

   What|Removed |Added

 CC||ehabk...@redhat.com




--- Comment #1 from Eduardo Habkost   2009-02-13 10:05:46 
EDT ---
> Summary:Portable x86 PC emulator

Is bochs-bios a "portable x86 PC emulator"?

> Patch0:   0001_bx-qemu.patch

I hope this doesn't break any guidelines on naming patches. I think using the
package name as a prefix for patch file names is good practice, even if not
explicitly required by Fedora.

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

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


[Bug 485424] New: Review Request: eclipse-birt - Eclipse-based reporting system

2009-02-13 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: eclipse-birt - Eclipse-based reporting system

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

   Summary: Review Request: eclipse-birt - Eclipse-based reporting
system
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: akurt...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://akurtakov.fedorapeople.org/eclipse-birt.spec
SRPM URL:
http://akurtakov.fedorapeople.org/eclipse-birt-2.3.1-1.fc10.src.rpm

Description: BIRT is an Eclipse-based open source reporting system for web
applications,
especially those based on Java and J2EE. BIRT has two main components: 
a report designer based on Eclipse, and a runtime component that you 
can add to your app server. BIRT also offers a charting engine that lets 
you add charts to your own application.

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

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


[Bug 485416] Review Request: msp430-gcc - Cross compiling GNU GCC for the MSP430

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Steve Whitehouse   2009-02-13 09:40:24 
EDT ---
Ok, I'll try and find time to have a look at this over the weekend.

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

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


[Bug 485418] New: Review Request: vgabios - vga option rom for bochs/qemu

2009-02-13 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: vgabios - vga option rom for bochs/qemu

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

   Summary: Review Request: vgabios - vga option rom for
bochs/qemu
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: gco...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://glommer.fedorapeople.org/vgabios.spec
SRPM URL: http://glommer.fedorapeople.org/vgabios-0.6-0.1beta.fc11.src.rpm
Description:
vgabios is an LPGL implementation of a bios for a video card.
It is tied to plex86/bochs, althoug it will likely work on other
emulators. It is not intended for use in real cards.

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

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


[Bug 485416] Review Request: msp430-gcc - Cross compiling GNU GCC for the MSP430

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from Robert Spanton   2009-02-13 09:32:58 
EDT ---
Created an attachment (id=331830)
 --> (https://bugzilla.redhat.com/attachment.cgi?id=331830)
koji ppc64 build.log ending in failure.

As documented in specfile, build fails in koji for ppc64.  I attach the
build.log for this failure.

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

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


[Bug 485420] New: Review Request: openbios - Open Source implementation of IEEE 1275-1994

2009-02-13 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: openbios - Open Source implementation of IEEE 1275-1994

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

   Summary: Review Request: openbios - Open Source implementation
of IEEE 1275-1994
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: gco...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://glommer.fedorapeople.org/openbios.spec
SRPM URL: http://glommer.fedorapeople.org/openbios-1.0-0.1.svn450.fc11.src.rpm
Description: 
The OpenBIOS project provides you with most free and open source Open Firmware
implementations available. Here you find several implementations of
IEEE 1275-1994 (Referred to as Open Firmware) compliant firmware. Among its
features, Open Firmware provides an instruction set independent device
interface.
This can be used to boot the operating system from expansion cards without
native initialization code.

It is Open Firmware's goal to work on all common platforms, like x86, AMD64,
PowerPC, ARM and Mips. With its flexible and modular design, Open Firmware
targets servers, workstations and embedded systems, where a sane and unified
firmware is a crucial design goal and reduces porting efforts noticably.

Open Firmware is found on many servers and workstations and there are sever
commercial implementations from SUN, Firmworks, CodeGen, Apple, IBM and others.

In most cases, the Open Firmware implementations provided on this site rely on
an additional low-level firmware for hardware initialization, such as coreboot
or U-Boot.

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

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


[Bug 485416] New: Review Request: msp430-gcc - Cross compiling GNU GCC for the MSP430

2009-02-13 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: msp430-gcc - Cross compiling GNU GCC for the MSP430

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

   Summary: Review Request: msp430-gcc - Cross compiling GNU GCC
for the MSP430
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rspan...@zepler.net
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://users.ecs.soton.ac.uk/rds/rpm/mspgcc/msp430-gcc.spec
SRPM URL:
http://users.ecs.soton.ac.uk/rds/rpm/mspgcc/msp430-gcc-3.2.3-1.20090210cvs.fc10.src.rpm
Description: This is a Cross Compiling version of GNU GCC, which can be used to
compile for the msp430 platform, instead of for the native i386
platform.

The mspgcc project has maintained patches against gcc 3.2.3 for a long time. 
The documented procedure for installing mspgcc is taking their patches and
applying them against gcc.

I based this on the avr-gcc spec.

One rpmlint warning: 
msp430-gcc.spec:46: W: rpm-buildroot-usage %prep sed -e 's,find
$RPM_BUILD_ROOT,find $RPM_BUILD_ROOT%_bindir $RPM_BUILD_ROOT%_libexecdir,' $a >
$b

I don't believe this error is fatal.  The script needs to modify the
os_install_post script so that the build system doesn't try to process the
msp430 binaries with the tools for the host's architecture.

Build fails on ppc64 in koji, but succeeds on all the other arches (I shall
attach the error message to this bug shortly).

I'd really appreciate a review of this.  Thanks :-D

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

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


[Bug 485417] New: Review Request: bochs-bios - bios implementation from the bochs project

2009-02-13 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: bochs-bios - bios implementation from the bochs project

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

   Summary: Review Request: bochs-bios - bios implementation from
the bochs project
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: gco...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://glommer.fedorapeople.org/bochs-bios.spec
SRPM URL:
http://glommer.fedorapeople.org/bochs-bios-2.3.8-0.1.git36989b0d2.fc11.src.rpm
Description:
This is the BIOS to be used in the Bochs Project

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

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


[Bug 484704] Review Request: libapogee - Library for Apogee CCD Cameras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Sergio Pascual  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Sergio Pascual   2009-02-13 
09:15:39 EDT ---
New Package CVS Request
===
Package Name: libapogee
Short Description: Library for Apogee CCD Cameras
Owners: sergiopr
Branches: F-10 F-9
InitialCC:

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

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


[Bug 472525] Review Request: kio_sysinfo - KIO slave which shows basic system information

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jaroslav Reznik  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Jaroslav Reznik   2009-02-13 09:12:34 
EDT ---
New Package CVS Request
===
Package Name: kio_sysinfo
Short Description: KIO slave which shows basic system information
Owners: jreznik ltinkl kkofler rdieter than
Branches: F-9 F-10
InitialCC:

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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 CC||allis...@gmail.com




--- Comment #1 from Itamar Reis Peixoto   2009-02-13 
09:07:38 EDT ---
koji scratch build in dist-f11

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

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

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


[Bug 472525] Review Request: kio_sysinfo - KIO slave which shows basic system information

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Lukáš Tinkl   2009-02-13 09:07:12 EDT ---
APPROVED

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

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


[Bug 484704] Review Request: libapogee - Library for Apogee CCD Cameras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Jaroslav Reznik  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Jaroslav Reznik   2009-02-13 08:56:23 
EDT ---
Ok, thank you!

- license is OK
- build is OK

APPROVED

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

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


[Bug 485007] Review Request: rhnpush - Package uploader for the RHN Satellite/Spacewalk Server

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Marcela Maslanova  changed:

   What|Removed |Added

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




--- Comment #1 from Marcela Maslanova   2009-02-13 
08:46:35 EDT ---
FAIL source files match upstream
The source should be written as whole path.
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field (same as Perl) matches the actual license.
OK license is open source-compatible. License text not included upstream.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock (Rawhide/x86_64).
http://koji.fedoraproject.org/koji/taskinfo?taskID=1124650
OK debuginfo package isn't need.
OK rpmlint is silent.
OK final provides and requires look sane.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.

Shouldn't be test executed as a part of build process?
(btw you have ugly man page of rhnpush the synopsis part.)

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

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


[Bug 472525] Review Request: kio_sysinfo - KIO slave which shows basic system information

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lukáš Tinkl  changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #4 from Lukáš Tinkl   2009-02-13 08:37:56 EDT ---
RPMLINT:

[lti...@localhost ~/tmp/kio_sysinfo-20081121]$ rpmlint kio_sysinfo
kio_sysinfo.speckio_sysinfo-20081121-1.fc9.src.rpm
[lti...@localhost ~/tmp/kio_sysinfo-20081121]$ rpmlint
kio_sysinfo-20081121-1.fc9.src.rpm
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

MUST: Packages containing GUI applications must include a %{name}.desktop file,
and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.

The package cleanly built and meets all the required MUST's with the above
exception, I still think it's not necessary to do desktop-file-install as this
application provides a service, not a standalone application.

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

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


[Bug 466953] Review Request: perl-NOCpulse-Utils - NOCpulse utility packages

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Miroslav Suchy  changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #8 from Miroslav Suchy   2009-02-13 08:23:01 EDT 
---
New Package CVS Request
===
Package Name: perl-NOCpulse-Utils
Short Description: NOCpulse utility packages
Owners: msuchy
Branches: F-10, EL-4, EL-5
InitialCC: perl-sig

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

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


[Bug 472525] Review Request: kio_sysinfo - KIO slave which shows basic system information

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Lukáš Tinkl  changed:

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 485207] Review Request: perl-Test-Assert - Assertion methods for those who like JUnit

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Allisson Azevedo  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.

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


[Bug 485402] Review Request: libytnef - TNEF Stream Parser Library

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NOTABUG




--- Comment #1 from Itamar Reis Peixoto   2009-02-13 
08:13:02 EDT ---
this package alrealy exists in fedora, closing ticket

https://admin.fedoraproject.org/pkgdb/packages/name/libytnef

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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 485403 depends on bug 485402, which changed state.

Bug 485402 Summary: Review Request: libytnef - TNEF Stream Parser Library
https://bugzilla.redhat.com/show_bug.cgi?id=485402

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||NOTABUG



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

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


[Bug 485401] Review Request: KyaPanel - Servers Manager The easy way to admin Postfix and Samba Servers.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 485401 depends on bug 485402, which changed state.

Bug 485402 Summary: Review Request: libytnef - TNEF Stream Parser Library
https://bugzilla.redhat.com/show_bug.cgi?id=485402

   What|Old Value   |New Value

 Status|NEW |CLOSED
 Resolution||NOTABUG



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

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


[Bug 466953] Review Request: perl-NOCpulse-Utils - NOCpulse utility packages

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Marcela Maslanova  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mmasl...@redhat.com
   Flag||fedora-review+




--- Comment #7 from Marcela Maslanova   2009-02-13 
08:08:27 EDT ---
OK source files match upstream: fc962eb4a529f467911494d9d508512f
OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible. License text not included upstream.
OK latest version is being packaged.
OK BuildRequires are proper.
OK %clean is present.
OK package builds in mock (Rawhide/x86_64).
OK debuginfo package isn't need.
OK rpmlint is silent.
OK final provides and requires look sane.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK no duplicates in %files.
OK file permissions are appropriate.
OK no scriptlets present.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.

The installation is ugly as was mentioned before. But this is not blocker for a
review according to guidelines.

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

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


[Bug 485154] Review Request: perl-MooseX-Role-Cmd - Wrap system command binaries the Moose way

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Allisson Azevedo  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.

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


[Bug 485206] Review Request: perl-Hash-Util-FieldHash-Compat - Use Hash::Util::FieldHash or ties, depending on availability

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Allisson Azevedo  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.

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


[Bug 484704] Review Request: libapogee - Library for Apogee CCD Cameras

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Sergio Pascual   2009-02-13 
07:34:54 EDT ---
Spec URL: http://sergiopr.fedorapeople.org/libapogee.spec
SRPM URL: http://sergiopr.fedorapeople.org/libapogee-2.2-2.fc10.src.rpm

I have fixed the license and submitted a new bug about library-calling-exit

Regarding ppc ppc64, there was already a bug filled about the problem. The
solutions seems to be as easy as removing sys/io.h includes. 

https://sourceforge.net/tracker2/?func=detail&aid=2215787&group_id=90275&atid=593019

So I have created a patch that does exactly that. The package now builds in
koji:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1124525

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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Blocks||485401




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

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


[Bug 485401] Review Request: KyaPanel - Servers Manager The easy way to admin Postfix and Samba Servers.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Depends on||485403




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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Blocks|485401  |
  Alias||ytnef




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

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


[Bug 485401] Review Request: KyaPanel - Servers Manager The easy way to admin Postfix and Samba Servers.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Depends on|485403  |




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

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


[Bug 485401] Review Request: KyaPanel - Servers Manager The easy way to admin Postfix and Samba Servers.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Depends on||485403




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

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


[Bug 485403] New: Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 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: ytnef - Yerase's TNEF Stream Reader.

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

   Summary: Review Request: ytnef - Yerase's TNEF Stream Reader.
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ita...@ispbrasil.com.br
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ispbrasil.com.br/ytnef/ytnef.spec
SRPM URL: http://ispbrasil.com.br/ytnef/ytnef-2.6-2.src.rpm
Description: 

Yerase's TNEF Stream Reader.  Can take a TNEF Stream (winmail.dat) sent from
Microsoft Outlook (or similar products) and extract the attachments, including
construction of Contact Cards & Calendar entries.

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

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


[Bug 485402] Review Request: libytnef - TNEF Stream Parser Library

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Blocks||485403




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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Depends on||485402




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

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


[Bug 485403] Review Request: ytnef - Yerase's TNEF Stream Reader.

2009-02-13 Thread bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto  changed:

   What|Removed |Added

 Blocks||485401




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

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


  1   2   >