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

Auto submodule update: cmake - orion cmake configuration fails when CMAKE_C_COMPILED_ID is not set [OI-1869] (#176) #491

Merged
merged 1 commit into from
Aug 1, 2024

Conversation

swiftnav-svc-jenkins
Copy link
Contributor

@swiftnav-svc-jenkins swiftnav-svc-jenkins commented Jul 31, 2024

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

cmake 68306be8 -> 5b65c03c

This pull request was created by https://jenkins.ci.swift-nav.com/job/CI%20Infra/job/submodule-update/18075/.

…ot set [OI-1869]

## Jira Ticket

https://swift-nav.atlassian.net/browse/OI-1869

## Bug description

orion build fails with

```bash
CMake Error at third_party/orion-engine/third_party/auk/cmake/swift_nav/LanguageStandards.cmake:63 (if):
  if given arguments:

    "STREQUAL" "IAR"

  Unknown arguments specified
Call Stack (most recent call first):
  third_party/orion-engine/third_party/auk/cmake/swift_nav/SwiftTargets.cmake:351 (swift_set_language_standards)
  third_party/orion-engine/third_party/auk/cmake/swift_nav/SwiftTargets.cmake:373 (swift_add_target)
  proto/CMakeLists.txt:29 (swift_add_library)
```

CMake syntax allows using the variable without ${} to simultaneously
check if variable is defined, and variable content.


## Testing

Updated
`third_party/orion-engine/third_party/auk/cmake/swift_nav/LanguageStandards.cmake`
locally with the fix, CMake config runs correctly.

Triggered-By:	cmake	5b65c03ce8154a2a3a9ae06623849257f87d6dd9
Upstream-PR:	http://github.com/swift-nav/cmake/pull/176
@armallen armallen merged commit 5265add into master Aug 1, 2024
9 checks passed
@armallen armallen deleted the auto-submodule-pointer-update/id-5b65c03ce8 branch August 1, 2024 09:55
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.

2 participants