wagic icon indicating copy to clipboard operation
wagic copied to clipboard

travis release android builds(maybe others) are debug builds.

Open zethfoxster opened this issue 8 years ago • 12 comments

anyway to get the auto builds from travis to do release builds for android? havent checked others i would assume theyre being made as debug builds too.

zethfoxster avatar May 31 '16 22:05 zethfoxster

Sounds like Travis/Appveyor releases on Mac and Windows contains the testsuite as well and should not.

xawotihs avatar Jun 06 '16 18:06 xawotihs

Yeah the preformance difference from a true release build compared to debug is huge, very noticable.

zethfoxster avatar Jun 06 '16 20:06 zethfoxster

Windows Appveyor build does not show the test suite and is actually built in release. One checked.

xawotihs avatar Jun 06 '16 20:06 xawotihs

Noticed i cant get a true release build manually either. I go through the process, enter the password, install the release version apk...test suite and ai testing in menu.

zethfox avatar Jul 01 '16 03:07 zethfox

Travis_scripts.sh, the scripts here should be for "release" not "debug"

zethfoxster avatar Jul 01 '16 17:07 zethfoxster

if [ "$BUILD_ANDROID" = "YES" ]; then android-ndk-r9/ndk-build -C projects/mtg/Android -j4 $ANDROID list targets $ANDROID update project -t 1 -p projects/mtg/Android ant debug -f projects/mtg/Android/build.xml fi

Notice its building for debug

zethfoxster avatar Jul 01 '16 17:07 zethfoxster

https://gist.github.com/mariotaku/7a0c51955d14def2fa0e

Not sure if this would help us

zethfoxster avatar Jul 01 '16 19:07 zethfoxster

hey Xaw, is there anyway you can help us with this one? Ive tested 2 devices with both a personally built release build and a "release" debug build off our auto builds, the difference in preformence is astronomical.

zethfoxster avatar Jul 18 '16 23:07 zethfoxster

If you know what you changed to create your personal release built, we should be able to put it in Travis.

xawotihs avatar Jul 19 '16 05:07 xawotihs

I used command prompt, and where i mentioned earlier in this report i wrote "release" instead of debug.

But how do you tell travis to use a keystore?

zethfoxster avatar Jul 19 '16 10:07 zethfoxster

you are my heros, thank you. Android is not debug anymore from what im seeing. it wasnt so much about hiding the test suite options and stuff, but the difference between the release vs debug in preformence. now we just need to see if the rest of the releases are also not debug.

zethfoxster avatar Jul 20 '16 23:07 zethfoxster

Android, psp, win are good. Can an iphone/linux user confirm this is fixed.

zethfoxster avatar Jul 21 '16 16:07 zethfoxster