rstudio / r-system-requirements

System requirements for R packages
MIT License
122 stars 21 forks source link

Update README for CentOS 8 EOL and Rocky Linux 8 #128

Closed glin closed 1 year ago

glin commented 1 year ago

For https://github.com/rstudio/r-system-requirements/issues/126 - remove CentOS 8, add Rocky Linux 8, and note that Rocky 8 is still centos8 for backward compatiiblity.

glin commented 1 year ago

@jeroen, since GitHub doesn't let me tag you as a reviewer here

jeroen commented 1 year ago

Ah okay thanks for clarifying that.

jeroen commented 1 year ago

Is there any reason to not just search-and-replace centos8 with rockylinux8 in the rules? It is a bit confusing still...

glin commented 1 year ago

I forgot to mention yesterday that Rocky Linux 8 isn't officially supported by Posit (only RHEL 8 since the CentOS 8 EOL), and so it's not intended to be officially supported here either. But we do still rely on it internally for Package Manager's RHEL 8 binaries and still maintain it. Keeping it as centos8 was both to simplify things in Package Manager and not break existing users making a similar transition. We've also kept Package Manager's RHEL 8 binary repo URL as /centos8/ to not break existing users.

So yes, it is confusing, but there would've been more work and potential confusion if we switched it to rockylinux8.