Prefer $TMPDIR to /data/local/tmp on Android #4342
Merged
Conversation
This file contains hidden or 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
Newer devices can have the latter location read-only. (I observed this with Termux on a non-rooted Pixel 6.)
tanzislam
added a commit
to tanzislam/cryptopals
that referenced
this pull request
Aug 18, 2023
[Termux](https://termux.dev) is only available on the F-Droid app store. After installation, I had to install these packages with `pkg install`: - automake - binutils-is-llvm - clang - gettext - git - libcpufeatures - libtool - make and was able to do an out-of-source build in subdirectories of the $HOME directory. (The repo was cloned using the separate app MGit.) Currently this requires google/googletest#4342.
Collaborator
|
I generally agree with the change, but maybe we could reuse an existing utility function by declaring it in googletest/googletest/src/gtest.cc Line 6835 in 460ae98 |
vvb2060
added a commit
to vvb2060/BoringSSL_Android
that referenced
this pull request
Oct 29, 2024
paymenremora
added a commit
to paymenremora/BoringSSL_Android
that referenced
this pull request
Oct 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Newer devices can have the latter location read-only. (I observed this with Termux on a non-rooted Pixel 6.)
Fixes: #4167