Skip to content

Commit

Permalink
[CMakeToolchain] fix adding tools.android:ndk_path with spaces in path (
Browse files Browse the repository at this point in the history
#17379)

* fix adding tools.android:ndk_path with spaces in path

* fix test

* fix quotes
  • Loading branch information
kambala-decapitator authored Nov 26, 2024
1 parent 6bdd529 commit aa4920e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion conan/tools/cmake/toolchain/blocks.py
Original file line number Diff line number Diff line change
Expand Up @@ -375,7 +375,7 @@ class AndroidSystemBlock(Block):
{% if android_use_legacy_toolchain_file %}
set(ANDROID_USE_LEGACY_TOOLCHAIN_FILE {{ android_use_legacy_toolchain_file }})
{% endif %}
include({{ android_ndk_path }}/build/cmake/android.toolchain.cmake)
include("{{ android_ndk_path }}/build/cmake/android.toolchain.cmake")
""")

def context(self):
Expand Down
2 changes: 1 addition & 1 deletion test/integration/toolchains/cmake/test_cmaketoolchain.py
Original file line number Diff line number Diff line change
Expand Up @@ -831,7 +831,7 @@ def configure(self):
conan_toolchain = client.load(os.path.join(client.current_folder, "conan_toolchain.cmake"))
assert "set(ANDROID_PLATFORM android-23)" in conan_toolchain
assert "set(ANDROID_ABI x86_64)" in conan_toolchain
assert "include(/foo/build/cmake/android.toolchain.cmake)" in conan_toolchain
assert 'include("/foo/build/cmake/android.toolchain.cmake")' in conan_toolchain
client.run("create . --name=foo --version=1.0 " + settings)


Expand Down

0 comments on commit aa4920e

Please sign in to comment.