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

"NINJA" easyconfig will name collide with "Ninja" #13412

Closed
Micket opened this issue Jul 11, 2021 · 4 comments
Closed

"NINJA" easyconfig will name collide with "Ninja" #13412

Micket opened this issue Jul 11, 2021 · 4 comments
Milestone

Comments

@Micket
Copy link
Contributor

Micket commented Jul 11, 2021

I happened to see that there was a easyconfig named "NINJA" now, and, since some naming schemes, as well as our environment variables, ignore case, this will conflict with "Ninja".

https://github.com/easybuilders/easybuild-easyconfigs/blob/e679b333f74853d128e8fe3bfbb8725909e94d9e/easybuild/easyconfigs/n/NINJA/NINJA-0.97-cluster_only-GCC-10.2.0.eb

this package should be renamed ASAP

@Micket Micket added this to the next release (4.4.2?) milestone Jul 11, 2021
@jfgrimm
Copy link
Member

jfgrimm commented Jul 13, 2021

just make sure never to load Ninja and NINJA, and switch to a case-sensitive MNS ;)

in all seriousness, I'd suggest something like TWL-NINJA?

@jfgrimm
Copy link
Member

jfgrimm commented Jul 13, 2021

#13126 and its easyblock #2470 will need updating to match

@Micket
Copy link
Contributor Author

Micket commented Jul 30, 2021

I have no real suggestions for the name. Their about section is blank, and the acronym means nothing to me.

@boegel
Copy link
Member

boegel commented Aug 7, 2021

fixed in #13529, so closing this...

@boegel boegel closed this as completed Aug 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants