[Bug 428894] Review Request: rudesocket - Library (C++ API) for creating client sockets

2008-02-05 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: rudesocket - Library (C++ API) for creating client 
sockets


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 03:00 EST ---
Is this built for F-7 and F-8?

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

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


[Bug 431414] Review Request: planets - A celestial simulator

2008-02-05 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: planets - A celestial simulator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 03:06 EST ---
As this package uses ocaml, I think good to follow same what ocaml used in SPEC 
=
ExclusiveArch:  alpha armv4l %{ix86} ia64 x86_64 ppc sparc


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

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


[Bug 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-02-05 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-Net-eBay - Perl Interface to XML based eBay API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 03:34 EST ---
hey it worked now :)

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

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


[Bug 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-02-05 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-Net-eBay - Perl Interface to XML based eBay API


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 03:29 EST ---
hmm, weird, it works for me, I tested again this morning.

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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 04:23 EST ---

Greg, Thanks for the patch. I managed to get it build across all the
architectures with this patch applied. Unfortunate that other arch's besides x86
wont get voice support though. 

Revised spec and SRPM:

http://sundaram.fedorapeople.org/packages/gyachi.spec
http://sundaram.fedorapeople.org/packages/gyachi-1.1.0-6.fc8.src.rpm

Koji scratch build:

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

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

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


[Bug 426492] Review Request: jabbin - Instant messaging and VoIP Jabber client

2008-02-05 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: jabbin - Instant messaging and VoIP Jabber client


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 04:22 EST ---
Fixed
Spec URL: http://kurzawa.nonlogic.org/rpm/jabbin/jabbin.spec
SRPM URL: http://kurzawa.nonlogic.org/rpm/jabbin/jabbin-2.0-3.beta2a.fc8.src.rpm

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

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


[Bug 431249] Review Request: sawfish - An extensible window manager for the X Window System.

2008-02-05 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: sawfish - An extensible window manager for the X 
Window System.


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 04:46 EST ---
[EMAIL PROTECTED] ~]$ rpmlint sawfish-1.3.2-1.fc8.src.rpm 
W: sawfish summary-ended-with-dot An extensible window manager for the X Window
System.
W: sawfish invalid-license GPLv2
W: sawfish strange-permission sawfish.spec 0444
E: sawfish no-cleaning-of-buildroot %clean

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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 04:43 EST ---
Please check directory ownership issue.
For example, %_datadir/%name is not owned by any package.

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

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


[Bug 431249] Review Request: sawfish - An extensible window manager for the X Window System.

2008-02-05 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: sawfish - An extensible window manager for the X 
Window System.


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 04:52 EST ---
The Changelog says 
* Thu Jan 31 2008 Rahul Sundaram [EMAIL PROTECTED] -1.3.2-1
and the package is submitted by Ritesh Khadgaray ([EMAIL PROTECTED])
any specific reason for that?

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

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


[Bug 431381] Review Request: unicap - Library to access different kinds of (video) capture devices

2008-02-05 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: unicap - Library to access different kinds of (video) 
capture devices


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:08 EST ---
I've no 64 bit system around, so I unluckily didn't find that. /usr/lib seems 
somehow to be hardcoded and not /usr/lib(64) depending on the arch. Do you 
think, 
it fits to do a mv /usr/lib %{_libdir}, if we're on x86_64 etc. during build or 
should I better really fix the code, inclulde as patch and send upstream?

Shoulds are luckily no must, so I don't care about 1) - sorry
2) is accepted, will be corrected for the next build of course
3) seems just useless for non-packagers when reading the file

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

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


