[Bug 206836] Review Request: TurboJson - Python template plugin that supports json

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

Summary: Review Request: TurboJson - Python template plugin that supports json


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Package Review  |osgal




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:57 EST ---
(In reply to comment #4)
> - source files match do not upstream: You can no longer download 0.9.9, 
> they're
> released 1.0 beta.
> - latest version is not being packaged (see above)

1.0b is the current TurboGears release; it looks like TurboJson is still at
0.9.9 as far as I can tell.

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

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


[Bug 200499] Review Request:

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

Summary: Review Request: 


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:51 EST ---
If you are still willing to submit this package, upload new version of this
package. I got following rpmlint output

Review on this package =>

On SRPM i got
W: btsco-kmod-common mixed-use-of-spaces-and-tabs (spaces: line 2, tab: line 1)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic annoyance.  Use either spaces or tabs for indentation, not both.

=>USE sed -i -e 's|\t| |g' on SPEC file

on RPM i got
W: btsco-kmod-common incoherent-version-in-changelog 0.42.fc5.2 0.42-2.fc6
The last entry in %changelog contains a version identifier that is not
coherent with the epoch:version-release tuple of the package.

=> update Changelog entry 

E: btsco-kmod-common zero-length /usr/share/doc/btsco-kmod-common-0.42/NEWS
E: btsco-kmod-common zero-length /usr/share/doc/btsco-kmod-common-0.42/ChangeLog
E: btsco-kmod-common zero-length /usr/share/doc/btsco-kmod-common-0.42/AUTHORS

=>Remove files in SPEC if they are zero-length files

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

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


[Bug 206837] Review Request: TurboCheetah - TurboGears plugin to support use of Cheetah templates

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

Summary: Review Request: TurboCheetah - TurboGears plugin to support use of 
Cheetah templates


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Package Review  |orpie




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:47 EST ---
http://people.redhat.com/lmacken/SPECS/python-turbocheetah.spec
http://people.redhat.com/lmacken/RPMS/python-turbocheetah-0.9.5-2.src.rpm

* Sat Sep 30 2006 Luke Macken <[EMAIL PROTECTED]> - 0.9.5-2
- Rename to python-turbocheetah
- Install egg-info
- Add README

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

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


[Bug 206838] Review Request: TGFastData - Automatic user interface generation for TurboGears

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

Summary: Review Request: TGFastData - Automatic user interface generation for 
TurboGears


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Component|Package Review  |orpie




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:39 EST ---
http://people.redhat.com/lmacken/SPECS/python-tgfastdata.spec
http://people.redhat.com/lmacken/RPMS/python-tgfastdata-0.9a6-2.src.rpm

* Sat Sep 30 2006 Luke Macken <[EMAIL PROTECTED]> - 0.9a6-2
- Rename to python-tgfastdata
- Install egg-info

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

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


[Bug 191743] Review Request: sysprof - a sampling CPU profiler

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

Summary: Review Request: sysprof -  a sampling CPU profiler


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:38 EST ---
Anyway i tried with latest uploaded SRPM link. Mock build is failed with error
checking for cplus_demangle_opname in -liberty... no
configure: error: libiberty is required to compile sysprof
error: Bad exit status from /var/tmp/rpm-tmp.63043 (%build)


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

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


[Bug 206839] Review Request: TurboKid - Python template plugin that supports Kid templates

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

Summary: Review Request: TurboKid - Python template plugin that supports Kid 
templates


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE
  Component|Package Review  |orpie
OtherBugsDependingO|163779  |163778
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:27 EST ---
Imported into CVS, added to owners.list, requested FC-5 branch, and bumped to 
0.9.9.

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

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


[Bug 191743] Review Request: sysprof - a sampling CPU profiler

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

Summary: Review Request: sysprof -  a sampling CPU profiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:23 EST ---
I suggest you to change SPEC name to sysprof-kmod-common.spec and resubmit
package. This is according to kernel module packaging guidelines. Thoug its
given that userlan package should 
Provides: %{name}- kmod-common = %{version}
it will be good idea to have similar name given to SPEC also.


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

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


[Bug 191745] Review Request: sysprof-kmod - kernel module for the sysprof profiler

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

Summary: Review Request: sysprof-kmod - kernel module for the sysprof profiler


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:17 EST ---
I am not able to find sysprof-kmod-common package. When i tried to install
sysprof-kmod, i got following output.

error: Failed dependencies: 
sysprof-kmod-common = 1.0.3 is needed by
kmod-sysprof-1.0.3-1.2.6.18_1.2699.fc6.i686


In this case i suggest you to modify kmodtool to remove dependency of
sysprof-kmod-common for sysprof-kmod and also ask same on Fedora-extras mailing
list.

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

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


[Bug 191745] Review Request: sysprof-kmod - kernel module for the sysprof profiler

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

Summary: Review Request: sysprof-kmod - kernel module for the sysprof profiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 02:10 EST ---
I think kernels above 2.6.16 do not require any smp kernels so either SPEC need
to  have some kversion checking or remove smp build lines.
Here is Review for your package which i built for 2.6.18 which do not have smp
support so i remove those lines form SPEC

Package Built successfully for i386 FC6 development
Rpmlint on SRPM
W: sysprof-kmod strange-permission kmodtool 0666
A file that you listed to include in your package has strange
permissions. Usually, a file should have 0644 permissions.
=> This is well known rpmlint Warning on kmod packages. Forget it.

E: sysprof-kmod configure-without-libdir-spec
A configure script is run without specifying the libdir. configure
options must be augmented with something like --libdir=%{_libdir}.

I am not able to found any ./configure call in SPEC. Why this Error is coming 
Gianluca, any idea?


Rpmlint on RPM
W: kmod-sysprof summary-not-capitalized sysprof kernel module(s)
Summary doesn't begin with a capital letter.
=> This is well known rpmlint Warning on kmod packages. Forget it.

W: kmod-sysprof unstripped-binary-or-object
/lib/modules/2.6.18-1.2699.fc6/extra/sysprof/sysprof-module.ko
=> This is well known rpmlint Warning on kmod packages. Forget it.

W: kmod-sysprof no-documentation
The package contains no documentation (README, doc, etc).
You have to include documentation files.
=>Ignore this


Otherwise package looks ok.


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

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


[Bug 206707] Review Request: acerhk-kmod - Acer Hotkey driver

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

Summary: Review Request: acerhk-kmod - Acer Hotkey driver


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 01:58 EST ---
It looks to me that this package is not building on i386 FC6 and hanging in
setup precedure of mock build. Can you mock build your package and give its
results here.

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

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


[Bug 191745] Review Request: sysprof-kmod - kernel module for the sysprof profiler

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

Summary: Review Request: sysprof-kmod - kernel module for the sysprof profiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 01:18 EST ---
Will Do Full Review today

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

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


[Bug 208680] New: Review Request: ser2net - Proxy that allows tcp connections to serial ports

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




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

   Summary: Review Request: ser2net - Proxy that allows tcp
connections to serial ports
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://www.auroralinux.org/people/spot/review/ser2net.spec
SRPM URL: 
http://www.auroralinux.org/people/spot/review/ser2net-2.3-1.fc6.src.rpm
Description: 
ser2net provides a way for a user to connect from a network connection to a
serial port. It provides all the serial port setup, a configuration file to
configure the ports, a control login for modifying port parameters,
monitoring ports, and controlling ports.

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

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


[Bug 201077] gfs-kmod

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

Summary: gfs-kmod


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 01:09 EST ---
SRPM is also NOT rpmlint Silent
W: gfs-kmod strange-permission kmodtool 0755
A file that you listed to include in your package has strange
permissions. Usually, a file should have 0644 permissions.

=> This is well known rpmlint Warning on kmod packages. Forget it.

E: gfs-kmod invalid-spec-name gfs-kernel.spec
Your spec filename must end with '.spec'. If it's not the case, rename your
file and rebuild your package.

=> You Need to change SPEC file name from gfs-kernel.spec to gfs-kmod.spec

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

=> USE sed -i -e 's|\t| |g' gfs-kernel.spec


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

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


[Bug 201077] gfs-kmod

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

Summary: gfs-kmod


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 01:00 EST ---
I mockbuild this package on FC6 devel.  It got built successfully.
rpmlint is NOT silent
I: kmod-gfs checking
W: kmod-gfs summary-not-capitalized gfs kernel module(s)
Summary doesn't begin with a capital letter.

=>This is well known rpmlint Warning. Forget it.

W: kmod-gfs no-version-in-last-changelog
The last changelog entry doesn't contain a version. Please insert the
version that is coherent with the version of the package and rebuild it.

=> You miss to add changelong entry

W: kmod-gfs unstripped-binary-or-object
/lib/modules/2.6.18-1.2699.fc6/extra/gfs/gfs.ko

=>This is well known rpmlint Warning. Forget it.

W: kmod-gfs no-documentation
The package contains no documentation (README, doc, etc).
You have to include documentation files.

=>Not harmfull.

Everything looks OK. Update Chnagelog and give new package locations.

If nobody is willing to Sponsor this package I can do a Full Review and APPROVE
package provided FESCO is not having any problem to allow this kmod in FE.

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

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


[Bug 208436] Review Request: kphotobymail - Kphotobymail is a PyQT based application for uploading photos to flickr account.

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

Summary: Review Request: kphotobymail - Kphotobymail is a PyQT based 
application for uploading photos to flickr account.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 208678] Review Request: SimGear - Simulation library components

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

Summary: Review Request: SimGear - Simulation library components


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||208679
  nThis||




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

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


[Bug 208679] Review Request: vamos - Automotive simulation framework

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

Summary: Review Request: vamos - Automotive simulation framework


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||208678




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

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


[Bug 208679] New: Review Request: vamos - Automotive simulation framework

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




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

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


Spec URL: http://www.auroralinux.org/people/spot/review/vamos.spec
SRPM URL: 
http://www.auroralinux.org/people/spot/review/vamos-0.5.5-1.fc6.src.rpm   
Description: 
Vamos is an automotive simulation framework with an emphasis on thorough
physical modeling and good C++ design. Vamos includes a real-time,
first-person, 3D driving application.

It also includes caelum, which is a program to help make seamless sky boxes 
from panoramic photos. The image file is mapped onto a sphere and six views are 
shown. The views are from the center of the sphere looking up (top row), down 
(bottom row), front, right, back, and left (middle row). When these images are 
mapped to the faces of a cube and viewed from the center of the cube, the view 
in any direction is indistinguishable from the view from the center of the 
sphere. The images for the sky box are extracted by taking a screenshot.

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

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


[Bug 208678] New: Review Request: SimGear - Simulation library components

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




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

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


Spec URL: http://www.auroralinux.org/people/spot/review/SimGear.spec
SRPM URL: 
http://www.auroralinux.org/people/spot/review/SimGear-0.3.10-1.fc6.src.rpm
Description: 
SimGear is a set of open-source libraries designed to be used as building
blocks for quickly assembling 3d simulations, games, and visualization
applications.

This library is used in several open source simulation games, most notably, 
vamos and flightgear. Accordingly, this is a pre-requisite for those games to 
be packaged in Fedora.

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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

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

Summary: Review Request: gutenprint: Printer Drivers Package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 00:27 EST ---
I got some issues in package build i gave following command
plague-client build gutenprint gutenprint-5.0.0-0.14.fc6 fc6


Got results as 
Error: could not check out gutenprint-5.0.0-0.14.fc6 from
fedora-development-extras - output was:

cvs checkout: warning: cannot open /cvs/extras/CVSROOT/val-tags read/write:
Permission denied
cvs [checkout aborted]: no such tag gutenprint-5.0.0-0.14.fc6


What i missed?


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

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


[Bug 205075] Review Request: fwbackups - a user backup program, with support for automated backups and on-demand backups

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

Summary: Review Request: fwbackups - a user backup program, with support for 
automated backups and on-demand backups


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-30 00:20 EST ---
Sorry for taking so long to respond; work intruded a bit into my evenings.

Firstly, the Source0: URL is not valud; it looks like you need to downcase
"FWBackups".  However, after doing this I find that again the files are not the
same.  What's in the srpm needs to match precisely what is downloaded from the
web site.

Also, for your sanity I recommend something like:

Source0: http://www.diffingo.com/downloads/fwbackups/fwbackups-%{version}.tar.gz

so that you only have the update the version in one place when you update the
package.

Aside from that, you should clean up the commented lines in %install (and the
commented Provides as well).  And I wonder what you have your tabs set for,
since the indentation is all over the place on my screen.

The package installs fine and seems to work for me on FC5.  (I did not test
actually running a backup.)

So really it's down to the tarball actually matching what's at the upstream web
site, and a few minor specfile adjustments.

Review:
X source files match upstream.
* package meets naming and packaging guidelines.
X specfile is properly named, is cleanly written and uses macros consistently
(needs minor cleanups).
* dist tag is present.
* build root is correct.
* license field matches the actual license.
* license is open source-compatible.  License text included in package.
* latest version is being packaged.
* BuildRequires are proper.
* %clean is present.
* package builds in mock (development & FC5, x86_64 & i386).
* package installs properly
* package runs OK.  (I only did some basic tests and didn't actually run a 
backup.)
* rpmlint is silent.
* final provides and requires are sane:
   config(fwbackups) = 1.42.1-1.fc6
   fwbackups = 1.42.1-1.fc6
  =
   /bin/bash
   /usr/bin/python
   config(fwbackups) = 1.42.1-1.fc6
   pygtk2
   pygtk2-libglade
   python(abi) = 2.4
   redhat-artwork
* %check is not present; no test suite upstream.
* owns the directories it creates.
* doesn't own any directories it shouldn't.
* no duplicates in %files.
* file permissions are appropriate.
* 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.
* GUI app; .desktop files are supplied and installed properly.  No mimetype
keys, so no need to run update-mime-database. 

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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

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

