[Bug 456527] Review Request: sil-gentium-basic-fonts - Gentium Basic Font Family

2009-01-30 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=456527





--- Comment #23 from Nicolas Mailhot nicolas.mail...@laposte.net  2009-01-30 
03:32:38 EDT ---
(In reply to comment #22)

  (or sil-gentium-book-basic-fonts if you insist, 

This one won't work, use the other

-- 
Configure 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 483205] Review Request: eclipse-SystemTapGui - GUI interface for SystemTap

2009-01-30 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=483205


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||panem...@gmail.com




--- Comment #1 from Parag AN(पराग) panem...@gmail.com  2009-01-30 03:33:02 
EDT ---
I have not done any java or eclipse related package review yet but let me try
this package for preliminary review.

1) you should use links for sourceforge as
http://nchc.dl.sourceforge.net/sourceforge/stapgui/eclipse-SystemTapGui.spec
http://downloads.sourceforge.net/stapgui/eclipse-SystemTapGui-1.0-1.fc10.src.rpm

2) Source0 should be
Source0:
http://downloads.sourceforge.net/stapgui/org.eclipse.SystemTapGui.src.tar.gz

3) From
http://fedoraproject.org/wiki/Packaging/EclipsePlugins#Specfile_Template, you
should have %install as

%install
rm -rf $RPM_BUILD_ROOT
install -d -m 755 $RPM_BUILD_ROOT%{_datadir}/eclipse/dropins/SystemTapGui
unzip -q -d $RPM_BUILD_ROOT%{_datadir}/eclipse/dropins/SystemTapGui \
build/rpmBuild/org.eclipse.SystemTapGui.systemtap.feature.zip


3) From
http://fedoraproject.org/wiki/Packaging/EclipsePlugins#Specfile_Template, you
should have %files as
%files
%defattr(-,root,root,-)
%{_datadir}/eclipse/dropins/SystemTapGui

See package built successfully with above changes
http://koji.fedoraproject.org/koji/taskinfo?taskID=1092915

-- 
Configure 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 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict

2009-01-30 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=475037


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||panem...@gmail.com




--- Comment #3 from Parag AN(पराग) panem...@gmail.com  2009-01-30 03:40:56 
EDT ---
how can I use generate-tarball.sh? Provide usage and add same in 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 480538] Review Request: iptux -- a tool for sharing and transporting files and directories in Lan

2009-01-30 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=480538





--- Comment #8 from Liang Suilong liangsuil...@gmail.com  2009-01-30 03:44:02 
EDT ---
- Are you a member of the mailinglists?

I am so sorry that I am not a member

-- 
Configure 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 225852] Merge Review: gok

2009-01-30 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=225852


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

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




--- Comment #13 from Debarshi Ray debarshi@gmail.com  2009-01-30 03:46:51 
EDT ---
I added '%{?_smp_mflags}' and took the liberty to remove the mixed tabs and
spaces warnings from RPMLint.

* Fri Jan 30 2009 Debarshi Ray ri...@fedoraproject.org - 2.25.3-4
- Removed mixed use of tabs and spaces.
- Use parallel make.

+-+
| This package is APPROVED by me. |
+-+

Please close this review as NEXTRELEASE.

-- 
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 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict

2009-01-30 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=475037





--- Comment #4 from Rakesh Pandit rakesh.pan...@gmail.com  2009-01-30 
04:12:53 EDT ---
http://rakesh.fedorapeople.org/spec/stardict-dic-hi.spec
http://rakesh.fedorapeople.org/srpm/stardict-dic-hi-3.0.1-3.fc10.src.rpm

Yes very important to mention. Updated.

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

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


[Bug 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict

2009-01-30 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=475037


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

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




--- Comment #5 from Parag AN(पराग) panem...@gmail.com  2009-01-30 04:53:46 
EDT ---
Review:
+ package builds in mock.
 Koji build = http://koji.fedoraproject.org/koji/taskinfo?taskID=1093215
+ rpmlint is silent for SRPM and for RPM.
- packaged source file will does not match generated source as md5sum always
changes whenever we generate tarball from upstream source.
+ package meets naming and packaging guidelines.
+ specfile is properly named, is cleanly written
+ Spec file is written in American English.
+ Spec file is legible.
+ dist tag is present.
+ build root is correct.
+ license is open source-compatible.
+ License text is included in package.
+ %doc files present.
+ BuildRequires are proper.
+ defattr usage is correct.
+ %clean is present.
+ package installed properly.
+ Macro use appears rather consistent.
+ no static libraries.
+ no .pc file present.
+ no -devel subpackage exists.
+ no .la files.
+ no translations are available.
+ Does owns the directories it creates.
+ no duplicates in %files.
+ file permissions are appropriate.
+ no scriptlets are used.
+ Not a GUI app.

Suggestions:
1) preserve timestamps in cp command using cp -p

APPROVED.

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

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


[Bug 481389] Review Request: beesu - Beesu is a wrapper around su

2009-01-30 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=481389





--- Comment #10 from bee honeybeehoneyhive-red...@yahoo.com  2009-01-30 
04:59:17 EDT ---
Thank you Tom and Manuel!!!
i'm so happy and sure it's okay if spot will be the Fedora maintainer!! That's
very very good!!! you were so kind and honey with me and beesu!! thanks a
lot
i'll still floating around here, so just post a message if you need me to ask
me about anything!

bye

-- 
Configure 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 481071] Review Request: tex-musixtex - Sophisticated music typesetting

2009-01-30 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=481071


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

   What|Removed |Added

 CC||paska...@di.uoa.gr
 AssignedTo|nob...@fedoraproject.org|paska...@di.uoa.gr
   Flag||fedora-review?




--- Comment #1 from Sarantis Paskalis paska...@di.uoa.gr  2009-01-30 05:10:52 
EDT ---
Taking this review, since I own a couple of TeX font packages:

- Could you use the ctan archive for all the sources?

- Could you download the sources from the said archive with wget -N or
similar mechanism to preserve the timestamp on the files? (md5sums match)

- _texmf_main is declared in /etc/rpm/macros.texlive from texlive-texmf, so the
declaration -f _texmf is unnecessary. (btw if it is not a system macro, it
better not start with an _).  I believe the BuildRequires: kpsewhich can go
as well after this.

- Could you please specify a little more explicitly the directories in the
files section?   The spec is correct as is, but the directory ownerships are
not obvious to review, especially in the future. (You can reuse the variables
MUSIXTEXDIR etc).

- I think the package should update the map files (running updmap-sys).  Please
see tetex-font-kerkis in F-10 (or ctan-kerkis-fonts, subpackage tex-kerkis) in
rawhide for an example.

No major problems found, fix the above and I will approve 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 226550] Merge Review: xcdroast

2009-01-30 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=226550


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 CC||rra...@redhat.com




--- Comment #2 from Fabian Affolter fab...@bernewireless.net  2009-01-30 
05:25:09 EDT ---
Roman Rakus is the owner acc. to the PackageDB. Added as cc.

-- 
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 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 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=479903





--- Comment #15 from MERCIER bioinfornat...@gmail.com  2009-01-30 05:24:35 
EDT ---
In first, thanks for you answer.

