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

scx_utils::BpfBuilder: properly detect clang version in Ubuntu #13

Merged
merged 1 commit into from
Dec 7, 2023

Conversation

arighi
Copy link
Collaborator

@arighi arighi commented Dec 7, 2023

Apply the same logic of commit 00cd15a ("build: properly detect clang version in Ubuntu") in scx_utils as well.

This allows to build scx_utils properly in Ubuntu.

Apply the same logic of commit 00cd15a ("build: properly detect clang
version in Ubuntu") in scx_utils as well.

This allows to build scx_utils properly in Ubuntu.

Signed-off-by: Andrea Righi <andrea.righi@canonical.com>
@htejun htejun merged commit b0bf163 into sched-ext:main Dec 7, 2023
@htejun
Copy link
Contributor

htejun commented Dec 7, 2023

@arighi Once things look okay for ubuntu builds, please let me know. I'll cut another release so that the published scx_utils works for ubuntu out of the box.

@arighi
Copy link
Collaborator Author

arighi commented Dec 7, 2023

@arighi Once things look okay for ubuntu builds, please let me know. I'll cut another release so that the published scx_utils works for ubuntu out of the box.

@htejun With this additional fix applied I was able to build everything in scx on Ubuntu 24.04 (and I can use the test schedulers with my own sched-ext enabled Ubuntu kernel - that I'm planning to publish soon in a ppa).

I haven't packaged scx yet, so it still requires to manually apt install a bunch of dependent packages and build everything from source, but I'm planning to create a proper deb also for the scx stuff, one step at a time. :)

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.

None yet

2 participants