Date: Fri, 28 Jun 2013 16:41:38 +0200
From: Natanael Copa <ncopa@alpinelinux.org>
To: Dubiousjim <dubiousjim@gmail.com>
Cc: alpine-devel@lists.alpinelinux.org
Subject: Re: [alpine-devel] [PATCH 5/8] main/syslinux: add reboot entry
Message-ID: <20130628164138.56178a80@ncopa-desktop.alpinelinux.org>
In-Reply-To: <511cff4ffaf89ac9d5316663d6853a69b0d57368.1372318132.git.dubiousjim@gmail.com>
References: <cover.1372318132.git.dubiousjim@gmail.com>
	<511cff4ffaf89ac9d5316663d6853a69b0d57368.1372318132.git.dubiousjim@gmail.com>
Precedence: list
Mime-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit

On Thu, 27 Jun 2013 03:32:18 -0400
Dubiousjim <dubiousjim@gmail.com> wrote:

> We don't install /boot/reboot.c32, but if it's present (it can be copied from
> /usr/share/syslinux/reboot.c32), we add a menu entry for it---just like
> with memtest.
> 
> Add a comment to /etc/update-extlinux.conf stating this.
> 
> It'd already be possible to get a reboot entry using the
> /etc/update-extlinux.d/ folder, but this patch provides a more intelligent
> framework with nicer layout.
> 
> The syslinux sourceball also provides a shutdown module, but I couldn't get this to
> work and looking at the sources reveals it to be for machines with APM enabled.
> Not sure how many machines that applies to anymore.
> ---
>  main/syslinux/update-extlinux      | 7 +++++++
>  main/syslinux/update-extlinux.conf | 3 +++
>  2 files changed, 10 insertions(+)
> 
> diff --git a/main/syslinux/update-extlinux b/main/syslinux/update-extlinux
> index 0dc3601..1c5fc5c 100755
> --- a/main/syslinux/update-extlinux
> +++ b/main/syslinux/update-extlinux
> @@ -151,6 +151,13 @@ if [ -f "/boot/memtest" ]; then
>  	echo "" >> $conf.new
>  fi
>  
> +if [ -f "/boot/reboot.c32" ]; then
> +	everbose "Found reboot"
> +	echo "LABEL reboot" >> $conf.new
> +	echo "  MENU LABEL Reboot" >> $conf.new
> +        echo "  COM32 reboot.c32" >> $conf.new
> +	echo "" >> $conf.new
> +fi
>  
>  
>  if [ "$overwrite" != "1" ]; then
> diff --git a/main/syslinux/update-extlinux.conf b/main/syslinux/update-extlinux.conf
> index 59f9e60..7be434b 100644
> --- a/main/syslinux/update-extlinux.conf
> +++ b/main/syslinux/update-extlinux.conf
> @@ -42,3 +42,6 @@ default=grsec
>  # options to hand to xen hypervisor, useful ones are:
>  #    dom0_mem=256M (give domain-0 environment 256M ram)
>  xen_opts=dom0_mem=256M
> +
> +# neither of /boot/memtest or /boot/reboot.c32 are installed by default
> +# but entries for them will be auto-generated if they're present

ACK

-nc


---
Unsubscribe:  alpine-devel+unsubscribe@lists.alpinelinux.org
Help:         alpine-devel+help@lists.alpinelinux.org
---