Skip to content

Enforce Qt5#2157

Merged
jdtournier merged 1 commit intodevfrom
enforce_qt5
Nov 17, 2020
Merged

Enforce Qt5#2157
jdtournier merged 1 commit intodevfrom
enforce_qt5

Conversation

@Lestropie
Copy link
Copy Markdown
Member

Second half of #2142: where the first half (#2153) only alters documentation on master, this goes to dev, and involves removing support for anything prior to 5.5 and performing corresponding checks.

@Lestropie Lestropie added this to the 3.1.0 updates milestone Sep 4, 2020
@Lestropie Lestropie requested a review from jdtournier September 4, 2020 14:38
@Lestropie Lestropie self-assigned this Sep 4, 2020
- Removes precompiler directives that provided support for Qt versions prior to this threshold.
- Apply this requirement both during configure script and during build of GUI commands.
Copy link
Copy Markdown
Member

@jdtournier jdtournier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems to work fine on my Arch system. But then it's running Qt 5.15, and it's hard to force Qt4 back onto it, so I can't realistically check how it handles a Qt4 system.

But then, the main thing is that it still works with Qt >= 5.5, and tries to prevent the use of older versions, so it's all good as far as I'm concerned.

@jdtournier jdtournier merged commit 4973703 into dev Nov 17, 2020
@jdtournier jdtournier deleted the enforce_qt5 branch November 17, 2020 11:53
Lestropie added a commit that referenced this pull request Feb 7, 2021
- Qt 5.5 as per #2157.
- Python3 as per #2215.
@Lestropie Lestropie mentioned this pull request Mar 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants