[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 03:38 EST ---
Parag: Nils is the upstream for the package and - if I have understood correctly
- he implied that rather then incrementing the release tag, he prefers to
increase the version tag each time he modifies something. Basically this comes
down to the fact that he does not intend to really use the release tag, which is
why he ignores it in the Changelog. Only that this decision makes rpmlint 
unhappy.

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

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


[Bug 216536] Review Request: FuzzyOcr - Checks for specific keywords in image attachments, using gocr

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

Summary: Review Request: FuzzyOcr - Checks for specific keywords in image 
attachments, using gocr


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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


[Bug 221717] Review Request: agg - C++ rendering framework, move from core to shared

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

Summary: Review Request: agg - C++ rendering framework, move from core to shared


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:08 EST ---
(In reply to comment #14)
 (In reply to comment #11)
 
  The substantive issue is gpc, and in that case we're building with 
  --disable-gpc
  to disable building and including gpc in the final rpm
 
 It isn't enough, the srpm should not contain non free or
 legally problematic code. This should be reported upstream 
 anyway, upstream is violating gpc license and/or the GPL.

In fact it isn't sure that there is a violation as long as no 
binaries are redistributed (with gpc linked in), but still 
there should be no non-free file shipped in fedora srpm.

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

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


[Bug 222998] Review Request: iDesk - Desktop icons and background for minimal WMs

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

Summary: Review Request: iDesk - Desktop icons and background for minimal WMs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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


[Bug 209614] Review Request: wmmemload - windowmaker dock app

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

Summary: Review Request: wmmemload - windowmaker dock app


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:22 EST ---
What is the status of this submission?

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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:27 EST ---
According to http://fedoraproject.org/wiki/PackageReviewProcess you are supposed
to set flag fedora-review to ? and wait.  It's not your responsibility to look
for a reviewer, just let it rot here.

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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:36 EST ---
(In reply to comment #12)
 to set flag fedora-review to ? and wait.

No, my mistake.  The flag should be  , not ?.
I apologize that I'm messing with your bug.

So you are done now and should patiently wait.



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

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


[Bug 190189] Review Request: PyQt4: Python bindings for Qt4

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED




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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|DEFERRED|




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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Keywords|Reopened|




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

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


[Bug 226381] Merge Review: ruby

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

Summary: Merge Review: ruby


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




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

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


[Bug 226381] Merge Review: ruby

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

Summary: Merge Review: ruby


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:49 EST ---
Created an attachment (id=153401)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153401action=view)
missing directories list

For 1.8.6-2:

0. File entry:
* Static library
  - Is /usr/lib/libruby-static.a needed? If so, can this be split
into -static rpm?
Note: If this static archive cannot be removed, splitting into
  different subpackage is too late as F7T4 is frozen.

* Duplicate file entry
  - Please check the following.

[EMAIL PROTECTED] ruby]$ rpm -qlp /var/lib/mock/ruby-*i386*rpm | grep -v debug
| sort | uniq -d
/usr/lib/ruby
/usr/lib/ruby/1.8
/usr/share/ri/1.8/system/IRB/Context/irb_level-i.yaml
/usr/share/ri/1.8/system/IRB/ExtendCommand/Nop/irb-i.yaml
/usr/share/ri/1.8/system/IRB/ExtendCommandBundle/irb_context-i.yaml
/usr/share/ri/1.8/system/IRB/ExtendCommandBundle/irb_exit-i.yaml
/usr/share/ri/1.8/system/IRB/ExtendCommandBundle/irb_load-i.yaml
/usr/share/ri/1.8/system/IRB/ExtendCommandBundle/irb_original_method_name-c.yaml

/usr/share/ri/1.8/system/IRB/ExtendCommandBundle/irb_require-i.yaml
/usr/share/ri/1.8/system/IRB/IrbLoader/irb_load-i.yaml
/usr/share/ri/1.8/system/IRB/JobManager/irb-i.yaml
/usr/share/ri/1.8/system/IRB/JobManager/main_irb-i.yaml
/usr/share/ri/1.8/system/IRB/irb-c.yaml
/usr/share/ri/1.8/system/IRB/irb_abort-c.yaml
/usr/share/ri/1.8/system/IRB/irb_exit-c.yaml


* Missing directory ownership
  - Too many, the result is attached. 
1447 directories are not owned by any package.

( for f in `rpm -qlp /var/lib/mock/ruby-*i386*rpm | grep -v debug` ; do dirname
$f ; done ) | sort | uniq | LANG=C xargs rpm -qf | grep package

NOTE: Due to this, on my system many files related to ruby cannot be
  seen by non-root users. Several ruby modules package I currently
  submit for review request fail to be rebuilt.

* From build log:
  - For ruby-libs:

warning: File listed twice: /usr/lib/ruby
warning: File listed twice: /usr/lib/ruby/1.8
warning: File listed twice: /usr/lib/ruby/site_ruby
warning: File listed twice: /usr/lib/ruby/site_ruby/1.8
warning: File listed twice: /usr/lib/ruby/site_ruby/1.8/i386-linux
-

A. rpmlint - attached
Summary :
  - doc-file-dependency
Files included as %doc with executable permission adds additional
dependency to its package. This is generally avioded and documentations
should not have executable permission.

  - non-executable-script
Generally sourced scripts should not have shebangs. It there is some
exceptions for ruby package, please explain why.

  - spurious-executable-perm
Same as doc-file-dependency

  - wrong-script-interpreter
Setting interpreter the files under /usr/local is wrong.

Z. Question
* Automatically set CFLAGS
  - One strange thing for me is that fedora ruby rpms sets CFLAGS
as fedora specific compilation flags by default.

For example, currently ruby-zoom I submitted (bug 237381) is under
review. Then
- Just unpack ruby-zoom-0.2.2.tar.gz
- do cd ruby-zoom-0.2.2
- explicitly unset CFLAGS
- do ruby extconf.rb
Then Makefile is created. This Makefile sets CFLAGS as fedora specific
compilation flags by default. Is this a expected behavior?


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

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


[Bug 226381] Merge Review: ruby

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

Summary: Merge Review: ruby


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:50 EST ---
Created an attachment (id=153402)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153402action=view)
rpmlint log for ruby 1.8.6-2


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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:51 EST ---
[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||DEFERRED

So we can assume gdb not to be present in FC7?

Or is this an attempt of @RH's to demonstrate their ability to overrule the
community?





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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 04:58 EST ---
Ralf, I think this was a mistake, because the report was opened again one 
minute after it got closed.

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

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


[Bug 225807] Merge Review: glib-java

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

Summary: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG




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

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


[Bug 225807] Merge Review: glib-java

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

Summary: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|NOTABUG |




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

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


[Bug 225807] Merge Review: glib-java

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

Summary: Merge Review: glib-java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Keywords|Reopened|




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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:04 EST ---
(In reply to comment #15)
 Ralf, I think this was a mistake, because the report was opened again one 
 minute after it got closed.

I am more than willing to believe this, but ... given how this review and other
merge reviews (did not) proceeded doesn't give me much reason for trust.


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

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


[Bug 209614] Review Request: wmmemload - windowmaker dock app

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

Summary: Review Request: wmmemload - windowmaker dock app


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:12 EST ---
Hi Patrice, 

wmmemload uses internal libdockapp as well as wmcpuload. After a long time I've
about rewriten wmcpuload to work with libdockap 0.6 which is already in extras,
this will need the same work.

I'll get right on it once I submit wmcpuload again.



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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:15 EST ---
(In reply to comment #15)
 Ralf, I think this was a mistake, because the report was opened again one 
 minute after it got closed.

It was an attempt to get this Bug into the state NEW as formerly reopened Bugs
went to the state NEW (and they went to REOPENED state even before).
Unfortunately I found out the Bugzilla behavior changed.
It was being done to reinsert this Bug back to the list referenced from
http://fedoraproject.org/wiki/PackageReviewProcess by its link
https://bugzilla.redhat.com/bugzilla/buglist.cgi?product=Fedora+Extrascomponent=Package+Reviewbug_status=NEW
but unfortunately one would have to CLOSE this Bug completely and open a new one
to reach its NEW state again.

(In reply to comment #16)
 I am more than willing to believe this, but ... given how this review
...
 (did not) proceeded

The issues listed in Comment 1 were implemented in about 2 days - is it such a
tragical response time? During the implementation there was found one possible
security hole which was being requested to be evaluated by security response
entities but unfortunately this step took too long - it was not a real security
threat but noone was willing to give a certification for it.

It is a security policy to never disclose any security flaws until the fix is
known to protect the users of the product.  So far I am glad to be the White Hat
- not the Black Hat (relationship to Red Hat is just a coincidence here :-) ).


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

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


[Bug 203520] Review Request: evolution-brutus

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

Summary: Review Request: evolution-brutus
Alias: evolution-brutus

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:21 EST ---
Brian, by the way would you have a time to
review migemo review request I submitted (bug 236493)?

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

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


[Bug 225783] Merge Review: gdb

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

Summary: Merge Review: gdb


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:39 EST ---
rpmlint v. 0.80 is not silent on gdb src.rpm.
(rpmlint on the binary rpm is OK.)

Jan, your advice is needed for the not-applied-patch issue.
The rest are just a formal issues; is it OK with you if I fix them?

W: gdb patch-not-applied Patch197: gdb-6.5-bz198365-IPv6.patch
A patch is included in your package but was not applied.

W: gdb summary-ended-with-dot A GNU source-level debugger for C, C++, Java and
other languages.
Summary ends with a dot.

W: gdb unversioned-explicit-obsoletes gdb64
The specfile contains an unversioned Obsoletes: token, which will match all
older, equal and newer versions of the obsoleted thing.  This may cause update
problems, restrict future package/provides naming, and may match something it
was originally not inteded to match -- make the Obsoletes versioned if
possible.

E: gdb hardcoded-library-path in /usr/lib/libc.so
A library path is hardcoded to one of the following paths: /lib,
/usr/lib. It should be replaced by something like /%{_lib} or %{_libdir}.

W: gdb macro-in-%changelog check
W: gdb macro-in-%changelog p
Macros are expanded in %changelog too, which can in unfortunate cases lead
to the package not building at all, or other subtle unexpected conditions that
affect the build.  Even when that doesn't happen, the expansion results in
possibly rewriting history on subsequent package revisions and generally
odd entries eg. in source rpms, which is rarely wanted.  Avoid use of macros
in %changelog altogether, or use two '%'s to escape them, like '%%foo'.

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


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

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


[Bug 209614] Review Request: wmmemload - windowmaker dock app

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

Summary: Review Request: wmmemload - windowmaker dock app


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:44 EST ---
Ok. As a side note I am the libdockapp maintainer in fedora 
(was needed for wmacpi update). Be sure to use the libdockapp api
for command line args processing to have the -w option that
allows to run in a window. 

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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 05:46 EST ---
OK. As maintainer is also upstream developer so no issues then.

Review:
+ package builds in mock (development i386).
+ rpmlint is silent for for RPM.
- rpmlint is NOT silent for SRPM.
+ source files match upstream.
c817a01e5bf60e30458df5b4b90d27d7  hwbrowser-0.32.tar.gz
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text COPYING is included in package.
+ %doc is small so no need of -doc subpackage.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no static libraries.
+ no .pc files are present.
+ no -devel subpackage exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Desktop file installed correctly.
+ no scriptlets are used.
+ Provides: config(hwbrowser) = 0.32-1.fc7
+ GUI app.
APPROVED.


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

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


[Bug 170303] Review Request: google-perftools: Very fast malloc performance analysis tools

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

Summary: Review Request: google-perftools: Very fast malloc  performance 
analysis tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||
   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 07:08 EST ---
I believe, that period of pregnancy has ended :)

rpmlint: OK
MUST/SHOULD items: OK

APPROVED!

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

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


[Bug 170303] Review Request: google-perftools: Very fast malloc performance analysis tools

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

Summary: Review Request: google-perftools: Very fast malloc  performance 
analysis tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

URL|http://goog-|http://code.google.com/p/goo
   |perftools.sourceforge.net/  |gle-perftools/




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

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


[Bug 226387] Merge Review: samba

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

Summary: Merge Review: samba


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 07:34 EST ---
Created an attachment (id=153413)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153413action=view)
proposed .spec changes to handle winbindd -- winbind change

Consider this patch. Since %post section will be executed with the new code
(i.e. with winbind, not winbindd), just check for and rename winbindd
lock to winbind before the actual service stop.

And apply the latest winbind.init cosmetic changes to smb.init and nmb.init
too.

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

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


[Bug 226387] Merge Review: samba

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

Summary: Merge Review: samba


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


Bug 226387 depends on bug 203568, which changed state.

Bug 203568 Summary: split out a -devel package for libsmbclient
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=203568

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED



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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 08:06 EST ---
Parag, I already changed the version string in the changelog to 0.32-1, but
only in the upstream CVS, not the actual built package. Once a new version
(0.33) gets out, I'll surely remember why I put the -release in the changelog
and will continue to do so.

BTW, isn't someone supposed to close this if it is approved? Just curious...

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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #153353|0   |1
is obsolete||
 Attachment #153354|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 09:10 EST ---
Created an attachment (id=153418)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153418action=view)
mcpp.spec


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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 09:14 EST ---
Created an attachment (id=153419)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153419action=view)
mcpp-2.6.3-3.src.rpm

I have rewritten again the spec file in order to avoid duplication in %file,
and changed the installation of the documents.


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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 09:23 EST ---
Nils, once your package is approved you are supposed to ask creation of the CVS
bits (see warren's mail, cvs with flags ) and after you import the package and
build it in plague you are the one who should close this bz ticket.

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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 09:46 EST ---
Well,
* release number
  http://fedoraproject.org/wiki/Packaging/FrequentlyMadeMistakes
--
Increase the Release tag every time you upload a new package to avoid
confusion. The reviewer and other interested parties probably still have older
versions of your SRPM lying around to check what has changed between the old and
new packages; those get confused when the revision didn't change.
--
  So please bump release number.

  Well, then as I said on comment 6, would you do a pre-review of
  other person's review request or submit a new review request of other
  software? (usually people choose former)


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

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


[Bug 227579] Review Request: spr - Statistical pattern recognition

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

Summary: Review Request: spr - Statistical pattern recognition


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:32 EST ---
Imported and built.  Thanks!

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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:51 EST ---
(In reply to comment #14)
 Parag, I already changed the version string in the changelog to 0.32-1, but
 only in the upstream CVS, not the actual built package. Once a new version
 (0.33) gets out, I'll surely remember why I put the -release in the changelog
 and will continue to do so.
 
 BTW, isn't someone supposed to close this if it is approved? Just curious...

anyone can CLOSE this bug. The reason I have not closed it immediately is that I
thought you may also want to create EPEL cvs branch request.
As package is always in CVS there is no requirement for setting fedora-cvs flag
for Merge-Review packages.
So you can close this review.

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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #153418|0   |1
is obsolete||
 Attachment #153419|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:52 EST ---
Created an attachment (id=153424)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153424action=view)
mcpp-2.6.3-4.i386.rpm


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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #153424|0   |1
is obsolete||




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:53 EST ---
Created an attachment (id=153425)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153425action=view)
mcpp-2.6.3-4.src.rpm


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

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


[Bug 237742] Review Request: perl-Text-SimpleTable - Simple Eyecandy ASCII Tables

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

Summary: Review Request: perl-Text-SimpleTable - Simple Eyecandy ASCII Tables
Alias: Text-SimpleTable

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:56 EST ---
You are building with TEST_POD disabled:
...

t/02podskipped
all skipped: set TEST_POD to enable this test
t/03podcoverageskipped
all skipped: set TEST_POD to enable this test
...

Enabling them reveals these bugs:

+ TEST_POD=1
+ ./Build test
t/01useok
t/02podok
t/03podcoverageNOK 1 
#   Failed test 'Pod coverage on Text::SimpleTable'
#   in /usr/lib/perl5/vendor_perl/5.8.8/Test/Pod/Coverage.pm at line 126.
# Coverage for Text::SimpleTable is 66.7%, with 1 naked subroutine:
#   new
# Looks like you failed 1 test of 1.
t/03podcoveragedubious   
Test returned status 1 (wstat 256, 0x100)
DIED. FAILED test 1
Failed 1/1 tests, 0.00% okay
t/04tables.ok
Failed Test   Stat Wstat Total Fail  Failed  List of Failed
---
t/03podcoverage.t1   256 11 100.00%  1
Failed 1/4 test scripts, 75.00% okay. 1/7 subtests failed, 85.71% okay.
error: Bad exit status from /var/tmp/rpm-tmp.78813 (%check)

Though Pod-tests are not really important, this doesn't necessarily provide
trust into this package's quality.


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

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


[Bug 237373] Review Request: mcpp - Alternative C/C++ preprocessor

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

Summary: Review Request: mcpp -  Alternative C/C++ preprocessor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 10:58 EST ---
Created an attachment (id=153427)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153427action=view)
mcpp.spec

Increased the release number.

I have mis-uploaded a binary rpm, sorry.

As for pre-review or other review request, please give me a few days of time.


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

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


[Bug 203520] Review Request: evolution-brutus

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

Summary: Review Request: evolution-brutus
Alias: evolution-brutus

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:02 EST ---
(In reply to comment #98)
 Brian, by the way would you have a time to
 review migemo review request I submitted (bug 236493)?

Yeah, I'll give it a look later today.



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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:04 EST ---
Manuel, Parag, as this is a merge review I don't think I should be asking for
any CVS creation just yet: It will be in the combined Core+Extras for Fedora 7
and is part of RHEL5 anyway...

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

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


[Bug 237813] New: Review Request: taxipilot - Game where you pilot a taxi in space

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




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

   Summary: Review Request: taxipilot - Game where you pilot a taxi
in space
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.atrpms.net/~hdegoede/taxipilot.spec
SRPM URL: http://people.atrpms.net/~hdegoede/taxipilot-0.9.1-1.fc7.src.rpm
Description:
Game where you pilot a taxi in space, the objective is to pick up passengers
waiting on a number of platforms and to drop them where they want to go.
That's basically it.

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

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


[Bug 237817] New: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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




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

   Summary: Review Request: pam_thinkfinger - Fingerprint PAM and
usb libraries for Fedora
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://belial.dead.li/~jmp/thinkfinger/Fedora/thinkfinger.spec
SRPM URL: 
http://belial.dead.li/~jmp/thinkfinger/Fedora/SRPMS/pam_thinkfinger-0.3-2.src.rpm
Description: 
ThinkFinger is a driver for the UPEK/SGS Thomson Microelectronics fingerprint
reader (USB ID 0483:2016). The device is being found either as a standalone USB
device, built into USB keyboards or built into laptops.  The following laptop
vendors are using the device:

- Dell
- IBM/Lenovo
- Toshiba

Toshiba is shipping their laptops either with the UPEK/SGS Thomson
Microelectronics fingerprint reader or with a fingerprint reader built by
AuthenTec. The AuthenTec fingerprint reader is *not* supported by ThinkFinger.

SONY laptops with the UPEK/SGS Thomson Microelectronics fingerprint reader are
not supported.

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

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


[Bug 226276] Merge Review: perl

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

Summary: Merge Review: perl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:17 EST ---
*** Bug 237564 has been marked as a duplicate of this bug. ***

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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||237760
  nThis||




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

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


[Bug 231774] Review Request: perl-DBIx-POS - Define a dictionary of SQL statements in a POD dialect (POS)

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

Summary: Review Request: perl-DBIx-POS - Define a dictionary of SQL statements 
in a POD dialect (POS)
Alias: perl-DBIx-POS

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:27 EST ---
So we have 

a) D::P - Artistic or GPL
b) C::S - Artistic
c) License Tag - Artistic or GPL

If the license tag is Artistic or GPL, doens't it imply that the entire package
can be relicensed under the choice of Artistic or GPL?  Nothing I see in
Artistic license for C::S allows that AFAICS.



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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:38 EST ---
A few random questions regarding the design of the program itself, rather than
the package:
- Why does pam_thinkfinger need uinput when pam_usb (http://www.pamusb.org/)
doesn't?
- Why does it need threading (which is likely to cause problems with PAM) (apart
from a bad library design?)

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

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


[Bug 234791] Review Request: perl-Email-Send - Module for sending email

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

Summary: Review Request: perl-Email-Send -  Module for sending email
Alias: perl-Email-Send

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:38 EST ---

*** APPROVED ***


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

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


[Bug 234791] Review Request: perl-Email-Send - Module for sending email

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

Summary: Review Request: perl-Email-Send -  Module for sending email
Alias: perl-Email-Send

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:48 EST ---
New Package CVS Request
===
Package Name: perl-Email-Send
Short Description: Module for sending email
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6
InitialCC: [EMAIL PROTECTED]


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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:51 EST ---
Bastien,
  I was expecting these :-)
- uinput has been added by the maintainer to send \n after the swiped image. Now
this was working fine with the code from 0.2.. To me this is just a gadget and I
will add --with-uinput in configure to give the choice. Ubuntu,debian and other
folks already spoke to me about this.

- threading... same as above! a choice from the maintainer and that I will find
time sometime to get them out. Feel free to assist ;-)
Jose

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

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


[Bug 237045] Review Request: perl-Hash-WithDefaults - Class for hashes with key-casing requirements supporting defaults

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

Summary: Review Request: perl-Hash-WithDefaults - Class for hashes with 
key-casing requirements supporting defaults
Alias: Hash-WithDefaults

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 225892] Merge Review: hwbrowser

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

Summary: Merge Review: hwbrowser


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 11:58 EST ---
Me too I was thinking about EPEL when I was speaking about CVS. Anyway, I'd say
that the answer to closing the bug is here:
https://www.redhat.com/archives/fedora-extras-list/2007-February/msg00380.html

Please do correct me if I am wrong.

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

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


[Bug 237705] Review Request: xclip - Command line clipboard grabber

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

Summary: Review Request: xclip - Command line clipboard grabber


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:00 EST ---
New Package CVS Request
===
Package Name: xclip
Short Description: Command line clipboard grabber
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6


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

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


[Bug 237045] Review Request: perl-Hash-WithDefaults - Class for hashes with key-casing requirements supporting defaults

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

Summary: Review Request: perl-Hash-WithDefaults - Class for hashes with 
key-casing requirements supporting defaults
Alias: Hash-WithDefaults

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:15 EST ---
Package Review
==

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

=== REQUIRED ITEMS ===
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Rpmlint output: None
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format 
%{name}.spec.
 [x] Package meets the  Packaging Guidelines.
 [x] Package is licensed with an open-source compatible license and meet other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL or Artistic
 [-] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is written in American English.
 [x] Spec file for the package is legible.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package: e769b5f381e2b55ec53429b847f086c1
 MD5SUM upstream package: e769b5f381e2b55ec53429b847f086c1
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: FC-6 / i386
 [x] Package is not known to require ExcludeArch, OR:
 Arches excluded:
 Why:
 [x] All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package is not relocatable.
 [x] Package must own all directories that it creates.
 [-] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [-] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: FC-6 / i386
 [-] Package should compile and build into binary rpms on all supported
architectures.
 Tested on:
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files are correct.
 [-] File based requires are sane.
 [x] Latest version is packaged.

=== Issues ===
1.

=== Final Notes ===
1.



*** APPROVED ***


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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:16 EST ---
Isn't this a duplicate of the already available/approved 'thinkfinger' package? 
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229291

Perhaps you could talk to the maintainer of that package and offer improvements
and/or help co-maintain it? 

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

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


[Bug 237045] Review Request: perl-Hash-WithDefaults - Class for hashes with key-casing requirements supporting defaults

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

Summary: Review Request: perl-Hash-WithDefaults - Class for hashes with 
key-casing requirements supporting defaults
Alias: Hash-WithDefaults

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:20 EST ---
New Package CVS Request
===
Package Name: perl-Hash-WithDefaults
Short Description: Class for hashes with key-casing requirements supporting 
defaults
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: [EMAIL PROTECTED]

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

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


[Bug 237045] Review Request: perl-Hash-WithDefaults - Class for hashes with key-casing requirements supporting defaults

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

Summary: Review Request: perl-Hash-WithDefaults - Class for hashes with 
key-casing requirements supporting defaults
Alias: Hash-WithDefaults

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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

Summary: Review Request: perl-JSON - Parse and convert to JSON (JavaScript 
Object Notation)
Alias: perl-JSON

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:24 EST ---
(In reply to comment #3)
 Isn't this a duplicate of the already available/approved 'thinkfinger' 
 package? 
 https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=229291
 
 Perhaps you could talk to the maintainer of that package and offer 
 improvements
 and/or help co-maintain it? 

It is indeed. As Jose is also helping upstream, it would be good if the original
package could include some of Jose's patches.

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

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


[Bug 237817] Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries for Fedora

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

Summary: Review Request: pam_thinkfinger - Fingerprint PAM and usb libraries 
for Fedora


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:26 EST ---
Also the guy is aware of this, CCing him.


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

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


[Bug 237817] Merge some additional packages

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

Summary: Merge some additional packages


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: |Merge some additional
   |pam_thinkfinger -   |packages
   |Fingerprint PAM and usb |
   |libraries for Fedora|
  Component|Package Review  |thinkfinger
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
  QAContact|fedora-package- |[EMAIL PROTECTED]
   |[EMAIL PROTECTED]   |




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:28 EST ---
Moving to thinkfinger. Julian, could you check with Jose which patches you'd
want to include in the package? The README might be useful as well.

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

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


[Bug 221027] Review Request: LabPlot - Data Analysis and Visualization

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

Summary: Review Request: LabPlot - Data Analysis and Visualization


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:28 EST ---
Well, please change release number properly.
Other things are okay.

-
   This package (LabPlot) is APPROVED by me
-

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

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


[Bug 237416] Review Request: Berusky - 2D logic game based on an ancient puzzle Sokoban.

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

Summary: Review Request: Berusky - 2D logic game based on an ancient puzzle 
Sokoban.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:32 EST ---
Good:
+ Tar ball maches with upstream for package berusky-data

Bad:
- The is no version 1.1 of berusky, there is only a 1.0 version.
- Tab ball differ from upstream for package berusky.
- Strange directory /var/games in package berusky-data. Perhaps it shold be
/usr/games?
 


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

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


[Bug 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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

Summary: Review Request: perl-JSON - Parse and convert to JSON (JavaScript 
Object Notation)
Alias: perl-JSON

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:34 EST ---
Package Review
==

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

=== REQUIRED ITEMS ===
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Rpmlint output: None
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format 
%{name}.spec.
 [x] Package meets the  Packaging Guidelines.
 [x] Package is licensed with an open-source compatible license and meet other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL or Artistic
 [-] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is written in American English.
 [x] Spec file for the package is legible.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 MD5SUM this package: c1e50efa98c9f7c1907b0dfd6b2656a9
 MD5SUM upstream package: c1e50efa98c9f7c1907b0dfd6b2656a9
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: FC-6 / i386
 [x] Package is not known to require ExcludeArch, OR:
 Arches excluded:
 Why:
 [x] All build dependencies are listed in BuildRequires, except for any that are
listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package is not relocatable.
 [x] Package must own all directories that it creates.
 [-] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf %{buildroot} (or
$RPM_BUILD_ROOT).
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [-] Reviewer should test that the package builds in mock.
 Tested on: FC-6 / i386
 [-] Package should compile and build into binary rpms on all supported
architectures.
 Tested on:
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files are correct.
 [-] File based requires are sane.
 [x] Latest version is packaged.

=== Issues ===
1.

=== Final Notes ===
1.



*** APPROVED ***


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

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


[Bug 225522] Review Request: cinepaint - CinePaint is a tool for manipulating images

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

Summary: Review Request: cinepaint - CinePaint is a tool for manipulating images


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:35 EST ---
Created an attachment (id=153429)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=153429action=view)
mock build log of cinepaint 0.22.0-1 on FC-devel i386 (autoconf added)

Mock build failed FC-devel i386 because of missing BR
about autoconf.

Even if I added autoconf as BR, still mockbuild failed
as attached.

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

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


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

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

Summary: Merge Review: gimp-data-extras


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||CANTFIX
   Flag|fedora-review?  |fedora-review-




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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:38 EST ---
ping?

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

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


[Bug 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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

Summary: Review Request: perl-JSON - Parse and convert to JSON (JavaScript 
Object Notation)
Alias: perl-JSON

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:39 EST ---
New Package CVS Request
===
Package Name: perl-JSON
Short Description: Parse and convert to JSON (JavaScript Object Notation)
Owners: [EMAIL PROTECTED]
Branches: FC-5, FC-6, devel
InitialCC: [EMAIL PROTECTED]

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

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


[Bug 237041] Review Request: perl-JSON - Parse and convert to JSON (JavaScript Object Notation)

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

Summary: Review Request: perl-JSON - Parse and convert to JSON (JavaScript 
Object Notation)
Alias: perl-JSON

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 226455] Merge Review: system-config-date

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

Summary: Merge Review: system-config-date


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 225248] Merge Review: ant

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

Summary: Merge Review: ant


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 12:50 EST ---
God:
+ BR fixed.
+ Rpmlint was OK (with warnings)

*** APPrOVED ***

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

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


[Bug 203520] Review Request: evolution-brutus

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

Summary: Review Request: evolution-brutus
Alias: evolution-brutus

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 13:15 EST ---
New Package CVS Request
===
Package Name: evolution-brutus
Short Description: Brutus based Exchange connector for Evolution
Owners: [EMAIL PROTECTED]
Branches: 
InitialCC: [EMAIL PROTECTED]

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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 13:18 EST ---
I'm doing terrible with this review :) Thanks for the prodding.
I'll fix up the macros when I import to CVS

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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 13:19 EST ---
New Package CVS Request
===
Package Name: stgit
Short Description: StGIT provides similar functionality to Quilt on top of GIT
Owners: [EMAIL PROTECTED]
Branches: FC-6
InitialCC: 

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

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


[Bug 203520] Review Request: evolution-brutus

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

Summary: Review Request: evolution-brutus
Alias: evolution-brutus

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 231758] Review Request: perl-Workflow - Simple, flexible system to implement workflows

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

Summary: Review Request: perl-Workflow - Simple, flexible system to implement 
workflows
Alias: perl-Workflow

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 233783] Review Request: vegastrike-data - Data files for Vega Strike

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

Summary: Review Request: vegastrike-data - Data files for Vega Strike


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 234784] Review Request: perl-Email-Date - Find and format date headers

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

Summary: Review Request: perl-Email-Date - Find and format date headers
Alias: perl-Email-Date

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:06 EST ---
Package Change Request
==
Package Name: stgit
Updated Fedora Owners: [EMAIL PROTECTED], [EMAIL PROTECTED]

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

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


[Bug 225114] Review Request: jamin - JACK Audio Mastering interface

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

Summary: Review Request: jamin - JACK Audio Mastering interface


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:07 EST ---
Just to confirm...

No one has reviewed this for 3 months. Florin, 
do you still want to import this to Fedora?

This is just I want to make it sure if you are
still watching this review request.

(In reply to comment #2)
 I don't know if I have to bump the Release number after every rebuild like
 this, so I'm just using the same number. 
You *have to*.

Well, from just I glanced at your spec file:
* Source URL
  - Please refer to
http://fedoraproject.org/wiki/Packaging/SourceURL
for sourceforge.net source URL

* Using autotool
  - Generally calling autotool is not preferred for
several reasons.
If possible, patch against configure, not configure.in,
for example

* Deprecated categories
  - Categories Application X-Fedora are both deprecated
for desktop file categories and should be removed.

* Directory ownership
  - _Perhaps_ %{_libdir}/ladspa is not owned by any package
(I have not tried to rebuild this package)

* %exclude
  - Please don't use %exclude when it can. Just remove
the file at %install stage.

* Mimeinfo
  - Please check mimeinfo section of
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets

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

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


[Bug 234791] Review Request: perl-Email-Send - Module for sending email

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

Summary: Review Request: perl-Email-Send -  Module for sending email
Alias: perl-Email-Send

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 227032] Review Request: asm-1.5.3-2jpp - A code manipulation tool to implement adaptable systems

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

