Bug 46635

Summary: [1.3] join kozyrevid@
Product: Team Accounts Reporter: kozyrevid
Component: joinAssignee: Gleb F-Malinovskiy <glebfm>
Status: ASSIGNED --- QA Contact: Andrey Cherepanov <cas>
Severity: normal    
Priority: P5 CC: glebfm, grenka, kozyrevid, ldv, liannnix, liannnix
Version: unspecified   
Hardware: x86_64   
OS: Linux   
URL: https://altlinux.org/Team/Join
Attachments:
Description Flags
public ssh key
none
public gpg key
none
new ssh key
none
new gpg key
none
fixed gpg key
none
single uid gpg key
none
gpg key none

Description kozyrevid 2023-06-23 10:44:25 MSK
Created attachment 13585 [details]
public ssh key
Comment 1 kozyrevid 2023-06-23 10:50:33 MSK
Created attachment 13586 [details]
public gpg key
Comment 2 kozyrevid 2023-06-23 11:00:14 MSK
Псевдоним: kozyrevid 
Почта: kozyrevid@basealt.ru
Ментор: Андрей Лимачко <liannnix@altlinux.org>
Цели: научиться собирать пакеты
Comment 3 kozyrevid 2023-06-23 12:43:15 MSK
Личная почта: uyraq2001@gmail.com
Comment 4 Andrey Limachko 2023-07-02 19:22:15 MSK
Менторство подтверждаю.
Comment 5 Gleb F-Malinovskiy 2023-08-04 13:49:47 MSK
(In reply to kozyrevid from comment #1)
> Created attachment 13586 [details]
> public gpg key

https://www.altlinux.org/Team/Join/Candidate 
> В ключе должны быть имя в формате "<First name> <Last name>"

Ещё комментарий, совпадающий с вашим ником не имеет никакого смысла.  Лучше никакого комментария, чем бессмысленный.
Comment 6 kozyrevid 2023-08-25 14:33:05 MSK
Created attachment 14209 [details]
new ssh key
Comment 7 kozyrevid 2023-08-25 14:33:49 MSK
Created attachment 14210 [details]
new gpg key
Comment 8 Gleb F-Malinovskiy 2023-08-30 12:18:19 MSK
(In reply to kozyrevid from comment #7)
> Created attachment 14210 [details]
> new gpg key
> В ключе должны быть имя в формате "<First name> <Last name>"
Comment 9 kozyrevid 2023-08-30 18:43:22 MSK
Created attachment 14281 [details]
fixed gpg key
Comment 10 Gleb F-Malinovskiy 2023-08-30 19:23:25 MSK
(In reply to kozyrevid from comment #9)
> Created attachment 14281 [details]
> fixed gpg key
В этом файле сразу два uid, но один из них явно лишний.
Comment 11 kozyrevid 2023-08-31 09:50:37 MSK
Created attachment 14286 [details]
single uid gpg key
Comment 12 Gleb F-Malinovskiy 2023-08-31 15:44:25 MSK
(In reply to kozyrevid from comment #11)
> Created attachment 14286 [details]
> single uid gpg key
Здесь тоже два. :)
Comment 13 kozyrevid 2023-09-04 13:42:59 MSK
Created attachment 14325 [details]
gpg key
Comment 14 Gleb F-Malinovskiy 2023-11-08 20:19:23 MSK
Актуально ли ещё?
Comment 15 kozyrevid 2023-11-09 20:16:26 MSK
Да, актуально.
Comment 16 Gleb F-Malinovskiy 2023-11-24 16:33:28 MSK
Ментор есть, ключи в порядке.
T/J/S -> 1.3.
Comment 17 Andrey Limachko 2024-02-26 21:03:20 MSK
Поработали с кандидатом (в рамках передачи опыта) над сборкой python3 пакета socli.
https://github.com/gautamkrishnar/socli

В результате получилось два репозитория: с наработками кандидата и моими дополнениями.
https://github.com/uyraq2001/socli_alt/tree/wip
https://github.com/liannnix/socli/tree/wip

В процессе выяснилось, что для работы socli не хватает python3 модуля Py-StackExchange, который и предлагается собрать кандидату.
https://github.com/lucjon/Py-StackExchange

Следующим шагом предлагается собрать модуль pytest-runner, который у нас протух и давно выкинут из Sisyphus. Он, собственно, требуется для запуска тестов socli.
https://packages.altlinux.org/ru/sisyphus/srpms/python-module-pytest-runner/
https://github.com/pytest-dev/pytest-runner/
Comment 18 Grigory Ustinov 2024-02-26 21:27:47 MSK
(Ответ для Andrey Limachko на комментарий #17)
> Следующим шагом предлагается собрать модуль pytest-runner, который у нас
> протух и давно выкинут из Sisyphus. Он, собственно, требуется для запуска
> тестов socli.
> https://packages.altlinux.org/ru/sisyphus/srpms/python-module-pytest-runner/
> https://github.com/pytest-dev/pytest-runner/

А можно не надо? Посмотрите пожалуйста, как это делается в других пакетах. И в дальнейшем подумайте ещё раз, прежде чем собирать в сизиф устаревшие пакеты.
Comment 19 Andrey Limachko 2024-02-27 03:07:24 MSK
(Ответ для Grigory Ustinov на комментарий #18)
> (Ответ для Andrey Limachko на комментарий #17)
> > Следующим шагом предлагается собрать модуль pytest-runner, который у нас
> > протух и давно выкинут из Sisyphus. Он, собственно, требуется для запуска
> > тестов socli.
> > https://packages.altlinux.org/ru/sisyphus/srpms/python-module-pytest-runner/
> > https://github.com/pytest-dev/pytest-runner/
> 
> А можно не надо? Посмотрите пожалуйста, как это делается в других пакетах. И
> в дальнейшем подумайте ещё раз, прежде чем собирать в сизиф устаревшие
> пакеты.

Извиняюсь, косяк. Подумал, что это очередная приблуда для особо забористого запуска тестов и даже смотреть не стал.
Сейчас посмотрел, оно deprecated настолько, что об этом написали в README на github.
Спасибо.
Comment 20 Andrey Limachko 2024-04-18 17:40:10 MSK
Кандидат предложил на ревью пакет:
https://github.com/uyraq2001/Py-StackExchange/tree/first_build

Привожу свои замечания:
1. Имя пакета должно иметь вид python3-module-%pypi_name, где %pypi_name - это имя модуля с pypi.org (py-stackexchange).
2. Название репозитория должно совпадать с полным именем пакета.
3. Название spec-файлы должно совпадать с именем пакета.
4. Не стоит оставлять в spec'е комментарии, не несущие смысла.
5. Строка 67: Если уж нет man'ов, то и папку под них создавать не нужно.
6. 33-34 Provides здесь не нужены. Они должны сгенерироваться автоматически.
7. Необходимо включить тесты.
8. Добавить нормальный changelog.


По некоторым вопросам требуется "помощь зала":
1. Нужно ли упаковывать __pycache__?
Comment 21 kozyrevid 2024-04-24 01:42:50 MSK
Исправил большую часть замечаний:
- переименовал пакет, spec и репозиторий (https://github.com/uyraq2001/python3-module-py-stackexchange.git)
- убрал man'ы
- убрал Provides
- исправил changelog

При попытке добавить тесты возникла такая проблема: pytest подает с кодом 5 (никакие тесты не найдены). На сколько я понял, проблема заключается в том, что хоть я и прописал `%pyproject_run_pytest -ra %buildroot/testsuite.py`, rpm использует exec из дефолтного tox.ini: `pytest -vra` (правда мне не удалось найти этот файл в билдруте hasher'а). Но как выполнить нужный мне тест я так и не понял.