Re: [make-initrd] [PATCH v6 05/22] bootchain-core, bootchain-waitdev: whitespace

[Leonid Krivoshein <klark.devel@gmail.com>]

26.10.2021 14:14, Alexey Gladkov пишет:
> On Tue, Oct 26, 2021 at 12:51:14PM +0300, Leonid Krivoshein wrote:
>> 26.10.2021 12:20, Alexey Gladkov пишет:
>>> On Sun, Oct 24, 2021 at 08:20:39PM +0300, Leonid Krivoshein wrote:
>>>> Signed-off-by: Leonid Krivoshein <klark.devel@gmail.com>
>>>> ---
>>>>    .../data/bin/bootchain-sh-functions           | 28 +++++++++++--------
>>>>    features/bootchain-core/data/sbin/chaind      |  6 ++--
>>>>    .../data/lib/bootchain/waitdev                |  1 +
>>>>    3 files changed, 21 insertions(+), 14 deletions(-)
>>>>
>>>> diff --git a/features/bootchain-core/data/bin/bootchain-sh-functions b/features/bootchain-core/data/bin/bootchain-sh-functions
>>>> index fa6201f..f337767 100644
>>>> --- a/features/bootchain-core/data/bin/bootchain-sh-functions
>>>> +++ b/features/bootchain-core/data/bin/bootchain-sh-functions
>>>> @@ -9,9 +9,11 @@ __bootchain_sh_functions=1
>>>>    handlerdir=/lib/bootchain
>>>>    [ "${ROOT-}" = pipeline ] && mntdir=/dev/pipeline || mntdir=/dev/bootchain
>>>> +
>>>>    check_parameter()
>>>>    {
>>>>    	local v
>>>> +
>>>>    	eval "v=\"\${$1-}\""
>>>>    	[ -n "$v" ] ||
>>>>    		fatal "Parameter '$1' required"
>>>> @@ -25,20 +27,22 @@ get_parameter()
>>>>    resolve_target()
>>>>    {
>>>>    	local target="$1"
>>>> +
>>>>    	case "${target%%/*}" in
>>>> -		'')
>>>> -			;;
>>>> -		pipe[0-9]|pipe[0-9][0-9]|pipe[0-9][0-9][0-9])
>>>> -			target="$mntdir/dst/step${target:4}"
>>>> -			;;
>>>> -		*)
>>>> -			if [ -z "${prevdir-}" ]; then
>>>> -				message "no previous stop to use"
>>>> -				return
>>>> -			fi
>>>> -			target="$prevdir/${target#/}"
>>>> -			;;
>>>> +	'')
>>>> +		;;
>>>> +	pipe[0-9]|pipe[0-9][0-9]|pipe[0-9][0-9][0-9])
>>>> +		target="$mntdir/dst/step${target:4}"
>>>> +		;;
>>>> +	*)
>>>> +		if [ -z "${prevdir-}" ]; then
>>>> +			message "no previous stop to use"
>>>> +			return
>>>> +		fi
>>>> +		target="$prevdir/${target#/}"
>>>> +		;;
>>> Это неправильное форматирование. Остальной код отформатирован иначе:
>>>
>>> git grep -A1 'case "' features/
>> Понял, это я прощёлкал. Переделаю.
>>
>>
>>>>    	esac
>>>> +
>>>>    	printf '%s' "$target"
>>>>    }
>>>> diff --git a/features/bootchain-core/data/sbin/chaind b/features/bootchain-core/data/sbin/chaind
>>>> index d401101..08e5e58 100755
>>>> --- a/features/bootchain-core/data/sbin/chaind
>>>> +++ b/features/bootchain-core/data/sbin/chaind
>>>> @@ -7,6 +7,7 @@ message_time=1
>>>>    pidfile="/var/run/$PROG.pid"
>>>>    logfile="/var/log/$PROG.log"
>>>> +
>>>>    exit_handler()
>>>>    {
>>>>    	local rc="$?"
>>>> @@ -15,6 +16,7 @@ exit_handler()
>>>>    	exit $rc
>>>>    }
>>>> +
>>>>    [ ! -f "$pidfile" ] ||
>>>>    	fatal "already running"
>>>> @@ -60,8 +62,8 @@ while [ -n "$chainsteps" ]; do
>>>>    			export name callnum datadir destdir prevdir
>>>> -			rc=0
>>>> -			"$exe" || rc=$?
>>>> +				rc=0
>>>> +				"$exe" || rc=$?
>>> А это вообще ошибка в форматировании.
>> Переделаю. Да, здесь whitespace слишком преждевременно (#05->16) для
>> уменьшения diff. Вопрос в том, как лучше переделать? Сделать отдельный
>> коммит whitespace перед коммитом, где тут ожидается изменение (#16), или
>> чёрт с ним, со сдвигом? Просто удалятся и добавятся две строки, по diff и
>> так будет понятно, что это старый код.
> Оставь пока так. Просто в следующей версии выкини этот коммит.

OK


>>>>    			if [ "$rc" != 0 ]; then
>>>>    				[ "$rc" != 2 ] ||
>>>> diff --git a/features/bootchain-waitdev/data/lib/bootchain/waitdev b/features/bootchain-waitdev/data/lib/bootchain/waitdev
>>>> index 74af6ea..bc81673 100755
>>>> --- a/features/bootchain-waitdev/data/lib/bootchain/waitdev
>>>> +++ b/features/bootchain-waitdev/data/lib/bootchain/waitdev
>>>> @@ -20,5 +20,6 @@ while [ -n "$devspec" ]; do
>>>>    			break
>>>>    		fi
>>>>    	fi
>>>> +
>>>>    	sleep 1
>>>>    done
>>>> -- 
>>>> 2.24.1
>>>>
>>>> _______________________________________________
>>>> Make-initrd mailing list
>>>> Make-initrd@lists.altlinux.org
>>>> https://lists.altlinux.org/mailman/listinfo/make-initrd
>>>>
>> -- 
>> Best regards,
>> Leonid Krivoshein.
>>
>> _______________________________________________
>> Make-initrd mailing list
>> Make-initrd@lists.altlinux.org
>> https://lists.altlinux.org/mailman/listinfo/make-initrd

-- 
Best regards,
Leonid Krivoshein.