Skip to content

Linux collaterals in single build dir - #11915

Merged
maloel merged 6 commits into
realsenseai:developmentfrom
maloel:linux
Jun 18, 2023
Merged

Linux collaterals in single build dir#11915
maloel merged 6 commits into
realsenseai:developmentfrom
maloel:linux

Conversation

@maloel

@maloel maloel commented Jun 15, 2023

Copy link
Copy Markdown
Contributor

Windows already does this -- so this does the same in Linux:

  • Puts everything together, so it's easy to find and create artifacts
  • Turns out, cmake --build . --config <> doesn't work in Linux!
    • See here: "For single configuration generators like Makefile Generators ... the configuration is specified at configure time by the CMAKE_BUILD_TYPE variable"
    • So I changed all the GHA runners to use it in Linux/Mac/Android
    • This can default to be empty, so the default is Release
  • Had to change LibCI to expect collaterals in same place as Windows

Plus fixed some warnings on Linux...

Note that this has to be accompanied by additional PRs in other repos for our CI.

Tracked on [LRS-817]

@maloel
maloel requested a review from Nir-Az June 15, 2023 05:41
@maloel

maloel commented Jun 15, 2023

Copy link
Copy Markdown
Contributor Author

See this build if you want to check anything:
https://rsjenkins.iil.intel.com/view/LRS/job/LRS_release_flow_pipeline/5246/

@maloel

maloel commented Jun 15, 2023

Copy link
Copy Markdown
Contributor Author

I'm rerunning libCI (manually) - found an issue with another script.
Note that once all the PRs go in, we have to update all branches with development right away or they likely won't pass libCI or Jenkins builds.

@maloel

maloel commented Jun 18, 2023

Copy link
Copy Markdown
Contributor Author

LibCI with all the branches set:
https://rsjenkins.iil.intel.com/job/LRS_libci_pipeline/5130/

@maloel
maloel merged commit 8a0f72d into realsenseai:development Jun 18, 2023
@maloel
maloel deleted the linux branch June 18, 2023 06:38
maloel added a commit to maloel/librealsense that referenced this pull request Aug 7, 2023
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