Skip to content
Permalink
Browse files

remove sip workaround (fixes #17038, refs #16071)

  • Loading branch information
jef-n committed Aug 20, 2017
1 parent 9018414 commit 7e4345a9db640b852c25a11ff99df107a6960400
Showing with 0 additions and 1 deletion.
  1. +0 −1 python/core/qgscoordinatetransform.sip
@@ -15,7 +15,6 @@
class QgsCoordinateTransform : QObject
{
%TypeHeaderCode
extern PyObject *sipExportedExceptions__core[2]; // workaround: sipExportedExceptions__core is only defined in the first sip part
#include <qgscoordinatetransform.h>
%End

7 comments on commit 7e4345a

@3nids

This comment has been minimized.

Copy link
Member

@3nids 3nids replied Aug 22, 2017

getting an error here
image
I did clean without success...

@3nids

This comment has been minimized.

Copy link
Member

@3nids 3nids replied Aug 22, 2017

reverting this allows me to compile....any idea @jef-n ?

@jef-n

This comment has been minimized.

Copy link
Member Author

@jef-n jef-n replied Aug 22, 2017

upgrade sip? which version are you using?

@3nids

This comment has been minimized.

Copy link
Member

@3nids 3nids replied Aug 22, 2017

4.19.1

@jef-n

This comment has been minimized.

Copy link
Member Author

@jef-n jef-n replied Aug 22, 2017

@jef-n

This comment has been minimized.

Copy link
Member Author

@jef-n jef-n replied Aug 22, 2017

Does that help for you?

diff --git a/python/core/qgscoordinatetransform.sip b/python/core/qgscoordinatetransform.sip
index f9b7854c32..900967d37f 100644
--- a/python/core/qgscoordinatetransform.sip
+++ b/python/core/qgscoordinatetransform.sip
@@ -15,7 +15,9 @@
 class QgsCoordinateTransform : QObject
 {
 %TypeHeaderCode
+#if (SIP_VERSION >= 0x041300 && SIP_VERSION < 0x041303)
 extern PyObject *sipExportedExceptions__core[2];  // workaround: sipExportedExceptions__core is only defined in the first sip part
+#endif
 #include <qgscoordinatetransform.h>
 %End

Not sure whether 4.19.2 is also affected though...

@3nids

This comment has been minimized.

Copy link
Member

@3nids 3nids replied Aug 22, 2017

yep, it does the trick!

Please sign in to comment.
You can’t perform that action at this time.