Skip to content

Updated project to work again#3

Open
DeexCoding wants to merge 2 commits into
mmozeiko:masterfrom
DeexCoding:master
Open

Updated project to work again#3
DeexCoding wants to merge 2 commits into
mmozeiko:masterfrom
DeexCoding:master

Conversation

@DeexCoding

Copy link
Copy Markdown

Updated project, because it did not work, but it came at the cost of compatibility:

  • No default paths for ANDROID_NDK, ANDROID_SDK, JAVA_JDK in a.cmd
  • Android 6 is the minimum version now

@mmozeiko

Copy link
Copy Markdown
Owner

I think you misunderstood why I have those ANDROID_NDK/SDK/JAVA_JDK env variables in bat file.

They were never installed in specific location. You can place them wherever you want. I had then in those locations, so it's easier for me to hardcode them. But the bat file first checks for env variables. So if you have set env variables then it will automatically use them, not hardcoded locations.

So there's no reason to remove them. If you have custom locations, just set env variables and that's it.

Other changes are fine.

@DeexCoding

Copy link
Copy Markdown
Author

Sorry for not updating earlier, exam season was really tough.

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.

2 participants