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.

Sign up for GitHub

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

Jump to bottom

fix versioning check #2818

Merged
merged 1 commit into from
Apr 9, 2013
Merged

fix versioning check #2818

merged 1 commit into from
Apr 9, 2013

Conversation

karlitschek
Copy link
Contributor

The logic was indeed wrong as @arkascha pointed out. Should be fixed now.
@arkascha @Raydiation @DeepDiver1975

@BernhardPosselt
Copy link
Contributor

👍

@karlitschek karlitschek mentioned this pull request Apr 9, 2013
Closed
@arkascha
Copy link
Contributor

arkascha commented Apr 9, 2013

👍
I tried the patch in my local installation (manually replaced the line) and indeed ownCloud succeeded in installing the imprint app afterwards. I cannot say anything about other effects, though ;-)

karlitschek pushed a commit that referenced this pull request Apr 9, 2013
Merge pull request #2818 from owncloud/fix_versioncheck
aef66e0 
fix versioning check
@karlitschek karlitschek merged commit aef66e0 into stable5 Apr 9, 2013
@karlitschek karlitschek deleted the fix_versioncheck branch April 9, 2013 12:35
@lock lock bot locked as resolved and limited conversation to collaborators Aug 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@karlitschek @BernhardPosselt @arkascha