[Bug 218744] New: Review Request: sofia-sip - SIP user agent library

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




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

   Summary: Review Request: sofia-sip - SIP user agent library
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://sofia-sip.org/~ppessi/fedora/sofia-sip.spec
SRPM URL: http://sofia-sip.org/~ppessi/fedora/sofia-sip-1.12.4-0.src.rpm
Description: This is my first package, so I need a sponsor.

Package compiles on mock with i386, but I'd like to hear about PPC results.

--8--8--8--8--8--8--8--8--8--8--8--8--8--8--8--8--
Sofia-SIP is an open-source SIP User-Agent library, compliant 
with the IETF RFC3261 specification. It can be used as 
a building block for SIP client software for uses such as VoIP, 
IM, and many other real-time and person-to-person communication 
services. The primary target platform for Sofia-SIP is 
GNU/Linux. Sofia-SIP is based on a SIP stack developed at 
the Nokia Research Center. Sofia-SIP is licensed under the LGPL.

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

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


[Bug 218744] Review Request: sofia-sip - SIP user agent library

2006-12-07 Thread 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: sofia-sip - SIP user agent library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 218367] Review Request: compat-guile - Guile compatibility package

2006-12-07 Thread 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: compat-guile - Guile compatibility package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 03:47 EST ---
I'll review this.
Btw, I need it too for geda packages. Good job GĂ©rard :)

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 03:53 EST ---
(In reply to comment #19)
 The audacious header /usr/lib/audacious/beepctrl.h does expose glib to clients
 such as conky that use this interface for audacious status.  I don't disagree
 that audacious.pc exposes too much, but I would also characterize this issue 
 as
 something for the audacious devs to address.

Indeed. But it is something which annoys the conky packager ;-)

 Meantime, building conky with:
 
 AUDACIOUS_CFLAGS=`pkg-config --cflags glib-2.0` \
 AUDACIOUS_LIBS=-laudacious `pkg-config --cflags glib-2.0` \
 ./configure --enable-audacious ...
 
 can be used to override the pkg-config --cflags --libs audacious excess and
 will yield a trimmer set of true deps for the conky binary (w.r.t. 
 audacious)

I don't think this is the proper way to handle that issue, the
right fix is to fix what is broken (the audacious pkgconfig file).
It is not such a problematic issue that it needs a workaround.

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 04:01 EST ---
(In reply to comment #19)
 The audacious header /usr/lib/audacious/beepctrl.h does expose glib to clients

Does conky uses directly the glib functions? If not, it is better
to have glib-devel as an indirect dependency through audacious-devel.
Thus if audacious stops using the glib, conky wouldn't need to be modified.
If glib-devel is a direct dependency of conky it would be better to
list it in buildrequires, but it doesn't seems to be the case since
conky itself don't include the glib headers (unless I am missing 
something).

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

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


[Bug 218744] Review Request: sofia-sip - SIP user agent library

2006-12-07 Thread 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: sofia-sip - SIP user agent library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 04:12 EST ---
I can provide you info about owerpc-builds.

First of all, it doesn't builds at all

[EMAIL PROTECTED] SPECS]$ LANG=C rpmbuild -ba sofia-sip.spec 
error: parse error in expression
error: /usr/src/redhat/SPECS/sofia-sip.spec:25: parseExpressionBoolean returns 
-1
error: Package has no %description: sofia-sip
[EMAIL PROTECTED] SPECS]$ 

Please add description.

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 04:38 EST ---
* rpmlint is silent
* follow guidelines
* sane provides
* buildrequires seem right
* match upstream
c856556d4372226f99cf7e9a888e9118  conky-1.4.4.tar.bz2
* doc not content

A remark (not a blocker, may be changed later, the group is not
used for anything currently), but Applications/System seems wrong
to me, something like 'User Interface/X' would be better in my 
opinion.

A blocker: the COPYING is a BSD like license, while the whole is
under the GPL. First of all (especially if upstream authors listen 
that report ;-) the GPL notice should also be in the tarball, along
with the BSD license, named, for example COPYING.GPL. But it is
not an obligation for you as a packager to add the license if it isn't
done upstream. However, in cases like conky, where there are more than 
one license covering parts of code, and especially in that case, with
a COPYING which doesn't match the package license, some clarification
is required. I see 2 way to do that clarification:

* add a file with the appropriate name, stating something along

Most of the conky code is covered by the BSD license in COPYING,
some files are GPL, and other files lack attribution and seem to be in 
the public domain.

* do an audit of the code and add file with the summary, with
something like (you can also have a look at what I did for grads, in
/usr/share/doc/grads-1.9b4/grads-copyright_summary)


Files under the GPL:
libtcp-portmon.h  
libtcp-portmon.c  
audacious.h