Summary: Review Request: asm-1.5.3-2jpp - A code manipulation tool to implement 
adaptable systems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:37 EST ---
This package was originally an indirect dependency of maven2, but is no longer
needed.

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

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


[Bug 227048] Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM Conformance Test Suite

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

Summary: Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM 
Conformance Test Suite


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
OtherBugsDependingO|163776  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:39 EST ---
This package was needed by maven2, but that is no longer the case.

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

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


[Bug 227099] Review Request: plexus-cdc-1.0-0.a4.2jpp - Plexus Component Descriptor Creator

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

Summary: Review Request: plexus-cdc-1.0-0.a4.2jpp - Plexus Component Descriptor 
Creator


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]   |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:40 EST ---
Re-assigning to reviewer. Package is in extras, please close.

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

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


[Bug 227122] Review Request: xmlbeans-2.1.0-3jpp - XML-Java binding tool

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

Summary: Review Request: xmlbeans-2.1.0-3jpp - XML-Java binding tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||WONTFIX
OtherBugsDependingO|163776  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 14:43 EST ---
This package used to be an indirect dependency of maven2, but was found to be
non-critical, and hence disabled for the time being.

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

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


[Bug 231758] Review Request: perl-Workflow - Simple, flexible system to implement workflows

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

Summary: Review Request: perl-Workflow - Simple, flexible system to implement 
workflows
Alias: perl-Workflow

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:01 EST ---
Imported, built, and all that goodness.  Thanks for the review! :)

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

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


[Bug 231755] Review Request: perl-Class-Observable - Allow other classes and objects to respond to events in yours

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

Summary: Review Request: perl-Class-Observable - Allow other classes and 
objects to respond to events in yours


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




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

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


[Bug 232612] Review Request: perl-Class-Observable - Allow other classes and objects to respond to events in yours

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

Summary: Review Request: perl-Class-Observable - Allow other classes and 
objects to respond to events in yours


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




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

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


[Bug 231753] Review Request: perl-DBD-Mock - Mock database driver for testing

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

Summary: Review Request: perl-DBD-Mock - Mock database driver for testing
Alias: perl-DBD-Mock

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




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

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


[Bug 231754] Review Request: perl-Class-Factory - Base class for dynamic factory classes

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

Summary: Review Request: perl-Class-Factory - Base class for dynamic factory 
classes
Alias: perl-Class-Factory

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium




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

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


[Bug 234717] Review Request: python-spambayes - Bayesian anti-spam filter

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

Summary: Review Request: python-spambayes - Bayesian anti-spam filter
Alias: spambayes

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:03 EST ---
Trond, did you get your bugzilla accounts sorted out?

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

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


[Bug 225628] Merge Review: bsh

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

Summary: Merge Review: bsh


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:05 EST ---
Any update on this?

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

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


[Bug 230275] Review Request: varnish - High-performance HTTP accelerator

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

Summary: Review Request: varnish - High-performance HTTP accelerator


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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:06 EST ---
Ugh, fixec. src.rpm and spec in the same location.

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

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


[Bug 234717] Review Request: python-spambayes - Bayesian anti-spam filter

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

Summary: Review Request: python-spambayes - Bayesian anti-spam filter
Alias: spambayes

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





--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:19 EST ---
fwiw, rpmlint is clean on both src.rpm and built .noarch rpm.  neat.

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

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


[Bug 227946] Review Request: stgit - StGIT provides similar functionality to Quilt on top of GIT

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

Summary: Review Request: stgit - StGIT provides similar functionality to Quilt 
on top of GIT


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 227032] Review Request: asm-1.5.3-2jpp - A code manipulation tool to implement adaptable systems

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

Summary: Review Request: asm-1.5.3-2jpp - A code manipulation tool to implement 
adaptable systems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|WONTFIX |




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:26 EST ---
On second thought, I think I will keep this open. Although it is non-critical,
it should really be added into Fedora at some point..

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

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


[Bug 227048] Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM Conformance Test Suite

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

Summary: Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM 
Conformance Test Suite


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|CLOSED  |ASSIGNED
   Keywords||Reopened
 Resolution|WONTFIX |




--- Additional Comments From [EMAIL PROTECTED]  2007-04-25 15:26 EST ---
On second thought, I think I will keep this open. Although it is non-critical,
it should really be added into Fedora at some point..

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

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


  1   2   >