Bug 39041 - При указании ключа -r для update-kernel требуется ввести релиз в неожиданном формате
Summary: При указании ключа -r для update-kernel требуется ввести релиз в неожиданном ...
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: update-kernel (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Vitaly Chikunov
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on: 36949
Blocks:
  Show dependency tree
 
Reported: 2020-10-06 10:49 MSK by mattaku@altlinux.org
Modified: 2020-11-21 04:38 MSK (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description mattaku@altlinux.org 2020-10-06 10:49:16 MSK
https://bugzilla.altlinux.org/show_bug.cgi?id=36949
Comment 1 Vitaly Chikunov 2020-10-06 11:52:07 MSK
1. Наоборот, баг #36949 зависит от этого, а не этот от того. Так как течение пакетов - из Сизифа в p9.
2. Я могу переделать логику выбора версии в update_kernel -r, если хотите.
Comment 2 Vitaly Lipatov 2020-10-06 13:04:05 MSK
(Ответ для Vitaly Chikunov на комментарий #1)
> 1. Наоборот, баг #36949 зависит от этого, а не этот от того. Так как течение
> пакетов - из Сизифа в p9.
Поменять местами продукты багов? в баге на p9 есть описание задачи :)

> 2. Я могу переделать логику выбора версии в update_kernel -r, если хотите.
Было бы здорово!
Как я понимаю, -r должен поддерживать формат, в котором заданы имена каталогов /lib/modules.

Вообще в справке как-то странно описано назначение -r:
-r, --release     install kernel with specified release (alt10, alt22, etc)
Comment 3 Vitaly Chikunov 2020-10-09 15:22:57 MSK
Task #259508. Сделал и fix и feature request. Протестируйте, пожалуйста.
Comment 4 Vitaly Chikunov 2020-10-10 08:49:02 MSK
Прямая ссылка на коммит http://git.altlinux.org/people/vt/packages/?p=update-kernel.git;a=commitdiff;h=7232e37b6901c9f0a176dd5d39a2ffdb26b9a0f5 для возможного вашего review.
Comment 5 Vitaly Lipatov 2020-10-10 10:42:12 MSK
(Ответ для Vitaly Chikunov на комментарий #4)
> Прямая ссылка на коммит
> http://git.altlinux.org/people/vt/packages/?p=update-kernel.git;a=commitdiff;
> h=7232e37b6901c9f0a176dd5d39a2ffdb26b9a0f5 для возможного вашего review.

> Allow setting release in `uname -r' format.
Эта важная особенность не отражена в хелпе

и вообще, мы же
4.14.104-std-def-alt1

называем релизом ядра?
Comment 6 Vitaly Chikunov 2020-10-10 10:50:27 MSK
> Эта важная особенность не отражена в хелпе

Сказано "kernel with specified release" и "etc", я не придумал что ещё написать, кроме как убрать тренд, что это релиз тэгов.

Не знаю что "мы" называем, так предыдущий автор явно называл релизом только релиз тэг пакета. В uname(1):

       -r, --kernel-release
              print the kernel release

Так что поддержать такую же семантику вполне логично и может соответствовать ожиданиям пользователя. Если это и есть "kernel release", то что не отражено в хелпе если сказано "kernel with specified release"?
Comment 7 Vitaly Chikunov 2020-10-10 10:54:27 MSK
s/что это релиз тэгов/из релиз тэгов/
Comment 8 Vitaly Lipatov 2020-10-10 11:49:25 MSK
(Ответ для Vitaly Chikunov на комментарий #6)
> > Эта важная особенность не отражена в хелпе
> 
> Сказано "kernel with specified release" и "etc", я не придумал что ещё
> написать, кроме как убрать тренд, что это релиз тэгов.
> 
> Не знаю что "мы" называем, так предыдущий автор явно называл релизом только
> релиз тэг пакета. В uname(1):
Посмотрел коммиты, да, это странно.
Но смысла в таком указании нет. Если уж указывать, то kernel release. 

> 
>        -r, --kernel-release
>               print the kernel release
> 
> Так что поддержать такую же семантику вполне логично и может соответствовать
> ожиданиям пользователя. Если это и есть "kernel release", то что не отражено
> в хелпе если сказано "kernel with specified release"?
Тогда надо в хелпе пример поменять с alt10 на 5.7.19-std-def-alt1
И написать словосочетание kernel release
Comment 9 Vitaly Chikunov 2020-10-10 12:21:03 MSK
> И написать словосочетание kernel release

Я не лингвист, но тут его нет - "kernel with specified release"?
Comment 10 Vitaly Lipatov 2020-10-10 12:32:19 MSK
(Ответ для Vitaly Chikunov на комментарий #9)
> > И написать словосочетание kernel release
> 
> Я не лингвист, но тут его нет - "kernel with specified release"?

Давайте так:
-r, --release     install kernel with specified kernel release (f.e, 5.7.19-std-def-alt1)
Comment 11 Michael Shigorin 2020-10-10 15:54:00 MSK
(Ответ для Vitaly Lipatov на комментарий #10)
> (Ответ для Vitaly Chikunov на комментарий #9)
> > > И написать словосочетание kernel release
...чтоб было ясно, что не package release

> Давайте так:
> -r, --release     install kernel with specified kernel release (f.e,
> 5.7.19-std-def-alt1)
+1
Comment 12 Vitaly Chikunov 2020-10-12 17:23:25 MSK
http://git.altlinux.org/tasks/259508/

- расширен хелп
- добавлен ещё формат в -r
- попутно новая опция -l
Comment 13 Repository Robot 2020-11-21 04:38:56 MSK
update-kernel-0.9.20-alt1 -> sisyphus:

 Thu Nov 19 2020 Vitaly Chikunov <vt@altlinux> 0.9.20-alt1
 - update-kernel: Fix and improve -r option (closes: #39041).
 - update-kernel: Add --list option.