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

update GNU coreutils version in GnuTests workflow #2919

Merged
merged 1 commit into from
Jan 27, 2022

Conversation

jtracey
Copy link
Contributor

@jtracey jtracey commented Jan 26, 2022

Updates it to the next and latest release. This is necessary to pull the fixed invalid-j test that will, once all the outstanding join PRs get merged, allow join to pass all its GNU tests (#2634). Seems to make some other tests fail, so those might require more investigation, but presumably aren't because of bugs introduced in the GNU repo.

@jtracey jtracey mentioned this pull request Jan 26, 2022
10 tasks
@sylvestre
Copy link
Contributor

Ouch:

Warning: Congrats! The gnu test tests/ls/removed-directory is now passing!
Error: GNU test failed: tests/cp/attr-existing. tests/cp/attr-existing is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/ls/classify. tests/ls/classify is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/ls/selinux-segfault. tests/ls/selinux-segfault is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/ls/sort-width-option. tests/ls/sort-width-option is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/ls/zero-option. tests/ls/zero-option is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/misc/cksum. tests/misc/cksum is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/misc/cksum-a. tests/misc/cksum-a is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/misc/cksum-c. tests/misc/cksum-c is passing on 'main'. Maybe you have to rebase?
Error: GNU test failed: tests/misc/sm3sum. tests/misc/sm3sum is passing on 'main'. Maybe you have to rebase?

did you look at what is regressing?

@jtracey
Copy link
Contributor Author

jtracey commented Jan 26, 2022

I had not, but did so now:

So basically, no singular issue, just various updates to the tests. Some of these seem like new functionality that will have to be added, but at least the first one seems like an actual bug being caught.

@sylvestre
Copy link
Contributor

Sounds good, could you please create issues to keep track of the regressions?
thanks

@jtracey
Copy link
Contributor Author

jtracey commented Jan 27, 2022

I've updated my above comment with links to the respective issues.

@sylvestre
Copy link
Contributor

amazing, thanks :)

@sylvestre
Copy link
Contributor

let's regress then :)

@sylvestre sylvestre merged commit 53f2aab into uutils:main Jan 27, 2022
@sylvestre
Copy link
Contributor

sylvestre commented Jan 28, 2022

I documented the change here:
uutils/coreutils-tracking@a10f9b0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants