Re: git: d72c74a6c52c - main - loader: Add halt as an alias to poweroff machine

On Fri, 4 Jul 2025, Tom Jones wrote:

> The branch main has been updated by thj:
>
> URL: https://cgit.FreeBSD.org/src/commit/?id=d72c74a6c52cd382c5aaf8f28b5f94e102140a6d
>
> commit d72c74a6c52cd382c5aaf8f28b5f94e102140a6d
> Author:     Tom Jones <thj@FreeBSD.org>
> AuthorDate: 2025-07-04 06:37:19 +0000
> Commit:     Tom Jones <thj@FreeBSD.org>
> CommitDate: 2025-07-04 06:37:31 +0000
>
>    loader: Add halt as an alias to poweroff machine
>
>    Add halt as an alias to also poweroff a machine from the loader prompt,
>    this is consistent with ddb.

Would it make sense to add an alias 'reset' for 'reboot' as well?

I think ddb is the only one using reset, everything else uses reboot
(kind-of makes sense but still is confusing).


>    Reviewed by:    imp
>    Sponsored by:   The FreeBSD Foundation
>    Differential Revision: https://reviews.freebsd.org/D51132
> ---
> stand/efi/loader/main.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/stand/efi/loader/main.c b/stand/efi/loader/main.c
> index 70cdfb2e9328..3ef418d20df3 100644
> --- a/stand/efi/loader/main.c
> +++ b/stand/efi/loader/main.c
> @@ -1547,6 +1547,7 @@ command_seed_entropy(int argc, char *argv[])
> }
>
> COMMAND_SET(poweroff, "poweroff", "power off the system", command_poweroff);
> +COMMAND_SET(halt, "halt", "power off the system", command_poweroff);
>
> static int
> command_poweroff(int argc __unused, char *argv[] __unused)
>

-- 
Bjoern A. Zeeb                                                     r15:7