Comments (2)
Hi from the peanut gallery.
The problem comes from this: android.toolchain.cmake line 42
Inhibit all of CMake's own NDK handling code.
set(CMAKE_SYSTEM_VERSION 1)
When using the toolchain that is shipped with the ndk there's a specific check for CMAKE_SYSTEM_VERSION greater than 22 at line 46 of the generated cdep-dependencies-config.cmake
if(CMAKE_SYSTEM_NAME STREQUAL "Android")
if(CMAKE_SYSTEM_VERSION GREATER 22)
...
else()
message (FATAL_ERROR "") <-- that's the failure you're seeing.
If you use a system cmake or one that doesn't use android.toolchain.cmake then the CMAKE_SYSTEM_VERSION will be set correctly and there's no need for the workaround you're deleting here.
CMAKE_SYSTEM_VERSION is a correct check to ensure that the platform OS being targeted is at a specific platform. It is populated by buildSystemTargetPlatform. I'd venture this is a working as designed bug.
The only real suggestion I'd have is to place a new variable in android.toolchain.cmake that doesn't conflict with CMAKE_SYSTEM_VERSION and cdep can check both? This would eliminate the need for the workaround and mask the issue to users.
from cdep.
yeah, that sounds correct. thank you!
from cdep.
Related Issues (20)
- include should be abi capable HOT 1
- On android-studio-cmake.md it's not clear which build.gradle file to edit
- Error:ABIs [armeabie] are not available for platform...
- List of pros and cons of CDep
- more than one example
- CFlags ldFlags promotion HOT 1
- cdep-manifest.yml parsing
- improve authoring.md step 14 HOT 1
- order of dependencies HOT 2
- ndk-build spaces vs tabs
- Error: SHA256 on Step 13 of the authoring guide HOT 2
- ios support HOT 9
- CMAKE_SYSTEM_VERSION is 24 HOT 3
- gitlab support HOT 2
- libxml2 package
- Error with CMakeLists.txt HOT 1
- There is a vulnerability in SnakeYAML 1.17,upgrade recommended
- Security Policy violation Binary Artifacts HOT 134
- https://raw.githubusercontent.com/jomof/cdep/master/boot.yml HOT 1
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from cdep.