Skip to content

GNUmakefile: avoid bootstrap issue with mkdir#12521

Merged
cakebaker merged 1 commit into
uutils:mainfrom
oech3:make-system-mkdir
May 30, 2026
Merged

GNUmakefile: avoid bootstrap issue with mkdir#12521
cakebaker merged 1 commit into
uutils:mainfrom
oech3:make-system-mkdir

Conversation

@oech3
Copy link
Copy Markdown
Contributor

@oech3 oech3 commented May 30, 2026

Support the system mkdir is broken via INSTALL="cargo run ...".

@github-actions
Copy link
Copy Markdown

GNU testsuite comparison:

Skip an intermittent issue tests/date/date-locale-hour (fails in this run but passes in the 'main' branch)
Skipping an intermittent issue tests/misc/io-errors (passes in this run but fails in the 'main' branch)
Congrats! The gnu test tests/tail/tail-n0f is now passing!

@oech3 oech3 marked this pull request as ready for review May 30, 2026 07:14
@cakebaker
Copy link
Copy Markdown
Contributor

Hm, somehow I don't understand the reason for the changes of this PR. Why is using install "better" than using mkdir?

@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 30, 2026

This makes easier to install uutils on the system missing system coreutils by some reason. We only need to build uu_install at 1st.

@cakebaker cakebaker changed the title GNUmakefile: avoid bootsprap issue with mkdir GNUmakefile: avoid bootstrap issue with mkdir May 30, 2026
@cakebaker cakebaker merged commit b936d7c into uutils:main May 30, 2026
116 of 171 checks passed
@cakebaker
Copy link
Copy Markdown
Contributor

Thanks!

@oech3 oech3 deleted the make-system-mkdir branch May 30, 2026 08:12
@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 30, 2026

Can we just remove sort too?

@cakebaker
Copy link
Copy Markdown
Contributor

Probably yes.

@oech3
Copy link
Copy Markdown
Contributor Author

oech3 commented May 30, 2026

It might make's built-in...

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