[Bug 1445411] Review Request: python-cysignals - Interrupt and signal handling for Cython

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



https://bugzilla.redhat.com/show_bug.cgi?id=1445411



--- Comment #2 from Jerry James <loganjerry@xxxxxxxxx> ---
Created attachment 1274064
  --> https://bugzilla.redhat.com/attachment.cgi?id=1274064&action=edit
Patch to remove python preloading

Two quick notes: fedora-review says the spec file at the URL given above does
not match the spec file inside the SRPM.  Also, the cysignals-CSI script
attempts to "preload the right Python library."  However, Fedora's gdb is
already linked with the python 3 library.  The preload attempt is both
unnecessary and wrong, as it causes gdb to crash on startup.  I think that the
rest of %prep after %setup should be removed, and the attached patch applied
instead.  I haven't tried yet, but I expect that will fix the problem with
%check as well.

The first hunk of the patch tells gdb that it can load an init file from
anywhere.  I'm not sure that is necessary.  You might want to remove that part.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
You are always notified about changes to this product and component
_______________________________________________
package-review mailing list -- package-review@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to package-review-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite Conditions]     [KDE Users]

  Powered by Linux