Skip to content
This repository has been archived by the owner on May 10, 2021. It is now read-only.

Update setupEnv-android-mac.js #73

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

Luci2015
Copy link
Collaborator

  • copyright update
  • removal of unnecessary functions and imports
  • fixing the double PATH env variable creation for /platform-tools and /tools
  • roll-back the android_home as per the 'aemm platform install android' default location instead of Android Studio default sdk path
  • using quotes for existing PATH values so that errors for existing "space" char inside PATH be averted

- copyright update
- removal of unnecessary functions and imports
- fixing the double PATH env variable creation for /platform-tools and /tools
- roll-back the android_home as per the 'aemm platform install android' default location instead of Android Studio default sdk path
- using quotes for existing PATH values so that errors for existing "space" char inside PATH be averted
@codecov-io
Copy link

codecov-io commented Dec 17, 2018

Codecov Report

Merging #73 into master will decrease coverage by 0.08%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #73      +/-   ##
==========================================
- Coverage   38.25%   38.16%   -0.09%     
==========================================
  Files          25       25              
  Lines        1286     1289       +3     
  Branches      167      167              
==========================================
  Hits          492      492              
- Misses        794      797       +3
Impacted Files Coverage Δ
src/build-android.js 100% <ø> (ø) ⬆️
src/platform.js 23.52% <ø> (ø) ⬆️
src/config.js 100% <ø> (ø) ⬆️
src/aemm-config.js 100% <ø> (ø) ⬆️
src/build-ios.js 100% <ø> (ø) ⬆️
src/platform-ios.js 100% <ø> (ø) ⬆️
src/package-android.js 0% <ø> (ø) ⬆️
src/package.js 100% <ø> (ø) ⬆️
src/run-android.js 0% <ø> (ø) ⬆️
src/build.js 100% <ø> (ø) ⬆️
... and 12 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c84272...52f4998. Read the comment docs.

Fixing error for AVD creation fail by updating the ./android command to be adapted to the 27.0.1 build and platform.
This is just to escape the error, but the emulator will not work anymore since it will require /emulator/emulator from the Android Studio  Tools.
android-emulator -> using dedicated emulator; chaning path to emulator install
src/platform-android.js:
-> changing download source to be the android studio tools instead of a default sdk
-> use sdkmanager to download packages instead of deprecated ./android
-> use avdmanager to create AVD instead of deprecated ./android
-> updated packages to install along with platform android-27 (extras;google;google_play_services required for the gradle fodler that changed location since android-24)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants