Skip to content

Add Qt5 support #38

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Add Qt5 support #38

wants to merge 1 commit into from

Conversation

jcfr
Copy link
Member

@jcfr jcfr commented Feb 18, 2016

PythonQt_QT_VERSION cmake option is added ("4" or "5"). If "5",
PythonQt_Wrap_QtXYZ use a XYZ with the same Qt5 casing, if "4" it is lower
case, e.g. PythonQt_Wrap_QtWebKit vs PythonQt_Wrap_Qtwebkit

To be done:

  • Rebase Qt5 support on PythonQt trunk (some changes in this commit
    (e.g. in PythonQt.cpp) might not be due to a Qt5 support)
  • Fix XmlPatterns
  • Split generated_cpp folders regarding to Qt modules.
  • Remove findCharOffset() and use QByteArray methods instead
  • Bonus: Factorize method signature processings
  • Fix warning:
    src/PythonQtConversion.cpp: In static member function ‘static QVariant PythonQtConv::PyObjToQVariant(PyObject_, int)’:
    src/PythonQtConversion.cpp:1044:27: warning: deprecated conversion from string constant to ‘char_’ [-Wwrite-strings]

Based on Melven Zöllner work (
http://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)

PythonQt_QT_VERSION cmake option is added ("4" or "5"). If "5",
PythonQt_Wrap_QtXYZ use a XYZ with the same Qt5 casing, if "4" it is lower
case, e.g. PythonQt_Wrap_QtWebKit vs PythonQt_Wrap_Qtwebkit

To be done:
* Rebase Qt5 support on PythonQt trunk (some changes in this commit
(e.g. in PythonQt.cpp) might not be due to a Qt5 support)
* Fix XmlPatterns
* Split generated_cpp folders regarding to Qt modules.
* Remove findCharOffset() and use QByteArray methods instead
* Bonus: Factorize method signature processings
* Fix warning:
src/PythonQtConversion.cpp: In static member function ‘static QVariant PythonQtConv::PyObjToQVariant(PyObject*, int)’:
src/PythonQtConversion.cpp:1044:27: warning: deprecated conversion from string constant to ‘char*’ [-Wwrite-strings]

Based on Melven Zöllner work (
http://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
@jcfr
Copy link
Member Author

jcfr commented Feb 18, 2016

Created this PR so that it can be referenced in commit message and issue.

@jcfr jcfr closed this Feb 18, 2016
@jcfr jcfr deleted the add-qt5-support branch February 18, 2016 09:44
jcfr added a commit to jcfr/PythonQt that referenced this pull request Feb 18, 2016
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See commontk#38)
 * Arnaud Barre (@Alzathar):See commontk#15
 * Eric Heim (@eric-h):See commontk#36
mwoehlke-kitware pushed a commit to mwoehlke-kitware/PythonQt that referenced this pull request May 23, 2016
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See commontk#38)
 * Arnaud Barre (@Alzathar):See commontk#15
 * Eric Heim (@eric-h):See commontk#36
jcfr added a commit that referenced this pull request Jan 17, 2017
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Jun 11, 2017
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Jun 11, 2017
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Aug 8, 2017
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Apr 1, 2019
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Sep 5, 2019
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jcfr added a commit that referenced this pull request Aug 2, 2022
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See #38)
 * Arnaud Barre (@Alzathar):See #15
 * Eric Heim (@eric-h):See #36
jamesobutler pushed a commit to jamesobutler/PythonQt that referenced this pull request Dec 12, 2023
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See commontk#38)
 * Arnaud Barre (@Alzathar):See commontk#15
 * Eric Heim (@eric-h):See commontk#36
jamesobutler pushed a commit to jamesobutler/PythonQt that referenced this pull request Dec 13, 2023
Based on work from the following individual:
 * Melven Röhrig-Zöllner (https://sourceforge.net/p/pythonqt/discussion/631392/thread/5f20c176/)
 * Julien Finet (@finetjul): See commontk#38)
 * Arnaud Barre (@Alzathar):See commontk#15
 * Eric Heim (@eric-h):See commontk#36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants