Skip to content

Fix ROOT for musl libc#9260

Merged
Axel-Naumann merged 3 commits intoroot-project:masterfrom
balejk:musl
Nov 9, 2021
Merged

Fix ROOT for musl libc#9260
Axel-Naumann merged 3 commits intoroot-project:masterfrom
balejk:musl

Conversation

@balejk
Copy link
Copy Markdown

@balejk balejk commented Nov 8, 2021

It is not possible to compile ROOT with musl libc because in several places it relies on glibc and uses non-standard constructs. This PR aims to fix that, at least to such an extent so that compilation against musl does not fail.

@phsft-bot
Copy link
Copy Markdown

Can one of the admins verify this patch?

@vgvassilev
Copy link
Copy Markdown
Member

@phsft-bot build

@phsft-bot
Copy link
Copy Markdown

Starting build on ROOT-debian10-i386/cxx14, ROOT-performance-centos8-multicore/default, ROOT-ubuntu16/nortcxxmod, ROOT-ubuntu2004/soversion, mac1014/python3, mac11.0/cxx17, windows10/cxx14
How to customize builds

Copy link
Copy Markdown
Member

@Axel-Naumann Axel-Naumann left a comment

Choose a reason for hiding this comment

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

Fantastic, thank you so much!

@Axel-Naumann Axel-Naumann merged commit adefbc8 into root-project:master Nov 9, 2021
@balejk
Copy link
Copy Markdown
Author

balejk commented Nov 9, 2021

@Axel-Naumann Actually, this was not meant to be merged yet, I still haven't resolved all the build issues so it's still not possible to successfully compile with musl (glibc should be fine).

I imagined you would just review the proposed changes and let me know if there was anything problematic. But if you think these changes were fine, I will open another pull request once I have identified and found a potential solution to all the remaining issues.

@Axel-Naumann
Copy link
Copy Markdown
Member

Ha sorry :-) It works for our PR builds, it's certainly an improvement, and we're used to working incrementally. I hope I didn't cause too much trouble on your side! So yes, please keep them coming, especially if they are as nice as this one: removing code!

@balejk
Copy link
Copy Markdown
Author

balejk commented Nov 9, 2021

OK :-)

@balejk balejk deleted the musl branch November 9, 2021 14:14
bellenot added a commit to bellenot/root that referenced this pull request Jul 19, 2023
Prevent potential infinite recursion and simplify some of the code. This fixes the Github issue root-project#13233 and doesn't affect the already closed Jira issue [root-project#9260](https://sft.its.cern.ch/jira/browse/ROOT-9260)
bellenot added a commit that referenced this pull request Jul 25, 2023
Prevent potential infinite recursion and simplify some of the code. This fixes the Github issue #13233 and doesn't affect the already closed Jira issue [#9260](https://sft.its.cern.ch/jira/browse/ROOT-9260)
bellenot added a commit to bellenot/root that referenced this pull request Jul 25, 2023
Prevent potential infinite recursion and simplify some of the code. This fixes the Github issue root-project#13233 and doesn't affect the already closed Jira issue [root-project#9260](https://sft.its.cern.ch/jira/browse/ROOT-9260)
bellenot added a commit that referenced this pull request Jul 26, 2023
Prevent potential infinite recursion and simplify some of the code. This fixes the Github issue #13233 and doesn't affect the already closed Jira issue [#9260](https://sft.its.cern.ch/jira/browse/ROOT-9260)
maksgraczyk pushed a commit to maksgraczyk/root that referenced this pull request Jan 12, 2024
Prevent potential infinite recursion and simplify some of the code. This fixes the Github issue root-project#13233 and doesn't affect the already closed Jira issue [root-project#9260](https://sft.its.cern.ch/jira/browse/ROOT-9260)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants