[Bug 459675] Review Request: python-sybase - new package request

bugzilla at redhat.com bugzilla at redhat.com
Thu Oct 23 07:53:23 UTC 2008


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=459675





--- Comment #11 from Mamoru Tasaka <mtasaka at ioa.s.u-tokyo.ac.jp>  2008-10-23 03:53:22 EDT ---
(In reply to comment #10)
> The RPM *does* definitely work for me. It installs an egg file instead of
> sybasect.so. and Sybase.py. 

After reading https://fedoraproject.org/wiki/Packaging/Python/Eggs
it seems that Fedora allows this case. Then
* As dlopened module sybasect.so is in zipped egg file, debuginfo file
  is not created.
  Please follow
 
https://fedoraproject.org/wiki/Packaging/Debuginfo#Useless_or_incomplete_debuginfo_packages_due_to_other_reasons
-------------------------------------------------------------------
use %define debug_package %{nil}  in the specfile, and be sure 
to add a comment next to it explaining why it was done. 
-------------------------------------------------------------------


> I'm removing site.py, site.pyc, and site.pyo because these files already exist
> (from python-setuptools). 
Okay.

> I see what you mean about the .csvignore and pstumble, but I can't figure out
> how to do anything about it. The %{_doc} directory in $RPM_BUILD_ROOT doesn't
> exist yet in the %install section.
%doc LICENCE ChangeLog doc examples installs these files.
Just
--------------------------------------------------------------------
%setup -q
chmod 0644 doc/pstumble
rm -f doc/.cvsignore
--------------------------------------------------------------------
will suppress rpmlint error/warnings

By the way I am also waiting for your another review request submission
or a pre-review of other person's review request.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.




More information about the package-review mailing list