[Bug 484485] New: Review Request: perl-Fedora-App-ReviewTool - Application classes for reviewtool
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-Fedora-App-ReviewTool - Application classes for reviewtool https://bugzilla.redhat.com/show_bug.cgi?id=484485 Summary: Review Request: perl-Fedora-App-ReviewTool - Application classes for reviewtool Product: Fedora Version: rawhide Platform: All URL: http://camelus.fedorahosted.org OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cw...@alumni.drew.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Fedora-App-ReviewTool-0.04-1.fc10.src.rpm Description: Application classes for reviewtool. For the actual application, please install the 'reviewtool' package. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1110928 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484486] New: Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates
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-DateTime-Format-Excel - Convert between DateTime and Excel dates https://bugzilla.redhat.com/show_bug.cgi?id=484486 Summary: Review Request: perl-DateTime-Format-Excel - Convert between DateTime and Excel dates Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/DateTime-Format-Excel OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cw...@alumni.drew.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~cweyl/review/perl-DateTime-Format-Excel.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-DateTime-Format-Excel-0.2901-1.fc10.src.rpm Description: Excel uses a different system for its dates than most Unix programs. This module allows you to convert between a few of the Excel raw formats and DateTime objects, which can then be further converted via any of the other DateTime::Format::* modules, or just with DateTime's methods.If you happen to be dealing with dates between 1 Jan 1900 and 1 Mar 1900 please read the notes on EPOCHS. If you're wanting to handle actual spreadsheet files, you may find the Spreadsheet::WriteExcel manpage and the Spreadsheet::ParseExcel manpage of use. Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1110940 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484487] New: Review Request: perl-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript
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-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript https://bugzilla.redhat.com/show_bug.cgi?id=484487 Summary: Review Request: perl-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript Product: Fedora Version: rawhide Platform: All URL: http://search.cpan.org/dist/Template-Plugin-JavaScript OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: cw...@alumni.drew.edu QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fedorapeople.org/~cweyl/review/perl-Template-Plugin-JavaScript.spec SRPM URL: http://fedorapeople.org/~cweyl/review/perl-Template-Plugin-JavaScript-0.01-1.fc10.src.rpm Description: Template::Plugin::JavaScript is a TT filter that filters text so it can be safely used in JavaScript quotes. e.g: [% USE JavaScript %] document.write([% FILTER js %] Here's some text going on. [% END %]); will become: document.write(\nHere\'s some text going on.\n); Koji build: http://koji.fedoraproject.org/koji/taskinfo?taskID=1110951 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484483] Review Request: ldc - D Programming Language Compiler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484483 Jussi Lehtola jussi.leht...@iki.fi changed: What|Removed |Added CC||jussi.leht...@iki.fi --- Comment #1 from Jussi Lehtola jussi.leht...@iki.fi 2009-02-07 04:11:33 EDT --- Uhh, so you haven't made a RPM? You have to make one before someone can review it: http://fedoraproject.org/wiki/PackageMaintainers/CreatingPackageHowTo Also, you need to get sponsored before you can get the package into Fedora: http://fedoraproject.org/wiki/PackageMaintainers/Join This looks more like a wishlist entry to me. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481224] Review Request: rabbitmq-server - An AMQP server written in Erlang
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481224 Ruben Kerkhof ru...@rubenkerkhof.com changed: What|Removed |Added CC||ru...@rubenkerkhof.com --- Comment #6 from Ruben Kerkhof ru...@rubenkerkhof.com 2009-02-07 04:19:57 EDT --- The choice to start a service by default is left to the user, to do that he has to run 'chkconfig rabbitmq on' after installing the rpm. The rationale behind it is that after installation no unwanted services are started without the users knowledge, and him having the time to secure the ports with firewall rules etc. Since RabbitMQ listens on the network it shouldn't be started by default. Does that make sense? Ruben -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483205] Review Request: eclipse-systemtapgui - GUI interface for SystemTap
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483205 --- Comment #21 from Anithra anit...@linux.vnet.ibm.com 2009-02-07 05:54:38 EDT --- The above rpm , spec file contains all the other changes as well. Repeating the link: SPEC URL: http://nchc.dl.sourceforge.net/sourceforge/stapgui/eclipse-systemtapgui.spec SRPM URL: http://downloads.sourceforge.net/stapgui/eclipse-systemtapgui-1.0-4.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483205] Review Request: eclipse-systemtapgui - GUI interface for SystemTap
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483205 --- Comment #20 from Anithra anit...@linux.vnet.ibm.com 2009-02-07 05:52:08 EDT --- (In reply to comment #19) Anithra: on the website there is a screenshot showing graphing. Is that functionality included here? Does it use Draw2D, GEF, BIRT, or something else? graphing functionality is included, it uses eclipse-swt, not sure if that needs to be included in BuildRequires , added it anyway. SPEC URL: http://nchc.dl.sourceforge.net/sourceforge/stapgui/eclipse-systemtapgui.spec SRPM URL: http://downloads.sourceforge.net/stapgui/eclipse-systemtapgui-1.0-4.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481141] Review Request: pycryptsetup - Python bindings for cryptsetup 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=481141 --- Comment #9 from Dan Horák d...@danny.cz 2009-02-07 08:14:16 EDT --- Martin, you should close this bug as CLOSED/NEXTRELEASE after importing and building the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484483] Review Request: ldc - D Programming Language Compiler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484483 manuel wolfshant wo...@nobugconsulting.ro changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG --- Comment #2 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-07 08:29:10 EDT --- This is not a package review request but some twisted form of entry for a wishlist. Feel free to reopen if/when you can provide packages for review. Otherwise please pay a visit to https://fedoraproject.org/wiki/PackageMaintainers/WishList -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484487] Review Request: perl-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=484487
manuel wolfshant wo...@nobugconsulting.ro changed:
What|Removed |Added
Status|NEW |ASSIGNED
AssignedTo|nob...@fedoraproject.org|wo...@nobugconsulting.ro
Flag||fedora-review+
--- Comment #1 from manuel wolfshant wo...@nobugconsulting.ro 2009-02-07
08:48:42 EDT ---
Package Review
==
Key:
- = N/A
x = Check
! = Problem
? = Not evaluated
=== REQUIRED ITEMS ===
[x] Package is named according to the Package Naming Guidelines.
[x] Spec file name must match the base package %{name}, in the format
%{name}.spec.
[x] Package meets the Packaging Guidelines including the Perl specific items
[x] Package successfully compiles and builds into binary rpms on at least one
supported architecture.
Tested on: devel/x86_64
[x] Rpmlint output:
source RPM: empty
binary RPM:empty
[x] Package is not relocatable.
[x] Buildroot is correct
(%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n))
[x] Package is licensed with an open-source compatible license and meets other
legal requirements as defined in the legal section of Packaging Guidelines.
[x] License field in the package spec file matches the actual license.
License type: GPL+ or Artistic (same as perl)
[x] If (and only if) the source package includes the text of the license(s) in
its own file, then that file, containing the text of the license(s) for the
package is included in %doc.
[x] Spec file is legible and written in American English.
[x] Sources used to build the package matches the upstream source, as provided
in the spec URL.
SHA1SUM of package: 97bd729f5deaee2b043c1fe2aff4067dae7dbb8c
Template-Plugin-JavaScript-0.01.tar.gz
[x] Package is not known to require ExcludeArch
[x] All build dependencies are listed in BuildRequires, except for any that
are listed in the exceptions section of Packaging Guidelines.
[-] The spec file handles locales properly.
[-] ldconfig called in %post and %postun if required.
[x] Package must own all directories that it creates.
[x] Package requires other packages for directories it uses.
[x] Package does not contain duplicates in %files.
[x] Permissions on files are set properly.
[x] Package has a %clean section, which contains rm -rf $RPM_BUILD_ROOT.
[x] Package consistently uses macros.
[x] Package contains code, or permissable content.
[-] Large documentation files are in a -doc subpackage, if required.
[x] Package uses nothing in %doc for runtime.
[-] Header files in -devel subpackage, if present.
[-] Static libraries in -devel subpackage, if present.
[-] Package requires pkgconfig, if .pc files are present.
[-] Development .so files in -devel subpackage, if present.
[-] Fully versioned dependency in subpackages, if present.
[x] Package does not contain any libtool archives (.la).
[-] Package contains a properly installed %{name}.desktop file if it is a GUI
application.
[x] Package does not own files or directories owned by other packages.
=== SUGGESTED ITEMS ===
[x] Latest version is packaged.
[x] Package does not include license text files separate from upstream.
[-] Description and summary sections in the package spec file contains
translations for supported Non-English languages, if available.
[x] Reviewer should test that the package builds in mock.
Tested on: koji scratch build
[x] Package should compile and build into binary rpms on all supported
architectures.
Tested on: standard perl module, noarch
[?] Package functions as described.
[-] Scriptlets must be sane, if used.
[-] The placement of pkgconfig(.pc) files is correct.
[-] File based requires are sane.
[x] make check passes
*** APPROVED ***
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484483] Review Request: ldc - D Programming Language Compiler
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484483 --- Comment #3 from Orlando Arias orlandoar...@gmail.com 2009-02-07 08:53:06 EDT --- Ugh, how embarrassing, should have read the wiki on this. I didn't want to make a tisted form of entry for a wish list. I managed to build it, but not the required method. I do not have the time to become a contributor right now since I won't be able to keep up with code maintenance. Feel free to delete this, guys. Sorry. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483438] Review Request: libindi - Instrument Neutral Distributed Interface
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483438 Sergio Pascual sergio.pa...@gmail.com changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Sergio Pascual sergio.pa...@gmail.com 2009-02-07 08:56:43 EDT --- New Package CVS Request === Package Name: libindi Short Description: Instrument Neutral Distributed Interface Owners: sergiopr Branches: F-10 F-9 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 483116] Review Request: grnotify - Google Reader Notifier
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483116 --- Comment #3 from Guillaume Kulakowski llaum...@gmail.com 2009-02-07 09:59:18 EDT --- Version 1.1.2 SPEC: http://llaumgui.fedorapeople.org/review/grnotify/grnotify.spec SRPMS: http://llaumgui.fedorapeople.org/review/grnotify/grnotify-1.1.2-2.fc10.noarch.rpm RPM: http://llaumgui.fedorapeople.org/review/grnotify/grnotify-1.1.2-2.fc10.src.rpm Commment: rpmlint is silent -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483696] Review Request: libdmapsharing - library for DAAP and DPAP
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483696 --- Comment #1 from W. Michael Petullo m...@flyn.org 2009-02-07 10:04:30 EDT --- Spec URL: http://www.flyn.org/SRPMS/libdmapsharing.spec SRPM URL: http://www.flyn.org/SRPMS/libdmapsharing-1.9.0.1-2.fc10.src.rpm Description: libdmapsharing implements the DMAP protocols. This includes support for DAAP and DPAP. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484507] New: Review Request: php-channel-ezc
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
Summary: Review Request: php-channel-ezc
https://bugzilla.redhat.com/show_bug.cgi?id=484507
Summary: Review Request: php-channel-ezc
Product: Fedora
Version: 10
Platform: noarch
OS/Version: Linux
Status: NEW
Severity: medium
Priority: low
Component: Package Review
AssignedTo: nob...@fedoraproject.org
ReportedBy: llaum...@gmail.com
QAContact: extras...@fedoraproject.org
CC: nott...@redhat.com, fedora-package-review@redhat.com
Estimated Hours: 0.0
Classification: Fedora
This package adds the eZ Components channel which allows PEAR packages
from this channel to be installed.
SPEC:
http://llaumgui.fedorapeople.org/review/ez_components/php-channel-ezc.spec
SRPM:
http://llaumgui.fedorapeople.org/review/ez_components/php-channel-ezc-1-1.fc10.src.rpm
RPM:
http://llaumgui.fedorapeople.org/review/ez_components/php-channel-ezc-1-1.fc10.noarch.rpm
rpmlint:
buil...@enterprise ~ rpmlint php-*
php-channel-ezc.noarch: W: no-documentation
php-channel-ezc.src:21: W: unversioned-explicit-provides php-channel(ezc)
php-channel-ezc.src:22: W: unversioned-explicit-provides
php-channel(%{channel})
php-channel-ezc.spec:21: W: unversioned-explicit-provides php-channel(ezc)
php-channel-ezc.spec:22: W: unversioned-explicit-provides
php-channel(%{channel})
2 packages and 1 specfiles checked; 0 errors, 5 warnings.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484507] Review Request: php-channel-ezc - eZ Components
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484507 Guillaume Kulakowski llaum...@gmail.com changed: What|Removed |Added Summary|Review Request: |Review Request: |php-channel-ezc |php-channel-ezc - eZ ||Components -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484507] Review Request: php-channel-ezc
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484507 Guillaume Kulakowski llaum...@gmail.com changed: What|Removed |Added Blocks||484509 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484509] New: Review Request: php-ezc-Base - eZ Components Base
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: php-ezc-Base - eZ Components Base https://bugzilla.redhat.com/show_bug.cgi?id=484509 Summary: Review Request: php-ezc-Base - eZ Components Base Product: Fedora Version: 10 Platform: noarch OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: llaum...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Depends on: 484507 Estimated Hours: 0.0 Classification: Fedora Base is a part of eZ Components : https://bugzilla.redhat.com/show_bug.cgi?id=484507 The Base package provides the basic infrastructure that all packages rely on. Therefore every component relies on this package. SPEC: http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-Base.spec SRPM: http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-Base-1.6-1.fc10.src.rpm RPM: http://llaumgui.fedorapeople.org/review/ez_components/php-ezc-Base-1.6-1.fc10.noarch.rpm rpmlint: buil...@enterprise ~ rpmlint php-ezc-Base.* 2 packages and 1 specfiles checked; 0 errors, 0 warnings. Pear CompatInfo: +-+-+---+++ | Files | Version | C | Extensions | Constants/Tokens | +-+-+---+++ | usr/share/pear/ezc/*| 5.0.2 | 1 | pcre | ...CTORY_SEPARATOR | | | | | spl| __FILE__ | | | | || abstract | | | | || catch | | | | || final | | | | || implements | | | | || interface | | | | || private| | | | || protected | | | | || public | | | | || throw | | | | || try| +-+-+---+++ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474430] Review Request: python-uulib - Python interface to RFC 4122 compliant UUID objects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474430 --- Comment #10 from Ray Van Dolson ra...@bludgeon.org 2009-02-07 11:43:00 EDT --- Thanks Jason! RPM adds this implicit requires based on the BuildRequires I'm assuming? Makes sense and I'll remove the explicit Requires from the final version of the package. Thanks for your work on this. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484511] New: Review Request: crunchyfrog - A multi-engine SQL client and database front-end
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: crunchyfrog - A multi-engine SQL client and database front-end https://bugzilla.redhat.com/show_bug.cgi?id=484511 Summary: Review Request: crunchyfrog - A multi-engine SQL client and database front-end Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ivazquez...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://ivazquez.fedorapeople.org/packages/crunchyfrog/crunchyfrog.spec SRPM URL: http://ivazquez.fedorapeople.org/packages/crunchyfrog/crunchyfrog-0.3.3-1.src.rpm Description: CrunchyFrog is a SQL client and database front-end. It's mainly written for GNOME but it should run in other environments too. Supported databases: * PostgreSQL (requires psycopg2) * MySQL (requires MySQLdb) * SQLite (requires python-sqlite3) * Oracle (requires cx_Oracle) * LDAP (requires python-ldap) * MSSQL (requires pymssql) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474430] Review Request: python-uulib - Python interface to RFC 4122 compliant UUID objects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474430 Ray Van Dolson ra...@bludgeon.org changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Ray Van Dolson ra...@bludgeon.org 2009-02-07 11:53:00 EDT --- For posterity's sake, here are the final spec and SRPM: http://rayvd.fedorapeople.org/python-uuid/python-uuid.spec http://rayvd.fedorapeople.org/python-uuid/python-uuid-1.30-3.el5.src.rpm New Package CVS Request === Package Name: python-uuid Short Description: Python interface to RFC 4122 compliant UUID objects Owners: rayvd Branches: EL-5 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 480887] Review Request: kguitar - Guitar Tabulature Music Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480887 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp Bug 480887 depends on bug 481071, which changed state. Bug 481071 Summary: Review Request: tex-musixtex - Sophisticated music typesetting https://bugzilla.redhat.com/show_bug.cgi?id=481071 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #4 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-02-07 13:23:10 EDT --- First of all: Would you explain why the license term in licenseMusicXML.html are related to this application? I fear that the clause - This Agreement is governed by the laws of the State of California and the intellectual property laws of the United States of America. -- renders this license non-free. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480887] Review Request: kguitar - Guitar Tabulature Music Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480887 --- Comment #5 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2009-02-07 13:28:12 EDT --- (In reply to comment #4) Would you explain why the license term in licenseMusicXML.html are related to this application? s|why|how| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484487] Review Request: perl-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484487 --- Comment #2 from Chris Weyl cw...@alumni.drew.edu 2009-02-07 14:41:05 EDT --- New Package CVS Request === Package Name: perl-Template-Plugin-JavaScript Short Description: Encodes text to be safe in JavaScript Owners: cweyl Branches: F-9 F-10 devel 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 484487] Review Request: perl-Template-Plugin-JavaScript - Encodes text to be safe in JavaScript
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484487 Chris Weyl cw...@alumni.drew.edu changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480254] Review Request: deletemail - A non-interactive tool for deleting mails
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480254 Fabian Affolter fab...@bernewireless.net changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from Fabian Affolter fab...@bernewireless.net 2009-02-07 16:35:27 EDT --- New Package CVS Request === Package Name: deletemail Short Description: A non-interactive tool for deleting mails Owners: fab Branches: F-9 F-10 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480859] Review Request: diffuse - graphical diff 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=480859
Michael Schwendt bugs.mich...@gmx.net changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||bugs.mich...@gmx.net
AssignedTo|nob...@fedoraproject.org|bugs.mich...@gmx.net
Flag||fedora-review?
--- Comment #9 from Michael Schwendt bugs.mich...@gmx.net 2009-02-07
16:43:13 EDT ---
you're missing somethint in changelog
No. The first %changelog entry quoted in comment 4 is fine.
macronify more.
No. The request in comment 5 leads to macro-madness. Only in some cases it is
convenient to replace the program name with %{name} -- e.g. if you want to
reuse a spec file for other packages. It doesn't happen often that a program
changes its name frequently, so that using %name everywhere (even in URL and
%files lists) would be justified.
* The guidelines want the .desktop file to be validated in the %install
section:
https://fedoraproject.org/wiki/Packaging/Guidelines#desktop-file-install_usage
* The guidelines include specific shell code for scrollkeeper-update and
desktop-data-base usage:
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#Scrollkeeper
https://fedoraproject.org/wiki/Packaging/ScriptletSnippets#desktop-database
* Several directories are not included in %files:
%dir %{_datadir}/gnome/help/diffuse
%dir %{_datadir}/gnome/help/diffuse/C
%dir %{_datadir}/omf/diffuse
https://fedoraproject.org/wiki/Packaging/UnownedDirectories
* Rest is good.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479230] Review Request: pidgin-privacy-please - Stop spam bots for Pidgin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479230 --- Comment #18 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:20:10 EDT --- pidgin-privacy-please-0.5.2-2.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 474768] Review Request: jpilot-backup - An enhanced backup plugin for J-Pilot
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=474768 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:19:14 EDT --- jpilot-backup-0.53-5.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/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=459535 --- Comment #47 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:19:53 EDT --- backup-manager-0.7.7-7.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467416] Review Request: mingw32-cairo - MinGW Windows Cairo library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467416 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Resolution|RAWHIDE |NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479230] Review Request: pidgin-privacy-please - Stop spam bots for Pidgin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479230 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/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=459535 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467416] Review Request: mingw32-cairo - MinGW Windows Cairo library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467416 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:19:30 EDT --- mingw32-cairo-1.8.0-6.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483728] Review Request: kde-plasma-ihatethecashew - Gets rid of the cashew on KDE Workspace
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483728 --- Comment #32 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:21:45 EDT --- kde-plasma-ihatethecashew-0.2c-2.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update kde-plasma-ihatethecashew'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1468 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483728] Review Request: kde-plasma-ihatethecashew - Gets rid of the cashew on KDE Workspace
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483728 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #31 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:19:21 EDT --- kde-plasma-ihatethecashew-0.2c-2.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update kde-plasma-ihatethecashew'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1444 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479723] Review Request: cgit - A fast webinterface for git
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479723 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #9 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:19:38 EDT --- cgit-0.8.2-1.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update cgit'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1447 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent Client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=484229
Michael Schwendt bugs.mich...@gmx.net changed:
What|Removed |Added
CC||bugs.mich...@gmx.net
--- Comment #12 from Michael Schwendt bugs.mich...@gmx.net 2009-02-07
17:16:48 EDT ---
* Source0 points to a web page. The following Source0 location actually points
to a wget/curl-downloadable tarball:
http://downloads.sf.net/qbittorrent/qbittorrent-1.3.0.tar.gz
./configure --prefix=$RPM_BUILD_ROOT%{_prefix}
This --prefix is wrong. That's too much risk that the buildroot path finds its
way into compiled/built files. Use --prefix=%{_prefix} only.
gmake %{?_smp_mflags}
gmake is a softlink to make, why not just use make from package make
then? ;)
%doc README INSTALL NEWS COPYING AUTHORS TODO Changelog
File INSTALL is irrelevant to the package users.
* Run rpmlint on your src.rpm and built rpms and apply fixes as appropriate:
$ rpmlint qbittorrent-1.3.0-4.fc10.src.rpm
qbittorrent.src:27: W: configure-without-libdir-spec
qbittorrent.src:29: W: rpm-buildroot-usage %build ./configure
--prefix=$RPM_BUILD_ROOT%{_prefix}
qbittorrent.src:29: W: configure-without-libdir-spec
qbittorrent.src: E: description-line-too-long A Bittorrent client using C++ /
rb_libtorrent and a Qt4 Graphical User Interface.
1 packages and 0 specfiles checked; 1 errors, 3 warnings.
* The Warning about missing --libdir usage can be ignored.
* Testbuild will need Rawhide (F11 devel):
Checking for libtorrent-rasterbar = 0.14 ... no
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480483] Review Request: gaupol - Subtitle editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480483 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:23:43 EDT --- gaupol-0.13.1-1.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480483] Review Request: gaupol - Subtitle editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480483 --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:22:10 EDT --- gaupol-0.13.1-1.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479723] Review Request: cgit - A fast webinterface for git
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479723 --- Comment #10 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:22:34 EDT --- cgit-0.8.2-1.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update cgit'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1473 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 468465] Review Request: posterazor - Make your own poster
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=468465 --- Comment #7 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:25:37 EDT --- posterazor-1.5-2.fc9 has been pushed to the Fedora 9 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing-newkey update posterazor'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F9/FEDORA-2009-1506 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 459535] Review Request: backup-manager - A command line backup tool for GNU/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=459535 --- Comment #48 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:24:40 EDT --- backup-manager-0.7.7-7.fc9 has been pushed to the Fedora 9 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 479230] Review Request: pidgin-privacy-please - Stop spam bots for Pidgin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=479230 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:23:01 EDT --- pidgin-privacy-please-0.5.2-2.fc10 has been pushed to the Fedora 10 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 468465] Review Request: posterazor - Make your own poster
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=468465 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #6 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:23:10 EDT --- posterazor-1.5-2.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update posterazor'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1479 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483159] Review Request: sugar-xoirc - IRC client for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483159 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #4 from Fedora Update System upda...@fedoraproject.org 2009-02-07 17:24:51 EDT --- sugar-xoirc-0-0.1.20090129.fc10 has been pushed to the Fedora 10 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update sugar-xoirc'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/F10/FEDORA-2009-1497 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481516] Review Request: sugar-stopwatch - Simple stopwatch for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481516 --- Comment #2 from Fabian Affolter fab...@bernewireless.net 2009-02-07 18:19:29 EDT --- Thanks for the review. Here are the updated files Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-stopwatch.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-stopwatch-0-0.2.20090126.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481521] Review Request: sugar-clock - Clock activity for Sugar
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481521 --- Comment #4 from Fabian Affolter fab...@bernewireless.net 2009-02-07 18:18:49 EDT --- (In reply to comment #2) Please look at https://fedoraproject.org/wiki/Packaging:NamingGuidelines#Pre-Release_packages for how to use version and release tags on pre-released software. For example Version: 1 Release: 0.1.20090126git Thanks for the review. I left the 'git' away because I think that it doesn't matter which kind of VCS upstream is using. Here are the updated files Spec URL: http://fab.fedorapeople.org/packages/XO/sugar-clock.spec SRPM URL: http://fab.fedorapeople.org/packages/XO/sugar-clock-0-0.2.20090207.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent Client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=484229
--- Comment #13 from leigh scott leigh123li...@googlemail.com 2009-02-07
19:59:05 EDT ---
(In reply to comment #12)
* Source0 points to a web page. The following Source0 location actually points
to a wget/curl-downloadable tarball:
http://downloads.sf.net/qbittorrent/qbittorrent-1.3.0.tar.gz
./configure --prefix=$RPM_BUILD_ROOT%{_prefix}
This --prefix is wrong. That's too much risk that the buildroot path finds its
way into compiled/built files. Use --prefix=%{_prefix} only.
gmake %{?_smp_mflags}
gmake is a softlink to make, why not just use make from package make
then? ;)
%doc README INSTALL NEWS COPYING AUTHORS TODO Changelog
File INSTALL is irrelevant to the package users.
* Run rpmlint on your src.rpm and built rpms and apply fixes as appropriate:
$ rpmlint qbittorrent-1.3.0-4.fc10.src.rpm
qbittorrent.src:27: W: configure-without-libdir-spec
qbittorrent.src:29: W: rpm-buildroot-usage %build ./configure
--prefix=$RPM_BUILD_ROOT%{_prefix}
qbittorrent.src:29: W: configure-without-libdir-spec
qbittorrent.src: E: description-line-too-long A Bittorrent client using C++ /
rb_libtorrent and a Qt4 Graphical User Interface.
1 packages and 0 specfiles checked; 1 errors, 3 warnings.
* The Warning about missing --libdir usage can be ignored.
* Testbuild will need Rawhide (F11 devel):
Checking for libtorrent-rasterbar = 0.14 ... no
This isn't possible !
./configure --prefix=$RPM_BUILD_ROOT%{_prefix}
This --prefix is wrong. That's too much risk that the buildroot path finds its
way into compiled/built files. Use --prefix=%{_prefix} only.
I have read the packaging guidelines and found nothing that expressly prohibits
this , so fixing this error isn't mandatory , is this correct ?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent Client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=484229
--- Comment #14 from leigh scott leigh123li...@googlemail.com 2009-02-07
20:01:19 EDT ---
(In reply to comment #13)
(In reply to comment #12)
* Source0 points to a web page. The following Source0 location actually
points
to a wget/curl-downloadable tarball:
http://downloads.sf.net/qbittorrent/qbittorrent-1.3.0.tar.gz
./configure --prefix=$RPM_BUILD_ROOT%{_prefix}
This --prefix is wrong. That's too much risk that the buildroot path finds
its
way into compiled/built files. Use --prefix=%{_prefix} only.
gmake %{?_smp_mflags}
gmake is a softlink to make, why not just use make from package make
then? ;)
%doc README INSTALL NEWS COPYING AUTHORS TODO Changelog
File INSTALL is irrelevant to the package users.
* Run rpmlint on your src.rpm and built rpms and apply fixes as
appropriate:
$ rpmlint qbittorrent-1.3.0-4.fc10.src.rpm
qbittorrent.src:27: W: configure-without-libdir-spec
qbittorrent.src:29: W: rpm-buildroot-usage %build ./configure
--prefix=$RPM_BUILD_ROOT%{_prefix}
qbittorrent.src:29: W: configure-without-libdir-spec
qbittorrent.src: E: description-line-too-long A Bittorrent client using C++
/
rb_libtorrent and a Qt4 Graphical User Interface.
1 packages and 0 specfiles checked; 1 errors, 3 warnings.
* The Warning about missing --libdir usage can be ignored.
* Testbuild will need Rawhide (F11 devel):
Checking for libtorrent-rasterbar = 0.14 ... no
This isn't possible !
./configure --prefix=$RPM_BUILD_ROOT%{_prefix}
This --prefix is wrong. That's too much risk that the buildroot path finds
its
way into compiled/built files. Use --prefix=%{_prefix} only.
I have read the packaging guidelines and found nothing that expressly
prohibits
this , so fixing this error isn't mandatory , is this correct ?
I have also found this.
http://www.mail-archive.com/fedora-package-review@redhat.com/msg96285.html
Fix if possible.
nss.src: E: no-cleaning-of-buildroot %install
You should clean $RPM_BUILD_ROOT in the %clean section and just after the
beginning of %install section. Use rm -Rf $RPM_BUILD_ROOT
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent Client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=484229
--- Comment #15 from leigh scott leigh123li...@googlemail.com 2009-02-07
20:09:57 EDT ---
Here are the new spec and srpm
Spec URL:
http://dnmouse.org/fedora/qbittorrent_review/review_changes/new_2/qbittorrent.spec
SRPM URL:
http://dnmouse.org/fedora/qbittorrent_review/review_changes/new_2/qbittorrent-1.3.1-5.fc10.src.rpm
rpmlint -vi qbittorrent-1.3.1-5.fc10.x86_64.rpm qbittorrent.x86_64: I: checking
1 packages and 0 specfiles checked; 0 errors, 0 warnings.
rpmlint -vi qbittorrent-1.3.1-5.fc10.src.rpmqbittorrent.src: I: checking
qbittorrent.src:27: W: configure-without-libdir-spec
A configure script is run without specifying the libdir. configure options
must be augmented with something like --libdir=%{_libdir} whenever the script
supports it.
qbittorrent.src:30: W: rpm-buildroot-usage %build ./configure
--prefix=$RPM_BUILD_ROOT%{_prefix}
$RPM_BUILD_ROOT should not be touched during %build or %prep stage, as it will
break short circuiting.
qbittorrent.src:30: W: configure-without-libdir-spec
A configure script is run without specifying the libdir. configure options
must be augmented with something like --libdir=%{_libdir} whenever the script
supports it.
1 packages and 0 specfiles checked; 0 errors, 3 warnings.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484229] Review Request: qbittorrent - A bittorrent Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=484229 --- Comment #16 from leigh scott leigh123li...@googlemail.com 2009-02-07 20:12:39 EDT --- I have also found this. http://www.mail-archive.com/fedora-package-review@redhat.com/msg96285.html Fix if possible. nss.src: E: no-cleaning-of-buildroot %install You should clean $RPM_BUILD_ROOT in the %clean section and just after the beginning of %install section. Use rm -Rf $RPM_BUILD_ROOT Disregard this, I miss read 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 480887] Review Request: kguitar - Guitar Tabulature Music Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480887 --- Comment #6 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-07 20:17:03 EDT --- (In reply to comment #4) First of all: Would you explain why the license term in licenseMusicXML.html are related to this application? I fear that the clause - This Agreement is governed by the laws of the State of California and the intellectual property laws of the United States of America. -- renders this license non-free. I am not sure if we should include this license file in the package. As far as I understood this license applies to certain MusicXML document files. No such files are distributed by upstream tarball. The only relation between this license and kguitar is: kguitar is capable of handling MusicXML files. In Fedora, we have also other applications that can handle these MusicXML files. Tuxguitar and hydrogen are two examples that I can think of (they both do not distribute MusicXML document files.). Should I take this file off the package? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 480887] Review Request: kguitar - Guitar Tabulature Music Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=480887 --- Comment #7 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-07 20:19:06 EDT --- Just to clear confusion, in the above message MusicXML document files and MusicXML files mean the same. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 484535] New: Review Request: kde-plasma-networkmanagement - Plasmoid to control Network Manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: kde-plasma-networkmanagement - Plasmoid to control Network Manager https://bugzilla.redhat.com/show_bug.cgi?id=484535 Summary: Review Request: kde-plasma-networkmanagement - Plasmoid to control Network Manager Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: maths...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-review@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://benboeckel.net/packaging/kde-plasma-networkmanagement/kde-plasma-networkmanagement.spec SRPM URL: http://benboeckel.net/packaging/kde-plasma-networkmanagement/kde-plasma-networkmanagement-0.1-0.3.20090207svn.fc10.src.rpm Description: Plasmoid for KDE4 to control Network Manger. It still has some issues, but we (the KDE SIG) are looking to get it into Rawhide so it can get testing before the F11 beta and so we can evaluate it as a possible default for F11 if it matures in time, otherwise, we plan to at least offer it as an option. We also plan to push it out to F10 and possibly F9 once it is reliable enough for a non-default option. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 483301] Review Request: muse - Midi/Audio Music Sequencer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=483301 --- Comment #1 from Orcan 'oget' Ogetbil oget.fed...@gmail.com 2009-02-07 22:13:42 EDT --- New upstream release. I removed the patches that are no longer necessary. Spec URL: http://oget.fedorapeople.org/review/muse.spec SRPM URL: http://oget.fedorapeople.org/review/muse-1.0-0.1.rc1.fc10.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 462521] Review Request: simplyhtml - Application and a java component for rich text processing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=462521 --- Comment #4 from John Guthrie guth...@counterexample.org 2009-02-08 00:02:52 EDT --- (In reply to comment #3) (In reply to comment #2) [-] Package doesn't run I recommend creating a small shell script to run the program with the correct CLASSPATH -- if you just try to run the jar file, it doesn't find the gnu-regexp classes. Something like this: #!/bin/sh exec java -cp `build-classpath gnu-regexp javahelp2 simplyhtml` \ com.lightdev.app.shtm.App When and where would I want to do this? During compile-time perhaps? Never mind this comment. I was totally mis-parsing what you were trying to say. I figured it out. ;-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 467421] Review Request: mingw32-gtk-vnc - MinGW Windows port of VNC client GTK widget
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=467421 Michel Alexandre Salim michel.syl...@gmail.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #6 from Michel Alexandre Salim michel.syl...@gmail.com 2009-02-08 01:20:27 EDT --- All remaining issues have been fixed -- 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 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 Alex Lancaster al...@users.sourceforge.net changed: What|Removed |Added CC||al...@users.sourceforge.net Flag||needinfo?(fab...@bernewirel ||ess.net) --- Comment #8 from Alex Lancaster al...@users.sourceforge.net 2009-02-08 01:56:48 EDT --- There are serious problems with the Requires. Although vhybridize is found at build-time as you have: BuildRequires: vhybridize this is not sufficient as the autodeps searcher in RPM does not search the resulting package and automatically add the dependent packages (unlike for Perl) and all dependent Python packages must be added *explicitly* in the .spec file, otherwise it fails on install: $ pipviewer Traceback (most recent call last): File /usr/bin/pipviewer, line 5, in module from pkg_resources import load_entry_point File /usr/lib/python2.5/site-packages/pkg_resources.py, line 2562, in module working_set.require(__requires__) File /usr/lib/python2.5/site-packages/pkg_resources.py, line 626, in require needed = self.resolve(parse_requirements(requirements)) File /usr/lib/python2.5/site-packages/pkg_resources.py, line 524, in resolve raise DistributionNotFound(req) # XXX put more info here pkg_resources.DistributionNotFound: vhybridize=0.5.9 Ditto for PyQt pygtk2 PyOpenGL, and you need the runtime versions of these (add any minimum version Requires as well): Requires: PyQt, pygtk, PyOpenGL = 2.0, vhybridize = 0.5.9 Also it seems that PyOpenGL isn't sufficient to get this to work, as I have this installed $ rpm -q PyOpenGL PyOpenGL-3.0.0-0.9.b8.fc10.noarch but it still fails at runtime. $ pipviewer Traceback (most recent call last): File /usr/bin/pipviewer, line 5, in module from pkg_resources import load_entry_point File /usr/lib/python2.5/site-packages/pkg_resources.py, line 2562, in module working_set.require(__requires__) File /usr/lib/python2.5/site-packages/pkg_resources.py, line 626, in require needed = self.resolve(parse_requirements(requirements)) File /usr/lib/python2.5/site-packages/pkg_resources.py, line 524, in resolve raise DistributionNotFound(req) # XXX put more info here pkg_resources.DistributionNotFound: OpenGL=2.0 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 --- Comment #9 from Alex Lancaster al...@users.sourceforge.net 2009-02-08 02:06:32 EDT --- The last OpenGL problem appears to be a problem with the information in the egg file: /usr/lib/python2.5/site-packages/pipviewer-0.3.9-py2.5.egg-info/requires.txt it contains OpenGL=2.0 vhybridize=0.5.9 but the package is called PyOpenGL not OpenGL, if you change OpenGL - PyOpenGL it starts. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 --- Comment #11 from Alex Lancaster al...@users.sourceforge.net 2009-02-08 02:11:22 EDT --- Also the summary has multiple typos: Pipviewer is a visualizer for multiple alignements ^^^ alignments of genomic sequences. It highlights conserved regions and allows basic anotations. ^^^ annotations Its main goal is to find conserved probes for the construction of gene order data sets. Selected regions marked as 'probes' can be expoxted to fasta format. It can also retreive ^^^ retrieve gene annotations form ^^^ from the NBCI and display this information along the alignement. ^^^ alignment -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 481508] Review Request: pipviewer - Visualizer for multiple alignments of genomic sequences
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=481508 --- Comment #10 from Alex Lancaster al...@users.sourceforge.net 2009-02-08 02:07:37 EDT --- Note to reviewer: you should ensure you test the package at run-time on at least one release before approving a package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 462521] Review Request: simplyhtml - Application and a java component for rich text processing
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=462521
--- Comment #5 from John Guthrie guth...@counterexample.org 2009-02-08
02:34:43 EDT ---
(In reply to comment #2)
I have posted a new spec file and SRPM. Here are the URLs:
Spec: http://www.guthrie.info/RPMS/f10/simplyhtml.spec
SRPM: http://www.guthrie.info/RPMS/f10/simplyhtml-0.12.5-2.fc10.src.rpm
[-] source files match upstream:
I followed the instructions in the spec file and ended up with
something with a different md5sum than the file in the SRPM. The
readme.txt and gpl.txt files do match though
You could just use the upstream .zip file directly, though. Just change
your %setup line to
%setup -q -c %{name}-%{version}
and it'll create the directory when it needs to.
For the 0.12.5 version, the upstream is now offering the source in .tar.gz
format. I am now using that unmodified.
[-] latest version is being packaged.
It looks like upstream has released 0.12.5
This is fixed. 0.12.5 is now packaged.
[-] package builds in mock.
error: %patch without corresponding Patch: tag
You should have %patch0, not %patch, on line 62
For the remainder of this review I made this change
This is fixed.
[-] rpmlint is silent.
One warning to deal with:
simplyhtml.src: W: mixed-use-of-spaces-and-tabs (spaces: line 3, tab: line
20)
The specfile mixes use of spaces and tabs for indentation, which is a
cosmetic
annoyance. Use either spaces or tabs for indentation, not both.
This is fixed.
[-] Package doesn't run
I recommend creating a small shell script to run the program with the
correct CLASSPATH -- if you just try to run the jar file, it doesn't
find the gnu-regexp classes.
Something like this:
#!/bin/sh
exec java -cp `build-classpath gnu-regexp javahelp2 simplyhtml` \
com.lightdev.app.shtm.App
I did this almost verbatim except that I added simplyhtml-help to the
build-classpath command.
[-] Other source files are included:
Please check the status and the necessity of using the files in
src/com/sun and de/calcom as they appear to come from other projects.
The short story is that from what I can tell, these files actually are imported
into the files for this project. I will talk more about this later when it is
not 2AM and I am needing sleep. ;-)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
