Skip to content

minor fix for error message install to /proc/self/mem#12718

Open
zhw2101024 wants to merge 1 commit into
uutils:mainfrom
zhw2101024:error_message_using_proc_self_mem_as_dst
Open

minor fix for error message install to /proc/self/mem#12718
zhw2101024 wants to merge 1 commit into
uutils:mainfrom
zhw2101024:error_message_using_proc_self_mem_as_dst

Conversation

@zhw2101024

@zhw2101024 zhw2101024 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Fixes #12695.
Not added a test since no behavior changes in this PR.

@sylvestre

Copy link
Copy Markdown
Contributor

i still would like to get a test :)
thanks

@zhw2101024 zhw2101024 force-pushed the error_message_using_proc_self_mem_as_dst branch from 04caff0 to 9330514 Compare June 8, 2026 14:56
@zhw2101024

Copy link
Copy Markdown
Contributor Author

i still would like to get a test :) thanks

The test has been commited.

Comment thread tests/by-util/test_install.rs
@zhw2101024 zhw2101024 force-pushed the error_message_using_proc_self_mem_as_dst branch from 9330514 to 70a439f Compare June 8, 2026 16:30
@github-actions

github-actions Bot commented Jun 8, 2026

Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/pr/bounded-memory (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/retry (fails in this run but passes in the 'main' branch)
Skip an intermittent issue tests/tail/tail-n0f (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Note: The gnu test tests/cp/link-heap is now being skipped but was previously passing.
Note: The gnu test tests/printf/printf-surprise is now being skipped but was previously passing.
Congrats! The gnu test tests/expand/bounded-memory is now passing!

@zhw2101024 zhw2101024 force-pushed the error_message_using_proc_self_mem_as_dst branch from 70a439f to 74ced61 Compare June 9, 2026 02:28
@zhw2101024

Copy link
Copy Markdown
Contributor Author

@oech3 all checks passed.

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.

minorbug(install): using /proc/self/mem as a dst leads to show raw rust errors not standard human readable errors

3 participants