[Bug 542740] Review Request: f2c - The f2c Fortran to C/C++ conversion program and static libraries

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


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


Terje Røsten terje...@phys.ntnu.no changed:

   What|Removed |Added

 CC||terje...@phys.ntnu.no




--- Comment #7 from Terje Røsten terje...@phys.ntnu.no  2009-12-01 03:30:34 
EDT ---
I don't see the need to continue the strange release tag, 
just change 3.0.3 to 4.

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

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


[Bug 484644] Review Request: screenlets - Fully themeable mini-apps

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


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





--- Comment #24 from Edouard Bourguignon ma...@linuxed.net  2009-12-01 
03:30:25 EDT ---
Only the weather icons are concerned by the restrictive license?

I've just opened a ticket as you suggested
https://fedorahosted.org/design-team/ticket/105

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

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


[Bug 526998] Review Request: volpack - Portable library for fast volume rendering

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


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





--- Comment #16 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 03:34:38 EDT ---
volpack-1.0c7-3.fc11 has been submitted as an update for Fedora 11.
http://admin.fedoraproject.org/updates/volpack-1.0c7-3.fc11

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

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


[Bug 526998] Review Request: volpack - Portable library for fast volume rendering

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


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





--- Comment #17 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 03:35:21 EDT ---
volpack-1.0c7-3.fc12 has been submitted as an update for Fedora 12.
http://admin.fedoraproject.org/updates/volpack-1.0c7-3.fc12

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

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


[Bug 526998] Review Request: volpack - Portable library for fast volume rendering

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


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





--- Comment #15 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 03:33:14 EDT ---
volpack-1.0c7-3.el5 has been submitted as an update for Fedora EPEL 5.
http://admin.fedoraproject.org/updates/volpack-1.0c7-3.el5

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

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


[Bug 540885] Review Request: cableswig - Create interfaces to interpreted languages for templated code

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


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





--- Comment #2 from Mario Ceresa mrcer...@gmail.com  2009-12-01 03:42:59 EDT 
---
Hello Mamoru, thanks for your comment. 
I looked at fedora people, but it seems I'd need to be already sponsored in
order to get the space. 

A friend of mine could host them for a while at:
ftp://85.54.198.199/FEDORA/cableswig-cvs20091120-1.fc12.src.rpm
ftp://85.54.198.199/FEDORA/cableswig.spec
ftp://85.54.198.199/FEDORA/InsightToolkit-3.16.0-1.fc12.src.rpm
ftp://85.54.198.199/FEDORA/InsightToolkit.spec

I'm looking forward to hearing from you soon,

Mario

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

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


[Bug 483250] Review Request: chordii - Print songbooks (lyrics + chords)

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


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





--- Comment #20 from Johan Vromans jvrom...@squirrel.nl  2009-12-01 03:52:08 
EDT ---
In a way, yes.
I managed to contact one of the original authors and he promised to make the
necessary changes.
That was several weeks ago but I didn't get a response since.
I'll try again.

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

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


[Bug 225286] Merge Review: aspell

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


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


Roman Rakus rra...@redhat.com changed:

   What|Removed |Added

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




--- Comment #19 from Roman Rakus rra...@redhat.com  2009-12-01 03:59:48 EDT 
---
# MUST: Each package must have a %clean section, which contains rm -rf
%{buildroot} (or $RPM_BUILD_ROOT). [16]
ok

# MUST: Each package must consistently use macros. [17]
ok

# MUST: The package must contain code, or permissable content. [18]
ok

# MUST: Large documentation files must go in a -doc subpackage. (The definition
of large is left up to the packager's best judgement, but is not restricted to
size. Large can refer to either size or quantity). [19]
ok

# MUST: If a package includes something as %doc, it must not affect the runtime
of the application. To summarize: If it is in %doc, the program must run
properly if it is not present. [19]
ok

# MUST: Header files must be in a -devel package. [20]
ok

# MUST: Static libraries must be in a -static package. [21]
ok

# MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'
(for directory ownership and usability). [22]
ok

# MUST: If a package contains library files with a suffix (e.g. libfoo.so.1.1),
then library files that end in .so (without suffix) must go in a -devel
package. [20]
ok

# MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency: Requires: %{name} =
%{version}-%{release} [23]
ok

# MUST: Packages must NOT contain any .la libtool archives, these must be
removed in the spec if they are built.[21]
ok

# MUST: Packages containing GUI applications must include a %{name}.desktop
file, and that file must be properly installed with desktop-file-install in the
%install section. If you feel that your packaged GUI application does not need
a .desktop file, you must put a comment in the spec file with your explanation.
[24]
ok

# MUST: Packages must not own files or directories already owned by other
packages. The rule of thumb here is that the first package to be installed
should own the files or directories that other packages may rely upon. This
means, for example, that no package in Fedora should ever share ownership with
any of the files or directories owned by the filesystem or man package. If you
feel that you have a good reason to own a file or directory that another
package owns, then please present that at package review time. [25]
ok

# MUST: At the beginning of %install, each package MUST run rm -rf %{buildroot}
(or $RPM_BUILD_ROOT). [26]
ok

# MUST: All filenames in rpm packages must be valid UTF-8. [27]
ok