Files covered by the BSD license in COPYING:
conky.h
conky.c
remoted.c


Files covered by a BSD license (from http://www.musicpd.org):
libmpdclient.c
libmpdclient.h

No clear license, GPL compatible?
 * Besed on code published in _Mastering Algorithms in C_
 * by Kyle Loudon (O'Reilly 1999).
 * Modified by Philip Kovacs 
hash.h
hash.c

No clear license, seems to be public domain:
 *  $Id: ftp.h 130 2005-08-21 22:10:54Z brenden1 $
ftp.c
ftp.h

 * $Id: mpd.c 598 2006-03-16 18:29:23Z jasper_la $
mpd.c

 *  $Id: linux.c 738 2006-11-08 03:06:42Z pkovacs $
linux.c

No license, no author, certainly public domain
xmms2.c
netbsd.c





Doing a full audit of the source takes more time, but it allows 
to find problematic files. In that case, we have

hash.h
hash.c
 which have clearly 2 authors, but no license. No license (since the 
Bern convention) means a restrictive license (no right to redistribute
or modify). So it needs clarification from Philip.

For ftp.c, ftp.h, mpd.c and linux.c, I am not sure that a rcs Id acts
as an author identification, but if it is the case, then there is the
same issue than for hash.c, otherwise they may be considered public
domain.

(as a side note, if I recall well interfaces cannot have their 
copyright enforced, so if I am not wrong so the .h licenses are 
not really problematic).

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




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

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


[Bug 218599] Review Request: klibido - NNTP (Usenet) file grabber for KDE

2006-12-07 Thread 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: klibido - NNTP (Usenet) file grabber for KDE


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 04:55 EST ---
Oops. Silly mistake, actually had the necessary BR's in there, but for some
strange reason removed it just before I uploaded the -2 release... :-? That's
what you get for being hasty...

Anyhow, here's the new build:
Spec URL: http://www.snoekie.com/rpm/klibido.spec
SRPM URL: http://www.snoekie.com/rpm/klibido-0.2.5-3.src.rpm

Changes:
- Added missing BuildRequires: kdelibs-devel db4-devel


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

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


[Bug 218744] Review Request: sofia-sip - SIP user agent library

2006-12-07 Thread 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: sofia-sip - SIP user agent library


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 05:00 EST ---
Now all builds fine at my powerpc-box.


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

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


[Bug 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 07:04 EST ---
forgot to mention you need to add scriptlets also.
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets?highlight=%28scriptlets%29#head-7103f6c38d1b5735e8477bdd569ad73ea2c49bda

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

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


[Bug 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 07:07 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM and RPMS.
+ source files md5sum. No upstream is available.
0f1d30b02182ff1ec463b3bc37e9326c  resapplet-0.1.1.tar.bz2
+ 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 included in package.
+ %doc is small; no -doc subpackage required.
+ %doc does not affect runtime.
+ COPYING included in %doc.
+ 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 or any subpackages exists
+ no .la files.
+ translations are available
+ Dose owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Desktop files are handled in usual way.
- Not Used gtk-update-icon-cache scriptlet

Add scriptlet update package.
Cool applet tested successfully on FC6 i386 system.

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

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


[Bug 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 07:10 EST ---
oops rpmlint is NOT silent but taking it as silent as only remaining warning is
of no-url-tag and which can be considerable as there is no upstream
(here i am aussming that packages in Fedora Core are existing without URL tag,
so does can be allowed in FE)

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

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


[Bug 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 07:17 EST ---
Got one reference for no-url-tag (bug 200422) where Tim put tarball on somw web
space. Can you also do that? I think you can use people.redhat.com also. So
Update  url tag in SPEC also.

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

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


[Bug 218574] Review Request: perl-Net-SNPP - Perl modules for the Simple Network Pager Protocol

2006-12-07 Thread 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-SNPP - Perl modules for the Simple Network 
Pager Protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 08:20 EST ---
Thanks for the review!  Fixed the summary, imported and built for development.

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

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


[Bug 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 08:20 EST ---
I don't think a empty URL tag is a problem. I'd say put a comment before the
Source0 line in the spec that points to the srpms from Opensuse where you
obtained the sources.

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

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


[Bug 218768] New: Review Request: poppler-extras - PDF rendering library extras (qt/qt4)

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




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

   Summary: Review Request: poppler-extras -  PDF rendering library
extras (qt/qt4)
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/poppler-extras.spec
SRPM URL: 
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/poppler-extras-0.5.4-1.src.rpm
Description:
PDF rendering library extras, including qt and qt4 wrappers.

This is my first quick-n-dirty attempt at building the qt/qt4 poppler wrappers. 
 I've done some minimal testing of the -qt wrapper with kdegraphics' kpdf, and 
it *seems* to work. (:

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

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


[Bug 218769] New: Review Request: hunspell-en - english dictionaries for hunspell

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




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

   Summary: Review Request: hunspell-en - english dictionaries for
hunspell
   Product: Fedora Core
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: http://people.redhat.com/caolanm/hunspell/hunspell-en.spec
SRPM URL: 
http://people.redhat.com/caolanm/hunspell/hunspell-en-0.20040623-1.src.rpm
Description: Move English dictionaries for hunspell out of openoffice.org and 
into a separate package.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218600] Review Request: resapplet - Resolution Switching Applet

2006-12-07 Thread 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: resapplet - Resolution Switching Applet


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 09:17 EST ---
I'm still not sure about the correct placement of pixmap files. Can you please
review the spec file and help me with it. The place should be consistent but now
there are two directories where icons are stored. 

I've updated the spec file with the scripplet (just for one location) and
comment above the Source0.

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

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


[Bug 218769] Review Request: hunspell-en - english dictionaries for hunspell

2006-12-07 Thread 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: hunspell-en - english dictionaries for hunspell


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||207571
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 09:20 EST ---
The side effect of this is that if firefox (etc) use the patch of bug 218762 and
firefox and thunderbird link their dictionary dir to /usr/share/myspell then we
can drop the duplicated english OOo dictionaries, the firefox english
dictionaries and the thunderbird english dictionaries in one fell swoop.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218769] Review Request: hunspell-en - english dictionaries for hunspell

2006-12-07 Thread 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: hunspell-en - english dictionaries for hunspell


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 09:24 EST ---
rpmlint output - none

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/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 218787] New: Review Request: python-pycurl - A Python interface to libcurl

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




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

   Summary: Review Request: python-pycurl - A Python interface to
libcurl
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-1.fc6.spec
SRPM URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-1.fc6.spec
Description:

PycURL is a Python interface to libcurl. PycURL can be used to fetch
objects identified by a URL from a Python program, similar to the
urllib Python module. PycURL is mature, very fast, and supports a lot
of features.

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

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


[Bug 218176] Review Request: gchempaint - A 2D chemical formulae drawing tool

2006-12-07 Thread 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: gchempaint - A 2D chemical formulae drawing tool


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 10:38 EST ---
Oops.. I didn't aware that I have received your mail.

Well, rpmlint complaint about undefined non-weak symbol
can be gained when you execute rpmlint agaist installed
rpm (not against rpm package), i.e.
-
[EMAIL PROTECTED] ~]$ rpmlint gchempaint
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
g_printable_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_set_buffer
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_get_iter_location
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_get_buffer
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_text_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_ellipse_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_bpath_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_get_pango_layout
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
g_printable_export_svg
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rich_text_ext_get_iter_at_location
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_group_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_rect_ext_get_type
W: gchempaint undefined-non-weak-symbol /usr/lib/libgchempaint.so.0.6.6
gnome_canvas_line_ext_get_type
W: gchempaint non-conffile-in-etc /etc/gconf/schemas/gchempaint-arrows.schemas
-
Well, gnome_canvas_???are also defined in sources under libgcpcanvas,
however, those files are not included in libgchempaint.so.

Perhaps it is not expected that this problem will be fixed in 0.6.X version
because fixing this requires entire reconstruction of source.

A. It may be possible that this has been already fixed in development
   version. Would you package 0.7.5 version of gchempaint?
   Then I can check if upgrading to 0.7.5 solves this problem
B. If 0.7.5 still has this problem, would you ask upstream?

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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 10:56 EST ---
According to Packaging Naming it would be better, if a package's name were
pycurl, wouldn't it? There is no need to use 'python-' because first 'py'
letters suggest that it is a python package.
Nevertheless, there is a more serious problem namely package build fails on
x86_64. To fix it you have to change %files section to make it looks like that:
%files
%defattr(-,root,root,-)
%doc COPYING ChangeLog README TODO examples doc tests
%{python_sitearch}/curl
%{python_sitearch}/pycurl.so

I have also removed INSTALL file as it isn't needed when package's already
installed.

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

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


[Bug 215258] Review Request: clucene - A C++ port of Lucene

2006-12-07 Thread 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: clucene - A C++ port of Lucene


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 11:02 EST ---
Ok, I've updated to the latest stable release and added a make check in the 
build.

Spec URL: ftp://czar.eas.yorku.ca/pub/clucene/clucene.spec
SRPM URL: ftp://czar.eas.yorku.ca/pub/clucene/clucene-0.9.16a-1.src.rpm


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

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


[Bug 218722] Review Request: perl-File-ExtAttr - Perl extension for accessing extended attributes of files

2006-12-07 Thread 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-File-ExtAttr - Perl extension for accessing 
extended attributes of files


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 11:20 EST ---
Imported and building for devel; branches requested for FC-[56].

Thanks for the review!

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 12:37 EST ---
Created an attachment (id=143068)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143068action=view)
Diff to xl2tpd.spec

(In reply to comment #9)
 - the space vs tab issue was still there because that error is in the
original
   tarball. I dont want to release the usptream version just for this, so I
   added a temp sed line to the install process.
For this purpose, applying the patch attached will be more
smart.

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 12:47 EST ---
Created an attachment (id=143071)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=143071action=view)
l2tpd - xl2tpd log

 
 What's left now is:
 [EMAIL PROTECTED] devel]# rpmlint
/usr/src/redhat/RPMS/x86_64/xl2tpd-1.1.06-4.x86_64.rpm
 E: xl2tpd non-readable /etc/xl2tpd/l2tp-secrets 0600
 E: xl2tpd non-readable /etc/ppp/chap-secrets.sample 0600
 W: xl2tpd dangerous-command-in-%post mv
 [EMAIL PROTECTED] devel]# rpmlint 
 /usr/src/redhat/SRPMS/xl2tpd-1.1.06-4.src.rpm 
 
 the secrets are supposed to be root-only readable.
 the mv is discussed above.

I checked this for to make sure that your script works,
and it is okay (at least it seems okay for me).

Please check if the log attached is what you expect.

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 12:51 EST ---
Now I approve this package

---
  This package (xl2tpd) is APPROVED by me.
---

Please consider the patch for dealing with init file tab issue
attached on comment #9 .

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 12:53 EST ---
looks good to me.

I used your sed command

spec URL: ftp://ftp.xelerance.com/xl2tpd/xl2tpd.spec
SRPM URL: ftp://ftp.xelerance.com/xl2tpd/xl2tpd-1.1.06-5.src.rpm

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|RELEASE_PENDING




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

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


[Bug 218229] Review Request: php-pear-Image-Color - Manage and handles color data and conversions

2006-12-07 Thread 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: php-pear-Image-Color - Manage and handles color data 
and conversions
Alias: pear-Image-Color

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776  |163778
  nThis||




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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:09 EST ---
Please change the status to CLOSED NEXTRELEASE when
rebuilding for FE-devel is done.

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

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


[Bug 218408] Review Request: xl2tpd - replacement of l2tpd

2006-12-07 Thread 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: xl2tpd - replacement of l2tpd


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|RELEASE_PENDING |CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 174377] Review Request: gnu-smalltalk - GNU Smalltalk

