-
-
Notifications
You must be signed in to change notification settings - Fork 50
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
Enable CRB repo on alma9, PowerTools on alma8 #299
Conversation
otherwise, the second `:` (and everything that follows) wrongly gets interpreted as payload for the first append command
This reverts commit 231bb36.
I'm fine for this PR to get squashed, if someone prefers that. I didn't squash the PR myself to show why I arrived at the solution here. Note that the following sed-command Line 7 in 7023391
is pretty gnarly to understand, and in particular, I couldn't find a way to chain another append-command afterwards. I added some comments to explain a bit what's going on. Whereas the regex for cos7 unconditionally disables something (so adding |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This LGTM but I'd prefer if someone else can also take a look before merging because I'm not that in touch with this part of the stack.
Ping @conda-forge/core, would be nice if someone could review. :) |
With the switch to alma8 & alma9, some of our CDTs are now split across separate repos. In particular, we need
PowerTools
on alma8, resp.CRB
on alma9 (which is essentially the same thing but got renamed). See for example conda-forge/cdt-builds@e118347As an example, the following failed:
That package is available under
CRB
on alma9, so enable this repo.