[Bug 230586] Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs

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

Summary: Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of 
xdg-user-dirs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 03:09 EST ---
New versions at:
http://people.redhat.com/alexl/temp/xdg-user-dirs-gtk.spec
http://people.redhat.com/alexl/temp/xdg-user-dirs-gtk-0.1-2.fc7.src.rpm


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

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


[Bug 229154] Review Request: konwert - Converter of character encodings

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

Summary: Review Request: konwert - Converter of character encodings


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 04:29 EST ---
Can't reproduce the problem on my machine.

~Daniil.

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

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


[Bug 230586] Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs

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

Summary: Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of 
xdg-user-dirs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 05:41 EST ---
GOOD

- rpmlint checks do not return anything either on source or on binary package
- package meets naming guidelines
- package meets packaging guidelines
- license ( GPL ) OK, text in %doc, matches source
- spec file legible, in am. english
- source matches upstream, sha1sum 
09ccb33dd147d969ec33a9e3d08daa1ca0c37202  xdg-user-dirs-gtk-0.1.tar.gz
- package compiles on devel (x86_64)
- no missing BR
- MINOR: unnecessary BR: pkgconfig is pulled in by gtk2-devel
- no locales, despite what the build log seems to say
- not relocatable
- owns all files/directories that it creates, does not take ownership of foreign
files/directories
- no duplicate files
- permissions ok
- %clean ok
- macro use consistent
- code, not content
- no need for -docs
- nothing in %doc affects runtime
- MUSTFIX the .desktop file does not contain all needed entries:
/usr/bin/desktop-file-validate *desktop
user-dirs-update-gtk.desktop: error: required key Type not found
user-dirs-update-gtk.desktop: warning: non-standard key OnlyShownIn lacks the
X- prefix
According to http://fedoraproject.org/wiki/Packaging/Guidelines#desktop there
are a few missing entries
- MUSTFIX: still according to
http://fedoraproject.org/wiki/Packaging/Guidelines#desktop, It is not simply
enough to just include the .desktop file in the package, one MUST run
desktop-file-install in %install (and have BuildRequires: desktop-file-utils),
to help ensure .desktop file safety and spec-compliance.
- no .la, .pc, static content
- devel not needed


SHOULD

- builds without problems in mock/devel/x86_64
- does not segfault at run. I am not a Gnome user, so I cannot really test if it
creates what it supposed to create; a dry run in strace seems to appear OK


Please fix the desktop issues (I suggest either a patch or going with the
simpler version of a separate file included as Source1) and I will approve the
package. You might also remove pkgconfig from BR if you wish since it's not
really needed to be included separately.


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

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


[Bug 229154] Review Request: konwert - Converter of character encodings

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

Summary: Review Request: konwert - Converter of character encodings


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 05:53 EST ---
Xavier, would you review this review again?

Note: I cannot still approve this request.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 06:06 EST ---
Created an attachment (id=149100)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=149100action=view)
build log on ppc64 using the last src rpm

build log on ppc64

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

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


[Bug 230586] Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs

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

Summary: Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of 
xdg-user-dirs


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:03 EST ---
I fixed the desktop file upstream and made a new release:
http://people.redhat.com/alexl/temp/xdg-user-dirs-gtk.spec
http://people.redhat.com/alexl/temp/xdg-user-dirs-gtk-0.2-1.fc7.src.rpm

However, i disagree with using desktop-file-install. I'm using a desktop file as
per the autostart spec, not for something in the menus, so the guidelines don't
really affect this use. (And in fact, it would be pretty weird to use it in this
case.)

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

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


[Bug 226129] Merge Review: man-pages-ru

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

Summary: Merge Review: man-pages-ru


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:08 EST ---
Fixed in man-pages-ru-0.97-2.fc7.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:17 EST ---
(In reply to comment #58)
 becouse latest perl sucks ...
 filled bug report.
 
ok perl is solved nowdays perl-devel is required.

Now python sucks it report wrong lookup paths (old 2.4) instead of 2.5 ...
I still fight to look into, so will come up with -11 shortly.


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

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


[Bug 187621] Review Request: blam

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

Summary: Review Request: blam


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:22 EST ---
After some consideration I've come to the conclusion that I'm really lazy: I'm
hereby giving up any claim of maintainership of blam. You take it!

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

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


[Bug 226128] Merge Review: man-pages-pl

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

Summary: Merge Review: man-pages-pl


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:28 EST ---
Fixed in man-pages-pl-0.24-3.fc7.

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

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


[Bug 230586] Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of xdg-user-dirs

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

Summary: Review Request: xdg-user-dirs-gtk - gnome/gtk+ integration of 
xdg-user-dirs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:42 EST ---
Except for Source0 which is wrong (you have left 0.1 instead of 0.2) the rest is
OK. New sha1sum for source file is
4184311e587a49b73a1fc5a00811bcc2d2df2fa7  /tmp/xdg-user-dirs-gtk-0.2.tar.gz
4184311e587a49b73a1fc5a00811bcc2d2df2fa7  
/home/wolfy/xdg-user-dirs-gtk-0.2.tar.gz


Your choice of using the desktop file seems (to me at least) to correspond with
http://standards.freedesktop.org/autostart-spec/autostart-spec-0.5.html#id2452086
so I guess that in this particular case the guidelines do not apply.

Package APPROVED. Please do not forget to fix Source0 before uploading to cvs.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:43 EST ---
(In reply to comment #62)
 (In reply to comment #60)
 
  one nit:
  /var/tmp/rpm-tmp.60412: line 48: ogdi-config: command not found
  
  You havent build this in mock again !!!
  
  can you show me output of this:
  yum list installed | grep e2fsprogs-devel
  
  Please wait for -10 becouse i need update this package for latest python 
plus 
  perl changes in devel.
 
  yum list installed | grep e2fsprogs-devel
 e2fsprogs-devel.ppc641.39-7.fc6 installed 
   

ah you miss this 32 bit lib !
'yum install e2fsprogs-devel.ppc'
please install it and than post result.

But i would like remind you to build it from scratch in mock session.




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

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


[Bug 226795] Review Request: sdcc - Small Device C Compiler

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

Summary: Review Request: sdcc - Small Device C Compiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:52 EST ---
Done

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

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


[Bug 230324] Review Request: avrdude -Software for programming Atmel AVR Microcontroller

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

Summary: Review Request: avrdude -Software for programming Atmel AVR 
Microcontroller


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:55 EST ---
Done

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

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


[Bug 226122] Merge Review: man-pages-da

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

Summary: Merge Review: man-pages-da


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 07:58 EST ---
Fixed in man-pages-da-0.1.1-13.fc7.

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

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


[Bug 229476] Review Request: xblast - Lay bombs and Blast the other players of the field (SDL version)

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

Summary: Review Request: xblast - Lay bombs and Blast the other players of the 
field (SDL version)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:01 EST ---
CVS done

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

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


[Bug 229477] Review Request: xblast-data - Data files for the game xblast

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

Summary: Review Request: xblast-data - Data files for the game xblast


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:01 EST ---
CVS done

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

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


[Bug 225957] Merge Review: k3b

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

Summary: Merge Review: k3b


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:04 EST ---
err... 
$ brew latest-pkg dist-fc7 k3b
Build Tag   Built by
    
k3b-1.0.0-0.rc6.1.fc7 dist-fc7  harald


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

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


[Bug 225957] Merge Review: k3b

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

Summary: Merge Review: k3b


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




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

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


[Bug 230585] Review Request: xdg-user-dirs - Handling of well known folders in homedir

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

Summary: Review Request: xdg-user-dirs - Handling of well known folders in 
homedir


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:06 EST ---
CVS done

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

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


[Bug 229927] Review Request: mecab - Yet Another Part-of-Speech and Morphological Analyzer

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

Summary: Review Request: mecab - Yet Another Part-of-Speech and Morphological 
Analyzer


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:18 EST ---
Hans, will you have a time to review mecab-jumandic again?

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

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


[Bug 226121] Merge Review: man-pages-cs

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

Summary: Merge Review: man-pages-cs


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:21 EST ---
Fixed in man-pages-cs-0.16-7.fc7.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:25 EST ---
updated
Spec URL: http://openrisc.rdsor.ro/gdal.spec
SRPM URL: http://openrisc.rdsor.ro/gdal-1.4.0-10.src.rpm

should build x86_64, i386 in mock.

oh, please do 'setarch i686' before launch your mock session for i386 ...




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

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


[Bug 226195] Merge Review: newt

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

Summary: Merge Review: newt


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|MODIFIED




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:33 EST ---
Ok, should be fixed in newt-0.52.6-1.fc7.

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

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


[Bug 225237] Merge Review: acpid

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

Summary: Merge Review: acpid


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||RAWHIDE




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:34 EST ---
Ok, closing it as RAWHIDE.

Read ya, Phil



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

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


[Bug 225957] Merge Review: k3b

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

Summary: Merge Review: k3b


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:37 EST ---
Hrm, looking closer, turns out besides .la files, Ed didn't have any
suggestions.  Ed, anything else blocking approval here?

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

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


[Bug 208737] Review Request: ivman - Generic handler for HAL events

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

Summary: Review Request: ivman - Generic handler for HAL events


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 08:38 EST ---
When orphaning, is the same procedure to be followed?
I mean the package is not retired, simply orphaned.

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

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


[Bug 230710] New: Review Request: boolstuff - Disjunctive Normal Form boolean expression library

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




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

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


Spec URL: http://www.environnement.ens.fr/perso/dumas/fc-srpms/boolstuff.spec
SRPM URL: 
http://www.environnement.ens.fr/perso/dumas/fc-srpms/boolstuff-0.1.10-1.src.rpm

Description: 
This library contains an algorithm that converts a boolean expression
binary tree into the Disjunctive Normal Form.  The NOT operator
is supported.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:16 EST ---
Created an attachment (id=149114)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=149114action=view)
mockbuild log of gdal-1.4.0-10.fc7.i386

This time mockbuild on FC-devel i386 is okay (1.4.0-10).
Check the log if this is what you expect.

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:28 EST ---
Thanks a lot Tasaka !
Yes its fine, no files are missing obviously from packages.

Can give help to review issues for final approval ?

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

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


[Bug 225765] Merge Review: fonts-japanese

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

Summary: Merge Review: fonts-japanese


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:31 EST ---
Well, I only saw three warning like:
W: fonts-japanese no-url-tag
W: fonts-japanese rpm-buildroot-usage %build %configure --disable-bold
--disable-italic --with-fontdir=$RPM_BUILD_ROOT%{bmpfontdir}
W: fonts-japanese mixed-use-of-spaces-and-tabs (spaces: line 149, tab: line 1)

I may be missing something but how did you see %config stuff?
spaces vs tabs issue and listing-files-twice issue was fixed in CVS with
revision 1.21 FYI.

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

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


[Bug 167147] Review Request: aqsis - 3D Rendering system

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

Summary: Review Request: aqsis - 3D Rendering system


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:34 EST ---
It seems that a2ps pdiff (print diff) and bug 223657
PerceptualDiff are of no relation.

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

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


[Bug 218556] Review Request: ecryptfs-utils - Linux eCryptfs utilities

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

Summary: Review Request: ecryptfs-utils - Linux eCryptfs utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:38 EST ---
Mar  2 14:25:26 rawhide64 kernel: 
=
[ INFO: possible recursive locking detected ]
2.6.20-1.2949.fc7 #1
-
bash/3868 is trying to acquire lock:
(inode-i_mutex){--..}, at: [802624d6] mutex_lock+0x2a/0x2e

but task is already holding lock:
(inode-i_mutex){--..}, at: [802624d6] mutex_lock+0x2a/0x2e

other info that might help us debug this:
1 lock held by bash/3868:
 #0:  (inode-i_mutex){--..}, at: [802624d6] mutex_lock+0x2a/0x2e

stack backtrace:

Call Trace:
[802a30ad] __lock_acquire+0x151/0xbc4
[8843e114] :ecryptfs:ecryptfs_filldir+0x0/0x7d
[802a3f16] lock_acquire+0x4c/0x65
[802624d6] mutex_lock+0x2a/0x2e
[80262312] __mutex_lock_slowpath+0xff/0x299
[802624d6] mutex_lock+0x2a/0x2e
[80235ab6] vfs_readdir+0x61/0xb1
[80226604] filldir+0x0/0xc5
[8843e374] :ecryptfs:ecryptfs_readdir+0x6c/0xb2
[80226604] filldir+0x0/0xc5
[80235ad1] vfs_readdir+0x7c/0xb1
[8023927f] sys_getdents+0x7a/0xc4
[8025c24a] tracesys+0x71/0xe1


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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 09:43 EST ---
Created an attachment (id=149119)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=149119action=view)
mock build log of grass-6.2.1-8.fc7.i386

mockbuild of 6.2.1-8 doesn't seem okay
(and I couldn't find -9 or 6.2.3?)

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

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


[Bug 225708] Merge Review: dovecot

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

Summary: Merge Review: dovecot


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:01 EST ---
Hi Jef,
thanks for your review, I applied your diff and commited, just changed the
section removing .la/.a files to use find instead. My comments follow:


(In reply to comment #1)
 ? MUST: Every binary RPM package which stores shared library files (not just
 symlinks) in any of the dynamic linker's default paths, must call ldconfig in
 %post and %postun. If the package has multiple subpackages with libraries, 
 each
 subpackage should also have a %post/%postun section that calls /sbin/ldconfig.
   Comment:  shared libs exist in /usr/lib/dovecot but they appear to be simple
 plugins for dovecot's own runtime use and not meant for linking.  if this is 
 the
 case, then no corrections need to be made. Please confirm that the items in
 /usr/lib/dovecot are not meant to be dynamically linkable libraries.

I confirm that.

 E: dovecot configure-without-libdir-spec
 ?  I am not sure what rpmlint is trying to tell us here.

This is probably a rpmlint bug, the libdir is passed by %configure itself.

 E: dovecot non-readable /etc/pki/dovecot/certs/dovecot.pem 0600
 E: dovecot non-standard-uid /var/lib/dovecot dovecot
 E: dovecot non-standard-gid /var/lib/dovecot dovecot
 E: dovecot non-standard-dir-perm /var/lib/dovecot 0750
 E: dovecot non-standard-gid /var/run/dovecot/login dovecot
 E: dovecot non-standard-dir-perm /var/run/dovecot/login 0750
 E: dovecot non-readable /etc/pki/dovecot/private/dovecot.pem 0600
  all of these rpmlint errors appear to be bogus to me. Please confirm that
 the permissions and ownership are as intended for these.

Yes, they are.

 W: dovecot dangerous-command-in-%pre rm
 W: dovecot dangerous-command-in-%post mv
 W: dovecot dangerous-command-in-%preun userdel
   I think these warnings are bogus. Though you may want to look back over 
   the use of the rm and mv commands to see if they are still needed.
   I think I understand why the restart_flag logic is present.  
   But I do not understand why the ssl cert manipulation logic block is in 
  
   %post.  All the file location testing and conditional use of mv. What
   cases trigger the mv commands? Is this logic meant for now EOL'd fedora 
   and rhl releases?

Yeah, the certificate paths used to be different, this block moves them to new
location.


(In reply to comment #4)
 If there is no -devel package, does that stop someone from being able to build
 something like http://wiki.dovecot.org/LDA/Sieve?highlight=%28dovecot-sieve%29
 against it? I know that Sieve will not build the way dovecot is currently
 packaged, because the Sieve program needs to be able to find a file called
 dovecot-config in the compiled Dovecot sources. I do not know what the 
 correct
 way to handle this but I ask that you take my comments into consideration, in
 case someone would like to use Dovecot-Sive with this package.

I didn't find any easy way to make dovecot-sieve compile against packaged
version. It just wants access to the dovecot build dir. I might create a -devel
package in the future as upstream added an option to install headers, but the
location of things is probably still not ok. Regarding dovecot-sieve, we'll
probably build that from the same source package.

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

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


[Bug 229826] Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko

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

Summary: Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs+ |fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:03 EST ---
Package co-maintainer Change Request
===
Package Name: chmsee
Short Description: A Gtk+2 CHM document viewer
Owners: [EMAIL PROTECTED], [EMAIL PROTECTED]
Branches: FC-6


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

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


[Bug 229929] Review Request: mecab-jumandic - JUMAN dictorionary for MeCab

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

Summary: Review Request: mecab-jumandic - JUMAN dictorionary for MeCab


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

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




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:09 EST ---
Sorry I hadn't assigned this to myself yet so I didn't see any of your replies
(oops). I'll get around to reviewing this as time permits.


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

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


[Bug 229826] Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko

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

Summary: Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:10 EST ---
(In reply to comment #29)
 Package co-maintainer Change Request
 ===

 Branches: FC-6

Only for FC-6? Not for all the branches? 



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

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


[Bug 230726] New: Review Request: xmoto-edit - X-Moto level editor

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




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

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


Spec URL: http://zanoni.jcomserv.net/extras/xmoto-exit/xmoto-edit.spec
SRPM URL: 
http://zanoni.jcomserv.net/extras/xmoto-exit/xmoto-edit-0.2.4-1.src.rpm
Description: Level editor or xmoto.  With the last release, this was split out 
of the upstream tarball.  The spec is basically a modified xmoto.spec.  rpmlint 
is clean for srpm, rpm and debuginfo.

I am already sponsored.

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

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


[Bug 225608] Merge Review: basesystem

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

Summary: Merge Review: basesystem


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:20 EST ---
Sorry it took a while to get to do the fixes.

Here first my comments on the findings:

 W: basesystem summary-ended-with-dot The skeleton package which defines a
simple Red Hat Linux system.

Fixed

 W: basesystem invalid-license public domain

Fixed, now using Public Domain (as requested below).

 W: basesystem no-url-tag

There is now upstream for this package, so the only option would be to either
make it http://www.redhat.com/ or http://www.fedoraproject.org/

 W: basesystem prereq-use setup filesystem

Fixed. Now it's Requires(Pre): setup filesystem

 W: basesystem hardcoded-path-in-buildroot-tag /var/tmp/basesystem-root

Fixed. Using the latest recommended BuildRoot 

 E: basesystem no-cleaning-of-buildroot %install

Fixed. Added empty %install section

 E: basesystem no-cleaning-of-buildroot %clean

Fixed. Added empty %clean section

 W: basesystem no-%prep-section

Fixed. Added empty %prep section

 W: basesystem no-%build-section

Fixed. Added empty %build section

 W: basesystem no-%install-section

Fixed. Added empty %install section

 W: basesystem no-%clean-section

Fixed. Added empty %clean section


  RPM:
 W: basesystem summary-ended-with-dot The skeleton package which defines a
simple Red Hat Linux system.

Fixed. See above.

 W: basesystem invalid-license public domain

Fixed. See above.

 W: basesystem no-url-tag

Possible fixes listed above.

 W: basesystem no-documentation

basesystem doesn't have a source, nor does it contain any files. so unecessary.


Random issues:
 * Change Red Hat Linux to Fedora (both in summary and description). 
 blocker.

Fixed.

 * What is the version 8.0?! I can't say this follows naming guidelines.

Version of basesystem is arbitrary.

 * Make release integer (6?).

Fixed, but release can (and often will) consiste of X.Y.Z components.

 * Using Prereq is bad. Change to Requires. blocker.

Absolutely agreed. Fixed.

 * Capitalize Public Domain.

Fixed. See above.

 * Change BuildRoot to
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

Fixed. Used the even newer recommended BuildRoot entry as per fedoraproject.org

 * description says this should be the first package installed on a system, but
the package Prereq's setup and filesystem. Confusing.

Uhm, yea. Thats one of the things i'd like to go over with some folks and
discuss on f-d-l. Imo the correct order of those 3 packages would be:

  basesystem
  filesystem
  setup

For the simple reason as basesystem (as the description also already says)
should be the first package on a system. Then filesystem, as it creates the
basic directory structure of the system. And third setup, which contains the
basic setup for the system. I think one of the problems though is that in order
to basesystem, filesystem and setup pulled in glibc requires basesystem, which
then in turn pulls in filesystem and setup via the Requires(Pre):

[EMAIL PROTECTED] tmp]$ rpm -q --whatrequires basesystem
glibc-2.5-9

[EMAIL PROTECTED] tmp]$ rpm -q --whatrequires filesystem
basesystem-8.0-5.1.1
lockdev-1.0.1-10
SysVinit-2.86-14
mkinitrd-5.1.19.2-1
nautilus-2.16.2-5.el5

[EMAIL PROTECTED] tmp]$ rpm -q --whatrequires setup
filesystem-2.4.0-1
basesystem-8.0-5.1.1
dump-0.4b41-2.fc6
sendmail-8.13.8-2.el5

* Add empty sections for %prep, %build, %install, and %clean. blocker.

Fixed. Added empty sections for those.

* Theoretically, the %files section should contain a %defattr line.

Fixed. Added it, just for completeness.

Summary: Overall should contain now nearly all recommended fixes. Only 2
questions are:

- What to do with URL? Really not happy about any arbitrary URL there.
- Discuss on f-d-l how to go about fixing the requires chain.

Read ya, Phil

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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:21 EST ---
Oops, bad URLs:
Spec URL: http://zanoni.jcomserv.net/extras/xmoto-edit/xmoto-edit.spec
SRPM URL: 
http://zanoni.jcomserv.net/extras/xmoto-edit/xmoto-edit-0.2.4-1.src.rpm

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

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


[Bug 230344] Review Request: bacula - Cross platform network backup for Linux, Unix, Mac and Windows.

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

Summary: Review Request: bacula - Cross platform network backup for Linux, 
Unix, Mac and Windows.


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:31 EST ---
I'm glad to see that, unlike upstream, you've split the storage daemon off from
the director package.

I can't look at this in detail at the moment as I'm very busy with $DAYJOB but
I'll keep an eye on this ticket and will hopefully be able to comment more 
later on.

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

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


[Bug 222039] Review Request: ogdi - Open Geographic Datastore Interface

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

Summary: Review Request: ogdi - Open Geographic Datastore Interface


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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


Bug 222042 depends on bug 222039, which changed state.

Bug 222039 Summary: Review Request: ogdi - Open Geographic Datastore Interface
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222039

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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


Bug 227646 depends on bug 222039, which changed state.

Bug 222039 Summary: Review Request: ogdi - Open Geographic Datastore Interface
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=222039

   What|Old Value   |New Value

 Resolution||NEXTRELEASE
 Status|ASSIGNED|CLOSED



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

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


[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version 
control repositories


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:37 EST ---
$ rpmlint -I forbidden-selinux-command-in-%post
forbidden-selinux-command-in-%post :
A command which requires intimate knowledge about a specific SELinux
policy type was found in the scriptlet. These types are subject to change
on a policy version upgrade. Use the restorecon command which queries the
currently loaded policy for the correct type instead.

See also bug 214605 (the reporter, Steve, might be able to provide more info how
to do the SELinux stuff better if needed).

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO|163778, 177841  |163779
  nThis||
   Flag||fedora-review+




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:38 EST ---
Okay.
--
  This package (gdal) is APPROVED by me.
--

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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




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

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


[Bug 229826] Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko

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

Summary: Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:41 EST ---
(In reply to comment #30)
 
  Branches: FC-6
 
 Only for FC-6? Not for all the branches? 
 

It is said that devel branch is always implied. Isn't it?


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

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


[Bug 222042] Review Request: GDAL - Geospatial Data Abstraction Library

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

Summary: Review Request: GDAL - Geospatial Data Abstraction Library


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:46 EST ---
New Package CVS Request
===
Package Name: gdal
Short Description: Geospatial Data Abstraction Library
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 devel
InitialCC:

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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:48 EST ---
updated.
Spec URL: http://openrisc.rdsor.ro/grass.spec
SRPM URL: http://openrisc.rdsor.ro/grass-6.2.3-9.src.rpm

changelog reflect the details.
should build now in mock i386/x86_64


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

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


[Bug 229826] Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko

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

Summary: Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:49 EST ---
(In reply to comment #31)

 It is said that devel branch is always implied. Isn't it?

If I'm not wrong not for co-maintainers. A co-maintainer may
only co-maintain a given branch.



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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:50 EST ---
good.
always rushy mistake ...

Spec URL: http://openrisc.rdsor.ro/grass.spec
SRPM URL: http://openrisc.rdsor.ro/grass-6.2.1-9.src.rpm

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

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


[Bug 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

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

Summary: Review Request: ntop - A network traffic probe similar to the UNIX top 
command
Alias: ntop

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:54 EST ---
(In reply to comment #56)
 Here is an update, but note:
 1) Ntop is terminating on my system after 5-20 minutes.
I ran for about 11 hours but no problem occurred...

 2) javascript generated graphics are not displaying on my system
I am not familiar with javascript...
I will review this later. Please wait...

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

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


[Bug 229826] Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko

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

Summary: Review Request: Chmsee - a GTK2 CHM viewer based on chmlib and gecko


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:56 EST ---
In fact unless I am wrong, there a 2 way of becoming co-maintainer,
be added to owners.list, or have an entry in pkg.acl. Since the
fedora-cvs request is to be added to owners.list, you are right
that branches are irrelevant. 

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

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


[Bug 226511] Merge Review: unifdef

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

Summary: Merge Review: unifdef


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 10:59 EST ---
Well, since unifdef is largely unmaintained as far as I can tell, shall we drop
the package? Would you like to co-maintain sunifdef? :)

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

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


[Bug 229342] Review Request: nfs4-acl-tools - ACL utilities for NFSv4

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

Summary: Review Request: nfs4-acl-tools - ACL utilities for NFSv4


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 11:21 EST ---
 I note 0.3.0 came out yesterday.
Updated to this version which now include man pages.

 W: nfs4-acl-tools no-version-in-last-changelog
 W: nfs4-acl-tools-debuginfo no-version-in-last-changelog
 W: nfs4-acl-tools no-documentation
 W: nfs4-acl-tools empty-%post
 W: nfs4-acl-tools empty-%preun
 W: nfs4-acl-tools empty-%postun
All fixed by applying patch in Comment #5 (thank you very much!)

 Did you intend to build the graphical bits?
No, since I was not able to build them. I'll add them
in an upcoming release.
 you shouldn't put the buildroot in --prefix or --bindir.  But the
 package fails to install if you do that. 
With the addition of the man pages, I also had to add --mandir to
this configuration line... But by setting these during the 
configuration, removed the need to set the DESTDIR during
the make install 

Updated Spec is in: 
http://people.redhat.com/steved/nfs4-acl-tools/nfs4-acl-tools.spec

Updated SRPM is in 
http://people.redhat.com/steved/nfs4-acl-tools/nfs4-acl-tools-0.3.0-1.fc7.src.rpm


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

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


[Bug 228425] Review Request: gtkpod - Graphical song management program for Apple's iPod

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

Summary: Review Request: gtkpod - Graphical song management program for Apple's 
iPod
Alias: gtkpod-review

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version 
control repositories


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 11:34 EST ---
(In reply to comment #12)
 Well, I was guessing that Python programs run best within mod_python, like 
 Perl
 programs do under mod_perl.

This is not always the case; moin for example, runs significantly faster as a
FastCGI application (e.g. with mod_fcgid) than with mod_python.

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

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


[Bug 230738] Review Request: jwm - Very light window manager

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

Summary: Review Request: jwm - Very light window manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Version|devel   |fc6




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

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


[Bug 230738] First package - jwm - Very light window manager

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

Summary: First package - jwm - Very light window manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|Review Request: jwm - Very  |First package - jwm - Very
   |light window manager|light window manager




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

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


[Bug 208737] Review Request: ivman - Generic handler for HAL events

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

Summary: Review Request: ivman - Generic handler for HAL events


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 12:08 EST ---
I would say you should follow the procedure to cvs rm the files, and put a
dead.package file in CVS. That way people will know it's not built. If someone
wants to bring it back they can do so from that state. 

Thats probibly going to have to happen to all the other orphans at some 
point... 

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

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


[Bug 229154] Review Request: konwert - Converter of character encodings

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

Summary: Review Request: konwert - Converter of character encodings


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 12:25 EST ---
Ok, i will

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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 12:27 EST ---
What's the %{_mandir}/mang/xmoto-edit.6.gz?  Why not to use man6 instead?

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

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


[Bug 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

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

Summary: Review Request: ntop - A network traffic probe similar to the UNIX top 
command
Alias: ntop

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 12:27 EST ---
Well, for 3.3-0.3.20060227cvs.fc7:

* conditional dependency (check again...)
--
checking for pcre_refcount in -lpcre... no
--
  - Perhaps adding BuildRequires: pcre-devel will
change this to yes.

--
checking security/pam_appl.h usability... no
checking security/pam_appl.h presence... no
--
   - Perhaps pam-devel

--
checking for i686-redhat-linux-gnu-mysql_config... no
checking for mysql_config... no
--
   - mysql-devel or so?

* Unused source?
  - What is Source4?

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

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


[Bug 230738] Review Request - jwm - Very light window manager

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

Summary: Review Request - jwm - Very light window manager


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

Summary|First package - jwm - Very  |Review Request - jwm - Very
   |light window manager|light window manager
OtherBugsDependingO||177841
  nThis||




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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 12:57 EST ---
Looks like a typo.  Should be man6.  I see it in xmoto, too.  I'll fix both. 
Should provide that back here?  Do you see anything else out of sorts?

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

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


[Bug 225615] Merge Review: binutils

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

Summary: Merge Review: binutils


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


Bug 225615 depends on bug 223678, which changed state.

Bug 223678 Summary: binutils: non-failsafe install-info use in scriptlets
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=223678

   What|Old Value   |New Value

 Resolution||RAWHIDE
 Status|NEW |CLOSED



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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:01 EST ---
On second look, it's set that way in the Makefile.  Should I patch it and send
it upstream?

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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:11 EST ---
(In reply to comment #3)
 Do you see anything else out of sorts?

I haven't even chcecked whether package's working, but at first sight I see 
that you ought to remove X-Fedora category and, if it exists, get rid of 
Application category as well. Also, if you put icons in %{_datadir}/icons/
hicolor you should add Requires: hicolor-icon-theme (some time ago it was 
discussed on list). Later I'm going to try to build this package and hopefully 
make the full review.

(In reply to comment #4)
 On second look, it's set that way in the Makefile.  Should I patch it and send
 it upstream?

IMHO you should.

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

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


[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version 
control repositories


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 CC||[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:11 EST ---
Regarding the SELinux issue, I think the best option is to:

(a) use %{_localstatedir}/cache/viewvc instead of %{_localstatedir}/spool/viewvc
(this is just the way I usually do it though)
(b) file a bug against selinux-policy requesting that the context type of
/var/cache/viewvc(/.*)? be set to httpd_cache_t
(c) remove all SElinux-related content from your own package

Dan is generally quite responsive to such requests.

Whilst waiting for the selinux-policy update to appear (updates appear quite
frequently, so it's unlikely to be long), you could just put a README.SELinux
file in that suggested the context changes to do manually if needed.

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

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


[Bug 230762] New: Review Request: xawtv - TV applications for video4linux compliant devices

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




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

   Summary: Review Request: xawtv - TV applications for video4linux
compliant devices
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://dmitry.butskoy.name/xawtv/xawtv.spec
SRPM URL: http://dmitry.butskoy.name/xawtv/xawtv-3.95-0.src.rpm

Description: 
Xawtv is a simple xaw-based TV program which uses the bttv driver or
video4linux. Xawtv contains various command-line utilities for
grabbing images and .avi movies, for tuning in to TV stations, etc.
Xawtv also includes a grabber driver for vic.


Additional info:
xawtv was in Fedora Core until FC2 (now tvtime is a base apps to watch TV).

Xawtv seems to be a first project capable to show TV, written by the same 
people as v4l drive.

Besides xawtv program itself, xawtv package includes a lot of another 
useful things: command line tools, console radio apps, frame buffer tv, 
teletext web daemon etc. These addional tools are the main reason to package 
xawtv for Fedora.

This package includes ability to write avi files with motion jpeg video, as 
well as raw YUV 4:2:0 called yuv4mpeg. All those seem to have no legal 
issues (avi support is already in Fedora (dvgrab, xine-lib), mjpeg and yum4mpeg 
are just a stream of images (jpeg and raw YUV respectively)).

Another distros like to split xawtv to several subpackages. I dislike that, 
because initial Fedora's xawtv was a single package, there is no common 
agreement how to split (various distos do it differently), splitting itself 
seem to be unuseful (even some pure cmdline tools require X11 libs etc.)

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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:16 EST ---
Created an attachment (id=149136)
 -- (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=149136action=view)
mock build log of grass-6.2.1-9.1.fc7.i386

Still was not good.. I added
libtiff-devel libpng-devel for BuildRequires.

Check if this build log is what you expect.
(Note: I just rebuilt grass and have not checked any of
 grass) Especially, check if some no return on configure
can be ignored.

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

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


[Bug 227646] Review Request: grass - GRASS (Geographic Resources Analysis Support System)

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

Summary: Review Request: grass -  GRASS (Geographic Resources Analysis Support 
System)


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
OtherBugsDependingO|163776, 177841  |163778
  nThis||
   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:17 EST ---
I once go to sleep...

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

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


[Bug 226276] Merge Review: perl

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

Summary: Merge Review: perl


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:32 EST ---
Alright, let's discuss this change on fedora-perl-devel and make the call by 
Monday.

See:
https://www.redhat.com/archives/fedora-perl-devel-list/2007-March/msg9.html
and replies.

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

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


[Bug 228294] Review Request: gkrellm-sun - Sun clock plugin for GKrellM

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

Summary: Review Request: gkrellm-sun - Sun clock plugin for GKrellM


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:33 EST ---
New Package CVS Request
===
Package Name: gkrellm-sun
Short Description: Sun clock plugin for GKrellM
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 devel EL-4 EL-5
InitialCC: 

(I still don't have rights to set the fedora-cvs flag...)

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

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


[Bug 228293] Review Request: gkrellm-moon - Moon clock plugin for GKrellM

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

Summary: Review Request: gkrellm-moon - Moon clock plugin for GKrellM


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:34 EST ---
New Package CVS Request
===
Package Name: gkrellm-moon
Short Description: Moon clock plugin for GKrellM
Owners: [EMAIL PROTECTED]
Branches: FC-5 FC-6 devel EL-4 EL-5
InitialCC: 

(I still don't have rights to set the fedora-cvs flag...)

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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:46 EST ---
I've created the patch and removed X-Fedora.  I'll send the patch upstream after
approval.

Spec URL: http://zanoni.jcomserv.net/extras/xmoto-edit/xmoto-edit.spec
SRPM URL: 
http://zanoni.jcomserv.net/extras/xmoto-edit/xmoto-edit-0.2.4-2.src.rpm

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

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


[Bug 216534] Review Request: gocr - GNU Optical Character Recognition program

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

Summary: Review Request: gocr - GNU Optical Character Recognition program


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-review?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 13:48 EST ---
(In reply to comment #12)
 Ooops, sorry it is included. The only issue is with files with author
 and no license.

0.44 has been released with the licenses added.

http://www.cora.nwra.com/~orion/fedora/gocr.spec
http://www.cora.nwra.com/~orion/fedora/gocr-0.44-1.fc6.src.rpm




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

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


[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version 
control repositories


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 17:44 EST ---
Another bump (-6) here:

ftp://ftp.rexursive.com/pub/viewvc/viewvc.spec
ftp://ftp.rexursive.com/pub/viewvc/viewvc-1.0.3-6.FC6.src.rpm

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

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


[Bug 230798] New: Review Request: perl-Test-Simple - A basic Perl testing framework for creating tests to be run either standalone or under Test::Harness.

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




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

   Summary: Review Request: perl-Test-Simple - A basic Perl testing
framework for creating tests to be run either standalone
or under Test::Harness.
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.redhat.com/mkpai/.SPEC/perl-Test-Simple.spec
SRPM URL: http://people.redhat.com/mkpai/.SRPM/perl-Test-Simple-0.67-1.src.rpm
Description:  This is a basic Perl testing framework for creating tests to be 
run either standalone or under Test::Harness.

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

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


[Bug 227101] Review Request: plexus-container-default-1.0-0.a8.2jpp - Default Plexus Container

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

Summary: Review Request: plexus-container-default-1.0-0.a8.2jpp - Default 
Plexus Container


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 17:52 EST ---
New Package CVS Request
===
Package Name: plexus-container-default
Short Description: Default Plexus Container
Owners: [EMAIL PROTECTED]
Branches: 
InitialCC: 

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

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


[Bug 227106] Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities

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

Summary: Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 17:54 EST ---
New Package CVS Request
===
Package Name: plexus-utils
Short Description: Plexus Common Utilities
Owners: [EMAIL PROTECTED] 
Branches: 
InitialCC: 

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

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


[Bug 230738] Review Request - jwm - Very light window manager

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

Summary: Review Request - jwm - Very light window manager
Alias: jwm

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


[EMAIL PROTECTED] changed:

   What|Removed |Added

  Alias||jwm




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

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


[Bug 227048] Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM Conformance Test Suite

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

Summary: Review Request: dom2-core-tests-0.0.1-0.20040405.1jpp - DOM 
Conformance Test Suite


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 AssignedTo|[EMAIL PROTECTED]   |[EMAIL PROTECTED]




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 17:59 EST ---
Reassigning to package owner to be built into rawhide.

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

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


[Bug 227106] Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities

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

Summary: Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 227101] Review Request: plexus-container-default-1.0-0.a8.2jpp - Default Plexus Container

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

Summary: Review Request: plexus-container-default-1.0-0.a8.2jpp - Default 
Plexus Container


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




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

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


[Bug 219025] Review Request: ntop - A network traffic probe similar to the UNIX top command

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

Summary: Review Request: ntop - A network traffic probe similar to the UNIX top 
command
Alias: ntop

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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:04 EST ---
(In reply to comment #59)
 Well, for 3.3-0.3.20060227cvs.fc7:
 
 * conditional dependency (check again...)
 --
 checking for pcre_refcount in -lpcre... no
 --
   - Perhaps adding BuildRequires: pcre-devel will
 change this to yes.


This can be used for pattern matching of incoming payloads, so I have enabled 
it.

 --
 checking security/pam_appl.h usability... no
 checking security/pam_appl.h presence... no
 --
- Perhaps pam-devel
 
 --

I looked at the code and all this would give is the ability of the web interface
to display that you have pam_appl.h available.  It doesn't use it anywhere else
that I can find.  

 checking for i686-redhat-linux-gnu-mysql_config... no
 checking for mysql_config... no
 --
- mysql-devel or so?

This can be used for longer term storage of net flow data, so I've enabled it,
and added mysql-server as a requires.


 * Unused source?
   - What is Source4?

Not currently used.  I was looking for a better way to set the ntop password,
but I didn't finish this.  I removed it.


Spec URL: http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/ntop.spec
SRPM URL:
http://www.symetrix.com/~bjohnson/projects/Fedora-Extras/ntop-3.3-0.4.20060227cvs.fc6.src.rpm

* Fri Mar 02 2007 Bernard Johnson [EMAIL PROTECTED] - 3.3-0.4.20060227cvs
- add pcre-devel to BR so payloads can be matched
- remove unused Source4 line
- enabled mysql storage of net flow data



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

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


[Bug 227106] Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities

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

Summary: Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:09 EST ---
does this need or not need cvs admin attention?


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

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


[Bug 230802] New: Review Request: perl-Callback - Object interface for function callbacks

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




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

   Summary: Review Request: perl-Callback - Object interface for
function callbacks
   Product: Fedora Extras
   Version: devel
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: normal
  Priority: normal
 Component: Package Review
AssignedTo: [EMAIL PROTECTED]
ReportedBy: [EMAIL PROTECTED]
 QAContact: fedora-package-review@redhat.com


Spec URL: http://people.redhat.com/mkpai/.SPEC/perl-Callback.spec
SRPM URL: http://people.redhat.com/mkpai/.SRPM/perl-Callback-1.06-1.src.rpm
Description: Object interface for function callbacks

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

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


[Bug 230798] Review Request: perl-Test-Simple - A basic Perl testing framework for creating tests to be run either standalone or under Test::Harness.

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

Summary: Review Request: perl-Test-Simple - A basic Perl testing framework for 
creating tests to be run either standalone or under Test::Harness.


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:15 EST ---
This is my first package. I need a sponsor. I have also submitted another Perl
package at https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=230802 .

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

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


[Bug 230806] Review Request: tasks - To-do list

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

Summary: Review Request: tasks - To-do list


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

OtherBugsDependingO||177841
  nThis||




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

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


[Bug 230726] Review Request: xmoto-edit - X-Moto level editor

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

Summary: Review Request: xmoto-edit - X-Moto level editor


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |NEEDINFO
 AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
   Flag||fedora-review?,
   ||needinfo?([EMAIL PROTECTED])




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:40 EST ---
REVIEW: 
** it seems that package has some unneeded dependencies, it built fine in mock 
with: SDL_mixer-devel, libGL-devel, libjpeg-devel, zlib-devel and libpng-devel. 
So you can get rid of curl-devel, ode-devel, lua-devel, libGLU-devel and 
replace bzip2-devel with zlib-devel (if there's something wrong with this set 
of dependencies, please let me know)

** does it make sense not to require main xmoto package? I understand that 
there's no need xmoto to be installed to make levels for it, but there's a 
Play Level button in xmoto-edit which ends up with very ugly sh: xmoto: 
command not found. Thus add xmoto dependency.

** also, you have to add versioned xmoto dependency because of that: file /usr/
bin/xmoto-edit from install of xmoto-edit-0.2.4-2.fc7 conflicts with file from 
package xmoto-0.2.2-2.fc6. Now I don't know whether it would be better to have 
xmoto = 0.2.4 dependency or xmoto = %{version}. If xmoto and xmoto-edit 
versions always match, the second solution will be much more sane.

** rpmlint complain about W: xmoto-edit mixed-use-of-spaces-and-tabs (spaces: 
line 1, tab: line 12) so get rid of that

** [EMAIL PROTECTED] ~]$ desktop-file-validate 
/usr/share/applications/fedora-xmoto-
edit.desktop
/usr/share/applications/fedora-xmoto-edit.desktop: warning: The 'Application' 
category is not defined by the desktop entry specification.  Please use one of 
AudioVideo, Audio, Video, Development, Education, Game, Graphics, 
Network, Office, Settings, System, Utility instead

So just get rid of Application category in your desktop file. Install the 
newest rawhide version of desktop-file utils if you can't see this error on 
your desktop-file-validate.

** There's /usr/share/xmoto-edit/xmoto.bin file which contains a lot of binary 
(?) data. Is it arch-dependent? If so, you cannot put this file in %{_datadir}. 
Unfortunetaly, I'm unable to identify what kind of data it is, maybe you know 
something more?

** Desktop file contains Comment=Xmoto LEvel Editor - just change LEvel→Level

Other things seems ok.

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

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


[Bug 227061] Review Request: isorelax-0.1-0.20041111.2jpp - Public interfaces useful for applications to support RELAX Core

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

Summary: Review Request: isorelax-0.1-0.2004.2jpp - Public interfaces 
useful for applications to support RELAX Core


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:43 EST ---
New Package CVS Request
===
Package Name: isorelax
Short Description: Public interfaces for RELAX Core
Owners: [EMAIL PROTECTED]
Branches: devel
InitialCC: [EMAIL PROTECTED],[EMAIL PROTECTED]

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

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


[Bug 226795] Review Request: sdcc - Small Device C Compiler

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

Summary: Review Request: sdcc - Small Device C Compiler


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 227041] Review Request: bea-stax-1.2.0-0.rc1.2jpp - Streaming API for XML

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

Summary: Review Request: bea-stax-1.2.0-0.rc1.2jpp - Streaming API for XML


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:48 EST ---
New Package CVS Request
===
Package Name: bea-stax
Short Description: Streaming API for XML
Owners: [EMAIL PROTECTED]
Branches: devel
InitialCC: [EMAIL PROTECTED],[EMAIL PROTECTED]



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

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


[Bug 227106] Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities

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

Summary: Review Request: plexus-utils-1.2-1jpp - Plexus Common Utilities


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:50 EST ---
(In reply to comment #6)
 does this need or not need cvs admin attention?

Hmm, I might have been given some bad information. I tried to upload this myself
but it only halfed worked (it fails halfway through, so I now have an empty
devel directory).

So I guess only cvs admins can upload new projects? 



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

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


[Bug 230512] Review Request: viewvc- Browser interface for CVS and SVN version control repositories

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

Summary: Review Request: viewvc-  Browser interface for CVS and SVN version 
control repositories


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:54 EST ---
(In reply to comment #12)
 Yeah, I was also thinking about this. On one hand, if I don't require this and
 someone configures one of the two options (CVS or SVN), it may or may not 
 work,
 depending on the current state of their system. So, I thought, better err on 
 the
 side of caution and make sure at least minimum requirements for both (since
 Fedora ships both CVS and SVN) are satisfied, so that things just work. But, 
 if
 there is a better way to do this, I'd be happy to change...

Well, looking at the packages required by each of those that you require
(subversion, rcs).  They have similarities and not too many differences.  IMHO,
it is not unreasonable to list both as requires rather than create meta packages
with the sole purpose of pulling the right dependencies.

# repoquery --resolve --requires subversion | sort
apr-0:1.2.7-10.i386  (-- already required by httpd)
apr-util-0:1.2.8-1.fc6.i386 (-- already required by httpd)
bash-0:3.1-16.1.i386 (-- already required by httpd)
coreutils-0:5.97-12.3.fc6.i386 (-- already required by httpd)
db4-0:4.3.29-9.fc6.i386 (-- already required by httpd)
e2fsprogs-libs-0:1.39-7.fc6.i386
expat-0:1.95.8-8.2.1.i386 (-- already required by httpd)
glibc-0:2.5-10.fc6.i686 (-- already required by almost everything)
krb5-libs-0:1.5-13.i386 (-- needed if cvs is installed)
neon-0:0.25.5-5.1.i386 (-- already required by rpm)
openldap-0:2.3.27-4.i386 (-- already required by httpd)
openssl-0:0.9.8b-8.3.fc6.i686 (-- already required by httpd)
perl-4:5.8.8-10.i386 (-- needed if cvs is installed)
perl-URI-0:1.35-3.noarch
python-0:2.4.4-1.fc6.i386  (-- required for viewvc)
subversion-0:1.4.2-2.fc6.i386
zlib-0:1.2.3-3.i386 (-- already required by httpd)

# repoquery --resolve --requires rcs | sort
glibc-0:2.5-10.fc6.i686

But more likely, if you're using viewvc with cvs repositories you also have cvs
installed (although this is not a hard requirement)

# repoquery --resolve --requires cvs | sort
bash-0:3.1-16.1.i386
cvs-0:1.11.22-6.fc6.i386
glibc-0:2.5-10.fc6.i686
info-0:4.8-14.fc6.i386
krb5-libs-0:1.5-13.i386
pam-0:0.99.6.2-3.16.fc6.i386
perl-4:5.8.8-10.i386
vim-minimal-2:7.0.191-2.fc6.i386
zlib-0:1.2.3-3.i386

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

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


[Bug 230324] Review Request: avrdude -Software for programming Atmel AVR Microcontroller

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

Summary: Review Request: avrdude -Software for programming Atmel AVR 
Microcontroller


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


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 227042] Review Request: byaccj-1.11-2jpp - Parser Generator with Java Extension

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

Summary: Review Request: byaccj-1.11-2jpp - Parser Generator with Java Extension


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





--- Additional Comments From [EMAIL PROTECTED]  2007-03-02 18:55 EST ---
New Package CVS Request
===
Package Name: byaccj
Short Description: Parser Generator with Java Extension
Owners: [EMAIL PROTECTED]
Branches: devel
InitialCC: [EMAIL PROTECTED],[EMAIL PROTECTED]

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

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


  1   2   >