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

From: "Антон Мидюков" <midyukov-anton@ya.ru>
To: devel-distro@lists.altlinux.org
Subject: Re: [devel-distro] [PATCH] lib/boot.mk: do not define BOOTLOADER type for e2k
Date: Mon, 7 Aug 2023 11:52:05 +0700
Message-ID: <ff9b2d73-1de6-bed2-7110-c7e73ab394ab@ya.ru> (raw)
In-Reply-To: <ZM4bFTues6INXt8w@glebfm.altlinux.org>

05.08.2023 16:49, Gleb Fotengauer-Malinovskiy пишет:
> 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 на что-то влияет.
> 

Сделал, чтобы проверка не отрабатывала, если IMAGE_PACKTYPE=isodata.
И затем принял патч.
В e2k просто конфиг. Смысл его генерировать в mkimage, если в mkimage-profiles
генерируется другой конфиг, я не вижу.

>> Но скорее всего эта проверка стала излишней после упразднения в этом 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
> 
> 
> _______________________________________________
> devel-distro mailing list
> devel-distro@lists.altlinux.org
> https://lists.altlinux.org/mailman/listinfo/devel-distro

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>