You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
creates a Nix expression that builds a utility Android SDK that has its licenses accepted. Instead of copying stuff to ~/.status/Android/Sdk we create symlinks to nixpkgs' androidenv package and add some files that contain the license file hashes symbolizing the acceptance. It's possible that a future change to sdkmanager might break this scheme, but there are hundreds of other projects using the same workaround (outside of nix community), so we're not really doing bleeding edge stuff here. We always have the ability to revert to the previous way.
does away with looking at whether an ANDROID_SDK_ROOT folder exists to determine whether to run scripts/setup and instead creates a hidden/ignored file in the repo as a marker.
moves to building Android artifacts in a pure environment (passing --pure to nix-shell so that the host environment has less chances of contaminating the build).
The second commit moves to using a nix package for fastlane on Android, removing the last barrier to pure builds.
Review notes
Looks like we'll need to reimport the project in snyk so that it can learn the new location for Gemfile.lock.
Testing notes
These changes should not affect the functionality, so the automated build should be enough of a test.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR consists of 2 smaller PRs:
~/.status/Android/Sdk
we create symlinks to nixpkgs' androidenv package and add some files that contain the license file hashes symbolizing the acceptance. It's possible that a future change tosdkmanager
might break this scheme, but there are hundreds of other projects using the same workaround (outside of nix community), so we're not really doing bleeding edge stuff here. We always have the ability to revert to the previous way.ANDROID_SDK_ROOT
folder exists to determine whether to runscripts/setup
and instead creates a hidden/ignored file in the repo as a marker.--pure
tonix-shell
so that the host environment has less chances of contaminating the build).Review notes
Looks like we'll need to reimport the project in snyk so that it can learn the new location for Gemfile.lock.
Testing notes
These changes should not affect the functionality, so the automated build should be enough of a test.
status: ready