Skip to content
This repository has been archived by the owner on Jul 4, 2023. It is now read-only.

qscintilla2 fails to build on 10.11: sip files generating qt5 API, but only qt4 installed #41464

Closed
ejtttje opened this issue Jul 7, 2015 · 13 comments

Comments

@ejtttje
Copy link

ejtttje commented Jul 7, 2015

When generating the python bindings, the sip files all die with error:

./sipQsciQsciLexerCSharp.cpp:194:43: error: no viable conversion from 'const QMetaMethod' to 'const char *'
        QsciLexerCSharp::disconnectNotify(a0);
                                          ^~
/Volumes/Xcode/Brew/Cellar/qt/4.8.7/lib/QtCore.framework/Versions/4/Headers/qobject.h:291:47: note: passing argument to parameter 'signal' here
    virtual void disconnectNotify(const char *signal);
                                              ^
./sipQsciQsciLexerCSharp.cpp:212:40: error: no viable conversion from 'const QMetaMethod' to 'const char *'
        QsciLexerCSharp::connectNotify(a0);
                                       ^~
/Volumes/Xcode/Brew/Cellar/qt/4.8.7/lib/QtCore.framework/Versions/4/Headers/qobject.h:290:44: note: passing argument to parameter 'signal' here
    virtual void connectNotify(const char *signal);
                                           ^

QT4 used the char* argument for these methods, QT5 switched it to use QMetaMethod instead.
So it looks like sip is generating QT5 calls even though I only have QT4 installed? Not sure if this is a bug/issue with sip, or something in the build configuration for qscintilla...

@ejtttje ejtttje changed the title qscintilla2 failed to build on 10.11: sip files generating qt5 API, but only qt4 installed qscintilla2 fails to build on 10.11: sip files generating qt5 API, but only qt4 installed Jul 7, 2015
@DomT4 DomT4 added the 10.11 label Jul 8, 2015
@tdsmith
Copy link
Contributor

tdsmith commented Jul 16, 2015

Can you share the output of brew gist-logs qscintilla2 please?

@apjanke
Copy link
Contributor

apjanke commented Sep 14, 2015

I'm seeing a similar error on my 10.11 beta test box. Here's my brew gist-logs.

https://gist.github.com/13d029521a2952a49870

@yarko
Copy link

yarko commented Sep 19, 2015

I'm seeing this (exact same thing) on os/x 10.10.5

@ecerrillo
Copy link

@tdsmith
Copy link
Contributor

tdsmith commented Oct 3, 2015

¯_(ツ)_/¯ I don't know how to debug this. It looks like everything is configured correctly: https://gist.github.com/anonymous/d190af428dff04c61643#file-04-python-L15-L30

I suggest reaching out to the qscintilla developers. It looks like it's been raised on this thread: https://www.riverbankcomputing.com/pipermail/qscintilla/2015-September/001067.html -- the correct qmake is being invoked and PyQt4 is being correctly selected.

@schoeps
Copy link

schoeps commented Oct 5, 2015

I can build qscintilla2 on El Capitan without the python bindings which I do not need (for Octave). May I suggest that the user can disable the python bindings? see e.g. https://gist.github.com/6d3f1008b9912f70f7ce

@tdsmith
Copy link
Contributor

tdsmith commented Oct 5, 2015

@dakcarto
Copy link
Contributor

dakcarto commented Oct 7, 2015

Ran into this, I then bumped my local formulae of pyqt and qscintilla2 to versions 4.11.4 and 2.9 respectively and had no problems compiling/installing (even on OS X 10.7). Not sure if this addresses this particular issue, but it is worth noting that the version bumps may be a good first step towards a resolution.

@apjanke
Copy link
Contributor

apjanke commented Oct 7, 2015

That looks like a fix. If I bump both versions, they build and install on OS X 10.11 for me. Submitting PRs for the version bumps...

#44726

@ecerrillo
Copy link

It works now!

@McPo
Copy link

McPo commented Oct 19, 2015

I made the changes referenced in the PR, and disabled the OSX sandbox.

Continues to fail with same issue

An internal error occured.  Please report all the output from the program,
including the following traceback, to support@riverbankcomputing.com.
Traceback (most recent call last):
  File "configure.py", line 1664, in <module>
    _main(sys.argv, module_config)
  File "configure.py", line 1637, in _main
    target_config.introspect_pyqt(module_config)
  File "configure.py", line 860, in introspect_pyqt
    from PyQt4 import QtCore
RuntimeError: the sip module implements API v11.0 to v11.2 but the PyQt4.QtCore module requires API v10.1

READ THIS: https://git.io/brew-troubleshooting

These open issues may also help:
Qscintilla2 2.9 https://github.com/Homebrew/homebrew/pull/44726
qscintilla2 doesn't support pyqt5 (only pyqt4?) https://github.com/Homebrew/homebrew/issues/37591
qscintilla2 fails to build on 10.11: sip files generating qt5 API, but only qt4 installed https://github.com/Homebrew/homebrew/issues/41464

@boltomli
Copy link

I rebuilt pyqt and sip, etc. then install 2.9 successfully. You might try that.

@UniqMartin
Copy link
Contributor

The qscintilla formula has been updated to 2.9.1 very recently. Closing as apparently this should resolve the issue reported here. (Please report back if that's not the case.)

@Homebrew Homebrew locked and limited conversation to collaborators Jul 10, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests