Please do not reply directly to this email. All additional comments should be made in the comments box of this bug report. Summary: Review Request: scim-python - a python wrapper for SCIM input platform https://bugzilla.redhat.com/show_bug.cgi?id=306431 ------- Additional Comments From phuang@xxxxxxxxxx 2007-09-26 04:07 EST ------- (In reply to comment #3) > - I think the License field should say LGPLv2+ > > - gettext-devel will pull in gettext anyway > > - "%{_libdir}/scim*/*" are owned by scim so it should be > "%{_libdir}/scim*/*/%{name}*.so" say Those problems have been fixed and files have been refreshed. Please check it. Thanks. =================================================================== --- scim-python.spec.in (revision 49) +++ scim-python.spec.in (working copy) @@ -3,14 +3,14 @@ Release: 1%{?dist} Summary: Python IMEngine for Smart Common Input Method platform -License: LGPLv2 +License: LGPLv2+ Group: System Environment/Libraries URL: http://code.google.com/p/scim-python/ Source0: http://scim-python.googlecode.com/files/%{name}-%{version}.tar.bz2 BuildRoot: %{_tmppath}/%{name}-%{version}-%{release}-root-%(%{__id_u} -n) -BuildRequires: scim-devel, gettext, gettext-devel, autoconf, automake, libtool, python -Requires: scim python-enchant +BuildRequires: scim-devel, gettext-devel, libtool, python-devel, pygtk2-devel, perl(XML::Parser) +Requires: scim, python-enchant, pygtk2 %description Python IMEngine for SCIM. @@ -40,9 +40,8 @@ %defattr(-,root,root,-) %doc AUTHORS COPYING README TODO %{_libdir}/python*/site-packages/*.so -%{_libdir}/scim*/* -%{_datadir}/scim-python/engine/* -%{_datadir}/scim-python/setupui/* +%{_libdir}/scim*/*/%{name}*.so +%{_datadir}/scim-python/* %{_datadir}/scim/icons/* -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug, or are watching someone who is. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review