Date: Tue, 2 Mar 2021 13:44:45 -0500 From: Nathan Whitehorn <nwhitehorn@freebsd.org> To: rgrimes@freebsd.org Cc: src-committers@freebsd.org, dev-commits-src-all@freebsd.org, dev-commits-src-main@freebsd.org Subject: Re: git: 2c26d77d989a - main - Remove /boot/efi from mtree, missed in 0b7472b3d8d2. Message-ID: <a20a180b-4264-61f9-ae44-bec18290a22b@freebsd.org> In-Reply-To: <202103021826.122IQfMN047888@gndrsh.dnsmgr.net> References: <202103021826.122IQfMN047888@gndrsh.dnsmgr.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On 3/2/21 1:26 PM, Rodney W. Grimes wrote: >> The branch main has been updated by nwhitehorn: >> >> URL: https://cgit.FreeBSD.org/src/commit/?id=2c26d77d989abe48c662eeb6f52f7e4c9b81680c >> >> commit 2c26d77d989abe48c662eeb6f52f7e4c9b81680c >> Author: Nathan Whitehorn <nwhitehorn@FreeBSD.org> >> AuthorDate: 2021-03-02 16:47:00 +0000 >> Commit: Nathan Whitehorn <nwhitehorn@FreeBSD.org> >> CommitDate: 2021-03-02 16:49:41 +0000 >> >> Remove /boot/efi from mtree, missed in 0b7472b3d8d2. >> >> This had prevented the bootconfig step from determining if an ESP exists, >> resulting in its unconditional setup. On BIOS-booted amd64, this wasn't >> harmful, just unnecessary, but it resulted in failed installations on >> non-EFI-supporting platforms like powerpc64. >> >> MFC after: 3 days >> --- >> etc/mtree/BSD.root.dist | 2 -- >> release/tools/arm.subr | 1 + >> 2 files changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/etc/mtree/BSD.root.dist b/etc/mtree/BSD.root.dist >> index f734f7891429..1dc9e179b0fc 100644 >> --- a/etc/mtree/BSD.root.dist >> +++ b/etc/mtree/BSD.root.dist >> @@ -18,8 +18,6 @@ >> rockchip tags=package=runtime >> .. >> .. >> - efi >> - .. >> firmware >> .. >> loader.conf.d tags=package=bootloader >> diff --git a/release/tools/arm.subr b/release/tools/arm.subr >> index 343d9f3a7034..2f91490c0859 100644 >> --- a/release/tools/arm.subr >> +++ b/release/tools/arm.subr >> @@ -184,6 +184,7 @@ arm_install_base() { >> TARGET_ARCH=${EMBEDDED_TARGET_ARCH} \ >> DESTDIR=${DESTDIR} KERNCONF=${KERNEL} \ >> ${CONF_FILES} installworld installkernel distribution >> + chroot ${CHROOTDIR} mkdir -p ${DESTDIR}/boot/efi > This fails to apply the proper owner/group and mode values > using what ever defaults are in place of the process running > the build. This script is riddled with such issues (see the line immediately below and copied here, for instance). Presumably, as run in the official build environment to generate SD-card images, this works -- if it didn't, large sections of the rest of the script would not either. It did not seem reasonable to fix it for this one case only or to retool the entire script at present, though some of this code should probably be looked at sooner rather than later -- I started with vmimage.subr already, but the rest will take a while. -Nathan >> chroot ${CHROOTDIR} mkdir -p ${DESTDIR}/boot/msdos >> >> arm_create_user >>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?a20a180b-4264-61f9-ae44-bec18290a22b>