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

adjust testsuite to pass also with newer coreutils-9.6 #167

Merged
merged 3 commits into from
Jan 20, 2025
Merged

Conversation

mvidner
Copy link
Member

@mvidner mvidner commented Jan 20, 2025

Problem

cat used to quote its invocation path in its error message, then started using only its basename

https://build.opensuse.org/package/live_build_log/openSUSE:Factory:Staging:A/yast2-core/standard/x86_64

[ 109s] Running /home/abuild/rpmbuild/BUILD/yast2-core-5.0.2/agent-system/testsuite/ag_system.test/ag_system.exp ...
[ 109s] Running bash...
[ 112s] FAIL: Wrong stderr for bash

Solution

Call cat without path, to make the message the same in both versions

Testing

  • Detected by existing tests

Screenshots

No

cat used to quote its invocation path in its error message,
then started using only its basename
fix many warnings like:
[   23s] ./../../libycp/src/include/ycp/ycpless.h:35:30: warning: 'template<class _Arg1, class _Arg2, class _Result> struct std::binary_function' is deprecated [-Wdeprecated-declarations]
@mvidner mvidner merged commit 524bc44 into master Jan 20, 2025
2 checks passed
@mvidner mvidner deleted the coreutils-cat branch January 20, 2025 13:23
Copy link

✅ Autosubmission job #12868997237 successfully finished
✅ Created submit request #1239032

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