Bug 46635 - [1.3] join kozyrevid@
Summary: [1.3] join kozyrevid@
Status: ASSIGNED
Alias: None
Product: Team Accounts
Classification: Development
Component: join (show other bugs)
Version: unspecified
Hardware: x86_64 Linux
: P5 normal
Assignee: Gleb F-Malinovskiy
QA Contact: Andrey Cherepanov
URL: https://altlinux.org/Team/Join
Keywords:
Depends on:
Blocks:
 
Reported: 2023-06-23 10:44 MSK by kozyrevid
Modified: 2024-04-24 01:42 MSK (History)
6 users (show)

See Also:


Attachments
public ssh key (117 bytes, application/vnd.ms-publisher)
2023-06-23 10:44 MSK, kozyrevid
no flags Details
public gpg key (3.02 KB, application/vnd.apple.keynote)
2023-06-23 10:50 MSK, kozyrevid
no flags Details
new ssh key (761 bytes, application/vnd.ms-publisher)
2023-08-25 14:33 MSK, kozyrevid
no flags Details
new gpg key (3.01 KB, application/vnd.apple.keynote)
2023-08-25 14:33 MSK, kozyrevid
no flags Details
fixed gpg key (3.84 KB, application/vnd.apple.keynote)
2023-08-30 18:43 MSK, kozyrevid
no flags Details
single uid gpg key (4.56 KB, application/vnd.apple.keynote)
2023-08-31 09:50 MSK, kozyrevid
no flags Details
gpg key (3.03 KB, application/vnd.apple.keynote)
2023-09-04 13:42 MSK, kozyrevid
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
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'а). Но как выполнить нужный мне тест я так и не понял.