Summary: Review Request: gutenprint: Printer Drivers Package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 23:51 EST ---
(In reply to comment #81)
> Parag: You'll need to setup your install the client-side tools [...]

That should read "...need to install and setup the client-side tools..."

Apologies for any confusion.



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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

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

Summary: Review Request: gutenprint: Printer Drivers Package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 23:44 EST ---
Parag: You'll need to setup your install the client-side tools and whatnot for
the build system, the import the package and tag then enqueue a build for it.
For more information please see:
http://fedoraproject.org/wiki/Extras/Contributors#head-0956b12959af46cfe0aa12d09ed15e573bfd9ef4

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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

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

Summary: Review Request: gutenprint: Printer Drivers Package


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 23:35 EST ---
Kevin,
 Thnaks very much for your time and sponsoring this package. I have imported
package. Do I need to build it on build server? If yes how can i do that?

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

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


[Bug 208675] New: Review Request: miau - Full-featured IRC bouncer

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




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

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


Spec URL: http://beer.tclug.org/fedora-extras/miau/miau.spec
SRPM URL: http://beer.tclug.org/fedora-extras/miau/miau-0.6.2-2.src.rpm
Description: 
miau is a smart and versatile IRC-bouncing tool for *NIX. The difference
between miau and other bouncers is that miau will go on IRC as soon as
it is launched, guarding or attempting to get your nick. Control over
the session can be taken as with other bouncers, by simply connecting to
miau (and providing a password) like you would connect to a normal IRC
server. On disconnect, miau is able to stay in the channels and to
reintroduce them to your client on your next connect. Other handy
features are message logging, flood protection, and DCC bouncing.

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

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


[Bug 207763] Review Request: ddd - GUI for several command-line debuggers

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

Summary: Review Request: ddd - GUI for several command-line debuggers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 21:54 EST ---
Built, thanks for the reviewing.

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

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


[Bug 207806] Review Request: evolution-bogofilter - A plugin for bogofilter support in evolution

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

Summary: Review Request: evolution-bogofilter -  A plugin for bogofilter 
support in evolution


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 21:04 EST ---
All the issues above fixed.

New SRPM:
http://www.auroralinux.org/people/spot/review/evolution-bogofilter-0.2.0-3.fc6.src.rpm
New SPEC: 
http://www.auroralinux.org/people/spot/review/evolution-bogofilter.spec

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

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


[Bug 208208] Review Request: MegaMek - a portable, network-enabled BattleTech engine

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

Summary: Review Request: MegaMek - a portable, network-enabled BattleTech engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 20:42 EST ---
(In reply to comment #7)
> OK, add this line to classpath.security:
> 
> securerandom.source=file:/dev/random
> 
> and re-run.

Same deal.


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

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


[Bug 191743] Review Request: sysprof - a sampling CPU profiler

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

Summary: Review Request: sysprof -  a sampling CPU profiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|177841  |
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 19:12 EST ---
New package addressing comment #1 (and comment #8...)

http://giallu.homelinux.org/fedora/sysprof.spec
http://giallu.homelinux.org/fedora/sysprof-1.0.3-2.src.rpm

Please note I have been sponsored, so now it only need an official review from
another regular contributor to have this piece of software in Fedora.


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

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


[Bug 198816] Review Request: gfs-utils - global file system userland utilities

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

Summary: Review Request: gfs-utils - global file system userland utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]
OtherBugsDependingO||177841
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 18:01 EST ---
I can, but would need to look over this and your other submissions first.  I
hope to have some time to spend on extras reviews over the next couple of days.

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

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


[Bug 206989] Review Request: twinkle - A SIP Soft Phone

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

Summary: Review Request: twinkle - A SIP Soft Phone


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 16:28 EST ---
Using my own review checklist:
http://beer.tclug.org/fedora-extras/review-checklist-1.1.txt

1. No rpmlint output from the main or source RPM.  However, from the
debuginfo, I got:
W: twinkle-debuginfo hidden-file-or-dir 
/usr/src/debug/twinkle-0.8.1/src/gui/.moc
W: twinkle-debuginfo hidden-file-or-dir 
/usr/src/debug/twinkle-0.8.1/src/gui/.moc
W: twinkle-debuginfo hidden-file-or-dir /usr/src/debug/twinkle-0.8.1/src/gui/.ui
W: twinkle-debuginfo hidden-file-or-dir /usr/src/debug/twinkle-0.8.1/src/gui/.ui

I have no idea how important that is.

2. Package seems to adhere to the Package Naming Guidelines.
3. Spec is %{name}.spec
4. Package appears to meet the Packaging Guidelines.
5. Package is GPL...
6. ...and the spec agrees.
7. COPYING included in %doc
8. Spec is in American English.
9. Spec is legible.
10. Tarball MD5 matches upstream (245ffd3b4a6f968d8f80d42f6aad079c).
11. Packages build on FC5/i386, FC5/ppc, and FC6/i386.
12. n/a, unless it fails to build on x86_64.
13. Package built in Plague, so if there are any missing BRs, they're non-fatal
(and might silently corrupt this package!)
14. Package makes no effort to handle locales.  Oh well.
15. n/a, no shared libraries.
16. Package is not designed to be relocatable.  Are any?
17. Package owns all directories it creates (aside from those owned by 
filesystem).
18. No duplicate files.
19. Permissions look sane, and there's a %defattr entry.
20. Spec has a valid %clean section.
21. Macro use consistent.
22. Package contains code and (I believe) permissable content.
23. Documentation is minimal enough...
24. ...and shouldn't affect runtime.
25. n/a, no header files or static libraries.
26. n/a, no .pc files.
27. n/a, no library files.
28. n/a, no -devel subpackage.
29. n/a, no .la files.
30. Desktop file appears to get installed in the prescribed manner.
31. Nothing else owns anything with the name "twinkle," so I think we're safe.
32. Release tag contains %{?dist}, unsurprisingly.
33. n/a, already contains COPYING.
34. n/a, translations probably not available.
35. Package builds in Plague/Mock.
36. I can't verify x86_64, but it builds fine on i386 and ppc.  Kevin said it
built under x86_64, though, so we should be good there.
37. Works beautifully under i386.  (I don't have any other test environments.)
38. n/a, no scriptlets.
39. n/a, no subpackages.

Sorry this review took as long as it did; I actually wanted to test the
application to see how well it would work for my own use.  It communicated just
fine with another (closed-source) SIP softphone.

Unless the rpmlint/debuginfo warnings are of particular concern (anyone?), I
think we can probably call twinkle APPROVED.

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

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


[Bug 201077] gfs-kmod

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

Summary: gfs-kmod


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 16:20 EST ---
I've fixed the comments so it provides a version number.

The documentation is provided in the user level package (which is why there are
no docs).

Also, the package name follows the kernel package name guidlines (which is why
it is not capitalized).

New packages here:
http://people.redhat.com/cfeist/gfs-kmod.spec
http://people.redhat.com/cfeist/gfs-kmod-0.1.0-4.2.6.18_1.2708.fc6.src.rpm

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

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


[Bug 198816] Review Request: gfs-utils - global file system userland utilities

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

Summary: Review Request: gfs-utils - global file system userland utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 16:14 EST ---
Jason,

Yes I do need sponsorship for this package.  Can you sponsor me?

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

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


[Bug 208613] Review Request: libgcj - separate libgcj srpm

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

Summary: Review Request: libgcj - separate libgcj srpm


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 16:05 EST ---
Huh? It seems way late in the game for this.

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

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


[Bug 208613] Review Request: libgcj - separate libgcj srpm

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

Summary: Review Request: libgcj - separate libgcj srpm


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 15:58 EST ---
(In reply to comment #1)
> I presume this is targeted at FC-7?

No, Jesse wanted this for FC-6.


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

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


[Bug 208009] Review Request: perl-Wx - Interface to the wxWidgets cross-platform GUI toolkit

2006-09-29 Thread bugzilla
Please 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-Wx - Interface to the wxWidgets cross-platform 
GUI toolkit


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||208636
  nThis||




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

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


[Bug 208636] Review Request: perl-PPI-Tester - A wxPerl-based interactive PPI debugger/tester

2006-09-29 Thread bugzilla
Please 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-PPI-Tester - A wxPerl-based interactive PPI 
debugger/tester


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||208009




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

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


[Bug 208636] New: Review Request: perl-PPI-Tester - A wxPerl-based interactive PPI debugger/tester

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




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

   Summary: Review Request: perl-PPI-Tester - A wxPerl-based
interactive PPI debugger/tester
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-PPI-Tester.spec

SRPM URL:
ftp://perl.di.uminho.pt/pub/fedora/perl-PPI-Tester-0.06-1.src.rpm

Description:
This package implements a wxWindows desktop application which provides
the ability to interactively test the PPI perl parser.

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

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


[Bug 195365] Review Request: etcnet - /etc/net network configuration system

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

Summary: Review Request: etcnet - /etc/net network configuration system


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


Bug 195365 depends on bug 205932, which changed state.

Bug 205932 Summary: hardcoded net-scripts dependency
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=205932

   What|Old Value   |New Value

 Resolution||WONTFIX
 Status|NEW |CLOSED



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

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


[Bug 208208] Review Request: MegaMek - a portable, network-enabled BattleTech engine

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

Summary: Review Request: MegaMek - a portable, network-enabled BattleTech engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 15:35 EST ---
OK, add this line to classpath.security:

securerandom.source=file:/dev/random

and re-run.


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

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


[Bug 196379] Review Request: kdeartwork-extras: Artwork Extras, including xscreensaver-based screensavers

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

Summary: Review Request: kdeartwork-extras: Artwork Extras, including 
xscreensaver-based screensavers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:54 EST ---
I will review this later.

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

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


[Bug 206989] Review Request: twinkle - A SIP Soft Phone

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

Summary: Review Request: twinkle - A SIP Soft Phone


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:45 EST ---
Parag: Strictly speaking, you're a vetted contributor, and Kevin certainly is,
so you *can* do official reviews.  If you're afraid of screwing something up, so
be it; I'm continually expecting to make glaring mistakes and get flamed for it,
but that won't stop me from reviewing. :-)

Since I don't do enough reviews (and readily acknowledge this), I think I'll
take this one on.  Looks like something that would certainly benefit Fedora.

Starting the review...well, as soon as my devel branch rsync finishes (I *swear*
I just synched it).

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

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


[Bug 208208] Review Request: MegaMek - a portable, network-enabled BattleTech engine

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

Summary: Review Request: MegaMek - a portable, network-enabled BattleTech engine


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:34 EST ---
(In reply to comment #3)
> (In reply to comment #2)
> > Created an attachment (id=137352)
 --> 
(https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=137352&action=view) 
[edit] [edit]
> > screenshot just after startup
> > 
> > The back window draws and then the front window appears to take over.  If I
> > switch windows and then switch back, nothing is re-drawn.
> 
> Can you paste the contents of /usr/lib/security/classpath.security here?

$ cat /usr/lib/security/classpath.security
security.provider.1=gnu.java.security.provider.Gnu
security.provider.2=gnu.javax.crypto.jce.GnuCrypto
security.provider.3=gnu.javax.crypto.jce.GnuSasl
security.provider.4=gnu.javax.net.ssl.provider.Jessie
security.provider.5=gnu.javax.security.auth.callback.GnuCallbacks
security.provider.6=org.bouncycastle.jce.provider.BouncyCastleProvider
security.provider.7=org.metastatic.jessie.provider.Jessie
security.provider.8=gnu.crypto.jce.GnuCrypto



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

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


[Bug 208584] Review Request: mirrormagic - Puzzle game where you steer a beam of light using mirrors

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

Summary: Review Request: mirrormagic - Puzzle game where you steer a beam of 
light using mirrors


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:32 EST ---
MUST items:
 * rpmlint doesn't show anything
 * package is named well
 * spec file name is good
 * package meets Packaging Guidelines
 * package is licensed with an GPL open-source compatible license
 * License field in spec file matches actual license
 * license file is included in %doc
 * md5sums are matching (32fd3909c1e27f493d89bc2276da6744)
 * package successfully compiles on x86_64
 * BuildRequires listed well
 * no locales
 * proper %post and %postun sections
 * not relocatable
 * package owns directories well
 * no duplicates in %files
 * every %files section includes %defattr
 * proper %clean section
 * macros used well
 * .desktop file looks good

THINGS to do:
 * this is not the blocker, but probably SDL-devel BR is unnecessary (I
haven't tested it)
 * the much more important problem is related with running the game on x86_64.
When I'm trying to do it, I get following message:

$ mirrormagic
mirrormagic: gadget incomplete (missing Bitmap)
mirrormagic: aborting

$ mirrormagic -v
mirrormagic: warning: cannot open configuration file '/home/ecik/.mirrormagic/
setup.conf'
mirrormagic: warning: using default setup values
mirrormagic: warning: cannot open configuration file '/home/ecik/.mirrormagic/
levelsetup.conf'
mirrormagic: warning: using default setup values
mirrormagic: warning: cannot open configuration file '/home/ecik/.mirrormagic/
levelsetup/classic_deflektor/levelsetup.conf'
mirrormagic: warning: using default setup values
mirrormagic: gadget incomplete (missing Bitmap)
mirrormagic: aborting

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

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


[Bug 197732] Review Request: optipng - a PNG optimizer and converter

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

Summary: Review Request: optipng - a PNG optimizer and converter


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:32 EST ---
I tend to forget to finally submit my comments here, everything is done already
except that I did not send my comment here.

SPEC: http://www-users.kawo2.rwth-aachen.de/~tmaas/fedora/optipng.spec
SRPM:
http://www-users.kawo2.rwth-aachen.de/~tmaas/fedora/repo/optipng-0.5.4-1.src.rpm

Sorry for the delay.

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

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


[Bug 187706] Review Request: alsa-oss - Userspace OSS emulation

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

Summary: Review Request: alsa-oss - Userspace OSS emulation


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:15 EST ---
Just passing on a suggestion Thorsten mentioned to me on IRC when I inquired
about alsa-oss; he said that the version of alsa-oss should probably be in sync
with the version of alsa-lib in Core.  For fc6, that's currently 1.0.12; for
fc5, it's 1.0.11rc2.  I'm not positive there are major potential implications if
they're not aligned (nor is he, I don't think), but it seems like a logical
enough idea.

I'd really like to see this make it into Extras, personally.  My father-in-law
nagged me about the package enough so that I rolled my own (basically just an
update of the one from fc1) before Kevin [Fenzi] pointed me toward this open
review.  Hack job or not, it resolved his Flash issue, and I suspect it'll make
other people similarly happy. :-)

Thanks! *patiently waits for the next iteration of the package*

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

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


[Bug 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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


Bug 207607 depends on bug 207173, which changed state.

Bug 207173 Summary: Review Request: pymsn - Python libraries for MSN Messenger 
network
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=207173

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 208436] Review Request: kphotobymail - Kphotobymail is a PyQT based application for uploading photos to flickr account.

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

Summary: Review Request: kphotobymail - Kphotobymail is a PyQT based 
application for uploading photos to flickr account.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 14:05 EST ---
Done.

Spec URL: http://freeshell.in/~kushal/kphotobymail.spec
SRPM URL: http://freeshell.in/~kushal/kphotobymail-0.3.2-4.src.rpm

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

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


[Bug 207173] Review Request: pymsn - Python libraries for MSN Messenger network

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

Summary: Review Request: pymsn - Python libraries for MSN Messenger network


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 208613] Review Request: libgcj - separate libgcj srpm

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

Summary: Review Request: libgcj - separate libgcj srpm


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:53 EST ---
I presume this is targeted at FC-7?

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

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


[Bug 197732] Review Request: optipng - a PNG optimizer and converter

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

Summary: Review Request: optipng - a PNG optimizer and converter


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:51 EST ---
Ping Till. 

Do you still want to submit this package? 
Any idea when you might address the items from comments #10 and #11?

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

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


[Bug 193106] Review Request: gtkmozembedmm

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

Summary: Review Request: gtkmozembedmm


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:49 EST ---
Any ideas on the failure from comment #11?

I can provide access to a x86_64 test machine if you would like... 

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

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


[Bug 208436] Review Request: kphotobymail - Kphotobymail is a PyQT based application for uploading photos to flickr account.

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

Summary: Review Request: kphotobymail - Kphotobymail is a PyQT based 
application for uploading photos to flickr account.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:49 EST ---
Looks good. Only remaining issue I see is that you're using spaces instead of
tabs in your desktop-file-install call (line 34). rpmlint complained about it,
so, just fix it before you commit.

APPROVED and SPONSORED. Do your paperwork, when I see it come across, I'll
sponsor you.

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

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


[Bug 208436] Review Request: kphotobymail - Kphotobymail is a PyQT based application for uploading photos to flickr account.

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

Summary: Review Request: kphotobymail - Kphotobymail is a PyQT based 
application for uploading photos to flickr account.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:45 EST ---
Fixed.. :)

Spec URL: http://freeshell.in/~kushal/kphotobymail.spec
SRPM URL: http://freeshell.in/~kushal/kphotobymail-0.3.2-3.src.rpm

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

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


[Bug 208613] Review Request: libgcj - separate libgcj srpm

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

Summary: Review Request: libgcj - separate libgcj srpm


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED] |[EMAIL PROTECTED]




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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 208613] New: Review Request: libgcj - separate libgcj srpm

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




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

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


Spec URL: http://people.redhat.com/~tromey/libgcj-srpm/libgcj41.spec
SRPM URL: http://people.redhat.com/~tromey/libgcj-srpm/libgcj-4.1.1-20.src.rpm

Description:

This is a separate source RPM for libgcj.  The idea is to allow pushing
libgcj updates separately from GCC updates, enabling faster bug fix
turnaround.

There's a separate patch for the GCC spec file.  I've sent a version of it
to Jakub, but it needs an update.  This patch reduces the build requirements
of GCC; for instance we no longer need gtk to build the GCC rpms.

There is a helper script in the libgcj SRPM to create the initial source
tar from the gcc srpm.  I think maintenance-wise the best approach is to
start with the current gcc sources and then apply libgcj patchese as needed.
Rebasing to a new gcc is simply and automated with the script.

With my patch the gcc spec file still builds libgcj.  This lets us continue
to run 'make check' to verify that compiler changes don't break the libgcj
build.  However, the libgcj built here has reduced functionality (e.g., 
no AWT peers -- this is why we can remove build requirements) and is not
packaged.

With this change, fastjar and grepjar move from the libgcj RPM into the
gcc RPM.

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

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


[Bug 188974] Review Request: mesa-libGLw - Xt / Motif OpenGL widgets

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

Summary: Review Request: mesa-libGLw - Xt / Motif OpenGL widgets


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:22 EST ---
Imported with Patrice's changes, thanks all!

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

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


[Bug 196379] Review Request: kdeartwork-extras: Artwork Extras, including xscreensaver-based screensavers

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

Summary: Review Request: kdeartwork-extras: Artwork Extras, including 
xscreensaver-based screensavers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:15 EST ---

Spec URL: http://kde-redhat.unl.edu/apt/kde-redhat/SPECS/kdeartwork-extras.spec
SRPM URL:
http://kde-redhat.unl.edu/apt/kde-redhat/all/SRPMS.stable/kdeartwork-extras-3.5.4-2.src

%changelog
* Tue Aug 29 2006 Rex Dieter  3.5.4-2
- used unversioned Requires: kdebase

* Tue Jul 25 2006 Rex Dieter  3.5.4-1
- kde-3.5.4


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

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


[Bug 199108] Review Request: gutenprint: Printer Drivers Package

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

Summary: Review Request: gutenprint: Printer Drivers Package


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:15 EST ---
one issue: 

- #rm -rf %{buildroot} in clean? That shouldn't be commented... perhaps it was 
while you were testing something?

Fix that before you import it? 

I think all the issues I can see are solved, so this package is APPROVED. 
Don't forget to close this NEXTRELASE when you have imported and built it. 

Thanks for your patience with this long process. 

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

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


[Bug 208436] Review Request: kphotobymail - Kphotobymail is a PyQT based application for uploading photos to flickr account.

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

Summary: Review Request: kphotobymail - Kphotobymail is a PyQT based 
application for uploading photos to flickr account.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:10 EST ---
A few things need to be fixed here:

You need to add:
BuildRequires: desktop-file-utils

Upstream source does not match source in SRPM, you need to reconcile that:
ecfa19fa8618da1314b3964f3d4b39c731e3d1ee  kphotobymail-0.3.2.tar.gz.UPSTREAM
70cf51f8641f315b0b6f738c0740a351fe5eb912  kphotobymail-0.3.2.tar.gz

Don't use the name of the program in the summary. A recommended change:

Summary:A PyQT based application for uploading photos to flickr

Also, while not mandatory, I highly recommend that you append %{?dist} to the
end of the Release string. See http://fedoraproject.org/wiki/Packaging/DistTag

Good:

- rpmlint checks return:
W: kphotobymail conffile-without-noreplace-flag /etc/kphotobymail.data
Should be safe to ignore.

E: kphotobymail non-executable-script
/usr/lib/python2.4/site-packages/Kphotobymail/__init__.py 0644
Should be safe to ignore.

W: kphotobymail mixed-use-of-spaces-and-tabs (spaces: line 14, tab: line 4)

Just fix line 14 to use tabs instead of spaces.

- package meets naming guidelines
- license (GPL) OK, text in %doc, matches source
- spec file legible, in am. english
- package compiles on devel (x86)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- desktop file ok

Bad:

- source does NOT match upstream

Fix the blockers and I'll revisit this.

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

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


[Bug 196379] Review Request: kdeartwork-extras: Artwork Extras, including xscreensaver-based screensavers

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

Summary: Review Request: kdeartwork-extras: Artwork Extras, including 
xscreensaver-based screensavers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 13:09 EST ---
Then, where is the newest SPEC file and SRPM?

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

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


[Bug 196379] Review Request: kdeartwork-extras: Artwork Extras, including xscreensaver-based screensavers

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

Summary: Review Request: kdeartwork-extras: Artwork Extras, including 
xscreensaver-based screensavers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:56 EST ---
fc6 (and newer)

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

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


[Bug 196379] Review Request: kdeartwork-extras: Artwork Extras, including xscreensaver-based screensavers

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

Summary: Review Request: kdeartwork-extras: Artwork Extras, including 
xscreensaver-based screensavers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:53 EST ---
Is this targeted for FC6 or FC7?

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:28 EST ---
OK, really thanks for the review!

%changelog
* Fri Sep 29 2006  Alain Portal  0.11.3-5
  - Fix typo in pickit2.perms


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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:23 EST ---
OK, thanks for the review!

%changelog
* Fri Sep 29 2006  Alain Portal  0.2.1-6
  - Remove useless update-desktop-database


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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:20 EST ---
Well, /etc/security/console.perms.d/pickit2.perms says:
-
=/dev/pickit2*
 0600  0600 roo
-
Perhaps the last word must be "root". Just fix it.


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

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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 12:06 EST ---
Well, accroding to 
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets :
desktop file does not have mimetype key, so
calling update-desktop-database is not necessary.

Other things are okay.

 This package (pikloops) is APPROVED by me.

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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 10:46 EST ---
(In reply to comment #16)
> (In reply to comment #14)
 
> But perhaps now, you could have duplicate entry. Let me know.

The result is: only one entry is shown, no duplicate.
I will check if the other things to be fixed is left.

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

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


[Bug 208584] New: Review Request: mirrormagic - Puzzle game where you steer a beam of light using mirrors

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




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

   Summary: Review Request: mirrormagic - Puzzle game where you
steer a beam of light using mirrors
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.atrpms.net/~hdegoede/mirrormagic.spec
SRPM URL: http://people.atrpms.net/~hdegoede/mirrormagic-2.0.2-1.fc6.src.rpm
Description:
MirrorMagic is a game where you shoot around obstacles to collect energy using
your beam. It is similar to "Mindbender" (Amiga) from the same author. The goal
is to work out how to get around obstacles to shoot energy containers with your
beam, thereby opening the path to the next level. Included are many levels
familiar from the games "Deflektor" and "Mindbender".

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

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


[Bug 201006] Review Request: HelixPlayer

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

Summary: Review Request: HelixPlayer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 10:40 EST ---
Imported and built, thanks a lot !

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 10:16 EST ---
Spec URL: http://linuxelectronique.free.fr/download/fedora/5/SPECS/piklab.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/piklab-0.11.3-4.src.rpm

%changelog
* Wed Sep 29 2006  Alain Portal  0.11.3-4
  - Fix typo for installing pickit2.perms
  - Add Application category in desktop file
  - Fix symlink: english is the only language where common directory is in
  LANG directory while for other, common is in LANG/docs directory


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

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


[Bug 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 10:05 EST ---
ok, once you do that the filesystem package is APPROVED
I will give a review on butterfly soon

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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 09:24 EST ---
(In reply to comment #14)
> > > I don't understand why I have to relogin to see the new entry in the 
> menu...
> > Umm.. I don't know either because I use GNOME...

G! I found... That was because there wasn't "Application" in categories.

But perhaps now, you could have duplicate entry. Let me know.
Spec URL: 
http://linuxelectronique.free.fr/download/fedora/5/SPECS/pikloops.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/pikloops-0.2.1-5.src.rpm

%changelog
* Thu Sep 28 2006  Alain Portal  0.2.1-5
  - Update %%{SOURCE1} to add Application category



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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 09:15 EST ---
Spec URL: 
http://linuxelectronique.free.fr/download/fedora/5/SPECS/pikloops.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/pikloops-0.2.1-4.src.rpm

%changelog
* Thu Sep 28 2006  Alain Portal  0.2.1-4
  - Update %%{SOURCE1} which don't contain "OnlyShowIn=KDE;" any more


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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 09:07 EST ---
(In reply to comment #13)
> (In reply to comment #12)
> > (In reply to comment #11)
> > > 
> > > Do you see some duplicate in KDE?
> > 
> > No, only one.
> > I don't understand why I have to relogin to see the new entry in the 
menu...
> Umm.. I don't know either because I use GNOME...
>  
> > > In GNOME (I use GNOME), when the line
> > > OnlyShowIn=KDE; is removed, the PiKLoops seems to appear only in
> > > "Programming" entry and to have no duplicates. 
> > 
> > And you see nothing if it isn't removed?
> 
> Yes, actually I see nothing.

OK, I'll update the desktop file.


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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 09:04 EST ---
(In reply to comment #21)

> > No. KDE recently changed /usr/share/doc/HTML/$LANG/common 
> > in /usr/share/doc/HTML/$LANG/docs/common.
> 
> Well, then what package owns /usr/share/doc/HTML/en/docs/common
> _NOW_ ? I use rawhide, and the newest rawhide kdelibs-3.5.4-6.fc6
> owns /usr/share/doc/HTML/en/common, however, my system doesn't have
> /usr/share/doc/HTML/en/docs/common directory.

The /usr/share/doc/HTML/$LANG/docs/common is owned by kde-i18n-$LANG.
You right, as english is the default language /usr/share/doc/HTML/en/common is 
owned by kdelibs. But I thought that /usr/share/doc/HTML/en/docs/common was 
owned by kde-i18n-English, but this package don't exist ;-)
This is the exception.
So, for all languages but english, common is in $LANG/docs/, for english, it 
is in $LANG/

> So my opinition is:
> * If some package actually owns /usr/share/doc/HTML/en/docs/common,
>   this package should require the package (I am now trying to search
>   for it by yum, however for some reason I don't know yum is very
>   slow for now!!)
> * If no package owns the directory _NOW_, 
>/usr/share/doc/HTML/en/piklab/common should point to ../common
>   as before till some package gets to 
own /usr/share/doc/HTML/en/docs/common.
>   
> > >Also, while this is not documented, the files under
> > >/etc/security/console.perms.d/ seem to have the names like
> > >-.perms (like 50-default.perms).
> > 
> > I don't know how to choose a number.
> > This configuration is taken from http://piklab.sourceforge.net/support.php
> > section "for distributions using udev and PAM.
> 
> This is not documented and you don't have to add the number if
> you don't want to.

As I don't know choosing a number, I'll don't add. I'll see later when I'll 
get more informations.

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 09:03 EST ---
(In reply to comment #21)
> 
> Well, then what package owns /usr/share/doc/HTML/en/docs/common
> _NOW_ ? 

I finished searching for this by yum. The result is, no package
owns this (in rawhide).

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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:54 EST ---
(In reply to comment #12)
> (In reply to comment #11)
> > 
> > Do you see some duplicate in KDE?
> 
> No, only one.
> I don't understand why I have to relogin to see the new entry in the menu...
Umm.. I don't know either because I use GNOME...
 
> > In GNOME (I use GNOME), when the line
> > OnlyShowIn=KDE; is removed, the PiKLoops seems to appear only in
> > "Programming" entry and to have no duplicates. 
> 
> And you see nothing if it isn't removed?

Yes, actually I see nothing.

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

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


[Bug 201006] Review Request: HelixPlayer

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

Summary: Review Request: HelixPlayer


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:50 EST ---
OK, looks good, APPROVED.

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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:40 EST ---
(In reply to comment #11)
> 
> Do you see some duplicate in KDE?

No, only one.
I don't understand why I have to relogin to see the new entry in the menu...

> In GNOME (I use GNOME), when the line
> OnlyShowIn=KDE; is removed, the PiKLoops seems to appear only in
> "Programming" entry and to have no duplicates. 

And you see nothing if it isn't removed?



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

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


[Bug 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||205343
  nThis||




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

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


[Bug 205343] Review Request: cohoba - Cohoba is a GNOME interface for Telepathy.

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

Summary: Review Request: cohoba -  Cohoba is a GNOME interface for Telepathy.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  BugsThisDependsOn||207607




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:35 EST ---
It looks like all the issues in comment #7 & comment #10 have been addressed. 
The only blocker that I see is that the package needs to require the
telepathy-filesystem package since it has files in the
%{_datadir}/telepathy/managers/ directory.  Therefore this bug also depends on
Bug #207607 which has that package.

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

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


[Bug 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:34 EST ---
Yeah, I can make the filesystem a noarch package.

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:18 EST ---
(In reply to comment #20)
> (In reply to comment #19)
> >/usr/share/doc/HTML/en/piklab/common ../doc/common
> >- Well, this warning itself is no problem, however, the problem
> >  is that this symlink is broken.
> >  Perhaps this should point to ../common  .
> 
> No. KDE recently changed /usr/share/doc/HTML/$LANG/common 
> in /usr/share/doc/HTML/$LANG/docs/common.

Well, then what package owns /usr/share/doc/HTML/en/docs/common
_NOW_ ? I use rawhide, and the newest rawhide kdelibs-3.5.4-6.fc6
owns /usr/share/doc/HTML/en/common, however, my system doesn't have
/usr/share/doc/HTML/en/docs/common directory.

So my opinition is:
* If some package actually owns /usr/share/doc/HTML/en/docs/common,
  this package should require the package (I am now trying to search
  for it by yum, however for some reason I don't know yum is very
  slow for now!!)
* If no package owns the directory _NOW_, 
   /usr/share/doc/HTML/en/piklab/common should point to ../common
  as before till some package gets to own /usr/share/doc/HTML/en/docs/common.
  
> > 3. Other things I have noticed:
> >- Well, 
> > 
> > /etc/security/console.perms.d/pickit1.perms
> > /etc/security/console.perms.d/pickit2.perms
> > 
> >  These two files are same. Acutally spec file says:
> > %{__install} -pm 644 %{SOURCE3} \
> >%{buildroot}%{_sysconfdir}/security/console.perms.d/pickit1.perms
> > %{__install} -pm 644 %{SOURCE3} \
> >%{buildroot}%{_sysconfdir}/security/console.perms.d/pickit2.perms
> > 
> No, this is an error, the second should be %{SOURCE4}

Okay. Just fix as it should be.

> 
> >Also, while this is not documented, the files under
> >/etc/security/console.perms.d/ seem to have the names like
> >-.perms (like 50-default.perms).
> 
> I don't know how to choose a number.
> This configuration is taken from http://piklab.sourceforge.net/support.php
> section "for distributions using udev and PAM.

This is not documented and you don't have to add the number if
you don't want to.


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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:11 EST ---
(In reply to comment #10)
> (In reply to comment #9)
> > One question.
> > 
> > Why does /usr/share/applications/kde/pikloops.desktop have
> > OnlyShowIn=KDE; ?
> > 
> It seems to me that was to prevent duplicate desktop entry in menu.
> Do I have to remove it?

Do you see some duplicate in KDE? In GNOME (I use GNOME), when the line
OnlyShowIn=KDE; is removed, the PiKLoops seems to appear only in
"Programming" entry and to have no duplicates. 



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

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


[Bug 199681] Review Request: slab

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

Summary: Review Request: slab


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:11 EST ---
Good catch, this was caused by some missing build-requires (nl: libtool,
intltools, wireless-tools). Added those and uploaded the fixed releases to.

To make things more interesting, in fedora-devel its wireless-tools-devel and in
fc5 its just wireless-tools, so i've added some distro specific conditional BR's
 to make it work in both situations; After these modifications, it mock builds
correctly again under fc5 & fc6/devel

The results can be found at at:

http://chabotc.nl/slab/slab.spec
http://chabotc.nl/slab/slab-0.6.2-4.src.rpm



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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 08:03 EST ---
(In reply to comment #9)
> One question.
> 
> Why does /usr/share/applications/kde/pikloops.desktop have
> OnlyShowIn=KDE; ?
> 
> I tried to remove this, and PiKLoops icon appears on "Programming"
> session, so I think this can be shown in GNOME, too.

It seems to me that was to prevent duplicate desktop entry in menu.
Do I have to remove it?


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

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


[Bug 208254] Review Request: pikloops - Code generator for PIC delays

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

Summary: Review Request: pikloops - Code generator for PIC delays


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 07:48 EST ---
One question.

Why does /usr/share/applications/kde/pikloops.desktop have
OnlyShowIn=KDE; ?

I tried to remove this, and PiKLoops icon appears on "Programming"
session, so I think this can be shown in GNOME, too.

Other things seem to be okay.

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 07:20 EST ---
(In reply to comment #19)
> Okay. Full review for piklab.
> 
> 1. From http://fedoraproject.org/wiki/Packaging/Guidelines
> 
> * Use rpmlint
> W: piklab dangling-relative-symlink \
>/usr/share/doc/HTML/en/piklab/common ../doc/common
>- Well, this warning itself is no problem, however, the problem
>  is that this symlink is broken.
>  Perhaps this should point to ../common  .

No. KDE recently changed /usr/share/doc/HTML/$LANG/common 
in /usr/share/doc/HTML/$LANG/docs/common.

See 
https://www.redhat.com/archives/fedora-extras-list/2006-September/msg00794.html 
and some follow up.


> 2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines :
>= Nothing.
> 
> 3. Other things I have noticed:
>- Well, 
> 
> /etc/security/console.perms.d/pickit1.perms
> /etc/security/console.perms.d/pickit2.perms
> 
>  These two files are same. Acutally spec file says:
> %{__install} -pm 644 %{SOURCE3} \
>%{buildroot}%{_sysconfdir}/security/console.perms.d/pickit1.perms
> %{__install} -pm 644 %{SOURCE3} \
>%{buildroot}%{_sysconfdir}/security/console.perms.d/pickit2.perms
> 
>I suspect only one of these two are necessary.

No, this is an error, the second should be %{SOURCE4}

>Also, while this is not documented, the files under
>/etc/security/console.perms.d/ seem to have the names like
>-.perms (like 50-default.perms).

I don't know how to choose a number.
This configuration is taken from http://piklab.sourceforge.net/support.php
section "for distributions using udev and PAM.



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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 06:57 EST ---
Okay. Full review for piklab.

1. From http://fedoraproject.org/wiki/Packaging/Guidelines

* Use rpmlint
W: piklab dangling-relative-symlink \
   /usr/share/doc/HTML/en/piklab/common ../doc/common
   - Well, this warning itself is no problem, however, the problem
 is that this symlink is broken.
 Perhaps this should point to ../common  .

2. From http://fedoraproject.org/wiki/Packaging/ReviewGuidelines :
   = Nothing.

3. Other things I have noticed:
   - Well, 

/etc/security/console.perms.d/pickit1.perms
/etc/security/console.perms.d/pickit2.perms

 These two files are same. Acutally spec file says:
%{__install} -pm 644 %{SOURCE3} \
   %{buildroot}%{_sysconfdir}/security/console.perms.d/pickit1.perms
%{__install} -pm 644 %{SOURCE3} \
   %{buildroot}%{_sysconfdir}/security/console.perms.d/pickit2.perms

   I suspect only one of these two are necessary.
   Also, while this is not documented, the files under
   /etc/security/console.perms.d/ seem to have the names like
   -.perms (like 50-default.perms).

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

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


[Bug 203520] Review Request: evolution-brutus

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

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

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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 05:30 EST ---
Well, I cannot formally review this package because
I am not a sponsor, so I hope someone who has sponsor
membership would review this.

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

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


[Bug 176581] Review Request: fnord -- A very fast HTTP server

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

Summary: Review Request: fnord -- A very fast HTTP server


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


Bug 176581 depends on bug 173459, which changed state.

Bug 173459 Summary: Review Request: initng
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=173459

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 173459] Review Request: initng

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

Summary: Review Request: initng


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 05:03 EST ---
(In reply to comment #344)
> I'd say simply replace the old files with the new ones.

Ok. I'll do that. Well, since initng and initng-ifiles now has been successfully
built in FE-devel, I guess it's time to close this ticket. A big thanks to
everyone that has been involved.

This is comment #345. This bug was open for 10 months and 12 days. Over and out.

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 05:03 EST ---
Spec URL: http://linuxelectronique.free.fr/download/fedora/5/SPECS/piklab.spec
SRPM URL: 
http://linuxelectronique.free.fr/download/fedora/5/SRPMS/piklab-0.11.3-3.src.rpm

%changelog
* Wed Sep 29 2006  Alain Portal  0.11.3-3
  - Add doc about how to complete full feature installation
  - Fix typo on security pathname
  - Short summary
  - Add BR ncurses-devel for FC6


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

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


[Bug 207607] Review Request: telepathy-butterfly - MSN connection manager for Telepathy

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

Summary: Review Request: telepathy-butterfly - MSN connection manager for 
Telepathy


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 04:51 EST ---
can you switch telepathy-filesystem to noarch?

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

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


[Bug 208250] Review Request: piklab - Development environment for applications based on PIC and dsPIC microcontrollers

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

Summary: Review Request: piklab - Development environment for applications 
based on PIC and dsPIC microcontrollers


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 04:42 EST ---
(In reply to comment #16)
> (In reply to comment #15)
> > (In reply to comment #14)
> > > The problem is to know if I have to add this BR or if this BR is missing 
in 
> > a 
> > > package from which pikloop depends.
> > 
> > piklab of course :-)
> 
> Well, configure requests the existence of -lcurses.
> in FC5, this is in ncurses, which are installed by minimal buildroot.
> However, in FC6, libcurses.so is moved to ncurses-devel.
> 
> So you have to add ncurses-devel to BuildRequires and the problem
> for rebuilding is resolved.
> 
> I have not yet reviewed this fully, however,
> * rpmlint complaints about long summary. It says that summary should
>   have no longer than 79 characters.

I really don't understand, my rpmlint don't complaints!
I'll fix...

> * Please add "ncurses-devel" to BuildRequiers (this should be no problem
>   for FC5, too).

Before doing this change, I'm waiting one hour or two for an answer to my 
question on Extras list.

> * Perhaps /etc/security/consoles.perms.d/ should be
>   /etc/security/console.perms.d/ ? (the latter is owned by pam)

Oups, you're right, it's a typo.


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

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


[Bug 207173] Review Request: pymsn - Python libraries for MSN Messenger network

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

Summary: Review Request: pymsn - Python libraries for MSN Messenger network


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778  |163779
  nThis||




--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 04:35 EST ---
Good:

- rpmlint checks return nothing
- package meets naming guidelines
- package meets packaging guidelines
- license (LGPL) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream
- package compiles on devel (x86_64)
- no missing BR
- no unnecessary BR
- no locales
- not relocatable
- owns all directories that it creates
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- no need for .desktop file

APPROVED

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

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


[Bug 205343] Review Request: cohoba - Cohoba is a GNOME interface for Telepathy.

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

Summary: Review Request: cohoba -  Cohoba is a GNOME interface for Telepathy.


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





--- Additional Comments From [EMAIL PROTECTED]  2006-09-29 04:27 EST ---
http://fedora.hoentjen.eu/telepathy/cohoba.spec
http://fedora.hoentjen.eu/telepathy/cohoba-0.0.4-1.src.rpm

I think all issues are fixed, cohoba is updated to latest release which has a
message when no connection managers are found.


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

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


  1   2   >