[Bug 305561] Review Request: bluecurve-icon-theme - Bluecurve icon theme

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

Summary: Review Request: bluecurve-icon-theme - Bluecurve icon theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 02:00 EST ---
should be good to go

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

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


[Bug 275871] Review Request: aldrin - Modular music sequencer/tracker

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

Summary: Review Request: aldrin - Modular music sequencer/tracker


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 02:02 EST ---
New patch seems to work well, then 

diff -up aldrin-0.11/share/aldrin/router.py.docfix
aldrin-0.11/share/aldrin/router.py
--- aldrin-0.11/share/aldrin/router.py.docfix   2007-09-25 21:16:54.0 
+0200
+++ aldrin-0.11/share/aldrin/router.py  2007-09-25 21:17:19.0 +0200
@@ -698,8 +698,8 @@ class ParameterDialog(gtk.Dialog):
uri = filenameify(self.pluginloader.get_uri())
name = filenameify(self.pluginloader.get_name())
helpfilepaths = [
-   filepath('../doc/zzub/plugins/' + uri + '/index.html'),
-   filepath('../doc/zzub/plugins/' + name + '/index.html'),
+   filepath('../doc/libzzub-0.2.3/plugins/' + uri +
'/index.html'),
+   filepath('../doc/libzzub-0.2.3/plugins/' + name +
'/index.html'),
]
for path in helpfilepaths:
print searching for '%s'... % path
-
  It may be better that this aldrin require specific version
  of pyzzub, i.e.
  Requires: pyzzub = 0.2.3

-- 
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 306431] New: Review Request: scim-python - a python wrapper for SCIM input platform

2007-09-26 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=306431

   Summary: Review Request: scim-python - a python wrapper for SCIM
input platform
   Product: Fedora
   Version: devel
  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://groups.google.com/group/scim-python/web/scim-python.spec
SRPM URL: 
http://groups.google.com/group/scim-python/web/scim-python-0.1.2-1.fc7.src.rpm
Description: a python wrapper for SCIM input platform. With it, IM developers 
may use python language to create Input Methods. This package also includes a 
sample input engine - English Writer.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:08 EST ---
need fix
1)Add  BuildRequires:  perl(XML::Parser),pygtk2-devel
2) Remove BuildRequires: autoconf, automake

-- 
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 234581] Review Request: Mapnik cartography library - first package, need sponsor.

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

Summary: Review Request: Mapnik cartography library - first package, need 
sponsor.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:08 EST ---
Upstream is on vacation for the next couple of weeks with no Internet access. 
I'll ping them when they return and try to get this moving.

-- 
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 286691] Review Request: libgssglue - renaming libgssapi to libgssglue

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

Summary: Review Request: libgssglue - renaming libgssapi to libgssglue


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:18 EST ---
thanks for your updates to SPEC.
But sorry I can't approve this package as I can still see package 
file /etc/gssapi_mech.conf from install of libgssglue-0.1-2.fc8 conflicts with
file from package libgssapi-0.11-1.fc7

What workaround I can suggest here is to add 
Provides: libgssapi = 0.11-1.fc7
Obsoletes: libgssapi = 0.11-1.fc7
below following line in SPEC file
Requires: krb5-libs = 1.5

When built with above changes and tried to install on F7 machine as it gave me
 rpm -Uvh libgssglue-0.1-2.fc7.i386.rpm libgssglue-devel-0.1-2.fc7.i386.rpm
error: Failed dependencies:
libgssapi.so.2 is needed by (installed) libtirpc-0.1.7-7.fc7.i386
libgssapi.so.2 is needed by (installed) nfs-utils-1.1.0-3.fc7.i386
libgssapi.so.2(libgssapi_CITI_2) is needed by (installed)
libtirpc-0.1.7-7.fc7.i386
libgssapi.so.2(libgssapi_CITI_2) is needed by (installed)
nfs-utils-1.1.0-3.fc7.i386

That mean you need to ask all packages that depends on libgssapi.so.2 to build
against libgssglue.



-- 
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 294641] Review Request: aboot - A bootloader which can be started from the SRM console.

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

Summary: Review Request: aboot - A bootloader which can be started from the SRM 
console.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:17 EST ---
Duh? Why 4Suite!?

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:28 EST ---
scim-python.spec and scim-python-0.1.2-1.fc7.src.rpm have been refreshed. Please
check it.

Below is diff from scim-python.spec.
Thanks.

===
--- scim-python.spec.in (revision 49)
+++ scim-python.spec.in (working copy)
@@ -9,8 +9,8 @@
 Source0:http://scim-python.googlecode.com/files/%{name}-%{version}.tar.bz2
 BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
-BuildRequires:  scim-devel, gettext, gettext-devel, autoconf, automake,
libtool, python
-Requires:   scim python-enchant
+BuildRequires:  scim-devel, gettext, gettext-devel, libtool, python-devel,
pygtk2-devel, perl(XML::Parser)
+Requires:   scim, python-enchant, pygtk2
 
 %description
 Python IMEngine for SCIM.


-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 03:43 EST ---
- I think the License field should say LGPLv2+

- gettext-devel will pull in gettext anyway

- %{_libdir}/scim*/* are owned by scim so it should be
  %{_libdir}/scim*/*/%{name}*.so say

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:07 EST ---
(In reply to comment #3)
 - I think the License field should say LGPLv2+
 
 - gettext-devel will pull in gettext anyway
 
 - %{_libdir}/scim*/* are owned by scim so it should be
   %{_libdir}/scim*/*/%{name}*.so say

Those problems have been fixed and files have been refreshed. Please check it.
Thanks.

===
--- scim-python.spec.in (revision 49)
+++ scim-python.spec.in (working copy)
@@ -3,14 +3,14 @@
 Release:1%{?dist}
 Summary:Python IMEngine for Smart Common Input Method platform
 
-License:LGPLv2
+License:LGPLv2+
 Group:  System Environment/Libraries
 URL:http://code.google.com/p/scim-python/
 Source0:http://scim-python.googlecode.com/files/%{name}-%{version}.tar.bz2
 BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
 
-BuildRequires:  scim-devel, gettext, gettext-devel, autoconf, automake,
libtool, python
-Requires:   scim python-enchant
+BuildRequires:  scim-devel, gettext-devel, libtool, python-devel, pygtk2-devel,
perl(XML::Parser)
+Requires:   scim, python-enchant, pygtk2
 
 %description
 Python IMEngine for SCIM.
@@ -40,9 +40,8 @@
 %defattr(-,root,root,-)
 %doc AUTHORS COPYING README TODO
 %{_libdir}/python*/site-packages/*.so
-%{_libdir}/scim*/*
-%{_datadir}/scim-python/engine/*
-%{_datadir}/scim-python/setupui/*
+%{_libdir}/scim*/*/%{name}*.so
+%{_datadir}/scim-python/*
 %{_datadir}/scim/icons/*


-- 
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 225761] Merge Review: fonts-chinese

2007-09-26 Thread 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: fonts-chinese


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:10 EST ---
Can you prepare a draft spec file and I will give feedback.
Look at the new fonts-korean.spec as an example.

-- 
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 248617] Review Request: cdcollect - Simple CD/DVD catalog for GNOME

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

Summary: Review Request: cdcollect - Simple CD/DVD catalog for GNOME


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:11 EST ---
updated Spec URL: http://fedora.danny.cz/cdcollect.spec
updated SRPM URL: http://fedora.danny.cz/cdcollect-0.6.0-3.src.rpm

changes
- fixed URLs
- removed unneeded BR: pkgconfig

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:19 EST ---
 Those problems have been fixed

Thanks. :)

 and files have been refreshed. Please check it.

Could you please provide the urls for the spec and srpm for each revision.
The release number should be bumped too please.

You need to own %{_datadir}/scim-python/ too.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:22 EST ---
rpmlint says

scim-python.i386: W: symlink-should-be-relative
/usr/lib/scim-1.0/IMEngine/scim-python.so 
/usr/lib/python2.5/site-packages/scim.so
scim-python.i386: W: symlink-should-be-relative
/usr/lib/scim-1.0/SetupUI/scim-python-setup.so
/usr/lib/python2.5/site-packages/scim.so

I think it is not a blocker but you might prefer to use a relsymlink.

-- 
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 253812] Package Review: taipeifonts (split from fonts-chinese).

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

Summary: Package Review: taipeifonts (split from fonts-chinese).


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:27 EST ---
ugh, sorry, buildreqs listed should also have been under Good.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:30 EST ---
* For python sitelib/arch directory, please follow:
  http://fedoraproject.org/wiki/Packaging/Python

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

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


[Bug 305281] Review Request: cpl - ESO (European Southern Observatory) library for automated astronomical data-reduction tasks

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

Summary: Review Request: cpl - ESO (European Southern Observatory) library for 
automated astronomical data-reduction tasks


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: cpl - ESO   |Review Request: cpl - ESO
   |(European Sothern   |(European Southern
   |Observatory) library for|Observatory) library for
   |automated astronomical data-|automated astronomical data-
   |reduction tasks |reduction tasks




-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:36 EST ---
* Rebuild fails at least on x86_64
  http://koji.fedoraproject.org/koji/taskinfo?taskID=175148

  - Note: the build.log says:

RPM build errors:
File not found by glob:
/var/tmp/scim-python-0.1.2-1.fc8-root-kojibuilder/usr/lib64/python*/site-packages/*.so
Installed (but unpackaged) file(s) found:
   /usr/lib/python2.5/site-packages/scim.la
   /usr/lib/python2.5/site-packages/scim.so

scim.so must be installed under python_sitearch (i.e.
under /usr/lib64), not under python_sitelib

-- 
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 253812] Package Review: taipeifonts (split from fonts-chinese).

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

Summary: Package Review: taipeifonts (split from fonts-chinese).


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 04:48 EST ---
How about
http://ftp2.tnc.edu.tw/pub3/CLE/CLE/devel/wjwu/slackware/slackware-9.1/source/taipeifonts-1.2/%{name}-%{version}.tar.gz
for the source url?

Do you have a better one?

Also I just noticed the Buildroot should updated to the current 
recommendations:
http://fedoraproject.org/wiki/Packaging/Guidelines#head-b4fdd45fa76cbf54c885ef0836361319ab962473


-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 05:07 EST ---
(In reply to comment #5)
 Could you please provide the urls for the spec and srpm for each revision.
 The release number should be bumped too please.

Sorry. I refreshed the files on web. I do not reserve the old version.

 You need to own %{_datadir}/scim-python/ too.
Fixed. add %dir %{_datadir}/scim-python

(In reply to comment #6)
 rpmlint says
 
 scim-python.i386: W: symlink-should-be-relative
 /usr/lib/scim-1.0/IMEngine/scim-python.so 
 /usr/lib/python2.5/site-packages/scim.so
 scim-python.i386: W: symlink-should-be-relative
 /usr/lib/scim-1.0/SetupUI/scim-python-setup.so
 /usr/lib/python2.5/site-packages/scim.so
 
 I think it is not a blocker but you might prefer to use a relsymlink.

It has a little difficulty to create relsymlink now. If it is not blocker, I
will try fix it later. 

(In reply to comment #8)
 * Rebuild fails at least on x86_64
Fixed. Please retry it.
Thanks

The new URLs:
http://groups.google.com/group/scim-python/web/scim-python.spec-2
http://groups.google.com/group/scim-python/web/scim-python-0.1.2-2.fc7.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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:04 EST ---
 * cool. So many comments. thanks to Jens and Mamoru.

 *rpmlint on SRPM says 
scim-python.src:54: W: macro-in-%changelog dir
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'.

* on RPM it said
scim-python.i386: W: symlink-should-be-relative
/usr/lib/scim-1.0/IMEngine/scim-python.so 
/usr/lib/python2.5/site-packages/scim.so
Absolute symlinks are problematic eg. when working with chroot environments.

scim-python.i386: W: symlink-should-be-relative
/usr/lib/scim-1.0/SetupUI/scim-python-setup.so
/usr/lib/python2.5/site-packages/scim.so
Absolute symlinks are problematic eg. when working with chroot environments.


 * And build.log showed 
warning: File listed twice: /usr/lib/python2.5/site-packages/scim.so

So from %files section you should remove 
%{_libdir}/python*/site-packages/*.so

* otherwise package looks ok.



-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:17 EST ---
Still seems not okay.

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

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:31 EST ---
(In reply to comment #11)
 Still seems not okay.
 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=175238

Could you tell me how to build it on koji? When I update the spec file, I may
try it by self.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:47 EST ---
You can try by:

koji build --scratch target srpm_you_want_to_try
Currently target can be dist-f8 or dist-fc7-updates-candidate.

If rebuild ends successfully, the result binary rpms are put under
htt://koji.fedoraproject.org/scratch/your_FAS_name/task_task_id/

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:48 EST ---
(In reply to comment #14)

 If rebuild ends successfully, the result binary rpms are put under
 htt://koji.fedoraproject.org/scratch/your_FAS_name/task_task_id/

Of course this is http://

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 06:46 EST ---
(In reply to comment #11)
 Still seems not okay.
 
 http://koji.fedoraproject.org/koji/taskinfo?taskID=175238

I modified to fix rpmlint checking warning on SRPM and build warning for rpm.
And I also fix the build problem for X86_64 arch, but without test. :-/

The new URLs:
http://groups.google.com/group/scim-python/web/scim-python.spec-3
http://groups.google.com/group/scim-python/web/scim-python-0.1.2-3.fc7.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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:05 EST ---
It passed the koji build. 
The problem is in a Makefile.am in the tarball. It does not install the .so file
into /usr/lib64/... in x86_64 arch, and it still install .so into /usr/lib/...


-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:01 EST ---
I think if macro is not getting expanded to lib64.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:15 EST ---
In that case you can use arch specific patch for Makefile like say
%ifarch x86_64
%patch1 -p1
%endif

and need to change accordingly at other places in 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 266261] Review Request: knm_new-fonts - 12x12 JIS X 0208 Bitmap font

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

Summary: Review Request: knm_new-fonts - 12x12 JIS X 0208 Bitmap font


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 248617] Review Request: cdcollect - Simple CD/DVD catalog for GNOME

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

Summary: Review Request: cdcollect - Simple CD/DVD catalog for GNOME


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:11 EST ---
Review:
+ package builds in mock (development i386).
+ rpmlint is silent for SRPM
- rpmlint complains for RPM as
  cdcollect.i386: E: no-binary
  cdcollect.i386: E: only-non-binary-in-usr-lib
   But these can be ignored for mono packages.
+ source files match upstream.
33f71604b9dfb84497b4bc2fce69e89b  cdcollect-0.6.0.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 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.
+ no .la files.
+ translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ GConf scriptlets are used.
+ Desktop file handled correctly.
+ Package cdcollect-0.6.0-3.fc8 -
  Provides: config(cdcollect) = 0.6.0-3.fc8 mono(cdcollect) = 0.2.2825.11093
  Requires: /bin/sh config(cdcollect) = 0.6.0-3.fc8 gnome-sharp gtk-sharp2 =
2.8.0 mono-core = 1.1.17 mono-data-sqlite sqlite = 3.3.5
+ GUI app.

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 253175] Review Request: jisksp16-1990-fonts - 16x16 JIS X 0212:1990 Bitmap font

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

Summary: Review Request: jisksp16-1990-fonts - 16x16 JIS X 0212:1990 Bitmap font


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 305581] Review Request: bluecurve-kde-theme - Bluecurve KDE theme

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

Summary: Review Request: bluecurve-kde-theme - Bluecurve KDE theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:19 EST ---
Where's the KWin theme?

-- 
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 305631] Review Request: bluecurve-qt-engine - Bluecurve qt engine

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

Summary: Review Request: bluecurve-qt-engine - Bluecurve qt engine


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:21 EST ---
Is the BR kdebase-devel actually needed here? I think it isn't, it's needed 
only for the KWin theme, which isn't part of this package.

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

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


[Bug 244936] Review Request: dbench - Filesystem load benchmarking tool

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

Summary: Review Request: dbench - Filesystem load benchmarking tool


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:25 EST ---
Is this still waiting for any builds or we can CLOSE this review ticket?

-- 
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 248617] Review Request: cdcollect - Simple CD/DVD catalog for GNOME

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

Summary: Review Request: cdcollect - Simple CD/DVD catalog for GNOME


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:27 EST ---
New Package CVS Request
===
Package Name: cdcollect
Short Description: Simple CD/DVD catalog for GNOME
Owners: sharkcz
Branches: F-7
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:32 EST ---
* For scim-python*.so:
  - Well, this package installs scim-python.so under
/usr/lib/scim-1.0/IMEngine/ (on i386), however on my rawhide
system the directory /usr/lib/scim-1.0/IMEngine/ does not exist
and _perhaps_ this must be installed under
/usr/lib/scim-1.0/1.4.0/IMEngine/ (this directory is owned
by scim-1.4.7-5.fc8)

  - Similary, /usr/lib/scim-1.0/SetupUI does not exist on my system,
and /usr/lib/scim-1.0/1.4.0/SetupUI/ is owned by scim

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 07:54 EST ---
If you don't already the scim pkgconfig config from scim-devel should
be used to give you the correct install location.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 08:02 EST ---
(In reply to comment #19)
 * For scim-python*.so:
   - Well, this package installs scim-python.so under
 /usr/lib/scim-1.0/IMEngine/ (on i386), however on my rawhide
 system the directory /usr/lib/scim-1.0/IMEngine/ does not exist
 and _perhaps_ this must be installed under
 /usr/lib/scim-1.0/1.4.0/IMEngine/ (this directory is owned
 by scim-1.4.7-5.fc8)
 
   - Similary, /usr/lib/scim-1.0/SetupUI does not exist on my system,
 and /usr/lib/scim-1.0/1.4.0/SetupUI/ is owned by scim

yes. you are right. Other scim-* say scim-hangul, scim-anthy are installing
under /usr/lib/scim-1.0/1.4.0/IMEngine/ and /usr/lib/scim-1.0/1.4.0/SetupUI/

For this you need to patch configure.in
run autoconf in 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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 08:08 EST ---
SCIM_MODULEDIR=`$PKG_CONFIG --variable=moduledir scim` should help.

-- 
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 306431] Review Request: scim-python - a python wrapper for SCIM input platform

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

Summary: Review Request: scim-python - a python wrapper for SCIM input platform


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 08:22 EST ---
or just directly patch configure file but that made me to have %files section
for that as
%{_libdir}/scim-1.0/1.4.0/IMEngine/scim-python.so
%{_libdir}/scim-1.0/1.4.0/SetupUI/scim-python-setup.so
instead to have it as
%{_libdir}/scim-1.0/*/IMEngine/scim-python.so
%{_libdir}/scim-1.0/*/SetupUI/scim-python-setup.so
which is preferred by most of other scim-* packages

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 305581] Review Request: bluecurve-kde-theme - Bluecurve KDE theme

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

Summary: Review Request: bluecurve-kde-theme - Bluecurve KDE theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:34 EST ---
It's here: http://rstrode.fedorapeople.org/bluecurve-kwin-theme/checkout/

I apparently didn't file a review request for it.  Will do so now (and double
check that I didn't miss any others)

-- 
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 305441] redhat-artwork should be split up into multiple packages

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

Summary: redhat-artwork should be split up into multiple packages
Alias: ArtworkSplit

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||306951




-- 
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 306951] New: Review Request: bluecurve-kwin-theme - Bluecurve kwin theme

2007-09-26 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=306951

   Summary: Review Request: bluecurve-kwin-theme - Bluecurve kwin
theme
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-
[EMAIL PROTECTED],[EMAIL PROTECTED],[EMAIL PROTECTED]
com,[EMAIL PROTECTED],[EMAIL PROTECTED]


Spec URL:
http://rstrode.fedorapeople.org/bluecurve-kde-theme/checkout/bluecurve-kde-theme.spec

SRPM URL:
http://rstrode.fedorapeople.org/bluecurve-kde-theme/checkout/bluecurve-kde-theme-1.0.0-1.fc8.src.rpm

Description:
This package contains the Bluecurve kwin theme.

-- 
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 286691] Review Request: libgssglue - renaming libgssapi to libgssglue

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

Summary: Review Request: libgssglue - renaming libgssapi to libgssglue


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:39 EST ---
 Provide me package links as you did in initial comment.
I just assumed since the spec file and srpm were in the
exact same place you could continue to used the original
links. Sorry for such a bad assumption. I'll guess I go head 
and cut and past them from the original posting.

 What workaround I can suggest here is to add 
 Provides: libgssapi = 0.11-1.fc7
 Obsoletes: libgssapi = 0.11-1.fc7
 below following line in SPEC file
 Requires: krb5-libs = 1.5
updated spec and srpm.
Spec URL: http://people.redhat.com/steved/libgssglue/libgssglue.spec
SRPM URL: 
http://people.redhat.com/steved/libgssglue/libgssglue-0.1-2.fc8.src.rpm

 rpm -Uvh libgssglue-0.1-2.fc7.i386.rpm libgssglue-devel-0.1-2.fc7.i386.rpm
 error: Failed dependencies:
Of course there will be dependency problem because I
have not been able to check this package in!
As soon as I can check this package in, those 
dependency problem will go away because I will 
be able to update the packages at are depending
it. Please note *all* the packages that are dependent
on this I maintain. 


-- 
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 306951] Review Request: bluecurve-kwin-theme - Bluecurve kwin theme

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

Summary: Review Request: bluecurve-kwin-theme - Bluecurve kwin theme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:38 EST ---
err, the urls are wrong,

Spec URL:
http://rstrode.fedorapeople.org/bluecurve-kwin-theme/checkout/bluecurve-kwin-theme.spec

SRPM URL:
http://rstrode.fedorapeople.org/bluecurve-kwin-theme/checkout/bluecurve-kwin-theme-1.0.0-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 305531] Review Request: bluecurve-gdm-theme - Bluecurve GDM theme

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

Summary: Review Request: bluecurve-gdm-theme - Bluecurve GDM theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:43 EST ---
Well, it doesn't actually use a *trademarked* logo (this is just the old Fedora
font on a big blue background, right?)

-- 
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 305531] Review Request: bluecurve-gdm-theme - Bluecurve GDM theme

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

Summary: Review Request: bluecurve-gdm-theme - Bluecurve GDM theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:47 EST ---
Okay, yea, let's make this one system-logos then.

-- 
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 230164] Review Request: rt73usb-firmwa re - Firmware for Ralink® RT2571W/RT2671 A/B/G network adaptors

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

Summary: Review Request: rt73usb-firmware - Firmware for Ralink® RT2571W/RT2671 
A/B/G network adaptors


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:47 EST ---
ok,
this package can be 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 230164] Review Request: rt73usb-firmwa re - Firmware for Ralink® RT2571W/RT2671 A/B/G network adaptors

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

Summary: Review Request: rt73usb-firmware - Firmware for Ralink® RT2571W/RT2671 
A/B/G network adaptors


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|182235  |
  nThis||
   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:54 EST ---
New Package CVS Request
===
Package Name:  rt73usb-firmware
Short Description: Firmware for Ralink® RT2571W/RT2671 A/B/G network adaptors
Owners:kwizart
Branches:  F-7
InitialCC: empty
Commits by cvsextras: yes

Thx for 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 290581] Review Request: madan-fonts - Font for Nepali

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

Summary: Review Request: madan-fonts - Font for Nepali


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 09:53 EST ---
Updated again:
Spec URL: http://rbhalera.fedorapeople.org/madan-fonts/madan-fonts.spec
SRPM URL: 
http://rbhalera.fedorapeople.org/madan-fonts/madan-fonts-1.0-4.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 286691] Review Request: libgssglue - renaming libgssapi to libgssglue

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

Summary: Review Request: libgssglue - renaming libgssapi to libgssglue


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:06 EST ---
* thanks very much that finally you got my point that links must be provided
each time to reviewer to avoid confusion between old and new SPEC.

* But that doesn't mean you should not increment release tag. And for above
change, do you think its so small to previous SPEC you provided?
No. You must update release tag and provide new links for final review.

*Remember to run rpmlint on all SRPM,RPM files.


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

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


[Bug 305531] Review Request: bluecurve-gdm-theme - Bluecurve GDM theme

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

Summary: Review Request: bluecurve-gdm-theme - Bluecurve GDM theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:01 EST ---
No, I mean if it's not trademarked, you can just put the graphic in the theme.
Or leave it in fedora-logos, either or.

-- 
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 305531] Review Request: bluecurve-gdm-theme - Bluecurve GDM theme

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

Summary: Review Request: bluecurve-gdm-theme - Bluecurve GDM theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:11 EST ---
ah right, that's even nicer

-- 
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 305711] Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM theme

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

Summary: Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM 
theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:15 EST ---
This patch got lost here:
http://cvs.fedora.redhat.com/viewcvs/rpms/redhat-artwork/devel/redhat-artwork-7.0.0-kdm-userlist.patch?rev=1.1view=markup

-- 
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 306951] Review Request: bluecurve-kwin-theme - Bluecurve kwin theme

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

Summary: Review Request: bluecurve-kwin-theme - Bluecurve kwin theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:12 EST ---
Than's patch to fix bug #276731:
http://cvs.fedora.redhat.com/viewcvs/rpms/redhat-artwork/devel/redhat-artwork-7.0.0-kwin-bluecurve.patch?rev=1.1view=markup
got lost again. Can this please be committed to the upstream tree?

-- 
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 306951] Review Request: bluecurve-kwin-theme - Bluecurve kwin theme

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

Summary: Review Request: bluecurve-kwin-theme - Bluecurve kwin theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:22 EST ---
sure

-- 
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 305711] Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM theme

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

Summary: Review Request: fedoraflyinghigh-kdm-theme - Fedora Flying High KDM 
theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 10:29 EST ---
thanks, applied

-- 
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 305441] redhat-artwork should be split up into multiple packages

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

Summary: redhat-artwork should be split up into multiple packages
Alias: ArtworkSplit

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||307171




-- 
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 307171] New: Review Request: bluecurve-kdm-theme - Bluecurve KDM theme

2007-09-26 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=307171

   Summary: Review Request: bluecurve-kdm-theme - Bluecurve KDM
theme
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-
[EMAIL PROTECTED],[EMAIL PROTECTED],[EMAIL PROTECTED]
com,[EMAIL PROTECTED],[EMAIL PROTECTED]


Spec URL:
http://rstrode.fedorapeople.org/bluecurve-kdm-theme/checkout/bluecurve-kdm-theme.spec

SRPM URL:
http://rstrode.fedorapeople.org/bluecurve-kdm-theme/checkout/bluecurve-kdm-theme-1.0.0-1.fc8.src.rpm

Description:
This package contains the Bluecurve kdm theme.

-- 
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 294641] Review Request: aboot - A bootloader which can be started from the SRM console.

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

Summary: Review Request: aboot - A bootloader which can be started from the SRM 
console.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|4Suite  |Package Review




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 11:28 EST ---
Sometimes the component gets changed randomly.  I don't know if this is a
firefox bug or a bugzilla bug.  It's no big deal; I just change it back when I
have something else to add.

-- 
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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools

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

Summary: Review Request: scsi-target-utils  - SCSI target daemon and tools


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 11:51 EST ---
What is going on here? It's over two months since the package was approved and
I can't find anything in CVS.

There are some bugs too:

 o license policy has changed since review:
 http://fedoraproject.org/wiki/Licensing
   License should be changed to GPLv2

 o build not honor compiler flags:
 http://tinyurl.com/2n3qqh

 o should build latest release:
   new release from 2007-08-03 available

I have created an update package fixing these bugs, it's here:

SPEC: http://terjeros.fedorapeople.org/scsi-target-utils/scsi-target-utils.spec
SRPM:
http://terjeros.fedorapeople.org/scsi-target-utils/scsi-target-utils-0.0-0.20070803snap.fc7.src.rpm


Do you still want to maintain this package for Fedora?


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

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


[Bug 305561] Review Request: bluecurve-icon-theme - Bluecurve icon theme

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

Summary: Review Request: bluecurve-icon-theme - Bluecurve icon theme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 11:59 EST ---
Almost.

Just fix the license for real before you import it. 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 244936] Review Request: dbench - Filesystem load benchmarking tool

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

Summary: Review Request: dbench - Filesystem load benchmarking tool


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 11:57 EST ---

Have pushed an update. Closing

-- 
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 305581] Review Request: bluecurve-kde-theme - Bluecurve KDE theme

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

Summary: Review Request: bluecurve-kde-theme - Bluecurve KDE theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:01 EST ---
It needs kdebase-devel because configure complains if it is not there...
Just mock-build it and see.

-- 
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 286691] Review Request: libgssglue - renaming libgssapi to libgssglue

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

Summary: Review Request: libgssglue - renaming libgssapi to libgssglue


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 11:58 EST ---

 * But that doesn't mean you should not increment release tag. And for above
 change, do you think its so small to previous SPEC you provided?
I did, I went from -1 to -2 and group all the comments under
the RPM review changelog which I think *very* appropriate. 
Why artificially increase version numbers for basically the 
same bug (or process in this case). Also, Isn't  up the 
the discretion of the maintainer to decide when and
why to increase a version number? I would surely hope so.

 No. You must update release tag and provide new links for final review.
No. I feel the way I'm managing this tag for my package is just
fine. Now if so choose not to let this package into fedora due to
silly issue like tag number (after working on this for 3 weeks)
so be it... 

 *Remember to run rpmlint on all SRPM,RPM files.
I did and I see no errors. 



-- 
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 305631] Review Request: bluecurve-qt-engine - Bluecurve qt engine

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

Summary: Review Request: bluecurve-qt-engine - Bluecurve qt engine


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:04 EST ---
the configure script complains if it is not there...

-- 
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 305621] Review Request: bluecurve-metacity-theme - Bluecurve metacity theme

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

Summary: Review Request: bluecurve-metacity-theme - Bluecurve metacity theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:03 EST ---
but then you need to make this package require bluecurve-gnome-theme ?

-- 
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 275231] Review Request: php-channel-phing - Adds phing channel to PEAR

2007-09-26 Thread bugzilla
Please 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-channel-phing - Adds phing channel to PEAR


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:40 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 275211] Review Request: php-channel-phpdb - Adds phpdb channel to PEAR

2007-09-26 Thread bugzilla
Please 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-channel-phpdb - Adds phpdb channel to PEAR


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:37 EST ---
cvs done. 

There isn't really any procedure to review a seperate spec for el branches. 
Perhaps you could do some testing and get it all working for epel and then
request branches? 


-- 
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 248617] Review Request: cdcollect - Simple CD/DVD catalog for GNOME

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

Summary: Review Request: cdcollect - Simple CD/DVD catalog for GNOME


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:44 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 230164] Review Request: rt73usb-firmwa re - Firmware for Ralink® RT2571W/RT2671 A/B/G network adaptors

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

Summary: Review Request: rt73usb-firmware - Firmware for Ralink® RT2571W/RT2671 
A/B/G network adaptors


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 12:42 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 305621] Review Request: bluecurve-metacity-theme - Bluecurve metacity theme

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

Summary: Review Request: bluecurve-metacity-theme - Bluecurve metacity theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 13:09 EST ---
i don't know, aside from the dir ownership issue, it makes more sense for
bluecurve-gnome-theme to require bluecurve-metacity-theme, not the other way
around...

-- 
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 294641] Review Request: aboot - A bootloader which can be started from the SRM console.

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

Summary: Review Request: aboot - A bootloader which can be started from the SRM 
console.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 13:14 EST ---
A few complaints:

the manpages are executable, which rpmlint dutifully complains about:
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/e2writeboot.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/sdisklabel.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man1/isomarkboot.1.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/swriteboot.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/abootconf.8.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man1/netabootwrap.1.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man5/aboot.conf.5.gz
  aboot.alpha: W: spurious-executable-perm /usr/share/man/man8/aboot.8.gz

I notice that the normal set of compiler flags aren't used.  Now, this is a
bootloader so I can understand why, although there are userspace programs
included which perhaps should be built like any other userspace program.  Given
that this is for Alpha, though, I can't even be truly sure what the proper
compilation flags are.

This package does not meet the versioning guidelines; when 1.0 is released, it
will sort lower than the current package name.  The guidelines specify the
proper version and release to be used as:
  1.0-0.2.pre20040408
You can increment the '2' for each new revision, and when 1.0 is released you
can just use 1.0-1 without worrying about any sorting issues.

There's a COPYING file in the tarball, which must be included in the package.

* source files match upstream:
   a8ae8f2bf549c1cc79ea66a0a11c8db5c0257ce0d94b97418eb1c658723b12d2  
   aboot-1.0_pre20040408.tar.bz2
X package does not meet versioning guidelines.
* specfile is properly named, is cleanly written and uses macros consistently.
* summary is OK.
* description is OK.
* dist tag is present.
* build root is OK.
* license field matches the actual license.
* license is open source-compatible.
X license text included in tarball but not in package.
* latest version is being packaged.
* BuildRequires are proper.
? compiler flags are appropriate.
* %clean is present.
* package builds in mock (development, alpha).
* debuginfo package looks complete.
X rpmlint has valid complaints.
* final provides and requires are sane.
* %check is not present; no test suite upstream.  I have no way to test this 
  pacakge.
* no shared libraries are added to the regular linker search paths.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
X file permissions are not appropriate (executable manpages)
* no scriptlets present.
* code, not content.
* documentation is small, so no -docs subpackage is necessary.
* %docs are not necessary for the proper functioning of the package.
* no headers.
* no pkgconfig files.
* no static libraries.
* no libtool .la files.

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

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


[Bug 307601] New: Review Request: libHX - General-purpose library (needed by recent pam_mount)

2007-09-26 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=307601

   Summary: Review Request: libHX - General-purpose library (needed
by recent pam_mount)
   Product: Fedora
   Version: devel
  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://till.fedorapeople.org/review/libHX.spec
SRPM URL: http://till.fedorapeople.org/review/libHX-1.10.1-1.fc7.src.rpm
Description:

A library for:
- A+R/B trees to use for lists or maps (associative arrays)
- Deques (double-ended queues) (Stacks (LIFO) / Queues (FIFOs))
- platform independent opendir-style directory access
- platform independent dlopen-style shared library access
- auto-storage strings with direct access
- command line option (argv) parser
- shell-style config file parser
- platform independent random number generator with transparent
  /dev/urandom support
- various string, memory and zvec ops

-- 
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 307601] Review Request: libHX - General-purpose library (needed by recent pam_mount)

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

Summary: Review Request: libHX - General-purpose library (needed by recent 
pam_mount)
Alias: libHX

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||libHX
URL||http://jengelh.hopto.org/f/l
   ||ibHX/




-- 
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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools

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

Summary: Review Request: scsi-target-utils  - SCSI target daemon and tools


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 14:28 EST ---
My fault. I must have just done the RHEL sources and forgot to do FC for some
reason.

If you want to maintain it or help, that is fine with 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 173054] Review Request: wavpack - completely open audiocodec

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

Summary: Review Request: wavpack - completely open audiocodec


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC|fedora-extras-  |
   |[EMAIL PROTECTED] |
 CC||fedora-package-
   ||[EMAIL PROTECTED]
 CC|fedora-package- |
   |[EMAIL PROTECTED]   |
 CC||fedora-extras-
   ||[EMAIL PROTECTED]
   Severity|normal  |medium
   Priority|normal  |medium
Product|Fedora Extras   |Fedora

[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 15:00 EST ---
Package Change Request
==
Package Name: wavpack
New Branches: EL-4 EL-5

-- 
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 307791] New: Review Request: twitux - a Twitter client for the Gnome desktop

2007-09-26 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=307791

   Summary: Review Request: twitux - a Twitter client for the Gnome
desktop
   Product: Fedora
   Version: devel
  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://icon.fedorapeople.org/f/twitux.spec
SRPM URL: http://icon.fedorapeople.org/f/twitux-0.50-1.fc7.src.rpm
Description:
Twitux is a Twitter client for the Gnome 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 305441] redhat-artwork should be split up into multiple packages

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

Summary: redhat-artwork should be split up into multiple packages
Alias: ArtworkSplit

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||307801




-- 
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 307801] New: Review Request: bluecurve-gnome-theme - Bluecurve GNOME theme

2007-09-26 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=307801

   Summary: Review Request: bluecurve-gnome-theme - Bluecurve GNOME
theme
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: low
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL:
http://rstrode.fedorapeople.org/bluecurve-gnome-theme/checkout/bluecurve-gnome-theme.spec

SRPM URL:
http://rstrode.fedorapeople.org/bluecurve-gnome-theme/checkout/bluecurve-gnome-theme-1.0.0-1.fc8.src.rpm

Description:
This package contains the Bluecurve GNOME meta theme.

-- 
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 307821] New: Review Request: main package name here - short summary here

2007-09-26 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=307821

   Summary: Review Request: main package name here - short
summary here
   Product: Fedora
   Version: devel
  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://ccrma.stanford.edu/planetccrma/extras/sooperlooper.spec
SRPM URL: 
http://ccrma.stanford.edu/planetccrma/extras/sooperlooper-1.2.0-1.src.rpm

Description: 
SooperLooper is a realtime software looping sampler in the spirit of
Gibson's Echoplex Digital Pro. If used with a low-latency kernel and
the proper audio buffer configuration it is capable of truly realtime
live looping performance.

(was https://bugzilla.redhat.com/show_bug.cgi?id=200941)

-- 
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 307891] New: libvpd - C++ library for system vpd access

2007-09-26 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=307891

   Summary: libvpd - C++ library for system vpd access
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://superb-west.dl.sourceforge.net/sourceforge/linux-diag/libvpd.spec
SRPM URL: 
http://superb-west.dl.sourceforge.net/sourceforge/linux-diag/libvpd-1.3.4-1.src.rpm
Description: A C++ Library used for accessing information collected by the 
program lsvpd.  The lsvpd package also uses this library for storing the system 
vpd collection.

A request to add lsvpd will follow.

-- 
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 307901] New: Review Request: lsvpd - A utility to list device Vital Product Data (VPD) information.

2007-09-26 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=307901

   Summary: Review Request: lsvpd - A utility to list device Vital
Product Data (VPD) information.
   Product: Fedora
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: [EMAIL PROTECTED]
CC: fedora-package-review@redhat.com,[EMAIL PROTECTED]


Spec URL: 
http://superb-west.dl.sourceforge.net/sourceforge/linux-diag/lsvpd.spec
SRPM URL: 
http://superb-west.dl.sourceforge.net/sourceforge/linux-diag/lsvpd-1.3.4-1.src.rpm
Description: This utility lists device Vital Product Data (VPD), which includes 
the following information and more: vendor, version, revision level, serial 
number. This utility has been tested on ppc64, ppc, and i386 platforms.

-- 
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 244948] Review Request: perl-Config-Std - Yet Another Way of Storing Configuration Files

2007-09-26 Thread bugzilla
Please 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-Config-Std - Yet Another Way of Storing 
Configuration Files


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 15:52 EST ---
You are still missing build dependencies on perl(Test::Pod) and
perl(Test::Pod::Coverage) so you miss some test suite coverage; this changes the
test results to:
  All tests successful, 2 tests skipped.
  Files=11, Tests=29,  0 wallclock secs ( 0.42 cusr +  0.15 csys =  0.57 CPU)
It's good to have full test coverage but you don't seem to want to add those two
bbuild dependencies for whatever reason so I won't block on it.

The license field is fixed, so rpmlint is quiet.  The following issue remains:
X package version is not compliant with guidelines.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, 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 245708] Review Request: scsi-target-utils - SCSI target daemon and tools

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

Summary: Review Request: scsi-target-utils  - SCSI target daemon and tools


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 16:01 EST ---
Ok, I can take it.

I did a updated package with some style changes, can notting have a quick look
and (hopefully) ack?

SPEC: http://terjeros.fedorapeople.org/scsi-target-utils/scsi-target-utils.spec
SRPM:
http://terjeros.fedorapeople.org/scsi-target-utils/scsi-target-utils-0.0-1.20070803snap.fc7.src.rpm

You then have to set fedora-cvs to ? and say something like this:

Package Change Request
==
Package Name: scsi-target-utils 
Change Owner To: terjeros

Thanks for the nice work and quick reply!

-- 
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 304301] Review Request: pysol - A Python Solitaire game collection

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

Summary: Review Request: pysol -  A Python Solitaire game collection


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 16:11 EST ---
This builds fine and rpmlint is clean; I'll run through the checklist as soon as
I get home this evening.  But unfortunately it doesn't run:

ImportError: No module named pysolsoundserver

This happens even if I pass --nosound.  It looks like there's code to detect
that the module isn't there (look for warn_pysolsoundserver) but the import
isn't wrapped in a try block.  When I get home I'll have to see if I can't hack
it a bit.  Or this could grow a dependency on the sound server, except that it
would have to be a circular dependency, and in that case they should probably
just be bundled together in one 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 248617] Review Request: cdcollect - Simple CD/DVD catalog for GNOME

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

Summary: Review Request: cdcollect - Simple CD/DVD catalog for GNOME


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 16:33 EST ---
imported and build
set ExcludeArch: ppc64 as Mono doesn't exist there

-- 
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 307791] Review Request: twitux - a Twitter client for the Gnome desktop

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

Summary: Review Request: twitux - a Twitter client for the Gnome desktop


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 16:52 EST ---
Do you really need the %post and %postun update-desktop-database?

-- 
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 305531] Review Request: bluecurve-gdm-theme - Bluecurve GDM theme

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

Summary: Review Request: bluecurve-gdm-theme - Bluecurve GDM theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:02 EST ---
I made the change suggested in comment 5, fwiw (and the similar change for the
kdm theme)

-- 
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 307821] Review Request: sooperlooper - Realtime software looping sampler

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

Summary: Review Request: sooperlooper - Realtime software looping sampler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:01 EST ---
GPL is not a valid value for the License-Tag, see:
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines#head-f21ae23bf2f278444e2c385463cfa74a502396b8

Afaik, these categories in the desktop file should not be used:
X-Fedora Application

And I guess these, too:
X-PlanetCCRMA
X-Digital_Processing
X-Jack

Maybe you want to add this category:
Audio

-- 
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 305581] Review Request: bluecurve-kde-theme - Bluecurve KDE theme

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

Summary: Review Request: bluecurve-kde-theme - Bluecurve KDE theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:07 EST ---
or if we merge this with bluecurve-qt-engine, then this issue becomes moot.  I'm
still curious for opinions on 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 305581] Review Request: bluecurve-kde-theme - Bluecurve KDE theme

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

Summary: Review Request: bluecurve-kde-theme - Bluecurve KDE theme


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:06 EST ---
that's because configure has:

KDE_SET_PREFIX
KDE_CHECK_FINAL
KDE_USE_QT
AC_PATH_KDE

to pull in $(kde_datadir)

I don't know what those macros, I just copied it verbatim from redhat-artwork. 
For now i'll add the BuildRequires, but we might be able to trim that and drop
the BuildReq

-- 
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 244948] Review Request: perl-Config-Std - Yet Another Way of Storing Configuration Files

2007-09-26 Thread bugzilla
Please 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-Config-Std - Yet Another Way of Storing 
Configuration Files


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:11 EST ---
I've added a dependency on Test::Pod but not on Test::Pod::Coverage, as it built
without, and made it rpmlint-clean, which I believe brings package version into
compliance with guidelines.

SPEC:  http://fetter.org/rpm/perl-Config-Std.spec
SRPM:  http://fetter.org/rpm/perl-Config-Std-v0.0.4-5.fc7.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 302351] Review Request: higlayout - Easy to use and powerful layout manager for Java

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

Summary: Review Request: higlayout - Easy to use and powerful layout manager 
for Java


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:14 EST ---
New Package CVS Request
===
Package Name:  higlayout
Short Description: Easy to use and powerful layout manager for Java
Owners:jwrdegoede
Branches:  F-7 devel
InitialCC: empty
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 307791] Review Request: twitux - a Twitter client for the Gnome desktop

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

Summary: Review Request: twitux - a Twitter client for the Gnome desktop


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:15 EST ---
Created an attachment (id=207541)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=207541action=view)
Patches spelling of Connecting


-- 
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 307791] Review Request: twitux - a Twitter client for the Gnome desktop

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

Summary: Review Request: twitux - a Twitter client for the Gnome desktop


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |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 302351] Review Request: higlayout - Easy to use and powerful layout manager for Java

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

Summary: Review Request: higlayout - Easy to use and powerful layout manager 
for Java


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:12 EST ---
(In reply to comment #1)
 BR: java-devel should be replaced by BR: java-javadoc (even if sinjdoc
 doesn't do crosslinked javadocs at the moment) - java-gcj-compat-devel 
 provides
 java-devel.  Otherwise looks good to me, feel free to import and fix before 
 the
 first build.

Nothing provides java-javadoc, thats why I put java-devel there, but you are
right, that is the same as java-gcj-compat-devel. So I'll take that out before
import.

Thanks for the review!

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

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


[Bug 307791] Review Request: twitux - a Twitter client for the Gnome desktop

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

Summary: Review Request: twitux - a Twitter client for the Gnome desktop


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





--- Additional Comments From [EMAIL PROTECTED]  2007-09-26 17:14 EST ---
APPROVED, with following notes:
- Dependency on glib2-devel unnecessary (pulled in by libsexy-devel)
- Requires(post), Requires(postun), %post and %postun also unnecessary
- Connecting is misspelled in upstream code and translation, patch attached.  
  Please apply and forward to upstream.

MUST

• rpmlint: OK
• package name: OK
• spec file name: OK
• package guideline-compliant: OK
• license complies with guidelines: OK
• license field accurate: OK
• license file not deleted: OK
• spec in US English: OK
• spec legible: OK
• source matches upstream: OK
• builds under = 1 archs, others excluded: OK
• build dependencies complete: OK*
• locales handled using %find_lang: OK
• no dupes in %files: OK
• permission: OK
• %clean RPM_BUILD_ROOT: OK
• macros used consistently: OK
• Package contains code: OK
• desktop file uses desktop-file-install: OK*
• clean buildroot before install: OK
• filenames UTF-8: OK

SHOULD
• package build in mock on all architectures
• package functioned as described: OK
• require package not files: OK

-- 
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 305761] Review Request: fedorainfinity-screensaver-theme - Fedora Infinity screensaver theme

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

Summary: Review Request: fedorainfinity-screensaver-theme - Fedora Infinity 
screensaver theme


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




-- 
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


  1   2   >