Skip to content
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

Remove all conditional checks for Qt 4.x #5505

Merged
merged 1 commit into from
Apr 21, 2017

Conversation

vladimiroff
Copy link
Contributor

Now that #2558 has been closed there is no point of having to check whether Qt is <5.0.

@CLAassistant
Copy link

CLAassistant commented Jan 30, 2017

CLA assistant check
All committers have signed the CLA.

@michaelstingl
Copy link
Contributor

#2558 has been replaced by #5470 (2.3.1 release)

@ogoffart
Copy link
Contributor

ogoffart commented Feb 6, 2017

We still have to wait a bit before we can totally stop supporting Qt 4.
Then we will have to decide what's the minimal version of Qt we support. And we can also drop all the old Qt 5.x.

@guruz guruz added this to the 2.4.0 milestone Mar 24, 2017
@guruz guruz mentioned this pull request Apr 15, 2017
@guruz
Copy link
Contributor

guruz commented Apr 20, 2017

@vladimiroff This has merge conflicts now, could you re-base?

@guruz
Copy link
Contributor

guruz commented Apr 21, 2017

thanks @vladimiroff

@guruz guruz merged commit 3a31694 into owncloud:master Apr 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants