-
Notifications
You must be signed in to change notification settings - Fork 217
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Autocomplettion with some IDE #170
Comments
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 21, 2017
- Add other paramether to `setup_arduino_libraries` for includes: `INCLUDE_PATHS` - Add other paramether to `setup_arduino_library` for includes: `INCLUDE_PATHS` - Removed manual `-I` as manual `COMPILE_FLAGS` with `set_target_properties` in setup_arduino_library` - Add the more CMake semantic `target_include_directories` in `setup_arduino_library`
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 21, 2017
- Add other parameter to `get_arduino_flags` for get the include paths as a list: INCLUDE_PATHS_VAR - Usage on `GENERATE_ARDUINO_LIBRARY`, `setup_arduino_core`, `setup_arduino_library` and `setup_arduino_target` functions
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 21, 2017
- Removed `set(${VAR_NAME}_INCLUDES ${LIB_INCLUDES} PARENT_SCOPE)` from `setup_arduino_library` because it is automatically now, the `target_include_directories` on this function is in `PUBLIC` scope. Read more on: https://cmake.org/cmake/help/v3.0/command/target_include_directories.html - Removed `set(${VAR_NAME}_INCLUDES ${LIB_INCLUDES} PARENT_SCOPE)` from `setup_arduino_libraries` cuzy the same... previous.
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 21, 2017
- Some fix after test.
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 21, 2017
working on issue queezythegreat#170. do not close this yet
denisacostaq
added a commit
to denisacostaq/arduino-cmake
that referenced
this issue
Mar 23, 2017
- Add matching macro for mcu, the compiler make this by default, but we need notice the IDE about this.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Some IDE know cmake, and can not auto-complete from some include paths, it build ok because the compiler receive the include path with -I flag, but it is no know in the others levels of the build system, for example, qtcreator.
The text was updated successfully, but these errors were encountered: