[Bug 479413] Review Request: pywebdav - WebDAV library for Python

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Dan Horák d...@danny.cz changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #8 from Dan Horák d...@danny.cz  2009-01-21 03:18:44 EDT ---
imported and built

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

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

[Bug 225977] Merge Review: ksh

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Michal Hlavinka mhlav...@redhat.com changed:

   What|Removed |Added

   Flag|needinfo?(mhlav...@redhat.c |
   |om) |




--- Comment #5 from Michal Hlavinka mhlav...@redhat.com  2009-01-21 03:18:27 
EDT ---
 Absolute symlinks are problematic eg. when working with chroot environments.

yes, but because only common directory for /bin/ksh and /usr/bin/ksh is /
directory, so chroot will never work here even with relative path

I think there is no package that require 'ksh' at all, so yes... I'll remove
this link.

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

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

[Bug 426751] Review Request: ghc-X11 - A Haskell binding to the X11 graphics library.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #43 from Jens Petersen peter...@redhat.com  2009-01-21 03:19:20 
EDT ---
Thanks Yaakov, will try to take a look at it soon.

(The bug will soon have its first birthday so hope it can be resolved soon.:)

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

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

[Bug 426751] Review Request: ghc-X11 - A Haskell binding to the X11 graphics library.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #44 from Jens Petersen peter...@redhat.com  2009-01-21 03:24:16 
EDT ---
(In reply to comment #42)
 Note: AFAIK, we don't have the macros needed in F10 right now, so this only
 builds in rawhide.  The SRPM is output from mock using rawhide. Once the 
 macros
 are in F10, supporting it there should be trivial.

Ok, finally I built ghc-6.10.1-7.fc10 for F10.

http://koji.fedoraproject.org/koji/buildinfo?buildID=79616

Unfortunately still need to update haddock and gtk2hs before it can be pushed
to F10 updates.  (With the number libs going in fedora now I suspect it is
going to get harder and less common to backport ghc going forward but let us
see.)

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

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

[Bug 253813] Package Review: cjkunifonts (split from fonts-chinese) [renamed to cjkuni-fonts]

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #45 from Jens Petersen peter...@redhat.com  2009-01-21 03:28:29 
EDT ---
http://koji.fedoraproject.org/koji/buildinfo?buildID=79605

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

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

[Bug 480771] Review Request: tunneler - Clone of legendary Tunneler game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Felix Kaechele fe...@fetzig.org  2009-01-21 03:54:29 EDT 
---
I didn't meant to sound harsh. If you felt offended by the way I wrote my
comment I hereby apologize for that.

For reference here is the Guideline on desktop files:
https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage
It says:
For new packages, do not apply a vendor tag to desktop files.

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

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

[Bug 480863] Review Request: mythes-ro - Romanian thesaurus

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Caolan McNamara caol...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Caolan McNamara caol...@redhat.com  2009-01-21 04:02:56 
EDT ---
New Package CVS Request
===
Package Name: mythes-ro
Short Description: Romanian thesaurus
Owners: caolanm
Branches: devel
InitialCC:

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from _pjp_ pj.pan...@yahoo.co.in  2009-01-21 04:05:02 EDT ---
 Nowhere. You are supposed to use the original tarballs + patches inside of
 your rpm.

  Why? I don't think that's a good idea. I think it'll be easy for me to
maintain the patched sources, may be on my local svn.

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

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

[Bug 480771] Review Request: tunneler - Clone of legendary Tunneler game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #8 from Šimon Lukašík luka...@atlas.cz  2009-01-21 04:06:42 EDT 
---
 https://fedoraproject.org/wiki/Packaging:Guidelines#desktop-file-install_usage
Imho it has changed during this review. I saw old vendor tag here few hours
ago.

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

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

[Bug 480771] Review Request: tunneler - Clone of legendary Tunneler game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
04:20:59 EDT ---
(In reply to comment #9)
 It was just wiki package was not updated.

wiki page was (not updated)

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

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

[Bug 480771] Review Request: tunneler - Clone of legendary Tunneler game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
04:18:27 EDT ---
Actually this desktop-file-install usage change (i.e. that
for new packages --vendor=fedora is removed) took place in
last October:

http://www.redhat.com/archives/fedora-devel-list/2008-October/msg02273.html
http://fedoraproject.org/wiki/Packaging/Minutes/20081021

It was just wiki package was not updated.

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

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

[Bug 225808] Merge Review: gmime

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?(al...@redhat.com)




--- Comment #5 from Debarshi Ray debarshi@gmail.com  2009-01-21 04:31:37 
EDT ---
Ping?

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

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

[Bug 225852] Merge Review: gok

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?(dav...@redhat.com
   ||)




--- Comment #3 from Debarshi Ray debarshi@gmail.com  2009-01-21 04:29:49 
EDT ---
Ping?

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

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

[Bug 225865] Merge Review: gstreamer-plugins-base

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

   Flag||needinfo?(a...@redhat.com)




--- Comment #4 from Debarshi Ray debarshi@gmail.com  2009-01-21 04:32:31 
EDT ---
Ping?

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

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

[Bug 480771] Review Request: tunneler - Clone of legendary Tunneler game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
04:40:33 EDT ---
So please remove --vendor=fedora from desktop-file-install.

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

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

[Bug 224245] Merge Review: squirrelmail

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #20 from Tomas to...@users.sourceforge.net  2009-01-21 04:48:32 
EDT ---
(In reply to comment #19)
 Tomas, if you could explain me (irc?) something about the utf-8 vs. CJK mess,
 it would be really appreciated. I've got this package with some fixes for CJK
 and that confuses me... I thought utf-8 should be able to handle it, so I'm
 little bit confused about all CJK workarounds/patches.

If you need explanation about patches in squirrelmail-1.4.16-1.fc10.src.rpm,
see information in bugzilla.

squirrelmail-1.4.6-japanese-multibyte-view-body.patch - RH bug #194457
squirrelmail-1.4.6-japanese-multibyte-view-text.patch - RH bug #195452
squirrelmail-1.4.6-zenkaku-subject-convert.patch - RH bug #196017

All patches written by RH/Fedora and information about them was never submitted
to upstream. I can't explain why scripts are modified without seeing broken
emails. zenkaku-subject-convert.patch might be needed because according to
Wikipedia article on Katakana, hankaku (halfwidth form of katakana) is not
supported by ISO-2022-JP.

If you need explanation why SquirrelMail Japanese translation works that way -
it was written that way by third party (risumail.jp) contributors in
SquirrelMail 1.3-dev branch. In SquirrelMail 1.4.4 and older utf-8 and charset
conversion support was very simple and limited to message display. Only
SquirrelMail 1.4.5 improved charset handling in compose. I suspect that
risumail.jp just wanted to get things working and they did it in their own
works for us, lets ignore other part of the world way.

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
04:58:39 EDT ---
(In reply to comment #10)

   Why? I don't think that's a good idea. I think it'll be easy for me to
 maintain the patched sources, may be on my local svn.

it's more easy to track what's changed if you use the original tarballs +
patches inside of your rpm., and if you like your package reviewed please work
in this way

include small patchs like daemontools-0.76-errno.patch and others

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

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

[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/Linux

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #37 from Gratien D'haese gratien.dha...@it3.be  2009-01-21 
05:32:56 EDT ---
1/ basic rpmbuild/rpmlint tests:

$ rpmbuild -ba SPECS/backup-manager.spec
= No issues here. Good.

$ rpmlint -vi SRPMS/backup-manager-0.7.7-7.fc9.src.rpm
RPMS/noarch/backup-manager-0.7.7-7.fc9.noarch.rpm
backup-manager.src: I: checking
backup-manager.src: W: strange-permission backup-manager.spec 0755
A file that you listed to include in your package has strange permissions.
Usually, a file should have 0644 permissions.

backup-manager.noarch: I: checking
2 packages and 0 specfiles checked; 0 errors, 1 warnings.
= pls. fix the permission of the spec file before starting the rpmbuild
process

2/ check spec file:

The spec file contains 'Requires' lines which are not really needed - see:
http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

Requires:   /bin/bash
Requires:   bzip2
Requires:   gzip
Requires:   diffutils
Requires:   sed
Requires:   tar
Requires:   which

3/ install  usage backup-manager:
- install rpm : ok
- usage : ok
- removal of rpm : ok

Good your almost there.

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

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

[Bug 460959] Review Request: libkml - A KML library written in C++ with bindings to other languagues

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #19 from Peter Lemenkov lemen...@gmail.com  2009-01-21 05:58:57 
EDT ---
Will review during this week (probably). Unfortunately, I have no access to
internet at home.

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

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

[Bug 446976] Review Request: tkabber - Tkabber is a Free and Open Source client for the Jabber instant messaging system.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Peter Lemenkov lemen...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(lemen...@gmail.co |
   |m)  |




--- Comment #10 from Peter Lemenkov lemen...@gmail.com  2009-01-21 06:11:02 
EDT ---
Alexey Lubimov promised me in private email, that he will renew tkabber package
soon. So, I think we should wait for a while.

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from _pjp_ pj.pan...@yahoo.co.in  2009-01-21 06:19:40 EDT ---
 it's more easy to track what's changed if you use the original tarballs +
 patches inside of your rpm., and if you like your package reviewed please
 work in this way

  Uff...alright; I've made the changes. Please have a loot at the following

SPEC: http://pjp.dgplug.org/tools/daemontools.spec
SRPM: http://pjp.dgplug.org/tools/daemontools-0.76-1.fc10.src.rpm

Thank you.

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

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

[Bug 480916] New: Review Request: w_scan - Tool for scanning DVB-C/T transponders

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: w_scan - Tool for scanning DVB-C/T transponders

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

   Summary: Review Request: w_scan - Tool for scanning DVB-C/T
transponders
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fe...@fetzig.org
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://heffer.fedorapeople.org/review/w_scan.spec
SRPM URL: http://heffer.fedorapeople.org/review/w_scan-20081106-1.fc10.src.rpm
Description: The w_scan tool is similar to dvbscan from dvb-apps. However it
does not require initial tuning data and thus is able to find more channels.
Note that w_scan does not support DVB-S (only DVB-C/T).

rpmlint is silent on this one

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

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

[Bug 467396] Review Request: mingw32-freetype - Free and portable font rendering engine

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 05:53:12 EDT ---
mingw32-freetype-2.3.8-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-freetype-2.3.8-1.fc10

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

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

[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/Linux

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Christoph Wickert fed...@christoph-wickert.de changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #38 from Christoph Wickert fed...@christoph-wickert.de  
2009-01-21 06:48:13 EDT ---
(In reply to comment #36)
 @Christoph  Your review is finish ? The review is approve ? Because this
 review block my sponsorships and the pidgin-privacy-please integration...

Why that? I thought Aurelien already said he will sponsor you.

I'm not really happy with the hardcoded paths bin /backup manager, but on the
other hand I don't want to write a ./configure. This should be done upstream.

I see backup-manager-0.7.7-7.fc10 fixes all issues from comment #23, but
instead of
  %{name}-%{version}-configtpl.patch
you should use
  %{name}-0.7.7-configtpl.patch
Otherwise you will have to delete and re-add the file in cvs on every update.

Nevertheless this is your decision and you can fix it after import, so I will
APPROVE the package now.


(In reply to comment #37)
 The spec file contains 'Requires' lines which are not really needed - see:
 http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

This list is for _BuildRequries_ in the build system but not for _Requires_ of
the package itself.
Also note that your third test from comment #28 is not supposed to work.

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

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

[Bug 467407] Review Request: mingw32-sqlite - MinGW Windows port of sqlite embeddable SQL database engine

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 05:53:29 EDT ---
mingw32-sqlite-3.6.6.2-1.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-sqlite-3.6.6.2-1.fc10

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

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

[Bug 468376] Review Request: mingw32-SDL - MinGW Windows port of SDL cross-platform multimedia library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 05:53:50 EDT ---
mingw32-SDL-1.2.13-4.fc10 has been submitted as an update for Fedora 10.
http://admin.fedoraproject.org/updates/mingw32-SDL-1.2.13-4.fc10

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from _pjp_ pj.pan...@yahoo.co.in  2009-01-21 06:56:15 EDT ---
Please consider this new

SRPM: http://pjp.dgplug.org/tools/daemontools-0.76-2.fc10.src.rpm

Thank you.

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


_pjp_ pj.pan...@yahoo.co.in changed:

   What|Removed |Added

URL|http://pjp.dgplug.org/tools |http://cr.yp.to/daemontools
   |/daemontools-0.76.tar.gz|/daemontools-0.76.tar.gz




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

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

[Bug 464430] Review Request: k12linux-quick-start-guide - Doc describing how to enable LTSP on Fedora Live LTSP.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #46 from Patrice Dumas pertu...@free.fr  2009-01-21 08:04:22 EDT 
---
Is this built?

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

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

[Bug 225302] Merge Review: automake

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Karsten Hopp kars...@redhat.com  2009-01-21 08:38:45 EDT 
---
NEWS file fixed in automake-1.10.2-2. Can't fix the ansi2knr stuff as this is
used by some m4 macros and will be compiled when needed.

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #14 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
08:48:05 EDT ---
1 - instead changing conf-cc please use  daemontools-0.76-errno.patch as Patch0

2 - debug info package is not being created for some reason, we need to figure
why

3 - should be good to have a svscanboot [UPS] running with upstart, Mamoru is
the best person to answer about what directory, because /service should not be
used, may be somthing like /var/run/daemontools  ?


UPS -
https://bugs.launchpad.net/ubuntu/+source/daemontools-installer/+bug/66615

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

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

[Bug 480589] Review Request: ctan-cm-lgc-fonts - CM-LGC Type1 fonts

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Sarantis Paskalis paska...@di.uoa.gr changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Comment #6 from Sarantis Paskalis paska...@di.uoa.gr  2009-01-21 09:15:26 
EDT ---
Built in rawhide.  Edited comps. tetex-font-cm-lgc EOLed.

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

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

[Bug 475097] Review Request: gimp-fourier-plugin - A fourier transformation plugin for GIMP.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
09:19:31 EDT ---
! First of all:
  Please check your srpm / rebuilt binary rpms using rpmlint
  (in rpmlint rpm). This will detect some generic packaging errors.

For 0.3.2-1:
* odd issue in srpm
  - Please change the permissions of all files in srpm to 0644.

* disttag
  - Please consider to use %{?dist} tag:
https://fedoraproject.org/wiki/Packaging/DistTag

* Macros in SourceURL
  - Using %{name}, %{version}, especially %{version} in SourceURL
is recommended:
https://fedoraproject.org/wiki/Packaging/SourceURL#Using_.25.7Bversion.7D

* Misc rpmlint issue

W: summary-ended-with-dot A simple plug-in to do fourier transform on you
image.
E: description-line-too-long A simple plug-in to do fourier ...

  - Summary must not end with dot.
  - Please shorten the length of a line in %description. It should
not exceed 79 characters.

* License
  - The licene tag GPL is invalid on Fedora.
https://fedoraproject.org/wiki/Packaging/LicensingGuidelines#GPL_and_LGPL
For this package the license tag should be GPLv3+.

* (Build)Requires
  - Please check if BuildRequires: gimp is needed.
  - Requires: fftw3 is definitely unneeded. Such library related
dependencies are automatically detected by rpmbuild itself and
rpmbuild correctly adds those dependencies.
! Requires: gimp is needed for this case because of directory
  ownership issue

* compiler flags
  - Fedora specific compiler flags are not honored correctly.
https://fedoraproject.org/wiki/Packaging/Guidelines#Compiler_flags
This can be specified by %optflags (you can check what compiler
flags must be honored by $rpm --eval %optflags)
! As a result, currently debuginfo rpm creation is incomplete.

* $RPM_BUILD_ROOT vs %{buildroot}
  - Choose one. Please don't use both.

* Directory ownership issue
  - The directory %{_libdir}/gimp/2.0{,/plug-ins} themselves should not
be owned by this package:
   
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership

* Documents
  - Please add the following file(s) to %doc.
-
README
gpl.txt
-
! By the way README files are not in UTF-8, and this has
  CRLF, LF line terminators. Please fix these.

* %changelog
  - We recommend that one line is put between each %changelog entry
 (it will be helpful in Fedora CVS), like:
-
* Mon Jan 19 2009 Fabian Deutsch fabian.deutsch at gmx.de 0.3.2-1
- Updated to 0.3.2
- Includes license note
- Small hack around gimptool-2.0 bug.

* Sun Dec 07 2008 Fabian Deutsch fabian.deutsch at gmx.de 0.3.1-1
- Version 0.3.1
- Initial.
-

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

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

[Bug 480944] Review Request: perl-Test-Harness-Straps - Detailed analysis of test results

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Marcela Maslanova mmasl...@redhat.com changed:

   What|Removed |Added

 CC||ska...@redhat.com




--- Comment #1 from Marcela Maslanova mmasl...@redhat.com  2009-01-21 
09:34:33 EDT ---
Test::Harness doesn't use Straps. These are included in this package only for
backward compatibility. 

Details of problem are mentioned in this thread:
www.nntp.perl.org/group/perl.perl5.porters/2009/01/msg143565.html

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

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

[Bug 467376] Review Request: mingw32-pixman - MinGW Windows Pixman library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Richard W.M. Jones rjo...@redhat.com  2009-01-21 09:33:58 
EDT ---
These are all rpmlint warnings/errors that we'd normally ignore.
I'll explain here why:

script-without-shebang [...] libpixman-1.la
 - We use *.la files when linking.  This is explicitly allowed in
   the MinGW guidelines that were passed.
   http://fedoraproject.org/wiki/Packaging/MinGW#Libraries_.28DLLs.29

devel-file-in-non-devel-package
 - All MinGW packages are for development only, so when we passed
   the guidelines we didn't opt to have a separate -devel subpackage.
   See bug 468987 for why this warning isn't suppressed.

spurious-executable-perm
 - See https://bugzilla.redhat.com/show_bug.cgi?id=467397#c4

arch-independent-package-contains-binary-or-object
 - See bug 468989.

non-standard-dir-in-usr i686-pc-mingw32
 - This breaks the FHS, but is permitted by the MinGW guidelines.

/me should put this information in a wiki page ...

/me starts https://fedoraproject.org/wiki/MinGW/Rpmlint 

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

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

[Bug 480567] Review Request: dnssec-conf - DNSSEC and DLV configuration and priming tool

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Paul Wouters p...@xelerance.com  2009-01-21 09:38:31 EDT 
---
Fixed all issues (bad and TODO)

Spec URL: ftp://ftp.xelerance.com/dnssec-conf/dnssec-conf.spec
SRPM URL: ftp://ftp.xelerance.com/dnssec-conf/dnssec-conf-1.13-1.src.rpm

Thanks for the review!

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

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

[Bug 480859] Review Request: diffuse - graphical diff tool (first package, seeking sponser)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


François Kooman fkoo...@tuxed.net changed:

   What|Removed |Added

 CC||fkoo...@tuxed.net




--- Comment #1 from François Kooman fkoo...@tuxed.net  2009-01-21 09:38:38 
EDT ---
(Just some remarks from quickly looking over the package)

- For your Source line see:
https://fedoraproject.org/wiki/PackagingDrafts/SourceUrl, maybe also make it
Source0 instead of just Source.

- See https://fedoraproject.org/wiki/Packaging:Python for specific Python
packaging guidelines

- See https://fedoraproject.org/wiki/PackageMaintainers/CreatingPackageHowTo
for a template of a spec file (or use rpmdev-newspec), it might be nice to
create your spec file based on this or modify yours to better match the
template.

rpmlint output:

[fkoo...@franek SPECS]$ rpmlint diffuse.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.
[fkoo...@franek SPECS]$ 

[fkoo...@franek SRPMS]$ rpmlint diffuse-0.2.15-1.fc10.src.rpm 
diffuse.src: W: summary-not-capitalized graphical tool for comparing and
merging text files
diffuse.src: W: non-standard-group Development/Tools/Version Control
diffuse.src: W: no-url-tag
1 packages and 0 specfiles checked; 0 errors, 3 warnings.
[fkoo...@franek SRPMS]$ 

[fkoo...@franek noarch]$ rpmlint diffuse-0.2.15-1.fc10.noarch.rpm 
diffuse.noarch: W: summary-not-capitalized graphical tool for comparing and
merging text files
diffuse.noarch: W: non-standard-group Development/Tools/Version Control
diffuse.noarch: W: no-url-tag
diffuse.noarch: W: conffile-without-noreplace-flag /etc/diffuserc
diffuse.noarch: E: invalid-desktopfile /usr/share/applications/diffuse.desktop
1 packages and 0 specfiles checked; 1 errors, 4 warnings.
[fkoo...@franek noarch]$

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

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

[Bug 480944] New: Review Request: perl-Test-Harness-Straps - Detailed analysis of test results

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: perl-Test-Harness-Straps - Detailed analysis of test 
results

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

   Summary: Review Request: perl-Test-Harness-Straps - Detailed
analysis of test results
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mmasl...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL:
http://mmaslano.fedorapeople.org/perl-Test-Harness-Straps/perl-Test-Harness-Straps.spec
SRPM URL:
http://mmaslano.fedorapeople.org/perl-Test-Harness-Straps/perl-Test-Harness-Straps-0.30-1.fc10.src.rpm
Description: 
Test::Harness is limited to printing out its results. This makes analysis
of the test results difficult for anything but a human. To make it easier
for programs to work with test results, we provide Test::Harness::Straps.
Instead of printing the results, straps provide them as raw data. You can
also configure how the tests are to be run.

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

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

[Bug 480859] Review Request: diffuse - graphical diff tool (first package, seeking sponser)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto ita...@ispbrasil.com.br changed:

   What|Removed |Added

 CC||ita...@ispbrasil.com.br
 Depends on||177841




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

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

[Bug 480859] Review Request: diffuse - graphical diff tool (first package, seeking sponser)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
10:11:39 EDT ---
please don't gzip man pages.

gzip -9 %{buildroot}/usr/share/man/man1/diffuse.1

all files in /usr/share/man are magically gziped by rpm

please macronify your %files section

please replace all hardcoded paths below with macros.

/usr/bin/diffuse
/usr/share/diffuse/
/usr/share/applications/diffuse.desktop
/usr/share/gnome/help/diffuse/C/diffuse.xml
/usr/share/man/man1/diffuse.1.gz
/usr/share/omf/diffuse/diffuse-C.omf
/usr/share/pixmaps/diffuse.png

for more info about macros please l@@k


https://fedoraproject.org/wiki/Packaging/RPMMacros

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

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

[Bug 480859] Review Request: diffuse - graphical diff tool (first package, seeking sponser)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Miroslav Lichvar mlich...@redhat.com changed:

   What|Removed |Added

 CC||mlich...@redhat.com




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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #15 from Ralf Corsepius rc040...@freenet.de  2009-01-21 10:15:54 
EDT ---
(In reply to comment #14)

 2 - debug info package is not being created for some reason, we need to figure
 why
The cause is this *.spec not acknowledging RPM_OPT_FLAGS.

In other words: You must somehow propagate $RPM_OPT_FLAGS from inside of the
*.spec into the build-machinery.

One way is to edit compile-cc inside %prep (e.g. using sed).


4. You patch's file name is weird. The standard extensions for patches is
*.diff or *.patch.

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

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

[Bug 478007] Review Request: vmware-requirements - Installs packages needed for VMware's virtualization programs to run

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Denis Leroy de...@poolshark.org changed:

   What|Removed |Added

 CC||de...@poolshark.org




--- Comment #11 from Denis Leroy de...@poolshark.org  2009-01-21 10:25:24 EDT 
---
I use VMWare a lot but I do have a hard time seeing this making it into Fedora
even if brought to FeSCO. David I think it would be more productive to package
this for RpmFusion.

 in real employment world we use xen/kvm/qemu

No offense, but you don't know what you're talking about. VMWare's share of the
virtualization market is about 80%. The rest is Microsoft. The FOSS solutions
are in the noise (unfortunately).

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

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

[Bug 187318] Review Request: mondo

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #48 from Gratien D'haese gratien.dha...@it3.be  2009-01-21 
10:28:53 EDT ---
(In reply to comment #47)
As mention in comment #46:

for file in COPYING INSTALL NEWS README TODO AUTHORS Changelog; do

= only above files should be converted if needed during the build

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #17 from Ralf Corsepius rc040...@freenet.de  2009-01-21 10:42:57 
EDT ---
(In reply to comment #16)

 if you like to contribute with a patch to speed up this review, I believe it
 will welcome
Well, for didactical reasons, I prefer candidate maintainers to write such
things themselves, because Fedora needs maintainers who have an understanding
of what they are doing.

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

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

[Bug 478470] Review Request: mrpt - The Mobile Robot Programming Toolkit (MRPT)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #17 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
10:43:44 EDT ---
For 0.6.5-0.1:

* Tarball
  - For svn repo based tarball, please follow
https://fedoraproject.org/wiki/Packaging/SourceURL#Using_Revision_Control

* BuildRequires
  - Some reviewers may say that BuildRequires: tex(latex) tex(dvips) is
preferred.

(In reply to comment #14)
 btw, if I got it right, a package, say foo-1.2.3-0.1.20090118svn MUST have 
 an
 associated tarball 
 foo-1.2.3.tar.gz, without any indication of the snapshot?? 
Including revision number in tarball name is highly recommeded.

* Dependency for -devel subpackage
  - Requires: opencv is not needed.
  - For this package Requires: opencv-devel does not seem to be needed,
either.

* Compiler flags
  - On some parts Fedora specific compiler flags are not honored.
For example:
--
   179  /usr/bin/cmake -E cmake_progress_report
/builddir/build/BUILD/mrpt-0.6.5/CMakeFiles 
   180  [  1%] 
   181  Building CXX object
otherlibs/aria-2.5.1/src/CMakeFiles/mrpt-aria.dir/ArAction.o
   182  cd /builddir/build/BUILD/mrpt-0.6.5/otherlibs/aria-2.5.1/src 
/usr/bin/c++   -D_FILE_OFFSET_BITS=64 -D_LARGE_FILES -D__WXGTK__
-Dmrpt_aria_EXPORTS -O2 -fPIC -D_REENTRANT -fno-exceptions -pthread -fPIC
-I/usr/include/opencv -isystem /usr/lib/wx/include/gtk2-unicode-release-2.8
-isystem /usr/include/wx-2.8 -I/builddir/build/BUILD/mrpt-0.6.5/.
-I/builddir/build/BUILD/mrpt-0.6.5/include/mrpt-config/unix
-I/builddir/build/BUILD/mrpt-0.6.5/include
-I/builddir/build/BUILD/mrpt-0.6.5/include/mrpt/otherlibs
-I/builddir/build/BUILD/mrpt-0.6.5/include/mrpt/otherlibs/aria   -o
CMakeFiles/mrpt-aria.dir/ArAction.o -c
/builddir/build/BUILD/mrpt-0.6.5/otherlibs/aria-2.5.1/src/ArAction.cpp
--
(I usually check this by seeking for the word FORTIFY)

* Directory ownership issue
  - -doc subpackage should own %{_datadir}/doc/mrpt-doc/ directory.


Also I will wait for your another review request or your pre-review
of other person's review request.

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

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

[Bug 480727] Review Request: daemontools: is a collection of tools for managing UNIX services.

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #16 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
10:27:50 EDT ---
 
  2 - debug info package is not being created for some reason, we need to 
  figure
  why
 The cause is this *.spec not acknowledging RPM_OPT_FLAGS.
 
 In other words: You must somehow propagate $RPM_OPT_FLAGS from inside of the
 *.spec into the build-machinery.
 
 One way is to edit compile-cc inside %prep (e.g. using sed).
 
 
 4. You patch's file name is weird. The standard extensions for patches is
 *.diff or *.patch.

if you like to contribute with a patch to speed up this review, I believe it
will welcome

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

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

[Bug 479903] gdesklets plug-in, Cycle through a collection of pictures

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


MERCIER bioinfornat...@gmail.com changed:

   What|Removed |Added

   Flag|needinfo?(bioinfornat...@gm |
   |ail.com)|




--- Comment #12 from MERCIER bioinfornat...@gmail.com  2009-01-21 11:03:19 
EDT ---
1 and 2/ The new spec file are update. The new src.rpm file are update And the
new rpm file update

rpmlint rpmbuild/SPECS/gdesklet-slideshow.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

rpmlint rpmbuild/RPMS/noarch/gdesklet-slideshow-0.9-1.fc10.noarch.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

rpmlint rpmbuild/SRPMS/gdesklet-slideshow-0.9-1.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.


3/ this is not a GUI but a plugin for gdesklet so they are not .desktop file

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

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

[Bug 473367] Review Request: autotrust - DNSKEY trust anchor update utility that uses RFC-5011

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Paul Wouters p...@xelerance.com  2009-01-21 11:16:07 EDT 
---
Thanks for the addition comments Adam, here is the new release:

Spec URL: ftp://ftp.xelerance.com/autotrust/autotrust.spec
SRPM URL:
ftp://ftp.xelerance.com/autotrust/autotrust-0.2.1-0.1.rc1.fc10.src.rpm

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

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

[Bug 479903] gdesklets plug-in, Cycle through a collection of pictures

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


MERCIER bioinfornat...@gmail.com changed:

   What|Removed |Added

 Blocks||177841




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

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

[Bug 473367] Review Request: autotrust - DNSKEY trust anchor update utility that uses RFC-5011

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Adam Tkac at...@redhat.com  2009-01-21 11:35:59 EDT ---
(In reply to comment #9)
 Thanks for the addition comments Adam, here is the new release:
 

Well, package looks better but I still can't give you review, two problems are
remaining:

autotrust.x86_64: W: incoherent-version-in-changelog 0.2.1-0
- please use 0.2.1-0.1.rc1 instead

Drop --sysconfdir configure parameter, it is not needed (as written in comment
#5)

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

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

[Bug 479903] gdesklets plug-in, Cycle through a collection of pictures

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from MERCIER bioinfornat...@gmail.com  2009-01-21 11:35:40 
EDT ---
rpm and src.rpm for F9:
http://bioinformatiques.free.fr/gdesklet-slideshow-0.9-2.fc9.noarch.rpm
http://bioinformatiques.free.fr/gdesklet-slideshow-0.9-2.fc9.src.rpm

-

rpm and src.rpm for F10:
http://bioinformatiques.free.fr/gdesklet-slideshow-0.9-2.fc10.noarch.rpm
http://bioinformatiques.free.fr/gdesklet-slideshow-0.9-2.fc10.src.rpm

The spec file
http://bioinformatiques.free.fr/gdesklet-slideshow.spec

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

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

[Bug 479796] Review Request: ghc-haskell-src-exts - Library for Manipulating Haskell source

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #5 from Conrad Meyer kon...@tylerc.org  2009-01-21 11:47:26 EDT 
---
Updated; see:
http://konradm.fedorapeople.org/fedora/SPECS/ghc-haskell-src-exts.spec
http://konradm.fedorapeople.org/fedora/SRPMS/ghc-haskell-src-exts-0.4.8-2.fc9.src.rpm

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

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

[Bug 480999] Review Request: python-webflash - Portable flash messages for WSGI apps

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

 Blocks||468230




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

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

[Bug 468230] Review Request: TurboGears2 - Next generation Front-to-back web development megaframework built on Pylons

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Luke Macken lmac...@redhat.com changed:

   What|Removed |Added

 Depends on||480999




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

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

[Bug 480999] New: Review Request: python-webflash - Portable flash messages for WSGI apps

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: python-webflash - Portable flash messages for WSGI apps

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

   Summary: Review Request: python-webflash - Portable flash
messages for WSGI apps
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lmac...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://lmacken.fedorapeople.org/python-webflash.spec
SRPM URL:
http://lmacken.fedorapeople.org/rpms/python-webflash-0.1-0.1.a8.fc10.src.rpm
Description:
WebFlash is a library to display flash messages in python web applications.
These messages are usually used to provide feedback to the user (eg: you
changes have been saved, your credit card number has been stolen, ...). One
important characteristic they must provide is the ability to survive a redirect
(ie: display the message in a page after being redirected from a form
submission).

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

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

[Bug 460894] Review Request: ocsigen - Web programming framework

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Stéphane Glondu st...@glondu.net changed:

   What|Removed |Added

 CC||st...@glondu.net




--- Comment #11 from Stéphane Glondu st...@glondu.net  2009-01-21 12:09:09 
EDT ---
There might be problems building with OCaml 3.11.0, see:

  http://caml.inria.fr/mantis/view.php?id=4699

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

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

[Bug 481004] New: Review Request: pg_top - top for postgresql

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: pg_top - top for postgresql

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

   Summary: Review Request: pg_top - top for postgresql
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: ita...@ispbrasil.com.br
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://ispbrasil.com.br/pg_top/pg_top.spec
SRPM URL: http://ispbrasil.com.br/pg_top/pg_top-3.6.2-2.fc11.src.rpm
Description: 

pg_top is 'top' for PostgreSQL processes. See running queries,
query plans, issued locks, and table and index statistics.

koji scratch build

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

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

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

[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
12:29:01 EDT ---
I don't unterstand you issue: 

I have done the following:
1.) Build your package locally.
2.) Install it
3.) Start blender, so the wrapper script should create the symlinks
4.) View the ~/.blender/scripts directory

I have got the following expected result:

lrwxrwxrwx 1 s4504kr s4504kr   42 21. Jan 18:24 LuxBlend_0.1.py -
/usr/share/blender/scripts/LuxBlend_0.1.py
lrwxrwxrwx 1 s4504kr s4504kr   43 21. Jan 18:24 LuxBlend_0.1.pyc -
/usr/share/blender/scripts/LuxBlend_0.1.pyc
lrwxrwxrwx 1 s4504kr s4504kr   43 21. Jan 18:24 LuxBlend_0.1.pyo -
/usr/share/blender/scripts/LuxBlend_0.1.pyo

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

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

[Bug 480567] Review Request: dnssec-conf - DNSSEC and DLV configuration and priming tool

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #3 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
12:41:10 EDT ---
Bad:
- The current upstream release is 1.12.
- License tag was not changed from GPLv2+ to GPLv2
I couldN't download 1.13 via spectool -g
- Please write $$RPM_BUILD_ROOT and %%{buildroot} in the %changelog stanza for
excaping the special chars '$' and '%'

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

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

[Bug 480528] Review Request: botan - Crypto library written in C++

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #6 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
12:42:23 EDT ---
I wan't to notify you, that 1.8.1 was released.

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

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

[Bug 479585] Review Request: megaupload-dl - Megaupload automatic downloader

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #20 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
12:45:04 EDT ---
PIng.

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

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

[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Nicolas Chauvet (kwizart) kwiz...@gmail.com  2009-01-21 
12:44:49 EDT ---
that's not what will append for real:
1- As the per user ~/.blender directory is already created no more files will
be copied to the user directory.
2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin expects
is to write in its config directory and in its content directory. (see the
readme.txt).
Please note that the permission you have showed are the perm of the symlinks,
not the perms of the files.

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

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

[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #11 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
12:53:20 EDT ---
(In reply to comment #10)
 that's not what will append for real:
 1- As the per user ~/.blender directory is already created no more files will
 be copied to the user directory.

Sorry, On eatch start of blender there will be creating symlink from
~/.blnder/scripts to /usr/share/blender/scripts.

 2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin expects
 is to write in its config directory and in its content directory. (see the
 readme.txt).

I can't find your readme.txt file

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

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

[Bug 481009] New: Review request: pothana2000-fonts - Unicode compliant OpenType font for Telugu

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review request: pothana2000-fonts - Unicode compliant OpenType font 
for Telugu

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

   Summary: Review request: pothana2000-fonts - Unicode compliant
OpenType font for Telugu
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: sshed...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Description:

Its Free OpenType font for Telugu created by
Dr. Tirumala Krishna Desikacharyulu. Font is 
Unicode version 5.1 compliant. 

SPEC: http://rakesh.gnulinuxcentar.org/pothana2000-fonts.spec
SRPM: rakesh.gnulinuxcentar.org/pothana2000-fonts-1.3-1.fc11.src.rpm

The source had a binary file called Pothana2k.exe which I removed and re
compressed the source to tar.gz using generate-tarball.sh. I have kept this
shell script as Source2.

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

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

[Bug 481009] Review request: pothana2000-fonts - Unicode compliant OpenType font for Telugu

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from sandeep shedmake sshed...@redhat.com  2009-01-21 13:01:46 
EDT ---
Updated SRPM url
http://rakesh.gnulinuxcentar.org/pothana2000-fonts-1.3-1.fc11.src.rpm

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

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

[Bug 479832] Review Request: mmpong - a massively multiplayer pong game

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-21 
13:03:58 EDT ---
Well, rpmlint rpmname should work and checking your rpms
by rpmlint is always recommended.


For 0.9-2
- BuildRequires: gcc is redundant:
  https://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

- I think people don't expect that $ yum install mmpong will
  install development packages (i.e. -devel).
  Rather I think that mmpong-client-caca should be renamed to
  mmpong.

- By the way Requires: %{name}-libs=%{version}-%{release}
  will seek for a rpm package with the name 
  %{name}-libs=%{version}-%{release} (not %{name}-libs), so
  anyway this needs fixing (I guess if you try to install
  this rpm you will see such errors.)

- The Group Libraries is not common. You can see the Group
  tag which Fedora regards as valid by:
  $ rpmlint -I non-standard-group

- Summary should begin with a capital letter.

- Usually dependencies within packages rebuilt from a srpm
  must be exact EVR (Epoch-Version-Release) specific, not
  with inequality

- For -client-gl subpackage, Requires: libgl is not needed.
  Such libraries based dependency is automatically detected by
  rpmbuild itself and rpmbuild correctly adds such dependency
  to rebuilt binary rpms.

- Also, would you explain why the dependency for libcaca(-devel) is
  EVR specific? (If there is some reason, please write
  it as comments in the spec file)

- %cmake macro already sets -DCMAKE_INSTALL_PREFIX:PATH, 
  -DCMAKE_INSTALL_LIBDIR:PATH (and -DLIB_SUFFIX=64 for 64 bits
  architecture), and CFLAGS and so on.
  Would you check if -DCMAKE_INSTALL_PREFIX=%{_prefix} 
  in your spec file really needed with using %cmake macro?

  By the way what %cmake macro does can be shown by
  $ rpm --eval %cmake . Also please check
  https://fedoraproject.org/wiki/Packaging/cmake

- On Fedora %run_ldconfig macro is not defined
  (Is this really working on your Fedora system?)

  (Also, as currently mmpong package is just empty, no scriptlets
   is needed... however also please see my suggestion about renaming
   -client-caca subpackage for this issue)

- For -libs subpackage, /sbin/ldconfig must be called on %post{,un}
  scriptlets. Add
-
%post libs -p /sbin/ldconfig
-
  for example.
  https://fedoraproject.org/wiki/Packaging/Guidelines#Shared_Libraries

- Any packages containing pkgconfig .pc files must have
  Requires: pkgconfig.
  https://fedoraproject.org/wiki/Packaging/Guidelines#Pkgconfig_Files

- By the way even on x86_64, mmpong.pc contains:
-
 3  libdir=${exec_prefix}/lib
-
  This is not correct because on 64 bits architecture libdir should
  point to /usr/lib64, not /usr/lib.

- The directory %{_includedir}/mmpong must be owned by -devel
  subpackage:
 
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
 
https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Wildcarding_Files_inside_a_Created_Directory

- Directories for installing man files are not right.
  For example, man files with suffix 6 should be installed under
  %{_mandir}/man6 (then rpmbuild automatically gzips this file)

- Again desktop files to be installed must be treated by
  desktop-file-{install,validate}: see
 
https://fedoraproject.org/wiki/Packaging/Guidelines#desktop-file-install_usage
  Currently mmpong-gl.desktop is a invalid desktop file
  and need fixing by desktop-file-install.

- -client-gl package must not own %_datadir/icons/hicolor
   directory and so on (also here please check
  
https://fedoraproject.org/wiki/Packaging/Guidelines#File_and_Directory_Ownership
   and check what directories are owned by hicolor-icon-theme)

- Usually daemon executables are installed under %{_sbindir} (i.e.
  /usr/sbin ) and some initscript is provided.
  * Would you check if mmpongd should be under %_bindir or
%_sbindir ? (I don't know well about mmpongd, however I guess
this should be under %_sbindir)
  * Also would you consider to supply sysv initscripts for mmpongd?
https://fedoraproject.org/wiki/Packaging/SysVInitScript

- As -client-gl installs some icons under %_datadir/icons/hicolor,
  please check
  https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#GTK.2B_icon_cache

- Please create %changelog
  https://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs

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

___
Fedora-package-review mailing list
Fedora-package-review@redhat.com

[Bug 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed:

   What|Removed |Added

Summary|gdesklets plug-in, Cycle|Review Request:
   |through a collection of |gdesklet-slideshow - Cycle
   |pictures|through a collection of
   ||pictures




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

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

[Bug 458090] Review Request: LuxRender - Lux Renderer, an unbiased rendering system

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Nicolas Chauvet (kwizart) kwiz...@gmail.com  2009-01-21 
13:16:43 EDT ---
(In reply to comment #11)
 (In reply to comment #10)
  that's not what will append for real:
  1- As the per user ~/.blender directory is already created no more files 
  will
  be copied to the user directory.
 
 Sorry, On eatch start of blender there will be creating symlink from
 ~/.blnder/scripts to /usr/share/blender/scripts.
It depends on where you consider your error is, so as you like: but quoting
blender-wrapper:
When  if [ ! -d ~/.blender ]; then 
install -d ~/.blender/scripts
ln -sf /usr/share/blender/scripts/bpydata ~/blender/scripts/bpydata
^^ this part is wrong - and a misunderstanding of what bpydata and
bpydata/config directories are
(namely the content and configuration directories for scripts that are meant to
be user writable).
But there is a typo (blender instead of .blender) so it doesn't work anyway.

Then,:
rm -r ~/.blender/scripts/*.p*
Why end-users aren't allowed to have their own scripts ?


  2- Nobody wants to write on LuxBlend_0.1.py?. Instead what the plugin 
  expects
  is to write in its config directory and in its content directory. (see the
  readme.txt).
 
 I can't find your readme.txt file
rpm -ql blender |grep readme.txt


I'm still unsure of the accurate solution, so it is just workinprogress
thoughts.

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

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

[Bug 479585] Review Request: megaupload-dl - Megaupload automatic downloader

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #21 from Jochen Schmitt joc...@herr-schmitt.de  2009-01-21 
14:01:10 EDT ---
Created an attachment (id=329637)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=329637)
blender wrapper script

I have create a fixed release of the blender-wrapper script. It may be nice, if
you can try out this script.

If the script is ok, I will include it in the next release of the blender
package.

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

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

[Bug 481004] Review Request: pg_top - top for postgresql

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||tcall...@redhat.com
 AssignedTo|nob...@fedoraproject.org|tcall...@redhat.com
   Flag||fedora-review?




--- Comment #1 from Tom spot Callaway tcall...@redhat.com  2009-01-21 
14:12:59 EDT ---
Review
===

Good:

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

Bad:

* Package is not using %{optflags} during compilation.
* There is no packaged documentation aside from the manpage
* The license text is not packaged as %doc
* %defattr(-,root,root) is used instead of %defattr(-,root,root,-)
* Patch1 is completely pointless.
* Missing BuildRequires: elfutils-libelf-devel

Please fix these issues and show a new SRPM.

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

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

[Bug 481022] Review Request: SPE it is a Python IDE editor released

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


MERCIER bioinfornat...@gmail.com changed:

   What|Removed |Added

Summary|SPE it is a Python IDE  |Review Request: SPE it is a
   |editor released |Python IDE editor released




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

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

[Bug 481022] New: SPE it is a Python IDE editor released

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: SPE it is a Python IDE editor released

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

   Summary: SPE it is a Python IDE editor released
   Product: Fedora
   Version: 10
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: bioinfornat...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
  Clone Of: https://bugzilla.redhat.com/show_bug.cgi?id=444134


Spec URL: http://bioinformatiques.free.fr/spe.spec
SRPM URL: http://bioinformatiques.free.fr/spe-0.8.4.h-3.fc10.src.rpm
Description: SPE it is a Python IDE editor released
__
$ rpmlint rpmbuild/SPECS/spe.spec 
0 packages and 1 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/SRPMS/spe-0.8.4.h-3.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

$ rpmlint rpmbuild/RPMS/noarch/spe-0.8.4.h-3.fc10.noarch.rpm 
spe.noarch: E: wrong-script-interpreter
/usr/lib/python2.5/site-packages/spe/_spe/Blender_signature.py BPY
spe.noarch: E: wrong-script-interpreter
/usr/lib/python2.5/site-packages/spe/_spe/spe_blender.py BPY
spe.noarch: E: wrong-script-interpreter
/usr/lib/python2.5/site-packages/spe/_spe/winpdb_blender.py BPY
1 packages and 0 specfiles checked; 3 errors, 0 warnings.


rpmlint don't know the shebang #!BPY This shebang makes it
possible Blender to recognize script as a plugins

#!BPY  It tells Blender that this is a Blender script, and therefore it will
consider it when scanning for scripts.
http://en.wikibooks.org/wiki/Blender_3D:_Noob_to_Pro/Advanced_Tutorials/Python_Scripting/Export_scripts

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

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

[Bug 480887] Review Request: kguitar - Guitar Tabulature Music Editor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-21 
14:50:38 EDT ---
Please use KGuitar is a powerful ...I am not a native speaker but AFAIK it's
a bit closer to correct English.

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

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

[Bug 480859] Review Request: diffuse - graphical diff tool

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

Summary|Review Request: diffuse -   |Review Request: diffuse -
   |graphical diff tool  (first |graphical diff tool
   |package, seeking sponser)   |
  Alias||diffuse




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

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

[Bug 481030] New: Review Request: pmd-emacs - an interface to PMD for (X)Emacs

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: pmd-emacs - an interface to PMD for (X)Emacs

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

   Summary: Review Request: pmd-emacs - an interface to PMD for
(X)Emacs
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: loganje...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://jjames.fedorapeople.org/pmd-emacs/pmd-emacs.spec
SRPM URL: http://jjames.fedorapeople.org/pmd-emacs/pmd-emacs-0.6-1.src.rpm
Description: pmd-emacs supports the invocation of PMD from within (X)Emacs. 
Specifically, it supplies function pmd-current-buffer and pmd-current-dir, and
enables parsing of the resulting list of PMD complaints so that, for example,
next-error will walk through the list correctly.

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

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

[Bug 480887] Review Request: kguitar - Guitar Tabulature Music Editor

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-01-21 
15:25:50 EDT ---
Right. Will do.

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

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

[Bug 481004] Review Request: pg_top - top for postgresql

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #2 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
15:26:06 EDT ---
done.

http://ispbrasil.com.br/pg_top/pg_top.spec
http://ispbrasil.com.br/pg_top/pg_top-3.6.2-3.fc10.src.rpm

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

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

[Bug 481034] New: Review Request: coccinelle - Semantic patching for Linux (spatch)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: coccinelle - Semantic patching for Linux (spatch)

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

   Summary: Review Request: coccinelle - Semantic patching for
Linux (spatch)
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: rjo...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://www.annexia.org/tmp/ocaml/coccinelle.spec
SRPM URL: http://www.annexia.org/tmp/ocaml/coccinelle-0.1.4-2.fc10.src.rpm
Description: 
Coccinelle is a tool to utilize semantic patches for manipulating C
code.  It was originally designed to ease maintenance of device
drivers in the Linux kernel.

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

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

[Bug 481034] Review Request: coccinelle - Semantic patching for Linux (spatch)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Richard W.M. Jones rjo...@redhat.com changed:

   What|Removed |Added

URL||http://www.emn.fr/x-info/co
   ||ccinelle/




--- Comment #1 from Richard W.M. Jones rjo...@redhat.com  2009-01-21 15:46:38 
EDT ---
Home page is http://www.emn.fr/x-info/coccinelle/
and there is a LWN article about the tool here:
  http://lwn.net/Articles/315686/

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

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

[Bug 480870] Review Request: crda - An 802.11 wireless regulatory compliance daemon

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bill Nottingham nott...@redhat.com changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|nott...@redhat.com
   Flag||fedora-review?




--- Comment #2 from Bill Nottingham nott...@redhat.com  2009-01-21 15:49:29 
EDT ---
MUST items:

- Package meets naming and packaging guidelines - OK
- Spec file matches base package name. - OK
- Spec has consistent macro usage. - ***

The 'v' in the version seems extraneous. In fact, there seem to be identical
tarballs on the upstream download site with and without the 'v'.

- Meets Packaging Guidelines. - OK

- License - BSD ***

The code doesn't specify a license, so it's assumed from the included LICENSE
file. Would be nice if the code made it explicit.

Given the keys/signing involved in the build process, it would have been highly
entertaining if this package was GPL3. Alas, it is not.

- License field in spec matches - OK
- License file included in package - OK
- Spec in American English - OK
- Spec is legible  - OK ***

'An' in the summary is probably superfluous.

- Sources match upstream md5sum: - OK

- Package needs ExcludeArch - N/A
- BuildRequires correct - OK
- Package has %defattr and permissions on files is good. - OK
- Package has a correct %clean section. - OK
- Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) - OK
- Package is code or permissible content. - OK
- Doc subpackage needed/used. - ***

There are READMEs for both the daemon and the wireless regdb that should
probably be in %doc.

- Packages %doc files don't affect runtime. - OK

- Headers/static libs in -devel subpackage. - N/A

- Package compiles and builds on at least one arch. - OK (tested rawhide
x86_64, F10 x86)
- Package has no duplicate files in %files. - OK
- Package doesn't own any directories other packages own. - OK
- Package owns all the directories it creates. - ***

Should own /usr/lib/crda

- No rpmlint output. - ***

crda.i386: W: non-conffile-in-etc /etc/udev/rules.d/regulatory.rules

See below about udev rules.

crda.i386: E: only-non-binary-in-usr-lib

Can be ignored.

- final provides and requires are sane: - ***

Arguably, should require udev.

SHOULD Items:

- Should build in mock. - OK (tested i386)
- Should build on all supported archs - didn't test
- Should function as described. - didn't test, don't have appropriate drivers
- Should have sane scriptlets. - N/A
- Should have dist tag - OK
- Should package latest version - OK

Random notes not covered above:
- We end up building two different upstreams into one package here. It could be
done with the wireless db built as a separate package, that includes its pubkey
in /etc/pki for the daemon to build later. If the daemon and the regulatory
information are going to be updated asynchronously, that might be worthwhile

- system udev rules should go in /lib/udev/rules.d, and usually are named
XX-regulatory.rules, where XX is some relative numeric priority

- the regulatory db is on /usr ... will it be needed before /usr is mounted?

- the usage flow seems to be 'user sets a domain - netlink message to kernel
- uevent - udev helper to daemon - reads database - new netlink message to
kernel - implements restrictions in driver. Seems overly convoluted, but..
meh.

- we remove the upstream key, and sign the regulatory db with our own key,
generated at build time (and then thrown away). As I understand it, this means
users won't be able to drop in new upstream releases of the regulatory db. Is
this intentional?

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

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

[Bug 481004] Review Request: pg_top - top for postgresql

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Tom spot Callaway tcall...@redhat.com changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #3 from Tom spot Callaway tcall...@redhat.com  2009-01-21 
15:52:21 EDT ---
Looks good, approved.

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

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

[Bug 481004] Review Request: pg_top - top for postgresql

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Itamar Reis Peixoto ita...@ispbrasil.com.br changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #4 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
16:00:26 EDT ---
New Package CVS Request
===
Package Name: pg_top
Short Description: pg_top is 'top' for PostgreSQL processes
Owners: itamarjp
Branches: F-10 F-9
InitialCC:

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

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

[Bug 456190] Review Request: dosemu - dos emulator

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #22 from Justin Zygmont solarflo...@gmail.com  2009-01-21 
16:04:45 EDT ---
I tried following all the sketchy guidelines, and following your advise, etc. 
I'm ready to just leave it with comments like this, I can build a fedora RPM.

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

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

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

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #51 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-21 
16:13:43 EDT ---
I can't answer this question for Devrim, but 2 years is alot of time.

Devrim have your own repo.

http://yum.pgsqlrpms.org/

and It's have a slony 2.0 rpm in his repo.

http://yum.pgsqlrpms.org/srpms/8.3/fedora/fedora-10-x86_64/

I don't know if him want to continue, but if he is not available I want to
contribute to have slony imported into fedora.




(In reply to comment #50)
 Devrim, what's up with this package, do you still want to proceed.
 I'm running out of patience after two years ;-)

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

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

[Bug 481040] New: Review Request: skyeye - integrated simulation environment for typical Embedded Computer Systems

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.

Summary: Review Request: skyeye - integrated simulation environment for typical 
Embedded Computer Systems

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

   Summary: Review Request: skyeye - integrated simulation
environment for typical Embedded Computer Systems
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: wo...@nobugconsulting.ro
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://wolfy.fedorapeople.org/skyeye/skyeye.spec
SRPM URL: http://wolfy.fedorapeople.org/skyeye/skyeye-1.2.6-2.rc1.fc7.src.rpm
Description: Originating from GDB/Armulator, the goal of SkyEye is to provide
an integrated simulation environment for typical Embedded Computer Systems. Now
it supports a series ARM architecture based microprocessors and Blackfin DSP
Processor. You can run some Embedded Operation System such as ARM Linux,
uClinux, uc/OS-II (ucos-ii) etc. in SkyEye, and analysis or debug them at
source level.

koji scratch build at:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1072452
EL-5 build in use for a couple of days in the company I work for.

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

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

[Bug 481040] Review Request: skyeye - integrated simulation environment for typical Embedded Computer Systems

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


manuel wolfshant wo...@nobugconsulting.ro changed:

   What|Removed |Added

URL||http://www.skyeye.org/index
   ||.shtml
 CC||cgoo...@yahoo.com.au




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

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

[Bug 480528] Review Request: botan - Crypto library written in C++

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #7 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:25:31 EDT ---
botan-1.8.0-2.fc9 has been pushed to the Fedora 9 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update botan'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/F9/FEDORA-2009-0698

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

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

[Bug 480380] Review Request: python-epdb - extended python debugger

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:28:53 EDT ---
python-epdb-0.11-1.fc9 has been pushed to the Fedora 9 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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

[Bug 480380] Review Request: python-epdb - extended python debugger

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:26:47 EDT ---
python-epdb-0.11-1.fc10 has been pushed to the Fedora 10 stable repository.  If
problems still persist, please make note of it in this bug report.

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

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

[Bug 467388] Review Request: mingw32-pdcurses - Curses library for MinGW

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #13 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:30:50 EDT ---
mingw32-pdcurses-3.4-3.fc10 has been pushed to the Fedora 10 stable repository.
 If problems still persist, please make note of it in this bug report.

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

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

[Bug 467398] Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #23 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:30:16 EDT ---
mingw32-gettext-0.17-7.fc10 has been pushed to the Fedora 10 stable repository.
 If problems still persist, please make note of it in this bug report.

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

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

[Bug 473412] Review Request: kopete-cryptography - Encrypts and signs messages in Kopete using the OpenPGP

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 467398] Review Request: mingw32-gettext - GNU libraries and utilities for producing multi-lingual messages

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 480646] Review Request: urlwatch - A tool for monitoring webpages for updates

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #10 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:27:38 EDT ---
urlwatch-1.7-2.fc9 has been pushed to the Fedora 9 testing repository.  If
problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing-newkey update urlwatch'.  You can
provide feedback for this update here:
http://admin.fedoraproject.org/updates/F9/FEDORA-2009-0731

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

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

[Bug 478640] Review Request: mingw32-dlfcn - Implements a wrapper for dlfcn (dlopen dlclose dlsym dlerror)

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:28:26 EDT ---
mingw32-dlfcn-0-0.3.r11.fc10 has been pushed to the Fedora 10 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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

[Bug 480380] Review Request: python-epdb - extended python debugger

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 467388] Review Request: mingw32-pdcurses - Curses library for MinGW

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Fedora Update System upda...@fedoraproject.org changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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

[Bug 473412] Review Request: kopete-cryptography - Encrypts and signs messages in Kopete using the OpenPGP

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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





--- Comment #34 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:31:26 EDT ---
kdenetwork-4.1.4-2.fc10, kdeaccessibility-4.1.4-1.fc10, kdeadmin-4.1.4-1.fc10,
kdeartwork-4.1.4-1.fc10, kdebase-runtime-4.1.4-1.fc10,
kdebase-workspace-4.1.4-1.fc10, kdebindings-4.1.4-1.fc10, kdeedu-4.1.4-1.fc10,
kde-filesystem-4-23.fc10, kdegames-4.1.4-1.fc10, kdegraphics-4.1.4-1.fc10,
kdelibs-4.1.4-2.fc10, kde-l10n-4.1.4-2.fc10, kdemultimedia-4.1.4-1.fc10,
kdepim-4.1.4-1.fc10, kdepimlibs-4.1.4-1.fc10, kdeplasma-addons-4.1.4-1.fc10,
kdesdk-4.1.4-1.fc10, kdetoys-4.1.4-1.fc10, kdeutils-4.1.4-1.fc10,
kopete-cryptography-1.3.0-8.fc10, kdebase-4.1.4-2.fc10 has been pushed to the
Fedora 10 stable repository.  If problems still persist, please make note of it
in this bug report.

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

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

[Bug 468376] Review Request: mingw32-SDL - MinGW Windows port of SDL cross-platform multimedia library

[bugzilla]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


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


Bug 468376 depends on bug 478640, which changed state.

Bug 478640 Summary: Review Request: mingw32-dlfcn - Implements a wrapper for 
dlfcn (dlopen dlclose dlsym dlerror)
https://bugzilla.redhat.com/show_bug.cgi?id=478640

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #11 from Fedora Update System upda...@fedoraproject.org  
2009-01-21 16:28:33 EDT ---
mingw32-SDL-1.2.13-4.fc10 has been pushed to the Fedora 10 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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