Re: [Xen-devel] [PATCH 2/3] xen: x86: drop the ".gz" suffix when installing

[Julien Grall <julien.grall@xxxxxxxxxx>]

On 07/17/2013 04:37 PM, Ian Campbell wrote:
> As Jan says it is pretty meaningless under /boot anyway. However I am slightly
> concerned about breaking bootloaders (or more specifically their help scripts
> which automatically generate config files). By inspection at leastr grub 2's
                                                                ^^
minor typo                                                     here

> update-grub script (as present in Debian Wheezy) seems to cope (it matches on
> xen* not xen*.gz)
> 
> Signed-off-by: Ian Campbell <ian.campbell@xxxxxxxxxx>

Acked-by: Julien Grall <julien.grall@xxxxxxxxxx>

> Cc: jbeulich@xxxxxxxx
> Cc: keir@xxxxxxx
> ---
>  xen/Makefile |   16 ++++++++--------
>  1 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/xen/Makefile b/xen/Makefile
> index 8bc8f3a..0a20662 100644
> --- a/xen/Makefile
> +++ b/xen/Makefile
> @@ -36,10 +36,10 @@ _build: $(TARGET)$(Z)
>  _install: Z=$(CONFIG_XEN_INSTALL_SUFFIX)
>  _install: $(TARGET)$(Z)
>       [ -d $(DESTDIR)/boot ] || $(INSTALL_DIR) $(DESTDIR)/boot
> -     $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir 
> $(TARGET))-$(XEN_FULLVERSION)$(Z)
> -     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) 
> $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z)
> -     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) 
> $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION)$(Z)
> -     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION)$(Z) 
> $(DESTDIR)/boot/$(notdir $(TARGET))$(Z)
> +     $(INSTALL_DATA) $(TARGET)$(Z) $(DESTDIR)/boot/$(notdir 
> $(TARGET))-$(XEN_FULLVERSION)
> +     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) 
> $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION).$(XEN_SUBVERSION)
> +     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) 
> $(DESTDIR)/boot/$(notdir $(TARGET))-$(XEN_VERSION)
> +     ln -f -s $(notdir $(TARGET))-$(XEN_FULLVERSION) 
> $(DESTDIR)/boot/$(notdir $(TARGET))
>       $(INSTALL_DATA) $(TARGET)-syms $(DESTDIR)/boot/$(notdir 
> $(TARGET))-syms-$(XEN_FULLVERSION)
>       if [ -r $(TARGET).efi -a -n '$(EFI_DIR)' ]; then \
>               [ -d $(DESTDIR)$(EFI_DIR) ] || $(INSTALL_DIR) 
> $(DESTDIR)$(EFI_DIR); \
> @@ -59,10 +59,10 @@ _uninstall: Z=$(CONFIG_XEN_INSTALL_SUFFIX)
>  _uninstall: D=$(DESTDIR)
>  _uninstall: T=$(notdir $(TARGET))
>  _uninstall:
> -     rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)$(Z)
> -     rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)$(Z)
> -     rm -f $(D)/boot/$(T)-$(XEN_VERSION)$(Z)
> -     rm -f $(D)/boot/$(T)$(Z)
> +     rm -f $(D)/boot/$(T)-$(XEN_FULLVERSION)
> +     rm -f $(D)/boot/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION)
> +     rm -f $(D)/boot/$(T)-$(XEN_VERSION)
> +     rm -f $(D)/boot/$(T)
>       rm -f $(D)/boot/$(T)-syms-$(XEN_FULLVERSION)
>       rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_FULLVERSION).efi
>       rm -f $(D)$(EFI_DIR)/$(T)-$(XEN_VERSION).$(XEN_SUBVERSION).efi
> 


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel