Skip to content

mktemp: remove duplicate code from uumain#12490

Draft
cakebaker wants to merge 1 commit into
uutils:mainfrom
cakebaker:mktemp_use_mktemp_fn_in_uumain
Draft

mktemp: remove duplicate code from uumain#12490
cakebaker wants to merge 1 commit into
uutils:mainfrom
cakebaker:mktemp_use_mktemp_fn_in_uumain

Conversation

@cakebaker
Copy link
Copy Markdown
Contributor

This PR removes some duplicate code from uumain and calls mktemp() instead, a function we provide for nushell and wasn't used in our code base so far.

@sylvestre
Copy link
Copy Markdown
Contributor

ouch, thanks :)

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

GNU test failed: tests/mktemp/mktemp. tests/mktemp/mktemp is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/cut/bounded-memory (passes in this run but fails in the 'main' branch)
Note: The gnu test tests/rm/many-dir-entries-vs-OOM is now being skipped but was previously passing.
Congrats! The gnu test tests/printf/printf-surprise is now passing!
Congrats! The gnu test tests/tail/tail-n0f is now passing!

@cakebaker cakebaker marked this pull request as draft May 27, 2026 12:11
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