[Bug 483730] Review Request: kde-plasma-translatoid - A Google Translation Plasmoid
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=483730 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|kde-plasma-translatoid-1.1. |kde-plasma-translatoid-1.1. |90_20100122svn-2.fc13 |90_20100122svn-2.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567107] Review Request: perl-Net-SSLGlue - Add/extend SSL support for common perl modules
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=567107 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Fixed In Version|perl-Net-SSLGlue-0.2-1.fc12 |perl-Net-SSLGlue-0.2-1.fc11 -- Configure 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 570113] Review Request: rubygem-right_aws - Interface classes for Amazon EC2/EBS, SQS, S3, SDB and ACF
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=570113 --- Comment #15 from Fedora Update System upda...@fedoraproject.org 2010-03-10 01:54:24 EST --- rubygem-right_aws-1.10.0-3.fc13 has been pushed to the Fedora 13 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update rubygem-right_aws'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/rubygem-right_aws-1.10.0-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 553345] Review Request: ibus-table-tv - The Thai and (Viqr) Vietnamese tables for IBus-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=553345 --- Comment #3 from Caius 'kaio' Chance ccha...@redhat.com 2010-03-10 02:14:32 EST --- Spec URL: http://kaio.fedorapeople.org/pkgs/ibus-table-tv.spec SRPM URL: http://kaio.fedorapeople.org/pkgs/ibus-table-tv-1.2.0.20100305-3.fc12.src.rpm Description: The Thai and (Viqr) Vietnamese tables for IBus-Table -- Configure 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 566406] Review Request: packETH - A GUI packet generator tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=566406 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-03-10 02:42:50 EST --- packETH-1.6.5-3.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/packETH-1.6.5-3.fc11 -- Configure 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 566406] Review Request: packETH - A GUI packet generator tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=566406 --- Comment #13 from Fedora Update System upda...@fedoraproject.org 2010-03-10 02:42:10 EST --- packETH-1.6.5-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/packETH-1.6.5-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566409] Review Request: python-SocksiPy - A Python SOCKS module
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=566409 --- Comment #11 from Fedora Update System upda...@fedoraproject.org 2010-03-10 02:47:24 EST --- python-SocksiPy-1.00-3.fc12 has been submitted as an update for Fedora 12. http://admin.fedoraproject.org/updates/python-SocksiPy-1.00-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566407] Review Request: perl-Number-Bytes-Human - Convert byte count to human readable 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=566407 --- Comment #16 from Fedora Update System upda...@fedoraproject.org 2010-03-10 02:45:33 EST --- perl-Number-Bytes-Human-0.07-3.fc11 has been submitted as an update for Fedora 11. http://admin.fedoraproject.org/updates/perl-Number-Bytes-Human-0.07-3.fc11 -- Configure 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 566407] Review Request: perl-Number-Bytes-Human - Convert byte count to human readable 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=566407 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-03-10 02:44:21 EST --- perl-Number-Bytes-Human-0.07-3.fc13 has been submitted as an update for Fedora 13. http://admin.fedoraproject.org/updates/perl-Number-Bytes-Human-0.07-3.fc13 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 562366] Review request: zinnia - online handwriting recognition system with machine learning
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=562366
--- Comment #21 from Liang Suilong liangsuil...@gmail.com 2010-03-10 03:20:06
EST ---
(In reply to comment #20)
rpmlint *
zinnia.src: W: summary-not-capitalized C online handwriting recognition system
with machine learning-
Online handwriting recognition system with machine learning
zinnia.src: W: spelling-error %description -l en_US customizable -
customization, customize, customarily
zinnia.src: W: non-coherent-filename zinnia-0.05-3.fc14.src.rpm
zinnia-0.05-3.fc14.x86_64.rpm-
zinnia.src: W: strange-permission zinnia-0.05.tar.gz 0777L
zinnia.src: W: strange-permission zinnia-0.05-bindings.patch 0777L
zinnia.src: W: strange-permission zinnia.spec 0777L-
chmod 644
zinnia.x86_64: W: summary-not-capitalized C online handwriting recognition
system with machine learning
zinnia.x86_64: W: spelling-error %description -l en_US customizable -
customization, customize, customarily
zinnia.x86_64: W: shared-lib-calls-exit /usr/lib64/libzinnia.so.0.0.0
e...@glibc_2.2.5
This warning still appears. I think that it needs a patch of source codes. I do
not know how to fix it. Could you help me write a patch?
zinnia.x86_64: E: zero-length /usr/share/doc/zinnia-0.05/ChangeLog
-
in %prep
find . -type f -name ChangeLog -size 0c -exec rm -f {} ';'
zinnia-devel.x86_64: W: no-documentation
Here I add %doc README COPYING in all %file sections. The warning disappears.
Is it OK?
zinnia-doc.noarch: W: file-not-utf8
/usr/share/doc/zinnia-doc-0.05/doc/zinnia.css-
In %prep
sed -i 's/\r//' doc/zinnia.css
It seems not to work. I add them in %prep
pushd doc
iconv -f latin1 -t utf8 zinnia.css zinnia.css.bak
mv -f zinnia.css.bak zinnia.css
popd
zinnia-perl.x86_64: W: no-documentation
zinnia-perl.x86_64: E: non-standard-executable-perm
/usr/lib64/perl5/auto/zinnia/zinnia.so 0555L-
chmod 755
zinnia-python.x86_64: W: no-documentation
zinnia-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/zinnia
['/usr/lib64']
zinnia-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/zinnia_convert
['/usr/lib64']
zinnia-utils.x86_64: E: binary-or-shlib-defines-rpath /usr/bin/zinnia_learn
['/usr/lib64']
-
After %configure
sed -i 's|^hardcode_libdir_flag_spec=.*|hardcode_libdir_flag_spec=|g'
libtool
sed -i 's|^runpath_var=LD_RUN_PATH|runpath_var=DIE_RPATH_DIE|g' libtool
zinnia-utils.x86_64: W: no-documentation
8 packages and 0 specfiles checked; 5 errors, 14 warnings.
I recommend the following changes:
#%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print get_python_lib())}
#%{!?python_sitearch: %global python_sitearch %(%{__python} -c from
distutils.sysconfig import get_python_lib; print get_python_lib(1))}
%if ! (0%{?fedora} 12 || 0%{?rhel} 5)
%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib()))}
%{!?python_sitearch: %global python_sitearch %(%{__python} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib(1)))}
%endif
-
%if ! (0%{?fedora} 12 || 0%{?rhel} 5)
%{!?python_sitelib: %global python_sitelib %(%{__python} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib()))}
%{!?python_sitearch: %global python_sitearch %(%{__python} -c from
distutils.sysconfig import get_python_lib; print(get_python_lib(1)))}
%endif
rm -f python/%{name}.pyc
-
find . -type f -name *.pyc -exec rm -f {} ';'
rm -f $RPM_BUILD_ROOT%{_libdir}/lib%{name}.la
rm -f $RPM_BUILD_ROOT%{perl_vendorarch}/auto/%{name}/%{name}.bs
rm -f $RPM_BUILD_ROOT%{perl_vendorarch}/auto/%{name}/.packlist
-
find $RPM_BUILD_ROOT -type f -name *.la -exec rm -f {} ';'
find $RPM_BUILD_ROOT -type f -name *.bs -size 0c -exec rm -f {} ';'
find $RPM_BUILD_ROOT -type f -name .packlist -exec rm -f {} ';'
Note:.bs file is useful if it's not a empty file
%description utils
The %{name}-utils package provide utils for zinnia library that
use %{name}.
-
%description utils
The %{name}-utils package provides utilities for zinnia library that
use %{name}.
python setup.py install --root $RPM_BUILD_ROOT
-
%{__python} setup.py install --root $RPM_BUILD_ROOT
Note: Choose %{__python} or python, not use both
CFLAGS=%{optflags}
-
CFLAGS=$RPM_OPT_FLAGS
Note: Choose %{optflags} or $RPM_OPT_FLAGS, not use both
%doc doc
-%doc doc/*
The package has the last warning:
zinnia.x86_64: W: shared-lib-calls-exit /usr/lib64/libzinnia.so.0.0.0
SPEC: https://dl.dropbox.com/u/1352061/zinnia.spec
SRPM: https://dl.dropbox.com/u/1352061/zinnia-0.05-4.fc12.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.
[Bug 563820] Review Request: globus-libxml2 - Globus Toolkit - LibXML2 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=563820 Mattias Ellert mattias.ell...@fysast.uu.se changed: What|Removed |Added Flag||fedora-cvs? --- Comment #4 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-03-10 03:45:05 EST --- Thank you for the review! New Package CVS Request === Package Name: globus-libxml2 Short Description: Globus Toolkit - LibXML2 Library caches Owners: ellert Branches: F-11 F-12 F-13 EL-4 EL-5 InitialCC: -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 570424] Review Request: transmission-remote-cli - A console client for the Transmission BitTorrent client
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=570424
--- Comment #4 from Satya Komaragiri satya.komarag...@gmail.com 2010-03-10
03:48:41 EST ---
[!] %{buildroot} is removed at beginning of %install
According to http://fedoraproject.org/wiki/Packaging/Guidelines#BuildRoot_tag,
cleaning up of buildroot isn't required from Fedora 10 onwards.
[!] Package contains latest upstream version
* While checking the sha1sum I noted there were some changes on the code just
today, maybe you want to update the file and the release tag accordingly
then?
Corrected.
[!] Package builds in mock
Tested on: F12/x86_64; see my note concerning README.md below
* The installation of the README.md is not okay yet. The file actually would
get installed in /, which obviuously is not the right place for documentation
files.
--snip--
Corrected
* You should write down in your ChangeLog what you actually changed in the
package or rather in the specfile
Corrected
* The release number should always begin with 0.
--snip--
Corrected.
The updated spec file may be found at:
http://sundaram.fedorapeople.org/packages/transmission-remote-cli.spec
And the updated SRPM is at:
http://sundaram.fedorapeople.org/packages/transmission-remote-cli-0.5.5-0.3.20100310git.fc12.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 225783] Merge Review: gdb
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=225783 --- Comment #30 from Jan Kratochvil jan.kratoch...@redhat.com 2010-03-10 04:51:24 EST --- (In reply to comment #29) (In reply to comment #28) gdb/osf-share/HP800/cma_thread_io.h is apparently BSD-ish license. Posted: Licensing: gdb/osf-share/ is GPL-incompatible http://sourceware.org/ml/gdb/2010-03/msg00061.html Just it IMO should not be tagged as BSD with advertising as the companies listed there are not Berkeley. https://fedoraproject.org/wiki/Licensing#Good_Licenses Going to use (currently without the gdb/osf-share/ before it gets resolved upstream): License: GPLv3+ and GPLv3+ with exceptions and GPLv2+ and GPL+ and LGPLv2+ and GFDL and Public Domain -- 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 512170] Review Request: pidgin-sipe - Pidgin plugin for connecting to MS Communications 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=512170 --- Comment #49 from Stefan Becker stefan.bec...@nokia.com 2010-03-10 05:21:07 EST --- FYI: 1.9.0 has just been released. New features: file transfer, NTLMv2 support I was wondering if you could look into syncing the Fedora RPM spec from the one in the release (contrib/rpm/pidgin-sipe.spec). At least you should update the summary descriptions entries from that 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 549821] Review Request: dcap - Client Tools for dCache
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=549821 --- Comment #4 from Steve Traylen steve.tray...@cern.ch 2010-03-10 05:57:50 EST --- Concerning addler32.c is it possible just to use the system zlib rather than this copy? Steve -- Configure 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 549821] Review Request: dcap - Client Tools for dCache
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=549821 --- Comment #5 from Steve Traylen steve.tray...@cern.ch 2010-03-10 06:03:15 EST --- Also given you are already creating your own tar ball then rm getopt.c is obviously trivial addition. Steve -- Configure 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 544295] Review Request: upower - Power management service
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=544295 Richard Hughes rich...@hughsie.com changed: What|Removed |Added Flag||fedora-cvs? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567937] Review Request: mpir - A library for arbitrary precision arithmetic
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=567937 Mark Chappell trem...@tremble.org.uk changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #9 from Mark Chappell trem...@tremble.org.uk 2010-03-10 06:42:24 EST --- MPIR now submitted for F-12/13-testing and in rawhide. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 571364] Review Request: man-db - Database cached manual pager suit
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=571364 --- Comment #2 from Terje Røsten terje...@phys.ntnu.no 2010-03-10 06:58:29 EST --- Fixed some issues: - fix man_db.conf location - build with various compress support - explicit file listing - use find_lang macro - redo install part - remove all translations spec: http://terjeros.fedorapeople.org/man-db/man-db.spec srpm: http://terjeros.fedorapeople.org/man-db/man-db-2.5.7-2.fc13.src.rpm koji: http://koji.fedoraproject.org/koji/taskinfo?taskID=2043624 with changes it works fine here, please feel free to use. -- Configure 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 543917] Review Request: check_postgres - PostgreSQL monitoring script
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=543917 --- Comment #24 from Xavier Bachelot xav...@bachelot.org 2010-03-10 07:54:59 EST --- ok, forget about my comment then. -- Configure 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 567086] Review Request: VXL - C++ Libraries for Computer Vision Research and Implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567086
--- Comment #4 from Chen Lei supercy...@163.com 2010-03-10 07:56:00 EST ---
(In reply to comment #3)
Hello Chen Lei!
thanks for your comments:
1) What do you mean add subpackage info after %description? Would you mind
writing a small example?
(ie:
%package devel
Summary: Headers for VXL library
Group: Development/Libraries
Requires: %{name} = %{version}-%{release}
%description devel
You should install this package if you would like to
develop code based on VXL.
- Here I think you should add some more text to clarify -
)
It's not error, just a spec file standard. You see almost all spec files add
subpackage info after %description, follow standard will increase the
readability of spec file.
See
http://cvs.fedoraproject.org/viewvc/rpms/glibc/devel/glibc.spec?revision=1.430view=markup
2) As for the bundled libs:
- I've already removed jpeg png zlib tiff geotiff mpeg2
- v3p/netlib dependency not removed because of heavily modifications
- v3p/dcmtk: it exists an rpm package at http://dicom.offis.de/dcmtk.php.en
but
there is none in yum and neither there is a review request. Should we try to
add it as well?
- All of the following libraries are being switched off:
-DBUILD_OXL:BOOL=OFF \
-DBUILD_CORE_GEOMETRY:BOOL=OFF \
-DBUILD_CORE_IMAGING:BOOL=OFF \
-DBUILD_CORE_NUMERICS:BOOL=ON \
-DBUILD_CORE_PROBABILITY:BOOL=OFF \
-DBUILD_CORE_SERIALISATION:BOOL=OFF \
-DBUILD_CORE_UTILITIES:BOOL=OFF \
-DBUILD_CORE_VIDEO:BOOL=OFF \
because the main intent for this package was to allow InsightToolkit (which is
blocked by this package review), my idea to speed up the approval was to
disable these functionalities as they are not used in Insight Toolkit and
require a significant effort to be decoupled and packaged as well.
If you can help in packaging and enabling these libraries, I would be more
than
We should packaging those libs, I can also help you to package some of them.
Howerver, if I try to package those libs, I'll need some others to review my
packages.
--
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are on the CC list for the bug.
___
package-review mailing list
package-review@lists.fedoraproject.org
https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 567086] Review Request: VXL - C++ Libraries for Computer Vision Research and Implementation
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=567086
--- Comment #5 from Mario Ceresa mrcer...@gmail.com 2010-03-10 08:42:10 EST
---
Hello Chen Lei!
thanks for your quick response.
I'm sorry to say that I still don't catch what you want me to change in the
spec file.
Please be patient: English is not my mother tongue so there might be some
subtleties that confounds me. Let's make an example:
#Example starts here#
[from vxl.spec]
%package devel
Summary: Headers for VXL library
Group: Development/Libraries
Requires: %{name} = %{version}-%{release}
%description devel
You should install this package if you would like to
develop code based on VXL.
[end vxl.spec]
[from glibc.spec]
%package xen
Summary: The GNU libc libraries (optimized for running under Xen)
Group: System Environment/Libraries
Requires: glibc = %{version}-%{release}, glibc-utils = %{version}-%{release}
%description xen
The standard glibc package is optimized for native kernels and does not
perform as well under the Xen hypervisor. This package provides alternative
library binaries that will be selected instead when running under Xen.
Install glibc-xen if you might run your system under the Xen hypervisor.
[end glibc.spec]
#End of example#
As devel is a subpackage of vxl I added %description devel to create vxl-devel
package. Then I added some descriptive info.
I would appreciate very much if you could rewrite the part of my example
between [from vxl.spec] - [end vxl.spec] to show me the change you have in
mind, so I can see what I'm missing. Sorry if it seems a stupid question.
On the other hand, on the library side, I'll be very happy to review any vxl
related lib that you might try to package, and help with adding them to vxl.
The only think that I'd like to ask is that, if the time required to package
them all would be considerable, we go on with the review for this subset only
of vxl and file later bugs requiring more functionalities.
I don't know if this could be a viable solution but it seems to me an
acceptable compromise between blocking indefinitely the InsightToolkit review
and packaging many scientific libs. I would love to hear what you think about
this point.
Thanks for your time and best regards
Mario
--
Configure 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 512170] Review Request: pidgin-sipe - Pidgin plugin for connecting to MS Communications 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=512170 --- Comment #50 from Stefan Becker stefan.bec...@nokia.com 2010-03-10 09:17:00 EST --- Any chances to add EPEL5 as build target? -- Configure 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 562366] Review request: zinnia - online handwriting recognition system with machine learning
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=562366 Chen Lei supercy...@163.com changed: What|Removed |Added Flag|fedora-review? |fedora-review+ --- Comment #24 from Chen Lei supercy...@163.com 2010-03-10 10:32:24 EST --- *** 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 562366] Review request: zinnia - online handwriting recognition system with machine learning
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=562366 --- Comment #25 from Liang Suilong liangsuil...@gmail.com 2010-03-10 10:53:14 EST --- New Package CVS Request === Package Name: zinnia Short Description: Online handwriting recognition system with machine learning Owners: liangsuilong Branches: F-11 F-12 F-13 devel InitialCC: liangsuilong Cvsextras Commits: yes -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 572233] New: Review Request: compat-tracker - Compatibilty libraries and tools for tracker-0.6.x
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. Summary: Review Request: compat-tracker - Compatibilty libraries and tools for tracker-0.6.x https://bugzilla.redhat.com/show_bug.cgi?id=572233 Summary: Review Request: compat-tracker - Compatibilty libraries and tools for tracker-0.6.x Product: Fedora Version: rawhide Platform: All OS/Version: Linux Status: NEW Severity: medium Priority: medium Component: Package Review AssignedTo: nob...@fedoraproject.org ReportedBy: dakin...@gmail.com QAContact: extras...@fedoraproject.org CC: nott...@redhat.com, fedora-package-rev...@redhat.com Estimated Hours: 0.0 Classification: Fedora Spec URL: http://deji.fedorapeople.org/compat-tracker.spec SRPM URL: http://deji.fedorapeople.org/compat-tracker-0.6.96-1.fc13.src.rpm Description: Tracker is a powerful desktop-neutral first class object database, tag/metadata database, search tool and indexer. It consists of a common object database that allows entities to have an almost infinite number of properties, metadata (both embedded/harvested as well as user definable), a comprehensive database of keywords/tags and links to other entities. THIS PACKAGE ONLY PROVIDES COMPATIBILITY FOR APPLICATIONS AND LIBRARIES THAT DEPENDS ON THE TRACKER-0.6 SERIES API. NOTICE: The main aim of this package proposal is to provide support for application(s) within Fedora that depend on the tracker-0.6.x series APIs, while the main tracker package itself is updated to the latest 0.7/0.8 series. The 0.7/0.8 series is in a lot ways a rewrite of tracker which provides substantial improvements to the operation of tracker over the 0.6.x series. -- Configure 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 549821] Review Request: dcap - Client Tools for dCache
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=549821 --- Comment #6 from Mattias Ellert mattias.ell...@fysast.uu.se 2010-03-10 10:54:14 EST --- (In reply to comment #4) Concerning addler32.c is it possible just to use the system zlib rather than this copy? I contacted upstream, and they have supplied a patch for this. I have updated the package: Spec URL: http://www.grid.tsl.uu.se/review/dcap.spec SRPM URL: http://www.grid.tsl.uu.se/review/dcap-2.44.0-2.fc12.src.rpm (In reply to comment #5) Also given you are already creating your own tar ball then rm getopt.c is obviously trivial addition. Upstream's patch also removes this file. I don't know if this part of the patch will stay - some claim the getopt.c is needed on windows. But for now the file is removed by the patch supplied by 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 563471] Tracker: Review Requests for Security Spin related packages
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=563471 Bug 563471 depends on bug 553281, which changed state. Bug 553281 Summary: Review Request: netsniff-ng - high performance linux network sniffer for packet inspection https://bugzilla.redhat.com/show_bug.cgi?id=553281 What|Old Value |New Value Status|ASSIGNED|ON_QA Resolution||ERRATA Status|ON_QA |CLOSED Bug 563471 depends on bug 566406, which changed state. Bug 566406 Summary: Review Request: packETH - A GUI packet generator tool https://bugzilla.redhat.com/show_bug.cgi?id=566406 What|Old Value |New Value Resolution||NEXTRELEASE 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 566406] Review Request: packETH - A GUI packet generator tool
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=566406 Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE --- Comment #15 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-03-10 11:30:53 EST --- Closing. -- Configure 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 566412] Review Request: wapiti - A web application vulnerability scanner
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=566412 Bug 566412 depends on bug 566409, which changed state. Bug 566409 Summary: Review Request: python-SocksiPy - A Python SOCKS module https://bugzilla.redhat.com/show_bug.cgi?id=566409 What|Old Value |New Value Status|ASSIGNED|CLOSED Resolution||NEXTRELEASE -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 563844] Review Request: zim - A Desktop Wiki Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=563844 --- Comment #12 from Cheese Lee cheese...@126.com 2010-03-10 11:43:55 EST --- - Fixed postun scriptlet Spec URL: https://dl.dropbox.com/u/612778/zim.spec SRPM URL: https://dl.dropbox.com/u/612778/zim-0.44-4.fc12.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 561448] Review Request: jffi - Java Foreign Function Interface
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=561448
--- Comment #9 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-03-10
12:25:52 EST ---
For -3:
* For ppc{,64}
- To compile your jffi.src on these architecture, adding
the below at the end of %prep is enough
--
%prep
...
...
...
sed -i.cpu -e '/m\$(MODEL)/d' jni/GNUmakefile libtest/GNUmakefile
%ifnarch %{ix86} x86_64
rm -rf test/
%endif
--
! Well, test fails on these archs, I don't know why (I don't
know java well).
http://koji.fedoraproject.org/koji/taskinfo?taskID=2044713
Other things seem okay.
--
Configure 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 563844] Review Request: zim - A Desktop Wiki Editor
Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=563844 --- Comment #13 from Mamoru Tasaka mtas...@ioa.s.u-tokyo.ac.jp 2010-03-10 12:50:44 EST --- Forgot to mention one more thing For -4: * rpmlint - zim.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/zim/_lib/xdot.py 0644L /usr/bin/env zim.noarch: E: non-executable-script /usr/lib/python2.6/site-packages/zim/_version.py 0644L /usr/bin/env - - These two scripts are not executed directly by user, so shebangs on these scripts are not needed. Please remove these shebangs. -- Configure 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 472303] Review Request: python-ropevim - Python module to add rope refactoring to vim
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=472303 --- Comment #2 from Toshio Ernie Kuratomi a.bad...@gmail.com 2010-03-10 13:07:52 EST --- I wasn't able to get the integration to work bug-free enough that I was comfortable supporting this so I've never imported it. It's been quite a while and I realize I don't have the time to work on making it work so i'm closing this review request -- someone else can take my work and update it if they feel they need to. -- Configure 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 472303] Review Request: python-ropevim - Python module to add rope refactoring to vim
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=472303 Toshio Ernie Kuratomi a.bad...@gmail.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||DEFERRED -- Configure 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 453422] Review Request: songbird - Mozilla based multimedia player
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=453422 --- Comment #124 from gat gatlinsulli...@yahoo.com 2010-03-10 13:10:36 EST --- When Fedora 13 comes out (the final version out of Beta and past the release candidates) if I install Songbird immediately do you believe that Songbird will work, or not? I tried to install Songbird 1.4 about a month after I had Fedora 12 installed, and a boat load of installations after installing the operating system I install Songbird, but for whatever reason it won't work correctly on non-fresh installations of Fedora. Songbird is the last component that I need to make Fedora my new GNU/Linux operating system and my laptop media center. -- Configure 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 503136] Review Request: csync - a file synchroniser utility
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=503136 --- Comment #14 from Alex Hudson (Fedora Address) fed...@alexhudson.com 2010-03-10 14:11:50 EST --- Ah, I was on vacation actually - but anyhow, I don't have a problem with Andreas taking this package on: I was hoping to use it more than I have ended up to. -- Configure 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 225783] Merge Review: gdb
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=225783 Jan Kratochvil jan.kratoch...@redhat.com changed: What|Removed |Added Flag||needinfo?(mc...@redhat.com) --- Comment #31 from Jan Kratochvil jan.kratoch...@redhat.com 2010-03-10 14:47:58 EST --- In CVS F-13/ as gdb-7.0.90.20100306-21.fc13. License: GPLv3+ and GPLv3+ with exceptions and GPLv2+ and GPL+ and LGPLv2+ and GFDL and BSD and Public Domain Used BSD for gdb/osf-share/ as it looks like: BSD License (no advertising) BSD Yes Yes Yes https://fedoraproject.org/wiki/Licensing/BSD#3ClauseBSD -- 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 225783] Merge Review: gdb
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=225783 --- Comment #32 from Jan Kratochvil jan.kratoch...@redhat.com 2010-03-10 15:11:50 EST --- In CVS F-13/ as gdb-7.0.90.20100306-22.fc13. Add GPLv2+ with exceptions for files like libtool.m4. License: GPLv3+ and GPLv3+ with exceptions and GPLv2+ and GPLv2+ with exceptions and GPL+ and LGPLv2+ and GFDL and BSD and Public Domain -- 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 #13 from Jan Kratochvil jan.kratoch...@redhat.com 2010-03-10 15:15:54 EST --- There should also be - IMO: License: GPLv3+ and GPLv3+ with exceptions and GPLv2+ and GPLv2+ with exceptions and GPL+ and LGPLv2+ and GFDL and Public Domain following: https://fedoraproject.org/wiki/Packaging/LicensingGuidelines GPLv3+ with exceptions: texinfo/texinfo.tex GPLv2+ with exceptions: libtool.m4 GPL+: include/opcode/arm.h Other licenses are easy to find. -- 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 553717] Review Request: libcrystalhd - Broadcom Crystal HD device interface 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=553717 Jarod Wilson ja...@wilsonet.com changed: What|Removed |Added Flag||needinfo?(pbrobin...@gmail. ||com) --- Comment #29 from Jarod Wilson ja...@wilsonet.com 2010-03-10 16:01:21 EST --- So... Does this mean we're good to go with approving this package, and I'll just add the firmware sub-package after the license for it gets sorted 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 439630] Review Request: jogl - Java bindings for OpenGL
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=439630 --- Comment #97 from Henrique LonelySpooky Junior henrique...@gmail.com 2010-03-10 16:06:40 EST --- I guess is time to reopen this review (and for gluegen too). Should I create a new one? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 439627] Review Request: gluegen - Java/JNI glue code generator to call out to ANSI C
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=439627 --- Comment #43 from Henrique LonelySpooky Junior henrique...@gmail.com 2010-03-10 16:06:59 EST --- I guess is time to reopen this review. Should I create a new one? -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 225783] Merge Review: gdb
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=225783 Matej Cepl mc...@redhat.com changed: What|Removed |Added Status|ASSIGNED|CLOSED Resolution||RAWHIDE --- Comment #34 from Matej Cepl mc...@redhat.com 2010-03-10 16:19:24 EST --- yes, of course, this is Merge Review. Closing as Rawhide. -- 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 553717] Review Request: libcrystalhd - Broadcom Crystal HD device interface 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=553717 Peter Robinson pbrobin...@gmail.com changed: What|Removed |Added Flag|fedora-review?, |fedora-review+ |needinfo?(pbrobin...@gmail. | |com)| --- Comment #30 from Peter Robinson pbrobin...@gmail.com 2010-03-10 16:34:07 EST --- Yep, I think so. 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 226490] Merge Review: thunderbird
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=226490 Matej Cepl mc...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mc...@redhat.com -- 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 225755] Merge Review: firefox
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=225755 Matej Cepl mc...@redhat.com changed: What|Removed |Added AssignedTo|nob...@fedoraproject.org|mc...@redhat.com -- 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 551651] Review Request: ArpON - Arp handler inspection
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=551651 --- Comment #20 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:17:15 EST --- CVS done (by process-cvs-requests.py). -- Configure 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 455067] Review Request: ferm - For Easy Rule Making
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=455067 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #26 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:14:51 EST --- CVS done. And yes, we don't care if the ticket is open; we just look for the status of 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 562366] Review request: zinnia - online handwriting recognition system with machine learning
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=562366 --- Comment #26 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:23:18 EST --- CVS done (by process-cvs-requests.py). -- Configure 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 566729] Review Request: CharLS - An optimized implementation of the JPEG-LS standard
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=566729 --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:24:43 EST --- CVS done (by process-cvs-requests.py). -- Configure 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 565376] Review Request: qstardict - StarDict clone written in Qt4
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=565376 --- Comment #10 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:24:17 EST --- CVS done (by process-cvs-requests.py). -- Configure 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 571397] Review Request: python-gudev
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=571397 Jason Tibbitts ti...@math.uh.edu changed: What|Removed |Added Flag|fedora-cvs? |fedora-cvs+ --- Comment #8 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:35:51 EST --- Please don't file a new package request for a package that already exists; it just makes me spend time figuring out why pkgdb fails to add the package. File a change request instead. I've added the F-13 branch. -- Configure 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 553717] Review Request: libcrystalhd - Broadcom Crystal HD device interface 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=553717 --- Comment #32 from Jason Tibbitts ti...@math.uh.edu 2010-03-10 17:41:42 EST --- CVS done (by process-cvs-requests.py). -- Configure 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 556346] Review Request: stage - A 2.5D multi-robot simulator
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=556346 --- Comment #15 from Rich Mattes richmat...@gmail.com 2010-03-10 18:32:47 EST --- I updated one of the patches to remove -O3 from the project's cflags. Spec URL: http://www.richmattes.com/rpms/stage/stage.spec SRPM URL: http://www.richmattes.com/rpms/stage/stage-3.2.2-6.fc12.src.rpm koji build: dist-f13 http://koji.fedoraproject.org/koji/taskinfo?taskID=2045239 dist-f12 http://koji.fedoraproject.org/koji/taskinfo?taskID=2045250 rpmlint: $ rpmlint stage.spec ../RPMS/i686/stage* ../RPMS/noarch/stage* ../SRPMS/stage* stage.i686: W: spelling-error Summary(en_US) multi - mulch, mufti stage.i686: W: spelling-error %description -l en_US scalable - salable, scalawag, scalar stage.i686: W: shared-lib-calls-exit /usr/lib/libstage.so.3.2.2 e...@glibc_2.0 stage-devel.i686: W: spelling-error %description -l en_US libstage - lib stage, lib-stage, multistage stage-devel.i686: W: no-documentation stage-playerplugin.i686: W: no-documentation stage.src: W: spelling-error Summary(en_US) multi - mulch, mufti stage.src: W: spelling-error %description -l en_US scalable - salable, scalawag, scalar 6 packages and 1 specfiles checked; 0 errors, 8 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 562469] Review Request: openvas-scanner - Open Vulnerability Assessment (OpenVAS) Scanner
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=562469
--- Comment #11 from Xavier Bachelot xav...@bachelot.org 2010-03-10 20:01:40
EST ---
the cron should be tagged as %config(noreplace). Also, it would be better to
store it in /etc/cron.d/openvas-scanner, it'll then be easier to tweak the
execution time.
The reload initscript target should be made a no op, as per the guidelines.
Sorry about my previous misleading comment. See
http://fedoraproject.org/wiki/Packaging:SysVInitScript#Required_Actions.
As already written above, prog variable could probably be changed to
openvas-scanner (and then the initscript tweaked accordingly) for nicer startup
messages.
I still find having the sysconfdir be owned by openvas-libraries strange. I'd
rather have that owned by both -client and -server.
It's not good to strip the plugins manually, it's better to have them stripped
by duringthe rpm build in order to have the symbols in the -debuginfo package.
As the files need to be installed executable to be stripped, you can remove the
exec bit in the %files section rather than in the %install section. It makes
the %files section a bit longer, but something like that should do :
%dir %{_libdir}/openvas
%dir %{_libdir}/openvas/plugins
%attr(644,-,-) %{_libdir}/openvas/plugins/*.nes
then the following lines can be removed from the %install section:
# Fix permissions on included plugins and strip binaries
chmod 644 $RPM_BUILD_ROOT%{_libdir}/openvas/plugins/*
strip $RPM_BUILD_ROOT%{_libdir}/openvas/plugins/*
This package and the client package are getting in a good shape now. A quick
comment (or more ;-)) from Huzaifa would be nice. I'll do the formal reviews
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 562469] Review Request: openvas-scanner - Open Vulnerability Assessment (OpenVAS) Scanner
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=562469 --- Comment #12 from Xavier Bachelot xav...@bachelot.org 2010-03-10 20:05:41 EST --- (In reply to comment #11) the cron should be tagged as %config(noreplace). Also, it would be better to store it in /etc/cron.d/openvas-scanner, it'll then be easier to tweak the execution time. oops, typo, I meant /etc/cron.d/openvas-sync-plugins. -- Configure 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 571502] Review Request: pidgin-birthday-reminder - Birthday Reminder plugin for Pidgin
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.
https://bugzilla.redhat.com/show_bug.cgi?id=571502
William Witt will...@witt-family.net changed:
What|Removed |Added
CC||will...@witt-family.net
--- Comment #1 from William Witt will...@witt-family.net 2010-03-10 21:12:51
EST ---
Below is an informal review:
(extract from http://fedoraproject.org/wiki/Packaging:ReviewGuidelines)
* MUST: rpmlint must be run on every package. The output should be posted in
the review.[1]
[ unama...@gimli:~ ] $rpmlint
/home/unamanic/rpmbuild/SRPMS/pidgin-birthday-reminder-1.4-1.fc12.src.rpm
/home/unamanic/rpmbuild/RPMS/x86_64/pidgin-birthday-reminder-1.4-1.fc12.x86_64.rpm
/home/unamanic/rpmbuild/RPMS/x86_64/pidgin-birthday-reminder-debuginfo-1.4-1.fc12.x86_64.rpm
pidgin-birthday-reminder.src: W: spelling-error Summary(en_US) plugin - plug
in, plug-in, plugging
pidgin-birthday-reminder.x86_64: W: spelling-error Summary(en_US) plugin -
plug in, plug-in, plugging
3 packages and 0 specfiles checked; 0 errors, 2 warnings.
* MUST: A package must own all directories that it creates. If it does not
create a directory that it uses, then it should require a package which does
create that directory. [13]
is # Pidgin package doesn't own sounds/pidgin/ dir a bug against pidgin?
* MUST: Each package must consistently use macros. [17]
changing $RPM_BUILD_ROOT to %{buildroot} for consitancy would improve
consistency
* SHOULD: The description and summary sections in the package spec file should
contain translations for supported Non-English languages, if available. [29]
are translations available?
--
Configure 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 570731] Review Request: ibus-handwrite - IBus handwrite project
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=570731 Jens Petersen peter...@redhat.com changed: What|Removed |Added Status|NEW |ASSIGNED Blocks||507684(F13Target) -- Configure 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 523877] Review Request: CBFlib - crystallography binary format 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=523877 --- Comment #15 from Chen Lei supercy...@163.com 2010-03-11 00:58:52 EST --- (In reply to comment #14) (In reply to comment #13) (In reply to comment #12) Here's the updated 0.9.0 version: Spec URL: http://www.stanford.edu/~fenn/packs/CBFlib.spec SRPM URL: http://www.stanford.edu/~fenn/packs/CBFlib-0.9.0-1.fc12.src.rpm I also included some of the bin programs, but I'm wondering if it would be best to prepend cbf to them, as names like convert_image are a bit ambiguous and potentially conflicting. I didn't split off -doc and -utils, I'd like to get some feedback from fedora review people on that. Please do not add utils to main package, it'll lead to multilib conflict. How will it lead to a multilib conflict? Its fine to install both the 32 and 64 bit binaries and libraries and run/use either. Hi Tim, See http://fedoraproject.org/wiki/PackagingDrafts/MultilibTricks#Splitting_libraries_into_separate_packages -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 523877] Review Request: CBFlib - crystallography binary format 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=523877 --- Comment #16 from Michael Schwendt mschwe...@gmail.com 2010-03-11 01:13:07 EST --- Take that with a grain of salt, please. It's a draft and not a mandatory item in the guidelines. Ordinary binary executables don't cause multilib conflicts. Tim is right about that. Further, there's a conditional: *If* you have a file conflict in a library package due to other files in the same package, splitting of the libs into their own package *may* help. Doing that doesn't prevent all multiarch problems, though, such as conflicts in the -devel 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 523877] Review Request: CBFlib - crystallography binary format 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=523877 --- Comment #17 from Chen Lei supercy...@163.com 2010-03-11 01:39:38 EST --- (In reply to comment #16) Take that with a grain of salt, please. It's a draft and not a mandatory item in the guidelines. Ordinary binary executables don't cause multilib conflicts. Tim is right about that. Further, there's a conditional: *If* you have a file conflict in a library package due to other files in the same package, splitting of the libs into their own package *may* help. Doing that doesn't prevent all multiarch problems, though, such as conflicts in the -devel package. You are right Michael, this is not a mandatory item in reviewing a package. We should be lenient with multilib conflicts if the package mainly act as a program such as qt-creator. Howerver, the main role of CBFlib is act as a dynamic lib, the programs included in the CBFlib are only utilities. So I think we should be careful with multilib conflicts in CBFlib, it's easy to split out a -utils subpackage. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566407] Review Request: perl-Number-Bytes-Human - Convert byte count to human readable 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=566407 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ASSIGNED|ON_QA --- Comment #17 from Fedora Update System upda...@fedoraproject.org 2010-03-11 02:13:53 EST --- perl-Number-Bytes-Human-0.07-3.fc12 has been pushed to the Fedora 12 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Number-Bytes-Human'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-Number-Bytes-Human-0.07-3.fc12 -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543248] Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON
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=543248 Fedora Update System upda...@fedoraproject.org changed: What|Removed |Added Status|ON_QA |CLOSED Fixed In Version||git-cola-1.4.1.2-2.fc12 Resolution||ERRATA -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 516312] Review Request: ueagle-atm4-firmware - Firmware for USB ADSL Modems based on Eagle IV Chipset
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=516312 --- Comment #19 from Fedora Update System upda...@fedoraproject.org 2010-03-11 02:18:21 EST --- ueagle-atm4-firmware-1.0-4.fc13 has been pushed to the Fedora 13 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 543248] Review Request: python-jsonpickle - A module that allows any object to be serialized into JSON
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=543248 --- Comment #14 from Fedora Update System upda...@fedoraproject.org 2010-03-11 02:17:42 EST --- git-cola-1.4.1.2-2.fc12, python-jsonpickle-0.3.1-1.fc12 has been pushed to the Fedora 12 stable repository. If problems still persist, please make note of it in this bug report. -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email --- You are receiving this mail because: --- You are on the CC list for the bug. ___ package-review mailing list package-review@lists.fedoraproject.org https://admin.fedoraproject.org/mailman/listinfo/package-review
[Bug 566407] Review Request: perl-Number-Bytes-Human - Convert byte count to human readable 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=566407 --- Comment #18 from Fedora Update System upda...@fedoraproject.org 2010-03-11 02:16:04 EST --- perl-Number-Bytes-Human-0.07-3.fc11 has been pushed to the Fedora 11 testing repository. If problems still persist, please make note of it in this bug report. If you want to test the update, you can install it with su -c 'yum --enablerepo=updates-testing update perl-Number-Bytes-Human'. You can provide feedback for this update here: http://admin.fedoraproject.org/updates/perl-Number-Bytes-Human-0.07-3.fc11 -- Configure 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
