Summary: | [1.3] join kozyrevid@ | ||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Product: | Team Accounts | Reporter: | kozyrevid | ||||||||||||||||
Component: | join | Assignee: | 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: |
|
Created attachment 13586 [details]
public gpg key
Псевдоним: kozyrevid Почта: kozyrevid@basealt.ru Ментор: Андрей Лимачко <liannnix@altlinux.org> Цели: научиться собирать пакеты Личная почта: uyraq2001@gmail.com Менторство подтверждаю. (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>" Ещё комментарий, совпадающий с вашим ником не имеет никакого смысла. Лучше никакого комментария, чем бессмысленный. Created attachment 14209 [details]
new ssh key
Created attachment 14210 [details]
new gpg key
(In reply to kozyrevid from comment #7) > Created attachment 14210 [details] > new gpg key > В ключе должны быть имя в формате "<First name> <Last name>" Created attachment 14281 [details]
fixed gpg key
(In reply to kozyrevid from comment #9) > Created attachment 14281 [details] > fixed gpg key В этом файле сразу два uid, но один из них явно лишний. Created attachment 14286 [details]
single uid gpg key
(In reply to kozyrevid from comment #11) > Created attachment 14286 [details] > single uid gpg key Здесь тоже два. :) Created attachment 14325 [details]
gpg key
Актуально ли ещё? Да, актуально. Ментор есть, ключи в порядке. T/J/S -> 1.3. Поработали с кандидатом (в рамках передачи опыта) над сборкой 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/ (Ответ для 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/ А можно не надо? Посмотрите пожалуйста, как это делается в других пакетах. И в дальнейшем подумайте ещё раз, прежде чем собирать в сизиф устаревшие пакеты. (Ответ для 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. Спасибо. Кандидат предложил на ревью пакет: 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__? Исправил большую часть замечаний: - переименовал пакет, 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'а). Но как выполнить нужный мне тест я так и не понял. |
Created attachment 13585 [details] public ssh key