Mail archive
alpine-aports

Re: [alpine-aports] [PATCH] main/openrc: fix hostname init for btrfs subvolumes

From: Natanael Copa <ncopa_at_alpinelinux.org>
Date: Sun, 15 Nov 2015 13:39:48 +0100

On Sun, 15 Nov 2015 01:43:09 +0000
Stuart Cardall <developer_at_it-offshore.co.uk> wrote:

> Busybox hostname doesn't seem to work on btrfs subvolumes (the hostname is set as "?")
>
> This shell oneliner sets the hostname as the first non blank line in /etc/hostname.
> If /etc/hostname is empty or does not exist the hostname is set to localhost.

Does it set hostname if its nonempty?

Does it set hostname if its empty and filesystem is something else than
btrfs?

I wonder why this is specific for btrfs.

> ---
> main/openrc/hostname.initd | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/main/openrc/hostname.initd b/main/openrc/hostname.initd
> index fdc7f0b..49098f4 100644
> --- a/main/openrc/hostname.initd
> +++ b/main/openrc/hostname.initd
> _at_@ -7,11 +7,7 @@ depend() {
> }
>
> start() {
> - if [ -f /etc/hostname ] ; then
> - opts="-F /etc/hostname"
> - else
> - opts="localhost"
> - fi
> + opts="$(grep -v '^$' /etc/hostname)" 2>/dev/null || opts="localhost"
> ebegin "Setting hostname"
> hostname $opts
> eend $?

Why not:

if [ -s /etc/hostname ]; then
    opts="-F /etc/hostname"
else
    opts="localhost"
fi

-nc


---
Unsubscribe:  alpine-aports+unsubscribe_at_lists.alpinelinux.org
Help:         alpine-aports+help_at_lists.alpinelinux.org
---
Received on Sun Nov 15 2015 - 13:39:48 GMT