[Bug 713584] Review Request: cogl - A library for using 3D graphics hardware to draw pretty pictures
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=713584 --- Comment #9 from Peter Robinson 2011-06-17 03:27:38 EDT --- (In reply to comment #8) > FYI: The *-devel package should own > /usr/include/cogl Fixed in git -- Configure 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 714036] New: Review Request: perl-PPI-PowerToys - Handy collection of small PPI-based utilities
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: perl-PPI-PowerToys - Handy collection of small PPI-based utilities https://bugzilla.redhat.com/show_bug.cgi?id=714036 Summary: Review Request: perl-PPI-PowerToys - Handy collection of small PPI-based utilities Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: ppi...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://ppisar.fedorapeople.org/perl-PPI-PowerToys/perl-PPI-PowerToys.spec SRPM URL: http://ppisar.fedorapeople.org/perl-PPI-PowerToys/perl-PPI-PowerToys-0.14-1.fc15.src.rpm Description: The PPI PowerToys are a small collection of utilities for working with Perl files, modules and distributions. -- Configure 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 714047] New: Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format https://bugzilla.redhat.com/show_bug.cgi?id=714047 Summary: Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sanjay.an...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://ankursinha.fedorapeople.org/nifticlib/nifticlib.spec SRPM URL: http://ankursinha.fedorapeople.org/nifticlib/nifticlib-2.0.0-1.fc15.src.rpm Description: Nifticlib is a set of C i/o libraries for reading and writing files in the nifti-1 data format. nifti-1 is a binary file format for storing medical image data, e.g. magnetic resonance image (MRI) and functional MRI (fMRI) brain images. -- Configure 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 714047] Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format
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=714047 --- Comment #1 from Ankur Sinha 2011-06-17 04:46:17 EDT --- [ankur@ankur SRPMS]$ rpmlint ../SPECS/nifticlib.spec nifticlib-2.0.0-1.fc15.src.rpm /var/lib/mock/fedora-rawhide-i386/result/*.rpm ../SPECS/nifticlib.spec:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/* ../SPECS/nifticlib.spec:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/ ../SPECS/nifticlib.spec: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found nifticlib.src: W: spelling-error Summary(en_US) nifti -> nifty nifticlib.src: W: spelling-error %description -l en_US nifti -> nifty nifticlib.src: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm nifticlib.src:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/* nifticlib.src:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/ nifticlib.src: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found nifticlib.i686: W: spelling-error Summary(en_US) nifti -> nifty nifticlib.i686: W: spelling-error %description -l en_US nifti -> nifty nifticlib.i686: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm nifticlib.i686: W: no-manual-page-for-binary nifti_stats nifticlib.i686: W: no-manual-page-for-binary nifti_tool nifticlib.i686: W: no-manual-page-for-binary nifti1_test nifticlib.src: W: spelling-error Summary(en_US) nifti -> nifty nifticlib.src: W: spelling-error %description -l en_US nifti -> nifty nifticlib.src: W: spelling-error %description -l en_US fMRI -> MRI, f MRI, firm nifticlib.src:53: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/* nifticlib.src:54: E: hardcoded-library-path in $RPM_BUILD_ROOT/usr/lib/ nifticlib.src: W: invalid-url Source0: http://downloads.sourceforge.net/nifticlib/nifticlib-2.0.0.tar.gz HTTP Error 404: Not Found nifticlib-devel.i686: W: no-documentation 6 packages and 1 specfiles checked; 6 errors, 16 warnings. The errors are irrelevant here. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714047] Review Request: nifticlib - A set of i/o libraries for reading and writing files in the nifti-1 data format
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=714047 Ankur Sinha changed: What|Removed |Added Alias||nifticlib -- Configure 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 683463] Review Request: trafficserver - Apache Traffic 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=683463 --- Comment #12 from Zhao Yongming 2011-06-17 06:14:06 EDT --- can someone give me some review and tell me what I need to do on this? we have aready put TS on FreeBSD ports, I don't want Linux to be too slow thanks guys. and here is the mock testing: [zym@unknown-10-62-163-x root]$ mock --rebuild rpmbuild/SRPMS/trafficserver-3.0.0-3.fc16.src.rpm INFO: mock.py version 1.1.10 starting... State Changed: init plugins INFO: selinux disabled State Changed: start INFO: Start(rpmbuild/SRPMS/trafficserver-3.0.0-3.fc16.src.rpm) Config(fedora-rawhide-x86_64) State Changed: lock buildroot State Changed: clean INFO: chroot (/var/lib/mock/fedora-rawhide-x86_64) unlocked and deleted State Changed: unlock buildroot State Changed: init State Changed: lock buildroot Mock Version: 1.1.10 INFO: Mock Version: 1.1.10 INFO: enabled root cache INFO: root cache aged out! cache will be rebuilt INFO: enabled yum cache State Changed: cleaning yum metadata INFO: enabled ccache State Changed: running yum State Changed: creating cache State Changed: unlock buildroot State Changed: setup State Changed: build INFO: Done(rpmbuild/SRPMS/trafficserver-3.0.0-3.fc16.src.rpm) Config(default) 11 minutes 45 seconds INFO: Results and/or logs in: /var/lib/mock/fedora-rawhide-x86_64/result State Changed: end [zym@unknown-10-62-163-x root]$ ls /var/lib/mock/fedora-rawhide-x86_64/result build.log trafficserver-3.0.0-3.fc16.src.rpm trafficserver-devel-3.0.0-3.fc16.x86_64.rpm root.log trafficserver-3.0.0-3.fc16.x86_64.rpm state.log trafficserver-debuginfo-3.0.0-3.fc16.x86_64.rpm [zym@unknown-10-62-163-x root]$ rpmlint /var/lib/mock/fedora-rawhide-x86_64/result/*.rpm trafficserver.src: I: enchant-dictionary-not-found en_US trafficserver.x86_64: W: hidden-file-or-dir /etc/trafficserver/body_factory/default/.body_factory_info trafficserver.x86_64: W: manual-page-warning /usr/share/man/man1/config_clock.1.gz 53: warning: macro `..' not defined trafficserver.x86_64: W: manual-page-warning /usr/share/man/man1/config_logging.1.gz 89: warning: macro `SS"custom' not defined (possibly missing space after `SS') trafficserver.x86_64: W: log-files-without-logrotate /var/log/trafficserver trafficserver.x86_64: W: no-manual-page-for-binary traffic_server trafficserver.x86_64: W: no-manual-page-for-binary traffic_manager trafficserver.x86_64: W: no-manual-page-for-binary traffic_line trafficserver.x86_64: W: no-manual-page-for-binary traffic_sac trafficserver.x86_64: W: no-manual-page-for-binary traffic_cop trafficserver.x86_64: W: no-manual-page-for-binary traffic_logstats trafficserver.x86_64: W: no-manual-page-for-binary traffic_logcat trafficserver.x86_64: W: one-line-command-in-%postun /sbin/ldconfig trafficserver.x86_64: W: service-default-enabled /etc/init.d/trafficserver trafficserver-devel.x86_64: W: no-documentation trafficserver-devel.x86_64: W: no-manual-page-for-binary tsxs 4 packages and 0 specfiles checked; 0 errors, 15 warnings. [zym@unknown-10-62-163-x 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 683463] Review Request: trafficserver - Apache Traffic 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=683463
--- Comment #13 from Jan-Frode Myklebust 2011-06-17
06:37:38 EDT ---
Only thing I find strange currently is that you install the daemon binaries to
/usr/bin instead of /usr/sbin/. Is that intentional ? Hmm.. I see the Gentoo
layout used has defined "sbindir: ${exec_prefix}/sbin" -- so it's strange it's
not putting anything there.
Also, it would probably be nice to restart ts on upgrades in %postun:
if [ $1 -eq 1 ] ; then
/sbin/service trafficserver condrestart &>/dev/null || :
fi
I think it's starting to look quite good, and I think it would be great to get
it into the package db as soon as possible to ease builds and also get more
testers. Unfortunately I'm just an EPEL packager, don't know if my reviews
counts for anything.
--
Configure 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 693425] Review Request: openerp - OpenERP business application
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=693425
--- Comment #58 from Alec Leamas 2011-06-17 06:51:33
EDT ---
Server review:
MUST
rpmlint must be run on the source rpm and all binary rpms...NOK
Seems that all warnings could be removed:
- The spelling errors could be handled by a hyphen
(addons -> add-ons, the US folks are just strange).
- Expand all tabs in specfile to blanks.
- Remove comment line w macros (29).
- Fix patches (see below)
- Don't use macros in changelog, expand them "by hand".
The spec file name must match the base package %{name}OK
The package must meet the Packaging Guidelines. TBD
Dependencies: Havn't really time for this ATM, but I notice
that much is improved. Let's presume it's ok
The package must be licensed with a Fedora approved license...: NOK
Once again, without checking the details, the License Field
in the spec does not reflect the breakdown in debian/copyright
See http://fedoraproject.org/wiki/Packaging:LicensingGuidelines
Separate License file must be in %doc...: NOK
- The only reasonable breakdown, debian/copyright is not present
The spec file must be written in American English: OK
To my understanding :)
The spec file for the package MUST be legible: OK
The build sources must match the upstream source...NOK
You have used patched source, not OK. Use the orignal source
and apply the patches in the spec file instead. It's easy
to remove once they are accepted. This will also fix the last
rpmlint warnings.
The package MUST successfully compile and build into binary rpms...: OK
Builds OK on f14 and f15.
f14: http://koji.fedoraproject.org/koji/taskinfo?taskID=3136833
f15: http://koji.fedoraproject.org/koji/taskinfo?taskID=3136871
All build dependencies must be listed in BuildRequires...OK
The spec file MUST handle locales properly...: OK
Packages must NOT bundle copies of system libraries: NOK
- contains bundled ftpserver (in Fedora: pyftpdlib)
- contains bundled rml2pdf (in Fedora: python-trml2pdf)
A package must own all directories that it creates. OK
Package must not list a file more than once in %file ..: OK
Permissions on files must be set properly...: OK
Each package must consistently use macros: OK
The package must contain code, or permissable content: OK
%doc must not affect the runtime of the application...: OK
Header files must be in a -devel package N/a
Packages must not own files or dir's owned by other ones...OK
All filenames in rpm packages must be valid UTF-8.: OK
SHOULD:
Package built on koji/mock: OK
Testing...TBD
Scriptlets should be sane: OK
(The test that update-desktop-database exists is not required:)
SUMMARY:
Besides minor editing in spec file the open issues are the bundled
libs, the use of patched source and that the different licenses
are not handled properly.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693425] Review Request: openerp - OpenERP business application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693425 --- Comment #59 from Alec Leamas 2011-06-17 06:55:17 EDT --- Client: I wait until a new variant is submitted, presuming that the the main server remarks are valid also for the client -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 712705] Review Request: gnome-shell-extension-icon-manager - A gnome-shell extension to manage icons in top bar
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=712705 Kevin Verma changed: What|Removed |Added CC||kevinve...@fedoraproject.or ||g --- Comment #4 from Kevin Verma 2011-06-17 07:07:22 EDT --- (In reply to comment #3) > Updated : > > Spec URL: > http://llaumgui.fedorapeople.org/review/gnome-shell-extension-icon-manager/gnome-shell-extension-icon-manager.spec > SRPM URL: > http://llaumgui.fedorapeople.org/review/gnome-shell-extension-icon-manager/gnome-shell-extension-icon-manager-0-0.1.gited463a5.fc15.src.rpm > RPM URL: > http://llaumgui.fedorapeople.org/review/gnome-shell-extension-icon-manager/gnome-shell-extension-icon-manager-0-0.1.gited463a5.fc15.noarch.rpm I tested the updated RPM and found that this extension does move task tray items out of messageTray into the top-pannel. Following the text of my configuration for org.gnome.shell.extentsions.icon-manager using dconf-editor: --snip-- ['a11y', 'gnote', 'parcellite', 'empathy', 'pidgin', 'workrave'] --snip-- I have only used this rpm to see if this extension works for given purpose, but I have not reviewed the RPM in particular. -- Configure 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 701332] Review Request: ghc-blaze-html - Fast HTML combinator library for Haskell
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=701332 --- Comment #4 from Jon Ciesla 2011-06-17 08:02:55 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713701] Review Request: tamil-typing-booster - Tamil typing booster table for ibus indic table
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=713701 --- Comment #3 from Jon Ciesla 2011-06-17 08:04:21 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693425] Review Request: openerp - OpenERP business application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693425 --- Comment #60 from Panos Christeas 2011-06-17 08:00:38 EDT --- > Packages must NOT bundle copies of system libraries: NOK > - contains bundled rml2pdf (in Fedora: python-trml2pdf) That's a bit unfair to OpenERP, as I have told you: the original lib is what we bundle, while "python-trml2pdf" one of Fedora is an old ver+hacks. > The build sources must match the upstream source...NOK The tar.gz's used are taken from: http://nightly.openerp.com/6.0/ so, they are what I expect to be tagged '6.0.3' soon. -- Configure 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 620112] Review Request: udpxy - UDP-to-HTTP multicast traffic relay daemon
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=620112 --- Comment #17 from Ivan Afonichev 2011-06-17 08:03:49 EDT --- Problem with Type=forking exists :( but reproduces rather rarely [root@vannew ~]# systemctl status udpxy.service udpxy.service - UDP-to-HTTP multicast traffic relay daemon Loaded: loaded (/lib/systemd/system/udpxy.service) Active: active (running) since Fri, 17 Jun 2011 15:49:55 +0400; 8min ago Process: 26428 ExecStart=/usr/bin/udpxy $OPTIONS (code=exited, status=0/SUCCESS) Main PID: 26430 (udpxy) CGroup: name=systemd:/system/udpxy.service └ 26430 /usr/bin/udpxy -S -p 4022 -a 127.0.0.1 -c 16 [root@vannew ~]# systemctl stop udpxy.service [root@vannew ~]# systemctl status udpxy.service udpxy.service - UDP-to-HTTP multicast traffic relay daemon Loaded: loaded (/lib/systemd/system/udpxy.service) Active: inactive (dead) since Fri, 17 Jun 2011 16:01:39 +0400; 2s ago Process: 26428 ExecStart=/usr/bin/udpxy $OPTIONS (code=exited, status=0/SUCCESS) Main PID: 26430 (code=exited, status=0/SUCCESS) CGroup: name=systemd:/system/udpxy.service [root@vannew ~]# systemctl start udpxy.service [root@vannew ~]# systemctl status udpxy.service udpxy.service - UDP-to-HTTP multicast traffic relay daemon Loaded: loaded (/lib/systemd/system/udpxy.service) Active: active (running) since Fri, 17 Jun 2011 16:01:48 +0400; 4s ago Process: 26539 ExecStart=/usr/bin/udpxy $OPTIONS (code=exited, status=0/SUCCESS) Main PID: 26541 (udpxy) CGroup: name=systemd:/system/udpxy.service └ 26541 /usr/bin/udpxy -S -p 4022 -a 127.0.0.1 -c 16 -- Configure 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 620112] Review Request: udpxy - UDP-to-HTTP multicast traffic relay daemon
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=620112 --- Comment #18 from nucleo 2011-06-17 08:15:46 EDT --- I don't see any reason why not to use Type=simple with -T option as always working solution (I tested both forking and simple many times). -- Configure 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 714141] New: Review Request: rubygem-multipart - Add multipart (file upload) support to Net::HTTP::Post
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-multipart - Add multipart (file upload) support to Net::HTTP::Post https://bugzilla.redhat.com/show_bug.cgi?id=714141 Summary: Review Request: rubygem-multipart - Add multipart (file upload) support to Net::HTTP::Post Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: vondr...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://people.redhat.com/vondruch/rubygem-multipart.spec SRPM URL: http://people.redhat.com/vondruch/rubygem-multipart-0.2.1-1.fc16.src.rpm Description: Multipart is a gem that adds support to multipart/form-encoded and multipart/mixed (file upload) to Net::HTTP::Post. Nothing more, nothing less. Currently it supports a file param with multiple files, but not multiple file params. Koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=3137438 -- Configure 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 712705] Review Request: gnome-shell-extension-icon-manager - A gnome-shell extension to manage icons in top bar
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=712705 --- Comment #5 from Mohamed El Morabity 2011-06-17 08:41:46 EDT --- The description is not true: the extension is configured through gsettings, not gconf. Plese fix it... And report the issue upstream. -- Configure 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 712705] Review Request: gnome-shell-extension-icon-manager - A gnome-shell extension to manage icons in top bar
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=712705 Mohamed El Morabity changed: What|Removed |Added Status|NEW |ASSIGNED AssignedTo|nob...@fedoraproject.org|pikachu.2...@gmail.com Flag||fedora-review? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714141] Review Request: rubygem-multipart - Add multipart (file upload) support to Net::HTTP::Post
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=714141 Vít Ondruch changed: What|Removed |Added Blocks||705508 -- Configure 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 683463] Review Request: trafficserver - Apache Traffic 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=683463 Daniel Walsh changed: What|Removed |Added CC||dwa...@redhat.com --- Comment #14 from Daniel Walsh 2011-06-17 08:58:29 EDT --- What is /etc/trafficserver/* Are these config files? I am trying to write some SELinux policy and I see traffic_manager modifying content in this directory /etc/trafficserver/log_hosts.config_2 Any reason these files are not in /var/lib/trafficserver? /etc should be treated as readonly. -- Configure 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 683463] Review Request: trafficserver - Apache Traffic 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=683463 --- Comment #15 from Jan-Frode Myklebust 2011-06-17 09:20:24 EDT --- Yes, /etc/trafficserver/* are configfiles, but I believe they can be modified by the traffic_manager (or traffic_server?), so it might be good to make it an selinux optional if one wants to allow this. Every time the configfiles are changed, the old version is rewritten to name.config_number+1. The traffic_manager is also a cluster manager, so config changes on one server can automatically be replicated to the other servers. -- Configure 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 713923] Review Request: heybuddy - identi.ca and status.net 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=713923 --- Comment #7 from Heiko Adams 2011-06-17 09:30:48 EDT --- Scratch builds can be found here: http://koji.fedoraproject.org/koji/taskinfo?taskID=3136853 -- Configure 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 639518] Review Request: gst-entrans - Plug-ins and tools for transcoding and recording with GStreamer
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=639518 --- Comment #9 from Remi Collet 2011-06-17 10:18:51 EDT --- Sorry for the long delay. Good work on the "split". Final minor issues. Licensing, See : http://fedoraproject.org/wiki/Packaging:LicensingGuidelines#Subpackage_Licensing So : - gstreamer-plugins-entrans-docs doesn't requires gst-entrans (which is ok), so you must add the license file in this sub-package. - gstreamer-plugins-entrans could also be install alone, so must also add the license file. - gst-entrans requires gstreamer-plugins-entrans, so license is not required (but you can keep it, with AUTHORS file, it's your choice) Move BuildRequires: gtk-doc To gstreamer-plugins-entrans-docs sub-package I think there is no legal issue, as no codec are provided. Notice : for the /usr/share/gtk-doc/html directory ownership, guidelines allow 2 solutions. - require an owner (gtk-doc) - own the directory (to avoid to require more packages) Both are ok. It's "your" choice (I think "gtk-doc" is required to generate the doc, not to read it...) Everything else seems ok. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713920] Review Request: csparse - Sparse Matrix software 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=713920 --- Comment #2 from Tom "spot" Callaway 2011-06-17 10:22:06 EDT --- Fair enough: Spec URL: http://www.auroralinux.org/people/spot/review/new/CSparse.spec SRPM URL: http://www.auroralinux.org/people/spot/review/new/CSparse-3.0.1-1.fc15.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 713920] Review Request: CSparse - Sparse Matrix software 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=713920 Tom "spot" Callaway changed: What|Removed |Added Summary|Review Request: csparse - |Review Request: CSparse - |Sparse Matrix software |Sparse Matrix software |library |library -- Configure 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 710137] Review Request: gedit-cossa - gedit plugin for GTK+ themes design
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=710137 Ignacio Casal Quinteiro (nacho) changed: What|Removed |Added Flag||fedora-cvs? --- Comment #6 from Ignacio Casal Quinteiro (nacho) 2011-06-17 10:45:22 EDT --- New Package SCM Request === Package Name: gedit-cossa Short Description: gedit plugin for GTK+ themes previewing Owners: nacho Branches: f15 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 706359] Review Request: lsx - List executables in a directory tree
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=706359 Pavel Alexeev (aka Pahan-Hubbitus) changed: What|Removed |Added CC||pa...@hubbitus.info --- Comment #8 from Pavel Alexeev (aka Pahan-Hubbitus) 2011-06-17 10:50:03 EDT --- Sorry, but not is it just equivalent to: find -type f -executable There much more powerfull also. -- Configure 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 710137] Review Request: gedit-cossa - gedit plugin for GTK+ themes design
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=710137 --- Comment #7 from Jon Ciesla 2011-06-17 10:53:49 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225974] Merge Review: krb5
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=225974 --- Comment #18 from Jon Ciesla 2011-06-17 10:58:29 EDT --- Any updates? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225615] Merge Review: binutils
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=225615 --- Comment #18 from Jon Ciesla 2011-06-17 10:57:50 EDT --- Ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 706359] Review Request: lsx - List executables in a directory tree
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=706359 --- Comment #9 from Petr Sabata 2011-06-17 10:56:41 EDT --- (In reply to comment #8) > Sorry, but not is it just equivalent to: > find -type f -executable > > There much more powerfull also. No, this is not recursive :) This is a simple tool for a simple thing. Try using that find construct every day -- you'll have to create an alias or a script. Or maybe a binary? And here we are. Also, this is going to be a dependency for future versions of dmenu, if you need some justifications for it to be in Fedora. -- Configure 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 226117] Merge Review: mailman
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226117 --- Comment #31 from Jon Ciesla 2011-06-17 10:58:38 EDT --- Ping? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713925] Review Request: naev - 2D space trading and combat game
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713925 --- Comment #1 from Tom "spot" Callaway 2011-06-17 11:00:43 EDT --- Updated SPEC and SRPM to reflect name change of CSparse: New Spec URL: http://www.auroralinux.org/people/spot/review/new/naev.spec New SRPM URL: http://www.auroralinux.org/people/spot/review/new/naev-0.5.0-2.fc15.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 226204] Merge Review: nss
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=226204 --- Comment #10 from Jon Ciesla 2011-06-17 11:03:07 EDT --- I see this is now emaldonado's package? Can you have a look at this so we can get it put to bed? Thanks! -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 524119] Review Request: nmon - Nigel's performance MONitor for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=524119 J.H.M. Dassen (Ray) changed: What|Removed |Added Blocks||714194 -- Configure 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 225974] Merge Review: krb5
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=225974 --- Comment #19 from Nalin Dahyabhai 2011-06-17 11:30:12 EDT --- Whoops, yes, that was fixed in 1.9-9, if I'm reading the changelog right. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 697836] Review Request: mmseq - Haplotype and isoform specific expression estimation for RNA-seq
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=697836 Fedora Update System changed: What|Removed |Added Status|ON_QA |MODIFIED -- Configure 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 697836] Review Request: mmseq - Haplotype and isoform specific expression estimation for RNA-seq
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=697836 --- Comment #13 from Fedora Update System 2011-06-17 11:26:52 EDT --- mmseq-0.9.11-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/mmseq-0.9.11-1.el5 -- Configure 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 693231] Review Request: gedit-collaboration - collaboration plugin for gedit
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=693231 Tomáš Bžatek changed: What|Removed |Added Flag||fedora-review+ --- Comment #10 from Tomáš Bžatek 2011-06-17 11:27:43 EDT --- (In reply to comment #9) > The correct URL is http://projects.gnome.org/gedit Corrected. Thank you very much for the review! -- Updated files: Spec URL: http://tbzatek.fedorapeople.org/gedit-collaboration/gedit-collaboration.spec SRPM URL: http://tbzatek.fedorapeople.org/gedit-collaboration/gedit-collaboration-3.0.0-1.fc16.src.rpm Description: Collaboration plugin for gedit -- Configure 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 225974] Merge Review: krb5
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=225974 Jon Ciesla changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NOTABUG Flag||fedora-review+ Last Closed||2011-06-17 11:32:37 --- Comment #20 from Jon Ciesla 2011-06-17 11:32:37 EDT --- Awesome, thanks! APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693231] Review Request: gedit-collaboration - collaboration plugin for gedit
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=693231 Tomáš Bžatek changed: What|Removed |Added Flag||fedora-cvs? --- Comment #11 from Tomáš Bžatek 2011-06-17 11:30:57 EDT --- New Package SCM Request === Package Name: gedit-collaboration Short Description: Collaboration plugin for gedit Owners: nacho tbzatek Branches: f15 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 426387] Merge reviews to be completed for F9
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=426387 Bug 426387 depends on bug 225974, which changed state. Bug 225974 Summary: Merge Review: krb5 https://bugzilla.redhat.com/show_bug.cgi?id=225974 What|Old Value |New Value Resolution||NOTABUG Status|ASSIGNED|CLOSED -- Configure 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 225626] Merge Review: brltty
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=225626 --- Comment #8 from Jon Ciesla 2011-06-17 11:42:43 EDT --- Ok, retrying on 4.2-4. rpm-buildroot-usage error is still there. synthesizers is misspelt in the description. Application is misspelt in the summary. Lots of file encoding and non-conffile-in-etc errors. private-shared-object-provides for /usr/lib/python2.7/site-packages/brlapi.so dangerous command in post still applies. no documentation warnings for python and tcl subpackages. Please fix or document the above. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713923] Review Request: heybuddy - identi.ca and status.net 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=713923
Julian Aloofi changed:
What|Removed |Added
Blocks||177841(FE-NEEDSPONSOR)
--- Comment #8 from Julian Aloofi 2011-06-17
11:42:20 EDT ---
(In reply to comment #5)
> I'm not yet a packager. I'm just a noob knocking on the door
Oh, okay. That means I can't do the review, because you have to find a sponsor
first. I'll set the FE-NEEDSPONSOR flag so sponsors will notice the bug and
know that you need one.
To make the review a bit easier and get you something to do while you're
waiting (it can take a while until some sponsor finds time on his hands to
guide you through the process), I'll just make a few quick comments on the spec
file:
- the language packs are overkill for such a small package, you can just leave
them out and include all translations in the main package :)
- Python packages must not Require: python, but set a BuildRequires on
python2-devel. The Requires on gtk2 is also not needed, as it gets pulled in by
pygtk2 anyway.
- why do you pull the source from bzr? It is much easier to maintain the
package if you use upstream release tarballs (currently version 0.2.1)
- rather use 644 instead of 755 as permissions when installing, these files
don't need to be executable
- putting all .py files into /usr/bin is not a good solution. Rather install a
shell script to /usr/bin that starts heybuddy (called "heybuddy") and put the
.py files to /usr/share/heybuddy/ (the macro for /usr/share/ is %{_datadir}).
Remember you'll have to change the "Exec=" in the .desktop file then.
- you should rather install the icon to /usr/share/pixmaps and call it
heybuddy.png. This way, you can just use Icon=heybuddy in the desktop file and
it will work out.
I hope this is not too discouraging to begin with :) If you have any questions
don't hesitate to ask.
You might also be interested in these two links (the list appears to be long,
but you'll get the feeling for how to do things in a little while ;) )
http://fedoraproject.org/wiki/Packaging/Guidelines
http://fedoraproject.org/wiki/Packaging:Python
--
Configure 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 225619] Merge Review: bluez-hcidump
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=225619 Jon Ciesla changed: What|Removed |Added Flag|fedora-review? |fedora-review+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225619] Merge Review: bluez-hcidump
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=225619 Jon Ciesla changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NOTABUG Last Closed||2011-06-17 11:49:00 --- Comment #8 from Jon Ciesla 2011-06-17 11:49:00 EDT --- Retrying on 2.0-2. rpmlint clean except for spelling errors and zero-length NEWS. You could remove that, but it's not critical. Otherwise it's good, the Source URL is good now. APPROVED. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693425] Review Request: openerp - OpenERP business application
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=693425 --- Comment #61 from Alec Leamas 2011-06-17 11:52:02 EDT --- As for rml2pdf, I think we agree on the overall situation, you are certainly right (and my phrasing is not really ideal) OTOH, from the distribution perspective, the simple fact is that openerp contains a modified version of something which already exists in Fedora, and this needs to be resolved. Let's hope the exception is granted! I'm just making a review, and one duty is to check the source vs the source URL. And they don't match. I'm sure they will, next time :) However, the requirement for the source url to match the actual source is not negotiable to my understanding. -- Configure 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 708535] Review Request: xenserverjava - Java SDK for XenServer
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=708535 David Nalley changed: What|Removed |Added Flag||fedora-cvs? --- Comment #2 from David Nalley 2011-06-17 12:07:33 EDT --- New Package SCM Request === Package Name: xenserverjava Short Description: Java SDK for XenServer Owners: ke4qqq Branches: f15, el6 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 226117] Merge Review: mailman
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=226117 Nils Philippsen changed: What|Removed |Added CC||jkal...@redhat.com --- Comment #32 from Nils Philippsen 2011-06-17 12:02:54 EDT --- Adding the current maintainer to Cc. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 693231] Review Request: gedit-collaboration - collaboration plugin for gedit
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=693231 --- Comment #12 from Jon Ciesla 2011-06-17 12:07:49 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 226157] Merge Review: module-init-tools
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=226157 --- Comment #15 from Jon Ciesla 2011-06-17 12:05:50 EDT --- Looking at 3.16-0.1 Lots of macros in the changelog. unversioned explicit obsoletes for modutils, -devel. Self-obsoletion of modutils Invalid URL for Source0 and Source1. Summary ends with dot. No URL tag. Several non-conffile in /etc. No man pages for insmod.static and weak-modules. Dangerous command in %pre, mv. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 713443] Review Request: yelp-tools - Create, manage, and publish documentation for Yelp
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=713443 Matthias Clasen changed: What|Removed |Added Status|NEW |CLOSED Resolution||RAWHIDE Last Closed||2011-06-17 12:14:03 --- Comment #7 from Matthias Clasen 2011-06-17 12:14:03 EDT --- Build is done by now, so closing this out -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 708535] Review Request: xenserverjava - Java SDK for XenServer
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=708535 --- Comment #3 from Jon Ciesla 2011-06-17 12:35:11 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714231] New: Review Request: mediawiki116-semantic - The semantic extension to mediawiki
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: mediawiki116-semantic - The semantic extension to mediawiki https://bugzilla.redhat.com/show_bug.cgi?id=714231 Summary: Review Request: mediawiki116-semantic - The semantic extension to mediawiki Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: jla...@redhat.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, ke...@scrye.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Clone Of: 490001 This is a review request for a compat version of mediawiki-semantic (originally reviewed in bug#490001) that can work with mediawiki116. Spec URL: http://jlaska.fedorapeople.org/rpms/mediawiki116-semantic.spec SRPM URL: http://jlaska.fedorapeople.org/rpms/mediawiki116-semantic-1.5.6-1.fc16.src.rpm Description: Semantic MediaWiki (SMW) is a free, open-source extension to MediaWiki – the wiki software that powers Wikipedia – that lets you store and query data within the wiki's pages. Semantic MediaWiki is also a full-fledged framework, in conjunction with many spinoff extensions, that can turn a wiki into a powerful and flexible “collaborative database”. All data created within SMW can easily be published via the Semantic Web, allowing other systems to use this data seamlessly. * koji scratch build (dist-f16) - http://koji.fedoraproject.org/koji/taskinfo?taskID=3138514 * rpmlint output [1] mediawiki116-semantic.noarch: W: spelling-error %description -l en_US spinoff -> spin off, spin-off, Spinoza [2] mediawiki116-semantic.noarch: E: incorrect-fsf-address /usr/share/doc/mediawiki116-semantic-1.5.6/COPYING [3] mediawiki116-semantic.src: W: spelling-error %description -l en_US spinoff -> spin off, spin-off, Spinoza [4] 2 packages and 1 specfiles checked; 1 errors, 2 warnings. [1,3] These are just warnings and I don't believe they are critical [2] I have notified upstream - https://bugzilla.wikimedia.org/show_bug.cgi?id=29458 -- Configure 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 713923] Review Request: heybuddy - identi.ca and status.net 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=713923 --- Comment #9 from Heiko Adams 2011-06-17 12:53:54 EDT --- Thanks for your tipps. I've updated the spec-file and srpm: Spec URL: https://dl.dropbox.com/s/oqch96tll8f22j8/heybuddy.spec?dl=1 SRPM URL: https://dl.dropbox.com/s/cvpjx7yheflau63/heybuddy-0.2.2bzr-3.20110616.fc15.src.rpm?dl=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 713122] Review Request: flyback - Git based backup software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713122 --- Comment #5 from Heiko Adams 2011-06-17 13:09:50 EDT --- Update: Spec URL: https://dl.dropbox.com/s/zjkybrv5vywsgod/flyback.spec?dl=1 SRPM URL: https://dl.dropbox.com/s/06n9mj1w2zwawe8/flyback-0.6.5-3.fc15.src.rpm?dl=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 713122] Review Request: flyback - Git based backup software
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=713122 Heiko Adams changed: What|Removed |Added Summary|Review Request: flyback - |Review Request: flyback - |Aplle's Timemachine for |Git based backup software |Linux | -- Configure 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 712203] Review Request: eclipse-mercurial - Mercurial plugin for Eclipse -- NEEDSPONSOR
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=712203 minoo ziaei changed: What|Removed |Added Summary|Review Request: |Review Request: |eclipse-mercurial - |eclipse-mercurial - |Mercurial plugin for|Mercurial plugin for |Eclipse |Eclipse -- 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963
leigh scott changed:
What|Removed |Added
CC||leigh123li...@googlemail.co
||m
--- Comment #1 from leigh scott 2011-06-17
13:38:09 EDT ---
this needs changing from
%files -n pyrpkg
%doc COPYING README
# For noarch packages: sitelib
%{python_sitelib}/*
to
%files -n pyrpkg
%defattr(-,root,root,-)
%doc COPYING README
%{python_sitelib}/pyrpkg
%{python_sitelib}/rpkg-1.0-py?.?.egg-info
as far as I know you should remove the comment from "rm -rf $RPM_BUILD_ROOT" in
the install section.
make these changes and I will approve it
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 712203] Review Request: eclipse-mercurial - Mercurial plugin for Eclipse
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=712203 minoo ziaei changed: What|Removed |Added Blocks||177841(FE-NEEDSPONSOR) Summary|Review Request: |Review Request: |eclipse-mercurial - |eclipse-mercurial - |Mercurial plugin for|Mercurial plugin for |Eclipse -- NEEDSPONSOR |Eclipse -- Configure 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 712203] Review Request: eclipse-mercurial - Mercurial plugin for Eclipse
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=712203 --- Comment #16 from Andrew Overholt 2011-06-17 14:09:17 EDT --- I've sponsored Minoo. -- Configure 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 712203] Review Request: eclipse-mercurial - Mercurial plugin for Eclipse
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=712203 Andrew Overholt 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963
--- Comment #2 from leigh scott 2011-06-17
13:48:34 EDT ---
Another point
BuildArch: noarch
BuildRequires: python-devel, python-setuptools
# We br these things for man page generation due to imports
BuildRequires: GitPython, koji, python-pycurl
%if 0%{?rhel} == 5 || 0%{?rhel} == 4
BuildRequires: python-hashlib
BuildRequires: python-argparse
%endif
needs to be
BuildArch: noarch
BuildRequires: python-devel, python-setuptools
# We br these things for man page generation due to imports
BuildRequires: GitPython, koji, python-pycurl
%if 0%{?rhel} == 6 || 0%{?rhel} == 4
BuildRequires: python-hashlib
BuildRequires: python-argparse
%endif
to build on RHEL6
--
Configure 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 683463] Review Request: trafficserver - Apache Traffic 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=683463 --- Comment #16 from Daniel Walsh 2011-06-17 13:46:34 EDT --- So all files in /etc/trafficserver can be modified by the traffic_manager? -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #3 from Jesse Keating 2011-06-17 14:20:34 EDT --- Good catches. That's what I get for uploading specs late at night. New stuff uploaded: Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.0-2.f14.src.rpm the defattr stuff is actually not necessary anymore. Rpm since 4.4 has a sane default. -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #4 from leigh scott 2011-06-17 14:32:36 EDT --- (In reply to comment #3) > the defattr stuff is actually not necessary anymore. Rpm since 4.4 has a sane > default. Thanks for the info. Sadly rpmlint on el6 still sees it as a error (when I get time I will file a bugreport against rpmlint). [leigh@main_pc SPECS]$ rpmlint rpkg-1.0-1.el6.src.rpm rpkg.src: I: enchant-dictionary-not-found en_US rpkg.src:74: E: files-attr-not-set rpkg.src:75: E: files-attr-not-set rpkg.src:76: E: files-attr-not-set P.S you forgot to update the spec file -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 710777] Review Request: foobnix - Simple and Powerful music player for Linux
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=710777 Pavel Alexeev (aka Pahan-Hubbitus) changed: What|Removed |Added CC||pa...@hubbitus.info Bug 710777 depends on bug 710772, which changed state. Bug 710772 Summary: Review Request: pylast - A Python interface to Last.fm API compatible social networks https://bugzilla.redhat.com/show_bug.cgi?id=710772 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED Bug 710777 depends on bug 710774, which changed state. Bug 710774 Summary: Review Request: python-vkontakte - vk.com (aka vkontakte.ru) API wrapper https://bugzilla.redhat.com/show_bug.cgi?id=710774 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED Bug 710777 depends on bug 710773, which changed state. Bug 710773 Summary: Review Request: python-lyricwiki - A Python interface to the http://lyrics.wikia.com https://bugzilla.redhat.com/show_bug.cgi?id=710773 What|Old Value |New Value Resolution||ERRATA Status|ON_QA |CLOSED -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #5 from Jesse Keating 2011-06-17 14:43:12 EDT --- Do you have a proxy or something setup? I did put a new spec file up there, and just checked it in my browser, it has the right content. -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #6 from Jesse Keating 2011-06-17 14:53:42 EDT --- Make that: Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.1-1.f14.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 226204] Merge Review: nss
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=226204 --- Comment #11 from Elio Maldonado Batiz 2011-06-17 14:52:51 EDT --- (In reply to comment #10) > I see this is now emaldonado's package? Can you have a look at this so we can > get it put to bed? Thanks! Hi Jon, What is the propsal in question that you would like me to comment on? The nss build system has changed substantially since you last reprted problems. Since 3.12.4 and Fedora-12 we have split nss into three packages: nss-util, nss-softokn, and nss. I'm sure that an rpmlinit run againts the latest srps will show different results. Quite a few error message still show up and I will be happy to discuss them with you. Some are in my list of things to fix but I am waiting to an opportune time to tacke them. By opprtune I mean very early in the relase cycle so that I can work with maintainers of packages that depend on nss to ensure we don't break them or anyone else. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are the QA contact for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #9 from Ryan Rix 2011-06-17 15:06:24 EDT --- Package Change Request == Package Name: python-netifaces New Branches: el5 el6 Owners: rrix InitialCC: Need epel branches. -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 leigh scott changed: What|Removed |Added Flag||fedora-review+ --- Comment #8 from leigh scott 2011-06-17 15:14:10 EDT --- source sha1sum matches upstream spec file conforms to packaging guide lines licence is acceptable and correct builds ok @ koji for F16 http://koji.fedoraproject.org/koji/taskinfo?taskID=3139353 Package 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #7 from Jesse Keating 2011-06-17 15:13:34 EDT --- Ahem. Spec URL: http://jkeating.fedorapeople.org/review/rpkg.spec SRPM URL: http://jkeating.fedorapeople.org/review/rpkg-1.1-2.f14.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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 leigh scott changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|leigh123li...@googlemail.co ||m -- Configure 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 665853] Review Request: h5py - A Python interface to the HDF5 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=665853
Steve Traylen changed:
What|Removed |Added
Flag|fedora-review? |fedora-review+
--- Comment #11 from Steve Traylen 2011-06-17 15:16:31
EDT ---
Builds in mock f15 x86_64.
rpmlint results:
$ rpmlint ./h5py.spec /var/lib/mock/fedora-15-x86_64/result/*.rpm
./h5py.spec: W: invalid-url Source0:
http://h5py.googlecode.com/files/h5py-1.3.1.tar.gz HTTP Error 404: Not Found
h5py.src: W: invalid-url Source0:
http://h5py.googlecode.com/files/h5py-1.3.1.tar.gz HTTP Error 404: Not Found
this is clean, known problem with googlecode.
- Package meets naming and packaging guidelines
- Spec file matches base package name.
Yes named after tar ball.
- Spec has consistant macro usage.
They are.
- Meets Packaging Guidelines.
Yes.
- License
BSD
- License field in spec matches
License looks to be consistantly BSD
- License file included in package
Yes license directory included.
- Spec in American English
yes
- Spec is legible.
yes
- Sources match upstream md5sum:
$ md5sum h5py-1.3.1.tar.gz ../SOURCES/h5py-1.3.1.tar.gz
cfef84992d33910a06371dc35becb71b h5py-1.3.1.tar.gz
cfef84992d33910a06371dc35becb71b ../SOURCES/h5py-1.3.1.tar.gz
- Package needs ExcludeArch
It odes not
- BuildRequires correct
Look good and passes mock
- Spec handles locales/find_lang
Not important.
- Package is relocatable and has a reason to be.
Not relocatable.
- Package has %defattr and permissions on files is good.
Fine
- Package has a correct %clean section.
Fine
- Package has correct buildroot
%{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n)
- Package is code or permissible content.
Yes
- Doc subpackage needed/used.
Not needed
- Packages %doc files don't affect runtime.
Thet don't
- Headers/static libs in -devel subpackage.
Not needed.
- Spec has needed ldconfig in post and postun
Not Needed.
- .pc files in -devel subpackage/requires pkgconfig
Not Needed.
- .so files in -devel subpackage.
Not Needed.
- -devel package Requires: %{name} = %{version}-%{release}
Not Needed.
- .la files are removed.
Not Needed.
- Package is a GUI app and has a .desktop file
Not Needed.
- Package compiles and builds on at least one arch.
Mock
- Package has no duplicate files in %files.
No
- Package doesn't own any directories other packages own.
It does not
- Package owns all the directories it creates.
It does.
- No rpmlint output.
See above.
- final provides and requires are sane:
They are indeed.
SHOULD Items:
- Should build in mock.
Yes
- Should build on all supported archs
Not checked but probablyu
- Should function as described.
%checks pass
- Should have sane scriptlets.
None
- Should have subpackages require base package with fully versioned depend.
Not relavent.
- Should package latest version
1.3.1 is newest except for a beta.
Issues:
None
Package 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #11 from Jon Ciesla 2011-06-17 15:58:47 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 Jon Ciesla changed: What|Removed |Added Flag|fedora-cvs+ |fedora-cvs? --- Comment #10 from Jon Ciesla 2011-06-17 15:57:41 EDT --- Setting the fedora-cvs flag. . . -- Configure 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 683463] Review Request: trafficserver - Apache Traffic 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=683463 --- Comment #17 from Jan-Frode Myklebust 2011-06-17 16:14:57 EDT --- dwalsh: I replied on the users@trafficserver mailinglist. Hopefull the developers there are following the discussion and can chime in with more knowledge. -- Configure 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #13 from Fedora Update System 2011-06-17 18:25:00 EDT --- python-netifaces-0.5-1.el5 has been submitted as an update for Fedora EPEL 5. https://admin.fedoraproject.org/updates/python-netifaces-0.5-1.el5 -- Configure 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #12 from Fedora Update System 2011-06-17 18:23:49 EDT --- python-netifaces-0.5-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/python-netifaces-0.5-1.el6 -- Configure 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #14 from Fedora Update System 2011-06-17 18:25:57 EDT --- python-netifaces-0.5-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/python-netifaces-0.5-1.fc15 -- Configure 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 709914] Review Request: python-netifaces: Python library to retrieve information about network interfaces
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=709914 --- Comment #15 from Fedora Update System 2011-06-17 18:27:11 EDT --- python-netifaces-0.5-1.fc14 has been submitted as an update for Fedora 14. https://admin.fedoraproject.org/updates/python-netifaces-0.5-1.fc14 -- Configure 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 708535] Review Request: xenserverjava - Java SDK for XenServer
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=708535 --- Comment #4 from Fedora Update System 2011-06-17 18:31:01 EDT --- xenserverjava-5.6.100.1-1.el6 has been submitted as an update for Fedora EPEL 6. https://admin.fedoraproject.org/updates/xenserverjava-5.6.100.1-1.el6 -- Configure 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 708535] Review Request: xenserverjava - Java SDK for XenServer
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=708535 --- Comment #5 from Fedora Update System 2011-06-17 18:31:08 EDT --- xenserverjava-5.6.100.1-1.fc15 has been submitted as an update for Fedora 15. https://admin.fedoraproject.org/updates/xenserverjava-5.6.100.1-1.fc15 -- Configure 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 708535] Review Request: xenserverjava - Java SDK for XenServer
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=708535 Fedora Update System changed: What|Removed |Added Status|ASSIGNED|MODIFIED -- Configure 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 Jesse Keating changed: What|Removed |Added Flag||fedora-cvs? --- Comment #9 from Jesse Keating 2011-06-17 19:20:19 EDT --- New Package SCM Request === Package Name: rpkg Short Description: Utility for interacting with rpm+git packaging systems Owners: jkeating Branches: f14 f15 el6 el5 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 693370] Review Request: elementary-gtk-theme - elementary GTK theme
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=693370 --- Comment #5 from Christoph Wickert 2011-06-17 19:44:43 EDT --- Hi Chris, do you agree with splitting 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 713963] Review Request: rpkg - Utility for interacting with rpm+git packaging systems
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=713963 --- Comment #10 from Jon Ciesla 2011-06-17 20:16:18 EDT --- Git done (by process-git-requests). -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ 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 #25 from Christoph Wickert 2011-06-17 22:25:46 EDT --- Tylon, I'd really like to finish this review and sponsor you. If you are busy of cannot maintain this package for other reasons, please let us know. -- Configure 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 714326] New: Review Request: libtpcmisc - Miscellaneous PET functions
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libtpcmisc - Miscellaneous PET functions https://bugzilla.redhat.com/show_bug.cgi?id=714326 Summary: Review Request: libtpcmisc - Miscellaneous PET functions Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sanjay.an...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://ankursinha.fedorapeople.org/libtpcmisc/libtpcmisc.spec SRPM URL: http://ankursinha.fedorapeople.org/libtpcmisc/libtpcmisc-1.4.8-1.fc15.src.rpm Description: Former libpet, the common PET C library, has been divided up in smaller sub-libraries that each handle a specific task. This library includes miscellaneous functions utilized in PET data processing. -- rpmlint solutions: [ankur@ankur ~]$ rpmlint /tmp/mock-repo/libtpcmisc-* ~/rpmbuild/SPECS/libtpcmisc.spec libtpcmisc.i686: W: spelling-error %description -l en_US libpet -> limpet, lib pet, lib-pet libtpcmisc.i686: W: no-documentation libtpcmisc.i686: W: no-manual-page-for-binary libtpcmisc libtpcmisc.src: W: spelling-error %description -l en_US libpet -> limpet, lib pet, lib-pet libtpcmisc-devel.i686: W: no-documentation 4 packages and 1 specfiles checked; 0 errors, 5 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714326] Review Request: libtpcmisc - Miscellaneous PET functions
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=714326 Ankur Sinha changed: What|Removed |Added Alias||libtpcmisc -- Configure 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 714327] Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures
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=714327 Ankur Sinha changed: What|Removed |Added Alias||libtpcimgio -- Configure 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 714327] New: Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures https://bugzilla.redhat.com/show_bug.cgi?id=714327 Summary: Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sanjay.an...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://ankursinha.fedorapeople.org/libtpcimgio/libtpcimgio.spec SRPM URL: http://ankursinha.fedorapeople.org/libtpcimgio/libtpcimgio-1.5.10-1.fc16.src.rpm Description: The libtpcimgio library is a collection of commonly used C files in Turku PET Centre for image file input and output procedures. Libtpcimgio library supports Analyze 7.5, Ecat 6.x, Ecat 7.x and partly interfile formats. [ankur@ankur ~]$ rpmlint /tmp/mock-repo/libtpcimgio-* ~/rpmbuild/SPECS/libtpcimgio.spec libtpcimgio.i686: W: spelling-error Summary(en_US) Turku -> Turk, Turks, Turk u libtpcimgio.i686: W: no-documentation libtpcimgio.i686: W: no-manual-page-for-binary libtpcimgio libtpcimgio.src: W: spelling-error Summary(en_US) Turku -> Turk, Turks, Turk u libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/sif.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_ana.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_e63.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7r.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/analyze.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat63p.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/libtpcimgiov.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgunit.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat63w.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/img_e7.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/sifio.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7w.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgmax.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/ecat7ml.c libtpcimgio-debuginfo.i686: E: incorrect-fsf-address /usr/src/debug/libtpcimgio/imgdecay.c libtpcimgio-devel.i686: W: no-documentation 4 packages and 1 specfiles checked; 16 errors, 5 warnings. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 714326] Review Request: libtpcmisc - Miscellaneous PET functions
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=714326 Ankur Sinha changed: What|Removed |Added Blocks||714328(xmedcon) -- Configure 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 714327] Review Request: libtpcimgio - Turku PET Centre for image file input and output procedures
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=714327 Ankur Sinha changed: What|Removed |Added Blocks||714328(xmedcon) -- Configure 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 714328] Review Request: xmedcon - A medical image conversion utility and 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=714328 Ankur Sinha changed: What|Removed |Added Blocks||666726(amide) -- Configure 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 666726] Review Request: amide - A Medical Image Data Examiner:
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=666726 Ankur Sinha changed: What|Removed |Added Depends on||714328(xmedcon) -- Configure 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 714328] Review Request: xmedcon - A medical image conversion utility and 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=714328 Ankur Sinha changed: What|Removed |Added Depends on||714326(libtpcmisc), ||714327(libtpcimgio) Alias||xmedcon -- Configure 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 714328] New: Review Request: xmedcon - A medical image conversion utility and library
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: xmedcon - A medical image conversion utility and library https://bugzilla.redhat.com/show_bug.cgi?id=714328 Summary: Review Request: xmedcon - A medical image conversion utility and library Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: sanjay.an...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Story Points: --- Spec URL: http://ankursinha.fedorapeople.org/xmedcon/xmedcon.spec SRPM URL: http://ankursinha.fedorapeople.org/xmedcon/xmedcon-0.10.7-1.fc16.src.rpm Description: This project stands for Medical Image Conversion and is released under the GNU's (L)GPL license. It bundles the C sourcecode, a library, a flexible command-line utility and a graphical front-end based on the amazing Gtk+ toolkit. Its main purpose is image conversion while preserving valuable medical study information. The currently supported formats are: Acr/Nema 2.0, Analyze (SPM), Concorde/uPET, DICOM 3.0, CTI ECAT 6/7, InterFile 3.3 and PNG or Gif87a/89a towards desktop applications. [ankur@ankur SRPMS]$ rpmlint /var/lib/mock/fedora-rawhide-i386/result/*.rpm ../SPECS/xmedcon.spec xmedcon.i686: W: spelling-error %description -l en_US Acr -> Ac, Ar, Apr xmedcon.i686: W: spelling-error %description -l en_US uPET -> u Pet, PET, u PET xmedcon.i686: W: invalid-license BSD with attribution xmedcon.i686: E: shlib-with-non-pic-code /usr/lib/libmdc.so.2.0.1 ^ I need to look into this :/ Any clues will be helpful. xmedcon.i686: W: shared-lib-calls-exit /usr/lib/libmdc.so.2.0.1 exit@GLIBC_2.0 xmedcon.i686: W: devel-file-in-non-devel-package /usr/bin/xmedcon-config xmedcon.i686: E: incorrect-fsf-address /usr/share/doc/xmedcon-0.10.7/COPYING xmedcon.i686: E: incorrect-fsf-address /usr/share/doc/xmedcon-0.10.7/COPYING.LIB xmedcon.src: W: spelling-error %description -l en_US Acr -> Ac, Ar, Apr xmedcon.src: W: spelling-error %description -l en_US uPET -> u Pet, PET, u PET xmedcon.src: W: invalid-license BSD with attribution xmedcon-debuginfo.i686: W: invalid-license BSD with attribution xmedcon-devel.i686: W: spelling-error %description -l en_US libmdc -> libido xmedcon-devel.i686: W: invalid-license BSD with attribution xmedcon-devel.i686: E: incorrect-fsf-address /usr/share/doc/xmedcon-devel-0.10.7/COPYING.LIB xmedcon-devel.i686: E: incorrect-fsf-address /usr/share/doc/xmedcon-devel-0.10.7/COPYING 4 packages and 1 specfiles checked; 5 errors, 11 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
