Bug 45885

Эта заявка недооформлена.
Можете переоткрыть баг когда решите её оформить.

Created attachment 13073 [details]
Прикрепляю публичную часть SSH ключа

Created attachment 13074 [details]
Прикрепляю публичную часть GPG ключа

Created attachment 13078 [details]
Прикрепляю публичную часть SSH ключа

Подтверждаю заявку.
Прошу секретаря зарегистрировать ключи.

Считаю что кандидат уже умеет генерировать ключи и готов к переходу на следующий этап.

(In reply to Danila from comment #4)
> Created attachment 13078 [details]
> Прикрепляю публичную часть SSH ключа
Ok.

(In reply to Danila from comment #3)
> Created attachment 13074 [details]
> Прикрепляю публичную часть GPG ключа
Я не понимаю, зачем нужен такой комментарий к ключу, но формально ключ, конечно, соответствует критериям.

Кандидат готов начать встаупление в team.

ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.     

T/J/S -> 2.3.

Прошу кандидата предоставить примеры пакетов на git.altlinux.org.

Добавил несколько пакетов: https://git.altlinux.org/people/skachedubov/packages/
1. extarct-dtb: инструмент для разделения образа ядра с добавленными dtb на отдельные файлы ядра и dtb.
2. StegoPy: инструмент для стеанографии методом LSB.
3. spi-tools: инструмент командной строки, который помогает использовать устройства Linux spidev.
4. vunit: - фреймворк для Unit-тестов для Verilog.

(Ответ для Danila Skachedubov на комментарий #11)
> Добавил несколько пакетов:
> https://git.altlinux.org/people/skachedubov/packages/
> 1. extarct-dtb: инструмент для разделения образа ядра с добавленными dtb на
> отдельные файлы ядра и dtb.
> 2. StegoPy: инструмент для стеанографии методом LSB.
> 3. spi-tools: инструмент командной строки, который помогает использовать
> устройства Linux spidev.
> 4. vunit: - фреймворк для Unit-тестов для Verilog.

Посмотрел первый пакет из списка.
Тег Packager надо убрать, он больше не используется.
Тег License сделать в соответствии с https://www.altlinux.org/Spec#License.

Исправить:
warning: Installed (but unpackaged) file(s) found:
    /usr/CHANGES.md
    /usr/LICENSE
    /usr/bin/extract-dtb
    /usr/lib/python3/site-packages/extract_dtb-1.2.4.dev0.dist-info/METADATA
    /usr/lib/python3/site-packages/extract_dtb-1.2.4.dev0.dist-info/entry_points.txt

Остальные пакеты пока не смотрел, сказанное выше актуально и для них.

Ещё актуально?

Внес изменения в пакеты:
https://git.altlinux.org/people/skachedubov/packages/
1. Удалил тэг Packager
2. Лицензии указал макросами
3. Исправил warning'и при сборке пакета в хэшере.

(Ответ для Danila Skachedubov на комментарий #14)
> Внес изменения в пакеты:
> https://git.altlinux.org/people/skachedubov/packages/
> 1. Удалил тэг Packager
> 2. Лицензии указал макросами
> 3. Исправил warning'и при сборке пакета в хэшере.

Посмотрел второй пакет из списка. Он не собирается.

Исправил ошибки в spec файлах:
1. Изменил домен в почте на правильный (altlinux.org).
2. Исправил путь в секции %install в пакете StegoPy

Первый пакет:
1) Смешивать свои коммиты с коммитами апстрима не очень хорошая идея по моему.
   Наверное, лучше свои коммиты делать в отдельной ветке.

Второй пакет:
1) первый пункт для первого пакета.
   https://www.altlinux.org/Spec#Version - про версии.
2) Почему не использовать %_bindir вместо %define install_dir /usr/bin/ ?

Третий пакет:
1) первый пункт для первого пакета.
2) зачем эта переменная - %define version 4.7.0?

Четвёртый пакет:
1) Первый пункт из первых трёх можно посмотреть здесь.
2) Ветку по умолчанию логичней назвать sisyphus и собирать из неё, зачем master?

Исправил ошибки:
Разделил ветки с апстримом и своими коммитами со сборкой
Исправил использование макросов в spec файлах
Разобрался с промежуточными версиями для пакета StegoPy

 - Исправил ошибку при указании промежуточной версии, созданной командой git describe
 - Установил дефолтный бранч sisyphus во всех репозиториях командой ssh gitery default-branch

extract-dtb:
1)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"


StegoPy:
1)Пакет не собирается.

2) v0.0.4beta-14-g823a92b - странная версия.
https://www.altlinux.org/Spec#Version

3)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"

4) из https://bugzilla.altlinux.org/show_bug.cgi?id=45885#c17
> 2) Почему не использовать %_bindir вместо %define install_dir /usr/bin/ ?

spi-tools:
1)Пакет не собирается.

vunit:
1) из https://bugzilla.altlinux.org/show_bug.cgi?id=45885#c12
> Тег Packager надо убрать, он больше не используется.

2)https://www.altlinux.org/Spec#%25description
"Длина каждой строки не должна превышать 72 символа"

Собирал пакеты командой gear-hsh --commit, исправив все недочеты и ошибки,  выявленные в предыдущей проверке, отписался в багзиллу, подсознательно считая, что закоммитил и запушил все изменения...
Прошу прощение за потраченное время на проверку(
Сейчас закоммитил и отправил все изменения, так же исправил длину строк в секции  %description.

StegoPy:
1) Странная версия - v0.0.4beta.14.g823a92b.
https://www.altlinux.org/Spec#Version
В примере показано как из этого делают версию.
2) Думаю стоит запушить тэг апстрима с помощью которого это получилось.

tar: . name=@name@-@version@ base=@name@-@version@ - это не ошибка, но base в данном случае дублирует действие по умолчанию (gear-rules(5)).

Запушил тег, для получения командой git describe --abbrev=1 HEAD^ количества коммитов до последнего коммита апстрима и для формирования уникального имени объекта.
Исправил версию. 
Да, без указания base, используется имя архива без суффикса, в моем случае это излишне.

(Ответ для Danila Skachedubov на комментарий #23)
> Запушил тег, для получения командой git describe --abbrev=1 HEAD^ количества
> коммитов до последнего коммита апстрима и для формирования уникального имени
> объекта.
> Исправил версию. 

В версии 0.0.4.0.14.g823a один лишний символ по сравнению с примером из https://www.altlinux.org/Spec#Version

Исправил версию 0.0.4.0.14.g823a. Удалил префикс "g".

Считаю, что кандидат готов к следующему шагу.
Прошу секретаря дать доступ к сборочнице.

За прошедшее время мы ужесточили правила про gpg-ключи и принимаем ключи только без комментария.  Можно было бы рассмотреть вопрос об исключении, но я думаю, что комментарий (developer) не имеет никакого смысла.

Кандидат, прошу обновить ключ.

Created attachment 17719 [details]
GPG ключ

Добрый день, прикладываю новый gpg-ключ.