readme: remove incorrect "not"#12310
Conversation
|
GNU testsuite comparison: |
Merging this PR will degrade performance by 12.08%
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
|
Sorry. I typo. |
| Bug reporters should use binary from latest commit. | ||
|
|
||
| Minimal compatible glibc version is same with ubuntu-latest runner. Use `coreutils-*-musl` if `coreutils-*-musl` is not compatible with your system. | ||
| Minimal compatible glibc version is same with ubuntu-latest runner. Use `coreutils-*-musl` if `coreutils-*-musl` is compatible with your system. |
There was a problem hiding this comment.
| Minimal compatible glibc version is same with ubuntu-latest runner. Use `coreutils-*-musl` if `coreutils-*-musl` is compatible with your system. | |
| Minimal compatible glibc version is same with ubuntu-latest runner. Use `coreutils-*-musl` if `coreutils-*-gnu` is not compatible with your system. |
My understanding is that this was the likely intent, based on the conversation at #12005.
There was a problem hiding this comment.
I think the README should document the exact minimum supported glibc version for coreutils compatibility.
With a recent commit the following sentence was added to the readme:
The sentence doesn't make sense. This PR removes the "not". However, it's also possible that the "not" is correct and the second
coreutils-*-muslis incorrect.