Skip to content
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

Static bundles for the IAR toolchain[DRAFT] #177

Merged
merged 3 commits into from
Aug 8, 2024

Conversation

reimerix
Copy link
Contributor

@reimerix reimerix commented Jul 31, 2024

We made a release to Asensing but the IAR library was not bundled properly and static linking was not done.
The plan is to use add_static_library_bundle but, the archive code is not compatible with IAR.

A flag was added that is only set during IAR compilation and installs.

Currently, we are using CMAKE but plan to move to Bazel after this patch release

Copy link

sonarcloud bot commented Aug 1, 2024

@sokhealy sokhealy marked this pull request as ready for review August 8, 2024 18:44
@sokhealy sokhealy requested a review from a team as a code owner August 8, 2024 18:44
@sokhealy sokhealy merged commit fa93409 into master Aug 8, 2024
2 checks passed
@sokhealy sokhealy deleted the reimerix/iar-static-bundle branch August 8, 2024 20:58
sbmueller pushed a commit to swift-nav/libsbp that referenced this pull request Nov 6, 2024
…178)" (#179) (#1440)

Automated PR by Jenkins. If CI has passed successfully, merge away!



**cmake**  65c9a396  ->  87ff9dfa
- 87ff9dfa : Revert "CMake patches for QNX Support (#178)"
(swift-nav/cmake#179)
-  5d9f739e  :  CMake patches for QNX Support (swift-nav/cmake#178)
- fa934097 : Static bundles for the IAR toolchain[DRAFT]
(swift-nav/cmake#177)
- 5b65c03c : orion cmake configuration fails when CMAKE_C_COMPILED_ID is
not set [O


This pull request was created by
https://jenkins.ci.swift-nav.com/job/CI%20Infra/job/submodule-update/18385/.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants