tube42 / drumon

FOSS drum app for Android
GNU General Public License v2.0
85 stars 13 forks source link

bump android studio and dependencies, compile with java 1.8 #16

Closed TacoTheDank closed 5 years ago

TacoTheDank commented 5 years ago

I did this through a text editor; no Android Studio.

tube42 commented 5 years ago

Thanks, this looks so much better!

I cannot max all versions to latest the way you do since there are some incompatibilities across build environments (f-droid vs travis vs my computer) and tool combinations (e.g. gradle vs Android SDK) that make this fail.

So I will probably need to pull this manually.

tube42 commented 5 years ago

added this PR manually, see commit 035ebce092770efe5dc66efd910e8e39bc13be31

TacoTheDank commented 5 years ago

Yeah, thank you. I figured I would do it this way to minimize incompatibilities, so that if anything didn't build, it would probably be something you would need to change on your end.

TacoTheDank commented 5 years ago

Curious, what computer setup do you use that somehow causes all these incompatibilities?

Also why did you rename the applicationId? That would mean F-Droid needs to build yet another app. https://github.com/tube42/drumon/commit/5a21ef6b1807c2bc0b94ca33771cc19bfeaf86dc#diff-7ae5a9093507568eabbf35c3b0665732R21

tube42 commented 5 years ago

Hmm, the ID change was not intentional.

Different machines have different combinations of the SDK and built tools. They are pretty large so not all of them can be installed on a small build server.