2006-12-07 Thread 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: gnu-smalltalk - GNU Smalltalk


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:13 EST ---
Should be fixed:

SRPM: http://www.herr-schmitt.de/pub/gnu-smalltalk/gnu-smalltalk-2.3-3.src.rpm
SPEC: http://www.herr-schmitt.de/pub/gnu-smalltalk/gnu-smalltalk.spec



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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:28 EST ---
Spec URL:
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-2.fc6.spec
SRPM URL:
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-2.fc6.spec

%changelog
* Thu Dec  7 2006 Jeffrey C. Ollie [EMAIL PROTECTED] - 7.15.5.1-2
- Remove INSTALL from the list of documentation
- Use python_sitearch for all of the files


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

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


[Bug 218230] Review Request: php-pear-Image-Canvas - Common interface to image drawing

2006-12-07 Thread 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: php-pear-Image-Canvas - Common interface to image 
drawing
Alias: pear-Image-Canvas

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:38 EST ---
Image/Canvas/PDF.php requires php-pecl-pdflib.

I provide php-pecl-pdflib and pdflib-lite on my repo (for a long time) but i
dont have submit them to review (yet) because i'm not certain that License is 
OK.

With pdflib present, test doesn't work because of Font used (Courier New
defined as default one in Image/Canvas.php not available).

