Bug 45612 - Enable python binding
Summary: Enable python binding
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: crun (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 enhancement
Assignee: Alexey Shabalin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-03-21 21:52 MSK by Alexey Gladkov
Modified: 2023-03-22 20:30 MSK (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexey Gladkov 2023-03-21 21:52:43 MSK
Пожалуйста, если возможно соберите биндинг для python.

Также я заметил в спеке есть строчка:
```
printf "/* autogenerated.  */\n#ifndef GIT_VERSION\n# define GIT_VERSION \"%s\"\n#endif\n" %git_commit > git-version.h
```
Это ошибка. %s должен быть заэкранирован (%%s), иначе rpm считает это макросом и ругается в лог:

warning: Macro %s not found
Comment 1 Andrew Vasilyev 2023-03-21 22:53:37 MSK
(Ответ для Alexey Gladkov на комментарий #0)
> Пожалуйста, если возможно соберите биндинг для python.

  Правильно ли я понимаю, что нужен файл
/usr/lib64/python3/site-packages/python_crun.so
  ?
  В какой пакет его паковать? (В rpm/crun.spec.in python binding
  не собирают, а /usr/lib64 вообще сносят).

> Также я заметил в спеке есть строчка:
> ```
> printf "/* autogenerated.  */\n#ifndef GIT_VERSION\n# define GIT_VERSION
> \"%s\"\n#endif\n" %git_commit > git-version.h
> ```
> Это ошибка. %s должен быть заэкранирован (%%s), иначе rpm считает это
> макросом и ругается в лог:
> 
> warning: Macro %s not found

  Спасибо, исправлено.
Comment 2 Andrew Vasilyev 2023-03-21 23:42:00 MSK
  Пожалуйста, попробуйте crun из задания #317173
Comment 3 Alexey Gladkov 2023-03-22 16:25:30 MSK
Похоже работает :)

Только я бы предложил положить модуль в отдельный подпакет. Так он будет лучше искаться в apt-cache и не принесёт дополнительных файлов тем кто сейчас хочет только crun. Хотя, тут вам виднее.
Comment 4 Andrew Vasilyev 2023-03-22 17:38:35 MSK
(Ответ для Alexey Gladkov на комментарий #3)
> Только я бы предложил положить модуль в отдельный подпакет. Так он будет
> лучше искаться в apt-cache и не принесёт дополнительных файлов тем кто

  Ну, там пока один файл на 50 кБ...

> сейчас хочет только crun. Хотя, тут вам виднее.

  Вопрос, что писать в Requires для этого подпакета? Он работоспособен
  без остального crun, или для него нужен основной пакет?
Comment 5 Alexey Gladkov 2023-03-22 17:49:05 MSK
(Ответ для Andrew Vasilyev на комментарий #4)
>   Ну, там пока один файл на 50 кБ...

Именно поэтому я и говорю, что вам виднее :)

> > сейчас хочет только crun. Хотя, тут вам виднее.
> 
>   Вопрос, что писать в Requires для этого подпакета? Он работоспособен
>   без остального crun, или для него нужен основной пакет?

python_crun линкуется с libcrun из основного пакета поэтому зависимость появится сама. Также если запаковать его правильно rpm-build-python3 добавит зависимость на правильный ABI питона (типа 'python3.10-ABI(64bit)').
Comment 6 Alexey Shabalin 2023-03-22 18:16:42 MSK
Конечно нужно делать подпакет, тут даже другие варианты не стоит рассматривать.
Comment 7 Andrew Vasilyev 2023-03-22 18:56:07 MSK
  Пожалуйста, проверьте установку и работу python3-module-crun из задания #317173.
  (Удалив старый пакет).
Comment 8 Alexey Gladkov 2023-03-22 18:59:58 MSK
Работает.
Comment 9 Repository Robot 2023-03-22 20:30:04 MSK
crun-1.8.2-alt2 -> sisyphus:

 Tue Mar 21 2023 Andrew A. Vasilyev <andy@altlinux> 1.8.2-alt2
 - build with python bindings (ALT #45612)