Skip to content

Restore ms_gsl < 4 support#8343

Merged
ktf merged 1 commit intoAliceO2Group:devfrom
davidrohr:dev_pull_request6
Mar 14, 2022
Merged

Restore ms_gsl < 4 support#8343
ktf merged 1 commit intoAliceO2Group:devfrom
davidrohr:dev_pull_request6

Conversation

@davidrohr
Copy link
Copy Markdown
Collaborator

Revert "Revert "Workaround: temporarily support multiple ms_gsl versi…

This reverts commit ce9e904.

@davidrohr davidrohr requested a review from a team as a code owner March 14, 2022 14:33
@ktf ktf merged commit 4ee1755 into AliceO2Group:dev Mar 14, 2022
@ktf
Copy link
Copy Markdown
Member

ktf commented Mar 14, 2022

Currently broken on some arch...

@davidrohr
Copy link
Copy Markdown
Collaborator Author

Actually, I don't see how it can be broken, the switch-to-gsl-4 PR was green when it was merged.

@ktf
Copy link
Copy Markdown
Member

ktf commented Mar 14, 2022

Not sure either. I suspect there might be some interference with system includes.

@ktf
Copy link
Copy Markdown
Member

ktf commented Mar 14, 2022

(or maybe simply people did not recompile enough...)

@AllaMaevskaya
Copy link
Copy Markdown
Contributor

AllaMaevskaya commented Mar 14, 2022 via email

@chiarazampolli
Copy link
Copy Markdown
Collaborator

I can confirm that updating alidist (before this commit, of course) fixed the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants