Skip to content

[20.10 backport] dockerd-rootless.sh: add typo guard#42070

Merged
tiborvass merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_rootless_typo_guard
Feb 25, 2021
Merged

[20.10 backport] dockerd-rootless.sh: add typo guard#42070
tiborvass merged 1 commit into
moby:20.10from
thaJeztah:20.10_backport_rootless_typo_guard

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

backport of #41960

- What I did

dockerd-rootless.sh install is a common typo of dockerd-rootless-setuptool.sh install.

Now dockerd-rootless.sh install shows human-readable error.

- How I did it

Added a human-readable error

- How to verify it

$ dockerd-rootless.sh  install -f
+ echo Did you mean 'dockerd-rootless-setuptool.sh install -f' ?
Did you mean 'dockerd-rootless-setuptool.sh install -f' ?
+ exit 1

- A picture of a cute animal (not mandatory but encouraged)
🐧

`dockerd-rootless.sh install` is a common typo of `dockerd-rootless-setuptool.sh install`.

Now `dockerd-rootless.sh install` shows human-readable error.

Signed-off-by: Akihiro Suda <[email protected]>
(cherry picked from commit 8dc6c10)
Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah added this to the 20.10.4 milestone Feb 24, 2021
@thaJeztah

Copy link
Copy Markdown
Member Author

@tianon @AkihiroSuda ptal

@tiborvass
tiborvass merged commit 035cb27 into moby:20.10 Feb 25, 2021
@thaJeztah
thaJeztah deleted the 20.10_backport_rootless_typo_guard branch February 25, 2021 08:38
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.

3 participants