Re: [gentoo-dev] [PATCH] mount-boot.eclass: Check if /boot is sane, but don't try to mount it.

[Michał Górny]

On Fri, 2019-12-06 at 15:45 +0100, Ulrich Mueller wrote:
> > > +	local fstabstate=$(awk '!/^[[:blank:]]*#|^\/dev\/BOOT/ && $2 == "/boot" \
> > > +		{print $2}' /etc/fstab)
> > The 'print' here is used as a boolean... why not use a boolean output
> > instead?
>
> Changed, it's printing a 1 now.

Well, I actually meant using exit status ;-).  Would be also have the
benefit of terminating early.

>
> > > +	local procstate=$(awk '$2 == "/boot" \
> > > +		{print gensub(/^(.*,)?(ro|rw)(,.*)?$/, "\\2", 1, $4)}' /proc/mounts)
> > Shouldn't this use /proc/self/mounts?
>
> I haven't actually changed this, it was /proc/mounts before (which is a
> symlink to self/mounts). Maybe we should keep it as-is for backwards
> compatibility? (But TBH, I have no idea with which kernel version
> /proc/self/mounts appeared.)

Ok.

>
> > > +	mount-boot_check_status
> > > +	if ! ( shopt -s failglob; : "${ROOT}"/boot/.keep* ) 2>/dev/null ; then
> > EROOT?
>
> Hm, we mount /boot, not ${EROOT}/boot. I've added an additional test for
> [[ ${EROOT:-/} == / ]] before touching the file.
>
> Patch v2 will follow.
>

Thanks.

-- 
Best regards,
Michał Górny