[Bug 433187] New: Review Request: boinc-client - A platform for distributed computing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433187 Summary: Review Request: boinc-client - A platform for distributed computing Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] This is my first package and I'm seeking a sponsor. Spec URL: http://www.fi.muni.cz/~xjakub/boinc/boinc-client.spec SRPM URL: http://www.fi.muni.cz/~xjakub/boinc/boinc-client-5.10.40-4.20080206svn.fc8.src.rpm Description: The Berkeley Open Infrastructure for Network Computing (BOINC) is a software platform for distributed computing: several initiatives of various scientific disciplines all compete for the idle time of desktop computers. BOINC will be part of the Fedora Astronomy spin. (http://fedoraproject.org/wiki/SIGs/Astronomy) The BOINC core client program is required to participate in any project that uses BOINC. A central server distributes work units and collects results via this client. When attaching a local machine to a project, it will also dynamically download the project's application program to be then wrapped by the BOINC core client. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433187] Review Request: boinc-client - A platform for distributed computing
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: boinc-client - A platform for distributed computing https://bugzilla.redhat.com/show_bug.cgi?id=433187 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||177841 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431216] Review Request: perl-Tk-ProgressBar-Mac - Mac ProgressBar for Perl::Tk
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Tk-ProgressBar-Mac - Mac ProgressBar for Perl::Tk https://bugzilla.redhat.com/show_bug.cgi?id=431216 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 04:47 EST --- see #431215 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431217] Review Request: perl-Tk-FileDialog - File dialog for Perl Tk
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Tk-FileDialog - File dialog for Perl Tk https://bugzilla.redhat.com/show_bug.cgi?id=431217 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NOTABUG --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 04:48 EST --- see #413215, but in the final state of webinject package development this was not required anymore -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431215] Review Request: perl-Tk-Stderr - capture and show stderr output for Perl::Tk
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: perl-Tk-Stderr - capture and show stderr output for Perl::Tk https://bugzilla.redhat.com/show_bug.cgi?id=431215 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||NOTABUG --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 04:46 EST --- I am sorry for the delay, but all 3 packages were a prerequisite for a package (webinject - web application monitor) I was working on for my now former employer. I really appreciate your reviews, but the life has changed and I must close the requests. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 05:00 EST --- Since bug 430978 is taking forever, I added libsoup-2.4 patch for evoltion-rss. I can now build it under rawhide. Another thing is I've dropped firefox-devel (or xulrunner-devel) until xulrunner is fixed. The new files are: SPEC: http://gnome.eu.org/evolution-rss.spec SRPM: http://gnome.eu.org/evolution-rss-0.0.7-6.fc8.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432905] Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer https://bugzilla.redhat.com/show_bug.cgi?id=432905 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 04:53 EST --- The release number must be updated : don't use 1.2.4.orig The http://repo.awardspace.com/contrib/blueproximity-1.2.4.orig-1.fc8.src was a 404 Please fix more bugs before request a new update : only 3 bugs were fixed (4 if the image name on the .desktop was fixed) For the local stuff look : http://fedoraproject.org/wiki/Packaging/Guidelines#head-8c605ebf8330f6d505f384e671986fa99a8f72ee -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432817] Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game https://bugzilla.redhat.com/show_bug.cgi?id=432817 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 05:29 EST --- Thanks for 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432817] Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game https://bugzilla.redhat.com/show_bug.cgi?id=432817 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 05:29 EST --- New Package CVS Request === Package Name: greyhounds Short Description: Greyhounds is a greyhounds racing and breeding game Owners: kurzawa Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431310] Review Request: color-filesystem - Color filesystem layout
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: color-filesystem - Color filesystem layout https://bugzilla.redhat.com/show_bug.cgi?id=431310 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 05:30 EST --- New Package CVS Request === Package Name: color-filesystem Short Description: Color filesystem layout Owners:kwizart Branches: F-8 InitialCC: empty Commits by cvsextras: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML
tidy project
https://bugzilla.redhat.com/show_bug.cgi?id=433110
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 05:31 EST ---
(In reply to comment #1)
BLOCKER: There is a mix of licenses for source package. You chosen wrong
license
for this package. Please, be careful when setting package license tags.
Fixed.
BLOCKER: Package does not match package-naming-guidelines. Your package is
named
python-tidy and upstream package uTidylib. I see, that you are using just one
part from this source, but I don't know, if we can mix that this way.
The name is correct (as I read this, make most sense too:
http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-8756a3bce652c376d7ba3908461b638784b6952d
BLOCKER: python-tidy.noarch: E: explicit-lib-dependency libtidy
Add version info for libtidy requirement.
Fixed.
Other comments:
I think a better group for this package is Development/Languages .
php-tidy is using that, ok.
Add version for python-devel. uTidylib requires python=2.3
fixed
This code is not required:
%{python_sitelib}/tidy
%if 0%{?fedora} = 9
%{python_sitelib}/%{oname}-%{version}-py%{pyver}.egg-info
You can use %{python_sitelib}/*
Both are valid, %if is safer when things changes, but ...
As I don't know, what to do with this license mix, I can't approve you package
now. If you can give me a link to more information.
You can ask author to remove path.py from this package and rename it's package
as you need. Otherwise you need to fix these problems.
See the fix, it's normal to have several licenses on a package.
spec: http://terjeros.fedorapeople.org/python-tidy/python-tidy.spec
srpm: http://terjeros.fedorapeople.org/python-tidy/python-tidy-0.2-2.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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 06:09 EST --- (In reply to comment #2) (In reply to comment #1) BLOCKER: There is a mix of licenses for source package. You chosen wrong license for this package. Please, be careful when setting package license tags. Fixed. Not fixed. Where do you see BSD in licenses? There is just one LICENSE file which contain MIT license. The name is correct (as I read this, make most sense too: http://fedoraproject.org/wiki/Packaging/NamingGuidelines#head-8756a3bce652c376d7ba3908461b638784b6952d OK, accepting this. Add version for python-devel. uTidylib requires python=2.3 fixed If you are using Requires: python =2.5, set same version for python-devel See the fix, it's normal to have several licenses on a package. On IRC guys told me, that public domain do not need to be mentioned here. Just fix the BSD to MIT. You can mention other licenses in comments of .spec file. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241487] Review Request: cduce - An XML-oriented functional language (ocaml)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cduce - An XML-oriented functional language (ocaml) https://bugzilla.redhat.com/show_bug.cgi?id=241487 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 06:23 EST --- (In reply to comment #13) Without the netcgi1 patch I can't get cduce to compile: Checking for package camlp4... ok Checking for package num... ok Checking for package pcre... ok Checking for package ulex... ok Checking for package cgi... ocamlfind: Package `cgi' not found failed ! Ok, we fixed it in the svn version. Now if cgi package is not found then cduce is build but dtd2cduce and and the web interface interpreter are not. Thanks for the bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 241487] Review Request: cduce - An XML-oriented functional language (ocaml)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cduce - An XML-oriented functional language (ocaml) https://bugzilla.redhat.com/show_bug.cgi?id=241487 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 06:31 EST --- (In reply to comment #14) Spec URL: http://annexia.org/tmp/ocaml/cduce.spec In cduce.spec the following lines mv AUTHORS AUTHORS.old iconv -f ISO-8859-1 -t UTF-8 AUTHORS.old AUTHORS are no longer useful since now the AUTHORS file is in UTF-8 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430417] Review Request: PersonalCopy-Lite-soundfont - Lite version of the PersonalCopy General Midi soundfont
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: PersonalCopy-Lite-soundfont - Lite version of the PersonalCopy General Midi soundfont https://bugzilla.redhat.com/show_bug.cgi?id=430417 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 07:18 EST --- Jindrich, would you check the comment 12 by Hans? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 07:34 EST --- Not fixed. Where do you see BSD in licenses? There is just one LICENSE file which contain MIT license. Better now? If you are using Requires: python =2.5, set same version for python-devel Ok. On IRC guys told me, that public domain do not need to be mentioned here. Just fix the BSD to MIT. You can mention other licenses in comments of .spec file. Here we go: spec: http://terjeros.fedorapeople.org/python-tidy/python-tidy.spec srpm: http://terjeros.fedorapeople.org/python-tidy/python-tidy-0.2-3.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 427824] Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: tmispell-voikko - An Ispell compatible front-end for spell-checking modules https://bugzilla.redhat.com/show_bug.cgi?id=427824 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 07:51 EST --- Upstream released the first stable version of 0.7. New Spec: http://vpv.fedorapeople.org/packages/tmispell-voikko.spec New SRPM: http://vpv.fedorapeople.org/packages/tmispell-voikko-0.7-0.3.fc8.src.rpm There should only be minor changes compared to the previous RCs. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431188] Review Request: minbar - Athan call and prayer times notification software
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: minbar - Athan call and prayer times notification
software
https://bugzilla.redhat.com/show_bug.cgi?id=431188
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 09:08 EST ---
For 0.2.1-3:
* Scriptlets
- Scriptlets for GConf are not right. (also scriptlets for %pre
is needed). Please check the subsection GConf of
http://fedoraproject.org/wiki/Packaging/ScriptletSnippets
- Also the scriptlets for GTK icon cache are not right.
Please check the same wiki page.
* Timestamp
- To keep timestamps on installed files, I recommend to use
---
make install DESTDIR=$RPM_BUILD_ROOT INSTALL=install -p
---
This method usually works for recent autotool-based Makefiles.
* Desktop file
- Why is the category 'Utility' listed twice?
* Documents
- Usually the file INSTALL is for people who want to build and install
softwares by themselves and is not needed for people using rpm system.
- Files under %_mandir are automatically marked as %doc.
* macros
- Please use macros correctly. For example /usr/bin must be %{_bindir}.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431188] Review Request: minbar - Athan call and prayer times notification software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: minbar - Athan call and prayer times notification software https://bugzilla.redhat.com/show_bug.cgi?id=431188 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 09:39 EST --- Much better. :-) APPROVED. Do not require EL-* builds, because there are no python-2.5 for this OS. Please, be careful when setting license fields in future. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:06 EST --- Much better. :-) APPROVED. Thanks! New Package CVS Request === Package Name: python-tidy Short Description: Python wrapper for tidy, from the HTML tidy project Owners: terjeros Branches: F-7 F-8 InitialCC: Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432310] Review Request: compat-guichan06 - compatibility package for older guichan release
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: compat-guichan06 - compatibility package for older guichan release https://bugzilla.redhat.com/show_bug.cgi?id=432310 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:24 EST --- I've tried building ballz with guichan-0.7.1, and it turns out that all that is needed is a simple rebuild. I though that you had already tried that? Anyways that means that a compat package for guichan-0.6 isn't (really) needed and if you do one for binary compatibility reasons, you should remove all the files which normally go to -devel after make install, and then indeed cal;l it compat-foo, without a -devel subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225639] Merge Review: cdrdao
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: cdrdao https://bugzilla.redhat.com/show_bug.cgi?id=225639 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:48 EST --- Harald, ping ? I've checked in GCC 4.3 fixes upstream, or you can use the patch from gcdmaster F-9. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433199] Review Request: anjuta - A GNOME development IDE for C/C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anjuta - A GNOME development IDE for C/C++ https://bugzilla.redhat.com/show_bug.cgi?id=433199 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||432542 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:24 EST --- Due to the absense of autogen in PPC64 this package can not be scratch built in Koji. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433199] Review Request: anjuta - A GNOME development IDE for C/C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anjuta - A GNOME development IDE for C/C++ https://bugzilla.redhat.com/show_bug.cgi?id=433199 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:27 EST --- Anjuta installs some template *.c and *.h which causes some rpmlint warnings: $ rpmlint anjuta anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gtk/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/anjuta-plugin/src/plugin.h anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/anjuta-plugin/src/plugin.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/xlib-dock/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/xlib/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gtk/src/callbacks.h anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gtk/src/callbacks.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/terminal/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gnome/src/callbacks.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gnome/src/callbacks.h anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/xlib-dock/src/wmgeneral.h anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/xlib-dock/src/wmgeneral.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/gnome/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/mkfile/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/sdl/src/main.c anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/project/xlib-dock/src/pixmaps.h anjuta.x86_64: W: devel-file-in-non-devel-package /usr/share/anjuta/indent_test.c -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432542] Review Request: autogen - Automated text file generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: autogen - Automated text file generator https://bugzilla.redhat.com/show_bug.cgi?id=432542 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||433199 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433199] New: Review Request: anjuta - A GNOME development IDE for C/C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433199 Summary: Review Request: anjuta - A GNOME development IDE for C/C++ Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://rishi.fedorapeople.org/anjuta.spec SRPM URL: http://rishi.fedorapeople.org/anjuta-2.2.3-1.fc8.src.rpm Description: Anjuta DevStudio is a versatile Integrated Development Environment (IDE) on GNOME Desktop Environment and features a number of advanced programming facilities. These include project management, application and class wizards, an on-board interactive debugger, powerful source editor, syntax highlighting, intellisense autocompletions, symbol navigation, version controls, integrated GUI designing and other tools. The documentation for this package is in anjuta-doc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430978] Review Request: libsoup22 - Backward compatibility package for libsoup 2.2 API
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: libsoup22 - Backward compatibility package for libsoup 2.2 API https://bugzilla.redhat.com/show_bug.cgi?id=430978 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|430070 | nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin
https://bugzilla.redhat.com/show_bug.cgi?id=430070
[EMAIL PROTECTED] changed:
What|Removed |Added
BugsThisDependsOn|430978 |
AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED]
Status|NEW |ASSIGNED
Flag||fedora-review?
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 10:56 EST ---
* License issue
src/evolution-import-rss.c GPLv2 (strict)
- %_bindir/evolution-import-rss is GPLv2
The other parts are GPLv2+
Conclusion:
- Following
http://fedoraproject.org/wiki/Packaging/LicensingGuidelines
You should write as following in spec file:
..
Group: Applications/Internet
License: GPLv2 and GPLv2+
..
%files -f %{name}.lang
%defattr(-,root,root,-)
# Only the following binaries is under GPLv2. Other
# parts are under GPLv2+.
%{_bindir}/evolution-import-rss
%{_sysconfdir}/gconf/schemas/evolution-rss.schemas
..
For 0.0.7-6:
* configure option
- The following options are not needed for %configure.
--prefix=%{_prefix} --sysconfdir=%{_sysconfdir} --libdir=%{_libdir}
You can check what %configure actually does by
$ rpm --eval %configure .
* Timestamps
- To keep timestamps on installed files, I recommend to use
make install DESTDIR=%{buildroot} INSTALL=install -p
This method usually works for recent autotool-based Makefiles.
* ldconfig
- Calling /sbin/ldconfig is not needed for %preun.
* rpmlint
- You can use rpmlint (in rpmlint package) to detect some generic
mistakes in your rpms.
Currently:
evolution-rss.i386: W: non-conffile-in-etc
/etc/gconf/schemas/evolution-rss.schemas
evolution-rss.i386: W: no-version-in-last-changelog
- The first one can be ignored.
- For second one:
* Please also write the EVR in %changelog (please check
http://fedoraproject.org/wiki/Packaging/Guidelines#Changelogs )
* Also, I recommend to insert one blank line between each changelog
entry like:
%changelog
* Sat Feb 16 2008 Lucian Langa [EMAIL PROTECTED] - 0.0.7-6
- Drop gecko requirements till xulrunner is fixed
* Tue Feb 12 2008 Lucian Langa [EMAIL PROTECTED] - 0.0.7-5
- buildroot fixes
This is useful when using Fedora CVS system.
* Some misc cleanup
- For consistency, as you write
%pre
if [ $1 -gt 1 ]; then
export GCONF_CONFIG_SOURCE=`gconftool-2 --get-default-source`
gconftool-2 --makefile-uninstall-rule \
%{_sysconfdir}/gconf/schemas/%{name}.schemas /dev/null || :
^^^
fi
Please use %{name} in %files entry.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 216103] Review Request: python-twisted-names - A Twisted DNS implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-twisted-names - A Twisted DNS implementation https://bugzilla.redhat.com/show_bug.cgi?id=216103 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:16 EST --- Thomas: I'm interested in EL-5 branch. Would you maintain it yourself or you won't mind if I maintained it? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 207265] Review Request: python-zope-interface - Zope interfaces package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-zope-interface - Zope interfaces package https://bugzilla.redhat.com/show_bug.cgi?id=207265 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:19 EST --- Paul: I'm interested in EL-5 branch. Would you maintain it yourself or you won't mind if I maintained it? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433199] Review Request: anjuta - A GNOME development IDE for C/C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anjuta - A GNOME development IDE for C/C++ https://bugzilla.redhat.com/show_bug.cgi?id=433199 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn|432542 | -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 216105] Review Request: python-twisted-web - Twisted web server, programmable in Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-twisted-web - Twisted web server, programmable in Python https://bugzilla.redhat.com/show_bug.cgi?id=216105 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:15 EST --- Thomas: I'm interested in EL-5 branch. Would you maintain it yourself or you won't mind if I maintained it? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433199] Review Request: anjuta - A GNOME development IDE for C/C++
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anjuta - A GNOME development IDE for C/C++ https://bugzilla.redhat.com/show_bug.cgi?id=433199 [EMAIL PROTECTED] changed: What|Removed |Added BugsThisDependsOn||432542 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432542] Review Request: autogen - Automated text file generator
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: autogen - Automated text file generator
https://bugzilla.redhat.com/show_bug.cgi?id=432542
[EMAIL PROTECTED] changed:
What|Removed |Added
OtherBugsDependingO|433199 |
nThis||
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:11 EST ---
(In reply to comment #6)
Strangely the latest libopts tarball from upstream is versioned 27.6:
http://gnu.glug-nith.org/libopts/rel27.6/
At the same time, Debian ships a libopts package that bears the same EVRA as
autogen: http://packages.debian.org/experimental/libopts25 I took this option.
Remember due to the multiple licensing scenario we have to separate the
libopts
(or autoopts) portion from the rest of autogen.
What do you suggest?
This is *only my opinion*
I think that if you want to name the libopts related subrpm as libopts
the version should be 31.0.6 (as autoopts-config --version surely
returns this value)
texlive maintainers use this method (i.e. they allow different versions
for subpackage), however I really don't want this.
_IMO_ it is better that libopts related packages are named to show explicitly
that they are subpackages of autogen, i.e. they should be
autogen-libopts-devel, for example (as we actually did so in tetex age)
with the same EVR as autogen main package.
(In reply to comment #7)
It looks to me that the *.autogen suffix is not required because no other
package seems to provide binaries of the same name. If that is so, we can drop
the dependency on %{_sbindir}/alternatives.
If you don't know why alternatives is used here (note that I
don't know either) it should just be dropped.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 208169] Review Request: python-twisted-core - An asynchronous networking framework written in Python
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-twisted-core - An asynchronous networking framework written in Python https://bugzilla.redhat.com/show_bug.cgi?id=208169 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:16 EST --- Thomas: I'm interested in EL-5 branch. Would you maintain it yourself or you won't mind if I maintained it? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432542] Review Request: autogen - Automated text file generator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: autogen - Automated text file generator https://bugzilla.redhat.com/show_bug.cgi?id=432542 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO||433199 nThis|| -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 416461] Review Request: xmms-pulse - XMMS output plugin for the PulseAudio sound server.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xmms-pulse - XMMS output plugin for the PulseAudio sound server. https://bugzilla.redhat.com/show_bug.cgi?id=416461 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:43 EST --- What I mean by ping? is that I want you to update your spec file/srpm followed by comment 2. If you update your spec/srpm, sponsor members (including Jason and me) may want to review your spec/srpm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:53 EST --- (In reply to comment #9) updated spec and srpm to reflect changes. To where? (please change the release number every time you modify your spec file to avoid 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432033] Review Request: crystalspace - Crystal Space a free 3D engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: crystalspace - Crystal Space a free 3D engine https://bugzilla.redhat.com/show_bug.cgi?id=432033 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:55 EST --- (In reply to comment #9) For 1.2-3: * As I wrote some missing dependency for -devel subpackage as Example, it seems that there are some more missing dependencies for -devel package - cegui-devel - zlib-devel (I guess adding more 2 BR listed above should be enough) Sorry, I missunderstood. I did a full check and couldn't find any other headers besides the 2 above which I've added * It seems that adding INSTALL=install -p to make install actually works http://koji.fedoraproject.org/koji/taskinfo?taskID=426911 Strange as the makefile is justb a skeleton calling jam, but if it works thats good! Added. ! This weekend I will try to check license issues (if any) for this package (more than 1 files needs checking) Okay, about your analysis, I agree, except that the movierecorder plugin really is GPLv2 and not GPLv2+, as it includes (and uses) the nuppelvideo.h file which says: /* This file is from the NuppelVideo project: * * (c) Roman Hochleitner [EMAIL PROTECTED] * NuppelVideo is distributed under the GNU GENERAL PUBLIC LICENSE version 2 */ This makes the list: main : GPLv2+ and GPLv2 -utils: GPLv2+ and GPLv2 -demos: GPLv2+ -devel: GPLv2+ -doc: So to make things easier, esp the , I've just added a License tag of: GPLv2+ and GPLv2 to the main package, and let all the subpackages inherent this. I've added a large comment above the main License tag explaining why it is what it is. Also I will contact upstream about this, as I believe they intend the core of crystalspace to be LGPL not GPL. Here is a new (hopefully the last) version: Spec URL: http://people.atrpms.net/~hdegoede/crystalspace.spec SRPM URL: http://people.atrpms.net/~hdegoede/crystalspace-1.2-4.fc9.src.rpm -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432033] Review Request: crystalspace - Crystal Space a free 3D engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: crystalspace - Crystal Space a free 3D engine https://bugzilla.redhat.com/show_bug.cgi?id=432033 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:07 EST --- Thanks I will check this after rebuilding this on koji again. By the way I will appreciate it if you would review my review request bug 433182 (IMO very simple) (In reply to comment #12) Also I will contact upstream about this, as I believe they intend the core of crystalspace to be LGPL not GPL. I guess so. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432905] Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer https://bugzilla.redhat.com/show_bug.cgi?id=432905 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 11:55 EST --- All of the above have been corrected: (Suggestions by both Benoît and Guidolin) New links are: SRPM url: http://repo.awardspace.com/contrib/blueproximity-1.2.4-2.fc8.src.rpm SPEC url: http://repo.awardspace.com/contrib/blueproximity.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:10 EST --- SPEC: http://gnome.eu.org/evolution-rss.spec SRPM: http://gnome.eu.org/evolution-rss-0.0.7-7.fc8.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431310] Review Request: color-filesystem - Color filesystem layout
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: color-filesystem - Color filesystem layout https://bugzilla.redhat.com/show_bug.cgi?id=431310 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:28 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433165] Review Request: aldo - A morse code tutor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aldo - A morse code tutor https://bugzilla.redhat.com/show_bug.cgi?id=433165 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:34 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432817] Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: greyhounds - Greyhounds is a greyhounds racing and breeding game https://bugzilla.redhat.com/show_bug.cgi?id=432817 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:29 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:31 EST --- 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433012] Review Request: cloudy - Spectral synthesis code to simulate conditions in interstellar matter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: cloudy - Spectral synthesis code to simulate conditions in interstellar matter https://bugzilla.redhat.com/show_bug.cgi?id=433012 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:32 EST --- This fails to build for me in rawhide. The build log is about 3000 lines long and there's a bunch of stuff there that looks like error output, such as this: Generating docs for file /builddir/build/BUILD/c07.02.01/source/hydr/builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `8' /builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `1' /builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `H' /builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `i' /builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `o' /builddir/build/BUILD/c07.02.01/source/hydro_vs_rates.h:66: Error: Unexpected character `n' but at the end I think what kills the build is: Error: dot: can't open lines__service_8h_2a4d8a850b7f0515271aa4e92c71bd9f_cgraph.dot Problems running dot: exit code=1, command='dot', arguments='lines__service_8h_2a4d8a850b7f0515271aa4e92c71bd9f_cgraph.dot -Tpng -o lines__service_8h_2a4d8a850b7f0515271aa4e92c71bd9f_cgraph.png' Error: dot: can't open lines__service_8h_07a0a39c77d5a3b6cd9758bd0dd51378_cgraph.dot Problems running dot: exit code=1, command='dot', arguments='lines__service_8h_07a0a39c77d5a3b6cd9758bd0dd51378_cgraph.dot -Tpng -o lines__service_8h_07a0a39c77d5a3b6cd9758bd0dd51378_cgraph.png' Error: dot: can't open lines__service_8h_aa6a111a511c4f07dc11b62233ae2177_cgraph.dot Problems running dot: exit code=1, command='dot', arguments='lines__service_8h_aa6a111a511c4f07dc11b62233ae2177_cgraph.dot -Tpng -o lines__service_8h_aa6a111a511c4f07dc11b62233ae2177_cgraph.png' Could not open file /builddir/build/BUILD/c07.02.01/doxygen/html/magnetic_8cpp.html for writing s_service.h... I thought it might have something to with too much parallelism on my eight-CPU builder, but I turned off parallel make and the error is unchanged. (The random-looking portions of the strings do not change.) The /builddir/build/BUILD/c07.02.01/doxygen/html directory does exist in the chroot, and I have no problems creating a file with that name, so I'm at a bit of a loss. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432971] Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library https://bugzilla.redhat.com/show_bug.cgi?id=432971 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:36 EST --- http://hg.lewk.org/TurboFlot/raw-file/836ad97edcc8/python-turboflot.spec http://pypi.python.org/packages/2.5/T/TurboFlot/python-turboflot-0.0.5-2.fc8.src.rpm * Sat Feb 16 2008 Luke Macken [EMAIL PROTECTED] - 0.0.5-2 - Add TurboGears to BuildRequires and Requires - Require python-simplejson - Fix description - Fix Source0 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:50 EST --- (In reply to comment #8) # Only the following binaries is under GPLv2. Other Actually the following binary is under GPLv2, sorry... Well, - This package itself is okay - As written in http://fedoraproject.org/wiki/PackageMaintainers/HowToGetSponsored A person who wants to get sponsord is requested to either submit another review request or do a pre-review of other person's review request. For you it seems that your another review request (bug 431683) will be approved with a little more fixes. --- This package (evolution-rss) is APPROVED by me --- Please follow the procedure written on: http://fedoraproject.org/wiki/PackageMaintainers/Join from Get a Fedora Account. At a point a mail should be sent to sponsor members which notifies that you need a sponsor. At the stage, please also write on this bug for confirmation that you requested for sponsorship and your FAS (Fedora Account System) name. Then I will sponsor you. If you want to import this package into Fedora 7/8, you also have to look at http://fedoraproject.org/wiki/Infrastructure/UpdatesSystem/Bodhi-info-DRAFT (after once you rebuilt this package on koji Fedora rebuilding system). If you have questions, please ask 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 431683] Review Request: rspam - Report as Spam Evolution Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: rspam - Report as Spam Evolution Plugin https://bugzilla.redhat.com/show_bug.cgi?id=431683 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:51 EST --- Removing NEEDSPONSOR (bug 430070) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control (+GUI for it)
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: anyRemote - Bluetooth remote control (+GUI for it)
https://bugzilla.redhat.com/show_bug.cgi?id=433135
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:00 EST ---
First of all, please submit submit one review request for one package.
i.e. for {k,g}anyremote, please submit other review requests and
set bug dependency properly.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:15 EST --- Thank you. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432905] Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: blueproximity - A tool that locks/unlocks your screen
when your bluetooth devices gets away/near from your computer
https://bugzilla.redhat.com/show_bug.cgi?id=432905
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 12:56 EST ---
In the .desktop the field Icon should be Icon=blueproximity_base
In %file section please use : %{_datadir}/applications/fedora-%{name}
Source0:
http://downloads.sourceforge.net/blueproximity/%{name}-%{version}.tar.gz
Is more easier to use wen the package will be update
pygtk2-libglade require pygtk2 which require pygobject2 so keep over pygobject2
and pygtk2
Please fix all rpmlint error and fixable warning :
- E: description-line-too-long
blueproximity.src: E: invalid-spec-name blueproximity-1.spec
blueproximity.src:69: W: macro-in-%changelog setup
blueproximity.src:73: W: macro-in-%changelog version
blueproximity.src:76: W: macro-in-%changelog name
(to fix that use %%setup, %%{name} ... into the changelog)
blueproximity.src: W: no-%build-section
(add an empty %build section)
blueproximity.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 5)
And please update the date into the changelog (unless you can curve time)
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 [EMAIL PROTECTED] changed: What|Removed |Added OtherBugsDependingO|177841 | nThis|| --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:05 EST --- Now I should be sponsoring you. Please follow Join wiki again. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 207265] Review Request: python-zope-interface - Zope interfaces package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-zope-interface - Zope interfaces package https://bugzilla.redhat.com/show_bug.cgi?id=207265 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:27 EST --- Lubomir: I'm OK to maintain this myself but I'll happily add you as co-maintainer for both Fedora and EPEL if you want. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432971] Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: python-turboflot - A TurboGears widget for Flot, a
jQuery plotting library
https://bugzilla.redhat.com/show_bug.cgi?id=432971
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:39 EST ---
Source URL still do not match upstream!
- MUST: The sources used to build the package must match the upstream source, as
provided in the spec URL. Reviewers should use md5sum for this task. If no
upstream URL can be specified for this package, please see the Source URL
Guidelines for how to deal with this.
- It is nicer to use %{version} in Source URL. It is not required, so you can
leave it as is is you wish.
If you fix source, I can approve this 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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:45 EST --- New Package CVS Request === Package Name: evolution-rss Short Description: This is an evolution plugin which enables evolution to read rss feeds. Owners: lucilaga Branches: F-7 F-8 InitialCC: lucilanga Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433220] New: Review Request: kAnyREmote - KDE frontend for anyremote
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433220 Summary: Review Request: kAnyREmote - KDE frontend for anyremote Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://downloads.sourceforge.net/anyremote/kanyremote-fedora.spec?use_mirror=osdn SRPM URL: http://downloads.sourceforge.net/anyremote/kanyremote-4.5-2.fc8.src.rpm?use_mirror=osdn Description: kAnyRemote package is KDE GUI frontend for anyRemote (http://anyremote.sourceforge.net/) - remote control on Linux through Bluetooth, InfraRed, Wi-Fi or TCP/IP connection. This is additional review request for 433135 and 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433110] Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-tidy - Python wrapper for tidy, from the HTML tidy project https://bugzilla.redhat.com/show_bug.cgi?id=433110 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:48 EST --- Built and pushed for rawhide, f7 and f8. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433219] New: Review Request: ganyremote - GTK frontend for anyremote
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433219 Summary: Review Request: ganyremote - GTK frontend for anyremote Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://downloads.sourceforge.net/anyremote/ganyremote-fedora.spec?use_mirror=osdn SRPM URL: http://downloads.sourceforge.net/anyremote/ganyremote-2.6-2.fc8.src.rpm?use_mirror=osdn Description: gAnyRemote package is GTK GUI frontend for anyRemote (http://anyremote.sourceforge.net/) - remote control on Linux through Bluetooth, InfraRed, Wi-Fi or TCP/IP connection. This is additional review request for 433135 and 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:52 EST --- The item Short Description in CVS Request template is actually Summary in the spec file. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anyRemote - Bluetooth remote control https://bugzilla.redhat.com/show_bug.cgi?id=433135 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:49 EST --- 2 Mamoru Tasaka: Bug 433219: Review Request: ganyremote - GTK frontend for anyremote Bug 433220: Review Request: kAnyREmote - KDE frontend for anyremote -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anyRemote - Bluetooth remote control https://bugzilla.redhat.com/show_bug.cgi?id=433135 [EMAIL PROTECTED] changed: What|Removed |Added Summary|Review Request: anyRemote - |Review Request: anyRemote - |Bluetooth remote control|Bluetooth remote control |(+GUI for 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 207265] Review Request: python-zope-interface - Zope interfaces package
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-zope-interface - Zope interfaces package https://bugzilla.redhat.com/show_bug.cgi?id=207265 [EMAIL PROTECTED] changed: What|Removed |Added Flag||fedora-cvs? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:26 EST --- Package Change Request == Package Name: python-zope-interface New Branches: EL-4 EL-5 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anyRemote - Bluetooth remote control https://bugzilla.redhat.com/show_bug.cgi?id=433135 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:55 EST --- OK -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: anyRemote - Bluetooth remote control https://bugzilla.redhat.com/show_bug.cgi?id=433135 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:55 EST --- OK -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 430070] Review Request: evolution-rss - Evolution RSS Reader Plugin
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: evolution-rss - Evolution RSS Reader Plugin https://bugzilla.redhat.com/show_bug.cgi?id=430070 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 13:57 EST --- .. I also mistyped the owner, in order to change to I need to file a Package Change Request ? ..as cvs admin requests are processed per half an hour ? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433165] Review Request: aldo - A morse code tutor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aldo - A morse code tutor https://bugzilla.redhat.com/show_bug.cgi?id=433165 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:23 EST --- aldo-0.7.5-2.fc8 has been submitted as an update for Fedora 8 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433165] Review Request: aldo - A morse code tutor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aldo - A morse code tutor https://bugzilla.redhat.com/show_bug.cgi?id=433165 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:24 EST --- aldo-0.7.5-2.fc7 has been submitted as an update for Fedora 7 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432971] Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: python-turboflot - A TurboGears widget for Flot, a
jQuery plotting library
https://bugzilla.redhat.com/show_bug.cgi?id=432971
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:26 EST ---
http://hg.lewk.org/TurboFlot/raw-file/9fb8a0f1bb73/python-turboflot.spec
http://pypi.python.org/packages/any/T/TurboFlot/python-turboflot-0.0.5-2.fc8.src.rpm
I recreated the rpm with the upstream sources listed in Source0. I also now use
%{version} in the Source0, and have modified the URL to point to the Python
Package Index, instead of my blog.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433165] Review Request: aldo - A morse code tutor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: aldo - A morse code tutor https://bugzilla.redhat.com/show_bug.cgi?id=433165 [EMAIL PROTECTED] changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:26 EST --- Thanks John and Kevin. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432905] Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: blueproximity - A tool that locks/unlocks your screen when your bluetooth devices gets away/near from your computer https://bugzilla.redhat.com/show_bug.cgi?id=432905 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:38 EST --- Benoît: Fixed this in: Spec URL: http://repo.awardspace.com/contrib/blueproximity.spec SRPM URL: http://repo.awardspace.com/contrib/blueproximity-1.2.4-3.fc8.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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432741] Review Request: sarai-fonts - Free Sarai Hindi Truetype Font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: sarai-fonts - Free Sarai Hindi Truetype Font https://bugzilla.redhat.com/show_bug.cgi?id=432741 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:35 EST --- Review: + package builds in mock (development i386). + rpmlint is silent for SRPM and for RPM. + source files match upstream url b5f792b2fb76b7448c96f6a8f77b1390 Sarai_07.ttf + 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. + font is open source-compatible. + License text is included in package. + %clean is present. + package installed properly. + Macro use appears rather consistent. + Package contains 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. + fonts scriptlets present. + no duplicates in %files. + file permissions are appropriate. + Not a GUI App. 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 429221] Review Request: dzcomm - Dzcomm a RS-232 API/lib
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dzcomm - Dzcomm a RS-232 API/lib https://bugzilla.redhat.com/show_bug.cgi?id=429221 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:54 EST --- (In reply to comment #8) Still not * Actually this does not build. http://koji.fedoraproject.org/koji/taskinfo?taskID=422215 * Fedora specific compilation flags are yet not honored. Please recheck the section Compiler flags of http://fedoraproject.org/wiki/Packaging/Guidelines You can check what flags are used for Fedora now by `rpm --eval %optflags`. * The installation directory is perhaps wrong. Installing lib/unix/libdzcom.a to /var/tmp/dz0.9.9i-2.fc9-root-mockbuild/lib Shared libraries must be installed into %_libdir or /%_lib (in some special cases). On 64 bits this is /usr/lib64, not /usr/lib. Perhaps you want to use %configure macro (In reply to comment #4) * BuildRoot does not match Fedora guidelines. The package has its own root dir in the zipfile... (I don't know the procedure with this) What do you mean by this? Okay the CFLAGS should be honored now. I changed the BuildRoot to match what is suggested in the guidelines. I can not figure out why the %configure macro doesn't work. When I enable it the build fails right after (or during?) the find-debuginfo.sh... It seems the script is looking for the directory /var/tmp/$RPM_BUILD_ROOT which is not there. Any hints? Thanks, -Ian -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 432971] Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: python-turboflot - A TurboGears widget for Flot, a jQuery plotting library https://bugzilla.redhat.com/show_bug.cgi?id=432971 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 15:09 EST --- Much better. :-) 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433182] Review Request: pcmanfm - Extremly fast and lightweight file manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pcmanfm - Extremly fast and lightweight file manager https://bugzilla.redhat.com/show_bug.cgi?id=433182 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:23 EST --- Created an attachment (id=295109) -- (https://bugzilla.redhat.com/attachment.cgi?id=295109action=view) PATCH fixing 64 bit crash -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433182] Review Request: pcmanfm - Extremly fast and lightweight file manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pcmanfm - Extremly fast and lightweight file manager https://bugzilla.redhat.com/show_bug.cgi?id=433182 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:21 EST --- I've done a full review, result below. humor Unfortunately I cannot approve this package as every package should have atleast one patch. Luckily I found a 64 bit bug, causing the package to fail to work (crash) on 64 bit. So I've written a patch fixing this so that we can meet the every package should have atleast one patch rule. /humor MUST FIX * Apply the to be attached patch fixing crashing on 64 bit. The patch may look strange for a 64 bit patch, but it really fixes a 64 bit issue: -The header in question contains guards to protect it from multiple inclusion -These guards where copy pasted from another header without changing the define checked and set, therefor this header things its already included while its not -This causes various prototypes to not be declared, and the compiler to assume their return type is int (aka 32 bits) -However some return a pointer, which should be 64 bit and now gets truncated to 32 bits. -Boom (aka crash) Should Fix -- * Send the 64 bit patch upstream * Ask upstream to use freedesktop.org icon installation location instead of obsolete /usr/share/pixmaps Although there is a MUST FIX, I trust you to include the patch so I'm going to set the approved flag, please include the patch before building the first time. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433182] Review Request: pcmanfm - Extremly fast and lightweight file manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: pcmanfm - Extremly fast and lightweight file manager https://bugzilla.redhat.com/show_bug.cgi?id=433182 [EMAIL PROTECTED] changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 14:22 EST --- Like I said I'm going to set the approved flag, second 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, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] New: Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433228 Summary: Review Request: distcc - Distributed C/C++ compilation Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://www.poolshark.org/src/distcc.spec SRPM URL: http://www.poolshark.org/src/distcc-2.18.3-1.fc8.src.rpm Description: distcc is a program to distribute compilation of C or C++ code across several machines on a network. distcc should always generate the same results as a local compile, is simple to install and use, and is often two or more times faster than a local compile. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] New: Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433225 Summary: Review Request: dvipdfmx - A DVI to PDF translator Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://jgu.fedorapeople.org/dvipdfmx.spec SRPM URL: http://jgu.fedorapeople.org/dvipdfmx-20071115-1.fc9.src.rpm Description: The dvipdfmx (formerly dvipdfm-cjk) project provides an eXtended version of the dvipdfm, a DVI to PDF translator developed by Mark A. Wicks. Commentary: Another package currently part of texlive, for which texlive is not upstream, hence the desire to split it out as a separate package. dvipdfm seems to be unmaintained, and so dvipdfmx should probably be considered to be upstream for dvipdfm. I note that texlive currently ships both dvipdfm and dvipdfmx. Can we just ship dvipddfxm - does it have all the functionality of dvipdfm? I have included the cdi-x.map stuff from texlive spec file, but I'm not 100 percent sure this is still needed. Currently this package obsoletes and provides dvipdfm. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433223] New: Review Request: gstreamermm - C++ wrapper for GStreamer library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. https://bugzilla.redhat.com/show_bug.cgi?id=433223 Summary: Review Request: gstreamermm - C++ wrapper for GStreamer library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: [EMAIL PROTECTED] ReportedBy: [EMAIL PROTECTED] QAContact: [EMAIL PROTECTED] CC: fedora-package-review@redhat.com,[EMAIL PROTECTED] Spec URL: http://www.poolshark.org/src/gstreamermm-0.9.2-1.fc8.src.rpm SRPM URL: http://www.poolshark.org/src/gstreamermm.spec Description: GStreamermm is a C++ wrapper library for the multimedia library GStreamer. It is designed to allow C++ development of applications that work with multi-media. GStreamermm is part of the gtkmm project. Notes: - Tarball comes with empty COPYING file, so the SRPM provides its own of LGPLv2+. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433135] Review Request: anyRemote - Bluetooth remote control
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: anyRemote - Bluetooth remote control
https://bugzilla.redhat.com/show_bug.cgi?id=433135
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 15:47 EST ---
Are You serious ? Using of %{name} is clearly specified at
http://fedoraproject.org/wiki/Packaging/Guidelines
Right, but you don't need to write:
%define name anyremote
%define version 4.3
Name: %{name}
Version: %{version}
You can write directly:
Name: anyremote
Version: 4.3
and use %{name} in the rest of the spec file without using %define Name
and %define 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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: distcc - Distributed C/C++ compilation https://bugzilla.redhat.com/show_bug.cgi?id=433228 [EMAIL PROTECTED] changed: What|Removed |Added Status|CLOSED |ASSIGNED Keywords||Reopened Resolution|DUPLICATE | --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:04 EST --- The bug you're pointing to is a stalled 2+ year old review. Please leave this open and let review go through, then we'll let Fesco rule on the competing reviews. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: distcc - Distributed C/C++ compilation https://bugzilla.redhat.com/show_bug.cgi?id=433228 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] ||lesup.org --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:15 EST --- I have been involded in bug #174883, and I agree with Denis. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 225941] Merge Review: java_cup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Merge Review: java_cup https://bugzilla.redhat.com/show_bug.cgi?id=225941 [EMAIL PROTECTED] changed: What|Removed |Added Severity|normal |medium Priority|normal |medium Product|Fedora Extras |Fedora Version|devel |rawhide [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:17 EST --- I took the package maintainership. Anyone willing to do the review would be very welcome! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433129] Review Request: gkrellm-weather - Weather plugin for GKrellM
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: gkrellm-weather - Weather plugin for GKrellM https://bugzilla.redhat.com/show_bug.cgi?id=433129 [EMAIL PROTECTED] changed: What|Removed |Added AssignedTo|[EMAIL PROTECTED]|[EMAIL PROTECTED] Status|NEW |ASSIGNED Flag||fedora-review? --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 15:29 EST --- URL and Source0 no longer exist, new ones appear to be http://makovick.googlepages.com/gkrellmplugins + the tarball URL there. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: distcc - Distributed C/C++ compilation https://bugzilla.redhat.com/show_bug.cgi?id=433228 [EMAIL PROTECTED] changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 15:55 EST --- *** This bug has been marked as a duplicate of 174883 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 174883] Review Request: distcc -- A free distributed C/C++ compiler system
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: distcc -- A free distributed C/C++ compiler system https://bugzilla.redhat.com/show_bug.cgi?id=174883 [EMAIL PROTECTED] changed: What|Removed |Added CC||[EMAIL PROTECTED] --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 15:55 EST --- *** Bug 433228 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug, or are watching the QA contact. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: distcc - Distributed C/C++ compilation https://bugzilla.redhat.com/show_bug.cgi?id=433228 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:28 EST --- Hi Denis and Laurent, please pardon my knee-jerk reaction of marking this as a duplicate. I've also been following bug #174883 and was hoping that it would make better progress. But perhaps that's too much optimism... :-) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dvipdfmx - A DVI to PDF translator https://bugzilla.redhat.com/show_bug.cgi?id=433225 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:31 EST --- dvipdfm is not obsoleted by dvipdfmx in my opinion, and still maintained, last version is from 05.03.2007 which is pretty recent. Both packages can cohabit. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dvipdfmx - A DVI to PDF translator https://bugzilla.redhat.com/show_bug.cgi?id=433225 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:43 EST --- Yeah, there was a minor release in 2007 which incorporated two patches, but no major development. Before that, last release was 2001. I agree that both can co-exist, but what functionality is provided by dvipdfm that's not in dvipdfmx? If there's good reason, I'll knock up a dvipdfm package, but if there's no technical reason, I'd rather not bother. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dvipdfmx - A DVI to PDF translator https://bugzilla.redhat.com/show_bug.cgi?id=433225 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:45 EST --- But yes, given that both can co-exist, I'll remove the Obsoletes. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 416461] Review Request: xmms-pulse - XMMS output plugin for the PulseAudio sound server.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: xmms-pulse - XMMS output plugin for the PulseAudio sound server. https://bugzilla.redhat.com/show_bug.cgi?id=416461 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 16:47 EST --- Updated 2-14-08 see http://thenerdshow.com/rpm for directory listing of latest work in progress -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433228] Review Request: distcc - Distributed C/C++ compilation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: distcc - Distributed C/C++ compilation
https://bugzilla.redhat.com/show_bug.cgi?id=433228
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:16 EST ---
Here's a quick and not-yet-complete review:
- ok : follows naming guidelines
- ok : source matches upstream:
88e4c15826bdbc5a3de0f7c1bcb429e558c6976d distcc-2.18.3.tar.bz2
88e4c15826bdbc5a3de0f7c1bcb429e558c6976d distcc-2.18.3.tar.bz2.1
- ok : license (GPLv2+) appears to be correct
- ok : license appears in source and included in %doc
- ok : spec file looks sane/legible
- ok : built using mock -r fedora-8-x86_64 distcc-2.18.3-1.fc8.src.rpm
- rpmlint reports:
distcc.x86_64: W: file-not-utf8 /usr/share/doc/distcc-2.18.3/NEWS
distcc-server.x86_64: W: incoherent-subsys /etc/init.d/distccd $prog
distcc-server.x86_64: W: incoherent-subsys /etc/init.d/distccd $prog
distcc-server.x86_64: W: incoherent-subsys /etc/init.d/distccd $prog
distcc-server.x86_64: W: incoherent-init-script-name distccd
where the file-not-utf8 warning can be ignored and the four other
warnings appear to be due to the package name being distcc-server
(and not distcc or distccd) which means they are also harmless
- ok : no need for ldconfig and no *.la files
- ok : dir ownership looks good
- ok : no need for -devel
- ok : use of %clean and rm -rf $RPM_BUILD_ROOT at start of %install
Nits:
- perhaps use Source0:
http://distcc.samba.org/ftp/distcc/distcc-%{version}.tar.bz2
which seems to be the prefered link from the web site
Next, I'll install it on a few machines and give it a 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, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: dvipdfmx - A DVI to PDF translator https://bugzilla.redhat.com/show_bug.cgi?id=433225 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:06 EST --- (In reply to comment #2) Yeah, there was a minor release in 2007 which incorporated two patches, but no major development. Before that, last release was 2001. That's not necessarily bad, it may also be that dvipdfm is mature. I agree that both can co-exist, but what functionality is provided by dvipdfm that's not in dvipdfmx? If there's good reason, I'll knock up a dvipdfm package, but if there's no technical reason, I'd rather not bother. I don't know exactly what you mean by technical reason. In any case dvipdfm is already in texlive, so maybe it is not worth bothering packaging it separately if it is not updated more often than texlive. However I think that a dvipdfm package should stay, especially if it works well and isn't updated often. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ristretto - Image-viewer for the Xfce desktop environment https://bugzilla.redhat.com/show_bug.cgi?id=351531 [EMAIL PROTECTED] changed: What|Removed |Added Attachment #265271|0 |1 is obsolete|| --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:34 EST --- Created an attachment (id=295113) -- (https://bugzilla.redhat.com/attachment.cgi?id=295113action=view) Trace of ristretto 0.0.16 crashing when the main area is clicked -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ristretto - Image-viewer for the Xfce desktop environment https://bugzilla.redhat.com/show_bug.cgi?id=351531 [EMAIL PROTECTED] changed: What|Removed |Added Attachment #265541|0 |1 is obsolete|| --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:35 EST --- Created an attachment (id=295114) -- (https://bugzilla.redhat.com/attachment.cgi?id=295114action=view) Trace of ristretto 0.0.16 crashing when trying to zoom -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 351531] Review Request: ristretto - Image-viewer for the Xfce desktop environment
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: ristretto - Image-viewer for the Xfce desktop environment https://bugzilla.redhat.com/show_bug.cgi?id=351531 --- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:44 EST --- Stephan, I just read your announcement for 0.0.17 and updated my package. Still no joy. You want me to do more traces? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug, or are watching someone who is. ___ Fedora-package-review mailing list Fedora-package-review@redhat.com http://www.redhat.com/mailman/listinfo/fedora-package-review
[Bug 433225] Review Request: dvipdfmx - A DVI to PDF translator
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: dvipdfmx - A DVI to PDF translator
https://bugzilla.redhat.com/show_bug.cgi?id=433225
--- Additional Comments From [EMAIL PROTECTED] 2008-02-17 17:22 EST ---
rpmlint says:
dvipdfmx.src: W: mixed-use-of-spaces-and-tabs (spaces: line 1, tab: line 9)
dvipdfmx.i386: E: zero-length /usr/share/doc/dvipdfmx-20071115/NEWS
It seems to me that version should be 0 and release should be
0.x.20071115. However, dvipdfmx in texlive is already at release 16, so
it seems to me that it can be
17.x.20071115. Or even x.20071115 with x beginning at 17.
Why the texlive-texmf BuildRequires?
The files
%{_texmf_main}/fonts/cmap/EUC-UCS2
%{_texmf_main}/fonts/cmap/UniKSCms-UCS2-H
%{_texmf_main}/fonts/cmap/UniKSCms-UCS2-V
are already owned by texlive-texmf-fonts, which package should own them?
In the texlive spec, there is, for the dvipdfmx subpackage:
# for cmap files
Requires: texlive-texmf-fonts = %{texlive_ver}
I think that it would be better to list explicitly the files
in %_bindir, to avoid surprises.
I suggest adding INSTALL='install -p' to make install.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug, or are watching someone who is.
___
Fedora-package-review mailing list
Fedora-package-review@redhat.com
http://www.redhat.com/mailman/listinfo/fedora-package-review