A temporary solution could be to disable Image/PDF.php

I will investigate on this...


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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:45 EST ---
(In reply to comment #1)
 According to Packaging Naming it would be better, if a package's name were
 pycurl, wouldn't it? There is no need to use 'python-' because first 'py'
 letters suggest that it is a python package.

The way that I read the naming guidelines pycurl would be allowed since there
was a py in the name, but python-pycurl was preferred.  In any case, there's a
Provide: pycurl so that yum install pycurl should work.

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

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


[Bug 218229] Review Request: php-pear-Image-Color - Manage and handles color data and conversions

2006-12-07 Thread 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: php-pear-Image-Color - Manage and handles color data 
and conversions
Alias: pear-Image-Color

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:50 EST ---
This package is used by php-pear-Image-Canvas in which tests works (so this
package works).

REVIEW
* source files match upstream:
a4a1a6f5c0f72d235930902cc1ee2e94  /tmp/Image_Color-1.0.2.tgz
a4a1a6f5c0f72d235930902cc1ee2e94  SOURCES/Image_Color-1.0.2.tgz
* package meets naming and packaging guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible (PHP).
* latest version is being packaged (1.0.2)
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development).
* package installs properly
* rpmlint is silent.
* final provides are sane:
php-pear(Image_Color) = 1.0.2
php-pear-Image-Color = 1.0.2-1.fc7
* %check is not present; no test suite provide.
* owns the directories it creates (see comment)
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* scriptlets are OK (pear install)
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.

