[Bug 744430] New: Review Request: libquvi-scripts - Embedded lua scripts that libquvi uses for parsing the media details
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: libquvi-scripts - Embedded lua scripts that libquvi uses for parsing the media details
https://bugzilla.redhat.com/show_bug.cgi?id=744430
Summary: Review Request: libquvi-scripts - Embedded lua scripts
that libquvi uses for parsing the media details
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: nicoleau.fabien(a)gmail.com
QAContact: extras-qa(a)fedoraproject.org
CC: notting(a)redhat.com,
package-review(a)lists.fedoraproject.org
Classification: Fedora
Story Points: ---
Type: ---
Spec URL: http://rpms.nicoleau-fabien.net/SPECS/libquvi-scripts.spec
SRPM URL:
http://rpms.nicoleau-fabien.net/SRPMS/libquvi-scripts-0.4.0-1.fc15.src.rpm
Description:
libquvi-scripts contains the embedded lua scripts that libquvi
uses for parsing the media details. Some additional utility
scripts are also included.
The quvi package (already in fedora) has been splitted : this package for the
scripts, an other one for the lib, and the third one for the quvi command.
rpmlint output :
[builder@FEDORABOX rpmbuild]$ rpmlint
/home/builder/rpmbuild/SRPMS/libquvi-scripts-0.4.0-1.fc15.src.rpm
/home/builder/rpmbuild/RPMS/x86_64/libquvi-scripts-0.4.0-1.fc15.x86_64.rpm
libquvi-scripts.src: W: spelling-error Summary(en_US) lua -> la, luau, lea
libquvi-scripts.src: W: spelling-error %description -l en_US lua -> la, luau,
lea
libquvi-scripts.x86_64: E: no-binary
libquvi-scripts.x86_64: W: devel-file-in-non-devel-package
/usr/lib64/pkgconfig/libquvi-scripts.pc
2 packages and 0 specfiles checked; 1 errors, 3 warnings.
This package build on koji :
http://koji.fedoraproject.org/koji/taskinfo?taskID=3414075
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 5 months
[Bug 750099] Review Request: airinv - C++ Simulated Airline Inventory Management System Library
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750099
Denis Arnaud <denis.arnaud_fedora(a)m4x.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Flag| |fedora-cvs?
--- Comment #4 from Denis Arnaud <denis.arnaud_fedora(a)m4x.org> 2011-12-04 13:30:57 EST ---
Thank you very much for that review, Thomas!
I have eventually found the bug hindering the generation of the PDF reference
manual. It has been fixed in the new upstream release (0.1.2).
-------------------------------------------------------------------------------
New Package SCM Request
=======================
Package Name: airinv
Short Description: C++ Simulated Airline Inventory Management System Library
Owners: denisarnaud
Branches: f15 f16 el5 el6
InitialCC: denisarnaud
-------------------------------------------------------------------------------
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 5 months
[Bug 522239] New: Review Request: qdex - Romanian offline dictionary
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: qdex - Romanian offline dictionary
https://bugzilla.redhat.com/show_bug.cgi?id=522239
Summary: Review Request: qdex - Romanian offline dictionary
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: low
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: mapleoin(a)fedoraproject.org
QAContact: extras-qa(a)fedoraproject.org
CC: notting(a)redhat.com, fedora-package-review(a)redhat.com
Estimated Hours: 0.0
Classification: Fedora
Spec URL: http://mapleoin.fedorapeople.org/pkgs/qdex/qdex.spec
SRPM URL: http://mapleoin.fedorapeople.org/pkgs/qdex/qdex-9.09-3.fc11.src.rpm
Description: qDex is an offline dictionary application that uses the database
of the Romanian word definitions created and maintained by Dexonline.ro
volunteers.
Hello!
I have a couple of problems with this package.
1. I've included only very few files from the package. Everything is packed
inside the executable. The Readme file details the installation process so I've
excluded it along with everything else (header files, html doc files which are
included in the binary, qdbm and zlib programs).
2. qdex gets all its definitions from http://dexonline.ro. By default it
downloads them to the current directory the first time it starts and any time
the user asks for it. I had patched it so that it would put the db file (some
20MB) in /var/lib/qdex, but then only root could update the db. So I removed
the patch and assumed that, since users will start qdex from the menu, the
database will always stay/get updates in the current user's home and not have
duplicates that way.
So this is the dillema: leave it the way it is now or patch the program to
remove the "manual update" button and make a separate -data package with the
database, that way the database only gets updated by fedora. However, dexonline
seems to be quite the active project, with more definitions added daily.
I don't know if I should be mentioning this anymore, but I'm not yet in the
packager group although I have found 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.
12 years, 5 months
[Bug 750099] Review Request: airinv - C++ Simulated Airline Inventory Management System Library
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=750099
Thomas Spura <tomspur(a)fedoraproject.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
AssignedTo|nobody(a)fedoraproject.org |tomspur(a)fedoraproject.org
Flag| |fedora-review+
--- Comment #3 from Thomas Spura <tomspur(a)fedoraproject.org> 2011-12-04 08:36:17 EST ---
Review:
- name ok
- descriptions ok
- BR ok
- doc has own COPYING and doesn't R the main package - ok
- library correct installed
- ldconfig called
- no static lib
- %files ok
- license ok
(headers are missing, but as you are part of upstream, you know you should
have some ;))
- check is there
- macros everywhere
- doc is noarch
- R pkgconfig ok as you are building for el*
- BR there
- defattr there
- sources match:
ccd19f9a48abe5d506c43a28544d96f5 airinv-0.1.1.tar.bz2
- koji build worked:
http://koji.fedoraproject.org/koji/taskinfo?taskID=3561152
- $ rpmlint /home/tom/rpmbuild/RPMS/*/airinv*
/home/tom/rpmbuild/SRPMS/airinv-0.1.1-1.fc16.src.rpm
5 packages and 0 specfiles checked; 0 errors, 0 warnings.
NOTICE:
- It looks strange, that _datadir/airinv is owned by the devel package and not
by the main package... Usually the main package would own such a directory and
put application data in it. But as only the devel package puts files there: ok
NEEDSWORK:
- This is in the build log:
Warning: the PDF reference manual
(/home/tom/rpmbuild/BUILD/airinv-0.1.1/doc/latex/refman.pdf) has failed to
build. You can perform a simple re-build (make in the doc/latex sub-directory).
and the refman.pdf is indeed missing. Please check why it fails and fix.
_____________________________________________________________________________
As there is only a minor doc fix needed, this package is:
_____________________________________________________________________________
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.
12 years, 5 months
[Bug 472144] New: Review Request: tvbrowser - Free EPG for over 500 stations.
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: tvbrowser - Free EPG for over 500 stations.
https://bugzilla.redhat.com/show_bug.cgi?id=472144
Summary: Review Request: tvbrowser - Free EPG for over 500
stations.
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: red(a)fedoraproject.org
QAContact: extras-qa(a)fedoraproject.org
CC: notting(a)redhat.com, fedora-package-review(a)redhat.com
Estimated Hours: 0.0
Classification: Fedora
Spec URL: http://red.fedorapeople.org/SRPMS/tvbrowser.spec
SRPM URL: http://red.fedorapeople.org/SRPMS/tvbrowser-2.7.1-0.3.fc10.i386.rpm
Description: TV-Browser is a free electronic program guide (EPG). It offers
more than 500 stations.
This is my first package (with a little starting aid from che on #fedora-de)
and I need a mentor.
When I build the rpm, I seem to have some small problems that I've not been
able to solve:
- find: `debug': No such file or directory
- some in-the-jars operation has a problem with a file within a jar that has
spaces in its filename:
+ /usr/lib/rpm/redhat/brp-java-repack-jars
cp: cannot stat `./bbc/rd/tvanytime/util/Copy': No such file or directory
cp: cannot stat `of': No such file or directory
cp: cannot stat `DVBLocatorToolbox': No such file or directory
- javadoc compilation fails in this version of tvbrowser. I commmented those
parts out for now and hope upstream will solve this.
Nevertheless, I can install and run the application just fine.
I was not sure, if it's ok to include out-of-the-box plugins/whatever in the
base package or if I need to create additional packages for those.
Comments and mentors welcome ;)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 5 months
[Bug 747008] New: Review Request: drupal7-advanced_help - Allows module developers to store their help outside the module system in html
by Red Hat Bugzilla
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: drupal7-advanced_help - Allows module developers to store their help outside the module system in html
https://bugzilla.redhat.com/show_bug.cgi?id=747008
Summary: Review Request: drupal7-advanced_help - Allows module
developers to store their help outside the module
system in html
Product: Fedora
Version: rawhide
Platform: All
OS/Version: Linux
Status: NEW
Severity: medium
Priority: medium
Component: Package Review
AssignedTo: nobody(a)fedoraproject.org
ReportedBy: peter.borsa(a)gmail.com
QAContact: extras-qa(a)fedoraproject.org
CC: notting(a)redhat.com,
package-review(a)lists.fedoraproject.org
Classification: Fedora
Story Points: ---
Type: ---
Spec URL: http://asrob.fedorapeople.org/SOURCES/drupal7-advanced_help.spec
SRPM URL:
http://asrob.fedorapeople.org/SOURCES/drupal7-advanced_help-1.0-0.1.beta1...
Description: The advanced help module allows module developers to store their
help outside the module system, in pure .html files. The files can be easily
translated simply by copying them into the right translations directory.
The entire system can appear in a pop-up or not as the module prefers.
rpmlint output:
$ rpmlint drupal7-advanced_help.spec
../SRPMS/drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm
../RPMS/noarch/drupal7-advanced_help-1.0-0.1.beta1.fc15.noarch.rpm
drupal7-advanced_help.src: W: spelling-error Summary(en_US) html -> HTML, ht
ml, ht-ml
drupal7-advanced_help.src: W: spelling-error %description -l en_US html ->
HTML, ht ml, ht-ml
drupal7-advanced_help.noarch: W: spelling-error Summary(en_US) html -> HTML, ht
ml, ht-ml
drupal7-advanced_help.noarch: W: spelling-error %description -l en_US html ->
HTML, ht ml, ht-ml
2 packages and 1 specfiles checked; 0 errors, 4 warnings.
koji output:
Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=3440261
Watching tasks (this may be safely interrupted)...
3440261 build (dist-f15, drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm):
free
3440261 build (dist-f15, drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm):
free -> open (x86-15.phx2.fedoraproject.org)
3440262 buildArch (drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm, noarch):
free
3440262 buildArch (drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm, noarch):
free -> open (x86-04.phx2.fedoraproject.org)
3440262 buildArch (drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm, noarch):
open (x86-04.phx2.fedoraproject.org) -> closed
0 free 1 open 1 done 0 failed
3440261 build (dist-f15, drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm):
open (x86-15.phx2.fedoraproject.org) -> closed
0 free 0 open 2 done 0 failed
3440261 build (dist-f15, drupal7-advanced_help-1.0-0.1.beta1.fc15.src.rpm)
completed successfully
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.
12 years, 5 months