Bug 40500 - `control openssl-gost enable` включает gost, но возвращает ошибку
Summary: `control openssl-gost enable` включает gost, но возвращает ошибку
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: openssl-gost-engine (show other bugs)
Version: unstable
Hardware: x86 Linux
: P5 normal
Assignee: manowar@altlinux.org
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-07-15 15:12 MSK by Vitaly Chikunov
Modified: 2021-07-15 21:17 MSK (History)
3 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Vitaly Chikunov 2021-07-15 15:12:03 MSK
# control openssl-gost enable; echo $?
control: openssl-gost: Requested enable, got enabled
1
Comment 1 manowar@altlinux.org 2021-07-15 15:14:34 MSK
А зачем "enable" вместо "enabled"? Это feature request?
Comment 2 Dmitry V. Levin 2021-07-15 15:18:48 MSK
(In reply to manowar@altlinux.org from comment #1)
> А зачем "enable" вместо "enabled"? Это feature request?

Очевидно, это сообщение об ошибке в пакете openssl-gost-engine.
Comment 3 Vitaly Chikunov 2021-07-15 15:25:20 MSK
А там надо писать enabled. Я этого не понял сразу. Sorry.

        enabled|enable)
        enable_gost
                ;;
        disabled|disable)
        disable_gost
                ;;
Comment 4 Dmitry V. Levin 2021-07-15 15:30:37 MSK
(In reply to Vitaly Chikunov from comment #3)
> А там надо писать enabled. Я этого не понял сразу. Sorry.
> 
>         enabled|enable)
>         enable_gost
>                 ;;
>         disabled|disable)
>         disable_gost
>                 ;;

control считает ошибкой ситуацию, когда выставленное состояние не совпадает с заказанным.

Зачем enable и disable, если состояния на самом деле enabled и disabled?
Comment 5 Vitaly Chikunov 2021-07-15 15:34:53 MSK
В общем интерфейс у control слегка запутывающий. Я закрываю баг.
Comment 6 Dmitry V. Levin 2021-07-15 15:41:20 MSK
.
Comment 7 manowar@altlinux.org 2021-07-15 18:44:31 MSK
(Ответ для Vitaly Chikunov на комментарий #3)
> А там надо писать enabled. Я этого не понял сразу. Sorry.

Нужно ориентироваться на вывод

    control openssl-gost help
    control openssl-gost list

>         enabled|enable)
>         enable_gost
>                 ;;
>         disabled|disable)
>         disable_gost
>                 ;;

  Понял, лишние ключевые слова нужно убрать. Но может и вправду лучше переделать на 'enable' и 'disable'? С другой стороны, я их выбирал, помню, исходя из вывода

   # control openssl-gost
   enabled

Вроде так понятнее, когда для текущего состояния (статуса) выводятся слова 'enabled' и 'disabled'. Но в этом случае и для включения нужно использовать именно их --- хак с синонимами не прошёл.
Comment 8 Vitaly Chikunov 2021-07-15 18:47:43 MSK
Моё мнение, что нужно оставить "enabled", так как я уже так коммичу в апстрим для CI тестов в докере для альтов.
Comment 9 manowar@altlinux.org 2021-07-15 18:49:56 MSK
О! У этого контрола есть пользователи! :-) Хорошо.
Comment 10 Vitaly Chikunov 2021-07-15 18:54:13 MSK
Теперь есть, так как без него тесты гостовых алгоритмов скипаются.
https://github.com/mimizohar/ima-evm-utils/commit/01cb906942b8a7042d4fd3cd7d0fd29c1532d589
Comment 11 manowar@altlinux.org 2021-07-15 19:41:48 MSK
Можно проверять: http://git.altlinux.org/tasks/279057/
Comment 12 manowar@altlinux.org 2021-07-15 19:51:28 MSK
Не, осечка. Видимо, нужно добавить сообщение об ошибке, как в других контролах.
Comment 13 Repository Robot 2021-07-15 20:24:24 MSK
openssl-gost-engine-1.1.0.3.0.255.ge3af41d.p1-alt4 -> sisyphus:

 Thu Jul 15 2021 Paul Wolneykien <manowar@altlinux> 1.1.0.3.0.255.ge3af41d.p1-alt4
 - Fix: Remove synonyms for 'enabled' and 'disabled' states (closes: 40500).
Comment 14 Vitaly Chikunov 2021-07-15 21:17:53 MSK
Спасибо!