Skip to content

Conversation

@thezofx
Copy link
Contributor

@thezofx thezofx commented Jul 24, 2020

Fixing Android app Build

thezofx added 6 commits July 23, 2020 14:15
 ext.kotlin_version = '1.2.30' -> '1.3.72'
classpath 'com.android.tools.build:gradle:3.1.3 -> 4.0.0
compileSdkVersion 27 -> 30
targetSdkVersion 27 -> 30

implementation 'com.android.support.constraint:constraint-layout:1.1.2'
-> implementation 'com.android.support.constraint:constraint-layout:1.1.+'

testImplementation 'junit:junit:4.12 - > testImplementation 'junit:junit:4.+'

added: implementation 'com.android.support:design:27.1.1'
added missing code
v1.200 -> https://github.com/googlevr/gvr-android-sdk
(code\mobile\android\libraries\jni)
updating Gradle and CMake
rewritten MainActivity : AppCompatActivity().onResume()
@ShootingKing-AM ShootingKing-AM merged commit 1952daa into PhoneVR-Developers:android-fix Jul 24, 2020
ShootingKing-AM added a commit that referenced this pull request Jan 28, 2021
Crash stacktrace:
```ml
pid: 10065, tid: 10136, name: GLThread 235  >>> viritualisres.phonevr <<<
uid: 10155
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x8
Cause: null pointer dereference
    eax 00257100  ebx da946db4  ecx 00000008  edx 00000002
    edi ac68dbb0  esi 00000080
    ebp ac68d948  esp ac68d910  eip da7e04fb

backtrace:
      #00 pc 000de4fb  /system/lib/libndk_translation.so (ndk_translation::InsnInterpreter::FaultyStore(ndk_translation::DecoderReg, ndk_translation::DecoderReg)+395) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #1 pc 000e1dc6  /system/lib/libndk_translation.so (ndk_translation::(anonymous namespace)::ArmInterpreter::GenFaultyStore(int, int)+54) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #2 pc 000f6a68  /system/lib/libndk_translation.so (ndk_translation::SemanticsDecoder::STR_immediate(ndk_translation::STR_immediate_Args const&)+744) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #3 pc 0012cf9b  /system/lib/libndk_translation.so (ndk_translation::ArmDecoder::TranslateThumbInsn(unsigned short const*)+5995) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #4 pc 000e1700  /system/lib/libndk_translation.so (ndk_translation::(anonymous namespace)::DecodeInsn(unsigned int, ndk_translation::(anonymous namespace)::ArmInterpreter*, ndk_translation::ProcessState*, ndk_translation::ArmDecoder*)+128) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #5 pc 000e1934  /system/lib/libndk_translation.so (ndk_translation::InterpretTrace(ndk_translation::ProcessState*)+340) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #6 pc 000d1b32  /system/lib/libndk_translation.so (ndk_translation_HandleNotTranslated+194) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #7 pc 000cfffa  /system/lib/libndk_translation.so (ndk_translation::ExecuteGuest(ndk_translation::ProcessState*, unsigned int)+266) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #8 pc 000d0274  /system/lib/libndk_translation.so (ndk_translation::(anonymous namespace)::RunGuestCall(unsigned int, int, int, unsigned int*)+260) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #9 pc 000d0399  /system/lib/libndk_translation.so (ndk_translation::GuestCall::RunResInt32(unsigned int)+41) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #10 pc 000cda44  /system/lib/libndk_translation.so (ndk_translation::(anonymous namespace)::RunGuestThread(void*)+164) (BuildId: 1962273ccbeb69757878abf4e17d95aa)
      #11 pc 000e42c4  /apex/com.android.runtime/lib/bionic/libc.so (__pthread_start(void*)+100) (BuildId: 57f4eb5e1df9fa8bd21032ad6be9823f)
      #12 pc 000785f7  /apex/com.android.runtime/lib/bionic/libc.so (__start_thread+71) (BuildId: 57f4eb5e1df9fa8bd21032ad6be9823f)
```

Updating ndk to r22 fixes this crash.
ShootingKing-AM added a commit that referenced this pull request Apr 25, 2023
* update ndk

* Update AGP and gradle (#3)

* Update AGP and gradle versions to latest Stable

* update toolchain

* Update build.gradle

* Update build.yml

* Update CMakeLists.txt

* disable test Job

* Delete appveyor.yml
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