[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489686 --- Comment #62 from Jussi Lehtola jussi.leht...@iki.fi 2009-06-15 02:10:06 EDT --- (In reply to comment #61) In summary, we would have: -- BuildRequires: cmake, atlas-devel, lapack-devel, boost-devel %package devel # Comment justifying the use of a 'Requires:' line Requires: boost-devel -- Is that fine? Or did I miss something? No, you missed boost-devel which is needed if armadillo headers use boost classes / something else only available in the boost header files. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505623] Review Request: python-urllib2_kerberos - Kerberos over HTTP Negotiate/SPNEGO support for urllib2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505623 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added CC||panem...@gmail.com --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-06-15 02:26:59 EDT --- Build failed at http://koji.fedoraproject.org/koji/taskinfo?taskID=1414339 Add BuildRequires: python-setuptools-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 505623] Review Request: python-urllib2_kerberos - Kerberos over HTTP Negotiate/SPNEGO support for urllib2
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505623 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|panem...@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 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=504709
Orcan 'oget' Ogetbil oget.fed...@gmail.com changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||oget.fed...@gmail.com
AssignedTo|nob...@fedoraproject.org|oget.fed...@gmail.com
Flag||fedora-review?
--- Comment #2 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15
02:42:24 EDT ---
Here are my notes for this package:
! The timestamps in the source gem file are all wrong. Please ask upstream to
correct this.
? rpmlint says
rubygem-gettext_activerecord-doc.noarch: W: no-documentation
Shouldn't the contents of this package be marked %doc ?
* Please remove the binary .mo files in %prep
? The COPYING file you are packaging claims LGPL as the license.
The source files say You may redistribute it and/or modify it under the same
license terms as Ruby.
Meanwhile the license tag says GPLv2 or Ruby
What's going on? :)
? What are these BuildRequires(check)'s for? (I never saw them before. And
Fedora guidelines don't mention them)
* A package must not contain any duplicate files in the %files listing.
build.log says
warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/gettext_activerecord-2.0.4/test/test_parser.rb
warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/gettext_activerecord-2.0.4/test/test_validations.rb
* The indentation seems wrong with this line
* extract messages from models with the rake task.
* Ruby packaging guidelines say that the %build section of the specfile should
be empty and the install should be performed with the command
gem install --local --install-dir %{buildroot}%{gemdir} --force %{SOURCE0}
Any reason why you are doing it differently?
- koji rawhide build seems fine:
http://koji.fedoraproject.org/koji/taskinfo?taskID=1414345
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=482216
--- Comment #12 from Rakesh Pandit rpan...@redhat.com 2009-06-15 02:41:03
EDT ---
(In reply to comment #10)
Here are my initial notes. I still have to check the licensing stuff, the doc
files and a few other things:
* Please remove %{version} from URL. Isn't this the actual website?
http://code.enthought.com/projects/mayavi/
* Source0 gives 404. Also, could you use %{name} and %{version} in Source0 and
wherever else they can be used?
! Patches should be explained and be submitted to upstream and upstream
tracker
links should be given (if available) as comments
? Is the group tag correct?
* The BR's python-devel, python-setupdocs, python-Traits don't seem necessary.
The package builds the same way without them.
python-setupdocs is required (build fails without it):
distutils.errors.DistutilsError: Could not find suitable
distribution for Requirement.parse('setupdocs=1.0')
Other two are not required. Removed them.
* rpmlint says:
Mayavi.x86_64: W: hidden-file-or-dir
/usr/lib64/python2.6/site-packages/enthought/mayavi/html/.buildinfo
Mayavi.x86_64: W: hidden-file-or-dir
/usr/lib64/python2.6/site-packages/enthought/tvtk/html/.buildinfo
Can we get rid of these files?
Done.
? What is this line for?
sed -i 's/\.dev$//g'
$RPM_BUILD_ROOT/%{python_sitearch}/%{name}-%{version}-*.egg-info/requires.txt
It removes .dev for versions in requires.txt, it is a known upstream problem ..
and is being taken care in svn.
? What is this file that gets packaged?
/usr/lib64/python2.6/site-packages/Mayavi-3.2.0-py2.6-nspkg.pth
Python script to locate enthought modules. Required, did not checked details.
! I don't think we need this anymore:
# unstripped-binary-or-object
#chmod +x $RPM_BUILD_ROOT/%{python_sitearch}/enthought/tvtk/array_ext.so
If you want to keep it, please escape the macro with an extra %
removed.
- koji rawhide and F-10 seem fine
http://koji.fedoraproject.org/koji/taskinfo?taskID=1408825
http://koji.fedoraproject.org/koji/taskinfo?taskID=1408830
? There is this release dependent BR:
%if 0%{?fedora} = 11
BuildRequires: numpy-f2py
%else
BuildRequires: numpy
%endif
Also there is this release independent R:
Requires: numpy
Is there an inconsistency here?
No, actually in F-11 distutils sub package was moved to numpy-f2py but numpy
still has it for other stable releases. It is a BR. But mayavi still uses numpy
for functioning (array calculations), which is a R.
! It seems like the description is made to span 70 columns. Can you make it
span 80 columns instead (as far as possible)?
I tried doing it .. but it messes it up, my emacs adjusted it better so I have
left it that way.
* Package requires
python-Traits = 3.1.0
python-TraitsBackendQt = 3.1.0
Can you add these versioned dependencies?
fixed.
* Packages containing GUI applications must include a %{name}.desktop file
(which applies to both the mayavi2 and tvtk_doc executables), and that file
must be properly installed with desktop-file-install in the %install section.
Follow:
http://fedoraproject.org/wiki/Packaging/Guidelines#Desktop_files
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
* Packages must not own files or directories already owned by other packages.
This package owns
/usr/lib64/python2.6/site-packages/enthought/
which is also owned by python-Traits. This needs attention.
Fixed.
* tvtk_doc exits via segmentation fault.
It works on my F11, tested on F10 also, may you paste the error message or any
crash dump ? I was not able to reproduce.
Will update once issues in other comments are also fixed.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505965] Review Request: supybot-meetbot - Plugin for Supybot for handling IRC meetings
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505965 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@gmail.com Flag||fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-06-15 02:49:16 EDT --- Review: + package builds in mock (rawhide i586). koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1414340 + rpmlint is silent for SRPM and for RPM. + source files match upstream url. + 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 is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Not a GUI application Should: 1) change %define to %global 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 505957] Review Request: perl-Syntax-Highlight-Perl-Improved - Highlighting of Perl Syntactical Structures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505957 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-06-15 02:54:02 EDT --- Thanks for the review. Asking for cvs. New Package CVS Request === Package Name: perl-Syntax-Highlight-Perl-Improved Short Description: Highlighting of Perl Syntactical Structures Owners: eseyman Branches: F-11 F-10 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505955] Review Request: perl-Module-Starter-Plugin-CGIApp - Template based module starter for CGI apps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505955 --- Comment #2 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-06-15 02:44:45 EDT --- My bad. That should be: Spec URL: http://people.parinux.org/~seyman/fedora/perl-Module-Starter-Plugin-CGIApp/perl-Module-Starter-Plugin-CGIApp.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Module-Starter-Plugin-CGIApp/perl-Module-Starter-Plugin-CGIApp-0.10-1.fc11.src.rpm Description: This module subclasses Module::Starter::Plugin::Template which in turn subclasses Module::Starter::Simple. This document only describes the methods which are overriden from those modules or are new. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #13 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15 02:58:50 EDT --- Alright thanks. This is the gdb backtrace when I exit via tvtk_doc OK or Cancel Program received signal SIGSEGV, Segmentation fault. 0x00320101362d in fini_context_translations () at setrans_client.c:243 243 free(prev_r2t_trans); It is selinux related. I don't know what is wrong. I made a fresh F-11 install last week. I will try this on an F-10 box tomorrow. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505955] Review Request: perl-Module-Starter-Plugin-CGIApp - Template based module starter for CGI apps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505955 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #3 from Parag AN(पराग) panem...@gmail.com 2009-06-15 03:02:26 EDT --- eview: + package builds in mock (rawhide i586). koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1414354 + rpmlint is silent for SRPM and for RPM. + source files match upstream url e5bab1d5d4fcf419b5b3f193ee8fe59d1688ae2c Module-Starter-Plugin-CGIApp-0.10.tar.gz + 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 is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=7, Tests=97, 2 wallclock secs ( 0.08 usr 0.01 sys + 1.44 cusr 0.12 csys = 1.65 CPU) + Package perl-Module-Starter-Plugin-CGIApp-0.10-1.fc10 = Provides: perl(Module::Starter::Plugin::CGIApp) = 0.10 Requires: /usr/bin/perl perl(Carp) perl(English) perl(ExtUtils::Command) perl(File::Basename) perl(File::Spec) perl(HTML::Template) perl(Module::Starter) perl(Module::Starter::App) perl(Module::Starter::Simple) perl(base) perl(strict) perl(warnings) + Not a GUI application 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 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493250 --- Comment #36 from Liang Suilong liangsuil...@gmail.com 2009-06-15 03:05:45 EDT --- (In reply to comment #34) you have small typo in %description you say that you have small tupo in %description. What do you mean? Do you mean that %description should be longer? Would I add this paragraph into %description? GTK+ does't has an buildin canvas widget. GooCanvas is wonderful. It is easy to use and has powerful and extensible way to create items in canvas. Just try it. For more documents, please read GooCanvas Manual and the demo programs provided in the source distribution in both perl-Goo::Canvas and GooCanvas. Or do you mean that I should set every line of %description longer than the present 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 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Depends on||505995 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #14 from Rakesh Pandit rpan...@redhat.com 2009-06-15 03:18:51 EDT --- (In reply to comment #10) ? What is this file that gets packaged? /usr/lib64/python2.6/site-packages/Mayavi-3.2.0-py2.6-nspkg.pth Removed. (In reply to comment #11) Here is the rest of the review: * As indicated by LICENSE_COLORBREWER.txt, parts of the file enthought/mayavi/core/lut/pylab_luts.py is distributed under ASL 1.1 and it should be added to the license tag. Done. * image_LICENSE.txt says Crystal: LGPL license as described in icon_LICENSE_CP.txt but icon_LICENSE_CP.txt contains the full text of GPLv2. Can you ask this upstream for clarification? Even if they say it is LGPL, we cannot ship this icon_LICENSE_CP.txt file as it is wrong. Will remove after asking from upstream after clearing doubts. * image_LICENSE_OOo is LGPLv3+ and this has to be added to the license tag. I think LGPLv2+ (as it has or later) covers LGPLv3+ also. Or am I wrong ? * docs/mayavi2.man should be packaged. Is this a man1? Yes included. ? Is the documentation under docs/source/ worth packaging? Yes included Will updated, once I get confirmation from upstream about license. Thanks, -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #15 from Rakesh Pandit rpan...@redhat.com 2009-06-15 03:19:45 EDT --- (In reply to comment #13) Alright thanks. This is the gdb backtrace when I exit via tvtk_doc OK or Cancel Program received signal SIGSEGV, Segmentation fault. 0x00320101362d in fini_context_translations () at setrans_client.c:243 243 free(prev_r2t_trans); It is selinux related. I don't know what is wrong. I made a fresh F-11 install last week. I will try this on an F-10 box tomorrow. Will try to reproduce and report upstream. Somehow I was not able to reproduce again. :( Checking ... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479800 --- Comment #6 from Jens Petersen peter...@redhat.com 2009-06-15 03:35:24 EDT --- Luckily and unknowningly I already packaged hscolour so it is already there. So please update ghc-haskell-src-exts. :) Jens -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505955] Review Request: perl-Module-Starter-Plugin-CGIApp - Template based module starter for CGI apps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505955 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-06-15 03:34:16 EDT --- Thanks. Requesting fedora-cvs. New Package CVS Request === Package Name: perl-Module-Starter-Plugin-CGIApp Short Description: Template based module starter for CGI apps Owners: eseyman Branches: F-11 F-10 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479800] Review Request: hlint - Provides Haskell Source Code Suggestions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479800 --- Comment #7 from Conrad Meyer kon...@tylerc.org 2009-06-15 03:40:38 EDT --- Ok, cool. It is late here, I will do this when I wake up tomorrow :). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493250 --- Comment #37 from Jan Klepek jan.kle...@hp.com 2009-06-15 03:47:33 EDT --- no, you have does't and probably you mean doesn't. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=504709
--- Comment #3 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15
03:48:09 EDT ---
Hi:
(In reply to comment #2)
! The timestamps in the source gem file are all wrong. Please ask upstream to
correct this.
- This is rather the issue on gem, not this gem specific.
? rpmlint says
rubygem-gettext_activerecord-doc.noarch: W: no-documentation
Shouldn't the contents of this package be marked %doc ?
- I don't think it is needed to mark as %doc the files in the rpm
which is already declared as document rpm
* Please remove the binary .mo files in %prep
- Done at %build
? The COPYING file you are packaging claims LGPL as the license.
The source files say You may redistribute it and/or modify it under the same
license terms as Ruby.
Meanwhile the license tag says GPLv2 or Ruby
What's going on? :)
- Actually from the source code this is licensed under the same
license as ruby. Note that ruby is licensed under GPLv2 or Ruby
(well, a little complicated...), so the license tag should
be so.
? What are these BuildRequires(check)'s for? (I never saw them before. And
Fedora guidelines don't mention them)
- The intention for this is that These BuildRequires is needed only
for %check.
* A package must not contain any duplicate files in the %files listing.
build.log says
warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/gettext_activerecord-2.0.4/test/test_parser.rb
warning: File listed twice:
/usr/lib/ruby/gems/1.8/gems/gettext_activerecord-2.0.4/test/test_validations.rb
- See the test case bug 505995. I thought this worked as expected.
Workarround applied.
* The indentation seems wrong with this line
* extract messages from models with the rake task.
- Fixed.
* Ruby packaging guidelines say that the %build section of the specfile should
be empty and the install should be performed with the command
gem install --local --install-dir %{buildroot}%{gemdir} --force %{SOURCE0}
Any reason why you are doing it differently?
- Actually it is under discussion whether gem install should completely
moved from %install to %prep or %build. This suggestion was from other people
but now I also think that gem install should not done at %install
directly
because:
- Actually when gem creates C module extension, rubygem guideline
already says gem --install should be done at %build because
of creating debuginfo rpm correctly (and this proposal was
from me)
- For this package, spec file contains %check. When gem is installed
under %buildroot, %check must be done also under %buildroot.
This is troublesome when %check (for this package rake test) creates
some additional files because this will cause some unneeded filed
to be packaged or causes installed but not packaged error.
http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord.spec
http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord-2.0.4-3.fc.src.rpm
* Mon Jun 15 2009 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp - 2.0.4-3
- Recreate gettext mo files (BR: rubygem(gettext))
- Change BR: ruby(sqlite) - rubygem(sqlite3-ruby)
- Some cleanups
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505845] Review Request: hyphen-cy - Welsh hyphenation patterns
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505845 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Caolan McNamara caol...@redhat.com 2009-06-15 04:04:53 EDT --- New Package CVS Request === Package Name: hyphen-cy Short Description: Welsh hyphenation rules Owners: caolanm Branches: devel InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502795] Review Request: mcu8051ide - IDE for MCS-51 based microcontrollers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502795 Shakthi Kannan shakthim...@gmail.com changed: What|Removed |Added Flag|needinfo?(shakthim...@gmail | |.com) | --- Comment #16 from Shakthi Kannan shakthim...@gmail.com 2009-06-15 04:01:53 EDT --- No Internet connectivity for the system that I have my Fedora certificate and ssh key with, for the past few days. Will do so by this coming weekend. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505770] Review Request: hyphen-gl - Galician hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505770 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Caolan McNamara caol...@redhat.com 2009-06-15 04:04:14 EDT --- New Package CVS Request === Package Name: hyphen-gl Short Description: Galician hyphenation rules Owners: caolanm Branches: devel InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505760] Review Request: hyphen-eu - Basque hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505760 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Caolan McNamara caol...@redhat.com 2009-06-15 04:03:26 EDT --- New Package CVS Request === Package Name: hyphen-eu Short Description: Basque hyphenation rules Owners: caolanm Branches: devel InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489686 --- Comment #63 from Conrad Sanderson conrads...@ieee.org 2009-06-15 04:17:18 EDT --- (In reply to comment #60) IMO the Requires: atlas-devel, lapack-devel of armadillo-devel are probably unnecessary. If one builds with g++ -larmadillo not with g++ -llapack -latlas -larmadillo then one doesn't need atlas-devel or lapack-devel and these can be safely dropped, since armadillo acts as a wrapper for the atlas and lapack libraries. Agreed to omitting lapack-devel. However, we need atlas-devel. Atlas headers are included by Armadillo headers. (line 85 of upstream include/armadillo, where a workaround for Ubuntu and other systems is present). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505578] Review Request: hunspell-se - Northern Sami hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505578 Caolan McNamara caol...@redhat.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Caolan McNamara caol...@redhat.com 2009-06-15 04:02:15 EDT --- New Package CVS Request === Package Name: hunspell-se Short Description: Northern Sami hunspell dictionaries Owners: caolanm Branches: devel InitialCC: caolanm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489686 --- Comment #64 from Conrad Sanderson conrads...@ieee.org 2009-06-15 04:35:00 EDT --- Updated spec and SRPM: http://arma.sf.net/fedora/armadillo.spec http://arma.sf.net/fedora/armadillo-0.6.11-8.fc10.src.rpm Spec file summary: main package: BuildRequires: cmake, boost-devel, lapack-devel, atlas-devel devel package: Requires: boost-devel, atlas-devel (and a note explaining why they're needed) install section: rm -f examples/Makefile.cmake I think these satisfy everyone's concerns. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506006] New: Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application https://bugzilla.redhat.com/show_bug.cgi?id=506006 Summary: Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: emmanuel.sey...@club-internet.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.parinux.org/~seyman/fedora/perl-Test-WWW-Mechanize-CGIApp/perl-Test-WWW-Mechanize-CGIApp.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Test-WWW-Mechanize-CGIApp/perl-Test-WWW-Mechanize-CGIApp-0.05-1.fc11.src.rpm Description: This package makes testing CGIApp based modules fast and easy. It takes advantage of Test::WWW::Mechanize to provide functions for common web testing scenarios. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506007] Review Request: perl-Sub-WrapPackages - Add wrappers around all the subroutines in packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506007 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Summary|Review Request: main |Review Request: |package name here - short |perl-Sub-WrapPackages - Add |summary here |wrappers around all the ||subroutines in packages -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506007] New: Review Request: main package name here - short summary here
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: main package name here - short summary here https://bugzilla.redhat.com/show_bug.cgi?id=506007 Summary: Review Request: main package name here - short summary here Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: emmanuel.sey...@club-internet.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.parinux.org/~seyman/fedora/perl-Sub-WrapPackages/perl-Sub-WrapPackages.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-Sub-WrapPackages/perl-Sub-WrapPackages-1.2-1.fc11.src.rpm Description: This is mostly a wrapper around Damian Conway's Hook::LexWrap module. This module allows you to wrap function calls and exits with functions of your choice. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506006] Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506006 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@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 506007] Review Request: perl-Sub-WrapPackages - Add wrappers around all the subroutines in packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506007 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Status|NEW |ASSIGNED CC||panem...@gmail.com AssignedTo|nob...@fedoraproject.org|panem...@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 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #16 from Rakesh Pandit rpan...@redhat.com 2009-06-15 05:05:30 EDT --- * image_LICENSE.txt says Crystal: LGPL license as described in icon_LICENSE_CP.txt but icon_LICENSE_CP.txt contains the full text of GPLv2. Can you ask this upstream for clarification? Even if they say it is LGPL, we cannot ship this icon_LICENSE_CP.txt file as it is wrong. Will remove after asking from upstream after clearing doubts. The only issue here was that all files named icon_*.txt in file are actually image_*.txt files. But image_LICENSE_CP.txt is not GPL, it is LGPL ? May you recheck and confirm ? Wrong naming, I have fixed. So, to me only issue remaining is crash .. I will am checking whether I can reproduce it. Once that is done will update. Thanks, -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #17 from Rakesh Pandit rpan...@redhat.com 2009-06-15 05:17:20 EDT --- http://rakesh.fedorapeople.org/srpm/Mayavi-3.2.0-3.fc11.src.rpm http://rakesh.fedorapeople.org/spec/Mayavi.spec Fixed All .. not able to reproduce crash :( May you check whether there is any other issues remaining ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226429] Merge Review: sqlite
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226429 --- Comment #21 from Panu Matilainen pmati...@redhat.com 2009-06-15 05:28:52 EDT --- Wohoo :) Thanks. -- 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 506006] Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506006 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-06-15 05:39:00 EDT --- Review: + package builds in mock (rawhide i586). koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1414513 + rpmlint is silent for SRPM and for RPM. + source files match upstream url 5caa9fe9ec918ad75657228ae0a41ac0a5efb108 Test-WWW-Mechanize-CGIApp-0.05.tar.gz + 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 is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=6, Tests=39, 2 wallclock secs ( 0.06 usr 0.01 sys + 1.29 cusr 0.12 csys = 1.48 CPU) + Package perl-Test-WWW-Mechanize-CGIApp-0.05-1.fc12.noarch = Provides: perl(Test::WWW::Mechanize::CGIApp) = 0.05 Requires: perl(HTTP::Request::AsCGI) perl(base) perl(strict) perl(warnings) + Not a GUI application When I try to install this package, I saw we need to add Requires: perl(CGI::Application) Requires: perl(Test::WWW::Mechanize) Requires: perl(Test::More) yum does not pick it up automatically. 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 502851] Review Request: pokerth - A Texas-Holdem poker game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502851 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2009-06-15 05:39:53 EDT --- pokerth-0.7-3.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/pokerth-0.7-3.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 494845] Review Request: xdrfile - A small C-library for reading and writing GROMACS trr and xtc files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=494845 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2009-06-15 05:40:10 EDT --- xdrfile-1.1-2.fc10 has been submitted as an update for Fedora 10. http://admin.fedoraproject.org/updates/xdrfile-1.1-2.fc10 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 494845] Review Request: xdrfile - A small C-library for reading and writing GROMACS trr and xtc files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=494845 --- Comment #12 from Fedora Update System upda...@fedoraproject.org 2009-06-15 05:41:34 EDT --- xdrfile-1.1-2.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/xdrfile-1.1-2.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506007] Review Request: perl-Sub-WrapPackages - Add wrappers around all the subroutines in packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506007 Parag AN(पराग) panem...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #1 from Parag AN(पराग) panem...@gmail.com 2009-06-15 05:43:04 EDT --- Review: + package builds in mock (rawhide i586). koji Build =http://koji.fedoraproject.org/koji/taskinfo?taskID=1414511 + rpmlint is silent for SRPM and for RPM. + source files match upstream url e93348e9d3d4cec8629f95509ab5cc949b7b0008 Sub-WrapPackages-1.2.tar.gz + 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 is present. + BuildRequires are proper. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains code, not content. + no headers or static libraries. + no .pc file present. + no -devel subpackage + no .la files. + no translations are available + Does owns the directories it creates. + no scriptlets present. + no duplicates in %files. + file permissions are appropriate. + make test gave All tests successful. Files=4, Tests=10, 1 wallclock secs ( 0.02 usr 0.00 sys + 0.10 cusr 0.05 csys = 0.17 CPU) + Package perl-Sub-WrapPackages-1.2-1.fc12.noarch = Provides: perl(Sub::WrapPackages) = 1.2 Requires: perl(Data::Dumper) perl(Hook::LexWrap) perl(strict) perl(vars) perl(warnings) + Not a GUI application 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 451744] Review Request: root - The CERN analyzer for high to medium energy physics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451744 --- Comment #34 from Kostas Georgiou k.georg...@imperial.ac.uk 2009-06-15 07:48:27 EDT --- The software does use fontconfig so the X side is OK AFAIK, the problem is in GL/postscript output. The major places that ttf files are requested directly that I can see are in: http://root.cern.ch/viewvc/trunk/graf2d/graf/src/TTF.cxx where freetype is used to load a font by path/name or number (MS with fallback to Free fonts). http://root.cern.ch/viewvc/trunk/graf3d/gl/src/TGLText.cxx where ftgl is used to load a font by fontnumber (table uses only MS fonts). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=493250
--- Comment #38 from Liang Suilong liangsuil...@gmail.com 2009-06-15
08:21:34 EDT ---
Created an attachment (id=347924)
-- (https://bugzilla.redhat.com/attachment.cgi?id=347924)
perl-Goo-Canvas 0.06-2 spec
Now I upload the latest spec file
Jan Klepek,
Now is %description right?
Mamoru Tasaka,
Now I correct %file
%{perl_vendorarch}/Goo/ is here.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 451744] Review Request: root - The CERN analyzer for high to medium energy physics
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=451744 --- Comment #35 from Kostas Georgiou k.georg...@imperial.ac.uk 2009-06-15 08:24:30 EDT --- sorry I was wrong root uses freetype but not fontconfig, so as I see it the plan should be * quick solution: point root to the DejaVu fonts and check that it still works OK * add fontconfig support -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506056] Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506056 Eric Christensen e...@christensenplace.us changed: What|Removed |Added CC||da...@gnsa.us Blocks||505940 Depends on||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 487097] Review Request: ReviewBoard - web based code review tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=487097 Cristian Ciupitu cristian.ciup...@yahoo.com changed: What|Removed |Added CC||cristian.ciup...@yahoo.com --- Comment #9 from Cristian Ciupitu cristian.ciup...@yahoo.com 2009-06-15 08:32:44 EDT --- Are you sure about the BuildRequires: python-setuptools-devel dependency? I was able to build the RPM without 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 506056] New: Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors https://bugzilla.redhat.com/show_bug.cgi?id=506056 Summary: Review Request: zikula-module-scribite - The scribite! module for Zikula allows integration of various text editors Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: e...@christensenplace.us QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sparks.fedorapeople.org/Packages/zikula-module-scribite/zikula-module-scribite.spec SRPM URL: http://sparks.fedorapeople.org/Packages/zikula-module-scribite/zikula-module-scribite-3.1-1.fc11.src.rpm Description: scribite! is a module for easy integration of WYSIWYG scripts Xinha, TinyMCE, FCKeditor, openWYSIWYG, NicEdit or YUI Rich Text Editor into textarea fields in order to make text editing a little bit nicer and more comfortable for users. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 465534] Resurrect awesfx package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=465534 Guido Grazioli guido.grazi...@gmail.com changed: What|Removed |Added CC||guido.grazi...@gmail.com --- Comment #6 from Guido Grazioli guido.grazi...@gmail.com 2009-06-15 08:39:47 EDT --- (In reply to comment #5) Since this is currently a package review ticket, and a package isn't actually being submitted here, I can only close it. I would try to assign it to a more appropriate component, except that it started at kernel for some reason. Bottom line: Fedora is a community project. If someone steps up to maintain the package, they can open a proper package review ticket and, once the review process is done, the package can reappear in Fedora. Without a maintainer, the package can't be in the distro. I have an approved package review here: https://bugzilla.redhat.com/show_bug.cgi?id=490061 should i ask for a cvs change package or cvs new package admin request? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226117] Merge Review: mailman
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226117 --- Comment #11 from Jon Ciesla l...@jcomserv.net 2009-06-15 08:47:22 EDT --- Now I get this on the SRPM: mailman.src: W: strange-permission mailman-crontab-edit 0755 A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. mailman.src: W: strange-permission mailman-migrate-fhs 0755 A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. mailman.src: W: strange-permission mailman-update-cfg 0755 A file that you listed to include in your package has strange permissions. Usually, a file should have 0644 permissions. mailman.src: W: mixed-use-of-spaces-and-tabs (spaces: line 63, tab: line 172) The specfile mixes use of spaces and tabs for indentation, which is a cosmetic annoyance. Use either spaces or tabs for indentation, not both. and the same as #9 on the RPM. -- 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 465534] Resurrect awesfx package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=465534 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added CC||jussi.leht...@iki.fi --- Comment #7 from Jussi Lehtola jussi.leht...@iki.fi 2009-06-15 08:45:07 EDT --- (In reply to comment #6) I have an approved package review here: https://bugzilla.redhat.com/show_bug.cgi?id=490061 should i ask for a cvs change package or cvs new package admin request? There is a CVS module for the package, so I'd say a cvs change request, but in any case it's done by the CVS admin who will surely do the right thing even if you make a new package admin request. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 226330] Merge Review: pychecker
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=226330
Vitezslav Crhonek vcrho...@redhat.com changed:
What|Removed |Added
Flag|needinfo? |
--- Comment #3 from Vitezslav Crhonek vcrho...@redhat.com 2009-06-15
10:26:54 EDT ---
(In reply to comment #1)
- Drop
Requires: python
and
BuildRequires: python
since both of these are redundant (first is automatically picked up and second
is a requirement of python-devel).
Fixed.
- Remove
CFLAGS=$RPM_OPT_FLAGS
from the build phase since this is a noarch package.
Fixed.
**
rpmlint output:
pychecker.noarch: W: summary-ended-with-dot A python source code checking
tool.
pychecker.noarch: E: tag-not-utf8 %changelog
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/Stack.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/OP.py 0644
pychecker.noarch: W: file-not-utf8 /usr/share/doc/pychecker-0.8.17/MAINTAINERS
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/warn.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/printer.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/CodeChecks.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/Warning.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/msgs.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/utils.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/Config.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/checker.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/function.py 0644
pychecker.noarch: E: non-executable-script
/usr/lib/python2.5/site-packages/pychecker/python.py 0644
pychecker.src: W: summary-ended-with-dot A python source code checking tool.
pychecker.src: E: tag-not-utf8 %changelog
pychecker.src: E: non-utf8-spec-file
/tmp/rpmlint.pychecker-0.8.17-8.fc10.src.rpm.t9ghWq/pychecker.spec
2 packages and 0 specfiles checked; 15 errors, 3 warnings.
- Fix the above. You can get rid of the non-executable script errors by
removing the shebang with sed in the setup phase, e.g.
sed -i -e '1d' pychecker/checker.py
Fixed.
- Maybe add -O1 --skip-build to the install argument of setup.py?
There are problems with '--skip-build'.
MUST: The spec file for the package is legible and macros are used
consistently. OK
MUST: The package must be named according to the Package Naming Guidelines. OK
MUST: The spec file name must match the base package %{name}. OK
MUST: The package must be licensed with a Fedora approved license and meet
the
Licensing Guidelines. OK
MUST: The License field in the package spec file must match the actual
license.
OK
- Source code contains no license heders, please ask upstream to add them.
MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. OK
MUST: The package MUST successfully compile and build into binary rpms. OK
MUST: The spec file MUST handle locales properly. OK
MUST: Optflags are used and time stamps preserved. OK
MUST: Packages containing shared library files must call ldconfig. OK
MUST: A package must own all directories that it creates or require the
package
that owns the directory. OK
MUST: Files only listed once in %files listings. OK
MUST: Permissions on files must be set properly. OK
MUST: Clean section exists. OK
MUST: Large documentation files must go in a -doc subpackage. OK
MUST: All relevant items are included in %doc. Items in %doc do not affect
runtime of application. OK
MUST: Header files must be in a -devel package. OK
MUST: Static libraries must be in a -static package. OK
MUST: Packages containing pkgconfig(.pc) files must 'Requires: pkgconfig'. OK
MUST: If a package contains library files with a suffix then library files
ending in .so must go in a -devel package. OK
MUST: In the vast majority of cases, devel packages must require the base
package using a fully versioned dependency. OK
MUST: Packages does not contain any .la libtool archives. OK
MUST: Desktop files are installed properly. OK
MUST: No file conflicts with other packages and no general names. OK
MUST: Buildroot cleaned before install. OK
SHOULD: %{?dist} tag is used in release. OK
SHOULD: If the package does not include license text(s) as separate files from
upstream, the packager should query upstream to include it. OK
SHOULD: The
[Bug 226330] Merge Review: pychecker
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226330 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Flag|fedora-review? |fedora-review+ --- Comment #4 from Jussi Lehtola jussi.leht...@iki.fi 2009-06-15 10:44:39 EDT --- Okay. Everything seems OK, but I can't check the rpmlint warnings since you haven't built the package in rawhide. Those were minor issues anyway. APPROVED -- 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 506100] New: Review Request: opennhrp - An NHRP implementation for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: opennhrp - An NHRP implementation for Linux https://bugzilla.redhat.com/show_bug.cgi?id=506100 Summary: Review Request: opennhrp - An NHRP implementation for Linux Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: tja...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://sites.google.com/site/tjarls/fedora/opennhrp.spec SRPM URL: http://sites.google.com/site/tjarls/fedora/opennhrp-0.10.3-3.fc10.src.rpm Description: OpenNHRP implements NBMA Next Hop Resolution Protocol (as defined in RFC 2332). It makes it possible to create dynamic multipoint VPN Linux router using NHRP, GRE and IPsec. It aims to be Cisco DMVPN compatible. As this is my first package submission, I am seeking a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506100] Review Request: opennhrp - An NHRP implementation for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506100 Charles Lopes tja...@gmail.com changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 502358] Review Request: mojomojo - Catalyst DBIx::Class powered Wiki
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=502358
Lorenzo Villani lvill...@binaryhelix.net changed:
What|Removed |Added
CC||lvill...@binaryhelix.net
--- Comment #6 from Lorenzo Villani lvill...@binaryhelix.net 2009-06-15
11:29:29 EDT ---
Today I tried to install this package but mojomojo refused to start because
perl-JSON was not installed.
On a side note: I also had to make /var/lib/mojomojo/{index,upload} writable
for the apache user.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 461393] Review Request: congruity - Application t o program Logitech® Harmony® universal remote control s
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=461393 Adam Williamson awill...@redhat.com changed: What|Removed |Added CC||awill...@redhat.com --- Comment #6 from Adam Williamson awill...@redhat.com 2009-06-15 12:14:33 EDT --- it'd be nice for this review to get resurrected - I used congruity this weekend but had to download it manually. it'd be good for congruity to ship a configuration file to make browsers use it for *.EZ* files automatically, but I'm not sure if that's possible, because they use generic MIME types: [r...@adam tmp]# file --mime-type *.EZ* Connectivity-1.EZHex: application/xml Connectivity-2.EZHex: application/xml Connectivity-3.EZHex: application/xml Connectivity.EZHex: application/xml ConnectivitySimple-1.EZHex: application/xml ConnectivitySimple-2.EZHex: application/xml ConnectivitySimple-3.EZHex: application/xml ConnectivitySimple.EZHex: application/xml LatestFirmware.EZUp:text/html Update-1.EZHex: application/xml Update-2.EZHex: application/xml Update.EZHex: application/xml and I'm not sure if you can package a file to tell browsers to open a certain file type with a certain program _by extension_ rather than MIME type (which is obviously what happens in Windows). I've asked caillon if he can help answer that question... -- Fedora Bugzappers volunteer triage team https://fedoraproject.org/wiki/BugZappers -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 453422] Review Request: songbird - Mozilla based multimedia player
Please do not reply directly 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 #72 from David Halik auralva...@gmail.com 2009-06-15 12:23:28 EDT --- 1.2 is due out shortly. I'll remove the $ORIGIN rpath in the next build and repost. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 493250] Review Request: perl-Goo-Canvas -- Goo::Canvas Perl interface to the GooCanvas
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=493250 --- Comment #39 from Jan Klepek jan.kle...@hp.com 2009-06-15 12:29:09 EDT --- (In reply to comment #38) Created an attachment (id=347924) -- (https://bugzilla.redhat.com/attachment.cgi?id=347924) [details] perl-Goo-Canvas 0.06-2 spec Now I upload the latest spec file Jan Klepek, Now is %description right? Did you uploaded correct version? I see in your attachment does't -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=504709
--- Comment #4 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15
12:46:58 EDT ---
(In reply to comment #3)
Hi:
Hi, thanks for the update!
(In reply to comment #2)
? The COPYING file you are packaging claims LGPL as the license.
The source files say You may redistribute it and/or modify it under the
same license terms as Ruby.
Meanwhile the license tag says GPLv2 or Ruby
What's going on? :)
- Actually from the source code this is licensed under the same
license as ruby. Note that ruby is licensed under GPLv2 or Ruby
(well, a little complicated...), so the license tag should
be so.
Okay, but the COPYING file that is being packaged says LGPL, which is wrong
according to what you say. Shouldn't we drop the wrong COPYING file?
? What are these BuildRequires(check)'s for? (I never saw them before. And
Fedora guidelines don't mention them)
- The intention for this is that These BuildRequires is needed only
for %check.
Thanks. Now the package fails to build on F-11
+ rake --trace makemo
/var/tmp/rpm-tmp.qYI3ky: line 41: rake: command not found
I moved rubygem(rake) from BR(check) to regular BR, but then I got
+ rake --trace makemo
/usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31: command not
found: allison --path
I couldn't figure out what package provides allison on F-11.
* Ruby packaging guidelines say that the %build section of the specfile
should be empty and the install should be performed with the command
gem install --local --install-dir %{buildroot}%{gemdir} --force
%{SOURCE0}
Any reason why you are doing it differently?
- Actually it is under discussion whether gem install should completely
moved from %install to %prep or %build. This suggestion was from other
people but now I also think that gem install should not done at %install
directly
because:
- Actually when gem creates C module extension, rubygem guideline
already says gem --install should be done at %build because
of creating debuginfo rpm correctly (and this proposal was
from me)
- For this package, spec file contains %check. When gem is installed
under %buildroot, %check must be done also under %buildroot.
This is troublesome when %check (for this package rake test) creates
some additional files because this will cause some unneeded filed
to be packaged or causes installed but not packaged error.
Do you know by any chance when these changes in the guidelines will be
approved? I can't find the relevant discussion. Maybe I'm looking at the wrong
mailing lists.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505955] Review Request: perl-Module-Starter-Plugin-CGIApp - Template based module starter for CGI apps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505955 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #5 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:21:38 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505957] Review Request: perl-Syntax-Highlight-Perl-Improved - Highlighting of Perl Syntactical Structures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505957 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:20:22 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505508] Review Request: acheck-rules - Rules for acheck
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505508 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:19:42 EDT --- F-9 branches are no longer permitted. Otherwise, CVS is 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 505770] Review Request: hyphen-gl - Galician hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505770 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:25:56 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505760] Review Request: hyphen-eu - Basque hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505760 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:24:09 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505578] Review Request: hunspell-se - Northern Sami hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505578 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:23:27 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505845] Review Request: hyphen-cy - Welsh hyphenation patterns
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505845 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #3 from Jason Tibbitts ti...@math.uh.edu 2009-06-15 13:26:30 EDT --- CVS done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 --- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15 13:26:48 EDT --- (In reply to comment #4) (In reply to comment #2) ? The COPYING file you are packaging claims LGPL as the license. The source files say You may redistribute it and/or modify it under the same license terms as Ruby. Meanwhile the license tag says GPLv2 or Ruby What's going on? :) - Actually from the source code this is licensed under the same license as ruby. Note that ruby is licensed under GPLv2 or Ruby (well, a little complicated...), so the license tag should be so. Okay, but the COPYING file that is being packaged says LGPL, which is wrong according to what you say. Shouldn't we drop the wrong COPYING file? - Removed. Thanks. Now the package fails to build on F-11 + rake --trace makemo /var/tmp/rpm-tmp.qYI3ky: line 41: rake: command not found - Ah.. fixed. I moved rubygem(rake) from BR(check) to regular BR, but then I got + rake --trace makemo /usr/lib/ruby/site_ruby/1.8/rubygems/custom_require.rb:31: command not found: allison --path I couldn't figure out what package provides allison on F-11. - allison is not packaged on Fedora yet (and I have not checked the dependency...) Do you know by any chance when these changes in the guidelines will be approved? I can't find the relevant discussion. Maybe I'm looking at the wrong mailing lists. - Well, I will re-propose this later on fedora-packaging (after I write a draft: formal proposal needs a draft). http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord-2.0.4-4.fc.src.rpm http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord.spec * Tue Jun 16 2009 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp - 2.0.4-4 - Remove COPYING for now - Change BR: now rubygem(rake) is needed to create gettext mo -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504710] Review Request: rubygem-gettext_rails - Localization support for Ruby on Rails(=2.3) by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504710 --- Comment #2 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15 13:35:58 EDT --- http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_rails-2.0.4-3.fc.src.rpm http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_rails.spec * Tue Jun 16 2009 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp - 2.0.4-3 - Remove COPYING for now - Recreate gettext .mo 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 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 --- Comment #6 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15 14:07:02 EDT --- Thank you. But this still fails to build on F-11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1415248 I don't understand why, it builds fine on rawhide. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 503136] Review Request: csync - a file synchroniser utility
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=503136 --- Comment #2 from Alex Hudson (Fedora Address) fed...@alexhudson.com 2009-06-15 14:17:20 EDT --- Hi Jussi, Thanks very much for reviewing this. Sorry it has taken me a little while to respond! I've updated the spec and SRPM: http://alexh.fedorapeople.org/reviews/csync/csync.spec http://alexh.fedorapeople.org/reviews/csync/csync-0.43.0-2.fc11.src.rpm * Summary for libs - you're right. I've tried to write something better. * target platform bits - I've had to do a push/pop because csync doesn't allow in-source builds, only out-of-source. I'm not sure the KDE4 cmake macro helps in this regard; https://fedoraproject.org/wiki/SIGs/KDE gives a similar Best practice (see bottom of page). The alternative would be to patch the build system, which would be possible, but isn't a system tested by upstream. * devel requires - yes, you're right again :) * need for -libs - I plan on packaging the pam-csync module, which uses the libraries rather than the binary, hence the separation. * directory ownership - again, you're right: sorry, I'm not sure how I missed that. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506100] Review Request: opennhrp - An NHRP implementation for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506100 Steve Traylen st...@traylen.net changed: What|Removed |Added CC||st...@traylen.net --- Comment #1 from Steve Traylen st...@traylen.net 2009-06-15 14:16:42 EDT --- Hi Charles, I cannot sponsor you but I can hopefully provide some useful comments: libcares.so.2()(64bit) is picked up as a dependency so c-ares is not needed as a Requirement. https://fedoraproject.org/wiki/Packaging/Guidelines#Explicit_Requires There are a few rpmlint errors that look to need clearing up: $ rpmlint ../RPMS/x86_64/opennhrp-0.10.3-3.fc11.x86_64.rpm opennhrp.x86_64: W: spurious-executable-perm /usr/share/man/man8/opennhrp.8.gz opennhrp.x86_64: W: spurious-executable-perm /usr/share/man/man8/opennhrp-script.8.gz opennhrp.x86_64: W: spurious-executable-perm /usr/share/man/man5/opennhrp.conf.5.gz opennhrp.x86_64: W: spurious-executable-perm /usr/share/man/man8/opennhrpctl.8.gz opennhrp.x86_64: E: executable-marked-as-config-file /etc/opennhrp/opennhrp.conf opennhrp.x86_64: E: script-without-shebang /etc/opennhrp/opennhrp.conf 1 packages and 0 specfiles checked; 2 errors, 4 warnings. you can details on any particular message with: $ rpmlint -I executable-marked-as-config-file Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 498723] Review Request: eZ Publish
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=498723 --- Comment #6 from Guillaume Kulakowski llaum...@gmail.com 2009-06-15 14:26:35 EDT --- The source URL (http://ez.no/content/download/261295/1832505/version/3/file/ezpublish-4.1.0-gpl.tar.gz) causes an error. 4.1.1 is also the current version 4.1.0 seems to be un-available. There is no rules for the next release URL. Requires: php-pear(components.ez.no/Archive) Requires: php-pear(components.ez.no/ConsoleTools) Requires: php-pear(components.ez.no/File) Requires: php-pear(components.ez.no/SystemInformation) Requires: php-pear(components.ez.no/Webdav) are also not necessary. eZ Publish use this eZ Components, it's some dependence. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 --- Comment #7 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15 14:56:53 EDT --- Ah.. http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord.spec http://mtasaka.fedorapeople.org/Review_request/Ruby_on_Rails/rubygem-gettext_activerecord-2.0.4-5.fc.src.rpm * Tue Jun 16 2009 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp - 2.0.4-5 - More BR: rubygem(activerecord) for recreating mo files dist-f12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1415364 dist-f11-updates-candidate: http://koji.fedoraproject.org/koji/taskinfo?taskID=1415362 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 --- Comment #8 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15 15:11:47 EDT --- Thanks. One last question: From the SPEC file: %global activerecord_req2.3.2 ... %description gettext_activerecord provides the localization for ActiveRecord-2.2 or later using Ruby-GetText-Package. There is a discrepancy here on version number. Am I missing something? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 Orcan 'oget' Ogetbil oget.fed...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #9 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15 15:15:07 EDT --- Well, that is not a blocker. You can answer it later. --- This package (rubygem-gettext_activerecord) is APPROVED by oget --- -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Flag||fedora-cvs? --- Comment #10 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15 15:31:08 EDT --- Thank you for quick review! (In reply to comment #8) %global activerecord_req2.3.2 ... %description gettext_activerecord provides the localization for ActiveRecord-2.2 or later using Ruby-GetText-Package. I will remove -2.2 or later part. New Package CVS Request === Package Name: rubygem-gettext_activerecord Short Description: Localization support for ActiveRecord by Ruby-GetText-Package Owners: mtasaka Branches: F-11 InitialCC: (nobody) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 482216] Review Request: Mayavi - The Mayavi scientific data 3-dimensional visualizer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=482216 --- Comment #18 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-06-15 15:48:33 EDT --- I tested this on F-10 and got the same segmentation fault. But the crash is on exit and doesn't affect the application during runtime. Meanwhile, as we discussed, the missing things in the specfile are the .desktop files, the Group tag and the license thing (... and LGPLv3+). I was confused by the image_LICENSE_CP.txt file. Yes it is LGPL, so we don't have a problem there. What confused me was the license title inside the file is GNU General Public License whereas the text below is LGPL. It would be good to let upstream know about this confusion. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489686 --- Comment #65 from Denis Arnaud denis.arnaud_fed...@m4x.org 2009-06-15 15:50:44 EDT --- The following shows that both lapack-devel and blas-devel contain only symbolic links on libraries and static libraries, but no header files: $ rpm -ql lapack-devel /usr/lib64/liblapack.a /usr/lib64/liblapack.so /usr/lib64/liblapack_pic.a $ rpm -ql blas-devel /usr/lib64/libblas.a /usr/lib64/libblas.so So, if I am not mistaken, armadillo-devel does not need to list those development packages in the 'Requires:' statement. Of course, they are needed when building the Armadillo packages, but not any longer after that (they are not needed by someone wanting to develop with armadillo-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 492815] Review Request: fife - Cross platform game creation framework
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=492815 Simon Wesp cassmod...@fedoraproject.org changed: What|Removed |Added Resolution|WONTFIX |NOTABUG --- Comment #2 from Simon Wesp cassmod...@fedoraproject.org 2009-06-15 15:50:08 EDT --- i decided won't fix, because there are a lot of open questions and fixes which shall be done for this package, which I can't do. (perhabs cantfix???) for exmple: settings.xml must be writeable to adopt changes in settings.xml done by the settingsmenu in the client, because there will be non settings.xml in the users home. and many many more problems. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 496635] Review Request: monodevelop-debugger-mdb - Mono Debugger Addin for MonoDevelop
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=496635 --- Comment #47 from Mauricio Henriquez buhochil...@gmail.com 2009-06-15 15:41:09 EDT --- (In reply to comment #46) Excellent. Mauricio, please go to the account system and apply for the packager group: https://admin.fedoraproject.org/accounts/ mmm, kind of lost here, I have a fedora account but I supose to join a group?, is that is the case, what is the mono group name? I'll sponsor you in and then you can import and build your packages. great.. Paul has agreed to guide you in your work (since I don't do very much with mono packaging) so when you do your cvs request you can put him on the package as well. more great... Note that once I've sponsored you, you'll be on step #4 here: http://fedoraproject.org/wiki/Package_Review_Process don't know what to do with that :-S ...thanks Toshio... -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505928] Review Request: hdapsd - Protects hard drives by parking head when fall is detected
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505928 Steve Traylen st...@traylen.net changed: What|Removed |Added CC||st...@traylen.net --- Comment #1 from Steve Traylen st...@traylen.net 2009-06-15 15:58:49 EDT --- Hi Thomasz, I am not able to sponsor you but I can hopefully offer some comments. There are some rpmlint errors: $ rpmlint ../RPMS/x86_64/hdapsd-20090401-1.fc11.x86_64.rpm hdapsd.x86_64: W: non-conffile-in-etc /etc/event.d/hdapsd hdapsd.x86_64: W: non-conffile-in-etc /etc/udev/rules.d/99-hdapsd.rules 1 packages and 0 specfiles checked; 0 errors, 2 warnings. which to me look like they could and should be fixed. Maybe add a %doc README AUTHORS ChangeLog these files never do any harm to be included. Steve. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 504709] Review Request: rubygem-gettext_activerecord - Localization support for ActiveRecord by Ruby-GetText-Package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=504709 --- Comment #11 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-06-15 16:03:41 EDT --- (By the way I packaged rubygem-allison as bug 506168, not strictly needed for this package, though) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506168] New: Review Request: rubygem-allison - A modern, pretty RDoc template
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-allison - A modern, pretty RDoc template https://bugzilla.redhat.com/show_bug.cgi?id=506168 Summary: Review Request: rubygem-allison - A modern, pretty RDoc template Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mtas...@ioa.s.u-tokyo.ac.jp QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-allison/rubygem-allison.spec SRPM URL: http://mtasaka.fedorapeople.org/Review_request/rubygem-allison/rubygem-allison-2.0.3-2.fc.src.rpm Description: A modern, pretty RDoc template. Koji scratch build F-12: http://koji.fedoraproject.org/koji/taskinfo?taskID=1415422 F-11: http://koji.fedoraproject.org/koji/taskinfo?taskID=1415462 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505578] Review Request: hunspell-se - Northern Sami hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505578 Caolan McNamara caol...@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 on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 489686] Review Request: armadillo - fast C++ matrix library with interfaces to LAPACK and ATLAS
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=489686 --- Comment #66 from Jussi Lehtola jussi.leht...@iki.fi 2009-06-15 16:09:12 EDT --- Whoa, this review is unusually long. A lot of static, though. (In reply to comment #65) The following shows that both lapack-devel and blas-devel contain only symbolic links on libraries and static libraries, but no header files: clip So, if I am not mistaken, armadillo-devel does not need to list those development packages in the 'Requires:' statement. Of course, they are needed when building the Armadillo packages, but not any longer after that (they are not needed by someone wanting to develop with armadillo-devel). And they are not, as you could have seen in #64. The only packages required by -devel are boost-devel and atlas-devel, which both contain headers that are used by the armadillo headers. libstdc++-devel should probably be required too. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505760] Review Request: hyphen-eu - Basque hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505760 Caolan McNamara caol...@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 on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505845] Review Request: hyphen-cy - Welsh hyphenation patterns
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505845 Caolan McNamara caol...@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 on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505770] Review Request: hyphen-gl - Galician hyphenation rules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505770 Caolan McNamara caol...@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 on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506100] Review Request: opennhrp - An NHRP implementation for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506100 --- Comment #2 from Charles Lopes tja...@gmail.com 2009-06-15 16:43:00 EDT --- Thank you Steve. I have updated the spec file: Spec URL: http://sites.google.com/site/tjarls/fedora/opennhrp.spec SRPM URL: http://sites.google.com/site/tjarls/fedora/opennhrp-0.10.3-4.fc11.src.rpm I used rpmlint on the spec file only and forgot to run it on the rpm. That will teach me about rushing the package cleanup. I corrected the license tag as well. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506174] Review Request: qtdmm - a digital multimeter readout software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506174 --- Comment #1 from Andrew Zabolotny anp...@mail.ru 2009-06-15 16:53:55 EDT --- Sorry, I missed the URL to the spec file. The correct one is: https://fedorahosted.org/fedora-electronic-lab/attachment/ticket/23/qtdmm.6.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 285801] Review Request: simias - Collection-Oriented Data Storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=285801 --- Comment #22 from Christopher Brown snecklif...@gmail.com 2009-06-15 16:38:49 EDT --- (In reply to comment #21) I've just successfully compiled newest ifolder 3.7.2 (build 9169) from these SuSE rpms http://download.opensuse.org/repositories/Mono:/UIA:/iFolder/MonoOpenSUSE_11.0/ on F11 and started to use it. So far so good. If anyone intrested I can share it. The spec file would be good. Thanks to the fedora infrastructure team I was able to recover my old data so the original link now works again :) However it is outdated - I'll try compiling again and then updating shortly. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 285801] Review Request: simias - Collection-Oriented Data Storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=285801 --- Comment #23 from Vilius Šumskas vil...@lnk.lt 2009-06-15 16:57:35 EDT --- Created an attachment (id=348006) -- (https://bugzilla.redhat.com/attachment.cgi?id=348006) spec file for build 9169 Here you go. Also I had to apply libflaim patch for this ticket. Included gSOAP seem to compile fine. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506176] New: Review Request: hunspell-ko - Korean hunspell dictionaries
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: hunspell-ko - Korean hunspell dictionaries https://bugzilla.redhat.com/show_bug.cgi?id=506176 Summary: Review Request: hunspell-ko - Korean hunspell dictionaries Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: caol...@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://people.redhat.com/caolanm/rpms/hunspell-ko.spec SRPM URL: http://people.redhat.com/caolanm/rpms/hunspell-ko-0.2.4-1.fc11.noarch.rpm Description: Korean hunspell dictionaries -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506174] New: Review Request: qtdmm - a digital multimeter readout software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: qtdmm - a digital multimeter readout software https://bugzilla.redhat.com/show_bug.cgi?id=506174 Summary: Review Request: qtdmm - a digital multimeter readout software Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: anp...@mail.ru QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: https://fedorahosted.org/fedora-electronic-lab/attachment/ticket/23/qtdmm.5.spec SRPM URL: http://cs.ozerki.net/zap/pub/rpm/qtdmm-0.8.12-1.fc9.src.rpm Description: QtDMM is a DMM readout software including a configurable recorder. The recorder features manual start, scheduled start (at a given time), triggered automatic start when given thresholds are reached, and the ability to display more than one value from the multimeter. It was written for Metex (and compatible like VOLTCRAFT) multimeter which use an 14 byte protocol. Later several more protocols have been added. # rpmlint qtdmm-0.8.12-1.fc9.src.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. Respective Fedora Electronic Lab ticket here: https://fedorahosted.org/fedora-electronic-lab/ticket/23 This is my first package ever submitted to Fedora, so I need a sponsor. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505955] Review Request: perl-Module-Starter-Plugin-CGIApp - Template based module starter for CGI apps
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505955 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-06-15 17:26:17 EDT --- perl-Module-Starter-Plugin-CGIApp-0.10-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Module-Starter-Plugin-CGIApp-0.10-1.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 285801] Review Request: simias - Collection-Oriented Data Storage
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=285801 --- Comment #24 from Christopher Brown snecklif...@gmail.com 2009-06-15 17:29:36 EDT --- (In reply to comment #23) Created an attachment (id=348006) -- (https://bugzilla.redhat.com/attachment.cgi?id=348006) [details] spec file for build 9169 Here you go. Thanks! Also I had to apply libflaim patch for this ticket. Included gSOAP seem to compile fine. That's cool though we'll need to use system gsoap for the review. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506007] Review Request: perl-Sub-WrapPackages - Add wrappers around all the subroutines in packages
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506007 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-06-15 17:47:43 EDT --- Thanks for the review, Panag! Requesting cvs. New Package CVS Request === Package Name: perl-Sub-WrapPackages Short Description: Add wrappers around all the subroutines in packages Owners: eseyman Branches: F-11 F-10 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506006] Review Request: perl-Test-WWW-Mechanize-CGIApp - Test::WWW::Mechanize for CGI::Application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506006 Emmanuel Seyman emmanuel.sey...@club-internet.fr changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Emmanuel Seyman emmanuel.sey...@club-internet.fr 2009-06-15 17:46:09 EDT --- (In reply to comment #1) When I try to install this package, I saw we need to add Requires: perl(CGI::Application) Requires: perl(Test::WWW::Mechanize) Requires: perl(Test::More) How were you able to see this ? I can remove those modules from my machine and still install the rpm. Requesting cvs. New Package CVS Request === Package Name: perl-Test-WWW-Mechanize-CGIApp Short Description: Test::WWW::Mechanize for CGI::Application Owners: eseyman Branches: F-11 F-10 InitialCC: perl-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 505957] Review Request: perl-Syntax-Highlight-Perl-Improved - Highlighting of Perl Syntactical Structures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=505957 --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-06-15 17:21:06 EDT --- perl-Syntax-Highlight-Perl-Improved-1.01-1.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Syntax-Highlight-Perl-Improved-1.01-1.fc11 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506100] Review Request: opennhrp - An NHRP implementation for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=506100 --- Comment #3 from Steve Traylen st...@traylen.net 2009-06-15 17:23:17 EDT --- Evening, I'm trying to become a packager myself so apologies if I am learning. During the make stage there is one error. It is not obvious that it matters but looks worrying? + make fatal: Not a git repository (or any of the parent directories): .git the build is perfectly successful though. $ rpmlint ../RPMS/x86_64/opennhrp-0.10.3-4.fc11.x86_64.rpm 1 packages and 0 specfiles checked; 0 errors, 0 warnings. good. Hmm, why are there what look to be scripts as below in etc? /etc/opennhrp/opennhrp-script and /etc/opennhrp/racoon-ph1down.sh Now a more fundamental problem: You have ipsec-tools = 0.8 but f11 contains: # repoquery --location ipsec-tools ftp://ftp.free.fr/mirrors/fedora.redhat.com/fedora/linux/updates/11/x86_64//ipsec-tools-0.7.2-1.fc11.x86_64.rpm i.e it cannot be satisfied without a newer ipsec-tools? Steve -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506185] New: Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid) https://bugzilla.redhat.com/show_bug.cgi?id=506185 Summary: Review Request: bespin - A theme for Qt/KDE4 (includes KWin, Qt, and a plasmoid) Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: gr33nn1...@gr33nn1nj4.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.gr33nn1nj4.com/rpms/bespin.spec SRPM URL: http://www.gr33nn1nj4.com/rpms/bespin-0.1r483-1.fc12.src.rpm Description: Renamed baghira, a theme that was upstream but is being removed. The author has decided to keep it as a personal hobby. Bespin is inspired by various design ideas found in screenshots. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 506186] New: Review Request: perl-p5-Palm - Palm OS utility functions
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-p5-Palm - Palm OS utility functions https://bugzilla.redhat.com/show_bug.cgi?id=506186 Summary: Review Request: perl-p5-Palm - Palm OS utility functions Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: emmanuel.sey...@club-internet.fr QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://people.parinux.org/~seyman/fedora/perl-p5-Palm/perl-p5-Palm.spec SRPM URL: http://people.parinux.org/~seyman/fedora/perl-p5-Palm/perl-p5-Palm-1.009-1.fc11.src.rpm Description: This module provide functions and handlers to manipulate files used by Palm PDAs (AddressBook, ToDo, Memo, ...). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
