From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on sa.local.altlinux.org X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00 autolearn=unavailable autolearn_force=no version=3.4.1 Date: Mon, 10 Feb 2020 10:51:47 +0100 From: Alexey Gladkov To: =?utf-8?B?0JDQvdGC0L7QvSDQnNC40LTRjtC60L7Qsg==?= Message-ID: <20200210095146.gswqfvspiy5xynmf@comp-core-i7-2640m-0182e6> Mail-Followup-To: =?utf-8?B?0JDQvdGC0L7QvSDQnNC40LTRjtC60L7Qsg==?= , devel-distro@lists.altlinux.org, Michael Shigorin References: <3e580857-e644-bc79-f860-2cabaa3badac@ya.ru> <214f9fb1-02d7-b821-9faa-6cda4243415d@ya.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <214f9fb1-02d7-b821-9faa-6cda4243415d@ya.ru> Cc: devel-distro@lists.altlinux.org Subject: Re: [devel-distro] =?utf-8?b?Z3J1Yi1lZmkg0LIg0LjQvdGB0YLQsNC70Y/RgtC+?= =?utf-8?b?0YDQtQ==?= X-BeenThere: devel-distro@lists.altlinux.org X-Mailman-Version: 2.1.12 Precedence: list Reply-To: Distributions development List-Id: Distributions development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 10 Feb 2020 09:51:52 -0000 Archived-At: List-Archive: On Mon, Feb 10, 2020 at 09:58:54AM +0700, Антон Мидюков wrote: > 12.01.2020 22:18, Антон Мидюков пишет: > > 09.01.2020 21:39, Антон Мидюков пишет: > > > 09.01.2020 2:29, Антон Мидюков пишет: > > > > > > > > [...] > > > > > > > > 4. В mki-copy-efiboot для grub-efi ядро копировать в boot, если > > > > его ещё там нет. Аналогично для других EFI_BOOTLOADER проверять, > > > > есть ли уже ядро в EFI/BOOT, и если есть не копировать. > > > > > > > Копировать ядро в boot, видимо, плохая идея. В qemu грузится, на > > > железе же ядро не находит. > > > > [...] > > > > > В связи с этим переделал задание 243937 (mkimage), всё хорошенько > > перепроверил и исправил очепятки. > > > > Количество правок сократил. Убрал упоминание несуществующего grubx86boot. > > > > Без патча > > 0017-sub.in-stage1-call-copy-kernel-instead-of-copy-BOOTL.patch для > > mkimage-profiles будет работать по-старому. > > > > Прошу аппрув для задания 243937 (mkimage). > Повторяю просьбу. Если что-то сделал неправильно, не молчите, пожалуйста. Письма про EPERM я игнорирую. Это не pull-request. Они значат лишь то, что кто-то собирает ваш пакет. В будущем пожалуйста присылайте нормальный запрос с описанием того, что вы делаете. Теперь конкретно по таску 243937. 1f2ab51466760f2c614f33d916d8b7fefa90dd95 ("mki-copy-kernel: initial target for copying kernel") Описание коммита не поясняет зачем он нужен вообще. Не вызывать mki-copy-$BOOTLOADER дважды ? Не вызывайте, если не нужно. Какую проблему вы решаете этим коммитом ? > [ x$EFI_DESTINATION != 'x' ] Пожалуйста, не пишите так. В mki-copy-kernel вы анализируете ${BOOTLOADER:-}. Такой переменной нет в mkimage. Если это новый параметр, то его нужно требовать в скрипте и описать. Есть параметр BOOT_TYPE. Вы его имели в виду ? Вы проверяете BOOTLOADER только на значения isolinux, syslinux, pxelinux, ieee1275boot. Типов загрузчиков в mkimage больше. Если они не должны обрабатываться, то задокументируйте это. Также вы вынесли код копирования ядра в отдельный таргет продублировав код из соответствующих mki-copy-$BOOTLOADER. Это плохое решение. Это приведёт к рассинхронизации кода в будущем. Также следующий кто будет добавлять новый вариант загрузчика скорее всего пропустит этот скрипт. Я не против вынести копирование ядра в отдельный таргет, если это необходимо. Но мне не нравится, как это реализовано. Если вы хотите это сделать, то убирайте соответствующий код из mki-copy-*. Например, пусть они сами вызывают новую утилиту с параметром `mki-copy-kernel $DESTDIR`. e0501c5fcf9bde41d1cfc88c0941b7282e23bcdb ("mki-copy-grubaa64boot, mki-pack-efionly-isoboot: replaced grub to EFI/BOOT") Опять же описание коммита не совсем понятно. Вы создаёте EFI/BOOT и копируете части grub туда потому что директории вне EFI могут быть не доступны. Когда не доступны ? Вы наверно имели в виду, что для grub во время загрузки может быть доступен только EFI раздел ? (просто угадываю сейчас). > +[ -f "$chroot/.image/$imgdir/EFI/BOOT/grub.cfg" ] || > + message "Warning: /.image/$imgdir/EFI/BOOT/grub.cfg: not found." Зачем этот варнинг, если внизу есть код, который создаёт этот конфиг ? > +mkdir $verbose -p EFI/BOOT Вот тут вы можете написать EFI/BOOT/fonts чтобы не проверять и создавать эту директорию ниже. -- Rgrds, legion