Should : remplace in %file
%{pear_phpdir}/*
by
%{pear_phpdir}/Image

More explicit and allow detection of new directory in next release (testdir 
p.e.)

APPROVED

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

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


[Bug 218229] Review Request: php-pear-Image-Color - Manage and handles color data and conversions

2006-12-07 Thread 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: php-pear-Image-Color - Manage and handles color data 
and conversions
Alias: pear-Image-Color

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




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

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


[Bug 174377] Review Request: gnu-smalltalk - GNU Smalltalk

2006-12-07 Thread 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: gnu-smalltalk - GNU Smalltalk


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 13:52 EST ---
Okay!!


  This package (gnu-smalltalk) is APPROVED by me.

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

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


[Bug 218831] New: Review Request: python-urljr - A common interface to urllib2 and curl for making HTTP requests

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




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

   Summary: Review Request: python-urljr - A common interface to
urllib2 and curl for making HTTP requests
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-urljr-1.0.1-1.fc6.spec
SRPM URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-urljr-1.0.1-1.fc6.src.rpm
Description:

URL-related utilities from JanRain, Inc. This package contains the
fetchers module, which provides a common interface to urllib2 and
curl for making HTTP requests.

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

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


[Bug 218831] Review Request: python-urljr - A common interface to urllib2 and curl for making HTTP requests

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

Summary: Review Request: python-urljr - A common interface to urllib2 and curl 
for making HTTP requests


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||218787




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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||218831
  nThis||




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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 14:09 EST ---
MUST items:
 * rpmlint is quiet
?* is the package named well?
?* is spec file name good?
 * package meets Packaging Guidelines
 * package is licensed with an LGPL open-source compatible license
 * License field in spec file matches actual license
 * license file is included in %doc
 * md5sums are matching (464cfbeba150d99d92a407c7c8b751de)
 * package successfully compiles on x86_64
 * BuildRequires listed well
 * no locales
 * no need to %post and %postun sections
 * not relocatable
 * package owns directories well
 * no duplicates in %files
 * %files section includes %defattr
 * proper %clean section
 * macros used well

(In reply to comment #3)
 In any case, there's a
 Provide: pycurl so that yum install pycurl should work.

I can't see it anywhere so add Provides: pycurl into the spec file and I'll
be happy to approve this package.

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

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


[Bug 215258] Review Request: clucene - A C++ port of Lucene

2006-12-07 Thread 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: clucene - A C++ port of Lucene


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 14:14 EST ---
God:
+ Tarball match with upstream version.

Bad:
- You shuld but INSTALL='%{_bindir}/install -c -p' into the make install line to
prevents the timestamps.

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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 14:30 EST ---
(Annoyed Grunt!) I coulda swore that I had it in there...  It should be there
now though:

Spec: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-3.fc6.spec
SRPM:
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-pycurl-7.15.5.1-3.fc6.src.rpm

%changelog
* Thu Dec  7 2006 Jeffrey C. Ollie [EMAIL PROTECTED] - 7.15.5.1-3
- Don't forget to Provide: pycurl!!!


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

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


[Bug 218367] Review Request: compat-guile - Guile compatibility package

2006-12-07 Thread 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: compat-guile - Guile compatibility package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 14:30 EST ---
#001 Parallel builds
%{?_smp_mflags} missing on make

#002 typo
Line 83
# Ouch! guile forgets to set it's onw shared lib path to use shared uninstalled

 own

#003 cleanups ?
[ $RPM_BUILD_ROOT != / ]  rm -rf $RPM_BUILD_ROOT

$RPM_BUILD_ROOT isn't defined up in the spec file ?
Is it useful ?

#004: missing url tag

#005: macro-in-%changelog
W: compat-guile macro-in-%changelog files
W: compat-guile macro-in-%changelog file
W: compat-guile macro-in-%changelog files
W: compat-guile macro-in-%changelog post
W: compat-guile macro-in-%changelog postun
W: compat-guile macro-in-%changelog attr
W: compat-guile macro-in-%changelog post
W: compat-guile macro-in-%changelog description
W: compat-guile macro-in-%changelog files
W: compat-guile macro-in-%changelog postun
W: compat-guile macro-in-%changelog PACKAGE_VERSION
W: compat-guile macro-in-%changelog attr

add a % more in front for those macros 

#007 mixed-use-of-spaces-and-tabs
W: compat-guile mixed-use-of-spaces-and-tabs (spaces: line 4, tab: line 86)

Other Rpmlints issues:
$rpmlint /home/chitlesh/rpmbuild/RPMS/i386/compat-libguile-1.6.7-1.i386.rpm
 dangling-relative-symlink
W: compat-libguile dangling-relative-symlink /usr/share/guile/slib 
../../share/slib
E: compat-libguile non-standard-executable-perm /usr/share/guile/1.6/scripts/c2x
0555
E: compat-libguile non-standard-executable-perm
/usr/share/guile/1.6/scripts/scan-md-module 0555
E: compat-libguile non-standard-executable-perm
/usr/share/guile/1.6/scripts/slurp 0555
E: compat-libguile non-standard-executable-perm
/usr/share/guile/1.6/scripts/make-module-catalog 0555
E: compat-libguile non-standard-executable-perm
/usr/share/guile/1.6/scripts/PROGRAM2 0555

$rpmlint 
/home/chitlesh/rpmbuild/RPMS/i386/compat-libguile-devel-1.6.7-1.i386.rpm
W: compat-libguile-devel no-dependency-on compat-libguile

Fix those, I'll approve it, However it builds successfully under mock.



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

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


[Bug 218839] New: Review Request: pyflakes - A Lint-like tool for Python

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




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

   Summary: Review Request: pyflakes - A Lint-like tool for Python
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/pyflakes-0.0-0.1.10526.fc6.spec
SRPM URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/pyflakes-0.0-0.1.10526.fc6.src.rpm
Description:

PyFlakes is a Lint-like tool for Python, like PyChecker. It is focused
on identifying common errors quickly without executing Python
code. Its primary advantage over PyChecker is that it is fast. You
don't have to sit around for minutes waiting for the checker to run;
it runs on most large projects in only a few seconds.

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

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


[Bug 218229] Review Request: php-pear-Image-Color - Manage and handles color data and conversions

2006-12-07 Thread 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: php-pear-Image-Color - Manage and handles color data 
and conversions
Alias: pear-Image-Color

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 15:00 EST ---
Added Image dir.  Built successfully on devel.  Requested CVS sync with FC5/6. 
Added entry to owners.list.

Thanks for the review.

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

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


[Bug 215258] Review Request: clucene - A C++ port of Lucene

2006-12-07 Thread 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: clucene - A C++ port of Lucene


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 15:09 EST ---
(In reply to comment #9)

 Is this a new packaging requirement? I've never seen anywhere in the 
 guildelines
 where it says this should be done.

No, this is not an official requirement, but it may helpfule to preserve the
timestamps of the files which will be go unmodified into a package. Please see
B/ #174377.

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

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


[Bug 218844] New: Review Request: python-yadis - Relying party support for the Yadis service discovery protocol

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




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

   Summary: Review Request: python-yadis - Relying party support for
the Yadis service discovery protocol
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-yadis-1.0.1-1.fc6.spec
SRPM URL: 
http://repo.ocjtech.us/misc/fedora/6/SRPMS/python-yadis-1.0.1-1.fc6.src.rpm
Description:

This code implements relying party support for the Yadis[1] service
discovery protocol.  The protocol was developed for use by
decentralized URL-based identity systems, but is useful for
advertising services provided by or on behalf of a certain URL.

To learn more about Yadis, see http://www.openidenabled.com/yadis

The JanRain Python OpenID library[2] uses this library for discovery
of OpenID service endpoints.  See that library for an example of
library usage.

1. http://yadis.org/
2. http://www.openidenabled.com/openid/libraries/python/

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

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


[Bug 218844] Review Request: python-yadis - Relying party support for the Yadis service discovery protocol

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

Summary: Review Request: python-yadis - Relying party support for the Yadis 
service discovery protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||218831, 218839




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

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


[Bug 218839] Review Request: pyflakes - A Lint-like tool for Python

2006-12-07 Thread 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: pyflakes - A Lint-like tool for Python


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||218844
  nThis||




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

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


[Bug 218831] Review Request: python-urljr - A common interface to urllib2 and curl for making HTTP requests

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

Summary: Review Request: python-urljr - A common interface to urllib2 and curl 
for making HTTP requests


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||218844
  nThis||




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

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


[Bug 215258] Review Request: clucene - A C++ port of Lucene

2006-12-07 Thread 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: clucene - A C++ port of Lucene


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 15:28 EST ---
Since this is not official, I don't think it should block this being accepted.
I've seen the bug you pointed to, but it isn't clear why/how helpful doing that
is. Besides, I've rarely seen this INSTALL option purposefully used in the
Fedora sphere. 

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

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


[Bug 218176] Review Request: gchempaint - A 2D chemical formulae drawing tool

2006-12-07 Thread 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: gchempaint - A 2D chemical formulae drawing tool


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 15:36 EST ---
Well, the problem with development version is the following: it requires
development version of gnome-chemistry-utils, which requires goffice = 0.3
(also an unstable development version, shipped neither by fc6 nor by rawhide).
So no matter if the problem was fixed in unstable branch, shipping it for fedora
would be a bit problematic. I can ask upstream for fix anyway, but IMO it is
very unlikely that someone would want to link to libgchempaint.so

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

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


[Bug 218176] Review Request: gchempaint - A 2D chemical formulae drawing tool

2006-12-07 Thread 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: gchempaint - A 2D chemical formulae drawing tool


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 15:45 EST ---
Upstream bug report:
https://savannah.nongnu.org/bugs/index.php?18480

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

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


[Bug 218844] Review Request: python-yadis - Relying party support for the Yadis service discovery protocol

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

Summary: Review Request: python-yadis - Relying party support for the Yadis 
service discovery protocol


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||218852
  nThis||




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

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


[Bug 218831] Review Request: python-urljr - A common interface to urllib2 and curl for making HTTP requests

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

Summary: Review Request: python-urljr - A common interface to urllib2 and curl 
for making HTTP requests


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||218852
  nThis||




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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 16:49 EST ---
Now it looks really good :)

Approved!

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

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


[Bug 218230] Review Request: php-pear-Image-Canvas - Common interface to image drawing

2006-12-07 Thread 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: php-pear-Image-Canvas - Common interface to image 
drawing
Alias: pear-Image-Canvas

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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 16:50 EST ---
I have updated the spec to remove this file, however I think it would be better
to add a php-pecl-pdflib package to Extras.  I think the license should be okay.
 If you want to put it up for review I can review it for you.  Once it's in I
can also patch this package to not use Courier New font in the tests.

Spec URL: http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas.spec
SRPM URL:
http://tkmame.retrogames.com/fedora-extras/php-pear-Image-Canvas-0.3.0-2.src.rpm

%changelog
* Thu Dec 07 2006 Christopher Stone [EMAIL PROTECTED] 0.3.0-2
- Remove PDF.php until php-pecl-pdflib is added to Extras


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

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


[Bug 218787] Review Request: python-pycurl - A Python interface to libcurl

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

Summary: Review Request: python-pycurl - A Python interface to libcurl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 17:04 EST ---
Thanks for the review!  Imported and built for devel, branches for FC-5 and FC-6
requested.

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

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


[Bug 218831] Review Request: python-urljr - A common interface to urllib2 and curl for making HTTP requests

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

Summary: Review Request: python-urljr - A common interface to urllib2 and curl 
for making HTTP requests


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


Bug 218831 depends on bug 218787, which changed state.

Bug 218787 Summary: Review Request: python-pycurl - A Python interface to 
libcurl
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=218787

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

2006-12-07 Thread 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: glump - A small web application to glue files from 
multiple sources


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 17:19 EST ---
Forbidden

You don't have permission to access /misc/fedora/6/SRPMS/glump-0.9.11-1.fc6.spec
on this server.

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

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


[Bug 218577] Review Request: glump - A small web application to glue files from multiple sources

2006-12-07 Thread 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: glump - A small web application to glue files from 
multiple sources


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 17:24 EST ---
(In reply to comment #1)

 You don't have permission to access 
 /misc/fedora/6/SRPMS/glump-0.9.11-1.fc6.spec
 on this server.

Whoops! Fixed...



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

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


[Bug 218887] New: Review Request: fxload - Firmware loader

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




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

   Summary: Review Request: fxload - Firmware loader
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://bart.ulyssis.org/fedora/fxload/fxload.spec
SRPM URL: http://bart.ulyssis.org/fedora/fxload/fxload-2002_04_11-2.fc6.src.rpm

Description: This program is conveniently able to download firmware into FX and 
FX2 EZ-USB devices, as well as the original AnchorChips EZ-USB.  It is
intended to be invoked by udev scripts when the unprogrammed device
appears on the bus.

This was part of the hotplug package which was removed in favor of udev. Some 
drivers like the wis-go7007 driver still use it to make udev load the initial 
firmware.
#178679 contains some more information.

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 19:53 EST ---
(In reply to comment #22)
 (In reply to comment #19)

 Does conky uses directly the glib functions? If not, it is better
 to have glib-devel as an indirect dependency through audacious-devel.
 Thus if audacious stops using the glib, conky wouldn't need to be modified.
 If glib-devel is a direct dependency of conky it would be better to
 list it in buildrequires, but it doesn't seems to be the case since
 conky itself don't include the glib headers (unless I am missing 
 something).

Yes, conky uses glib directly in audacious.c, i.e. #include glib.h.
The audacious status interface allocates gchar * strings that must be
freed by the client with g_free() -- song titles, etc.


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

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


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

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

Summary: Review Request: PyQt4: Python bindings for Qt4


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 20:09 EST ---
rpmbuild --rebuild PyQt4-4.1-1.src.rpm 
[ ... chug ...]
ERROR   0001: file '/usr/bin/pylupdate4' contains a standard 
rpath '/usr/lib64' in [/usr/lib64]
ERROR   0001: file '/usr/bin/pyrcc4' contains a standard rpath '/usr/lib64' in 
[/usr/lib64]

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

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


[Bug 208420] Review Request: conky - A system monitor for X originally based on the torsmo code

2006-12-07 Thread 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: conky - A system monitor for X originally based on the 
torsmo code


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 20:51 EST ---
(In reply to comment #23)
 Doing a full audit of the source takes more time, but it allows 
 to find problematic files. In that case, we have
 
 hash.h
 hash.c
  which have clearly 2 authors, but no license. No license (since the 
 Bern convention) means a restrictive license (no right to redistribute
 or modify). So it needs clarification from Philip.

That code originates from here:

http://www.kyleloudon.com/books, specifically the Read a sample link:
http://www.oreilly.com/catalog/masteralgoc/chapter/ch08.pdf

I believe the licensing falls under the O'Reilly policy stated here:

http://www.oreilly.com/pub/a/oreilly/ask_tim/2001/codepolicy.html

I have emailed the author for clarification in any event.

Phil


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

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


[Bug 215883] Review Request: idioskopos - C++ Introspection Library

2006-12-07 Thread 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: idioskopos - C++ Introspection Library
Alias: idioskopos

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 174377] Review Request: gnu-smalltalk - GNU Smalltalk

2006-12-07 Thread 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: gnu-smalltalk - GNU Smalltalk


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 21:50 EST ---
Paolo,

Jochen tried to rebuild 2.3-3, however, it failed with x86_64.
Testing intmath.st fails with segv. Would you check this problem?

Reference:
http://buildsys.fedoraproject.org/logs/fedora-development-extras/23119-gnu-smalltalk-2.3-3.fc7/x86_64/build.log

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

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


[Bug 218360] Review Request: evolution-plugin-remove-duplicates - Evolution plugin for removing duplicate mails

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

Summary: Review Request: evolution-plugin-remove-duplicates - Evolution plugin 
for removing duplicate mails


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-07 22:51 EST ---
http://hircus.org/fedora/evolution-plugin-remove-duplicates/evolution-remove-duplicates.spec
http://hircus.org/fedora/evolution-plugin-remove-duplicates/evolution-remove-duplicates-0.0.2-3.src.rpm

Rex Dieter pointed out that the evolution-plugin-2.8.pc in turn depends on
another .pc whose package was not required by evolution-devel . I've added a
temporary BR on e-d-s-devel for now, that can be removed once that packaging bug
is fixed.

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

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


[Bug 196393] Review Request: svrcore-devel

2006-12-07 Thread 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: svrcore-devel
Alias: svrcore-devel

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-12-08 00:41 EST ---
I'm looking into autotooling this.  It's not impossible but I'm going to have to
look into the coreconf stuff more closely to make sure I'm not missing something
in my initial pass.  Some questions:

* Why is this package called srvcore-devel?  The library looks like it's called
libsrvcore so the -devel extension looks extraneous. (This will also cause you
packaging headaches as there's usually a main package with the shared libs and a
subpackage named %{name}-devel for static libraries and headers.  If you did
that in this case you'd have svrcore-devel-devel.  So you can either rename at
the upstream level or the packaging level.)

* Is it okay if I change the directory structure or are you still committed to
building this in a mozilla tree where the directory structure is important?

* Does anyone understand coreconf and want to walk me through it on IRC -- if
not, I'll figure it out on my own.  It's just a question of speed and assurance
that I'm not taking a false start.

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

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


[Bug 218176] Review Request: gchempaint - A 2D chemical formulae drawing tool

2006-12-07 Thread 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: gchempaint - A 2D chemical formulae drawing tool


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





--- Additional Comments From [EMAIL PROTECTED]  2006-12-08 01:50 EST ---
Comment from Jean Brefort:
Hmm, not easily fixed. libgchempaint was not intended to be used by other
programs (it should be possible with 0.8.0 but this one is not released yet). At
the moment, libgcpcanvas is only used as a static library, so adding -lgcpcanvas
to LIBS for libgchempaint just makes libtool fail. If fixing this bug is a
requirement for you, the solution is to build a shared version of libgcpcanvas.
Tell me if you need it. Also, please apply patch for #18159 before releasing.

So, is fixing this bug a requirement? Al for the patch, I'll add it later today.

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

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