[Bug 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-02-05 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-Net-eBay - Perl Interface to XML based eBay API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:09 EST ---
Thx Parag. I'll fix the last problem before importing.

New Package CVS Request
===
Package Name: perl-Net-eBay
Short Description: Perl Interface to XML based eBay API
Owners: xavierb
Branches: F-8 EL-4 EL-5
InitialCC: 
Cvsextras Commits: yes



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

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


[Bug 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-02-05 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-Net-eBay - Perl Interface to XML based eBay API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:01 EST ---
Review:
+ package builds in mock (rawhide i386).
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream url
175674e914326a6af84564acfb57286b  Net-eBay-0.46.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 is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ no headers or static libraries.
+ no .pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ no scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ make test output is
t/00-load.# Testing Net::eBay 0.46, Perl 5.008008, /usr/bin/perl
ok
t/pod-coverageok
t/pod.ok
All tests successful.
Files=3, Tests=22,  1 wallclock secs ( 0.64 cusr +  0.05 csys =  0.69 CPU)
+ Package perl-Net-eBay-0.46-1.fc9 -
  Provides: perl(Net::eBay) = 0.46
  Requires: /usr/bin/perl perl(Carp) perl(Cwd) perl(Data::Dumper)
perl(DateTime::Precise) perl(Getopt::Long) perl(HTML::FormatText)
perl(HTML::PrettyPrinter) perl(HTML::TreeBuilder) perl(HTTP::Date)
perl(HTTP::Request::Common) perl(HTTP::Status) perl(LWP::UserAgent)
perl(Net::eBay) perl(Text::Format) perl(XML::Dumper) perl(XML::Simple)
perl(strict) perl(utf8) perl(vars) perl(warnings)

SHOULD:
   Remove  OPTIMIZE=$RPM_OPT_FLAGS from make coammand
APPROVED.


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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:42 EST ---
Also, Good to preserve timestamps by replacing make install command as given 
below
==
make install DESTDIR=$RPM_BUILD_ROOT INSTALL=install -p
==

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

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


[Bug 431381] Review Request: unicap - Library to access different kinds of (video) capture devices

2008-02-05 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: unicap - Library to access different kinds of (video) 
capture devices


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:49 EST ---
(In reply to comment #8)
 I've no 64 bit system around, so I unluckily didn't find that. /usr/lib seems 
 somehow to be hardcoded and not /usr/lib(64) depending on the arch. Do you 
 think, 
 it fits to do a mv /usr/lib %{_libdir}, if we're on x86_64 etc. during build 
 or 
 should I better really fix the code, inclulde as patch and send upstream?

 You better fix this in code and include patch in SRPM and send same to upstream
also.

 
 Shoulds are luckily no must, so I don't care about 1) - sorry
  ok.

 2) is accepted, will be corrected for the next build of course
  hope to see that in next SRPM update.
 3) seems just useless for non-packagers when reading the file
   ohh. I thought it covers also Runtime issues which are useful to end user
and also from configure output it said
 ***  Please read the file README.troubleshooting   *** 
 ***  if you have any trouble using this software.  *** 
NOTE:-It said using this software not compiling/packaging this software.

Anyway Ok if you will not include as its not a blocker :)

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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 05:35 EST ---
will review this.
I suggest replacing %files section of gyachi to
=
%files -f %{name}.lang
%defattr(-,root,root,-)
%doc ABOUT-NLS ChangeLog doc/KNOWN_PROBLEMS-gyvoice.txt doc/KNOWN_PROBLEMS.txt
doc/txt/COPYING doc/html/gyachi-help.html
%{_bindir}/%{name}*
%{_datadir}/applications/*%{name}.desktop
%{_datadir}/pixmaps/%{name}.png
%{_datadir}/%{name}
%dir %{_libdir}/%{name}
%{_libdir}/%{name}/%{name}-*
%dir %{_libdir}/%{name}/plugins
===

And remove
ExcludeArch line from SPEC.

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

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


[Bug 431414] Review Request: planets - A celestial simulator

2008-02-05 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: planets - A celestial simulator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:13 EST ---
Agreed. All of the above addressed, see original URLs.  NVR not incremented.

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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:09 EST ---
Revised spec and SRPM

http://sundaram.fedorapeople.org/packages/gyachi.spec
http://sundaram.fedorapeople.org/packages/gyachi-1.1.0-7.fc8.src.rpm

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

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


[Bug 431534] New: Review Request: gfeed - RSS feed reader

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




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

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


Spec URL: http://kurzawa.nonlogic.org/rpm/gfeed/gfeed.spec
SRPM URL: http://kurzawa.nonlogic.org/rpm/gfeed/gfeed-2.5.0-1.fc8.src.rpm
Description: gfeed is an RSS feed reader for Linux and Gtk+ 2.0. It is designed 
to be a 
simple yet highly customizable reader, not an aggregator, which means it is 
designed for easy integration into a desktop.

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

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


[Bug 238817] Review Request: qtcurve - Metapackage to install both sides of QtCurve

2008-02-05 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: qtcurve - Metapackage to install both sides of QtCurve


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:56 EST ---
My pointless comment was about the existence of the metapackage (as you seem
to agree), and not anything against qtcurve itself.

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

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


[Bug 431175] Review Request: xhotkeys - Hotkeys for the X-Window

2008-02-05 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: xhotkeys - Hotkeys for the X-Window


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:49 EST ---
Imported and built. Thanks again for review!

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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:32 EST ---

 URL:http://sourceforge.net/projects/%{name} is better

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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:38 EST ---
Fixed
Spec URL: http://kurzawa.nonlogic.org/rpm/gfeed/gfeed.spec
SRPM URL: http://kurzawa.nonlogic.org/rpm/gfeed/gfeed-2.5.0-2.fc8.src.rpm

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

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


[Bug 431373] Review Request: incollector - Information collector

2008-02-05 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: incollector - Information collector


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:48 EST ---
Imported and built. Thanks again for review!

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

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


[Bug 238817] Review Request: qtcurve - Metapackage to install both sides of QtCurve

2008-02-05 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: qtcurve - Metapackage to install both sides of QtCurve


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora
Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 06:46 EST ---
:-) I was about to construct package for QtCurve-0.55, and submit them to 
Fedora.

Kelly, can you update to QtCurve-0.55 in bug #238816 and bug #238815, and also 
provide the KDE4 style? I am using this style in both Gtk2/KDE3/Qt4 apps, and 
my desktop now looks better.

In my opinion, the current meta-package should not exist. Anyway, should it 
exist, then the requirements should be:
  kde-style-qtcurve = %{version}, gtk-qtcurve-engine = %{version}

(In reply to comment #1)
 imo, this is is kinda pointless, but who am I to judge? :)

Rex, you are the maintainer of gtk-qt-engine. I do not understand your point, 
here. In my opinion, using QtCurve for both KDE3, KDE4 and Gtk is a better 
option than gtk-qt-engine, because it offers a quite-uniform look for all the 
desktop, even for KDE3/KDE4. Until all KDE3 applications are ported to KDE4, I 
will prefere to use QtCurve instead of a mix of Oxygen (for KDE4) and any 
other style for Gtk2/KDE3/Qt3.

And, IMO, Oxygen is ugly.


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

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


[Bug 426492] Review Request: jabbin - Instant messaging and VoIP Jabber client

2008-02-05 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: jabbin - Instant messaging and VoIP Jabber client


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:31 EST ---
As I said in comment 16:
- This jabbin doesn't build on g++43 (i.e. rawhide)
  http://koji.fedoraproject.org/koji/taskinfo?taskID=396315

  I tried to fix this, however apparently it requires not a few
  fixes. I hope I have some time to try on this weekend, however
  I suggest that you ping to upstream.

- Builds on dist-f8-updates-candidate, however Fedora specific
  compilation flags are not honored correctly. More patches or
  some sed trick or so is needed.

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

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


[Bug 316141] Review Request: gridengine - Grid Engine - Distributed Computing Management software

2008-02-05 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: gridengine - Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:26 EST ---
(In reply to comment #27)
 http://www.cora.nwra.com/~orion/fedora/gridengine-6.1u3-4.fc8.src.rpm
 http://www.cora.nwra.com/~orion/fedora/gridengine.spec

Returns 404 (not found)...



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

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


[Bug 431414] Review Request: planets - A celestial simulator

2008-02-05 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: planets - A celestial simulator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:06 EST ---
(In reply to comment #5)
 Agreed. All of the above addressed, see original URLs.  NVR not incremented.
which you should have done actually.
That will help to differentiate between exiting SRPM I have downloaded and new
one to review it without confusion. Also, Changelog will suggest how review got
progressed with written changes in changelog.


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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 431414] Review Request: planets - A celestial simulator

2008-02-05 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: planets - A celestial simulator


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:20 EST ---
Fixed:

Spec URL: http://zanoni.jcomserv.net/fedora/planets/planets.spec
SRPM URL: http://zanoni.jcomserv.net/fedora/planets/planets-0.1.13-2.fc8.src.rpm

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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:41 EST ---
Is icon working for you? I installed rpm and when tried to run gfeed I got
options help only.

will review this by tomorrow.

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

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


[Bug 245919] Review Request: openvpn-auth-ldap - OpenVPN plugin for LDAP authentication

2008-02-05 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: openvpn-auth-ldap - OpenVPN plugin for LDAP 
authentication


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 07:42 EST ---
Oops, I didn't realize that this review was so old, that the initial packages
were still in the old location I used to use :-) Updated packages (2.0.3-3) are
here :
http://thias.fedorapeople.org/review/openvpn-auth-ldap/

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

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


[Bug 430417] Review Request: PersonalCopy-Lite-soundfont - Lite version of the PersonalCopy General Midi soundfont

2008-02-05 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: PersonalCopy-Lite-soundfont - Lite version of the 
PersonalCopy General Midi soundfont


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 08:17 EST ---
Well, I want to know how Jindrich thinks about your idea.

(In reply to comment #12)
all snip
Your idea seems good to me.

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

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


[Bug 238817] Review Request: qtcurve - Metapackage to install both sides of QtCurve

2008-02-05 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: qtcurve - Metapackage to install both sides of QtCurve


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   ||lesup.org




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 08:49 EST ---
Sorry Rex for the misunderstanding.


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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 09:00 EST ---
Yes icon is working. If you wrote example: gfeed -f http://linux.pl/rss.php;,
new windows will open and icon will show.

Thanks in advance for review.

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

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


[Bug 430360] Review Request: ggz-gtk-client - Gtk+ client libraries for GGZ gaming zone

2008-02-05 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: ggz-gtk-client - Gtk+ client libraries for GGZ gaming 
zone


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 09:31 EST ---
Brian, feel free to add yourself as comaintainer in any capacity you wish for 
dep's libggz-devel, ggz-client-libs (and I'll add myself here as mostly an 
observer, if you don't mind).

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

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


[Bug 296161] Review Request: ical - An X Window System-based calendar program

2008-02-05 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: ical - An X Window System-based calendar program


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||201449
  nThis||
 Status|NEEDINFO|CLOSED
 Resolution||NOTABUG
   Flag|needinfo?([EMAIL PROTECTED])|




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 09:34 EST ---
Once CLOSING.

If someone wants to import this package into Fedora, please file
a new review request and mark this bug as a duplicate of the new one.

Thank you!

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

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


[Bug 398601] Review Request: cairo-clock - Cairo-rendered on-screen clock

2008-02-05 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: cairo-clock - Cairo-rendered on-screen clock


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 09:32 EST ---
Would someone update this bug?

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

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


[Bug 251710] Review Request: potrace - Transforming bitmaps into vector graphics

2008-02-05 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: potrace - Transforming bitmaps into vector graphics


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 09:31 EST ---
CC-ing new Balint e-mail. Balint, would you update this bug?

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

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


[Bug 225250] Merge Review: antlr

2008-02-05 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: antlr


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]




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

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


[Bug 225250] Merge Review: antlr

2008-02-05 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: antlr


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 11:29 EST ---
rpmlint on SRPM:

antlr.src:375: W: macro-in-%changelog _bindir
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'.

Fix.

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

Fix.

antlr.src: W: non-standard-group Development/Code Generators
The value of the Group tag in the package is not valid.  Valid groups are:
Amusements/Games, Amusements/Graphics, Applications/Archiving,
Applications/Communications, Applications/Databases,
Applications/Editors, Applications/Emulators, Applications/Engineering,
Applications/File, Applications/Internet, Applications/Multimedia,
Applications/Productivity, Applications/Publishing, Applications/System,
Applications/Text, Development/Debug, Development/Debuggers,
Development/Languages, Development/Libraries, Development/System,
Development/Tools, Documentation, System Environment/Base, System
Environment/Daemons, System Environment/Kernel, System
Environment/Libraries, System Environment/Shells, User
Interface/Desktops, User Interface/X, User Interface/X Hardware Support.

I'd use Development/Tools.


rpmlint on rpms:

antlr.i386: W: devel-file-in-non-devel-package /usr/include/antlr/ANTLRUtil.hpp
A development file (usually source code) is located in a non-devel
package. If you want to include source code in your package, be sure to
create a development package.

etc. . .. 

Ok, due to the nature of the package.

antlr.i386: W: incoherent-version-in-changelog 2.7.7-1jpp.6 0:2.7.7-1jpp.6.fc8
The last entry in %changelog contains a version identifier that is not
coherent with the epoch:version-release tuple of the package.

Fix.

antlr-javadoc.i386: W: dangerous-command-in-%post rm
antlr-javadoc.i386: W: dangerous-command-in-%postun rm

It looks like these are necessary, so OK.


Need to include something about how the RHCLEAN tarball can be produced from the
upstream tarball.

Mock build error:

 /usr/bin/make -C lib/cpp/antlr all

make[3]: Entering directory `/builddir/build/BUILD/antlr-2.7.7/lib/cpp/antlr'
make[3]: Nothing to be done for `all'.
make[3]: Leaving directory `/builddir/build/BUILD/antlr-2.7.7/lib/cpp/antlr'

 /usr/bin/make -C lib/cpp/src all

make[3]: Entering directory `/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src'
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/ANTLRUtil.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/ASTFactory.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/ASTNULLType.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/ASTRefCount.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/BaseAST.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/BitSet.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/CharBuffer.cpp
*** compiling
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/CharScanner.cpp
In file included from
/builddir/build/BUILD/antlr-2.7.7/lib/cpp/src/../../../lib/cpp/src/CharScanner.cpp:10:
/builddir/build/BUILD/antlr-2.7.7/scripts/../lib/cpp/antlr/CharScanner.hpp: In
member function 'bool antlr::CharScannerLiteralsLess::operator
/builddir/build/BUILD/antlr-2.7.7/scripts/../lib/cpp/antlr/CharScanner.hpp:565:
error: 'strcasecmp' was not declared in this scope


   E R R O R 

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

2008-02-05 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/show_bug.cgi?id=221717


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 11:45 EST ---
At least gnash depends on agg. 

I can take care of this package, as a gnash dependency. Would you 
accept me as a co-maintainer?

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

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


[Bug 225676] Merge Review: dbus

2008-02-05 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: dbus


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 426387] Merge reviews to be completed for F9

2008-02-05 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 reviews to be completed for F9
Alias: F9MergeReviewTarget

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


Bug 426387 depends on bug 225676, which changed state.

Bug 225676 Summary: Merge Review: dbus
https://bugzilla.redhat.com/show_bug.cgi?id=225676

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE



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

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


[Bug 226669] Merge Review: zip

2008-02-05 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: zip


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]
 Status|MODIFIED|NEEDINFO
   Flag||needinfo?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 11:55 EST ---
The bug should certainly be assigned to Ruben.

But the comments should also be addressed, in my opinion
the package is not really ready for inclusion without those
comments addressed.

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

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


[Bug 225676] Merge Review: dbus

2008-02-05 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: dbus


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 11:55 EST ---
Approved.

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

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


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

2008-02-05 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/show_bug.cgi?id=221717


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 11:56 EST ---
Sure, you can even be the sole maintainer if you want.

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

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


[Bug 210007] Review Request: libtune - standard API to access the kernel tunables

2008-02-05 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: libtune - standard API to access the kernel tunables


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:01 EST ---
Yikes, so I need to retract the 'ACCEPT'/APPROVED part since I am not a sponser
(per
http://fedoraproject.org/wiki/PackageMaintainers/Join#head-a601c13b0950a89568deafa65f505b4b58ee869b:
First reviews for new packagers must be done by registered sponsors. Informal
reviews can be done by anyone interested.).

Next stage is actually for a sponsor to do the review.



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

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


[Bug 225252] Merge Review: apmd

2008-02-05 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: apmd


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:03 EST ---
rpmlint on srpm:

apmd.src:17: W: prereq-use chkconfig
The use of PreReq is deprecated. In the majority of cases, a plain Requires
is enough and the right thing to do. Sometimes Requires(pre), Requires(post),
Requires(preun) and/or Requires(postun) can also be used instead of PreReq.

Fix.

apmd.src: W: summary-ended-with-dot Advanced Power Management (APM) BIOS
utilities for laptops.
Summary ends with a dot.

Fix.

apmd.src: W: invalid-license GPL
The value of the License tag was not recognized.  Known values are:
Adobe, AFL, AGPLv1, AGPLv3, AMPAS BSD, ARL, ASL 1.0, ASL 1.0+,
ASL 1.1, ASL 1.1+, ASL 2.0, ASL 2.0+, APSL 2.0, APSL 2.0+,
Artistic 2.0, Artistic clarified, BitTorrent, Boost, BSD, BSD with
advertising, CeCILL, CDDL, CPL, Condor, Copyright only, Cryptix,
Crystal Stacker, EPL, eCos, EFL 2.0, EFL 2.0+, EU Datagrid, FTL,
Giftware, Glide, gnuplot, GPL+, GPL+ or Artistic, GPLv2+ or
Artistic, GPLv2, GPLv2 with exceptions, GPLv2+, GPLv2+ with
exceptions, GPLv3, GPLv3 with exceptions, GPLv3+, GPLv3+ with
exceptions, IBM, IJG, ImageMagick, iMatix, Imlib2, Intel ACPI,
Interbase, ISC, Jabber, JasPer, LGPLv2, LGPLv2 with exceptions,
LGPLv2+, LGPLv3, LGPLv3+, libtiff, LPL, LPPL, mecab-ipadic,
MIT, MPLv1.0, MPLv1.0+, MPLv1.1, MPLv1.1+, NCSA, NGPL, NOSL,
Netscape, Nokia, OpenLDAP, OpenPBS, OReilly, OSL 1.0, OSL 1.0+,
OSL 1.1, OSL 1.1+, OSL 2.0, OSL 2.0+, OSL 3.0, OSL 3.0+,
OpenSSL, Phorum, PHP, Public Domain, Python, QPL, RPSL, Ruby,
Sleepycat, SISSL, SLIB, SPL, TCL, UCD, Vim, VNLSL, VSL,
W3C, WTFPL, wxWindows, xinetd, Zend, ZPLv1.0, ZPLv1.0+,
ZPLv2.0, ZPLv2.0+, ZPLv2.1, ZPLv2.1+, zlib, CDL, FBSDDL, GFDL,
IEEE, OFSFDL, Open Publication, CC-BY, CC-BY-SA, DSL, Free Art,
Arphic, Baekmuk, Bitstream Vera, mplus, OFL, STIX, Utopia,
XANO, Redistributable, no modification permitted, Freely redistributable
without restriction.

Fix, should be GPLv2+

apmd.src: W: no-url-tag
The URL tag is missing.

Fix.

apmd.src: W: strange-permission apmscript 0755
A file that you listed to include in your package has strange
permissions. Usually, a file should have 0644 permissions.

apmd.src: W: strange-permission apmd.init 0755
A file that you listed to include in your package has strange
permissions. Usually, a file should have 0644 permissions.

apmd.src: W: strange-permission laptopmode 0755
A file that you listed to include in your package has strange
permissions. Usually, a file should have 0644 permissions.

Ok.


apmd.i386: E: executable-marked-as-config-file /etc/rc.d/init.d/apmd
Executables must not be marked as config files because that may
prevent upgrades from working correctly. If you need to be able to
customize an executable, make it for example read a config file in
/etc/sysconfig.

apmd.i386: E: executable-marked-as-config-file 
/etc/sysconfig/apm-scripts/laptopmode
Executables must not be marked as config files because that may
prevent upgrades from working correctly. If you need to be able to
customize an executable, make it for example read a config file in
/etc/sysconfig.

apmd.i386: E: executable-marked-as-config-file 
/etc/sysconfig/apm-scripts/apmscript
Executables must not be marked as config files because that may
prevent upgrades from working correctly. If you need to be able to
customize an executable, make it for example read a config file in
/etc/sysconfig.

These are fine for this package.

apmd.i386: W: conffile-without-noreplace-flag /etc/rc.d/init.d/apmd
A configuration file is stored in your package without the noreplace flag.
A way to resolve this is to put the following in your SPEC file:

%config(noreplace) /etc/your_config_file_here


apmd.i386: W: conffile-without-noreplace-flag 
/etc/sysconfig/apm-scripts/apmscript
A configuration file is stored in your package without the noreplace flag.
A way to resolve this is to put the following in your SPEC file:

%config(noreplace) /etc/your_config_file_here


apmd.i386: W: conffile-without-noreplace-flag 
/etc/sysconfig/apm-scripts/laptopmode
A 

[Bug 226147] Merge Review: mktemp

2008-02-05 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: mktemp


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:44 EST ---
than, in that case, could you complete the package EOL process for mktemp as
described in http://fedoraproject.org/wiki/PackageMaintainers/PackageEndOfLife
so we can close this?

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

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


[Bug 427674] Review Request: themonospot -application that can be used to scan an avi file and extract some informations about audio and video data flow

2008-02-05 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: themonospot -application that can be used to scan an 
avi file and extract some informations about audio and video data flow


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:33 EST ---
ping?

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

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


[Bug 316141] Review Request: gridengine - Grid Engine - Distributed Computing Management software

2008-02-05 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: gridengine - Grid Engine - Distributed Computing 
Management software


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:33 EST ---
(In reply to comment #28)
 Returns 404 (not found)...

Sorry, should work now.

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

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


[Bug 427010] Review Request: abcmusiex - Training music score (pentagram) reading

2008-02-05 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: abcmusiex - Training music score (pentagram) reading


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
   Flag||needinfo?([EMAIL PROTECTED]
   ||dep.org.bo)




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 12:35 EST ---
What is the status of this review request?

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

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


[Bug 428793] Review Request: xhtml2fo-style-xsl - Antenna House, Inc. XHTML to XSL:FO stylesheets

2008-02-05 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: xhtml2fo-style-xsl - Antenna House, Inc. XHTML to 
XSL:FO stylesheets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 428798] Review Request: OmegaT - Computer Aid Translation tool

2008-02-05 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: OmegaT - Computer Aid Translation tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 226557] Merge Review: xfig

2008-02-05 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: xfig


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are 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 429451] Review Request: magicmaze - Board game featuring a maze which the players change each turn

2008-02-05 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: magicmaze - Board game featuring a maze which the 
players change each turn


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: maze - Board|Review Request: magicmaze -
   |game featuring a maze which |Board game featuring a maze
   |the players change each turn|which the players change
   ||each turn
   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:20 EST ---
Thanks for the review!

New Package CVS Request
===
Package Name:  magicmaze
Short Description: Board game featuring a maze which the players change each 
turn
Owners:jwrdegoede
Branches:  F-8
InitialCC: 
Cvsextras Commits: Yes


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

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


[Bug 429199] Review Request: polyester3 - A style for KDE3

2008-02-05 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: polyester3 - A style for KDE3


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




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

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


[Bug 225289] Merge Review: at-spi

2008-02-05 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: at-spi


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
 Status|NEW |CLOSED
 Resolution||ERRATA
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:21 EST ---
Only one rpmlint message:

at-spi-python.i386: W: no-documentation
The package contains no documentation (README, doc, etc).
You have to include documentation files.

I'd split out something into this if possible, but it doesn't look like it.

No blockers at all.  An exemplary package.

APPROVED.

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

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


[Bug 428113] Review Request: perl-Net-eBay - Perl Interface to XML based eBay API

2008-02-05 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-Net-eBay - Perl Interface to XML based eBay API


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:28 EST ---
cvs done.

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

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


[Bug 429451] Review Request: magicmaze - Board game featuring a maze which the players change each turn

2008-02-05 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: magicmaze - Board game featuring a maze which the 
players change each turn


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:28 EST ---
cvs done.

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

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


[Bug 429451] Review Request: maze - Board game featuring a maze which the players change each turn

2008-02-05 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: maze - Board game featuring a maze which the players 
change each turn


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:17 EST ---
I finally got around to packaging up a new version released by upstream, which
includes a zlib license.txt now:
http://people.atrpms.net/~hdegoede/magicmaze.spec
http://people.atrpms.net/~hdegoede/magicmaze-1.0.2-1.fc9.src.rpm


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

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


[Bug 427674] Review Request: themonospot -application that can be used to scan an avi file and extract some informations about audio and video data flow

2008-02-05 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: themonospot -application that can be used to scan an 
avi file and extract some informations about audio and video data flow


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:17 EST ---
sorry, i working on a project for my factory :(

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

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


[Bug 415791] Review Request: netbsd-iscsi - User-space implementation of iSCSI target from NetBSD project

2008-02-05 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: netbsd-iscsi - User-space implementation of iSCSI 
target from NetBSD project
Alias: netbsd-iscsi-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 226353] Merge Review: quota

2008-02-05 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: quota


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:28 EST ---
No response that I've seen from the Packaging Committee.  Anyone else hear
anythin g off-list?

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

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


[Bug 429199] Review Request: polyester3 - A style for KDE3

2008-02-05 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: polyester3 - A style for KDE3


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 14:32 EST ---
Not much has changed since previous review, so I'll be brief:

clean, simple.

source matches upstream,
673d21db31eeaedf9b63cd09f6b247a8  polyester-1.0.3.tar.bz2

scratch build was already pruned, so did a local build, which is 
rpmlint-clean.

All I see:

1.  SHOULD: I'd much prefer avoiding the use of Conflicts here.  It will only 
help a rare corner case.  I'll leave that to you.


APPROVED.

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

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


[Bug 225307] Merge Review: awesfx

2008-02-05 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: awesfx


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 15:22 EST ---
Local build fails:

+ make
Making all in awelib
make[1]: Entering directory `/var/storage/redhat/BUILD/awesfx-0.5.0d/awelib'
if gcc -DHAVE_CONFIG_H -I. -I. -I../include -I../include-O2 -g -pipe -Wall
-Wp,-D_FORTIFY_SOURCE=2 -fexceptions -fstack-protector --param=ssp-buffer-size=4
-m32 -march=i386 -mtune=generic -fasynchronous-unwind-tables -MT awe_parm.o -MD
-MP -MF .deps/awe_parm.Tpo -c -o awe_parm.o awe_parm.c; \
then mv -f .deps/awe_parm.Tpo .deps/awe_parm.Po; else rm -f
.deps/awe_parm.Tpo; exit 1; fi
In file included from awe_parm.c:25:
../include/awe_parm.h:25:29: error: linux/awe_voice.h: No such file or directory
In file included from awe_parm.c:25:
../include/awe_parm.h:30: error: expected ')' before '*' token
../include/awe_parm.h:31: error: expected ')' before '*' token
awe_parm.c:109: error: expected ')' before '*' token
awe_parm.c:142: error: expected ')' before '*' token
make[1]: *** [awe_parm.o] Error 1
make[1]: Leaving directory `/var/storage/redhat/BUILD/awesfx-0.5.0d/awelib'
make: *** [all-recursive] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.94791 (%build)


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

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


[Bug 225294] Merge Review: authd

2008-02-05 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: authd


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 15:12 EST ---
rpmlint on srpm:

authd.src:7: W: unversioned-explicit-obsoletes pidentd
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.

authd.src:18: W: prereq-use xinetd
The use of PreReq is deprecated. In the majority of cases, a plain Requires
is enough and the right thing to do. Sometimes Requires(pre), Requires(post),
Requires(preun) and/or Requires(postun) can also be used instead of PreReq.

authd.src: W: no-url-tag
The URL tag is missing.


These need fixing.


rpmlint on rpms:

authd.i386: E: non-readable /etc/ident.key 0640
The file can't be read by everybody. If this is expected (for security
reasons), contact your rpmlint distributor to get it added to the list of
exceptions for your distro (or add it to your local configuration if you
installed rpmlint from the source tarball).

authd.i386: E: zero-length /etc/ident.key

authd.i386: W: conffile-without-noreplace-flag /etc/ident.key
A configuration file is stored in your package without the noreplace flag.
A way to resolve this is to put the following in your SPEC file:

%config(noreplace) /etc/your_config_file_here


These are OK.

authd.i386: W: dangerous-command-in-%post chown

Fix if possible, might want to use the full path to chown.

In addition to needing a URL tag, the Source tag needs a url pointing to the
upstream tarball.  If Fedora or RH are upstream, get a space on hosted.

License tag should be GPLv2+.

Error in rawhide mock build:

Executing(%build): /bin/sh -e /var/tmp/rpm-tmp.30946
+ umask 022
+ cd /builddir/build/BUILD
+ cd authd-1.4.3
+ LANG=C
+ export LANG
+ unset DISPLAY
+ make prefix=/usr
gcc -std=gnu99 -Wall -W -DNDEBUG -g -O -lcrypto -o in.authd authd.c
authd.c: In function 'created_pton_hex':
authd.c:198: error: 'struct in6_addr' has no member named 'in6_u'
authd.c:202: error: 'struct in6_addr' has no member named 'in6_u'
authd.c: In function 'initialize_crypto':
authd.c:784: warning: pointer targets in passing argument 4 of 'EVP_BytesToKey'
differ in signedness
make: *** [in.authd] Error 1
error: Bad exit status from /var/tmp/rpm-tmp.30946 (%build)


RPM build errors:
Bad exit status from /var/tmp/rpm-tmp.30946 (%build)
EXCEPTION: Command failed. See logs for output.
 # bash -l -c 'rpmbuild -bb --target i386 --nodeps
//builddir/build/SPECS/authd.spec'
Traceback (most recent call last):
  File /usr/lib/python2.5/site-packages/mock/trace_decorator.py, line 70, in 
trace
result = func(*args, **kw)
  File /usr/lib/python2.5/site-packages/mock/util.py, line 286, in do
raise mock.exception.Error, (Command failed. See logs for output.\n # %s %
(command,), ret)
Error: Command failed. See logs for output.
 # bash -l -c 'rpmbuild -bb --target i386 --nodeps
//builddir/build/SPECS/authd.spec'
LEAVE do -- EXCEPTION RAISED


Other than that, looks good, no other blockers.

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

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


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

2008-02-05 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/show_bug.cgi?id=221717





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 15:34 EST ---
I would have preferred not to be the sole maintainer, but, if you 
want to I can.

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

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


[Bug 431277] Review Request: ocfs2-tools - programs for managing Ocfs2 file systems

2008-02-05 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: ocfs2-tools - programs for managing Ocfs2 file systems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 15:45 EST ---
Created an attachment (id=294042)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=294042action=view)
Latest ocfs2-tools spec file.


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

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


[Bug 431277] Review Request: ocfs2-tools - programs for managing Ocfs2 file systems

2008-02-05 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: ocfs2-tools - programs for managing Ocfs2 file systems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Attachment #293957|0   |1
is obsolete||




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

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


[Bug 431277] Review Request: ocfs2-tools - programs for managing Ocfs2 file systems

2008-02-05 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: ocfs2-tools - programs for managing Ocfs2 file systems


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 431277] Review Request: ocfs2-tools - programs for managing Ocfs2 file systems

2008-02-05 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: ocfs2-tools - programs for managing Ocfs2 file systems


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 15:43 EST ---
(In reply to comment #13)
 Change the buildroot to
 
 %{_tmppath}/%{name}-%{version}-%{release}-root

Ahh, ok. Thanks - a new version of the package and spec have been uploaded:

Spec URL: http://oss.oracle.com/~mfasheh/fedora/ocfs2-tools.spec
SRPM URL:
http://oss.oracle.com/~mfasheh/fedora/ocfs2-tools-1.3.9-5.20080131git.fc8.src.rpm


 The part of the guidelines, I wanted you to refer to regarding static libs 
 was, 
 
 In general, packagers are strongly encouraged not to ship static libs unless 
 a
 compelling reason exists.
 
 If you do have compelling reasons, it would be good to specify that in the
 review. 

Upstream doesn't build dynamic libraries because the ocfs2-tools api changes 
quite rapidly as file system features are added / expanded. As a result, it's
impossible to provide any sort of abi stability, so static libraries are
provided instead. Generally, people writing ocfs2 programs are encouraged to
submit their software to [EMAIL PROTECTED] so that it can be
included in the upstream ocfs2-tools distribution. The libraries that are
provided in ocfs2-tools-devel are intended for interim development, and for a
small number of external programs including some fs test software.


 Other than those minor nit-picks, the latest spec looks ok to me. I did
 a scratch build of the latest ocfs2 srpm and it builds and installs correctly.

Thanks for testing!


 A sponsor has to do a official review and approve your spec before you get
 commit access.
 
 http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

Ok, I will add the FE-NEEDSPONSOR field to indicate my official request for
sponsorship.

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

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


[Bug 428925] Review Request: bongo - An easy-to-use mail and calendar system

2008-02-05 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: bongo - An easy-to-use mail and calendar system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 16:03 EST ---
There seems to be a problem with one of the included libraries:

[EMAIL PROTECTED] ~]# bongo-config install
IP address to run on [127.0.0.1]: 
DNS name to use as main hostname: [localhost]: 
Mail Domains (enter  to end adding domains): []: 
ERROR: Couldn't initialise auth subsystem

An strace reveals the following reason for the failure:

open(/usr/lib/bongo-auth/libauthsqlite3.so, O_RDONLY) = -1 ENOENT (No such
file or directory)

The directory is missing the needed symlink:

[EMAIL PROTECTED] bongo-auth]# ls -l libauthsqlite3.*
lrwxrwxrwx 1 root root23 2008-02-05 17:20 libauthsqlite3.so.0 -
libauthsqlite3.so.0.0.0
-rwxr-xr-x 1 root root 10612 2008-02-05 17:19 libauthsqlite3.so.0.0.0

After creating the link manually Bongo works as it should.


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

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


[Bug 429199] Review Request: polyester3 - A style for KDE3

2008-02-05 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: polyester3 - A style for KDE3


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 16:12 EST ---
(In reply to comment #1)
 All I see:
 
 1.  SHOULD: I'd much prefer avoiding the use of Conflicts here.  It will only 
 help a rare corner case.  I'll leave that to you.

But it won't hurd so I leave it in the spec.

 APPROVED.

Thanks!


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

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


[Bug 429199] Review Request: polyester3 - A style for KDE3

2008-02-05 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: polyester3 - A style for KDE3


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 16:15 EST ---
New Package CVS Request
===
Package Name: polyester3
Short Description: A style for KDE3
Owners: svahl
Branches: 
InitialCC: 
Cvsextras Commits: no

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

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


[Bug 430653] Review Request: baracuda - VNC system

2008-02-05 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: baracuda - VNC system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 16:48 EST ---
Uh, trademarks are not only about identical names, but also about confusingly 
similar names.

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

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


[Bug 225356] Review request: fonts-thai

2008-02-05 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: fonts-thai


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |rawhide




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

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


[Bug 226353] Merge Review: quota

2008-02-05 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: quota


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 17:01 EST ---
Add the new Fedora Quota maintainer...  Ondřej Vašík 

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

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


[Bug 430653] Review Request: baracuda - VNC system

2008-02-05 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: baracuda - VNC system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 17:26 EST ---
Unversioned obsoletes are not acceptable. The naming guidelines explain how to
handle obsoletes:

http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-3cfc1ea19d28975faad9d56f70a6ae55661d3c3d

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

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


[Bug 430619] Review Request: perl-Color-Library - Easy-to-use and comprehensive named-color library

2008-02-05 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-Color-Library - Easy-to-use and comprehensive 
named-color library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 17:03 EST ---
New Package CVS Request
===
Package Name: perl-Color-Library
Short Description: Easy-to-use and comprehensive named-color library
Owners: steve
Branches: F-7 F-8 EL-4 EL-5
InitialCC: perl-sig
Cvsextras Commits: yes


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

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


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

2008-02-05 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/show_bug.cgi?id=221717





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 17:32 EST ---
I have taken care of all my comments (and a bit more).

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

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


[Bug 426026] Review Request: gnome-menu-extended - Gnome Menu with KDE directory

2008-02-05 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: gnome-menu-extended - Gnome Menu with KDE directory


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEEDINFO|NEW
   Flag|needinfo?(pablo.martin- |
   |[EMAIL PROTECTED])  |




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 17:33 EST ---
Sorry to answer just now but the package has been a little difficult to be
perfect. I've replace the cp command by install, I don't know it do it right
or not. However, I've a problem yet, the entries Add KDE menu and   Restore
Applications Menu in same time in the menu. (I'm not sure is it normal but,
putting the KDE menu, it's restore the pirut entry which I've earsed)

Spec URL: http://home.scarlet.be/~tsi30161/gnome-menu-extended.spec
SRPM URL: http://home.scarlet.be/~tsi30161/gnome-menu-extended-0.7-1.fc8.src.rpm


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

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


[Bug 241319] Review Request: hotwire - Interactive hybrid text/graphical shell for developers and system administrators

2008-02-05 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: hotwire - Interactive hybrid text/graphical shell for 
developers and system administrators


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Product|Fedora Extras   |Fedora
Version|devel   |rawhide

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 18:34 EST ---
Package Change Request
==
Package Name: hotwire
New Branches: EL-5


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

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


[Bug 241319] Review Request: hotwire - Interactive hybrid text/graphical shell for developers and system administrators

2008-02-05 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: hotwire - Interactive hybrid text/graphical shell for 
developers and system administrators


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 19:52 EST ---
cvs done.

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

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


[Bug 429199] Review Request: polyester3 - A style for KDE3

2008-02-05 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: polyester3 - A style for KDE3


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 19:54 EST ---
cvs done. 

Any particular reason for the cvsextras commits: no? 


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

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


[Bug 430619] Review Request: perl-Color-Library - Easy-to-use and comprehensive named-color library

2008-02-05 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-Color-Library - Easy-to-use and comprehensive 
named-color library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 19:57 EST ---
cvs done.

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

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


[Bug 428894] Review Request: rudesocket - Library (C++ API) for creating client sockets

2008-02-05 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: rudesocket - Library (C++ API) for creating client 
sockets


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 20:43 EST ---
Not yet, the build failed due to missing includes for std c libraries. I got 
the flu before I could fix it, am working on it now.

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

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


[Bug 199154] Review Request: Slony-1 (postgresql-slony-engine)

2008-02-05 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: Slony-1 (postgresql-slony-engine)


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 21:05 EST ---
1.2.13 is on the way (tomorrow, I believe). I have many packaging errors, but
still I'll push it for review.

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

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


[Bug 431098] Review Request: gambas2 - IDE based on a basic interpreter with object extensions

2008-02-05 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: gambas2 - IDE based on a basic interpreter with object 
extensions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 21:23 EST ---
koji says libffi-devel is not found...

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

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

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


[Bug 431633] New: Review Request: libffi - High level programming interface to various calling conventions

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




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

   Summary: Review Request: libffi - High level programming
interface to various calling conventions
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://www.auroralinux.org/people/spot/review/new/libffi.spec
SRPM URL: 
http://www.auroralinux.org/people/spot/review/new/libffi-2.1-0.1.20080129svn131943.fc9.src.rpm
Description:

Compilers for high level languages generate code that follows certain
conventions. These conventions are necessary, in part, for separate
compilation to work. One such convention is the calling convention. The
calling convention is a set of assumptions made by the compiler about where
function arguments will be found on entry to a function. A calling
convention also specifies where the return value for a function is found.

Some programs may not know at the time of compilation what arguments are to
be passed to a function. For instance, an interpreter may be told at run-time
about the number and types of arguments used to call a given function. Libffi
can be used in such programs to provide a bridge from the interpreter program
to compiled code. The libffi library provides a portable, high level
programming interface to various calling conventions. This allows a programmer
to call any function specified by a call interface description at run-time.

Ffi stands for Foreign Function Interface. A foreign function interface is the
popular name for the interface that allows code written in one language to call
code written in another language. The libffi library really only provides the
lowest, machine dependent layer of a fully featured foreign function interface.
A layer must exist above libffi that handles type conversions for values passed
between the two languages.

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

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


[Bug 431633] Review Request: libffi - High level programming interface to various calling conventions

2008-02-05 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: libffi - High level programming interface to various 
calling conventions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||431098
  nThis||




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

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


[Bug 431098] Review Request: gambas2 - IDE based on a basic interpreter with object extensions

2008-02-05 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: gambas2 - IDE based on a basic interpreter with object 
extensions


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||431633




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 21:51 EST ---
Whoops. I forgot to submit that one first. :)

libffi is bugzilla 431633.

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

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


[Bug 431633] Review Request: libffi - High level programming interface to various calling conventions

2008-02-05 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: libffi - High level programming interface to various 
calling conventions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 22:07 EST ---
Would you elaborate why you want to ship libffi as a separate package?

libffi is part of GCC, is being built when bootstrapping GCC, technically
closely tied to GCC ...

That said, IMO, libffi should not be shipped separately.

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

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


[Bug 431633] Review Request: libffi - High level programming interface to various calling conventions

2008-02-05 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: libffi - High level programming interface to various 
calling conventions


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 22:35 EST ---
Or, if it is shipped for systems to use, perhaps it should be built from GCC?

Note that there is precedence for apps shipping their own libffi in the absence
of a system one. See g-wrap.

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

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


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

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

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


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2008-02-05 23:16 EST ---
(In reply to comment #20)
 From this comment it seems that finally you have to build
 though strap stage but you mentioned that you did only by
 quick stage.

I actually build all the way through to strap since this is what the dist
target of the ant build file does.  Strap is built just to be sure that
quick can bootstrap itself.  In other words it guarantees that you can replace
the bootstrap environment with quick and be able to build scala from scratch.

I will be uploading a new srpm and spec fixing the emacs issues pointed out by
Jonathan in the next couple days.

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

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


[Bug 428739] libgweather - A library for weather information

2008-02-05 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: libgweather - A library for weather information


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




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

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


[Bug 431534] Review Request: gfeed - RSS feed reader

2008-02-05 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: gfeed - RSS feed reader


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 00:54 EST ---
SHOULD:
   1)Change %{__install} to install 
   2)Add doc/out/html/*.html and scripts. But to avoid unnecessary installations
of Makefiles in scripts directory, you can add following line to end of %install
section

rm -f scripts/Makefile*

and change %doc to
%doc AUTHORS CHANGELOG COPYING README doc/out/html/*.html scripts




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

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


[Bug 428925] Review Request: bongo - An easy-to-use mail and calendar system

2008-02-05 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: bongo - An easy-to-use mail and calendar system


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





--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 01:29 EST ---
%{_libdir}/bongo-auth/libauthsqlite3.so is in the -devel package like the
guidelines demands. It should likely try to open .so.0

Try this:
SRPM: http://dnielsen.fedorapeople.org/bongo-0.3.1-2.fc8.src.rpm
SPEC: http://dnielsen.fedorapeople.org/bongo.spec

- 5 sec untested patch for the above .so problem


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

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


[Bug 431371] Review Request: gyachi - A Yahoo! chat client with Webcam and voice support

2008-02-05 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: gyachi - A Yahoo! chat client with Webcam and voice 
support


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2008-02-06 01:57 EST ---
Review:
+ package builds in mock (rawhide i386).
koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=398197
+ rpmlint is silent for SRPM and for RPM.
+ source files match upstream.
f833c873da22fe857d9a5edd3c0e9922  gyachi-1.1.0.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 is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ Compiler flags are honoured correctly.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists But -plugins subpackages exists.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Desktop file installed correctly
+ GUI app.

SHOULD:
   Remove ExcludeArch line from SPEC.

APPROVED.


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

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