ALT Linux Bugzilla – Full Text Bug Listing
| Summary: | Активна кнопка "назад" в alterator-setup после шага preinstall | ||||||
|---|---|---|---|---|---|---|---|
| Product: | Sisyphus | Reporter: | DVoropaev <voropaevdmtr> | ||||
| Component: | alterator-net-eth | Assignee: | Mikhail Efremov <sem> | ||||
| Status: | NEW --- | QA Contact: | qa-sisyphus | ||||
| Severity: | normal | ||||||
| Priority: | P5 | CC: | antohami, boyarsh, iv, jqt4, mike, rider, sem, vitty, zerg | ||||
| Version: | unstable | ||||||
| Hardware: | all | ||||||
| OS: | Linux | ||||||
| See Also: | https://bugzilla.altlinux.org/show_bug.cgi?id=43878 | ||||||
| Attachments: |
|
||||||
|
Description
DVoropaev
2022-09-28 13:53:00 MSK
> Во время установки тестового образа
> (https://git.altlinux.org/people/jqt4/public/?p=mkimage-profiles-rpi.git;
> a=commit;h=969e1719506edc48a719c8c9ef21f7a083afc7a2)
Тестовая сборка slinux-rpi-20220923-aarch64.img.xz, собранная на этом профиле
> Тестовая сборка slinux-rpi-20220923-aarch64.img.xz, собранная на этом профиле Образ, в котором наблюдается проблема, опубликован: http://beta.altlinux.org/simply/aarch64/slinux-10.1-beta2-rpi4-aarch64.img.xz (Ответ для jqt4 на комментарий #2) > > Тестовая сборка slinux-rpi-20220923-aarch64.img.xz, собранная на этом профиле > > Образ, в котором наблюдается проблема, опубликован: > http://beta.altlinux.org/simply/aarch64/slinux-10.1-beta2-rpi4-aarch64.img.xz Эта проблема во всех дистрибутивах есть. Её нет в стартеркитах, так как там нет шага preinstall. Доступность кнопки назад определяется в ui модуля alterator. В данном случае необходимо запретить кнопку назад в alterator-net-eth. Я полагаю, что аналогичная проблема присутствует в e2k, так как там нет шага grub. Но и возвращение обратно к установке загрузчика grub также кажется неправильным. (In reply to Антон Мидюков from comment #4) > Доступность кнопки назад определяется в ui модуля alterator. Это, кстати, немного странно: если я переставлю шаги, мне нужно будет править ui-файлы? Мне кажется, логичнее, когда доступность кнопки "назад" определяется не тем, что правится сейчас, а тем, куда она возвращает -- например, возвращаться на шаг preinstall никогда не нужно. (Ответ для Ivan A. Melnikov на комментарий #5) > (In reply to Антон Мидюков from comment #4) > > Доступность кнопки назад определяется в ui модуля alterator. > > Это, кстати, немного странно: если я переставлю шаги, мне нужно будет > править ui-файлы? Мне кажется, логичнее, когда доступность кнопки "назад" > определяется не тем, что правится сейчас, а тем, куда она возвращает -- > например, возвращаться на шаг preinstall никогда не нужно. Согласен. Но я посмотрел, что работает это именно так. У меня даже получилось кнопку назад сделать не активной. Правда, я не уверен, что сделал правильно. Подсмотрел, как сделано в alterator-grub. Как вариант, можно убрать возвращение назад у всех шагов, которые выполняются из установленной системы. diff --git a/alterator-net-eth/ui/net-eth/index.scm b/alterator-net-eth/ui/net-eth/index.scm index 8ab025a..b9072cb 100644 --- a/alterator-net-eth/ui/net-eth/index.scm +++ b/alterator-net-eth/ui/net-eth/index.scm @@ -361,12 +361,11 @@ (form-bind "bridge_del" "click" bridge-del) (form-bind "bridge_ch" "click" bridge-ch) - (or (global 'frame:next) + (or (and (global 'frame:next) (frame:back-activity #f)) (begin (form-bind "apply" "click" (lambda() (begin (commit-interface) (read-interface (form-value "name") (form-value "ipv"))))) (form-bind "reset" "click" reset-interface))))) -(frame:on-back (lambda() (or (commit-interface) 'cancel))) (frame:on-next (lambda() (or (commit-interface) 'cancel))) ; vim: ft=lisp expandtab (Ответ для Антон Мидюков на комментарий #6) > (Ответ для Ivan A. Melnikov на комментарий #5) > > (In reply to Антон Мидюков from comment #4) > > > Доступность кнопки назад определяется в ui модуля alterator. > > > > Это, кстати, немного странно: если я переставлю шаги, мне нужно будет > > править ui-файлы? Мне кажется, логичнее, когда доступность кнопки "назад" > > определяется не тем, что правится сейчас, а тем, куда она возвращает -- > > например, возвращаться на шаг preinstall никогда не нужно. > > Согласен. Но я посмотрел, что работает это именно так. У меня даже > получилось кнопку назад сделать не активной. Правда, я не уверен, что сделал > правильно. > Подсмотрел, как сделано в alterator-grub. > Как вариант, можно убрать возвращение назад у всех шагов, которые > выполняются из установленной системы. > > diff --git a/alterator-net-eth/ui/net-eth/index.scm > b/alterator-net-eth/ui/net-eth/index.scm > index 8ab025a..b9072cb 100644 > --- a/alterator-net-eth/ui/net-eth/index.scm > +++ b/alterator-net-eth/ui/net-eth/index.scm > @@ -361,12 +361,11 @@ > (form-bind "bridge_del" "click" bridge-del) > (form-bind "bridge_ch" "click" bridge-ch) > > - (or (global 'frame:next) > + (or (and (global 'frame:next) (frame:back-activity #f)) > (begin (form-bind "apply" "click" (lambda() (begin (commit-interface) > (read-interface > (form-value "name") (form-value "ipv"))))) > (form-bind "reset" "click" reset-interface))))) > > -(frame:on-back (lambda() (or (commit-interface) 'cancel))) > (frame:on-next (lambda() (or (commit-interface) 'cancel))) > > ; vim: ft=lisp expandtab Добавил исправление. Собрал, протестировал. Прошу принять. https://git.altlinux.org/tasks/313536/ |