1/ Why did you change it...?

 %dir %{_datadir}/gdesklets/Controls/%{_ctrlname}
 %dir %{_datadir}/gdesklets/Displays/%{_appname}
 %{_datadir}/gdesklets/Controls/%{_ctrlname}/*
 %{_datadir}/gdesklets/Displays/%{_appname}/*

someone on IRC #fedora-fr say is better.

i have upadte the spec file:
 %{_datadir}/gdesklets/Controls/ImageSlideShow/
 %{_datadir}/gdesklets/Displays/SlideShow/

2/  %setup -q -n %{_appname}
 tar -xf %{SOURCE1} -C ../

it's ok, i use %setup -q -c %{name}-%{version} -a1

3/ 
The LICENSE, MANIFEST, README and todo files are installed in addition to
%doc, and it is important to note that the LICENSE file must not be removed
from /usr/share/gdesklets/Displays/SlideShow/LICENSE - or else it cannot be
displayed by the desklet.

ok, it's done

4/ 
Opening Configure desklet and selecting a folder from which to load images
always displays an error dialog:

| The folder contents could not be displayed
| 
| Operation not supported

It seems it always starts in the root directory due to that.


yes i have the same error, i did'nt find the source code who make this error!

5/ they are no directory image 
i have see in Display/SideShow.script line 259:
Prefs.image_directory.value= value[:value.rfind( / )]
if you change the value by:
Prefs.image_directory.value= /home/user/images
is better.
but where put os.environ ['HOME']?

6/ i have update spec rpm and src rpm
http://bioinformatiques.free.fr/SlideShow-0.9-3.fc10.noarch.rpm

http://bioinformatiques.free.fr/SlideShow-0.9-3.fc10.src.rpm

http://bioinformatiques.free.fr/SlideShow.spec


7/ rpmlint:
$ rpmlint rpmbuild/SRPMS/SlideShow-0.9-3.fc10.src.rpm 
1 packages and 0 specfiles checked; 0 errors, 0 warnings.

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

$ rpmlint rpmbuild/SPECS/SlideShow.spec 
0 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 467409] Review Request: mingw32-atk - MinGW Windows Atk library

2009-01-30 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=467409


Bug 467409 depends on bug 467402, which changed state.

Bug 467402 Summary: Review Request: mingw32-glib2 - MinGW Windows GLib2 library
https://bugzilla.redhat.com/show_bug.cgi?id=467402

   What|Old Value   |New Value

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE



--- Comment #7 from Richard W.M. Jones rjo...@redhat.com  2009-01-30 05:43:54 
EDT ---
No one is really clear on what the .def file is for, except
that it duplicates(?) or enhances(?) linking information.
I'm loathed to remove or change such files when libtool
has created them, since I'm sure the libtool developers
know far more about this than I do.

In any case, the packaging guidelines allow *.def files :-)

Here is an updated package which fixes the other two points
that you raised.

Spec URL:
http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/atk/mingw32-atk.spec
SRPM URL:
http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-atk-1.25.2-2.fc11.src.rpm

* Fri Jan 30 2009 Richard W.M. Jones rjo...@redhat.com - 1.25.2-2
- Remove gtk-doc.
- Fix defattr line.

-- 
Configure 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 467409] Review Request: mingw32-atk - MinGW Windows Atk library

2009-01-30 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=467409


Erik van Pienbroek erik-fed...@vanpienbroek.nl changed:

   What|Removed |Added

 Status|NEW |ASSIGNED




--- Comment #8 from Erik van Pienbroek erik-fed...@vanpienbroek.nl  
2009-01-30 06:01:32 EDT ---
The .def file contains a list of all the functions which need to be exported in
the .DLL file. It is only used by the linker. After a .DLL has been made, the
file has no further use (unless you want to create Visual C++ import libraries,
but there are other ways to generate those from MinGW-compiled DLL's). When
compiling other libraries/applicaties the .def file isn't used, just the .dll
and .dll.a files.

These .def files aren't created by libtool, but by some individual packages
(like glib2, atk, pango and gtk2) as part of the make job. This can be seen at
http://svn.gnome.org/viewvc/atk/trunk/atk/Makefile.am?view=markup (lines 139 to
150 and 168-169)

There was a page on the MinGW site with more details about this, but
unfortunately it got lost:

  The Old MinGWiki pages are again offline as a result of action from SF due to
  flock issues with the phpwiki application. We'll be trying a different
approach.


Another small thing I've found with this package is the lack of a Requires:
pkgconfig

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

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


[Bug 481717] Review Request: ugene - genome analysis suite

2009-01-30 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=481717





--- Comment #5 from Ivan Efremov iefre...@unipro.ru  2009-01-30 06:03:56 EDT 
---
(In reply to comment #3)

 Again 'changelog', if you make any changes on the spec file, you must bump the
 release of the package.  More details at
 https://fedoraproject.org/wiki/Packaging:Guidelines#Changelogs .  

fixed

 If 'make install' is taking care of the .desktop file installation, then
 'validate' is the right way if the categories match the 'Desktop Entry
 Specification'.

done 

 
 Without any further investigations I think that 'Requires: qt' is not
 necessary, for me it looks like that rpm will pick this automatically.

It is necessaty since RPM won't pick versions automatically - ugene needs
particular qt version.

New links:
SPEC: http://ugene.unipro.ru/downloads/ugene.spec
SRPM: http://ugene.unipro.ru/downloads/ugene-1.3.2-2.fc9.src.rpm

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

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


[Bug 481071] Review Request: tex-musixtex - Sophisticated music typesetting

2009-01-30 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=481071





--- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-01-30 
06:18:35 EDT ---
Thanks. I have a few questions:

* SOURCE0 and SOURCE1 are already hosted in ctan. Can you give me a specific
page (if it is at a different place at ctan) where I should download them?

* I'm not very familiar with the map file updating. The package already
contains two map files in
   %{_texmf_main}/fonts/map/dvipdfm/%{texname}/
   %{_texmf_main}/fonts/map/dvips/%{texname}/
Do these map files need updated? If yes, why? (I successfully ran musixtex on
some musixtex documents to produce dvi, ps and pdf files without updating any
map file)
Or is there a system map file that needs to be updated using these map files?

Do I have to call updmap-sys once for each map 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 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict

2009-01-30 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=475037


Rakesh Pandit rakesh.pan...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




-- 
Configure 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 467409] Review Request: mingw32-atk - MinGW Windows Atk library

2009-01-30 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=467409





--- Comment #9 from Richard W.M. Jones rjo...@redhat.com  2009-01-30 06:22:26 
EDT ---
OK, try this one:

Spec URL:
http://hg.et.redhat.com/cgi-bin/hg-misc.cgi/fedora-mingw--devel/raw-file/tip/atk/mingw32-atk.spec
SRPM URL: still building it, but will be here:
http://www.annexia.org/tmp/mingw/fedora-rawhide/src/SRPMS/mingw32-atk-1.25.2-4.fc11.src.rpm

* Fri Jan 30 2009 Richard W.M. Jones rjo...@redhat.com - 1.25.2-4
- Remove gtk-doc.
- Fix defattr line.
- Requires pkgconfig.
- Remove the atk*.def 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 475037] Review Request: stardict-dic-hi - Hindi Dictionary for stardict

2009-01-30 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=475037





--- Comment #6 from Rakesh Pandit rakesh.pan...@gmail.com  2009-01-30 
06:24:27 EDT ---
Updated:
http://rakesh.fedorapeople.org/spec/stardict-dic-hi.spec
http://rakesh.fedorapeople.org/srpm/stardict-dic-hi-3.0.1-4.fc10.src.rpm

Thanks for review.

New Package CVS Request
===
Package Name: stardict-dic-hi
Short Description: Hindi Dictionary for stardict
Owners: rakesh
Branches: F-9 F-10
InitialCC:
Cvsextras Commits: yes

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

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


[Bug 483074] Review Request: perl-Exception-Base - Lightweight exceptions

2009-01-30 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=483074


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #2 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-30 
06:25:47 EDT ---
Package Review
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines including the Perl specific items
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: devel/x86_64
 [x] Rpmlint output:
source RPM: empty
binary RPM:empty
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic (same as perl)
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 SHA1SUM of package: 7325048b213204c8aab7fef0886e2ce517e43600 
Exception-Base-0.21.tar.gz
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: devel/x86_64
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: koji scratch build
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.
 [x] make check passes

==
Notes
==
Please drop the perl = 0:5.006 line. It's been ages since Fedora and RHEL are
shipping much newer versions. Not to mention that during build, perl is brought
in anyway by perl(Module::Build) and for runtime you have the
perl(MODULE_COMPAT) line.


*** APPROVED ***


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

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


[Bug 456527] Review Request: sil-gentium-basic-fonts - Gentium Basic Font Family

2009-01-30 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=456527





--- Comment #24 from Rahul Bhalerao rbhal...@redhat.com  2009-01-30 06:39:58 
EDT ---
Ok. I have changed the naming. The new spec and srpm are at:
http://rbhalera.fedorapeople.org/temp/sil-gentium-basic-fonts/sil-gentium-basic-fonts.spec

http://rbhalera.fedorapeople.org/temp/sil-gentium-basic-fonts/sil-gentium-basic-fonts-1.1-3.fc11.src.rpm

Not sure if it needs reapproval?

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

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


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

2009-01-30 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=479903


Michael Schwendt bugs.mich...@gmx.net changed:

   What|Removed |Added

 CC||luya_...@thefinalzone.net




--- Comment #16 from Michael Schwendt bugs.mich...@gmx.net  2009-01-30 
06:53:48 EDT ---
Maybe the gdesklets maintainer can comment on this?

Bottom of comment 14 is some oddity in Fedora's gdesklets package. It is
reproducible with the Calendar v0.62 Display in F10 gdesklets package.
Steps to reproduce:

1. Run Calendar desklet from date/time category.
2. Right-click on desklet display for Configure desklet pop-up menu.
3. Click Open button next to Month image input field.

Result: An error dialog on top of the file manager is displayed:

| The folder contents could not be displayed
|
| Operation not supported  [ OK ]

It looks like a bug in gdesklets or an API incompatibility.

-- 
Configure 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 481071] Review Request: tex-musixtex - Sophisticated music typesetting

2009-01-30 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=481071





--- Comment #3 from Sarantis Paskalis paska...@di.uoa.gr  2009-01-30 06:58:20 
EDT ---
(In reply to comment #2)
 Thanks. I have a few questions:
 
 * SOURCE0 and SOURCE1 are already hosted in ctan. Can you give me a specific
 page (if it is at a different place at ctan) where I should download them?

Oops, sorry, I misread the URL for the Source0 link.  My bad.

 * I'm not very familiar with the map file updating. The package already
 contains two map files in
%{_texmf_main}/fonts/map/dvipdfm/%{texname}/
%{_texmf_main}/fonts/map/dvips/%{texname}/
 Do these map files need updated? If yes, why? (I successfully ran musixtex on
 some musixtex documents to produce dvi, ps and pdf files without updating any
 map file)
 Or is there a system map file that needs to be updated using these map files?

The latter.  updmap-sys updates updmap.cfg which lists the active map files.  

In my system, musix.map is currently commented out in updmap.cfg, so it will
not be read.  I suspect that in your system the line musix.map is present in
your updmap.cfg either through custom editing or from a manual musixtex
install.  We must also care of removing (or commenting out) that line when
uninstalling.


 Do I have to call updmap-sys once for each map file?

No, we don't actually call updmap-sys to update the map files, but to point out
that a map file called musix.map exists in the respective directories (be they
dvips or dvipdfm is irrelevant).

I suspect that in your system the line musix.map is present in your updmap.cfg
either through custom editing or from a manual musixtex install.  We must also
care of removing (or commenting out) that line when uninstalling.

-- 
Configure 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 482827] Review Request: banshee-mirage - An Automatic Playlist Generation Extension for Banshee

2009-01-30 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=482827


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

   What|Removed |Added

 CC||ita...@ispbrasil.com.br




--- Comment #12 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-30 
07:05:48 EDT ---
 Branches: F9, F10, devel

the devel branch are always created, so you don't need to specify 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 483079] Review Request: perl-constant-boolean - Define TRUE and FALSE constants

2009-01-30 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=483079


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #2 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-30 
07:13:16 EDT ---
Package Review
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines including the Perl specific items
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: devel/x86_64
 [x] Rpmlint output:
source RPM: empty
binary RPM:empty
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic (same as perl)
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 SHA1SUM of package: 1b5c8fc4b76bd9bf61063a92fdfbf6394cac9641
constant-boolean-0.01.tar.gz
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.
 [-] Fully versioned dependency in subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: devel/x86_64
 [x] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: koji scratch build
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.
 [x] make check passes


*** APPROVED ***


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

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


[Bug 483074] Review Request: perl-Exception-Base - Lightweight exceptions

2009-01-30 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=483074


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Allisson Azevedo allis...@gmail.com  2009-01-30 07:20:08 
EDT ---
New Package CVS Request
===
Package Name: perl-Exception-Base
Short Description: Lightweight exceptions
Owners: allisson
Branches: F-9 F-10

-- 
Configure 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 483079] Review Request: perl-constant-boolean - Define TRUE and FALSE constants

2009-01-30 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=483079


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #3 from Allisson Azevedo allis...@gmail.com  2009-01-30 07:24:10 
EDT ---
New Package CVS Request
===
Package Name: perl-constant-boolean
Short Description: Define TRUE and FALSE constants
Owners: allisson
Branches: F-9 F-10

-- 
Configure 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 482863] Review Request: perl-MooseX-Daemonize - Role for daemonizing your Moose based application

2009-01-30 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=482863


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #6 from Allisson Azevedo allis...@gmail.com  2009-01-30 07:23:05 
EDT ---
New Package CVS Request
===
Package Name: perl-MooseX-Daemonize
Short Description: Role for daemonizing your Moose based application
Owners: allisson
Branches: F-9 F-10

-- 
Configure 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 482827] Review Request: banshee-mirage - An Automatic Playlist Generation Extension for Banshee

2009-01-30 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=482827


David Nielsen gnomeu...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||CURRENTRELEASE




--- Comment #13 from David Nielsen gnomeu...@gmail.com  2009-01-30 07:30:05 
EDT ---
http://koji.fedoraproject.org/koji/buildinfo?buildID=80907

-- 
Configure 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 453422] Review Request: songbird - Mozilla based multimedia player

2009-01-30 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=453422


Nicolas Chauvet (kwizart) kwiz...@gmail.com changed:

   What|Removed |Added

 CC||kwiz...@gmail.com




--- Comment #20 from Nicolas Chauvet (kwizart) kwiz...@gmail.com  2009-01-30 
07:54:15 EDT ---
Hi!

What is the status of this package ?
Is this the last package version !

Just one note for now:
* Why is it using zlib-static ?
zlib doesn't change that much nor produce a circle dependency. So the needs of
static linking doens't make sense IMO.
* The source rpm is definitely enormous it would seems easier to repack the
%{tarname}%{version}-860-vendor.tar.bz2 with cherry picking only what is needed
to be built internally for the given Fedora version(xulrunner/taglib/other?).
This can be made using a script that will have to be provided within the source
package. (as source100: for example).
*BTW you need to use Source0: instead of Source:


But having a script to repack the source dependencies isn't that a real
solution.
The best way would be to have a script provided from the Songbird upstream(hi
Stephen) To download a given (vanilla) version of a Songbird dependency, with
only the related patch needed by Songbird (if there is). And That for each
Songbird dependency. (following the VideoLan example, with vlc)

This will allow for the packager (and the other packagers, that will check the
Songbird Packager commits during the whole live for the package), to control
which will be the difference between the system package and eventually, to
override the use of an internal dependency if the Songbird patch is for another
platform or already merged in the new system version...


For now, I think we will need to use xulrunner internally, but what the current
status of the (system) xulrunner 3.1 usability with SongBird ? (xulrunner 3.1
is already present in Rawhide/F-11).

We could also ask the feodra taglib package maintainer if he will allow to 
backport the needed patch for F-11 (unless it breaks the taglib ABI).



/me is still rpmbuilding Songbird for F-10 x86_64

-- 
Configure 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 483026] Review Request: hatools - Improved shell scripting in High Availability environment

2009-01-30 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=483026





--- Comment #3 from Oliver Falk oli...@linux-kernel.at  2009-01-30 08:08:58 
EDT ---
Updated the spec, including TODO and BAD :-)
I did *not* increase the release number.

Thx Jochen for reviewing!

-- 
Configure 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 483025] Review Request: imms - Adaptive playlist framework tracking and adapting to your listening patterns

2009-01-30 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=483025





--- Comment #1 from Milos Jakubicek xja...@fi.muni.cz  2009-01-30 08:26:23 
EDT ---
Torch is already built in rawhide, hence I made a scratch build here:

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

I also added all the missing BR, had to add also automake + autoconf because of
the funny configure script provided in the source tarball:

cat configure
#!/bin/sh
rm $0
make
$0 $*

New SPEC: http://mjakubicek.fedorapeople.org/imms/imms.spec
New SRPM:
http://mjakubicek.fedorapeople.org/imms/imms-3.1.0-0.2.rc6.fc10.src.rpm

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

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


[Bug 473046] Review Request: miniupnpc - command line tool to control NAT in UPnP-enabled routers as Linksys, D-Link etc

2009-01-30 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=473046





--- Comment #22 from Fabian Affolter fab...@bernewireless.net  2009-01-30 
08:51:44 EDT ---
More details:

https://fedoraproject.org/wiki/Packaging:Guidelines#Trademarks_in_Summary_or_Description

-- 
Configure 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 476832] Review Request: mydns - serve DNS records directly from an SQL database

2009-01-30 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=476832





--- Comment #12 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-30 
08:53:48 EDT ---
fixed.


http://ispbrasil.com.br/mydns/mydns.spec
http://ispbrasil.com.br/mydns/mydns-1.2.8.22-1.fc10.src.rpm

I need only to replace this 2 command's by install instead using cp.

mkdir -p %{buildroot}%{_datadir}/locale
cp -a ./$database%{_datadir}/locale %{buildroot}%{_datadir}

any suggestion about 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 225707] Merge Review: dosfstools

2009-01-30 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=225707





--- Comment #9 from Stepan Kasal ska...@redhat.com  2009-01-30 09:25:15 EDT 
---
fixed in dosfstools-3.0.1-3

-- 
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 478300] Review Request: python-wifi - Python binding for the wireless extensions

2009-01-30 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=478300


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||debarshi@gmail.com
 AssignedTo|nob...@fedoraproject.org|debarshi@gmail.com
   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 453422] Review Request: songbird - Mozilla based multimedia player

2009-01-30 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=453422





--- Comment #21 from Stephen Lau ste...@songbirdnest.com  2009-01-30 09:31:57 
EDT ---
(In reply to comment #20)
 Hi!
 
 What is the status of this package ?
 Is this the last package version !

David (auralvance) has spun 1.0 Fedora RPMs here:
http://wiki.songbirdnest.com/Developer/Articles/Builds/Contributed_Builds

 Just one note for now:
 * Why is it using zlib-static ?
 zlib doesn't change that much nor produce a circle dependency. So the needs of
 static linking doens't make sense IMO.

It can use the system zlib.

 For now, I think we will need to use xulrunner internally, but what the 
 current
 status of the (system) xulrunner 3.1 usability with SongBird ? (xulrunner 3.1
 is already present in Rawhide/F-11).

Still not possible... unless the XULRunner has the patches we depend on,
Songbird will not likely work. :(

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

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


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

2009-01-30 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=483108





--- Comment #2 from Johan Vromans jvrom...@squirrel.nl  2009-01-30 09:38:13 
EDT ---
Thanks for the kind and constructive comments.
I've added a patch for the Makefile to eliminate the double install of the man
pages, and applied your other suggestions as well.
I'll submit a new review request.

As for the other questions: I have some more packages I'd like to submit but
since this is the first time I first try one package to get used to the
procedures.

-- 
Configure 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] New: Review Request: chordii - Print songbooks (lyrics + chords)

2009-01-30 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: chordii - Print songbooks (lyrics + chords)

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

   Summary: Review Request: chordii - Print songbooks (lyrics +
chords)
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jvrom...@squirrel.nl
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.squirrel.nl/pub/xfer/chordii.spec
SRPM URL: http://www.squirrel.nl/pub/xfer/chordii-4.2-2.src.rpm
Description: 
chordii provides guitar players and other musicians with a tool to
produce good looking, self-descriptive music sheets from text files.

chordii reads text files in chordpro format, containing the lyrics of
songs, the chords to be played, their descriptions and some other
optional data. It produces a PostScript document suitable for viewing
and printing.

chordii features include:
 - centered titles,
 - chord names above the words,
 - chord diagrams at the end of the songs,
 - multiple columns on a page,
 - multiple logical pages per physical pages (1, 2 or 4),
 - configurable fonts and sizes,
 - the complete ISO 8859-1 character set,
 - chorus marking,
 - automated chord transposition,
 - songbook creation: typeset multiple songs with page numbers and a
   table of contents.

This is my first package, so I'm looking for a sponsor.
The spec and srpm are rpmlint free of warnings and errors.

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

___
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-01-30 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


Johan Vromans jvrom...@squirrel.nl changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 459751] Review Request: osgGtk - Gtk and Gtkmm widgets for OpenSceneGraph

2009-01-30 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=459751


Debarshi Ray debarshi@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|debarshi@gmail.com
   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 451298] Review Request: purple-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-30 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=451298





--- Comment #34 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
09:55:22 EDT ---
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 451298] Review Request: purple-msn-pecan - Alternative MSN protocol plugin for libpurple

2009-01-30 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=451298


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 481173] Review Request: libnss-pgsql - NSS library that interface with PostgreSQL

2009-01-30 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=481173


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

   What|Removed |Added

 Status|ON_QA   |CLOSED
 Resolution||NEXTRELEASE




--- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
09:56:24 EDT ---
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 483108] Review Request: chordii - Print songbooks (lyrics + chords)

2009-01-30 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=483108





--- Comment #3 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-30 
09:56:56 EDT ---
As you please, I am waiting.

2-3-4 correct packages (depending on complexity and your abilities to fix -- if
needed -- errors) will[/might] bring sponsorship :)

Please do no forget to increment the release tag each time you submit a
modified spec.

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

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


[Bug 463767] Review Request: cloog - The Chunky Loop Generator

2009-01-30 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=463767





--- Comment #16 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
10:05:38 EDT ---
Again ping?

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

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


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

2009-01-30 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


Fabian Affolter fab...@bernewireless.net changed:

   What|Removed |Added

 CC||fab...@bernewireless.net




--- Comment #1 from Fabian Affolter fab...@bernewireless.net  2009-01-30 
10:04:04 EDT ---
After a quick look at your spec file, some comments.  

- Use '%defattr(-,root,root,-)' instead of '%defattr(-, root, root, 0755)'
- The 'URL:' should point to the upstream website.  The 'Source0' to the
upstream tarball.  Move 'URL:' to 'Source'
  https://fedoraproject.org/wiki/Packaging/SourceURL
- It would be nice if you are going to use the dist tag - 'Release: 
2%{?dist}'
- Change 'examples' to 'examples/' to add all example files.

To get sponsored -
https://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored

-- 
Configure 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 475471] Review Request: poi - Java API to Access Microsoft Format Files

2009-01-30 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=475471





--- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
10:09:31 EDT ---
ping 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 479832] Review Request: mmpong - a massively multiplayer pong game

2009-01-30 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=479832


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

   What|Removed |Added

   Flag||needinfo?(maintai...@mt2009
   ||.com)




--- Comment #8 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
10:10:00 EDT ---
ping?

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

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


[Bug 453422] Review Request: songbird - Mozilla based multimedia player

2009-01-30 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=453422





--- Comment #22 from David Halik auralva...@gmail.com  2009-01-30 10:11:16 
EDT ---
My mistake. I forgot to log in and update the ticket after the 1.0 release. :)
Yes, the 1.0 rpm's have been out for sometime. Here's the direct link to the
src.rpm and spec, all of which you can find in the url stevel pointed out.

As of the latest release we use system zlib.

http://rpm.rutgers.edu/fedora/songbird-1.0.0-1.fc10.src.rpm
http://rpm.rutgers.edu/fedora/songbird.spec

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

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


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

2009-01-30 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=483108


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||panem...@gmail.com




--- Comment #4 from Parag AN(पराग) panem...@gmail.com  2009-01-30 10:23:49 
EDT ---
Make sure which bug to mark duplicate I see this package submitted by submitter
twice. see bug483250

-- 
Configure 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-01-30 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


Parag AN(पराग) panem...@gmail.com changed:

   What|Removed |Added

 CC||panem...@gmail.com




--- Comment #2 from Parag AN(पराग) panem...@gmail.com  2009-01-30 10:24:53 
EDT ---
Make sure which bug to mark duplicate I see this package submitted by submitter
twice. see bug483108

-- 
Configure 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 473205] Review Request: gPlanarity - puzzle game

2009-01-30 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=473205





--- Comment #12 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
10:29:29 EDT ---
I will close this this bug as NOTABUG if no response from
the reporter is received within ONE WEEK.

-- 
Configure 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 483258] New: Review Request: perl-MooseX-LogDispatch - Logging Role for Moose

2009-01-30 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: perl-MooseX-LogDispatch - Logging Role for Moose

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

   Summary: Review Request: perl-MooseX-LogDispatch - Logging Role
for Moose
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: allis...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL:
http://allisson.fedorapeople.org/packages/perl-MooseX-LogDispatch/perl-MooseX-LogDispatch.spec

SRPM URL:
http://allisson.fedorapeople.org/packages/perl-MooseX-LogDispatch/perl-MooseX-LogDispatch-1.2000-1.fc10.src.rpm

Description: Log::Dispatch role for use with your Moose classes.

-- 
Configure 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-01-30 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 #3 from Johan Vromans jvrom...@squirrel.nl  2009-01-30 10:32:49 
EDT ---
*** Bug 483108 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 483108] Review Request: chordii - Print songbooks (lyrics + chords)

2009-01-30 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=483108


Johan Vromans jvrom...@squirrel.nl changed:

   What|Removed |Added

 Status|NEW |CLOSED
 Resolution||DUPLICATE




--- Comment #5 from Johan Vromans jvrom...@squirrel.nl  2009-01-30 10:32:49 
EDT ---


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

-- 
Configure 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 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose

2009-01-30 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=483258


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
   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 225852] Merge Review: gok

2009-01-30 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=225852


Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||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 481389] Review Request: beesu - Beesu is a wrapper around su

2009-01-30 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=481389


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

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #11 from Tom spot Callaway tcall...@redhat.com  2009-01-30 
10:46:15 EDT ---
New Package CVS Request
===
Package Name: beesu
Short Description: Beesu is a wrapper around su
Owners: spot
Branches: F-9 F-10 devel EL-4 EL-5
InitialCC: 

...and it's 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 443771] Review Request: lmms - Linux MultiMedia Studio

2009-01-30 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=443771





--- Comment #27 from Thomas Moschny thomas.mosc...@gmx.de  2009-01-30 
10:45:26 EDT ---
(In reply to comment #26)

 I've asked upstream for clarification. If I don't get an answer, I'll
 repack the tarfile.

See
http://sourceforge.net/mailarchive/forum.php?thread_name=49790675.5020701%40gmx.deforum_name=lmms-devel

-- 
Configure 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 483115] Review Request: allgeyer-fonts - Musical Notation True Type Fonts

2009-01-30 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=483115


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

   What|Removed |Added

   Flag||fedora-cvs+




--- Comment #4 from Tom spot Callaway tcall...@redhat.com  2009-01-30 
10:46:23 EDT ---
New Package CVS Request
===
Package Name: allgeyer-fonts
Short Description: Musical Notation True Type Fonts
Owners: spot
Branches: F-9 F-10 devel
InitialCC: 

... and it's 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 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose

2009-01-30 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=483258


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

   What|Removed |Added

   Flag|fedora-review?  |fedora-review+




--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-30 
10:51:23 EDT ---
Package Review
==

Key:
 - = N/A
 x = Check
 ! = Problem
 ? = Not evaluated

=== REQUIRED ITEMS ===
 [x] Package is named according to the Package Naming Guidelines.
 [x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
 [x] Package meets the Packaging Guidelines including the Perl specific items
 [x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
 Tested on: devel/x86_64
 [x] Rpmlint output:
source RPM: empty
binary RPM:empty
 [x] Package is not relocatable.
 [x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
 [x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
 [x] License field in the package spec file matches the actual license.
 License type: GPL+ or Artistic (same as perl)
 [x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
 [x] Spec file is legible and written in American English.
 [x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
 SHA1SUM of package: 1b8b177dc740b96c70d7bbe40aeea1dd1f5e6f73
MooseX-LogDispatch-1.2000.tar.gz
 [x] Package is not known to require ExcludeArch
 [x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
 [-] The spec file handles locales properly.
 [-] ldconfig called in %post and %postun if required.
 [x] Package must own all directories that it creates.
 [x] Package requires other packages for directories it uses.
 [x] Package does not contain duplicates in %files.
 [x] Permissions on files are set properly.
 [x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
 [x] Package consistently uses macros.
 [x] Package contains code, or permissable content.
 [-] Large documentation files are in a -doc subpackage, if required.
 [x] Package uses nothing in %doc for runtime.
 [-] Header files in -devel subpackage, if present.
 [-] Static libraries in -devel subpackage, if present.
 [-] Package requires pkgconfig, if .pc files are present.
 [-] Development .so files in -devel subpackage, if present.   
  [-] Fully versioned dependency in
subpackages, if present.
 [x] Package does not contain any libtool archives (.la).
 [-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
 [x] Package does not own files or directories owned by other packages.

=== SUGGESTED ITEMS ===
 [x] Latest version is packaged.
 [x] Package does not include license text files separate from upstream.
 [-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
 [x] Reviewer should test that the package builds in mock.
 Tested on: devel/x86_64
 [-] Package should compile and build into binary rpms on all supported
architectures.
 Tested on: standard perl module, noarch
 [?] Package functions as described.
 [-] Scriptlets must be sane, if used.
 [-] The placement of pkgconfig(.pc) files is correct.
 [-] File based requires are sane.
 [x] make check passes


*** APPROVED ***


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

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


[Bug 483258] Review Request: perl-MooseX-LogDispatch - Logging Role for Moose

2009-01-30 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=483258


Allisson Azevedo allis...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #2 from Allisson Azevedo allis...@gmail.com  2009-01-30 10:56:32 
EDT ---
New Package CVS Request
===
Package Name: perl-MooseX-LogDispatch
Short Description: Logging Role for Moose
Owners: allisson
Branches: F-9 F-10

-- 
Configure 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-01-30 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 #4 from Johan Vromans jvrom...@squirrel.nl  2009-01-30 11:03:05 
EDT ---
@Fabian: Thanks for your kind and constructive feedback.
I've made the suggested changes to the spec.

I documented in %changelog that I changed %defattr, and now rpmlint complains
about 'macro-in-%changelog'. I assume this can be safely ignored.

New files have been uploaded:
Spec URL: http://www.squirrel.nl/pub/xfer/chordii.spec
SRPM URL: http://www.squirrel.nl/pub/xfer/chordii-4.2-3.fc11.src.rpm

Last build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1094024

Thanks for your patience.

-- 
Configure 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-01-30 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 #5 from manuel wolfshant wo...@nobugconsulting.ro  2009-01-30 
11:10:26 EDT ---
Johan, use %% in the changelog in order to silence rpmlint. Otherwise the entry
is expanded as a macro.

-- 
Configure 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 483205] Review Request: eclipse-SystemTapGui - GUI interface for SystemTap

2009-01-30 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=483205


Andrew Overholt overh...@redhat.com changed:

   What|Removed |Added

 CC||overh...@redhat.com




--- Comment #2 from Andrew Overholt overh...@redhat.com  2009-01-30 11:12:53 
EDT ---
Is the camel casing necessary in the name?  Just curious ...

-- 
Configure 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 483205] Review Request: eclipse-SystemTapGui - GUI interface for SystemTap

2009-01-30 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=483205





--- Comment #3 from Andrew Overholt overh...@redhat.com  2009-01-30 11:23:55 
EDT ---
Also, there's something weird happening with the feature not being included in
the zips resulting from the build.  I suspect something odd in the feature's
build.properties or feature.xml.  The os=linux thing may be it ... try a
build without it and see if the feature is included in the zip.

-- 
Configure 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 442233] Review Request: oprofileui - user interface for analysing oprofile data

2009-01-30 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=442233


Dave Jones da...@redhat.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #14 from Dave Jones da...@redhat.com  2009-01-30 11:56:10 EDT ---
New Package CVS Request
===
Package Name: oprofileui
Short Description: GTK2 user interface for oprofile
Owners: da...@redhat.com
Branches: devek
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 483277] New: Review Request: nautilus-gdu - Nautilus extension for disk formatting

2009-01-30 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: nautilus-gdu - Nautilus extension for disk formatting

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

   Summary: Review Request: nautilus-gdu - Nautilus extension for
disk formatting
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: tbza...@redhat.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: http://tbzatek.fedorapeople.org/nautilus-gdu/nautilus-gdu.spec
SRPM URL:
http://tbzatek.fedorapeople.org/nautilus-gdu/nautilus-gdu-0.1-git20090130.1.fc11.src.rpm

Description:

Simple extension for formatting disks, usb sticks and other media,
available from Nautilus context menu. This extension is built on top.
of the gnome-disk-utility library with DeviceKit-disks as a backend.
Access to physical devices is controlled by PolicyKit.

- rpmlint reports no error, package builds successfully in koji

-- 
Configure 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 483277] Review Request: nautilus-gdu - Nautilus extension for disk formatting

2009-01-30 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=483277


Matthias Clasen mcla...@redhat.com changed:

   What|Removed |Added

 CC||mcla...@redhat.com
 AssignedTo|nob...@fedoraproject.org|mcla...@redhat.com




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

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


[Bug 476832] Review Request: mydns - serve DNS records directly from an SQL database

2009-01-30 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=476832


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

   What|Removed |Added

 Status|NEW |ASSIGNED
 AssignedTo|nob...@fedoraproject.org|mtas...@ioa.s.u-tokyo.ac.jp
   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 476832] Review Request: mydns - serve DNS records directly from an SQL database

2009-01-30 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=476832





--- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp  2009-01-30 
12:29:18 EDT ---
Well,

 I need only to replace this 2 command's by install instead using cp.
 
 mkdir -p %{buildroot}%{_datadir}/locale
 cp -a ./$database%{_datadir}/locale %{buildroot}%{_datadir}

If cp can be used here, you can just use cp.
I frequenty use both install and cp in one spec file.

By the way for .22-1:

* Working directory
--
#install doc about alternatives
install -Dp -m 644 %{SOURCE1} %{_builddir}/%{name}-%{base_version}
--
  - Well, even in %prep after %setup -q is finished, the working directory
is %{_builddir}/%{name}-%{base_version} for this case.

* Autotool recall
  - build.log says:
--
   649  + make -j4
   650  cd .  /bin/sh /builddir/build/BUILD/mydns-1.2.8/missing --run
autoheader
   651  /builddir/build/BUILD/mydns-1.2.8/missing: line 46: autoheader: command
not found
   652  WARNING: `autoheader' is missing on your system.  You should only need
it if
   653   you modified `acconfig.h' or `configure.ac'.  You might want
   654   to install the `Autoconf' and `GNU m4' packages.  Grab them
   655   from any GNU archive site.
   656  rm -f stamp-h1
   657  touch config.h.in
   658  cd .  /bin/sh ./config.status config.h
   659  config.status: creating config.h
   660  config.status: config.h is unchanged
   661  make  all-recursive
--
Still autotools are called. Please suppress this.

* cflags
  I didn't notice this before, however:
--
   613  checking CFLAGS... -O2 -g -pipe -Wall -Wp,-D_FORTIFY_SOURCE=2
-fexceptions -fstack-protector --param=ssp-buffer-size=4 -m32 -march=i386
-mtune=generic -fasynchronous-unwind-tables -fomit-frame-pointer
-finline-functions -ffast-math -funsigned-char
--
  - -fomit-frame-pointer is not allowed on Fedora because this makes
debugging very difficult
  - Also -ffast-math should be avoid unless you have some reasons
because this changes (decreases) the precision of math functions and 
introduces some assumption.

* Scriptlets
  - As I said in my previous comment, please add proper Requires(post) or
so.
(In reply to comment #11)
 * scriptlets
   - For SysV initscripts handling, please refer to
 
 https://fedoraproject.org/wiki/Packaging/SysVInitScript#InitscriptScriptlets
 (Note that there are some Requires(post) or so)
 
   - Add Requires(post): %{_sbindir}/alternatives (also Requires(preun))
 for -mysql and -pgsql subpackages 

By the way:
---
if [ $1 = 1 ]; then
   /sbin/chkconfig --add %{name}
fi
---
- if [ $1 = 1 ] is not needed. If service name is already registered
  in initscript symlinks, then chkconfig --add does nothing.
  Also, removing [ $1 = 1 ] can handle the case in which
  service name changes correctly.

* Info file
  - I checked the info file and actually mydns.info.gz is empty
(try gunzip).
I guess some BuildRequires are missing (perhaps texinfo)

* Directory ownership issue
  - %_datadir/%name is not owned by any packages.

* Gettext .mo files
  - Making gettext .mo files owned by two subpackages is not needed.
Please these files to pydns package.

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

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


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

2009-01-30 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 #6 from Johan Vromans jvrom...@squirrel.nl  2009-01-30 12:33:55 
EDT ---
@manuel: thanks! Fixed in a future version.

-- 
Configure 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 483286] Review Request: perl-Data-Report - A flexible plugin-driven reporting framework

2009-01-30 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=483286


Johan Vromans jvrom...@squirrel.nl changed:

   What|Removed |Added

 Blocks||177841




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

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


[Bug 483286] New: Review Request: perl-Data-Report - A flexible plugin-driven reporting framework

2009-01-30 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: perl-Data-Report - A flexible plugin-driven reporting 
framework

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

   Summary: Review Request: perl-Data-Report - A flexible
plugin-driven reporting framework
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: jvrom...@squirrel.nl
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report.spec
SRPM URL: http://www.squirrel.nl/pub/xfer/perl-Data-Report-0.10-1.fc11.src.rpm
Description:
Data::Report is a framework for report generation.

You define the columns, add the data row by row, and get reports in
text, HTML, CSV and so on. Textual ornaments like extra empty lines,
dashed lines, and cell lines can be added in a way similar to HTML
style sheets.

Last koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1094325

This is one of my first packages, so I'm looking for a sponsor.
The spec and srpm are rpmlint free of warnings and errors.

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

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


[Bug 483277] Review Request: nautilus-gdu - Nautilus extension for disk formatting

2009-01-30 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=483277





--- Comment #1 from Matthias Clasen mcla...@redhat.com  2009-01-30 13:16:34 
EDT ---
Package builds fine in mock.

rpmlint output:

3 packages and 0 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 476832] Review Request: mydns - serve DNS records directly from an SQL database

2009-01-30 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=476832





--- Comment #14 from Itamar Reis Peixoto ita...@ispbrasil.com.br  2009-01-30 
13:21:21 EDT ---
I will add a new .spec tomorrow.

I am planning to add a user and a group called mydns and make it own the file
/etc/mydns.conf, so mydns.conf will not readable by everyone, since this file
stores the user and password of mydns database connection.

-- 
Configure 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 483277] Review Request: nautilus-gdu - Nautilus extension for disk formatting

2009-01-30 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=483277





--- Comment #2 from Matthias Clasen mcla...@redhat.com  2009-01-30 13:25:38 
EDT ---
requires/provides/file list look sane

-- 
Configure 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 481071] Review Request: tex-musixtex - Sophisticated music typesetting

2009-01-30 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=481071





--- Comment #4 from Orcan 'oget' Ogetbil oget.fed...@gmail.com  2009-01-30 
13:41:38 EDT ---
Files updated:
Spec URL: http://oget.fedorapeople.org/review/tex-musixtex.spec
SRPM URL: http://oget.fedorapeople.org/review/tex-musixtex-0.114-2.src.rpm

%changelog
- Use standard macros from /etc/rpm/macros.texlive
- Drop BR: kpsewhich
- %%files section is made more explicit
- Update updmap.cfg via updmap-sys in postpostun


 (You can reuse the variables MUSIXTEXDIR etc).
I couldn't use them because rpmbuild fails when a %files entry doesn't start
with /.

-- 
Configure 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 459948] Review Request: libhildon - Hildon Application Framework - shared libraries

2009-01-30 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=459948





--- Comment #2 from Michel Alexandre Salim michel.syl...@gmail.com  
2009-01-30 13:46:32 EDT ---
Will update soon, thanks for the reports.

-- 
Configure 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 476471] Review Request: fedora-security-guide - A security guide for Linux

2009-01-30 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=476471


Bug 476471 depends on bug 482972, which changed state.

Bug 482972 Summary: .desktop file not created
https://bugzilla.redhat.com/show_bug.cgi?id=482972

   What|Old Value   |New Value

 Status|CLOSED  |ASSIGNED
 Resolution|WONTFIX |



-- 
Configure 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 483277] Review Request: nautilus-gdu - Nautilus extension for disk formatting

2009-01-30 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=483277





--- Comment #3 from Matthias Clasen mcla...@redhat.com  2009-01-30 13:50:57 
EDT ---
package name: ok
spec file name: ok
packaging guidelines: ok
license: ok
license field: ok
license file: ok
spec file language: ok
spec file legible: ok
sources: you should probably add a comment explaining how to get the tarball
from git. longer-term, we need to sort out hosting
buildable: ok
excludearch: ok
build deps: ok
locale handling: ok, no translations right now. sorting that out ties in to the
hosting question...
ldconfig: I don't think you need to call ldconfig in %post/%postun, since you
are not installing a shared library in the linker path, just a loadable module
relocatable: n/a
directory ownership: ok
duplicate files: ok
permissions: ok
%clean: ok
macro use: ok
permissible content: ok
large docs: n/a
%doc content: ok
headers: n/a
static libs: n/a
pc files: ok
shared libs: ok
devel subpackage: n/a
libtool archives: ok
gui apps: ok
file ownership: ok
%install: ok
utf8 filenames: ok

Summary: 
- add source comment
- remove unnecessary %post/%postun

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

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


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

2009-01-30 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=479903





--- Comment #17 from MERCIER bioinfornat...@gmail.com  2009-01-30 13:49:24 
EDT ---
i have found this file:
/usr/lib64/gdesklets/Controls/URI/__init__py
this file is call for openning a 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 227933] Review Request: libproj4 - Cartographic projection library

2009-01-30 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=227933


Orion Poplawski or...@cora.nwra.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NOTABUG
   Flag|fedora-review?, |
   |needinfo?(or...@cora.nwra.c |
   |om) |




--- Comment #8 from Orion Poplawski or...@cora.nwra.com  2009-01-30 13:54:48 
EDT ---
I believe because back in the day, that is what one did when one submitted a
review.

I don't have a burning need for this to be in Fedora.  But gdl uses this
library rather than the main PROJ.4 distribution.  Actually, gdl uses a very
old version (now) of libproj4, so we'll see where that moves.  I'll close for
now.

-- 
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 474980] Review Request: ovm - Open Verification Methodology : IEEE 1800 SystemVerilog standard

2009-01-30 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=474980


Frank Ch. Eigler f...@redhat.com changed:

   What|Removed |Added

 CC||f...@redhat.com




--- Comment #21 from Frank Ch. Eigler f...@redhat.com  2009-01-30 14:10:34 
EDT ---
Re. comment #19,
 As sysadmin I would definitely have preferred to do
 yum install ovm.el4 instead of tar xzf ovm-2.0.tar.gz

Until that time, you could make a local yum repo and publish the RPM there.

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

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


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

2009-01-30 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=478007


Jon Stanley jonstan...@gmail.com changed:

   What|Removed |Added

 Status|NEW |CLOSED
 CC||jonstan...@gmail.com
 Resolution||WONTFIX




--- Comment #17 from Jon Stanley jonstan...@gmail.com  2009-01-30 14:41:25 
EDT ---
FESCo has voted not to allow crutches for 3rd party packages, such as this one,
into Fedora.

This package is therefore blocked from Fedora.

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

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


[Bug 483301] New: Review Request: muse - Midi/Audio Music Sequencer

2009-01-30 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: muse - Midi/Audio Music Sequencer

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

   Summary: Review Request: muse - Midi/Audio Music Sequencer
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: oget.fed...@gmail.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://oget.fedorapeople.org/review/muse.spec
SRPM URL: http://oget.fedorapeople.org/review/muse-0.9-3.fc10.src.rpm
Description: 
MusE is a MIDI/Audio sequencer with recording and editing capabilities. It can
perform audio effects like chorus/flanger in realtime via LASH and it supports
Jack and ALSA interfaces. MusE aims to be a complete multitrack virtual studio
for Linux.

rpmlint is silent.

Epoch: 1 is there for PlanetCCRMA compatibility. See AudioCreation SIG wiki for
informations about PlanetCCRMA integration:
https://fedoraproject.org/wiki/AudioCreation

-- 
Configure 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 474980] Review Request: ovm - Open Verification Methodology : IEEE 1800 SystemVerilog standard

2009-01-30 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=474980





--- Comment #22 from Jon Stanley jonstan...@gmail.com  2009-01-30 14:56:10 
EDT ---
FESCo has voted to continue to block this content, on the basis that it does
not enhance the OS user experience.  Imagine this:

I'm Joe User, who wants to use ovm.  I type 'yum install ovm iverilog' and
expect to be able to do whatever ovm does (and I'll have to admit ignorance as
to exactly what this is - I'm not a hardware guy). I find that I can't do that,
and instead have to buy some expensive proprietary tool in order to make use of
this free content.

We're fine with the concept of ovm being in Fedora, but we also want something
that is able to use this content, even if it's at a very primitive level that
no one actually *would* use it in a production environment. The link that
Chitlesh posted earlier to upstream iverilog forums seemed to indicate that
they had other priorities other than making this work.

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

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


[Bug 477818] Review Request: gsql - Integrated database development tool for GNOME

2009-01-30 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=477818


Pavel Shevchuk stl...@gmail.com changed:

   What|Removed |Added

   Flag||fedora-cvs?




--- Comment #5 from Pavel Shevchuk stl...@gmail.com  2009-01-30 15:18:42 EDT 
---
New Package CVS Request
===
Package Name: gsql
Short Description: Integrated database development tool for GNOME
Owners: stalwart
Branches: F-9 F-10
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 483320] New: Review Request: CodeAnalyst - Performance Analysis Suite for AMD-based System (based on Oprofile)

2009-01-30 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: CodeAnalyst - Performance Analysis Suite for AMD-based 
System (based on Oprofile)

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

   Summary: Review Request: CodeAnalyst - Performance Analysis
Suite for AMD-based System (based on Oprofile)
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: low
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: suravee.suthikulpa...@amd.com
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora
Target Release: ---


Spec URL: https://nda.amd.com/dds/user/xdoc/41629
SRPM URL: https://nda.amd.com/dds/user/xdoc/41628
Description:
CodeAnalyst is a performance analysis suite. It provids graphical 
utilitys for running Oprofile, and analyzing result on AMD-based systems. 
CodeAnalyst includes several features to aid profile analysis such as 
inline-function analysis, code-block analysis, and a utility for 
profile comparison, DiffAnalayst.

-- 
Configure 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 483331] New: Review Request: asterisk-sounds-core - Core sounds for Asterisk

2009-01-30 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: asterisk-sounds-core - Core sounds for Asterisk

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

   Summary: Review Request: asterisk-sounds-core - Core sounds for
Asterisk
   Product: Fedora
   Version: rawhide
  Platform: All
OS/Version: Linux
Status: NEW
  Severity: medium
  Priority: medium
 Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: j...@ocjtech.us
 QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
   Estimated Hours: 0.0
Classification: Fedora


Spec URL: http://fedorapeople.org/~jcollie/asterisk-sounds-core.spec
SRPM URL:
http://fedorapeople.org/~jcollie/asterisk-sounds-core-1.4.14-1.fc10.src.rpm
Description: Core sound files for Asterisk.

After a long, long, wait the Asterisk sound files finally have a proper license
(CC-BY-SA) so we can finally package them!

Once the package has been reviewed and in development I'm going to be talking
to rel-eng about copying the binary RPMs to each supported version of Fedora as
these are rather large.

-- 
Configure 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 483331] Review Request: asterisk-sounds-core - Core sounds for Asterisk

2009-01-30 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=483331


Jeffrey C. Ollie j...@ocjtech.us changed:

   What|Removed |Added

 Blocks||428832




-- 
Configure 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 464013] Review Request: findbugs-bcel - Byte Code Engineering Library with findbugs extensions

2009-01-30 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=464013


Jerry James loganje...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




--- Comment #16 from Jerry James loganje...@gmail.com  2009-01-30 15:54:14 
EDT ---
Thank you, Mary Ellen Foster.

-- 
Configure 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 464014] Review Request: findbugs - Find bugs in Java code

2009-01-30 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=464014


Bug 464014 depends on bug 464013, which changed state.

Bug 464013 Summary: Review Request: findbugs-bcel - Byte Code Engineering 
Library with findbugs extensions
https://bugzilla.redhat.com/show_bug.cgi?id=464013

   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 479903] Review Request: gdesklet-slideshow - Cycle through a collection of pictures

2009-01-30 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=479903


Luya Tshimbalanga l...@fedoraproject.org changed:

   What|Removed |Added

 CC||l...@fedoraproject.org




--- Comment #18 from Luya Tshimbalanga l...@fedoraproject.org  2009-01-30 
15:56:09 EDT ---
(In reply to comment #16)
 Maybe the gdesklets maintainer can comment on this?
 
 Bottom of comment 14 is some oddity in Fedora's gdesklets package. It is
 reproducible with the Calendar v0.62 Display in F10 gdesklets package.
 Steps to reproduce:
 
 1. Run Calendar desklet from date/time category.
 2. Right-click on desklet display for Configure desklet pop-up menu.
 3. Click Open button next to Month image input field.
 
 Result: An error dialog on top of the file manager is displayed:
 
 | The folder contents could not be displayed
 |
 | Operation not supported  [ OK ]
 
 It looks like a bug in gdesklets or an API incompatibility.

I confirmed the bug that affect other desklets with image path and submitted it
to upstream.
https://bugs.launchpad.net/fedora/+source/gdesklets/+bug/323347

-- 
Configure 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 483016] Package Review: perl-NOCpulse-Debug - Perl debug output package

2009-01-30 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=483016


Jon Stanley jonstan...@gmail.com changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
 CC||jonstan...@gmail.com
   Flag||fedora-review?




--- Comment #2 from Jon Stanley jonstan...@gmail.com  2009-01-30 15:52:03 EDT 
---
I'll take care of this this evening, hopefully.

-- 
Configure 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 474980] Review Request: ovm - Open Verification Methodology : IEEE 1800 SystemVerilog standard

2009-01-30 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=474980





--- Comment #23 from Chitlesh GOORAH cgoo...@yahoo.com.au  2009-01-30 
16:25:06 EDT ---
(In reply to comment #20)
 not having iverilog in means that ovm is only available to a subset of people
 who have access to the closed proprietary tools.  We want you to get this in, 
 all we are asking is to provide a way for everyone to use the content.  even 
 if
 for now its limited in functionality.

iverilog is under the fedora umbrella since one or two years. perl-Verilog has
incorporated early perl support of SystemVerilog
http://www.veripool.org/search/index/verilog-perl?q=systemverilog

-- 
Configure 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 481527] Review Request: bucardo - asynchronous PostgreSQL replication system

2009-01-30 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=481527


Ruben Kerkhof ru...@rubenkerkhof.com changed:

   What|Removed |Added

 CC||ru...@rubenkerkhof.com




--- Comment #2 from Ruben Kerkhof ru...@rubenkerkhof.com  2009-01-30 16:30:34 
EDT ---
I think it would be better to use the upstream source tarball for Source0, and
remove the included, unwanted perl modules in the %prep phase. That way we can
verify if the tarball is the same as upstream's

-- 
Configure 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 459948] Review Request: libhildon - Hildon Application Framework - shared libraries

2009-01-30 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=459948





--- Comment #3 from Michel Alexandre Salim michel.syl...@gmail.com  
2009-01-30 16:42:16 EDT ---
Updated to 2.0.6; build requirements on libtool and gettext added:

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

http://salimma.fedorapeople.org/specs/maemo/libhildon.spec
http://salimma.fedorapeople.org/specs/maemo/libhildon-2.0.6-1.fc11.src.rpm

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

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


[Bug 482985] Renaming review: gentium-fonts to sil-gentium-fonts

2009-01-30 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=482985


Roozbeh Pournader rooz...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


[Bug 482993] Renaming review: doulos-fonts to sil-doulos-fonts

2009-01-30 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=482993


Roozbeh Pournader rooz...@gmail.com changed:

   What|Removed |Added

 Status|ASSIGNED|CLOSED
 Resolution||NEXTRELEASE




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

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


  1   2   >