Summary:
Fix rpmlint errors.
Check licences:
- Many times LGPL (v2.1) (with incorrect FSF address)
- aspell-0.60.6/modules/speller/default/affix.cpp: BSD (2 clause) 
- aspell-0.60.6/misc/po-filter.c: GPL (v2 or later) (with incorrect FSF
address) 
- aspell-0.60.6/ltmain.sh: GPL (v2 or later) 
- aspell-0.60.6/myspell/munch.c: BSD (2 clause) 
Remove using %{_datadir}/locale/*.

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

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


[Bug 542759] Review Request: mpqc - Ab-inito chemistry program

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


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





--- Comment #9 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-01 04:08:18 
EDT ---
- Are the .la files necessary for something? Get rid of them e.g. by running
 find %{buildroot}%{_libdir} -name *.la -exec rm -rf {} \;
at the end of %install.

- Try if --disable-static in %configure gets rid of the static library.

- Molrender needs to own
 %{_datadir}/molrender/
not just
 %{_datadir}/molrender/molrender.in

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

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


[Bug 542740] Review Request: f2c - The f2c Fortran to C/C++ conversion program and static libraries

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


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





--- Comment #8 from Jussi Lehtola jussi.leht...@iki.fi  2009-12-01 04:04:46 
EDT ---
(In reply to comment #7)
 I don't see the need to continue the strange release tag, 
 just change 3.0.3 to 4.  

Yes, I agree. You could even reset it to 1, since the package hasn't been in
Fedora for ages.

**

By the way, you're using Source, but Patch0. Please be consistent, and use
Source0 or drop the 0 from the patch.

**

Looking at changes on netlib, the latest entry is

Sat Apr 11 18:06:00 MDT 2009
  src/sysdep.c src/sysdeptest.c: tweak for MacOSX (include unistd.h).

so the version field should be updated to correspond to it (and the source be
updated if you are using the version from the old spec file).

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

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


[Bug 225286] Merge Review: aspell

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


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





--- Comment #20 from Roman Rakus rra...@redhat.com  2009-12-01 04:07:02 EDT 
---
Also check comment #17. Changes are not made in rawhide.

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

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


[Bug 495246] Review Request: perl-POE-Test-Loops - Reusable tests for POE::Loop authors

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


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


Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

 CC||steve.tray...@cern.ch
   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #5 from Steve Traylen steve.tray...@cern.ch  2009-12-01 04:09:55 
EDT ---
Package Change Request
==
Package Name: perl-POE-Test-Loops
New Branches: EL-5
Owners: stevetraylen


Hi
 I contacted Chris a few weeks ago and he was happy for me 
 to take perl-POE* in EPEL.

 Also explicitly for perl-POE-Test-Loops I asked again last week to confirm
 and no reply was given.

Steve

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

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


[Bug 542313] Review Request: QTeXEngine - Add-on for QtiPlot to enable the export of 2D plots to TeX

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


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





--- Comment #1 from supercyper supercy...@163.com  2009-12-01 04:17:06 EDT ---
koji build succeed http://koji.fedoraproject.org/koji/taskinfo?taskID=1839905

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

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


[Bug 541207] Review Request: liborigin2 - library for reading OriginLab Origin 7.5 project files

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


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





--- Comment #2 from supercyper supercy...@163.com  2009-12-01 04:17:02 EDT ---
koji build succeed http://koji.fedoraproject.org/koji/taskinfo?taskID=1839880

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

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


[Bug 541317] Review Request: perl-CGI-Application-Structured - Medium-weight, MVC, DB web framework

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


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





--- Comment #4 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 05:06:11 EDT ---
perl-CGI-Application-Structured-0.003-1.fc12 has been submitted as an update
for Fedora 12.
http://admin.fedoraproject.org/updates/perl-CGI-Application-Structured-0.003-1.fc12

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

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


[Bug 541317] Review Request: perl-CGI-Application-Structured - Medium-weight, MVC, DB web framework

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


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





--- Comment #5 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 05:06:16 EDT ---
perl-CGI-Application-Structured-0.003-1.fc11 has been submitted as an update
for Fedora 11.
http://admin.fedoraproject.org/updates/perl-CGI-Application-Structured-0.003-1.fc11

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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


Stefan Seidel redhat...@stefanseidel.info changed:

   What|Removed |Added

 CC||redhat...@stefanseidel.info




--- Comment #43 from Stefan Seidel redhat...@stefanseidel.info  2009-12-01 
05:08:31 EDT ---
Here are my 0.02 - I added build dependencies of gcc and gcc-c++, because it
won't build without them. Maybe also need to add make, I'm not sure.

Also, iconv {} -o {} gives a bunch of 0 byte files - not what is intented I
think. So I took the easy way out and used recode.

I was building this on CentOS 5.4, and with these changes and the removal of
lzma (package is not available here) and perl(extra) from the final
dependencies I was able to build and install and run 'dpkg -h' just fine.

--- dpkg.spec.latest2009-12-01 10:56:33.0 +0100
+++ dpkg.spec   2009-12-01 10:59:48.0 +0100
@@ -11,7 +11,7 @@
 Patch1: fedora-fix-paths.patch
 BuildRoot:  %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)

-BuildRequires:  zlib-devel, bzip2-devel, libselinux-devel, gettext,
ncurses-devel
+BuildRequires:  zlib-devel, bzip2-devel, libselinux-devel, gettext,
ncurses-devel, gcc, gcc-c++, recode

 %description

@@ -61,7 +61,7 @@
 chmod +x %{__perl_requires}

 # fix to get man pages translated to utf-8
-find man -type f -exec /usr/bin/iconv -f latin1 -t utf8 {} -o {} ';'
+find man -type f -exec /usr/bin/recode latin1..utf8 {} ';'

 %build
 %configure LDFLAGS='-lselinux' \
@@ -215,6 +215,10 @@


 %changelog
+* Tue Dec 1 2009 Stefan Seidel bugzilla...@stefanseidel.info - 1.15.4.1-2
+- add required build dependencies
+- fix wrong use of iconv
+
 * Tue Nov 10 2009 Andrew Colin Kissa and...@topdog.za.net - 1.15.4.1-1
 - Upgrade to latest upstream
 - review changes

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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





--- Comment #44 from Andrew Colin Kissa and...@topdog.za.net  2009-12-01 
05:19:28 EDT ---

- The dependencies are not required. -
http://fedoraproject.org/wiki/Packaging/Guidelines#Exceptions_2

I will test the other changes on fedora and update.

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

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


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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





--- Comment #2 from Mario Ceresa mrcer...@gmail.com  2009-12-01 05:32:38 EDT 
---
Following Mamoru comments on bug 540885, I looked at fedora people for an
alternative way to store the specs and srpms, but it seems I'd need to be
already sponsored in order to get the space. 

A friend of mine offered to host them for a while at:

ftp://85.54.198.199/FEDORA/InsightToolkit-3.16.0-1.fc12.src.rpm
ftp://85.54.198.199/FEDORA/InsightToolkit.spec

Hope this could ease the review process :)

Mario

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

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


[Bug 540885] Review Request: cableswig - Create interfaces to interpreted languages for templated code

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


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


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com
   Flag||fedora-review?




--- Comment #4 from Peter Lemenkov lemen...@gmail.com  2009-12-01 06:15:50 
EDT ---
I'll review it.

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

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


[Bug 540885] Review Request: cableswig - Create interfaces to interpreted languages for templated code

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


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


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

   What|Removed |Added

 CC||lemen...@gmail.com
 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #3 from Peter Lemenkov lemen...@gmail.com  2009-12-01 06:15:05 
EDT ---
Removing FE-NEEDSPONSOR - I just sponsored Mario

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

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


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|lemen...@gmail.com
   Flag||fedora-review?




--- Comment #4 from Peter Lemenkov lemen...@gmail.com  2009-12-01 06:16:28 
EDT ---
I'll review it.

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

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


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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


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

   What|Removed |Added

 CC||lemen...@gmail.com
 Blocks|177841(FE-NEEDSPONSOR)  |




--- Comment #3 from Peter Lemenkov lemen...@gmail.com  2009-12-01 06:15:18 
EDT ---
Removing FE-NEEDSPONSOR - I just sponsored Mario

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

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


[Bug 542991] New: Review Request: ArpON - Portable handler daemon with nice tools to handle all ARP aspects

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

Summary: Review Request: ArpON - Portable handler daemon with nice tools to 
handle all ARP aspects

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

   Summary: Review Request: ArpON - Portable handler daemon with
nice tools to handle all ARP aspects
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: s...@sandro-mathys.ch
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://red.fedorapeople.org/SRPMS/ArpON.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/ArpON-1.90-1.fc12.src.rpm
Description:
ArpON (Arp handler inspectiON) has a lot of features and makes Arp a bit
safer. This is possible using two kinds of anti Arp Poisoning techniques,
the first is based on SARPI or Static Arp Inspection, the second on
DARPI or Dynamic Arp Inspection approach.

Builds on i686 and x86_64:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1840199

rpmlint is clean:
$ rpmlint -i {SPECS,RPMS/i686,SRPMS}/ArpON*
3 packages and 1 specfiles checked; 0 errors, 0 warnings.

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

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


[Bug 542990] New: Review Request: root - Numerical data analysis framework

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

Summary: Review Request: root - Numerical data analysis framework

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

   Summary: Review Request: root - Numerical data analysis
framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: mattias.ell...@fysast.uu.se
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.grid.tsl.uu.se/review/root.spec
SRPM URL: http://www.grid.tsl.uu.se/review/root-5.24.00b-1.fc12.src.rpm

Description:
The ROOT system provides a set of object oriented frameworks with all
the functionality needed to handle and analyse large amounts of data
in a very efficient way. Having the data defined as a set of objects,
specialised storage methods are used to get direct access to the
separate attributes of the selected objects, without having to touch
the bulk of the data. Included are histogramming methods in 1, 2 and 3
dimensions, curve fitting, function evaluation, minimisation, graphics
and visualisation classes to allow the easy setup of an analysis
system that can query and process the data interactively or in batch
mode.

Thanks to the built in CINT C++ interpreter the command language, the
scripting, or macro, language and the programming language are all
C++. The interpreter allows for fast prototyping of the macros since
it removes the time consuming compile/link cycle. It also provides a
good environment to learn C++. If more performance is needed the
interactively developed macros can be compiled using a C++ compiler.

The system has been designed in such a way that it can query its
databases in parallel on MPP machines or on clusters of workstations
or high-end PCs. ROOT is an open system that can be dynamically
extended by linking external libraries. This makes ROOT a premier
platform on which to build data acquisition, simulation and data
analysis systems.

Koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=1839820

The ppc64 build fails with a segfault - so ExcludeArch is used

rpmlint output:

Root uses the CINT runtime C++ interpreter and therefore needs access to its
classed header files at runtime, so the headers are not devel files, even
though rpmlint complains about them.

Root also uses short snippets of C++ code in its configuration files that is
parsed by the CINT interpreter to do its plugin initialisation. The presence of
these C++ source files in the packages is not a packaging error, but rpmlint
complains about these too.

All the documentation is in the doc package so most of the other rpms complain
about missing documentation.

Filtering out the no-documentation and devel-file-in-non-devel-package
warnings, the following rpmlint output remains:

[ell...@ellert ~]$ rpmlint rpmbuild/RPMS/*/*root-* | sed -e /no-documentation/d
-e /devel-file-in-non-devel-package/d
root-doc.noarch: W: hidden-file-or-dir
/usr/share/doc/root-5.24.00b/test/RootShower/.rootshowerrc
root-doc.noarch: W: file-not-utf8 /usr/share/doc/root-5.24.00b/test/tstring.cxx
root-core.x86_64: E: rpath-in-buildconfig /usr/bin/root-config lines ['42']
root-proofd.x86_64: W: incoherent-init-script-name proofd ('root-proofd',
'root-proofdd')
root-rootd.x86_64: W: incoherent-init-script-name rootd ('root-rootd',
'root-rootdd')
79 packages and 0 specfiles checked; 1 errors, 2542 warnings.

