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) #1419

Conversation

swiftnav-svc-jenkins
Copy link
Collaborator

@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
Copy link

sonarcloud bot commented Jul 31, 2024

@sbmueller
Copy link
Contributor

sbmueller commented Nov 6, 2024

This PR's changes are a subset of #1440 , closing

@sbmueller sbmueller closed this Nov 6, 2024
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