Re: [devel-distro] [PATCH] lib/boot.mk: do not define BOOTLOADER type for e2k

[Gleb Fotengauer-Malinovskiy <glebfm@altlinux.org>]

[-- Attachment #1: Type: text/plain, Size: 1987 bytes --]

On Sat, Aug 05, 2023 at 09:48:33AM +0700, Антон Мидюков wrote:
> 05.08.2023 03:42, Gleb Fotengauer-Malinovskiy пишет:
> > It is unnecessary to define the BOOTLOADER type for e2k in this context.
> > The reason is that images for e2k use the mkimage isodata pack method,
> > which is unaffected by any value of the BOOTLOADER variable (BOOT_TYPE
> > in the mkimage).  Therefore, defining the BOOTLOADER type does not make
> > any sense.
> > 
> 
> Этот коммит ломает сборку на e2k, так как в sub.in/stage1/Makefile:
> ifeq (,$(BOOTLOADER))
> $(error BOOTLOADER not defined)
> endif

Ну тут получается, что либо туда, либо сюда.  Либо нужно сделать чтобы
BOOTLOADER=e2kboot и IMAGE_PACKTYPE=boot имели смысл вместе, либо не
считать, что отсутствие BOOTLOADER на что-то влияет.

> Но скорее всего эта проверка стала излишней после упразднения в этом Makefile
> copy-$(BOOTLOADER)
> 
> > Fixes: e8fcdaf8 ("lib/boot.mk: Set IMAGE_PACKTYPE to isodata for e2k")
> > ---
> >  lib/boot.mk | 1 -
> >  1 file changed, 1 deletion(-)
> > 
> > diff --git a/lib/boot.mk b/lib/boot.mk
> > index 2c58f378..e8940dc7 100644
> > --- a/lib/boot.mk
> > +++ b/lib/boot.mk
> > @@ -13,7 +13,6 @@ ifeq (,$(filter-out aarch64 riscv64 loongarch64,$(ARCH)))
> >  	@$(call try,BOOTLOADER,efiboot)
> >  endif
> >  ifeq (,$(filter-out e2k%,$(ARCH)))
> > -	@$(call try,BOOTLOADER,e2kboot)
> >  	@$(call set,IMAGE_PACKTYPE,isodata)
> >  endif
> >  ifeq (,$(filter-out ppc64le,$(ARCH)))
> 
> -- 
> С уважением, Антон Мидюков <antohami@altlinux.org>
> 
> _______________________________________________
> devel-distro mailing list
> devel-distro@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-distro

-- 
glebfm

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]