Bug 1410204 - [Patch] Fix build against sip-4.19
Summary: [Patch] Fix build against sip-4.19
Alias: None
Product: Fedora
Classification: Fedora
Component: PyQwt
Version: rawhide
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
Assignee: Tadej Janež
QA Contact: Fedora Extras Quality Assurance
Depends On:
TreeView+ depends on / blocked
Reported: 2017-01-04 18:38 UTC by Sandro Mani
Modified: 2017-01-10 14:52 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2017-01-10 14:31:13 UTC
Type: Bug

Attachments (Terms of Use)
Patch (54.75 KB, patch)
2017-01-04 18:38 UTC, Sandro Mani
no flags Details | Diff

Description Sandro Mani 2017-01-04 18:38:58 UTC
Created attachment 1237268 [details]

Attached patch fixes build against sip-4.19:


Comment 1 Tadej Janež 2017-01-10 14:31:13 UTC
(In reply to Sandro Mani from comment #0)
> Attached patch fixes build against sip-4.19:
> https://koji.fedoraproject.org/koji/taskinfo?taskID=17163962

Thanks, Sandro!

I've applied the patch (along with some other spec updates) and pushed it to rawhide:

Comment 2 Rex Dieter 2017-01-10 14:36:51 UTC
That's an odd patch, much of it  includes removal of stanzas like:

#if SIP_VERSION < 0x040800

which should not be evaluating to true in this context.  Any idea(s) why that is?  (ie, I'd rather fix the version comparison logic rather than working-around it)

Comment 3 Rex Dieter 2017-01-10 14:38:59 UTC
that said, we (and PyQwt upstream) probably have little to no interest maintaining compatibility for such old sip versions

Comment 4 Sandro Mani 2017-01-10 14:52:54 UTC
@Rex: The codebase was half-ported to newer SIP verison, i.e. where the
#if SIP_VERSION < 0x040800
appeared. To make it easier to grep through the codebase to ensure I ported everything, I removed the code for the old SIP versions.

Note You need to log in before you can comment on or make changes to this bug.