update-kernel-1.14-alt1 -> sisyphus:

 Sat Feb 10 2024 Vitaly Chikunov <vt@altlinux> 1.14-alt1
 - Distinguish --dry-run and --download-only options (ALT#49340).

Как мне кажется, текущее поведение после изменений некорректно.

Теперь поведение зависит от порядка аргументов:
1) update-kernel -d -n -v
+ apt-get install ... --dry-run kernel-image-un-def...

2) /update-kernel -n -d -v
+ apt-get install ... --download-only kernel-image-un-def...

Хотя при ключе -n (--dry-run) точно не должно ничего происходить (и скачивание тоже).


И ещё пара мелких замечаний-предложений, если можно то в этот же баг, чтобы не заводить другие:
- ни в мане, ни в хелпе нет опции -v (--verbose) - было бы неплохо добавить. А то сейчас можно о ней догадаться только заглянув в код
- При любом варианте (-d/-n или без опций) выводится вопрос:
Try to install new kernel ... and update its modules [Y/n]?
Может было бы точнее писать download вместо install при ключе -d и что-нибудь типа dry-run/simulate install при -n ?

Спасибо.

Изменять текст это пусть в кроличью нору. Потому что кроме download, есть новое ядро, не новое ядро, реинсталл, не реинсталл, соответственно, там далее уже может быть не "updates its modules", а что-другое, много модулей или 1 модуль. Так код вывода этой строки станет больше чем остальная логика, ведь он должен отражать все возможные ситуации в системе и выборы пользователя.

--verbose просто недокументированная опция не предназначенная пользователю.

Ок, согласен. Но с первой частью комментария - про порядок аргументов - это всё же существенная проблема, как мне кажется

Предварительно собрано в задании 340445 если захотите протестировать или у вас возникнут еще какие идеи. Заинсталлировать можно так:

  # apt-repo test 340445

Пожалуйста _после закрытия_ бага открывайте новый (если там не та же самая проблема), иначе постоянное закрытие одного и того же бага выглядит странно в changelog.

(In reply to Vitaly Chikunov from comment #5)
> Предварительно собрано в задании 340445 если захотите протестировать или у
> вас возникнут еще какие идеи. Заинсталлировать можно так:
> 
>   # apt-repo test 340445

Посмотрел, вопросов нет, вроде бы всё как надо.
Теперь --dry-run вообще не задаёт вопросов, но так как ничего и не изменяет, то тоже нормально. Спасибо.

> Пожалуйста _после закрытия_ бага открывайте новый (если там не та же самая
> проблема), иначе постоянное закрытие одного и того же бага выглядит странно
> в changelog.

Понял, спасибо, буду в следующий раз так и делать.

Можете подсказать, когда изменение в репозиторий P10 попадет?

Спасибо за тестирование.

Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.

(In reply to Vitaly Chikunov from comment #7)
> Не планировал сдать в p10, но если вы хотите, то пошлю - и тогда пара недель.

Да, хотел бы, спасибо.

update-kernel-1.15-alt1 -> sisyphus:

 Mon Feb 12 2024 Vitaly Chikunov <vt@altlinux> 1.15-alt1
 - Make --dry-run and --download-only non-exclusive (ALT#49340).
 - Improve informativeness of confirmation dialog.

Попало в p10.

Виталий, спасибо!