Skip to content

[GHA] Fix Minimal CMake version test - #12283

Merged
Nir-Az merged 5 commits into
realsenseai:developmentfrom
Tamir91:fix-GHA-CMake-minimal-version-test
Oct 11, 2023
Merged

[GHA] Fix Minimal CMake version test#12283
Nir-Az merged 5 commits into
realsenseai:developmentfrom
Tamir91:fix-GHA-CMake-minimal-version-test

Conversation

@Tamir91

@Tamir91 Tamir91 commented Oct 11, 2023

Copy link
Copy Markdown
Contributor

Created variable for working directory instead hardcoded links in the script.

@Tamir91
Tamir91 requested a review from Nir-Az October 11, 2023 14:23
Comment thread .github/workflows/static_analysis.yaml
@Tamir91
Tamir91 requested a review from Nir-Az October 11, 2023 16:39
Comment thread .github/workflows/static_analysis.yaml Outdated
echo "The test PASSED, current CMake version is $CURRENT_CMAKE_VERSION and it is not higher than ${EXPECTED_CMAKE_VERSION}"
else
echo "Error - LibRS minimal CMake version require is ${EXPECTED_CMAKE_VERSION} but on this build the minimal is $CURRENT_CMAKE_VERSION"
echo "Error - LibRS minimally CMake version required is ${EXPECTED_CMAKE_VERSION} but on this build a minimally CMake version is $CURRENT_CMAKE_VERSION"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The minimal CMake version required for LibRS is ${EXPECTED_CMAKE_VERSION} but on this build the minimal CMake version that works is $CURRENT_CMAKE_VERSION"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Renamed

@Tamir91
Tamir91 requested a review from Nir-Az October 11, 2023 19:34

@Nir-Az Nir-Az left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good thanks

@Nir-Az
Nir-Az merged commit b8133c3 into realsenseai:development Oct 11, 2023
@Tamir91
Tamir91 deleted the fix-GHA-CMake-minimal-version-test branch November 16, 2023 15:17
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