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

[dalgona] Support ubuntu 24.04 #14204

Merged
merged 1 commit into from
Oct 13, 2024
Merged

Conversation

hseok-oh
Copy link
Contributor

This commit updates cmake to support ubuntu 24.04.
It requires to install python 3.8 explicitly because the default python version is 3.12.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh [email protected]

This commit updates cmake to support ubuntu 24.04.
It requires to install python 3.8 explicitly because the default python version is 3.12.

ONE-DCO-1.0-Signed-off-by: Hyeongseok Oh <[email protected]>
@hseok-oh hseok-oh added the PR/ready for review It is ready to review. Please review it. label Oct 11, 2024
Copy link
Contributor

@seanshpark seanshpark left a comment

Choose a reason for hiding this comment

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

LGTM

@seanshpark seanshpark requested a review from jinevening October 13, 2024 21:12
# refer https://github.com/Samsung/ONE/issues/9962
find_package(PythonInterp 3.8 QUIET)
find_package(PythonLibs 3.8 QUIET)
if(CMAKE_VERSION VERSION_LESS 3.12)
Copy link
Contributor

@jinevening jinevening Oct 13, 2024

Choose a reason for hiding this comment

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

For review) cmake versions for Ubuntu

image

Copy link
Contributor

@jinevening jinevening left a comment

Choose a reason for hiding this comment

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

LGTM

@jinevening jinevening merged commit 0665747 into Samsung:master Oct 13, 2024
8 checks passed
@hseok-oh hseok-oh deleted the dalgona_noble branch November 19, 2024 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR/ready for review It is ready to review. Please review it.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants