Skip to content

uutils-coreutils@0.10.0: Add shim for timeout - #8360

Merged
z-Fng merged 1 commit into
ScoopInstaller:masterfrom
God-damnit-all:patch-5
Aug 6, 2026
Merged

uutils-coreutils@0.10.0: Add shim for timeout#8360
z-Fng merged 1 commit into
ScoopInstaller:masterfrom
God-damnit-all:patch-5

Conversation

@God-damnit-all

Copy link
Copy Markdown
Contributor

uutils coreutils 0.10.0 now has a Windows implementation for timeout, so it has been included.

  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

As an aside to anyone who reviews this: I'm thinking of doing a PR that changes the uutils.exe shim here to uutils-coreutils.exe (it made sense back when coreutils was pretty much all uutils put out but now they have several things, it's like having a microsoft.exe shim).

Similarly, I want to make the uutils shim in the microsoft implementation of coreutils named microsoft-coreutils.exe instead of uutils.exe (I think this was the result of a copy-and-paste).

Let me know if you think this is a good idea.

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

Summary by CodeRabbit

  • New Features
    • Added the timeout command to the uutils coreutils package for 64-bit, 32-bit, and ARM64 systems.

Walkthrough

The Scoop manifest adds the timeout executable for 64-bit packages and command aliases for 32-bit and ARM64 packages.

Changes

uutils timeout support

Layer / File(s) Summary
Add timeout binary mappings
bucket/uutils-coreutils.json
The manifest adds timeout.exe to the 64-bit binary list and adds timeout aliases for the 32-bit and ARM64 bundled coreutils.exe binaries.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the manifest version and the addition of the timeout shim.
Description check ✅ Passed The description explains the change and confirms both required checklist items.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@God-damnit-all

God-damnit-all commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

/verify

(I often forget to do this, sorry about that.)

@github-actions

github-actions Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

uutils-coreutils

  • Lint
  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

Check the full log for details.

@z-Fng z-Fng changed the title uutils-coreutils@0.10.0: add timeout.exe shim uutils-coreutils@0.10.0: Add shim for timeout Aug 6, 2026
z-Fng added a commit that referenced this pull request Aug 6, 2026
* Fix a copy-paste error. The shim name should be `microsoft-coreutils`
  instead of `uutils`.
  Relates to #8360 (comment).
@z-Fng
z-Fng merged commit 7d608a3 into ScoopInstaller:master Aug 6, 2026
3 checks passed
@z-Fng

z-Fng commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

As an aside to anyone who reviews this: I'm thinking of doing a PR that changes the uutils.exe shim here to uutils-coreutils.exe (it made sense back when coreutils was pretty much all uutils put out but now they have several things, it's like having a microsoft.exe shim).

I'd suggest keeping this as-is. The shim was added intentionally (a04240c). The uutils shim was kept because it was previously named uutils. And it's unclear what this change might break.

Similarly, I want to make the uutils shim in the microsoft implementation of coreutils named microsoft-coreutils.exe instead of uutils.exe (I think this was the result of a copy-and-paste).

Good catch on microsoft-coreutils - that was indeed a copy-paste error, which I've addressed in 6892daa.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants