Re: [devel] Q: postinst hook for firmware-*

[Konstantin Lepikhov <lakostis@altlinux.org>]

Hi Dmitry!

On 09/06/2017, at 04:37:35 PM you wrote:

> 
> содержимое ucode_handled дальше только проверяется на непустоту,
> можно записывать туда что-то более простое, например, ucode_handled=1
> 
ok

> > +	set +f
> 
> set +f дальше не используется и, видимо, не нужен.
ну как бы readlink может обломаться, если у нас нет /boot/vmlinuz. Я не
знаю, эта проверка есть в handle_kernel, я ее добавил по аналогии.

> 
> > +	f="$(readlink -e /boot/vmlinuz)"
> > +	if [ -n "$f" -a -n "$kernel_versions_handled" ]; then
> > +		VERSION=${f#$VMLINUZ_PREFIX-}
> > +		case "$kernel_version_handled" in
> > +			"* $VERSION*")
> > +			ucode_handled=
> > +			;;
> > +		esac
> > +	fi
> 
> Это лучше делать всего один раз после обработки цикла по файлам.
> Хотя почему это исключение сделано именно для /boot/vmlinuz, неочевидно.
я тут не очень понял логику, которая ниже по коду - если у нас выставлен
INITRD_AUTOUPDATE=all это значит что будет обновлятся initramfs для всех
ядер, или только тех, что попали в обновление? Почему там еще раз
проверяется является ли файл /boot/vmlinuz?

> 
> > +}
> > +
> >  while read f; do
> >  	case "$f" in
> >  		$VMLINUZ_PREFIX-[0-9].*-*-*)
> > @@ -107,10 +129,13 @@ while read f; do
> >  		$MODULES_PREFIX/*-*-*/*/*.ko*)
> >  			handle_module "$f"
> >  			;;
> > +		$UCODE_PREFIX/*-ucode/*)
> > +			handle_ucode "$f"
> 
> Получается, что здесь достаточно написать ucode_detected=1,
> 
> > +			;;
> >  	esac
> >  done
> 
> А сюда поместить сброс ucode_detected в случае, если он уже был обработан.
ok

-- 
WBR et al.