-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Downgrade zink + some other fixes #6752
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
Downgrade zink + some other fixes #6752
Conversation
This reverts commit f88fb3c. Mali devices are severely screwed over by this change.
This makes it override anything that could be set by the launcher, which is the point. Also it looks cleaner imo.
I'm really annoyed at how the version file keeps changing and it seems dumb
Using 8 is causing some really weird unreproducable shitfuckery that is likely related to caches. Co-authored-by: ItsWinterBush <[email protected]>
I don't see issues with this from a quick look at the code. |
build.gradle
Outdated
def command = Runtime.getRuntime().exec("git rev-list -1 HEAD " + project) | ||
def result = command.waitFor() | ||
if (result != 0) { | ||
throw new IOException("Command 'getGitHash()' exited with " + result) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Note for a future PR/commit:
Enforce failing the build if the repository isn't cloned with Git. Not cloning with Git breaks submodules and this code already.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
How do you suppose just letting it silently fail and not edit the versions file?
This is to fix `aar-metadata.properties (No such file or directory)` See leancodepl/patrol#2181 for similar issue
oh uh oops i mistapped |
LGTM. |
No description provided.