The rpath-in-buildconfig error is just a simple string match picked up by
rpmlint. Root was configured with --disable-rpath, and the root-config script
generated by configure does the right thing:

[ell...@ellert root]$ root-config --has-rpath
no

[ell...@ellert root]$ root-config --libs
-L/usr/lib64/root -lCore -lCint -lRIO -lNet -lHist -lGraf -lGraf3d -lGpad
-lTree -lRint -lPostscript -lMatrix -lPhysics -lMathCore -lThread -lz -pthread
-lm -ldl -rdynamic

The string with the rpath is present in the root-config file, but is only
printed out when root's configure was run without --disable-rpath. So there is
no need to patch the script.

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

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


[Bug 542320] Review Request: plexus-lang - Plexus language bundle access component

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Flag||fedora-review?




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

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


[Bug 542320] Review Request: plexus-lang - Plexus language bundle access component

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 AssignedTo|nob...@fedoraproject.org|fed...@matbooth.co.uk




--- Comment #3 from Mat Booth fed...@matbooth.co.uk  2009-12-01 08:27:36 EDT 
---
I will take this one.

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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





--- Comment #45 from Ralf Corsepius rc040...@freenet.de  2009-12-01 08:33:04 
EDT ---
(In reply to comment #43)

 Also, iconv {} -o {} gives a bunch of 0 byte files - not what is intented I
 think.
Correct. It's a classic beginner's mistake to redirect a program's output to 
its input. You need to use a temporary file.

 So I took the easy way out and used recode.
Please don't. Use iconv.

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

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


[Bug 525927] Review Request: incollector - Information collector for various kinds of information

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


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


Kalev Lember ka...@smartlink.ee changed:

   What|Removed |Added

 CC||ka...@smartlink.ee
 AssignedTo|nob...@fedoraproject.org|ka...@smartlink.ee
   Flag||fedora-review?




--- Comment #1 from Kalev Lember ka...@smartlink.ee  2009-12-01 08:35:10 EDT 
---
Taking for review.

I have never dealt with orphaned packages before, but it appears that you have
already uploaded a newer version into CVS. I'll base my review on the following
files instead:

Spec URL:
http://cvs.fedoraproject.org/viewvc/rpms/incollector/devel/incollector.spec?view=markuppathrev=incollector-1_2-1_fc13
SRPM URL:
http://kojipkgs.fedoraproject.org/packages/incollector/1.2/1.fc13/src/incollector-1.2-1.fc13.src.rpm

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

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


[Bug 226102] Merge Review: lockdev

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


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





--- Comment #1 from Miroslav Lichvar mlich...@redhat.com  2009-12-01 09:09:40 
EDT ---
Review follows:

NO source files match upstream.
- source URL gives 404

YES package meets naming and versioning guidelines.
YES specfile is properly named, is cleanly written and uses macros
consistently.
YES dist tag is present.
YES build root is correct. (no build root specified)
NO license field matches the actual license.
- the actual license seems to be LGPLv2, under LGPLv2+ is licensed only
baudboy.h
YES license is open source-compatible.
NO License text included in package.
NO latest version is being packaged.
- is there a reason for not packaging 1.0.3?
YES BuildRequires are proper.
YES compiler flags are appropriate.
YES %clean is present.
YES package builds in mock (Rawhide/x86_64).
YES debuginfo package looks complete.
YES rpmlint output is sane.
lockdev.x86_64: E: setgid-binary /usr/sbin/lockdev lock 02711
lockdev.x86_64: E: non-standard-executable-perm /usr/sbin/lockdev 02711
lockdev.x86_64: E: non-standard-executable-perm /usr/sbin/lockdev 02711

These are ok.

YES final provides and requires look sane.
N/A %check is present and all tests pass.
YES shared libraries are added to the regular linker search paths, ldconfig
called in %post and %postun.
YES owns the directories it creates.
YES doesn't own any directories it shouldn't.
YES no duplicates in %files.
YES file permissions are appropriate.
YES code, not content.
YES documentation is small, so no -docs subpackage is necessary.
- documentation should include some files like AUTHORS, ChangeLog
YES %docs are not necessary for the proper functioning of the package.
YES headers packaged in -devel subpackage
YES no pkgconfig files.
YES no libtool .la droppings.
YES not a GUI app.

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

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


[Bug 226102] Merge Review: lockdev

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


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


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

   What|Removed |Added

   Flag||fedora-review?




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

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


[Bug 225286] Merge Review: aspell

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


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


Ivana Varekova varek...@redhat.com changed:

   What|Removed |Added

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




--- Comment #21 from Ivana Varekova varek...@redhat.com  2009-12-01 09:20:54 
EDT ---
Fixed.

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

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


[Bug 525927] Review Request: incollector - Information collector for various kinds of information

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


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





--- Comment #2 from Thomas Janssen thom...@fedoraproject.org  2009-12-01 
09:28:12 EDT ---
Yes, the old one was really old. And there was some build problem with devel
(mass rebuild IIRC). Sorry, i forgot to update the spec and srpm here. Will do
that today.

-- 
Fedora Bugzappers volunteer triage team
https://fedoraproject.org/wiki/BugZappers

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

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


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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





--- Comment #5 from Peter Lemenkov lemen...@gmail.com  2009-12-01 09:53:09 
EDT ---
Notes:

* I'm not sure about name. Perhaps 'itk' would be better name than
InsightToolkit?
* No need to re-define %{name} and %{version}. Just put correct values to
proper fields (Name: and Version:) at the top of the spec.
* Source0 url should be corrected. See Source1 for example.
* %{_libdir}/%{name}/*.cmake should be placed in devel rather than in main
package, I believe. Also, I'm not sure this is a correct place to put CMake
files in. 
* Unowned directory - %{_libdir}/%{name} . Just list it as %dir in main
package's %files section.
* Source1 should be added as %doc in devel-subpackage
* In the vast majority of cases, devel packages must require the base package
using a fully versioned dependency: Requires: %{name} = %{version}-%{release}
* Files within 'Copyrght' foler must be packages as %doc.
* Consider, also, packaging of 'Docmentation' and 'Examples' folders.

Please, comment my notes, and I'll continue.

BTW, since I sponsored you, you can store files at fedorapeople.org

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

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


[Bug 525927] Review Request: incollector - Information collector for various kinds of information

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


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





--- Comment #3 from Kalev Lember ka...@smartlink.ee  2009-12-01 09:58:15 EDT 
---
Thomas, I already got the spec and srpm from pkgdb / koji, no need to duplicate
those here.

Fedora review incollector-1.2-1.fc13.src.rpm 2009-12-01

+ OK
! needs attention

rpmlint says:
incollector.i686: E: no-binary
incollector.i686: W: only-non-binary-in-usr-lib
2 packages and 1 specfiles checked; 1 errors, 1 warnings.

All these rpmlint warnings/errors are expected for mono packages and can be
ignored.

+ rpmlint output
+ package is named according to the Package Naming Guidelines
+ specfile name matches base package name
+ package meets Packaging Guidelines
+ The stated license (GPLv2+) is a Fedora approved license
! The license doesn't match actual package license

Every source file contains the following lines:
 *  Copyright (C) 2006-2007 Marcin Krystian Krzywonos
 *  License: GNU/GPL version 2

I think this means that the license tag should read 'GPLv2'

+ The package contains the license file (COPYING)
+ Spec file is written in American English
+ Spec file is legible
+ Upstream sources match sources in the srpm. md5sum:
6701ac13da29119cd6719be3edcf30aa  incollector-1.2.tar.gz
6701ac13da29119cd6719be3edcf30aa  Download/incollector-1.2.tar.gz
+ Package builds in koji
n/a ExcludeArch bugs filed
+ BuildRequires look sane
+ Spec file handles locales properly
n/a binary RPM with shared library files must call ldconfig in %post and
%postun
+ Package doesn't bundle copies of system libraries
+ Does not use Prefix: /usr
+ Package owns all directories it creates
+ No duplicate files in %files
+ Proper permissions and %files has %defattr
+ %clean contains rm -rf %{buildroot}
+ Consistent use of macros
+ Package must contain code or permissible content
+ Package must contain code or permissible content
n/a Large documentation files should go in -doc subpackage
+ Files marked %doc should not affect package
n/a Header files should be in -devel
n/a Static libraries should be in -static
n/a Packages containing pkgconfig (.pc) files need 'Requires: pkgconfig'
n/a Library files that end in .so must go in a -devel package
n/a -devel must require the fully versioned base
+ Packages should not contain libtool .la files
+ Proper .desktop file handling
+ Package doesn't own files or directories already owned by other packages
+ %install begins with rm -rf %{buildroot}
+ all filename are valid UTF-8


! The following pushd/popd commands in the spec file without anything in
between seem useless and should be removed:
  pushd po
  popd

! You have the following sed command to fix lib64 dir:
  sed -i 's|/usr/lib|%{_libdir}|' %{name}

I think a proper way to do that is fix script.in instead (that's something you
could also send upstream):
-exec @MONO@ @prefix@/lib/incollector/incollector.exe $MONO_EXTRA_ARGS $@
+exec @MONO@ @pkglibdir@/incollector.exe $MONO_EXTRA_ARGS $@

Attaching the patch to the bug report in a separate file too.

! Consider removing %{_datadir}/pixmaps/%{name}.ico at the end of %install. I
very much doubt anything uses .ico files in Fedora when there's a matching .png
file in %{_datadir}/pixmaps/.

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

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


[Bug 539387] Review Request: InsightToolkit - Medical imaging processing library

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


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





--- Comment #6 from Peter Lemenkov lemen...@gmail.com  2009-12-01 10:00:44 
EDT ---
* I just found, that ITK contains numerous bundled libraries, many of them are
duplication Fedora's system ones - see 'Utilities' directory. This should be
fixed (and necessary BuildRequires should be added).

* Also I'm anxious about the contents of 'Code/Patented' folder.

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

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


[Bug 525927] Review Request: incollector - Information collector for various kinds of information

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


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





--- Comment #4 from Kalev Lember ka...@smartlink.ee  2009-12-01 10:02:39 EDT 
---
Created an attachment (id=375079)
 -- (https://bugzilla.redhat.com/attachment.cgi?id=375079)
Patch to fix hardcoded @prefix@/lib/ dir in wrapper script

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

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


[Bug 542990] Review Request: root - Numerical data analysis framework

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


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


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

   What|Removed |Added

 CC||juanucl...@gmail.com




--- Comment #1 from Peter Lemenkov lemen...@gmail.com  2009-12-01 10:05:34 
EDT ---
*** Bug 451744 has been marked as a duplicate of this bug. ***

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

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


[Bug 451744] Review Request: root - The CERN analyzer for high to medium energy physics

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


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


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

   What|Removed |Added

 Resolution|CANTFIX |DUPLICATE




--- Comment #39 from Peter Lemenkov lemen...@gmail.com  2009-12-01 10:05:34 
EDT ---


*** This bug has been marked as a duplicate of 542990 ***

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

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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


Christoph Wickert cwick...@fedoraproject.org changed:

   What|Removed |Added

 CC||cwick...@fedoraproject.org




--- Comment #10 from Christoph Wickert cwick...@fedoraproject.org  2009-12-01 
10:43:12 EDT ---
sed is ok, but should be done in %pre, so that package doesn't need to be
touched after installation. If possible, try to preserver the timestams of the
files, see 
https://fedoraproject.org/wiki/Packaging/Guidelines#Timestamps
https://fedoraproject.org/wiki/PackageMaintainers/PackagingTricks

I wonder it tracker is really sufficient or if tracker-search-tool is required.
I think it is.

%{_bindir}/mintmenu runs gconftool-2, so Requires: Gconf2 is missing.

%{_bindir}/mintmenu should be installed in %{_libexecdir}

For licensing reasons, you are not allowed to include any Fedora icons in the
package. You must use the logos provided by the fedora-logos package and
Require: system-logos. system-logos is a virtual provides, so the package can
be rebranded.
If possible, use a generic name for the icon in the code, e.g.
icon-panel-menu.png which is provided by different themes and on several
distros such as Rhel or CentOS.

Not sure about license of the mint icon, I think best is to leave it out.

Disable building the empty debuginfo package with %global debug_package
%{nil} at the beginning of the spec.

/usr/lib64/bonobo/servers/mintMenu.server must not be executable, this is what
causes the script-without-shebang error in rpmlint. I'd rather use install than
cp because you can set proper permissions of the files.

(In reply to comment #6)
 How does one specify /usr/lib in a no arch package without using an absolute
 path?  This is where upstream places the files.

%{_prefix}/lib, see http://fedoraproject.org/wiki/Packaging:RPMMacros
But I agree with Terje that the noarch stuff should be moved to %{_datadir}.

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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





--- Comment #46 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-12-01 
10:56:45 EDT ---
(In reply to comment #43)
 Also, iconv {} -o {} gives a bunch of 0 byte files - not what is intented I
 think. So I took the easy way out and used recode.

[tasa...@localhost TEMP]$ echo 漢字が入った例文です  kanji.txt 
[tasa...@localhost TEMP]$ ls -al kanji.txt 
-rw--- 1 tasaka1 tasaka1 31 2009-12-02 00:52 kanji.txt
[tasa...@localhost TEMP]$ iconv -f UTF-8 -t EUC-JP kanji.txt -o kanji.txt 
[tasa...@localhost TEMP]$ ls -la kanji.txt 
-rw--- 1 tasaka1 tasaka1 21 2009-12-02 00:52 kanji.txt
[tasa...@localhost TEMP]$ iconv -f EUC-JP -t UTF-8 kanji.txt -o kanji.txt 
[tasa...@localhost TEMP]$ ls -al kanji.txt 
-rw--- 1 tasaka1 tasaka1 31 2009-12-02 00:53 kanji.txt

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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





--- Comment #47 from Andrew Colin Kissa and...@topdog.za.net  2009-12-01 
11:07:03 EDT ---

In rely to comment #46, #43

That's exactly what i thought, I have double checked on my fedora build
machine, i do not get any 0 byte files.

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

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


[Bug 226165] Merge Review: mt-st

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


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


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

   What|Removed |Added

 CC||dho...@redhat.com
   Flag||fedora-review?




--- Comment #4 from Miroslav Lichvar mlich...@redhat.com  2009-12-01 11:55:26 
EDT ---
Review follows:

OK source files match upstream:
fdd5f5ec673c9f630a102ceff7612774  mt-st-1.1.tar.gz

OK package meets naming and versioning guidelines.
OK specfile is properly named, is cleanly written and uses macros consistently.
OK dist tag is present.
OK build root is correct.
OK license field matches the actual license.
OK license is open source-compatible. License text included in package.
OK latest version is being packaged.
OK BuildRequires are proper.
OK compiler flags are appropriate.
OK %clean is present.
OK package builds in mock.
OK debuginfo package looks complete.
NO rpmlint is silent.
mt-st.x86_64: W: service-default-enabled /etc/rc.d/init.d/stinit
mt-st.x86_64: E: no-status-entry /etc/rc.d/init.d/stinit
mt-st.x86_64: W: no-reload-entry /etc/rc.d/init.d/stinit
mt-st.x86_64: E: subsys-not-used /etc/rc.d/init.d/stinit
mt-st.x86_64: W: incoherent-init-script-name stinit ('mt-st', 'mt-std')

Enabled by default and incoherent name are ok, but the script should be
probably updated to match the Fedora init script guidelines.

OK final provides and requires look sane.
N/A %check is present and all tests pass.
OK no shared libraries are added to the regular linker search paths.
OK owns the directories it creates.
OK doesn't own any directories it shouldn't.
OK no duplicates in %files.
OK file permissions are appropriate.
OK scriptlets are sane.
OK code, not content.
OK documentation is small, so no -docs subpackage is necessary.
OK %docs are not necessary for the proper functioning of the package.
OK no headers.
OK no pkgconfig files.
OK no libtool .la droppings.
OK not a GUI app.

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

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


[Bug 542028] Review Request: php-pear-Spreadsheet-Excel-Writer - Package for generating Excel spreadsheets

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


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





--- Comment #5 from Remi Collet fed...@famillecollet.com  2009-12-01 12:19:24 
EDT ---
Please, fix
 Requires: php-pear-(OLE)
 Requires: php-pear(OLE)

%clean section is mandatory

Please update to latest version 0.9.2 (great, this package seems back to life)
as it fixes a number of bugs, especially php 5.3 compatibility.

package.xml in 0.9.1 is an old V1 one, in 0.9.2 is a new V2

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

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


[Bug 542754] Review Request: artha - A handy thesaurus based on WordNet

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


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





--- Comment #6 from Roshan Singh singh.rosha...@gmail.com  2009-12-01 
12:31:21 EDT ---
I have made the necessary changes. Here are the updated files.

SPEC: http://lug.nitdgp.ac.in/users/roshan/artha/artha.spec
SRPM: http://lug.nitdgp.ac.in/users/roshan/artha/artha-0.9.1-2.fc11.src.rpm

The project artha is licensed under GPLv2, will it be okay to change it GPLv2+
?

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

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


[Bug 457210] Review Request: dpkg - Package maintenance system for Debian Linux

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


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





--- Comment #48 from Stefan Seidel redhat...@stefanseidel.info  2009-12-01 
12:30:36 EDT ---
Probably the effects of
http://sourceware.org/git/?p=glibc.git;a=commitdiff;h=ca668b298e9b8c2c191eae259fddfed5e03a8dd4

Someone complained at http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=439720

On my Debian Etch, Lenny and CentOS 5.4 Systems with glibc versions ranging
from 2.3.6 to 2.7 it gives 0 bytes, but not on Ubuntu glibc 2.10.1. Anyway,
here's the glibc-independent version:

+find man -type f -exec /usr/bin/iconv -f latin1 -t utf8 {} -o {}._utf8 ';'
-exec mv -f {}._utf8 {} ';'

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

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


[Bug 533723] Review Request: perl-Guard - Safe cleanup blocks

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


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


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

   What|Removed |Added

   Fixed In Version|1.021-1.fc11|1.021-1.el5




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

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


[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server

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


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


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

   What|Removed |Added

   Fixed In Version|3.17-2.fc12 |3.17-2.el5




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

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


[Bug 194559] Review Request: perl-Event

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


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





--- Comment #8 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:26:12 EDT ---
perl-Event-1.13-1.el5 has been pushed to the Fedora EPEL 5 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 194559] Review Request: perl-Event

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


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





--- Comment #9 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:29:03 EDT ---
perl-Event-1.13-1.el4 has been pushed to the Fedora EPEL 4 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 533723] Review Request: perl-Guard - Safe cleanup blocks

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


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





--- Comment #16 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:25:17 EDT ---
perl-Guard-1.021-1.el5 has been pushed to the Fedora EPEL 5 stable repository. 
If problems still persist, please make note of it in this bug report.

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

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


[Bug 526876] Review Request: php-pecl-gmagick - Provides a wrapper to the GraphicsMagick library

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


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





--- Comment #15 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:26:24 EDT ---
php-pecl-gmagick-1.0.2b1-3.el5 has been pushed to the Fedora EPEL 5 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 533721] Review Request: perl-Net-CIDR - Manipulate IPv4/IPv6 netblocks in CIDR notation

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


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


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

   What|Removed |Added

   Fixed In Version|0.13-2.fc12 |0.13-2.el5




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

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


[Bug 533721] Review Request: perl-Net-CIDR - Manipulate IPv4/IPv6 netblocks in CIDR notation

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


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





--- Comment #19 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:28:32 EDT ---
perl-Net-CIDR-0.13-2.el5 has been pushed to the Fedora EPEL 5 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server

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


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





--- Comment #12 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:24:59 EDT ---
globus-gridftp-server-3.17-2.el5 has been pushed to the Fedora EPEL 5 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server

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


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





--- Comment #13 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:25:29 EDT ---
globus-gridftp-server-3.17-2.el4 has been pushed to the Fedora EPEL 4 stable
repository.  If problems still persist, please make note of it in this bug
report.

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

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


[Bug 516517] Review Request: globus-gridftp-server - Globus Toolkit - Globus GridFTP Server

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


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


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

   What|Removed |Added

   Fixed In Version|3.17-2.el5  |3.17-2.el4




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

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


[Bug 526876] Review Request: php-pecl-gmagick - Provides a wrapper to the GraphicsMagick library

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


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


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
   Fixed In Version||1.0.2b1-3.el5
 Resolution||ERRATA




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

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


[Bug 526998] Review Request: volpack - Portable library for fast volume rendering

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


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


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

   What|Removed |Added

 Status|ASSIGNED|ON_QA




--- Comment #18 from Fedora Update System upda...@fedoraproject.org  
2009-12-01 13:27:00 EDT ---
volpack-1.0c7-3.el5 has been pushed to the Fedora EPEL 5 testing repository. 
If problems still persist, please make note of it in this bug report.
 If you want to test the update, you can install it with 
 su -c 'yum --enablerepo=updates-testing update volpack'.  You can provide
feedback for this update here:
http://admin.fedoraproject.org/updates/EL-5/FEDORA-EPEL-2009-0926

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

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


[Bug 536684] Review Request: eclipse-mdt-ocl - Implementation of the OCL OMG metamodel for Eclipse

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #5 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:45:37 EDT 
---
CVS done.

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

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


[Bug 529254] Review Request: i3 - Improved tiling window manager

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:48:28 EDT 
---
CVS done.

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

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


[Bug 537451] Review Request: eclipse-emf-query - Specify and execute queries against EMF models

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #6 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:46:17 EDT 
---
CVS done.

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

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


[Bug 515230] Review Request: python-dmidecode - python extension module to access DMI data

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #11 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:46:54 
EDT ---
CVS done.

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

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


[Bug 510195] Review Request: eclipse-slice2java - A plugin that integrates Eclipse with Ice object middleware

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #12 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:44:55 
EDT ---
CVS done.

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

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


[Bug 529255] Review Request: i3lock - A slightly improved version of slock

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #7 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 13:51:21 EDT 
---
Is there some reason the Short Description here is the one that I originally
complained about instead of the one that's currently in the spec?  I went ahead
and used the one from the spec, but if for some reason that's not what you
wanted then please let me know.

CVS done.

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

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


[Bug 529256] Review Request: i3status - Generates a status line for dzen2 or wmii

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 14:05:03 EDT 
---
CVS done.

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

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


[Bug 466379] Review Request: zfs-fuse - ZFS ported to Linux FUSE

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #29 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 14:08:05 
EDT ---
There's already an F-12 branch for this package; I'm not sure why it's being
requested here.  Make sure you do cvs update -d to get any newly created
directories when you update.

I've made the EL-5 branch.

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

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


[Bug 541154] Review Request: xfce-volumed - Daemon to add additional functionality to the volume keys of the keyboard

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |xfce4-volumed - Daemon to   |xfce-volumed - Daemon to
   |add additional  |add additional
   |functionality to the volume |functionality to the volume
   |keys of the keyboard|keys of the keyboard
   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 14:09:34 EDT 
---
CVS done; I corrected the summary of this ticket to match the requested package
name.

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

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


[Bug 539912] Review Request: javacsv - Stream-based Java library for reading and writing CSV and other delimited data

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #8 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 14:11:10 EDT 
---
CVS done.

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

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


[Bug 533765] Review Request: fedora-accessibility-guide-en-US - Using Fedora with a visual, hearing, or mobility impairment

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


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


Jason Tibbitts ti...@math.uh.edu changed:

   What|Removed |Added

   Flag|fedora-cvs? |fedora-cvs+




--- Comment #4 from Jason Tibbitts ti...@math.uh.edu  2009-12-01 14:10:23 EDT 
---
CVS done.

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

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


[Bug 543147] New: Review Request: mingw32-SDL_image - MinGW Windows port of the Image loading library for SDL

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

Summary: Review Request: mingw32-SDL_image - MinGW Windows port of the Image 
loading library for SDL

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

   Summary: Review Request: mingw32-SDL_image - MinGW Windows port
of the Image loading library for SDL
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fgfs.ste...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://riemens.org/fs/temp/mingw32-SDL_image.spec
SRPM URL: mingw32-SDL_image-1.2.7-1.fc12.src.rpm
Description:
Simple DirectMedia Layer (SDL) is a cross-platform multimedia library
designed to provide fast access to the graphics frame buffer and audio
device.  This package contains a simple library for loading images of
various formats (BMP, TIF, JPEG, PNG) as SDL surfaces.

%{_mingw32_description}

Note: see also
http://www.mail-archive.com/fedora-mi...@lists.fedoraproject.org/msg01505.html

Thanks,
Stefan

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

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


[Bug 543151] New: Review Request: erlang-exmpp - A library for the eXtensible Messaging and Presence Protocol

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

Summary: Review Request: erlang-exmpp - A library for the eXtensible Messaging 
and Presence Protocol

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

   Summary: Review Request: erlang-exmpp - A library for the
eXtensible Messaging and Presence Protocol
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: lemen...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://peter.fedorapeople.org/erlang-exmpp.spec
SRPM URL: http://peter.fedorapeople.org/erlang-exmpp-0.9.1-1.fc12.src.rpm
Description: Erlang XMPP library (exmpp) is a fast and scalable library for the
eXtensible
Messaging and Presence Protocol.


Koji scratchbuild:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1841907

rpmlint:
[pe...@sulaco SPECS]$ rpmlint ../RPMS/ppc/erlang-exmpp-*
erlang-exmpp.ppc: E: invalid-soname
/usr/lib/erlang/lib/exmpp-0.9.1/priv/lib/exmpp_xml_expat_legacy.so
exmpp_xml_expat_legacy.so
erlang-exmpp.ppc: E: invalid-soname
/usr/lib/erlang/lib/exmpp-0.9.1/priv/lib/exmpp_compress_zlib.so
exmpp_compress_zlib.so
erlang-exmpp.ppc: E: invalid-soname
/usr/lib/erlang/lib/exmpp-0.9.1/priv/lib/exmpp_xml_expat.so exmpp_xml_expat.so
erlang-exmpp.ppc: E: invalid-soname
/usr/lib/erlang/lib/exmpp-0.9.1/priv/lib/exmpp_stringprep.so
exmpp_stringprep.so
erlang-exmpp.ppc: E: invalid-soname
/usr/lib/erlang/lib/exmpp-0.9.1/priv/lib/exmpp_tls_openssl.so
exmpp_tls_openssl.so
2 packages and 0 specfiles checked; 5 errors, 0 warnings.
[pe...@sulaco SPECS]$

These messages can be ignored since they mean that so-name has no suffix (which
is normal for erlang C-modules).

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

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


[Bug 543154] New: Review Request: mingw32-SDL_mixer - Simple DirectMedia Layer's Sample Mixer Library

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

Summary: Review Request: mingw32-SDL_mixer - Simple DirectMedia Layer's Sample 
Mixer Library

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

   Summary: Review Request: mingw32-SDL_mixer - Simple DirectMedia
Layer's Sample Mixer Library
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: fgfs.ste...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.riemens.org/fs/temp/mingw32-SDL_mixer.spec
SRPM URL: http://www.riemens.org/fs/temp/mingw32-SDL_mixer-1.2.8-3.fc12.src.rpm
Description:
A simple multi-channel audio mixer for SDL. It supports 4 channels of
16 bit stereo audio, plus a single channel of music, mixed by the popular
MikMod MOD library.

%{_mingw32_description}

Note: see also
http://www.mail-archive.com/fedora-mi...@lists.fedoraproject.org/msg01505.html

Koji scratch build: https://koji.fedoraproject.org/koji/taskinfo?taskID=1841931

Thanks,
Stefan

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

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


[Bug 541154] Review Request: xfce4-volumed - Daemon to add additional functionality to the volume keys of the keyboard

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


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


Christoph Wickert cwick...@fedoraproject.org changed:

   What|Removed |Added

Summary|Review Request: |Review Request:
   |xfce-volumed - Daemon to|xfce4-volumed - Daemon to
   |add additional  |add additional
   |functionality to the volume |functionality to the volume
   |keys of the keyboard|keys of the keyboard
   Flag|fedora-cvs+ |fedora-cvs?




--- Comment #5 from Christoph Wickert cwick...@fedoraproject.org  2009-12-01 
14:25:55 EDT ---
(In reply to comment #4)
 CVS done; I corrected the summary of this ticket to match the requested 
 package
 name.  

Sorry, that was a typo in the cvs request. As you can see both the source and
the package name is xfce4-volumed.

New Package CVS Request
===
Package Name: xfce4-volumed
Short Description: Daemon to add additional functionality to the volume keys of
the keyboard
Owners: cwickert
Branches: F-11 F-12
InitialCC:

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

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


[Bug 543147] Review Request: mingw32-SDL_image - MinGW Windows port of the Image loading library for SDL

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


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





--- Comment #1 from Stefan Riemens fgfs.ste...@gmail.com  2009-12-01 14:30:11 
EDT ---
Forgot to mention the koji scratch build:
https://koji.fedoraproject.org/koji/taskinfo?taskID=1841893

Thanks,
Stefan

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

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


[Bug 543159] New: Review Request: zeitgeist - Framework providing Desktop activity awareness

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

Summary: Review Request: zeitgeist -  Framework providing Desktop activity 
awareness

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

   Summary: Review Request: zeitgeist -  Framework providing
Desktop activity awareness
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: dakin...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://deji.fedorapeople.org/zeitgeist.spec
SRPM URL: http://deji.fedorapeople.org/zeitgeist-0.3.0-1.fc12.src.rpm
Description: 
Zeitgeist is a service which logs the users's activities and events (files
opened, websites visites, conversations hold with other people, etc.) and makes
relevant information available to other applications. 

Note that this package only contains the daemon, which you can use
together with several different user interfaces.

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

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


[Bug 536684] Review Request: eclipse-mdt-ocl - Implementation of the OCL OMG metamodel for Eclipse

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #6 from Mat Booth fed...@matbooth.co.uk  2009-12-01 15:16:38 EDT 
---
Built successfully for rawhide, closing.

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

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


[Bug 537452] Review Request: eclipse-emf-validation - Verify the integrity of EMF models

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


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


Bug 537452 depends on bug 536684, which changed state.

Bug 536684 Summary: Review Request: eclipse-mdt-ocl - Implementation of the OCL 
OMG metamodel for Eclipse
https://bugzilla.redhat.com/show_bug.cgi?id=536684

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 537451] Review Request: eclipse-emf-query - Specify and execute queries against EMF models

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


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


Bug 537451 depends on bug 536684, which changed state.

Bug 536684 Summary: Review Request: eclipse-mdt-ocl - Implementation of the OCL 
OMG metamodel for Eclipse
https://bugzilla.redhat.com/show_bug.cgi?id=536684

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 537652] Review Request: eclipse-m2m-qvtoml - Implementation of Operational QVT for Eclipse

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


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


Bug 537652 depends on bug 536684, which changed state.

Bug 536684 Summary: Review Request: eclipse-mdt-ocl - Implementation of the OCL 
OMG metamodel for Eclipse
https://bugzilla.redhat.com/show_bug.cgi?id=536684

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 542436] Review Request: python-cloudfiles - Python language bindings for Rackspace CloudFiles API

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


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





--- Comment #5 from Andrew Turner acturne...@gmail.com  2009-12-01 15:35:28 
EDT ---
Updated re:Comment #4 above:-

http://c0456722.cdn.cloudfiles.rackspacecloud.com/python-cloudfiles.spec
http://c0456722.cdn.cloudfiles.rackspacecloud.com/python-cloudfiles-1.6.0-3.fc12.src.rpm

Thanks again for the guidance.

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

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


[Bug 537452] Review Request: eclipse-emf-validation - Verify the integrity of EMF models

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #7 from Mat Booth fed...@matbooth.co.uk  2009-12-01 16:23:20 EDT 
---
Built successfully in rawhide, closing.

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

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


[Bug 537454] Review Request: eclipse-emf-transaction - A model management layer for managing EMF resources

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


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


Bug 537454 depends on bug 537451, which changed state.

Bug 537451 Summary: Review Request: eclipse-emf-query - Specify and execute 
queries against EMF models
https://bugzilla.redhat.com/show_bug.cgi?id=537451

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



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

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


[Bug 537451] Review Request: eclipse-emf-query - Specify and execute queries against EMF models

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #7 from Mat Booth fed...@matbooth.co.uk  2009-12-01 16:23:09 EDT 
---
Built successfully in rawhide, closing.

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

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


[Bug 537454] Review Request: eclipse-emf-transaction - A model management layer for managing EMF resources

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #5 from Mat Booth fed...@matbooth.co.uk  2009-12-01 16:23:28 EDT 
---
Built successfully in rawhide, closing.

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

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


[Bug 537652] Review Request: eclipse-m2m-qvtoml - Implementation of Operational QVT for Eclipse

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


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


Mat Booth fed...@matbooth.co.uk changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #5 from Mat Booth fed...@matbooth.co.uk  2009-12-01 16:23:39 EDT 
---
Built successfully in rawhide, closing.

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

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


[Bug 537431] Review Request: mono-bouncycastle - Bouncy Castle Crypto Package for Mono

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


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





--- Comment #3 from Kalev Lember ka...@smartlink.ee  2009-12-01 16:46:18 EDT 
---
* Tue Dec 01 2009 Kalev Lember ka...@smartlink.ee - 1.5-2
- Use the system mono.snk key instead of regenerating on every build

Spec URL: http://kalev.fedorapeople.org/mono-bouncycastle.spec
SRPM URL: http://kalev.fedorapeople.org/mono-bouncycastle-1.5-2.fc13.src.rpm

Scratch build for rawhide now that nant is working again:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1842242

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

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


[Bug 188105] Review Request: torque

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


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


Steve Traylen steve.tray...@cern.ch changed:

   What|Removed |Added

 CC||steve.tray...@cern.ch
   Flag||fedora-cvs?




--- Comment #17 from Steve Traylen steve.tray...@cern.ch  2009-12-01 17:54:12 
EDT ---
Package Change Request
==
Package Name: torque
New Branches: EL-4 EL-5
Owners: stevetraylen
InitialCC: garrick


I sent mails to the owner on September 23rd and more explicitly
on November 25th requesting the EPEL branches.
Also bug 479672 has been present for a while.

Steve Traylen.

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

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


[Bug 540617] Review Request: django-lint - lint for (python) django web-framework

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


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


Graeme Gillies ggill...@redhat.com changed:

   What|Removed |Added

 CC||ggill...@redhat.com




--- Comment #2 from Graeme Gillies ggill...@redhat.com  2009-12-01 18:07:11 
EDT ---
Having a quick build and running rpmlint over the spec, SRPM, and RPM produce
the following errors warnings.

$ rpmlint django-lint-0.11-3.fc12.noarch.rpm 
django-lint.noarch: E: description-line-too-long Django Lint is a static
analysis tool that checks (or lints) projects and applications that use the
Django web development framework.
django-lint.noarch: E: description-line-too-long It reports on common
programming errors and bad code smells, including checking for nullable
CharField field types, the use of brittle or deprecated Django features (such
as auto_now_add) as well as the absence of recommended options in settings.py.
It aims to encourage the development of high-quality re-usable Django
applications.
django-lint.noarch: W: no-version-in-last-changelog
django-lint.noarch: E: non-executable-script
/usr/lib/python2.6/site-packages/DjangoLint/script.py 0644 /usr/bin/env
1 packages and 0 specfiles checked; 3 errors, 1 warnings.

$ rpmlint django-lint-0.11-3.fc12.src.rpm 
django-lint.src: E: description-line-too-long Django Lint is a static analysis
tool that checks (or lints) projects and applications that use the Django web
development framework.
django-lint.src: E: description-line-too-long It reports on common programming
errors and bad code smells, including checking for nullable CharField field
types, the use of brittle or deprecated Django features (such as auto_now_add)
as well as the absence of recommended options in settings.py. It aims to
encourage the development of high-quality re-usable Django applications.
django-lint.src: W: no-version-in-last-changelog
1 packages and 0 specfiles checked; 2 errors, 1 warnings.

These should be relatively trivial to fix (wrap the lines containing
description and change permissions on script.py)

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

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


[Bug 541724] Review Request: gtk-chtheme - Gtk+ 2.0 theme preview and selection made slick

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


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





--- Comment #1 from Vadim Raskhozhev iamde...@gmail.com  2009-12-01 19:28:49 
EDT ---
Sorry, the links above are broken due to dead hosting. Correct links are below:
Spec URL:
http://www.ddt.cs.vsu.ru/dexpl/rpms/fc12/gtk-chtheme-0.3.1/gtk-chtheme.spec
SRPM URL:
http://www.ddt.cs.vsu.ru/dexpl/rpms/fc12/gtk-chtheme-0.3.1/gtk-chtheme-0.3.1-2.fc12.src.rpm

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

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


[Bug 543248] Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON

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


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


Ben Boeckel maths...@gmail.com changed:

   What|Removed |Added

 Blocks||532499




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

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


[Bug 543248] New: Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON

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

Summary: Review Request: python-jsonpickle - A module that allows any object to 
be serialized into JSON

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

   Summary: Review Request: python-jsonpickle - A module that
allows any object to be serialized into JSON
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: maths...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://benboeckel.net/packaging/python-jsonpickle/python-jsonpickle.spec
SRPM URL:
http://benboeckel.net/packaging/python-jsonpickle/python-jsonpickle-0.2.0-1.fc12.src.rpm
Description:
This module allows python objects to be serialized to JSON in a similar fashion
to the pickle module.

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

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


[Bug 543248] Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON

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


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


Ryan Rix phrkonale...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||phrkonale...@gmail.com
 AssignedTo|nob...@fedoraproject.org|phrkonale...@gmail.com




--- Comment #1 from Ryan Rix phrkonale...@gmail.com  2009-12-01 19:46:07 EDT 
---
I got this

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

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


[Bug 542754] Review Request: artha - A handy thesaurus based on WordNet

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


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





--- Comment #7 from Arun SAG saga...@gmail.com  2009-12-01 21:06:32 EDT ---
(In reply to comment #6)
 I have made the necessary changes. Here are the updated files.
 The project artha is licensed under GPLv2, will it be okay to change it GPLv2+
 ?  


No.

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

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


[Bug 542740] Review Request: f2c - The f2c Fortran to C/C++ conversion program and static libraries

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


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





--- Comment #9 from Carl Byington c...@five-ten-sg.com  2009-12-01 21:59:09 
EDT ---
Source updated from netlib, 0 dropped from patch, patch redone to correspond to
the latest source, release tag reset to 1, alpha arch dropped. This latest f2c
seems to be compatible with the rest of ghemical.

http://www.five-ten-sg.com/f2c.spec
http://www.five-ten-sg.com/f2c-20090411-1.fc12.src.rpm

scratch build:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1842593

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

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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





--- Comment #11 from William Witt will...@witt-family.net  2009-12-01 
22:06:42 EDT ---

(In reply to comment #10)
 sed is ok, but should be done in %pre, so that package doesn't need to be
 touched after installation. If possible, try to preserver the timestams of the
 files, see 

moved sed-foo to %pre

 I wonder it tracker is really sufficient or if tracker-search-tool is 
 required.
 I think it is.

changed...

 %{_bindir}/mintmenu runs gconftool-2, so Requires: Gconf2 is missing.

Added

 %{_bindir}/mintmenu should be installed in %{_libexecdir}

I have to disagree on this one, libexdir takes it off the path.  The mintmenu
script allows the menu to be cleaned or or launched from the command line in
its own window.

 For licensing reasons, you are not allowed to include any Fedora icons in the
 package. You must use the logos provided by the fedora-logos package and
 Require: system-logos. system-logos is a virtual provides, so the package can
 be rebranded.
 If possible, use a generic name for the icon in the code, e.g.
 icon-panel-menu.png which is provided by different themes and on several
 distros such as Rhel or CentOS.
 
 Not sure about license of the mint icon, I think best is to leave it out.


removed both mint and fedora logos, replaced with
/usr/share/icons/Bluecurve/24x24/apps/icon-panel-menu.png

 
 Disable building the empty debuginfo package with %global debug_package
 %{nil} at the beginning of the spec.
 

Done

 /usr/lib64/bonobo/servers/mintMenu.server must not be executable, this is what
 causes the script-without-shebang error in rpmlint. I'd rather use install 
 than
 cp because you can set proper permissions of the files.

Done

 (In reply to comment #6)
  How does one specify /usr/lib in a no arch package without using an absolute
  path?  This is where upstream places the files.
 
 %{_prefix}/lib, see http://fedoraproject.org/wiki/Packaging:RPMMacros
 But I agree with Terje that the noarch stuff should be moved to %{_datadir}.  

I moved as much as I can to %{_datadir}, but as I said the bonobo stuff needs
to be in /usr/lib or /usr/lib64, using %{_prefix}/lib sets off hardcoded
errors. I don't think there is a way around that while still keeping the
package no arch, which it really should be.  If this is acceptable, I can keep
it this way.


[unama...@gimli ~]$ rpmlint
/home/unamanic/rpmbuild/RPMS/noarch/mintmenu-4.9.0-10.fc12.noarch.rpm
mintmenu.noarch: W: no-documentation
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
[unama...@gimli ~]$ rpmlint
/home/unamanic/rpmbuild/SRPMS/mintmenu-4.9.0-10.fc12.src.rpm
mintmenu.src:73: E: hardcoded-library-path in
%{buildroot}/usr/lib/linuxmint/mintMenu/mint*.png
mintmenu.src:74: E: hardcoded-library-path in
%{buildroot}/usr/lib/linuxmint/mintMenu/mint*.svg
mintmenu.src:82: E: hardcoded-library-path in %{_prefix}/lib/bonobo/servers/
mintmenu.src:83: E: hardcoded-library-path in
%{_prefix}/lib/bonobo/servers/mintMenu.server
mintmenu.src:91: E: hardcoded-library-path in
%{_prefix}/lib/bonobo/servers/mintMenu.server
1 packages and 0 specfiles checked; 5 errors, 0 warnings.

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=1842604
Spec URL: http://www.witt-family.net/mintmenu.spec
SRPM URL: http://www.witt-family.net/mintmenu-4.9.0-10.fc12.src.rpm

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

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


[Bug 529374] Review Request: ethos - Plugin framework for GLib

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


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





--- Comment #5 from Peter Robinson pbrobin...@gmail.com  2009-12-01 22:27:50 
EDT ---
Updated the spec to the latest official release.

SPEC as before.
SRPM: http://pbrobinson.fedorapeople.org/ethos-0.2.2-1.fc12.src.rpm
koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=1842620

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

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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





--- Comment #12 from William Witt will...@witt-family.net  2009-12-01 
22:33:45 EDT ---
Do not use the previous koji build

(In reply to comment #11)
 (In reply to comment #10)
  sed is ok, but should be done in %pre, so that package doesn't need to be
  touched after installation. If possible, try to preserver the timestams of 
  the
  files, see 
 
 moved sed-foo to %pre
 

realized the sed in %pre with is not what is needed, moved it back to %install. 

koji: https://koji.fedoraproject.org/koji/taskinfo?taskID=1842623
Spec URL: http://www.witt-family.net/mintmenu.spec
SRPM URL: http://www.witt-family.net/mintmenu-4.9.0-11.fc12.src.rpm

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

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


[Bug 542292] Review Request: mintmenu - gnome-panel menu from LinuxMint

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


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





--- Comment #13 from William Witt will...@witt-family.net  2009-12-01 
22:36:38 EDT ---
 koji should read: http://koji.fedoraproject.org/koji/taskinfo?taskID=1842625

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

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


[Bug 538465] Review Request: libmx - A clutter toolkit for Moblin

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


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





--- Comment #2 from Peter Robinson pbrobin...@gmail.com  2009-12-01 22:43:55 
EDT ---
New upstream
SRPM: http://pbrobinson.fedorapeople.org/libmx-0.2.0-1.fc12.src.rpm

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

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

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


  1   2   >