[Bug 615192] Review Request: cjkuni-fonts-ghostscript -Chinese Unicode TrueType font ghostscript files
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615192 --- Comment #2 from Peng Wu 2010-07-23 02:24:00 EDT --- According to review comments, fixes license issue. Here are new rpm spec for review: Spec URL: http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript.spec SRPM URL: http://pwu.fedorapeople.org/cjkuni-fonts-ghostscript/cjkuni-fonts-ghostscript-0.3-1.fc13.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603233] Review Request: zeromq - Software library for fast, message-based applications
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=603233
Parag AN(पराग) changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #7 from Parag AN(पराग) 2010-07-23 02:20:43 EDT
---
Review:
+ package builds in mock (rawhide i686).
koji Build =>http://koji.fedoraproject.org/koji/taskinfo?taskID=2345975
+ rpmlint output for SRPM and for RPM.
zeromq-utils.x86_64: W: spelling-error %description -l en_US zmq -> Zama, zoom,
Zarqa
5 packages and 0 specfiles checked; 0 errors, 1 warnings.
+ source files match upstream url (sha1sum)
cfe061d06431eccbbc95df2d257d07447c7f4cce zeromq-2.0.7.tar.gz
cfe061d06431eccbbc95df2d257d07447c7f4cce zeromq-2.0.7.tar.gz.srpm
+ 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.
+ license is open source-compatible.
+ License text is included in package.
+ %doc is present.
+ BuildRequires are proper.
+ %clean is present which is not needed now.
+ package installed properly.
+ Macro use appears rather consistent.
+ Package contains code, not content.
+ libzmq.pc file present.
+ no -devel subpackage
+ no .la files.
+ no translations are available
+ Does owns the directories it creates.
+ ldconfig scriptlets present.
+ no duplicates in %files.
+ file permissions are appropriate.
+ Package zeromq-2.0.7-3.fc14.x86_64 =>
Provides: libzmq.so.0()(64bit)
Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.3)(64bit) libc.so.6(GLIBC_2.3.2)(64bit)
libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.4)(64bit)
libgcc_s.so.1()(64bit) libgcc_s.so.1(GCC_3.0)(64bit) libm.so.6()(64bit)
libpthread.so.0()(64bit) libpthread.so.0(GLIBC_2.2.5)(64bit)
libstdc++.so.6()(64bit) libstdc++.so.6(CXXABI_1.3)(64bit)
libstdc++.so.6(GLIBCXX_3.4)(64bit) libuuid.so.1()(64bit)
libuuid.so.1(UUID_1.0)(64bit) libzmq.so.0()(64bit) rtld(GNU_HASH)
+ Package zeromq-devel-2.0.7-3.fc14.x86_64 =>
Provides: pkgconfig(libzmq) = 2.0.7
Requires: /usr/bin/pkg-config libzmq.so.0()(64bit)
+ Package zeromq-utils-2.0.7-3.fc14.x86_64 =>
Requires: libc.so.6()(64bit) libc.so.6(GLIBC_2.2.5)(64bit)
libc.so.6(GLIBC_2.3.4)(64bit) libc.so.6(GLIBC_2.4)(64bit)
libgcc_s.so.1()(64bit) libgcc_s.so.1(GCC_3.0)(64bit) libm.so.6()(64bit)
libpthread.so.0()(64bit) libstdc++.so.6()(64bit)
libstdc++.so.6(CXXABI_1.3)(64bit) libstdc++.so.6(GLIBCXX_3.4)(64bit)
libuuid.so.1()(64bit) libzmq.so.0()(64bit) rtld(GNU_HASH)
+ Not a GUI application
Suggestions
1) If package is needed only for F-13 and above then please follow
a) buildroot should be removed
b) %clean not needed
c) cleaning of buildroot at start of %install also not needed
2) Please add some empty line between %package and %description sections for
-devel and -utils subpackages
3) %description for -devel packages are generally generic. See
/etc/rpmdevtools/spectemplate-lib.spec
So please replace existing %description text with following
The %{name}-devel package contains libraries and header files for
developing applications that use %{name}.
4) %description for -utils should be simple like
Utility files for libzmq package
or
This package contains ZeroMQ related utility files,
e.g. zmq_forwarder, zmq_streamer and zmq_queue.
5) For me working Source URL is
http://zeromq.wdfiles.com/local--files/area:download/zeromq-%{version}.tar.gz
Please fix these before you go for cvs import.
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603233] Review Request: zeromq - Software library for fast, message-based applications
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603233 --- Comment #8 from Parag AN(पराग) 2010-07-23 02:21:42 EDT --- and yes there exists -devel and -utils subpackages, missed to modify my review template above -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603641] Review Request: python-fontMath - A set of objects for performing math operations on font data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603641 Parag changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Parag 2010-07-23 01:05:53 EDT --- Thanks for the review! New Package CVS Request === Package Name: python-fontMath Short Description: A set of objects for performing math operations on font data Owners: pnemade Branches: F-13 EL-6 InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603639] Review Request: python-ufo2fdk - A bridge between UFOs and the AFKDO
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603639 Parag changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Parag 2010-07-23 01:05:44 EDT --- Thanks for the review! New Package CVS Request === Package Name: python-ufo2fdk Short Description: A bridge between UFOs and the AFKDO Owners: pnemade Branches: F-13 EL-6 InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603640] Review Request: python-compositor - A simple OpenType GSUB and GPOS engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603640 Parag changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Parag 2010-07-23 01:05:48 EDT --- Thanks for the review! New Package CVS Request === Package Name: python-compositor Short Description: A simple OpenType GSUB and GPOS engine Owners: pnemade Branches: F-13 EL-6 InitialCC: i18n-team fonts-sig -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603640] Review Request: python-compositor - A simple OpenType GSUB and GPOS engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603640 --- Comment #5 from Parag 2010-07-23 00:40:03 EDT --- Updated as Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/python-compositor.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/python-compositor-0.2b-2.fc13.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603639] Review Request: python-ufo2fdk - A bridge between UFOs and the AFKDO
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603639 --- Comment #3 from Parag 2010-07-23 00:39:45 EDT --- Updated as Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/python-ufo2fdk.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/python-ufo2fdk-0.1-2.fc13.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 521723] Review Request: perl-Makefile-Parser - Simple parser for Makefiles
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=521723 Kevin Fenzi changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #10 from Kevin Fenzi 2010-07-23 00:31:51 EDT --- cvs done. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615695] Review Request: libaccounts-glib - Nokia Maemo Accounts base library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615695 --- Comment #2 from Chen Lei 2010-07-23 00:07:44 EDT --- New SRPM and SPEC: http://dl.dropbox.com/u/1338197/1/libaccounts-glib-0.39-2.fc13.src.rpm http://dl.dropbox.com/u/1338197/1/libaccounts-glib.spec Notes: I add a permanent link for meego SRPM, getting snapshots for dozens of packages from git repo is not easy for both packagers and reviewers. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561073] Review Request: gist - Absolutely the best command line gister
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=561073 --- Comment #8 from Michael Stahnke 2010-07-22 22:44:51 EDT --- Any change of a 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 547601] Review Request: pfacter - A perl port of facter
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=547601 --- Comment #6 from Michael Stahnke 2010-07-22 22:44:02 EDT --- Can anybody review the rest of this? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 537325] Review Request: lv2-fil-plugins - Four-band parametric equalizers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=537325 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #26 from Fedora Update System 2010-07-22 22:39:20 EDT --- lv2-fil-plugins-2.0-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update lv2-fil-plugins'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lv2-fil-plugins-2.0-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616289] Review Request: perl-MooseX-Clone - Fine grained cloning support for Moose objects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616289 --- Comment #9 from Fedora Update System 2010-07-22 22:38:49 EDT --- perl-MooseX-Clone-0.05-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-MooseX-Clone'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-MooseX-Clone-0.05-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 537325] Review Request: lv2-fil-plugins - Four-band parametric equalizers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=537325 --- Comment #27 from Fedora Update System 2010-07-22 22:39:25 EDT --- lv2-fil-plugins-2.0-3.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update lv2-fil-plugins'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/lv2-fil-plugins-2.0-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616289] Review Request: perl-MooseX-Clone - Fine grained cloning support for Moose objects
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616289 Fedora Update System changed: What|Removed |Added Status|MODIFIED|ON_QA --- Comment #8 from Fedora Update System 2010-07-22 22:32:52 EDT --- perl-MooseX-Clone-0.05-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-MooseX-Clone'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-MooseX-Clone-0.05-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613646] Review Request: twlog - A simple day to day logging program for ham radio
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613646 Fedora Update System changed: What|Removed |Added Fixed In Version|twlog-2.7-2.fc12|twlog-2.7-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613646] Review Request: twlog - A simple day to day logging program for ham radio
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613646 --- Comment #17 from Fedora Update System 2010-07-22 22:32:39 EDT --- twlog-2.7-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 563822] Review Request: globus-gram-job-manager-setup-sge - Globus Toolkit - Sun Grid Engine Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=563822 --- Comment #11 from Fedora Update System 2010-07-22 22:25:45 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc13, globus-gram-job-manager-setup-condor-4.4-1.fc13, globus-gram-job-manager-setup-pbs-4.1-1.fc13, globus-gram-job-manager-setup-lsf-2.5-1.fc13, globus-core-5.17-1.fc13, globus-common-11.5-1.fc13, globus-gsi-proxy-ssl-2.3-1.fc13, globus-openssl-module-1.3-1.fc13, globus-gsi-cert-utils-6.6-1.fc13, globus-gsi-callback-2.7-1.fc13, globus-gsi-credential-3.5-1.fc13, globus-gsi-proxy-core-4.5-1.fc13, globus-gss-assist-5.9-2.fc13, globus-proxy-utils-3.9-1.fc13, globus-ftp-client-5.3-2.fc13, globus-gass-copy-5.7-1.fc13, globus-rsl-assist-2.2-1.fc13, globus-rls-client-5.2-2.fc13, globus-rls-server-4.9-2.fc13, globus-gridftp-server-control-0.43-1.fc13, globus-gridftp-server-3.23-1.fc13, globus-gram-protocol-9.7-3.fc13, globus-gram-client-10.4-1.fc13, globus-gram-client-tools-8.2-1.fc13, globus-gram-job-manager-scripts-2.11-1.fc13, globus-gram-job-manager-10.59-2.fc13, globus-gram-job-manager-setup-sge-2.5-1.fc13, globus-openssl-5.1-2.fc13, globus-gatekeeper-5.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-gram-job-manager-setup-sge globus-openssl globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc13,globus-gram-job-manager-setup-fork-4.2-1.fc13,globus-gram-job-manager-setup-condor-4.4-1.fc13,globus-gram-job-manager-setup-pbs-4.1-1.fc13,globus-gram-job-manager-setup-lsf-2.5-1.fc13,globus-gatekeeper-5.5-2.fc13,globus-core-5.17-1.fc13,globus-openssl-5.1-2.fc13,globus-common-11.5-1.fc13,globus-gsi-proxy-ssl-2.3-1.fc13,globus-openssl-module-1.3-1.fc13,globus-gsi-cert-utils-6.6-1.fc13,globus-gsi-callback-2.7-1.fc13,globus-gsi-credential-3.5-1.fc13,globus-gsi-proxy-core-4.5-1.fc13,globus-gss-assist-5.9-2.fc13,globus-proxy-utils-3.9-1.fc13,globus-ftp-client-5.3-2.fc13,globus-gass-copy-5.7-1.fc13,globus-rsl-assist-2.2-1.fc13,globus-rls-client-5.2-2.fc13,globus-rls-server-4.9-2.fc13,globus-gridftp-server-control-0.43-1.fc13,globus-gridftp-server-3.23-1.fc13,globus-gram-protocol-9.7-3.fc13,globus-gram-client-10.4-1.fc13,globus-gram-client-tools-8.2-1.fc13,globus-gram-job-manager-scripts-2.11-1.fc13! ,globus-gram-job-manager-10.59-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516538] Review Request: globus-gram-job-manager-setup-condor - Globus Toolkit - Condor Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516538 --- Comment #13 from Fedora Update System 2010-07-22 22:25:29 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc13, globus-gram-job-manager-setup-condor-4.4-1.fc13, globus-gram-job-manager-setup-pbs-4.1-1.fc13, globus-gram-job-manager-setup-lsf-2.5-1.fc13, globus-core-5.17-1.fc13, globus-common-11.5-1.fc13, globus-gsi-proxy-ssl-2.3-1.fc13, globus-openssl-module-1.3-1.fc13, globus-gsi-cert-utils-6.6-1.fc13, globus-gsi-callback-2.7-1.fc13, globus-gsi-credential-3.5-1.fc13, globus-gsi-proxy-core-4.5-1.fc13, globus-gss-assist-5.9-2.fc13, globus-proxy-utils-3.9-1.fc13, globus-ftp-client-5.3-2.fc13, globus-gass-copy-5.7-1.fc13, globus-rsl-assist-2.2-1.fc13, globus-rls-client-5.2-2.fc13, globus-rls-server-4.9-2.fc13, globus-gridftp-server-control-0.43-1.fc13, globus-gridftp-server-3.23-1.fc13, globus-gram-protocol-9.7-3.fc13, globus-gram-client-10.4-1.fc13, globus-gram-client-tools-8.2-1.fc13, globus-gram-job-manager-scripts-2.11-1.fc13, globus-gram-job-manager-10.59-2.fc13, globus-gram-job-manager-setup-sge-2.5-1.fc13, globus-openssl-5.1-2.fc13, globus-gatekeeper-5.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-gram-job-manager-setup-sge globus-openssl globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc13,globus-gram-job-manager-setup-fork-4.2-1.fc13,globus-gram-job-manager-setup-condor-4.4-1.fc13,globus-gram-job-manager-setup-pbs-4.1-1.fc13,globus-gram-job-manager-setup-lsf-2.5-1.fc13,globus-gatekeeper-5.5-2.fc13,globus-core-5.17-1.fc13,globus-openssl-5.1-2.fc13,globus-common-11.5-1.fc13,globus-gsi-proxy-ssl-2.3-1.fc13,globus-openssl-module-1.3-1.fc13,globus-gsi-cert-utils-6.6-1.fc13,globus-gsi-callback-2.7-1.fc13,globus-gsi-credential-3.5-1.fc13,globus-gsi-proxy-core-4.5-1.fc13,globus-gss-assist-5.9-2.fc13,globus-proxy-utils-3.9-1.fc13,globus-ftp-client-5.3-2.fc13,globus-gass-copy-5.7-1.fc13,globus-rsl-assist-2.2-1.fc13,globus-rls-client-5.2-2.fc13,globus-rls-server-4.9-2.fc13,globus-gridftp-server-control-0.43-1.fc13,globus-gridftp-server-3.23-1.fc13,globus-gram-protocol-9.7-3.fc13,globus-gram-client-10.4-1.fc13,globus-gram-client-tools-8.2-1.fc13,globus-gram-job-manager-scripts-2.11-1.fc13! ,globus-gram-job-manager-10.59-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516538] Review Request: globus-gram-job-manager-setup-condor - Globus Toolkit - Condor Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516538 --- Comment #14 from Fedora Update System 2010-07-22 22:28:36 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc12, globus-gram-job-manager-setup-condor-4.4-1.fc12, globus-gram-job-manager-setup-pbs-4.1-1.fc12, globus-gram-job-manager-setup-lsf-2.5-1.fc12, globus-core-5.17-1.fc12, globus-common-11.5-1.fc12, globus-gsi-proxy-ssl-2.3-1.fc12, globus-openssl-module-1.3-1.fc12, globus-gsi-cert-utils-6.6-1.fc12, globus-gsi-callback-2.7-1.fc12, globus-gsi-credential-3.5-1.fc12, globus-gsi-proxy-core-4.5-1.fc12, globus-gss-assist-5.9-2.fc12, globus-proxy-utils-3.9-1.fc12, globus-ftp-client-5.3-2.fc12, globus-gass-copy-5.7-1.fc12, globus-rsl-assist-2.2-1.fc12, globus-rls-client-5.2-2.fc12, globus-rls-server-4.9-2.fc12, globus-gridftp-server-control-0.43-1.fc12, globus-gridftp-server-3.23-1.fc12, globus-gram-protocol-9.7-3.fc12, globus-gram-client-10.4-1.fc12, globus-gram-client-tools-8.2-1.fc12, globus-gram-job-manager-scripts-2.11-1.fc12, globus-gram-job-manager-10.59-2.fc12, globus-openssl-5.1-2.fc12, globus-gram-job-manager-setup-sge-2.5-1.fc12, globus-gatekeeper-5.5-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-openssl globus-gram-job-manager-setup-sge globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc12,globus-gram-job-manager-setup-fork-4.2-1.fc12,globus-gram-job-manager-setup-condor-4.4-1.fc12,globus-gram-job-manager-setup-pbs-4.1-1.fc12,globus-gram-job-manager-setup-lsf-2.5-1.fc12,globus-gatekeeper-5.5-2.fc12,globus-core-5.17-1.fc12,globus-openssl-5.1-2.fc12,globus-common-11.5-1.fc12,globus-gsi-proxy-ssl-2.3-1.fc12,globus-openssl-module-1.3-1.fc12,globus-gsi-cert-utils-6.6-1.fc12,globus-gsi-callback-2.7-1.fc12,globus-gsi-credential-3.5-1.fc12,globus-gsi-proxy-core-4.5-1.fc12,globus-gss-assist-5.9-2.fc12,globus-proxy-utils-3.9-1.fc12,globus-ftp-client-5.3-2.fc12,globus-gass-copy-5.7-1.fc12,globus-rsl-assist-2.2-1.fc12,globus-rls-client-5.2-2.fc12,globus-rls-server-4.9-2.fc12,globus-gridftp-server-control-0.43-1.fc12,globus-gridftp-server-3.23-1.fc12,globus-gram-protocol-9.7-3.fc12,globus-gram-client-10.4-1.fc12,globus-gram-client-tools-8.2-1.fc12,globus-gram-job-manager-scripts-2.11-1.fc12! ,globus-gram-job-manager-10.59-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 563822] Review Request: globus-gram-job-manager-setup-sge - Globus Toolkit - Sun Grid Engine Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=563822 --- Comment #12 from Fedora Update System 2010-07-22 22:28:51 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc12, globus-gram-job-manager-setup-condor-4.4-1.fc12, globus-gram-job-manager-setup-pbs-4.1-1.fc12, globus-gram-job-manager-setup-lsf-2.5-1.fc12, globus-core-5.17-1.fc12, globus-common-11.5-1.fc12, globus-gsi-proxy-ssl-2.3-1.fc12, globus-openssl-module-1.3-1.fc12, globus-gsi-cert-utils-6.6-1.fc12, globus-gsi-callback-2.7-1.fc12, globus-gsi-credential-3.5-1.fc12, globus-gsi-proxy-core-4.5-1.fc12, globus-gss-assist-5.9-2.fc12, globus-proxy-utils-3.9-1.fc12, globus-ftp-client-5.3-2.fc12, globus-gass-copy-5.7-1.fc12, globus-rsl-assist-2.2-1.fc12, globus-rls-client-5.2-2.fc12, globus-rls-server-4.9-2.fc12, globus-gridftp-server-control-0.43-1.fc12, globus-gridftp-server-3.23-1.fc12, globus-gram-protocol-9.7-3.fc12, globus-gram-client-10.4-1.fc12, globus-gram-client-tools-8.2-1.fc12, globus-gram-job-manager-scripts-2.11-1.fc12, globus-gram-job-manager-10.59-2.fc12, globus-openssl-5.1-2.fc12, globus-gram-job-manager-setup-sge-2.5-1.fc12, globus-gatekeeper-5.5-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-openssl globus-gram-job-manager-setup-sge globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc12,globus-gram-job-manager-setup-fork-4.2-1.fc12,globus-gram-job-manager-setup-condor-4.4-1.fc12,globus-gram-job-manager-setup-pbs-4.1-1.fc12,globus-gram-job-manager-setup-lsf-2.5-1.fc12,globus-gatekeeper-5.5-2.fc12,globus-core-5.17-1.fc12,globus-openssl-5.1-2.fc12,globus-common-11.5-1.fc12,globus-gsi-proxy-ssl-2.3-1.fc12,globus-openssl-module-1.3-1.fc12,globus-gsi-cert-utils-6.6-1.fc12,globus-gsi-callback-2.7-1.fc12,globus-gsi-credential-3.5-1.fc12,globus-gsi-proxy-core-4.5-1.fc12,globus-gss-assist-5.9-2.fc12,globus-proxy-utils-3.9-1.fc12,globus-ftp-client-5.3-2.fc12,globus-gass-copy-5.7-1.fc12,globus-rsl-assist-2.2-1.fc12,globus-rls-client-5.2-2.fc12,globus-rls-server-4.9-2.fc12,globus-gridftp-server-control-0.43-1.fc12,globus-gridftp-server-3.23-1.fc12,globus-gram-protocol-9.7-3.fc12,globus-gram-client-10.4-1.fc12,globus-gram-client-tools-8.2-1.fc12,globus-gram-job-manager-scripts-2.11-1.fc12! ,globus-gram-job-manager-10.59-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516537 --- Comment #18 from Fedora Update System 2010-07-22 22:28:31 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc12, globus-gram-job-manager-setup-condor-4.4-1.fc12, globus-gram-job-manager-setup-pbs-4.1-1.fc12, globus-gram-job-manager-setup-lsf-2.5-1.fc12, globus-core-5.17-1.fc12, globus-common-11.5-1.fc12, globus-gsi-proxy-ssl-2.3-1.fc12, globus-openssl-module-1.3-1.fc12, globus-gsi-cert-utils-6.6-1.fc12, globus-gsi-callback-2.7-1.fc12, globus-gsi-credential-3.5-1.fc12, globus-gsi-proxy-core-4.5-1.fc12, globus-gss-assist-5.9-2.fc12, globus-proxy-utils-3.9-1.fc12, globus-ftp-client-5.3-2.fc12, globus-gass-copy-5.7-1.fc12, globus-rsl-assist-2.2-1.fc12, globus-rls-client-5.2-2.fc12, globus-rls-server-4.9-2.fc12, globus-gridftp-server-control-0.43-1.fc12, globus-gridftp-server-3.23-1.fc12, globus-gram-protocol-9.7-3.fc12, globus-gram-client-10.4-1.fc12, globus-gram-client-tools-8.2-1.fc12, globus-gram-job-manager-scripts-2.11-1.fc12, globus-gram-job-manager-10.59-2.fc12, globus-openssl-5.1-2.fc12, globus-gram-job-manager-setup-sge-2.5-1.fc12, globus-gatekeeper-5.5-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-openssl globus-gram-job-manager-setup-sge globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc12,globus-gram-job-manager-setup-fork-4.2-1.fc12,globus-gram-job-manager-setup-condor-4.4-1.fc12,globus-gram-job-manager-setup-pbs-4.1-1.fc12,globus-gram-job-manager-setup-lsf-2.5-1.fc12,globus-gatekeeper-5.5-2.fc12,globus-core-5.17-1.fc12,globus-openssl-5.1-2.fc12,globus-common-11.5-1.fc12,globus-gsi-proxy-ssl-2.3-1.fc12,globus-openssl-module-1.3-1.fc12,globus-gsi-cert-utils-6.6-1.fc12,globus-gsi-callback-2.7-1.fc12,globus-gsi-credential-3.5-1.fc12,globus-gsi-proxy-core-4.5-1.fc12,globus-gss-assist-5.9-2.fc12,globus-proxy-utils-3.9-1.fc12,globus-ftp-client-5.3-2.fc12,globus-gass-copy-5.7-1.fc12,globus-rsl-assist-2.2-1.fc12,globus-rls-client-5.2-2.fc12,globus-rls-server-4.9-2.fc12,globus-gridftp-server-control-0.43-1.fc12,globus-gridftp-server-3.23-1.fc12,globus-gram-protocol-9.7-3.fc12,globus-gram-client-10.4-1.fc12,globus-gram-client-tools-8.2-1.fc12,globus-gram-job-manager-scripts-2.11-1.fc12! ,globus-gram-job-manager-10.59-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516539] Review Request: globus-gram-job-manager-setup-lsf - Globus Toolkit - LSF Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516539 --- Comment #15 from Fedora Update System 2010-07-22 22:28:41 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc12, globus-gram-job-manager-setup-condor-4.4-1.fc12, globus-gram-job-manager-setup-pbs-4.1-1.fc12, globus-gram-job-manager-setup-lsf-2.5-1.fc12, globus-core-5.17-1.fc12, globus-common-11.5-1.fc12, globus-gsi-proxy-ssl-2.3-1.fc12, globus-openssl-module-1.3-1.fc12, globus-gsi-cert-utils-6.6-1.fc12, globus-gsi-callback-2.7-1.fc12, globus-gsi-credential-3.5-1.fc12, globus-gsi-proxy-core-4.5-1.fc12, globus-gss-assist-5.9-2.fc12, globus-proxy-utils-3.9-1.fc12, globus-ftp-client-5.3-2.fc12, globus-gass-copy-5.7-1.fc12, globus-rsl-assist-2.2-1.fc12, globus-rls-client-5.2-2.fc12, globus-rls-server-4.9-2.fc12, globus-gridftp-server-control-0.43-1.fc12, globus-gridftp-server-3.23-1.fc12, globus-gram-protocol-9.7-3.fc12, globus-gram-client-10.4-1.fc12, globus-gram-client-tools-8.2-1.fc12, globus-gram-job-manager-scripts-2.11-1.fc12, globus-gram-job-manager-10.59-2.fc12, globus-openssl-5.1-2.fc12, globus-gram-job-manager-setup-sge-2.5-1.fc12, globus-gatekeeper-5.5-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-openssl globus-gram-job-manager-setup-sge globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc12,globus-gram-job-manager-setup-fork-4.2-1.fc12,globus-gram-job-manager-setup-condor-4.4-1.fc12,globus-gram-job-manager-setup-pbs-4.1-1.fc12,globus-gram-job-manager-setup-lsf-2.5-1.fc12,globus-gatekeeper-5.5-2.fc12,globus-core-5.17-1.fc12,globus-openssl-5.1-2.fc12,globus-common-11.5-1.fc12,globus-gsi-proxy-ssl-2.3-1.fc12,globus-openssl-module-1.3-1.fc12,globus-gsi-cert-utils-6.6-1.fc12,globus-gsi-callback-2.7-1.fc12,globus-gsi-credential-3.5-1.fc12,globus-gsi-proxy-core-4.5-1.fc12,globus-gss-assist-5.9-2.fc12,globus-proxy-utils-3.9-1.fc12,globus-ftp-client-5.3-2.fc12,globus-gass-copy-5.7-1.fc12,globus-rsl-assist-2.2-1.fc12,globus-rls-client-5.2-2.fc12,globus-rls-server-4.9-2.fc12,globus-gridftp-server-control-0.43-1.fc12,globus-gridftp-server-3.23-1.fc12,globus-gram-protocol-9.7-3.fc12,globus-gram-client-10.4-1.fc12,globus-gram-client-tools-8.2-1.fc12,globus-gram-job-manager-scripts-2.11-1.fc12! ,globus-gram-job-manager-10.59-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516540] Review Request: globus-gram-job-manager-setup-pbs - Globus Toolkit - PBS Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516540 --- Comment #13 from Fedora Update System 2010-07-22 22:28:47 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc12, globus-gram-job-manager-setup-condor-4.4-1.fc12, globus-gram-job-manager-setup-pbs-4.1-1.fc12, globus-gram-job-manager-setup-lsf-2.5-1.fc12, globus-core-5.17-1.fc12, globus-common-11.5-1.fc12, globus-gsi-proxy-ssl-2.3-1.fc12, globus-openssl-module-1.3-1.fc12, globus-gsi-cert-utils-6.6-1.fc12, globus-gsi-callback-2.7-1.fc12, globus-gsi-credential-3.5-1.fc12, globus-gsi-proxy-core-4.5-1.fc12, globus-gss-assist-5.9-2.fc12, globus-proxy-utils-3.9-1.fc12, globus-ftp-client-5.3-2.fc12, globus-gass-copy-5.7-1.fc12, globus-rsl-assist-2.2-1.fc12, globus-rls-client-5.2-2.fc12, globus-rls-server-4.9-2.fc12, globus-gridftp-server-control-0.43-1.fc12, globus-gridftp-server-3.23-1.fc12, globus-gram-protocol-9.7-3.fc12, globus-gram-client-10.4-1.fc12, globus-gram-client-tools-8.2-1.fc12, globus-gram-job-manager-scripts-2.11-1.fc12, globus-gram-job-manager-10.59-2.fc12, globus-openssl-5.1-2.fc12, globus-gram-job-manager-setup-sge-2.5-1.fc12, globus-gatekeeper-5.5-2.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-openssl globus-gram-job-manager-setup-sge globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc12,globus-gram-job-manager-setup-fork-4.2-1.fc12,globus-gram-job-manager-setup-condor-4.4-1.fc12,globus-gram-job-manager-setup-pbs-4.1-1.fc12,globus-gram-job-manager-setup-lsf-2.5-1.fc12,globus-gatekeeper-5.5-2.fc12,globus-core-5.17-1.fc12,globus-openssl-5.1-2.fc12,globus-common-11.5-1.fc12,globus-gsi-proxy-ssl-2.3-1.fc12,globus-openssl-module-1.3-1.fc12,globus-gsi-cert-utils-6.6-1.fc12,globus-gsi-callback-2.7-1.fc12,globus-gsi-credential-3.5-1.fc12,globus-gsi-proxy-core-4.5-1.fc12,globus-gss-assist-5.9-2.fc12,globus-proxy-utils-3.9-1.fc12,globus-ftp-client-5.3-2.fc12,globus-gass-copy-5.7-1.fc12,globus-rsl-assist-2.2-1.fc12,globus-rls-client-5.2-2.fc12,globus-rls-server-4.9-2.fc12,globus-gridftp-server-control-0.43-1.fc12,globus-gridftp-server-3.23-1.fc12,globus-gram-protocol-9.7-3.fc12,globus-gram-client-10.4-1.fc12,globus-gram-client-tools-8.2-1.fc12,globus-gram-job-manager-scripts-2.11-1.fc12! ,globus-gram-job-manager-10.59-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516537] Review Request: globus-gram-job-manager-setup-fork - Globus Toolkit - Fork Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516537 --- Comment #17 from Fedora Update System 2010-07-22 22:25:19 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc13, globus-gram-job-manager-setup-condor-4.4-1.fc13, globus-gram-job-manager-setup-pbs-4.1-1.fc13, globus-gram-job-manager-setup-lsf-2.5-1.fc13, globus-core-5.17-1.fc13, globus-common-11.5-1.fc13, globus-gsi-proxy-ssl-2.3-1.fc13, globus-openssl-module-1.3-1.fc13, globus-gsi-cert-utils-6.6-1.fc13, globus-gsi-callback-2.7-1.fc13, globus-gsi-credential-3.5-1.fc13, globus-gsi-proxy-core-4.5-1.fc13, globus-gss-assist-5.9-2.fc13, globus-proxy-utils-3.9-1.fc13, globus-ftp-client-5.3-2.fc13, globus-gass-copy-5.7-1.fc13, globus-rsl-assist-2.2-1.fc13, globus-rls-client-5.2-2.fc13, globus-rls-server-4.9-2.fc13, globus-gridftp-server-control-0.43-1.fc13, globus-gridftp-server-3.23-1.fc13, globus-gram-protocol-9.7-3.fc13, globus-gram-client-10.4-1.fc13, globus-gram-client-tools-8.2-1.fc13, globus-gram-job-manager-scripts-2.11-1.fc13, globus-gram-job-manager-10.59-2.fc13, globus-gram-job-manager-setup-sge-2.5-1.fc13, globus-openssl-5.1-2.fc13, globus-gatekeeper-5.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-gram-job-manager-setup-sge globus-openssl globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc13,globus-gram-job-manager-setup-fork-4.2-1.fc13,globus-gram-job-manager-setup-condor-4.4-1.fc13,globus-gram-job-manager-setup-pbs-4.1-1.fc13,globus-gram-job-manager-setup-lsf-2.5-1.fc13,globus-gatekeeper-5.5-2.fc13,globus-core-5.17-1.fc13,globus-openssl-5.1-2.fc13,globus-common-11.5-1.fc13,globus-gsi-proxy-ssl-2.3-1.fc13,globus-openssl-module-1.3-1.fc13,globus-gsi-cert-utils-6.6-1.fc13,globus-gsi-callback-2.7-1.fc13,globus-gsi-credential-3.5-1.fc13,globus-gsi-proxy-core-4.5-1.fc13,globus-gss-assist-5.9-2.fc13,globus-proxy-utils-3.9-1.fc13,globus-ftp-client-5.3-2.fc13,globus-gass-copy-5.7-1.fc13,globus-rsl-assist-2.2-1.fc13,globus-rls-client-5.2-2.fc13,globus-rls-server-4.9-2.fc13,globus-gridftp-server-control-0.43-1.fc13,globus-gridftp-server-3.23-1.fc13,globus-gram-protocol-9.7-3.fc13,globus-gram-client-10.4-1.fc13,globus-gram-client-tools-8.2-1.fc13,globus-gram-job-manager-scripts-2.11-1.fc13! ,globus-gram-job-manager-10.59-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516540] Review Request: globus-gram-job-manager-setup-pbs - Globus Toolkit - PBS Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516540 --- Comment #12 from Fedora Update System 2010-07-22 22:25:39 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc13, globus-gram-job-manager-setup-condor-4.4-1.fc13, globus-gram-job-manager-setup-pbs-4.1-1.fc13, globus-gram-job-manager-setup-lsf-2.5-1.fc13, globus-core-5.17-1.fc13, globus-common-11.5-1.fc13, globus-gsi-proxy-ssl-2.3-1.fc13, globus-openssl-module-1.3-1.fc13, globus-gsi-cert-utils-6.6-1.fc13, globus-gsi-callback-2.7-1.fc13, globus-gsi-credential-3.5-1.fc13, globus-gsi-proxy-core-4.5-1.fc13, globus-gss-assist-5.9-2.fc13, globus-proxy-utils-3.9-1.fc13, globus-ftp-client-5.3-2.fc13, globus-gass-copy-5.7-1.fc13, globus-rsl-assist-2.2-1.fc13, globus-rls-client-5.2-2.fc13, globus-rls-server-4.9-2.fc13, globus-gridftp-server-control-0.43-1.fc13, globus-gridftp-server-3.23-1.fc13, globus-gram-protocol-9.7-3.fc13, globus-gram-client-10.4-1.fc13, globus-gram-client-tools-8.2-1.fc13, globus-gram-job-manager-scripts-2.11-1.fc13, globus-gram-job-manager-10.59-2.fc13, globus-gram-job-manager-setup-sge-2.5-1.fc13, globus-openssl-5.1-2.fc13, globus-gatekeeper-5.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-gram-job-manager-setup-sge globus-openssl globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc13,globus-gram-job-manager-setup-fork-4.2-1.fc13,globus-gram-job-manager-setup-condor-4.4-1.fc13,globus-gram-job-manager-setup-pbs-4.1-1.fc13,globus-gram-job-manager-setup-lsf-2.5-1.fc13,globus-gatekeeper-5.5-2.fc13,globus-core-5.17-1.fc13,globus-openssl-5.1-2.fc13,globus-common-11.5-1.fc13,globus-gsi-proxy-ssl-2.3-1.fc13,globus-openssl-module-1.3-1.fc13,globus-gsi-cert-utils-6.6-1.fc13,globus-gsi-callback-2.7-1.fc13,globus-gsi-credential-3.5-1.fc13,globus-gsi-proxy-core-4.5-1.fc13,globus-gss-assist-5.9-2.fc13,globus-proxy-utils-3.9-1.fc13,globus-ftp-client-5.3-2.fc13,globus-gass-copy-5.7-1.fc13,globus-rsl-assist-2.2-1.fc13,globus-rls-client-5.2-2.fc13,globus-rls-server-4.9-2.fc13,globus-gridftp-server-control-0.43-1.fc13,globus-gridftp-server-3.23-1.fc13,globus-gram-protocol-9.7-3.fc13,globus-gram-client-10.4-1.fc13,globus-gram-client-tools-8.2-1.fc13,globus-gram-job-manager-scripts-2.11-1.fc13! ,globus-gram-job-manager-10.59-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516539] Review Request: globus-gram-job-manager-setup-lsf - Globus Toolkit - LSF Job Manager Setup
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=516539 --- Comment #14 from Fedora Update System 2010-07-22 22:25:34 EDT --- globus-gram-job-manager-setup-fork-4.2-1.fc13, globus-gram-job-manager-setup-condor-4.4-1.fc13, globus-gram-job-manager-setup-pbs-4.1-1.fc13, globus-gram-job-manager-setup-lsf-2.5-1.fc13, globus-core-5.17-1.fc13, globus-common-11.5-1.fc13, globus-gsi-proxy-ssl-2.3-1.fc13, globus-openssl-module-1.3-1.fc13, globus-gsi-cert-utils-6.6-1.fc13, globus-gsi-callback-2.7-1.fc13, globus-gsi-credential-3.5-1.fc13, globus-gsi-proxy-core-4.5-1.fc13, globus-gss-assist-5.9-2.fc13, globus-proxy-utils-3.9-1.fc13, globus-ftp-client-5.3-2.fc13, globus-gass-copy-5.7-1.fc13, globus-rsl-assist-2.2-1.fc13, globus-rls-client-5.2-2.fc13, globus-rls-server-4.9-2.fc13, globus-gridftp-server-control-0.43-1.fc13, globus-gridftp-server-3.23-1.fc13, globus-gram-protocol-9.7-3.fc13, globus-gram-client-10.4-1.fc13, globus-gram-client-tools-8.2-1.fc13, globus-gram-job-manager-scripts-2.11-1.fc13, globus-gram-job-manager-10.59-2.fc13, globus-gram-job-manager-setup-sge-2.5-1.fc13, globus-openssl-5.1-2.fc13, globus-gatekeeper-5.5-2.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update globus-gram-job-manager-setup-fork globus-gram-job-manager-setup-condor globus-gram-job-manager-setup-pbs globus-gram-job-manager-setup-lsf globus-core globus-common globus-gsi-proxy-ssl globus-openssl-module globus-gsi-cert-utils globus-gsi-callback globus-gsi-credential globus-gsi-proxy-core globus-gss-assist globus-proxy-utils globus-ftp-client globus-gass-copy globus-rsl-assist globus-rls-client globus-rls-server globus-gridftp-server-control globus-gridftp-server globus-gram-protocol globus-gram-client globus-gram-client-tools globus-gram-job-manager-scripts globus-gram-job-manager globus-gram-job-manager-setup-sge globus-openssl globus-gatekeeper'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/globus-gram-job-manager-setup-sge-2.5-1.fc13,globus-gram-job-manager-setup-fork-4.2-1.fc13,globus-gram-job-manager-setup-condor-4.4-1.fc13,globus-gram-job-manager-setup-pbs-4.1-1.fc13,globus-gram-job-manager-setup-lsf-2.5-1.fc13,globus-gatekeeper-5.5-2.fc13,globus-core-5.17-1.fc13,globus-openssl-5.1-2.fc13,globus-common-11.5-1.fc13,globus-gsi-proxy-ssl-2.3-1.fc13,globus-openssl-module-1.3-1.fc13,globus-gsi-cert-utils-6.6-1.fc13,globus-gsi-callback-2.7-1.fc13,globus-gsi-credential-3.5-1.fc13,globus-gsi-proxy-core-4.5-1.fc13,globus-gss-assist-5.9-2.fc13,globus-proxy-utils-3.9-1.fc13,globus-ftp-client-5.3-2.fc13,globus-gass-copy-5.7-1.fc13,globus-rsl-assist-2.2-1.fc13,globus-rls-client-5.2-2.fc13,globus-rls-server-4.9-2.fc13,globus-gridftp-server-control-0.43-1.fc13,globus-gridftp-server-3.23-1.fc13,globus-gram-protocol-9.7-3.fc13,globus-gram-client-10.4-1.fc13,globus-gram-client-tools-8.2-1.fc13,globus-gram-job-manager-scripts-2.11-1.fc13! ,globus-gram-job-manager-10.59-2.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613646] Review Request: twlog - A simple day to day logging program for ham radio
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613646 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|CLOSED Fixed In Version||twlog-2.7-2.fc12 Resolution||ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=561456 Fedora Update System changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||jnr-x86asm-0.1-2.fc13 Resolution||ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 613646] Review Request: twlog - A simple day to day logging program for ham radio
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=613646 --- Comment #16 from Fedora Update System 2010-07-22 22:23:41 EDT --- twlog-2.7-2.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 561456] Review Request: jnr-x86asm - Pure-java port of asmjit
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=561456 --- Comment #11 from Fedora Update System 2010-07-22 22:24:05 EDT --- jnr-x86asm-0.1-2.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616193] Review Request: freerdp - X Remote Desktop Protocol Client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=616193
Mads Kiilerich changed:
What|Removed |Added
CC||m...@kiilerich.com
--- Comment #3 from Mads Kiilerich 2010-07-22 21:50:07 EDT
---
Some brief comments from reading through the spec follows. Some of it is
duplicate of what is said on bug 603481, but some of it might explain something
(not) there.
> License:BSD and GPLv2+
The project as whole is GPLv2+ even though some parts "just" are
GPL-compatible. (Just like glibc doesn't mention BSD despite
/usr/share/doc/glibc-2.12/LICENSES.)
> %description
> FreeRDP is a X Remote Desktop Protocol Client for Windows Terminal Servers.
> It is a fork of the rdesktop project that intends to rapidly start moving
> forward and implement features that rdesktop lacks the most.
Actually, FreeRDP is the project, libfreerdp is the library, and xfreerdp is
the X client. (I have tried to convince upstream to create a simple client on
each platform and call it freerdp but haven't succeeded yet.) The way it is I
think it would be least confusing if the package was called xfreerdp like the
binary.
> BuildRequires: directfb-devel
The directfb port is far from ready yet and shouldn't be packaged
> BuildRequires: pcsc-lite-devel
configure is lying - that isn't implemented yet / any longer.
> BuildRequires: libsamplerate-devel
> BuildRequires: libao-devel
> # not enabled by default cause it does not yet work
> %{?_with_libvncserver:BuildRequires: libvncserver-devel}
Also old cruft that doesn't apply
> Requires: pcsc-lite
N/A
> %packagedirectfb
N/A
> %packagecommon
> Summary:Common libraries, keymaps and plugins for %{name}
> Group: System Environment/Libraries
>
> %descriptioncommon
> The %{name}-common package contains common libraries, keymaps and plugins for
> %{name} and %{name}-directfb.
It is mostly the core freerdp lib, so shouldn't it be called something with lib
instead?
I also find it confusing that the -devel package is for a -common package.
> %prep
> %setup -q
>
>
> %build
> %configure --enable-smartcard --with-dfb
N/A N/A
> --disable-static \
good idea
> %{?_with_libvncserver:--with-libvncserver}
N/A
> sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=""|g'
> libtool
> sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool
Are they really needed? AFAICS my packages without this hack don't have any
issues.
> %install
> rm -rf $RPM_BUILD_ROOT
> make install DESTDIR=$RPM_BUILD_ROOT INSTALL='install -p'
AFAIK -p isn't required by the guidelines. Just doing it on the packages where
the maintainer cares seems a bit odd.
> %doc AUTHORS COPYING README doc/{ChangeLog,TODO,rdp-keyboard.odg,*.txt}
Most of these files are outdated and no longer applies
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #10 from Mads Kiilerich 2010-07-22 21:41:49 EDT --- (In reply to comment #9) It is late/early here, so an incomplete response follows. I have updated the spec at http://kiilerix.fedorapeople.org/freerdp.spec . I will test and review it again and provide rpms "tomorrow". Comments are welcome before that. > - Whenever you change something, increase the release and add a changelog > entry, even during review. Yes. I was lazy because nobody had looked at it yet and no review was started. > - The header of the spec could be more legible. Some lines are indented, most > are not. I suggest to use the template from rpmdev-newspec. It was based on the old spec inherited from rdesktop. I have now aligned it more with newspec and your package so we can see the important differences. > - I don't think that xfreerdp should be a separate package. xfreerdp doesn't > add additional dependencies and is only 53K, so there is not much so save. Your "freerdp" package also only contains xfreerdp, AFAICS. That should be the only package that has dependencies to X, so I think it should be optional. Can you please clarify? > People should be able to run "yum install freerdp" to get the binary, just > like > it was with rdesktop. It's also hard for people to find the freerdp component > in bugzilla when they want to file a bug against xfreerdp. People should be able to install the freerdp package and get the freerdp binary. But there is no freerdp binary. It is called xfreerdp. Users should thus install the xfreerdp package to get xfreerdp. I would have preferred if the binarys name was freerdp, just like firefox is called firefox everywhere. But now the binary is xfreerdp. Comments? > - What is the use of the libs package? If you are making a libs package, you > should IMHO merge it with the plugins package that contains libraries too. libs contains the core functionality of freerdp. It consists of ordinary dynamic libraries that can be linked into an application (such as xfreerdp or remmina) and some helper stuff. Plugins are optional parts that can be loaded by the core libraries. They implement optional "channels" in the rdp protocol and can thus handle sound, printing, filetransfer, smartcard, etc. They have dependencies to other libraries, but they don't have to be used, and other plugins can provide the same or other functionality. For use on a minimal livecd it could perhaps be nice to have sound but avoid dependencies to cups and pcsc, so we might want to split it up even more. Comments? > Then > you could spilt the noarch and the arched content, but I don't think this adds > much value here. 96 k? Agreed, could be done, but no point. > - Subpackages are missing Group tag, thus they will all be > Applications/Communications instead of System Environment/Libraries, > Development/Libraries or whatever is appropriate. Fixed > - There are some BuildRequires missing. When I build your package in mock, I > see > > checking for PCSCLITE... no > checking for old version of PCSC... no > [...] > checking dmedia/audio.h usability... no > checking dmedia/audio.h presence... no > checking for dmedia/audio.h... no > checking sys/audioio.h usability... no > checking sys/audioio.h presence... no > checking for sys/audioio.h... no > checking for LIBAO... no > checking for ALSA... no > checking for LIBSAMPLERATE... no > [...] > checking for IceConnectionNumber in -lICE... no > > Thus you should add pcsc-lite-devel, libao-devel, alsa-lib-devel, > libsamplerate-devel and libICE-devel. configure contains a lot of cruft from rdesktop. I might fix it upstream when I come to it - unless somebody else does first. smartcard doesn't work at all ... yet. channels/rdpdr/smartcard/scard.c is dead. It was a bug that I had included --enable-smartcard. libao-devel: channels/rdpsnd/rdpsnd_libao.c is dead alsa-lib-devel: Fixed - I had missed that libsamplerate: only used by channels/rdpsnd/rdpsnd_dsp.c which is dead libICE: no clue why there is an option for it, but it is apparently not used (vncserver seems to dead too ... if it ever was alive) > - Require pcscd-lite because otherwise only pcsc-libs will pulled in. ^^^ > - Requires: pkgconfig s no longer needed for the devel package, rpmbuild will > determine that autumatically. Fixed > - --with-ipv6 is not needed, it is default. Fixed > - --with-sound and --with-crypto=openssl are not needed ether if the build > requirements are correct. However you might want to keep it to spot build > problems. (I have patches for NSS and will switch to use that when possible. Still polishing the patches. I also plan to implement pulseaudio - and smart card.) > - add --disable-static to prevent building of *.a files. Fixed (I was removing them together with .la files instead). > - I have add
[Bug 617400] Review Request: pfHandle - wrapper for tools to manage postfix mail queue
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617400 manuel wolfshant changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617405] New: Review Request: poezio - An IRC-like jabber (XMPP) console client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: poezio - An IRC-like jabber (XMPP) console client https://bugzilla.redhat.com/show_bug.cgi?id=617405 Summary: Review Request: poezio - An IRC-like jabber (XMPP) console client Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: lo...@louiz.org QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://louizatakk.fedorapeople.org/rpm/poezio.spec SRPM URL: http://louizatakk.fedorapeople.org/rpm/poezio-0.6.2-1.fc13.src.rpm Description: A jabber (XMPP) console client that aims at being similar to popular IRC clients such as Irssi or Weechat. Its main goal is to let the user connect to the Jabber network and join chat rooms without requiring any registration. Note: I'm the upstream developer of this software. rpmlint output: poezio.noarch: W: no-manual-page-for-binary poezio 2 packages and 1 specfiles checked; 0 errors, 1 warnings. The man page will be added in latter version (http://codingteam.net/project/poezio/bugs/show/1670) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 575529] Review Request: dwm - Dynamic window manager
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=575529
Christoph Wickert changed:
What|Removed |Added
CC||cwick...@fedoraproject.org
AssignedTo|nob...@fedoraproject.org|cwick...@fedoraproject.org
--- Comment #15 from Christoph Wickert 2010-07-22
19:25:52 EDT ---
(In reply to comment #5)
> In fact, you were right in using $RPM_BUILD_ROOT%{_datadir}/xsessions/ as PATH
> for the desktop file, but it still _must_ be installed using
> desktop-file-install.
No, this rule only applies to desktop files that create menu entries. We have
other desktop files for xsession entries, xfce4-panel plugins, kde services and
many more. These are not necessarily flowing the freedesktop specs and thus
cannot be installed with desktop-file-utils. This means that the desktop file
is ok for this package.
But there is another problem: Quoting from http://dwm.suckless.org/
Because dwm is customized through editing its source code, it’s pointless to
make binary packages of it. This keeps its userbase small and elitist. No
novices asking stupid questions. There are some distributions that provide
binary packages though.
Damien, are you sure you want to package this? If you are willing to maintain
this, I'll go ahead and do an official review, but IMHO you really should think
about this twice.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615669] Review Request: perl-Crypt-Cracklib - perl module to interact with libcrack
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=615669 --- Comment #7 from Carl Thompson 2010-07-22 18:42:15 EDT --- Another package I've submitted is https://bugzilla.redhat.com/show_bug.cgi?id=617400 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617400] New: Review Request: pfHandle - wrapper for tools to manage postfix mail queue
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: pfHandle - wrapper for tools to manage postfix mail queue https://bugzilla.redhat.com/show_bug.cgi?id=617400 Summary: Review Request: pfHandle - wrapper for tools to manage postfix mail queue Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: low Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fed...@red-dragon.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://www.red-dragon.com/pfHandle.spec SRPM URL: http://www.red-dragon.com/pfHandle-1-1.src.rpm Description: wrapper for tools to manage postfix mail queue Second package submitted, first was https://bugzilla.redhat.com/show_bug.cgi?id=615669 Need a sponsor This package and spec passed rpmlint with no errors This package built on koji for fedora 13 and rawhide with no errors. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616120] Review Request: spacecmd - Command-line interface to Spacewalk and Satellite servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616120 --- Comment #4 from Aron Parsons 2010-07-22 18:10:20 EDT --- Ah, I missed that part on the wiki. :-( Fixes have all been committed and are tagged as spacecmd-0.5.2-1. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616120] Review Request: spacecmd - Command-line interface to Spacewalk and Satellite servers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616120 --- Comment #3 from Maxim Burgerhout 2010-07-22 17:33:41 EDT --- You must add BuildRequires: python-devel to your specfile. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 575529] Review Request: dwm - Dynamic window manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=575529 --- Comment #14 from François Cami 2010-07-22 17:11:53 EDT --- And your current package does as well: 1 packages and 0 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 575529] Review Request: dwm - Dynamic window manager
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=575529 --- Comment #13 from François Cami 2010-07-22 17:08:52 EDT --- Hi Damien, It seems that window managers don't use Type nor Category (see the icewm and fluxbox packages for example). Could you update the desktop file in the package? Your current dwm.spec passes rpmlint: 0 packages and 1 specfiles checked; 0 errors, 0 warnings. Thanks -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616120] Review Request: spacecmd - Command-line interface to Spacewalk and Satellite servers
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=616120
--- Comment #2 from Aron Parsons 2010-07-22 16:50:10 EDT
---
The directory ownerships and %global fixes have been committed.
It seems that rpmbuild is not doing what it should in regards to byte compiling
the Python files. I tried the following to see if the spec was wrong:
- moving the *.py into %{python_sitelib}
- listing %{rhnroot}/spacecmd/*.py in the %files section
The only way I could get rpmbuild to byte-compile the Python files was to
explicitly call 'brp-python-bytecompile' in %install, which probably isn't
something that should be done. Any ideas on what's going on?
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616580] Review Request: tlomt-junction-fonts - a humanist sans serif font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616580 --- Comment #7 from Paul Flo Williams 2010-07-22 16:52:30 EDT --- 1. Package is called tlomt-junction-fonts because the Font Packaging guidelines say that a foundry name should be used. The existing example is tlomt-sniglet-fonts. 2. spectool worked for me with the full url. Did it fail for you? If so, I'll go back to the original method, no problem. 3. The version number comes from the file "Junction FONTLOG.txt", and it is also why the author uses "Junction 02" as the directory name in the zip file. Unfortunately, the font metadata still says "1.000", but the versions in fonts are notoriously unreliable, especially when they say 1.000. 4. The verbatim text of the license provided in the zip is provided as /usr/share/doc/tlomt-junction-fonts-2.0/Open Font License.txt. If you'd like me to generate a new version because the Source URL is no good, I'll do that tomorrow evening. Thank you very much for looking at the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569833] Review Request: drupal6 - An open-source content-management platform
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=569833
Paul W. Frields changed:
What|Removed |Added
CC||pfrie...@redhat.com
--- Comment #14 from Paul W. Frields 2010-07-22 16:10:55
EDT ---
With the full knowledge that Jon is indeed doing the work here and I'm just a
bystander (although an interested one!) :-) I would note that if this .htaccess
file is meant to be altered, it really should live in %{_sysconfdir}. Out of
the box, I would guess a substantial number of users will edit it to allow URL
rewrites, which is really desirable in lots of places.
Symlinking this to something like %{_sysconfdir}/httpd/conf.d/drupal6-site.conf
would suffice, right? I sure wouldn't advocate trying to munge the content
into the existing drupal.conf file, seems like an unnecessary maintenance load.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616120] Review Request: spacecmd - Command-line interface to Spacewalk and Satellite servers
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=616120
Maxim Burgerhout changed:
What|Removed |Added
Status|NEW |ASSIGNED
CC||ma...@wzzrd.com
AssignedTo|nob...@fedoraproject.org|ma...@wzzrd.com
Flag||fedora-review?,
||needinfo?(msu...@redhat.com
||)
--- Comment #1 from Maxim Burgerhout 2010-07-22 15:51:20 EDT
---
== MUST ==
[x] Run rpmlint on all parts of the package and post output
spacecmd.noarch: W: non-conffile-in-etc /etc/bash_completion.d/spacecmd
1 packages and 0 specfiles checked; 0 errors, 1 warnings.
I agree files in /etc/bash_completion.d should not have %config
[x] Package is named according to Naming Guidelines
[x] Specfile matches %{name}.spec
[x] Package matches Packaging Guidelines
[x] Package license matches Licensing Guidelines
[x] Actual license is in License tag in spec file
[x] License text in %doc if applicable
[x] Spec file is in English
[x] Spec file is legible
[x] MD5 sum of sources match upstream MD5 sum (through Source0)
[x] Must compile on at least *one* major architecture
[-] If package does not compile, it has ExcludeArch for that architecture,
with a bug filed for the architecture the package does not build on
[-] Build-time dependencies are in BuildRequires
[-] Uses %find_lang if appropriate; *not* %{_datadir}/locale/*
[-] Calls ldconfig in %post and %postun if carries libraries
[x] Does not carry copies of system libraries
[x] Doesn't do explicit relocatability
[!] Owns all directories it creates; if it uses a directory it does not create
itself, it should depend on the package that *does* create it
spacecmd should own /etc/bash_completion.d:
http://fedoraproject.org/wiki/PackagingGuidelines#Multiple_packages_own_files_in_a_common_directory_but_none_of_them_needs_to_require_the_others.
%{rhnroot} (/usr/share/rhn) is not owned by any packages:
https://fedoraproject.org/wiki/Packaging/UnownedDirectories#Common_Mistakes
spacecmd should either depend on the package that creates %{rhnroot} or own
it.
[x] Has sane permissions, defattr() in every %files section
[!] Consistently uses macros
According to
http://fedoraproject.org/wiki/Packaging/Guidelines#.25global_preferred_over_.25define
it is preferred to use %global instead of %define
[x] Contains only code or permissable content
[-] Large documentation goes in -doc package
[-] Header files go in -devel package
[-] static libraries go in -static package
[-] Libaries without a numeric suffix (.so.1.1) mostly go in -devel package
[-] Devel package requires base package like this
[-] All .la libtool archives are removed from the package
[-] GUI apps include a %{name}.desktop file
[x] Does not conflict with other packages in file ownership
[-] Filenames are in UTF-8
=== SHOULD ==
[-] If license text is not in separate file, query upstream to do so
[-] If available, the summary and description in the spec can be in other
languages than English too
[x] Test build in mock
[x] Builds on all architectures
[x] Runs
[x] Scriptlets in spec file must be sane
[-] Subpackages require the base package
[-] .pc files go in -devel, mostly, unless the package is a devel tool, like
gcc
[-] requires packages, not files in packages
[-] Try to have upstream create and add manpages for all binaries
There's a couple of issues with the package at the moment. They're marked with
a [!] above.
There is one bigger issue with the RPM as it is now though and that is the fact
that it does not contain *.pyc and *.pyo files when built in mock or Koji. The
RPM *should* contain those files in order to be able to properly cleaning up
when upgrading or uninstalling. See
http://fedoraproject.org/wiki/Packaging:Python#Files_to_include
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617340] Review Request: throttle - copy stdin to stdout at the specified speed (or lower)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=617340 François Cami changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617340] New: Review Request: throttle - copy stdin to stdout at the specified speed (or lower)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: throttle - copy stdin to stdout at the specified speed (or lower) https://bugzilla.redhat.com/show_bug.cgi?id=617340 Summary: Review Request: throttle - copy stdin to stdout at the specified speed (or lower) Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: fdc...@fcami.net QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://fcami.fedorapeople.org/srpms/throttle.spec SRPM URL: http://fcami.fedorapeople.org/srpms/throttle-1.2-2.fc13.src.rpm Description: http://fcami.fedorapeople.org/srpms/throttle.spec Throttle copies the standard input to the standard output while limiting bandwidth to the specified maximum. For instance, to avoid overloading a system while using dd to create a file on the hard drive, you would run: $ dd if=/dev/zero | throttle -m 20 | dd of=somefile bs=4096 count=10 Please note that I need a sponsor as I am not a packager yet. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606430] Review Request: DMitry - Deepmagic Information Gathering Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606430 Ian Baker changed: What|Removed |Added Flag|needinfo?(ianrichardba...@g | |mail.com) | --- Comment #4 from Ian Baker 2010-07-22 15:15:39 EDT --- Sorry for the late response, new house and new job! Thank you for your response and advice, I agree with your suggestions and will make the modifications as soon as I can. Thanks again. 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617318] New: Review Request: xpenguins - Cute little penguins everywhere
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: xpenguins - Cute little penguins everywhere https://bugzilla.redhat.com/show_bug.cgi?id=617318 Summary: Review Request: xpenguins - Cute little penguins everywhere Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: goe...@uddeborg.se QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: ftp://ftp.uddeborg.se/pub/xpenguins/xpenguins.spec SRPM URL: ftp://ftp.uddeborg.se/pub/xpenguins/xpenguins-2.2-1.src.rpm Description: XPenguins animates a friendly family of penguins in your root window. They drop in from the top of the screen, walk along the tops of your windows, up the side of your windows, levitate, skateboard, and do other similarly exciting things. Rpmlint gives a few comments about not cleaning. I will add this package only for future releases, and that cleaning is no longer necessary. Similarly for a comment about the missing buildroot tag. Finally it has a couple of comments about the spelling, but they are false positives. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569833] Review Request: drupal6 - An open-source content-management platform
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569833 --- Comment #13 from David Nalley 2010-07-22 14:21:14 EDT --- That's certainly what I'd do. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 569833] Review Request: drupal6 - An open-source content-management platform
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=569833 --- Comment #12 from Jon Ciesla 2010-07-22 14:13:26 EDT --- SPEC: http://zanoni.jcomserv.net/fedora/drupal6/drupal6.spec SRPM: http://zanoni.jcomserv.net/fedora/drupal6/drupal6-6.17-2.fc13.src.rpm David, you raise an interesting point. Looking at said file, I don't really see a reason for it to be a config file. If you(the admin) want to add directives, could you not craft your own file and drop it in /etc/httpd/conf.d/? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 615695] Review Request: libaccounts-glib - Nokia Maemo Accounts base library
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=615695
Martin Gieseking changed:
What|Removed |Added
CC||martin.giesek...@uos.de
--- Comment #1 from Martin Gieseking 2010-07-22
14:09:59 EDT ---
The package looks almost fine to me. However, I recommend to be more specific
in the %file sections:
%files
%defattr(-,root,root,-)
%doc COPYING AUTHORS
%dir %{_datadir}/backup-framework/
%dir %{_datadir}/backup-framework/applications/
%{_libdir}/%{name}.so.*
%{_datadir}/backup-framework/applications/accounts.conf
%files devel
%defattr(-,root,root,-)
%{_libdir}/%{name}.so
%{_libdir}/pkgconfig/%{name}.pc
%{_includedir}/%{name}/
Also, file ChangeLog doesn't contain the changelogs but refers to running
git-log to get them. Thus, you should drop this file for now.
The SRPM referenced in the spec file is no longer available upstream. Instead
of extracting the tarball from a temporarily available SRPM, I suggest to get a
snapshot from the git repo.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 Christoph Wickert changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|cwick...@fedoraproject.org Flag||fedora-review? --- Comment #9 from Christoph Wickert 2010-07-22 13:48:22 EDT --- Thanks Mads, you could also update to 0.7.2, so we can continue this review in the meantime. Too bad I didn't see this review when I did my package. My spec looks different from yours and of course I think mine is better. ;) Some comments on your package: - Whenever you change something, increase the release and add a changelog entry, even during review. - The header of the spec could be more legible. Some lines are indented, most are not. I suggest to use the template from rpmdev-newspec. - I don't think that xfreerdp should be a separate package. xfreerdp doesn't add additional dependencies and is only 53K, so there is not much so save. People should be able to run "yum install freerdp" to get the binary, just like it was with rdesktop. It's also hard for people to find the freerdp component in bugzilla when they want to file a bug against xfreerdp. - What is the use of the libs package? If you are making a libs package, you should IMHO merge it with the plugins package that contains libraries too. Then you could spilt the noarch and the arched content, but I don't think this adds much value here. - Subpackages are missing Group tag, thus they will all be Applications/Communications instead of System Environment/Libraries, Development/Libraries or whatever is appropriate. - There are some BuildRequires missing. When I build your package in mock, I see checking for PCSCLITE... no checking for old version of PCSC... no [...] checking dmedia/audio.h usability... no checking dmedia/audio.h presence... no checking for dmedia/audio.h... no checking sys/audioio.h usability... no checking sys/audioio.h presence... no checking for sys/audioio.h... no checking for LIBAO... no checking for ALSA... no checking for LIBSAMPLERATE... no [...] checking for IceConnectionNumber in -lICE... no Thus you should add pcsc-lite-devel, libao-devel, alsa-lib-devel, libsamplerate-devel and libICE-devel. - Require pcscd-lite because otherwise only pcsc-libs will pulled in. - Requires: pkgconfig s no longer needed for the devel package, rpmbuild will determine that autumatically. - --with-ipv6 is not needed, it is default. - --with-sound and --with-crypto=openssl are not needed ether if the build requirements are correct. However you might want to keep it to spot build problems. - add --disable-static to prevent building of *.a files. - I have added --with-dfb and built dfbfreerdp as a separate package to avoid X deps. - add more documentation please - merge whatever you think makes sense from my 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611080] Review Request: python-unittest2 - Backport of new unittest feature for Python 2.7 to Python 2.4+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=611080 Fabian Affolter changed: What|Removed |Added Flag||fedora-cvs? --- Comment #8 from Fabian Affolter 2010-07-22 13:27:18 EDT --- New Package CVS Request === Package Name: python-unittest2 Short Description: Backport of new unittest feature for Python 2.7 to Python 2.4+ Owners: fab Branches: F-12 F-13 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 611080] Review Request: python-unittest2 - Backport of new unittest feature for Python 2.7 to Python 2.4+
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=611080 Fabian Affolter changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mbaco...@redhat.com --- Comment #7 from Fabian Affolter 2010-07-22 13:26:44 EDT --- Thanks for the review, Martin. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616580] Review Request: tlomt-junction-fonts - a humanist sans serif font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616580 Jochen Schmitt changed: What|Removed |Added Flag||fedora-review? --- Comment #6 from Jochen Schmitt 2010-07-22 13:19:40 EDT --- Good: + basename of the SPEC file has the same name as the package. + package zip file matches with upstream one (md5sum: fe26a1f85d7f367781e0f8764552dff7) + Package has consistantly usage of rpm macros + Package is build for noarch + Package has a valid license tag + Package has OFL as a valid font license for fedora. + package has no subpackage + locel build works fine + Rpmlint is quite for binary package + Build works fine on koji + Package has small %doc stanza, so no seperate subpackage is require + Package has proper %Changelog Bad: - Why do you call the package tlomt-junction. the upstream sources has the name junction and the official fontname I see on forntforge is Junction too. - downloaded package wia spectool -g has not the upstream filename which was based by the query string. Sorry, this was my part. We should revert the change between -1 and -2 and add a comment why we don't put the URL in the Source0 tag. question: How to you have determinate the version number of your package. - Verbatin text of the license which is provided by the upstream is not provide by the package. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616779] Review Request: rubygem-json_pure - JSON implementation in pure Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616779 --- Comment #3 from Michal Fojtik 2010-07-22 12:57:01 EDT --- I think this could be also possible solution, but it would not be bad to have a 'pure' version packaged. Also IMHO it will be much more cleaner to have 'original' dependency for cucumber than modifying cucumber spec file. Cucumber and other projects uses this because it's a "little bit" faster that original one. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 604331] Review Request: redshift - Adjusts the color temperature of your screen according to time of day
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=604331 Mamoru Tasaka changed: What|Removed |Added Flag||needinfo?(miloskomarce...@n ||etscape.net) --- Comment #4 from Mamoru Tasaka 2010-07-22 12:50:35 EDT --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606430] Review Request: DMitry - Deepmagic Information Gathering Tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606430 Mamoru Tasaka changed: What|Removed |Added Flag||needinfo?(ianrichardba...@g ||mail.com) --- Comment #3 from Mamoru Tasaka 2010-07-22 12:50:12 EDT --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 578269] Review Request: xgospel - An X11 client for Internet Go Server
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=578269 --- Comment #28 from Mamoru Tasaka 2010-07-22 12:51:56 EDT --- I will close this bug as NOTABUG if no response is received from the reported within ONE WEEK. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 606421] Review Request: quickfix - Open Source FIX Engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=606421 Mamoru Tasaka changed: What|Removed |Added Flag||needinfo?(mar...@martindeng ||ler.com) --- Comment #9 from Mamoru Tasaka 2010-07-22 12:49:25 EDT --- ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 592770] Review Request: texmakerx - LaTex Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=592770 --- Comment #9 from hannes 2010-07-22 12:46:20 EDT --- Updated to latest version: http://dl.dropbox.com/u/3917796/texmakerx-1.9.9a-1.fc13.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616580] Review Request: tlomt-junction-fonts - a humanist sans serif font
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616580 Jochen Schmitt changed: What|Removed |Added Status|NEW |ASSIGNED -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 476056] Review Request: gnustep-back - The GNUstep backend library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=476056 Jochen Schmitt changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #18 from Jochen Schmitt 2010-07-22 12:41:30 EDT --- Packages built for devel and EL-6 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616779] Review Request: rubygem-json_pure - JSON implementation in pure Ruby
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616779 Mamoru Tasaka changed: What|Removed |Added CC||mtas...@ioa.s.u-tokyo.ac.jp --- Comment #2 from Mamoru Tasaka 2010-07-22 12:41:46 EDT --- (In reply to comment #1) > We already have 'json' packaged for Fedora. This one is needed for Cucumber > update. So 's/json_pure/json/' in cucumber (0.8.5, maybe?) wouldn't be enough? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 469931] Review Request: ipmiutil - IPMI Management Utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=469931 --- Comment #49 from Andy Cress 2010-07-22 12:11:00 EDT --- Much better, thanks. A successful rawhide build. [arcr...@ac4lht ipmiutil]$ cvs update -dP -A cvs update: Updating . cvs update: Updating EL-6 cvs update: Updating common cvs update: Updating devel U devel/.cvsignore U devel/import.log U devel/ipmiutil.spec U devel/sources [arcr...@ac4lht ipmiutil]$ cd devel [arcr...@ac4lht devel]$ make build /usr/bin/koji build dist-rawhide 'cvs://cvs.fedoraproject.org/cvs/pkgs?rpms/ip miutil/devel#ipmiutil-2_6_8-1_fc14' Created task: 2341823 Task info: http://koji.fedoraproject.org/koji/taskinfo?taskID=2341823 None Watching tasks (this may be safely interrupted)... 2341823 build (dist-rawhide, /cvs/pkgs:rpms/ipmiutil/devel:ipmiutil-2_6_8-1_fc14 ): open (x86-09.phx2.fedoraproject.org) 2341825 buildSRPMFromSCM (/cvs/pkgs:rpms/ipmiutil/devel:ipmiutil-2_6_8-1_fc14) : open (x86-12.phx2.fedoraproject.org) 2341825 buildSRPMFromSCM (/cvs/pkgs:rpms/ipmiutil/devel:ipmiutil-2_6_8-1_fc14) : open (x86-12.phx2.fedoraproject.org) -> closed 0 free 1 open 1 done 0 failed 2341879 buildArch (ipmiutil-2.6.8-1.fc14.src.rpm, i686): open (x86-12.phx2.fed oraproject.org) 2341878 buildArch (ipmiutil-2.6.8-1.fc14.src.rpm, x86_64): open (x86-20.phx2.f edoraproject.org) 2341878 buildArch (ipmiutil-2.6.8-1.fc14.src.rpm, x86_64): open (x86-20.phx2.f edoraproject.org) -> closed 0 free 2 open 2 done 0 failed 2341913 tagBuild (noarch): closed 2341879 buildArch (ipmiutil-2.6.8-1.fc14.src.rpm, i686): open (x86-12.phx2.fed oraproject.org) -> closed 0 free 1 open 4 done 0 failed 2341823 build (dist-rawhide, /cvs/pkgs:rpms/ipmiutil/devel:ipmiutil-2_6_8-1_fc14 ): open (x86-09.phx2.fedoraproject.org) -> closed 0 free 0 open 5 done 0 failed 2341823 build (dist-rawhide, /cvs/pkgs:rpms/ipmiutil/devel:ipmiutil-2_6_8-1_fc14 ) completed successfully [arcr...@ac4lht devel]$ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 555018] Review Request: gnac - An audio converter for GNOME (first package, seeking sponsor)
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=555018 --- Comment #16 from Taylon 2010-07-22 12:03:50 EDT --- Here is the package updated to the latest release: SRPM URL: http://taylon.fedorapeople.org/gnac-0.2.2-1.fc13.src.rpm SPEC URL: http://taylon.fedorapeople.org/gnac.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 562226] Review Request: ccl - Free Common Lisp implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=562226 --- Comment #9 from Alexander Kahl 2010-07-22 11:48:35 EDT --- Oh.. I just realized PPC isn't supported as a primary architecture anymore. I'll remove the PPC part. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603638] Review Request: python-defcon - A set of flexible objects for representing UFO data.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603638 --- Comment #4 from Parag 2010-07-22 11:41:49 EDT --- When I asked upstream if they can help to fix this testcase failure issue, Author "Tal Leming" wrote me => No. The tests work for me, so it is probably a problem with your installation or your platform. I don't have time to debug that for you. In any case, the same license that allows you to redistribute this code also allows you to modify it. If these are indeed bugs, you can fix them for your repackaged version. If you submit the patches to me, I will consider applying them to my code. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603666] Review Request: python-defconAppKit - A collection of Vanilla objects for use in font editing applications.
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=603666
Thomas Spura changed:
What|Removed |Added
Status|NEW |ASSIGNED
Depends on||603639
--- Comment #1 from Thomas Spura 2010-07-22
11:29:48 EDT ---
Review:
Good:
- name ok
- source match upstream: 95f50ded72ebc5f77a9063b41820929c
- no libs
- buildarch noarch
- no locales
- no *.la
- license ok
Needswork:
- Development/Tools would be better as group
- %files:
Please be a bit more explicit like in your other packages. (It seems you just
forgot it here)
- %{version} should be in url
- Missing R fonttools and python-appkit and python-ufo2fdk
- This one again seems to not install right:
$ rpmls noarch/python-defconAppKit-0.1-1.fc13.noarch.rpm
drwxr-xr-x /usr/lib/python2.6/site-packages/defconAppKit
-rw-r--r-- /usr/lib/python2.6/site-packages/defconAppKit-0.1-py2.6.egg-info
-rw-r--r-- /usr/lib/python2.6/site-packages/defconAppKit/__init__.py
-rw-r--r-- /usr/lib/python2.6/site-packages/defconAppKit/__init__.pyc
-rw-r--r-- /usr/lib/python2.6/site-packages/defconAppKit/__init__.pyo
drwxr-xr-x /usr/share/doc/python-defconAppKit-0.1
-rw-r--r-- /usr/share/doc/python-defconAppKit-0.1/License.txt
Don't know, what's the problem here...
There should be more subfolders in defconAppKit
- Where is Foundation module?
./Lib/defconAppKit/controls/fontInfoView.py:from Foundation import *
Maybe another missing dependency.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603639] Review Request: python-ufo2fdk - A bridge between UFOs and the AFKDO
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603639 Thomas Spura changed: What|Removed |Added Blocks||603666 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603640] Review Request: python-compositor - A simple OpenType GSUB and GPOS engine
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603640 --- Comment #4 from Thomas Spura 2010-07-22 11:17:11 EDT --- A forgot to add: Development/Tools would be better for group -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603640] Review Request: python-compositor - A simple OpenType GSUB and GPOS engine
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=603640
Thomas Spura changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #3 from Thomas Spura 2010-07-22
11:13:55 EDT ---
Review:
Good:
- name ok
- source match upstream: 932993d25d981426de3d0379880e19b4
- no *.la
- no libs
- no locales
- $ rpmlint ./noarch/python-compositor-0.2b-1.fc13.noarch.rpm
./python-compositor-0.2b-1.fc13.src.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
- license ok
- %files ok
- %doc ok
Needswork:
- missing R: fonttools again ;)
- %{version} in url
missing fonttools as R is a BLOCKER, rest is SHOULD.
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 502358] Review Request: mojomojo - Catalyst & DBIx::Class powered Wiki
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=502358 --- Comment #18 from Iain Arnell 2010-07-22 11:13:18 EDT --- (In reply to comment #16) Thanks for picking this one up. (And the rest you've done recently - I must owe you a whole bunch of shiny beads). > I do not have a rawhide handy to test, but I've never seen before the > construct > that you have used in the config file for httpd, that is: > > Alias /mojomojo/ /usr/bin/mojomojo_fastcgi.pl/ > > Options ExecCGI > Order allow,deny > Allow from all > AddHandler fcgid-script .pl > > > where /usr/bin/mojomojo_fastcgi.pl is a script, not a directory as the Alias > directive might seem to indicate. > > http://httpd.apache.org/docs/2.2/mod/mod_alias.html#alias does not reference > this type of use either. Is it really correct ? Yes, I didn't believe it when I first saw it either (I got the snippet from http://blog.hjksolutions.com/articles/2007/07/19/catalyst-deployment-with-apache-2-and-mod_fcgid). But it certainly works. The Catalyst::Engine::FastCGI docs claim that the trailing slashes are important for setting PATH_INFO properly. There should be no need to test under rawhide - this review has been around long enough that it works out of the box on F-13 (and probably F-12 too). I'll wait until I hear back from dwalsh on the selinux bits before posting an updated 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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603641] Review Request: python-fontMath - A set of objects for performing math operations on font data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603641 --- Comment #5 from Parag 2010-07-22 11:03:06 EDT --- Thanks. I will add it when I will import it in cvs and also will add it in other packages. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603639] Review Request: python-ufo2fdk - A bridge between UFOs and the AFKDO
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=603639
Thomas Spura changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #2 from Thomas Spura 2010-07-22
10:59:59 EDT ---
Review:
Good:
- name ok
- source match upstream:
bd9d9ea3d41169f3d1f50abbdaa0dbb0
- spec readable
- macros everywhere
- no libs
- no *.la
- no locales
- $ rpmlint ./python-ufo2fdk-0.1-1.fc13.src.rpm
noarch/python-ufo2fdk-0.1-1.fc13.noarch.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
- license ok
Needswork:
- %{version} in url
- missing R fonttools
__
Missing R is a BLOCKER, %{verion} in url is SHOULD.
__
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603641] Review Request: python-fontMath - A set of objects for performing math operations on font data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603641 --- Comment #4 from Thomas Spura 2010-07-22 10:52:34 EDT --- (In reply to comment #3) > But, Do I really need to BR:python-robofab? > > See successful koji scratch build without it > http://koji.fedoraproject.org/koji/taskinfo?taskID=2340303 Long answer: Currently the setup.py is complaining with: + /usr/bin/python setup.py build *** Warning: defcon requires RoboFab, see: robofab.com see: http://koji.fedoraproject.org/koji/getfile?taskID=2340304&name=build.log So it would be better to also BuildRequire it. For now it's not needed, only if they choose to add a testsuite and you want to run it. I'd feel just saver with adding it now, so upstream can change what they want. (but no blocker throught) Short answer: no :) -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 617228] New: Review Request: rubygem-rack1 -- Common API for connecting web frameworks, web servers and layers of software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: rubygem-rack1 -- Common API for connecting web frameworks, web servers and layers of software https://bugzilla.redhat.com/show_bug.cgi?id=617228 Summary: Review Request: rubygem-rack1 -- Common API for connecting web frameworks, web servers and layers of software Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: mastah...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://stahnma.fedorapeople.org/newrails/rubygem-rack1.spec SRPM URL: http://stahnma.fedorapeople.org/newrails/rubygem-rack1-1.1.0-1.el5.src.rpm Description: Rack provides a common API for connecting web frameworks, web servers and layers of software in between This is a specific package targeted at EPEL5. It will fill a need to allow rubygem-sinatra into EPEL, and possibly allow for a parallel version of rails. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 420161] Review Request: python-bugzilla - A python library for interacting with Bugzilla.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=420161 Seth Vidal changed: What|Removed |Added CC||skvi...@sethdot.org Flag|fedora-cvs+ |fedora-cvs? --- Comment #17 from Seth Vidal 2010-07-22 10:18:20 EDT --- Package Change Request == Package Name: python-bugzilla New Branches: EL-6 Owners: wwoods -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 590573] Review Request: cuneiform - OCR System
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=590573 --- Comment #4 from Peter Lemenkov 2010-07-22 09:47:20 EDT --- Oh, sorry - I just found that Sergey finally added missing BuildRequires (although he didn't mention it here). Yes, updating this package up to 1.0.0 would be great. Ping, Sergey! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 590573] Review Request: cuneiform - OCR System
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=590573 --- Comment #3 from Andrea Musuruane 2010-07-22 09:36:48 EDT --- Upstream has released v1.0.0. Is it possible to update this package to the latest version? BTW, is the review stalled? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558743] Merge Review: perl-Unicode-String - handle Unicode issues
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=558743
--- Comment #5 from Petr Pisar 2010-07-22 09:18:54 EDT ---
This spec file patch is needed to filter private library out from provides and
to proceed local rpmbuild (rm ask for read-only file removal):
Index: perl-Unicode-String.spec
===
RCS file: /cvs/pkgs/rpms/perl-Unicode-String/devel/perl-Unicode-String.spec,v
retrieving revision 1.22
diff -u -r1.22 perl-Unicode-String.spec
--- perl-Unicode-String.spec7 May 2010 07:18:08 - 1.22
+++ perl-Unicode-String.spec22 Jul 2010 13:09:11 -
@@ -18,6 +18,8 @@
Requires: perl(:MODULE_COMPAT_%(eval "`%{__perl} -V:version`"; echo
$version))
# not detected by auto provide scripts:
Requires: perl(MIME::Base64)
+%filter_from_provides /String\.so/d
+%filter_setup
%description
%{summary}.
@@ -42,7 +44,7 @@
DESTDIR=$RPM_BUILD_ROOT \
INSTALLARCHLIB=$RPM_BUILD_ROOT%{perl_archlib}
find $RPM_BUILD_ROOT -type f \( -name perllocal.pod -o -name .packlist \
- -o \( -name '*.bs' -empty \) \) -exec rm {} ';'
+ -o \( -name '*.bs' -empty \) \) -exec rm -f {} ';'
find $RPM_BUILD_ROOT -depth -type d -empty -exec rmdir {} ';'
chmod -R u+w $RPM_BUILD_ROOT/*
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558743] Merge Review: perl-Unicode-String - handle Unicode issues
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558743 Marcela Mašláňová changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE Flag||fedora-review+ --- Comment #4 from Marcela Mašláňová 2010-07-22 08:46:29 EDT --- SRPM and RPM shows: rpmlint ~/Downloads/perl-Unicode-String-2.09-15.fc14.* perl-Unicode-String.x86_64: W: private-shared-object-provides /usr/lib64/perl5/auto/Unicode/String/String.so String.so()(64bit) 2 packages and 0 specfiles checked; 0 errors, 1 warnings. That's quite usual for perl modules. ACCEPT -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 580447] Review Request: perl-Module-Build - Build and install Perl modules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=580447 --- Comment #12 from Marcela Mašláňová 2010-07-22 08:44:02 EDT --- Ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558743] Merge Review: perl-Unicode-String - handle Unicode issues
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558743 Paul Howarth changed: What|Removed |Added CC||p...@city-fan.org --- Comment #3 from Paul Howarth 2010-07-22 08:36:48 EDT --- (In reply to comment #2) > The in-cvs version produces followin rpmlint output now: > > (none): E: no installed packages by name x86_64/perl-Unicode-String-8 > 0 packages and 1 specfiles checked; 0 errors, 0 warnings. You missed pressing the shift key and got an 8 instead of a *. $ rpmlint ~/perl-Unicode-String-* perl-Unicode-String.x86_64: W: private-shared-object-provides /usr/lib64/perl5/auto/Unicode/String/String.so String.so()(64bit) 3 packages and 0 specfiles checked; 0 errors, 1 warnings. Using perl_default_filter would fix that. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 558743] Merge Review: perl-Unicode-String - handle Unicode issues
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=558743 Petr Pisar changed: What|Removed |Added CC||ppi...@redhat.com --- Comment #2 from Petr Pisar 2010-07-22 08:29:57 EDT --- The in-cvs version produces followin rpmlint output now: (none): E: no installed packages by name x86_64/perl-Unicode-String-8 0 packages and 1 specfiles checked; 0 errors, 0 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616354] Review Request: geronimo-jpa - Java persistence API implementation
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616354 --- Comment #2 from Stanislav Ochotnicky 2010-07-22 07:43:59 EDT --- Small changes to provides/symlinks: Spec URL: http://sochotni.fedorapeople.org/packages/geronimo-jpa.spec SRPM URL: http://sochotni.fedorapeople.org/packages/geronimo-jpa-1.1.1-2.fc13.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. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616808] Review Request: maven-changes-plugin - Plugin to support reporting of changes between releases
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616808 Stanislav Ochotnicky changed: What|Removed |Added Flag||fedora-cvs? --- Comment #3 from Stanislav Ochotnicky 2010-07-22 07:33:15 EDT --- Thanks for review. New Package CVS Request === Package Name: maven-changes-plugin Short Description: Plugin to support reporting of changes between releases Owners: sochotni Branches: InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 --- Comment #8 from Mads Kiilerich 2010-07-22 07:26:05 EDT --- I will merge with whatever there might be in bug 616193 and update to 0.7.3 (which will be released in a couple of days). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 581220] Review Request: qtsingleapplication - Qt library to start applications only once per user
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=581220 --- Comment #28 from leigh scott 2010-07-22 06:57:24 EDT --- (In reply to comment #27) > Hey folks: I made the new builds with the libraries split: > > F-12: https://admin.fedoraproject.org/updates/qtsingleapplication-2.6.1-3.fc12 > F-13: https://admin.fedoraproject.org/updates/qtsingleapplication-2.6.1-3.fc13 > devel: http://koji.fedoraproject.org/koji/buildinfo?buildID=185326 > > Please test and let me know how things go. I have tested qbittorrent and the new packages function as expected. http://koji.fedoraproject.org/koji/buildinfo?buildID=185966 Can we get buildroot overrides for F13 & F12? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603628] Review Request: aldusleaf-crimson-text-fonts - A latin font for the production of technical books and papers
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603628 Parag changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #8 from Parag 2010-07-22 06:52:15 EDT --- Built in rawhide. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603641] Review Request: python-fontMath - A set of objects for performing math operations on font data
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603641 --- Comment #3 from Parag 2010-07-22 06:50:26 EDT --- Updated package as Spec URL: http://paragn.fedorapeople.org/fedora-work/SPECS/python-fontMath.spec SRPM URL: http://paragn.fedorapeople.org/fedora-work/SRPMS/python-fontMath-0.2-2.fc13.src.rpm But, Do I really need to BR:python-robofab? See successful koji scratch build without it http://koji.fedoraproject.org/koji/taskinfo?taskID=2340303 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 616193] Review Request: freerdp - X Remote Desktop Protocol Client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=616193 Christoph Wickert changed: What|Removed |Added Status|NEW |CLOSED Resolution||DUPLICATE --- Comment #2 from Christoph Wickert 2010-07-22 06:47:55 EDT --- I wonder why I didn't find it when I packaged freerdp. *** This bug has been marked as a duplicate of bug 603481 *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 Christoph Wickert changed: What|Removed |Added CC||cwick...@fedoraproject.org --- Comment #7 from Christoph Wickert 2010-07-22 06:47:55 EDT --- *** Bug 616193 has been marked as a duplicate of this bug. *** -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603481] Review Request: freerdp - remote desktop protocol client
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603481 Christoph Wickert changed: What|Removed |Added Blocks||617144 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603641] Review Request: python-fontMath - A set of objects for performing math operations on font data
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=603641
Thomas Spura changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #2 from Thomas Spura 2010-07-22
06:31:33 EDT ---
Review:
Good:
- name ok
- source match upstream: 739ecc87c553180c4978c110230be6f7
- $ rpmlint ./python-fontMath-0.2-1.fc13.src.rpm
noarch/python-fontMath-0.2-1.fc13.noarch.rpm
2 packages and 0 specfiles checked; 0 errors, 0 warnings.
- no libs
- no locales
- license ok
- macros everywhere
- spec readable
Needswork:
- you should use %{version} in the source url
- you should also buildrequire python-robofab
- wouldn't be Development/Tools be better as a group?
- %files:
You don't own %{python_sitelib}/fontMath -> remove the '*'
_
the %files issue is a blocker, the rest SHOULD.
Approving anyway, because I'm sure, you'll change it.
_
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.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603638] Review Request: python-defcon - A set of flexible objects for representing UFO data.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603638 --- Comment #2 from Thomas Spura 2010-07-22 06:18:46 EDT --- Review: Good: - name ok - source matches upstream: 738f62fea9d78b5225706dc7e1a562be - latest version packaged - $ rpmlint ./python-defcon-0.1-1.fc13.src.rpm noarch/python-defcon-0.1-1.fc13.noarch.rpm 2 packages and 0 specfiles checked; 0 errors, 0 warnings. Needswork: - *** Warning: defcon requires FontTools, see: fonttools.sf.net *** Warning: defcon requires RoboFab, see: robofab.com -> Missing BR: python-robofab and fonttools (last one also R) - Some tests are failing, does upstream know of this? Ran 78 tests in 0.641s FAILED (failures=5) - You could build the documentation and include that into a -doc subpackage. (I didn't found it on an upstream website.) - Are you sure, this is installed correctly? >From __init__.py: from defcon.objects.font import Font 9 from defcon.objects.glyph import Glyph, addRepresentationFactory, removeRepresentationFactory 10 from defcon.objects.contour import Contour But there is no defcon.objects. etc installed, just in the source... It seems python setuptools or the setup.py are wrong. :( -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 603638] Review Request: python-defcon - A set of flexible objects for representing UFO data.
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=603638 --- Comment #3 from Thomas Spura 2010-07-22 06:22:46 EDT --- Forgot to add: - license ok - no *.la - buildarch noarch ok - no libs - no locales -> false installing and BR are the blockers, doc is should -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
