Created attachment 12902 [details] SSH public key Псевдоним: zerospirit email: zerospirit@yandex.ru Ментор: Сергей Турчин aka zerg Цель вступления: Научиться собирать пакеты
Created attachment 12903 [details] GPG key
Created attachment 12913 [details] new ssh key
Created attachment 12914 [details] new gpg key
.(In reply to Pavel Shilov from comment #2) > Created attachment 12913 [details] > new ssh key (In reply to Pavel Shilov from comment #3) > Created attachment 12914 [details] > new gpg key Ok.
Подтверждаю. Согласен быть ментором.
Актуально ли ещё?
(Ответ для Gleb F-Malinovskiy на комментарий #6) > Актуально ли ещё? Да.
Ментор есть, ключи в порядке. T/J/S -> 1.3.
(Ответ для Gleb F-Malinovskiy на комментарий #8) > Ментор есть, ключи в порядке. > T/J/S -> 1.3. Доступ на gitery отсуствует. Что-то от меня требуется для его получения?
(Ответ для Pavel Shilov на комментарий #9) > (Ответ для Gleb F-Malinovskiy на комментарий #8) > > Ментор есть, ключи в порядке. > > T/J/S -> 1.3. > > Доступ на gitery отсуствует. Что-то от меня требуется для его получения? Попросить ментора перевести заявку на этап 2.0, если он считает, что кандидат готов к этому.
Кандидат готов начать вступление.
ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3.
Подопечный готов собирать пакеты.
ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. Адрес подписан на devel@. T/J/S -> 3.6.
Подопечный готов отправлять пакеты в Сизиф.
Собрано на данный момент: https://packages.altlinux.org/ru/tasks/341489/ * yascreen * bpfmon https://packages.altlinux.org/ru/tasks/341587/ * zps * has * yank https://packages.altlinux.org/ru/tasks/344343/ * libnetconf2 * sysrepo https://packages.altlinux.org/ru/tasks/345305/ * fbcat * shellcat https://packages.altlinux.org/ru/tasks/344969/ * python3-module-pymsgbox * python3-module-pyscreeze * python3-module-pytweening * python3-module-mouseinfo * python3-module-pyautogui https://git.altlinux.org/tasks/356811/ * gobuster https://git.altlinux.org/tasks/356771/ * otree https://git.altlinux.org/tasks/356102/ * netmask https://git.altlinux.org/tasks/355829/ * tailspin https://git.altlinux.org/tasks/354295/ * bitcrook https://git.altlinux.org/tasks/353945/ * hostctl https://git.altlinux.org/tasks/352936/ * dsd-fme https://git.altlinux.org/tasks/353656/ * pwnat * tgcd https://git.altlinux.org/tasks/349574/ * rgbds * SameBoy
Собрал: https://git.altlinux.org/tasks/357348/ * lssh * lsshell * ent https://git.altlinux.org/tasks/357444/ * binsider
Собрал: https://git.altlinux.org/tasks/357897/ * systemctl-tui * netscanner
Собрал https://git.altlinux.org/tasks/360409/ * giflib * swayimg
Собрал https://git.altlinux.org/tasks/361547/ * amass
Призван рецензент (antohami@) для независимой оценки готовности кандидата. T/J/S -> 4.2.
Собрал https://git.altlinux.org/tasks/362276/ * python3-module-wikipediaapi.git * python3-module-pyttsx3.git https://git.altlinux.org/tasks/362367/ * python3-module-srt.git * python3-module-vosk-api.git
Я случайно глянул последние 4 пакета. А почему вы для сборки проектов из гитхаба не пользуетесь сборкой из апстримного гита с помощью схемы сборки из тэга?
Вот тут было сделано что-то совсем странное: https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git;a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe Зачем-то исходники были перенесены из подкаталога в корень. Зачем? Как обновлять планируете исходники в дальнейшем? Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку из апстримного тега. То есть смержится с апстримным git. И в .gear/rules добавить diff для получения патча, чтобы понимать, что всё смержено правильно.
То есть, видимо, все пакеты так собираются. Исходники в корень помещаются и пакуется всё вместе в архив исходников. Очень странный подход. Исходники апстрима принято отделять от тех изменений, которые вносит мантейнер, тем или иным способом.
(Ответ для Антон Мидюков на комментарий #25) > Исходники апстрима принято отделять от тех изменений, которые вносит мантейнер Свой спек в дереве исходников -- у нас куча таких пакетов издревле. В коммитах, да, чужие изменения отдельно от своих.
(Ответ для Sergey V Turchin на комментарий #26) > (Ответ для Антон Мидюков на комментарий #25) > > Исходники апстрима принято отделять от тех изменений, которые вносит мантейнер > Свой спек в дереве исходников -- у нас куча таких пакетов издревле. > В коммитах, да, чужие изменения отдельно от своих. Издревле и git'а не было. Ладно, фиг с этим странным методом упаковки. Вроде бы нет предписаний по упаковке, только рекомендации.
(Ответ для Grigory Ustinov на комментарий #23) > Я случайно глянул последние 4 пакета. А почему вы для сборки проектов из > гитхаба не пользуетесь сборкой из апстримного гита с помощью схемы сборки из > тэга? Пользуюсь, чтобы не тянуть ненужные коммиты, пока история не закомичена в Сизиф.
(Ответ для Антон Мидюков на комментарий #24) > Вот тут было сделано что-то совсем странное: > https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git; > a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe > > Зачем-то исходники были перенесены из подкаталога в корень. Зачем? > Как обновлять планируете исходники в дальнейшем? > > Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку > из апстримного тега. То есть смержится с апстримным git. И в .gear/rules > добавить diff для получения патча, чтобы понимать, что всё смержено > правильно. Произвел пересборку из апстримного тега https://git.altlinux.org/tasks/362409/
(Ответ для Pavel Shilov на комментарий #29) > (Ответ для Антон Мидюков на комментарий #24) > > Вот тут было сделано что-то совсем странное: > > https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git; > > a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe > > > > Зачем-то исходники были перенесены из подкаталога в корень. Зачем? > > Как обновлять планируете исходники в дальнейшем? > > > > Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку > > из апстримного тега. То есть смержится с апстримным git. И в .gear/rules > > добавить diff для получения патча, чтобы понимать, что всё смержено > > правильно. > > Произвел пересборку из апстримного тега > https://git.altlinux.org/tasks/362409/ У вас два верхних коммита с одинаковым commit message "2.53-alt1".
(Ответ для Антон Мидюков на комментарий #30) > (Ответ для Pavel Shilov на комментарий #29) > > (Ответ для Антон Мидюков на комментарий #24) > > > Вот тут было сделано что-то совсем странное: > > > https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git; > > > a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe > > > > > > Зачем-то исходники были перенесены из подкаталога в корень. Зачем? > > > Как обновлять планируете исходники в дальнейшем? > > > > > > Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку > > > из апстримного тега. То есть смержится с апстримным git. И в .gear/rules > > > добавить diff для получения патча, чтобы понимать, что всё смержено > > > правильно. > > > > Произвел пересборку из апстримного тега > > https://git.altlinux.org/tasks/362409/ > > У вас два верхних коммита с одинаковым commit message "2.53-alt1". Исправлено.
(Ответ для Pavel Shilov на комментарий #31) > (Ответ для Антон Мидюков на комментарий #30) > > (Ответ для Pavel Shilov на комментарий #29) > > > (Ответ для Антон Мидюков на комментарий #24) > > > > Вот тут было сделано что-то совсем странное: > > > > https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git; > > > > a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe > > > > > > > > Зачем-то исходники были перенесены из подкаталога в корень. Зачем? > > > > Как обновлять планируете исходники в дальнейшем? > > > > > > > > Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку > > > > из апстримного тега. То есть смержится с апстримным git. И в .gear/rules > > > > добавить diff для получения патча, чтобы понимать, что всё смержено > > > > правильно. > > > > > > Произвел пересборку из апстримного тега > > > https://git.altlinux.org/tasks/362409/ > > > > У вас два верхних коммита с одинаковым commit message "2.53-alt1". > > Исправлено. Так как вы не указали в спеке Patch: %name-%version-%release.patch то патч не попал в srpm, и мы не увидели, что есть отличия от апстрима. Возможно, эти изменения не нужны вовсе, тогда стоит разобраться с мержем, чтобы патч стал пустым.
(Ответ для Антон Мидюков на комментарий #32) > (Ответ для Pavel Shilov на комментарий #31) > > (Ответ для Антон Мидюков на комментарий #30) > > > (Ответ для Pavel Shilov на комментарий #29) > > > > (Ответ для Антон Мидюков на комментарий #24) > > > > > Вот тут было сделано что-то совсем странное: > > > > > https://git.altlinux.org/tasks/archive/done/_333/341489/gears/400/git?p=git; > > > > > a=commitdiff;h=f28f7dc9fccde2a24a0162dd0c083e7c8ac1cbfe > > > > > > > > > > Зачем-то исходники были перенесены из подкаталога в корень. Зачем? > > > > > Как обновлять планируете исходники в дальнейшем? > > > > > > > > > > Я бы рекомендовал перейти в данном пакете после такого безобразия на сборку > > > > > из апстримного тега. То есть смержится с апстримным git. И в .gear/rules > > > > > добавить diff для получения патча, чтобы понимать, что всё смержено > > > > > правильно. > > > > > > > > Произвел пересборку из апстримного тега > > > > https://git.altlinux.org/tasks/362409/ > > > > > > У вас два верхних коммита с одинаковым commit message "2.53-alt1". > > > > Исправлено. > > Так как вы не указали в спеке > Patch: %name-%version-%release.patch > > то патч не попал в srpm, и мы не увидели, что есть отличия от апстрима. > Возможно, эти изменения не нужны вовсе, тогда стоит разобраться с мержем, > чтобы патч стал пустым. Исправил сборку, на основе апстрима https://git.altlinux.org/tasks/362409/
(Ответ для Pavel Shilov на комментарий #33) > (Ответ для Антон Мидюков на комментарий #32) > > Так как вы не указали в спеке > > Patch: %name-%version-%release.patch > > > > то патч не попал в srpm, и мы не увидели, что есть отличия от апстрима. > > Возможно, эти изменения не нужны вовсе, тогда стоит разобраться с мержем, > > чтобы патч стал пустым. > > Исправил сборку, на основе апстрима https://git.altlinux.org/tasks/362409/ Я одобрил. Отправляйте.
Общие мелкие замечания, не требующие исправления сейчас (просто примите к сведению). В Summary не нужно ставить точку в конце. А в %description наоборот нужно ставить точку в конце предложения, даже если оно всего одно или состоит из макроса %summary. Длина строк в %description не должна превышать 80 символов. Да и в целом нужно стремиться к тому, чтобы строки в спеке были не длиннее 80 символов. Теперь к существенному: https://packages.altlinux.org/ru/tasks/344969/ * python3-module-pymsgbox * python3-module-pyscreeze * python3-module-pytweening * python3-module-mouseinfo * python3-module-pyautogui Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось включить тесты? Комментарии в спеке не увидел, как и в commit message, на этот счёт.
(Ответ для Антон Мидюков на комментарий #35) > Общие мелкие замечания, не требующие исправления сейчас (просто примите к > сведению). > В Summary не нужно ставить точку в конце. А в %description наоборот нужно > ставить точку в конце предложения, даже если оно всего одно или состоит из > макроса %summary. > Длина строк в %description не должна превышать 80 символов. > Да и в целом нужно стремиться к тому, чтобы строки в спеке были не длиннее > 80 символов. Принял к сведению, спасибо. > Теперь к существенному: > https://packages.altlinux.org/ru/tasks/344969/ > * python3-module-pymsgbox > * python3-module-pyscreeze > * python3-module-pytweening > * python3-module-mouseinfo > * python3-module-pyautogui > > Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось > включить тесты? Комментарии в спеке не увидел, как и в commit message, на > этот счёт. Комментарии в спек добавлял, как минимум есть в пакете python3-module-pyautogui #imposible run tests witout graphical interface это графические модули и для тестов нужна графика.
(Ответ для Pavel Shilov на комментарий #36) > (Ответ для Антон Мидюков на комментарий #35) > > Теперь к существенному: > > https://packages.altlinux.org/ru/tasks/344969/ > > * python3-module-pymsgbox > > * python3-module-pyscreeze > > * python3-module-pytweening > > * python3-module-mouseinfo > > * python3-module-pyautogui > > > > Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось > > включить тесты? Комментарии в спеке не увидел, как и в commit message, на > > этот счёт. > > Комментарии в спек добавлял, как минимум есть в пакете > python3-module-pyautogui > #imposible run tests witout graphical interface > это графические модули и для тестов нужна графика. Ладно. Идём дальше. https://git.altlinux.org/tasks/355829/gears/200/git?p=git;a=commitdiff;h=e32f51b3bf714c48581e99f40d8d1bce28309dbb Тут вижу такое: %global __find_debuginfo_files %nil %add_debuginfo_skiplist %_bindir Что за проблема возникла? Расскажите.
(Ответ для Антон Мидюков на комментарий #37) > (Ответ для Pavel Shilov на комментарий #36) > > (Ответ для Антон Мидюков на комментарий #35) > > > Теперь к существенному: > > > https://packages.altlinux.org/ru/tasks/344969/ > > > * python3-module-pymsgbox > > > * python3-module-pyscreeze > > > * python3-module-pytweening > > > * python3-module-mouseinfo > > > * python3-module-pyautogui > > > > > > Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось > > > включить тесты? Комментарии в спеке не увидел, как и в commit message, на > > > этот счёт. > > > > Комментарии в спек добавлял, как минимум есть в пакете > > python3-module-pyautogui > > #imposible run tests witout graphical interface > > это графические модули и для тестов нужна графика. > > Ладно. Идём дальше. > > https://git.altlinux.org/tasks/355829/gears/200/git?p=git;a=commitdiff; > h=e32f51b3bf714c48581e99f40d8d1bce28309dbb > > Тут вижу такое: > %global __find_debuginfo_files %nil > %add_debuginfo_skiplist %_bindir > > Что за проблема возникла? Расскажите. Возможно возникла ошибка с генерацией подпакетов с дополнительной отладочной информацией. Сейчас сказать трудно. Перепроверил и пересобрал. https://git.altlinux.org/tasks/355829/
(Ответ для Pavel Shilov на комментарий #38) > (Ответ для Антон Мидюков на комментарий #37) > > (Ответ для Pavel Shilov на комментарий #36) > > > (Ответ для Антон Мидюков на комментарий #35) > > > > Теперь к существенному: > > > > https://packages.altlinux.org/ru/tasks/344969/ > > > > * python3-module-pymsgbox > > > > * python3-module-pyscreeze > > > > * python3-module-pytweening > > > > * python3-module-mouseinfo > > > > * python3-module-pyautogui > > > > > > > > Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось > > > > включить тесты? Комментарии в спеке не увидел, как и в commit message, на > > > > этот счёт. > > > > > > Комментарии в спек добавлял, как минимум есть в пакете > > > python3-module-pyautogui > > > #imposible run tests witout graphical interface > > > это графические модули и для тестов нужна графика. > > > > Ладно. Идём дальше. > > > > https://git.altlinux.org/tasks/355829/gears/200/git?p=git;a=commitdiff; > > h=e32f51b3bf714c48581e99f40d8d1bce28309dbb > > > > Тут вижу такое: > > %global __find_debuginfo_files %nil > > %add_debuginfo_skiplist %_bindir > > > > Что за проблема возникла? Расскажите. > > Возможно возникла ошибка с генерацией подпакетов с дополнительной отладочной > информацией. Сейчас сказать трудно. Перепроверил и пересобрал. > https://git.altlinux.org/tasks/355829/ Поэтому нужно всегда писать комментарии. Тут ошибки с опакечиванием: https://git.altlinux.org/tasks/355829/gears/300/git?p=git;a=commitdiff;h=c9a0e8aa54cf33d9ff3cc1bac412811f74a2f480 Вы опакечиваете каталоги, которые принадлежат другим пакетам: %_datadir/bash-completion/ %dir %_datadir/fish/ %_datadir/fish/vendor_completions.d/ %_datadir/zsh_completion.d/ Не надо так делать. Опакечивайте только файлы этого пакета. Неопакеченные каталоги - это не ошибки, а предупреждения. Нужно понимать, почему каталог не опакечен, и принимать решение на основе этого. В данном случае нужно было игнорировать их. Также вопрос, а действительно ли нужно указывать: Requires: bash-completion ?
(Ответ для Антон Мидюков на комментарий #35) > Да и в целом нужно стремиться к тому, чтобы строки в спеке были не длиннее 80 символов. Это уже давно отменено лично Линусом.
(Ответ для Антон Мидюков на комментарий #39) > (Ответ для Pavel Shilov на комментарий #38) > > (Ответ для Антон Мидюков на комментарий #37) > > > (Ответ для Pavel Shilov на комментарий #36) > > > > (Ответ для Антон Мидюков на комментарий #35) > > > > > Теперь к существенному: > > > > > https://packages.altlinux.org/ru/tasks/344969/ > > > > > * python3-module-pymsgbox > > > > > * python3-module-pyscreeze > > > > > * python3-module-pytweening > > > > > * python3-module-mouseinfo > > > > > * python3-module-pyautogui > > > > > > > > > > Во всех этих пакетах закоммичены проверки. Расскажите, почему не получилось > > > > > включить тесты? Комментарии в спеке не увидел, как и в commit message, на > > > > > этот счёт. > > > > > > > > Комментарии в спек добавлял, как минимум есть в пакете > > > > python3-module-pyautogui > > > > #imposible run tests witout graphical interface > > > > это графические модули и для тестов нужна графика. > > > > > > Ладно. Идём дальше. > > > > > > https://git.altlinux.org/tasks/355829/gears/200/git?p=git;a=commitdiff; > > > h=e32f51b3bf714c48581e99f40d8d1bce28309dbb > > > > > > Тут вижу такое: > > > %global __find_debuginfo_files %nil > > > %add_debuginfo_skiplist %_bindir > > > > > > Что за проблема возникла? Расскажите. > > > > Возможно возникла ошибка с генерацией подпакетов с дополнительной отладочной > > информацией. Сейчас сказать трудно. Перепроверил и пересобрал. > > https://git.altlinux.org/tasks/355829/ > > Поэтому нужно всегда писать комментарии. > > Тут ошибки с опакечиванием: > https://git.altlinux.org/tasks/355829/gears/300/git?p=git;a=commitdiff; > h=c9a0e8aa54cf33d9ff3cc1bac412811f74a2f480 > > Вы опакечиваете каталоги, которые принадлежат другим пакетам: > %_datadir/bash-completion/ > %dir %_datadir/fish/ > %_datadir/fish/vendor_completions.d/ > %_datadir/zsh_completion.d/ > > Не надо так делать. Опакечивайте только файлы этого пакета. > Неопакеченные каталоги - это не ошибки, а предупреждения. Нужно понимать, > почему каталог не опакечен, и принимать решение на основе этого. В данном > случае нужно было игнорировать их. > > Также вопрос, а действительно ли нужно указывать: > Requires: bash-completion > ? Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. Убрал зависимость от bash-completion https://git.altlinux.org/tasks/355829/
(Ответ для Pavel Shilov на комментарий #41) > Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. > Убрал зависимость от bash-completion > https://git.altlinux.org/tasks/355829/ Одобрил. Идём дальше. https://git.altlinux.org/tasks/356102/gears/200/git?p=git;a=commitdiff;h=81870913e3ad19840ce3caead095787f9a132ff4 Нет необходимости добавлять gcc и automake в BuildRequires, так как они всегда есть в hasher. Зачем удаляете /usr/share/info? %autoreconf вместо ./autogen не справится? https://git.altlinux.org/tasks/353656/gears/200/git?p=git;a=commitdiff;h=8bf40e75c72330f684c7fbaaf3c146994ca15231 А тут через чур длинное Summary https://git.altlinux.org/tasks/352936/gears/200/git?p=git;a=commitdiff;h=4cff6f877afbf1f698562193cca8d0405d1763ed По ошибке добавили вместо спека его резервную копию dsd-fme.spec~ И там же два коммита с одинаковым описанием.
(Ответ для Антон Мидюков на комментарий #42) > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > всегда есть в hasher. В таком случае надо добавлять hasher в BuildRequires. Или локальная сборка без hasher у нас уже игнорируется?
(Ответ для Sergey V Turchin на комментарий #43) > (Ответ для Антон Мидюков на комментарий #42) > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > > всегда есть в hasher. > В таком случае надо добавлять hasher в BuildRequires. > Или локальная сборка без hasher у нас уже игнорируется? Я не требую это исправлять. Просто к сведению.
(Ответ для Антон Мидюков на комментарий #44) > Я не требую это исправлять. Просто к сведению. Если к сведению, то готовить пакет удобнее в виртуалке без всяких hasher при помощи gear-rpm, что я, например, всегда и делаю.
(Ответ для Антон Мидюков на комментарий #42) > (Ответ для Pavel Shilov на комментарий #41) > > Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. > > Убрал зависимость от bash-completion > > https://git.altlinux.org/tasks/355829/ > > Одобрил. Идём дальше. > > https://git.altlinux.org/tasks/356102/gears/200/git?p=git;a=commitdiff; > h=81870913e3ad19840ce3caead095787f9a132ff4 > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > всегда есть в hasher. > Зачем удаляете /usr/share/info? Удалил потому, что возникает ошибка со сборкой netmask.info.xz > %autoreconf вместо ./autogen не справится? С данным пакетом разбираюсь, %autoreconf - справляется. > https://git.altlinux.org/tasks/353656/gears/200/git?p=git;a=commitdiff; > h=8bf40e75c72330f684c7fbaaf3c146994ca15231 > > А тут через чур длинное Summary Исправил https://git.altlinux.org/tasks/353656/ > https://git.altlinux.org/tasks/352936/gears/200/git?p=git;a=commitdiff; > h=4cff6f877afbf1f698562193cca8d0405d1763ed > > По ошибке добавили вместо спека его резервную копию dsd-fme.spec~ > И там же два коммита с одинаковым описанием. Исправил https://git.altlinux.org/tasks/352936/
(In reply to Sergey V Turchin from comment #43) > (Ответ для Антон Мидюков на комментарий #42) > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > > всегда есть в hasher. > В таком случае надо добавлять hasher в BuildRequires. > Или локальная сборка без hasher у нас уже игнорируется? Я думаю, Антон имел в виду rpm-build, а не hasher.
(Ответ для Dmitry V. Levin на комментарий #47) > Я думаю, Антон имел в виду rpm-build, а не hasher. Да, видимо. Да и мне слегка почудился gcc-c++. По крайней мере, это не влекло за собой никаких проблем последние пару десятков лет. С большой вероятностью ещё пару не будет.
(Ответ для Pavel Shilov на комментарий #46) > (Ответ для Антон Мидюков на комментарий #42) > > (Ответ для Pavel Shilov на комментарий #41) > > > Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. > > > Убрал зависимость от bash-completion > > > https://git.altlinux.org/tasks/355829/ > > > > Одобрил. Идём дальше. > > > > https://git.altlinux.org/tasks/356102/gears/200/git?p=git;a=commitdiff; > > h=81870913e3ad19840ce3caead095787f9a132ff4 > > > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > > всегда есть в hasher. > > Зачем удаляете /usr/share/info? > > Удалил потому, что возникает ошибка со сборкой netmask.info.xz > > > %autoreconf вместо ./autogen не справится? > > С данным пакетом разбираюсь, %autoreconf - справляется. > Исправил https://git.altlinux.org/tasks/356102/ /usr/share/info пакуется.
(Ответ для Pavel Shilov на комментарий #49) > (Ответ для Pavel Shilov на комментарий #46) > > (Ответ для Антон Мидюков на комментарий #42) > > > (Ответ для Pavel Shilov на комментарий #41) > > > > Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. > > > > Убрал зависимость от bash-completion > > > > https://git.altlinux.org/tasks/355829/ > > > > > > Одобрил. Идём дальше. > > > > > > https://git.altlinux.org/tasks/356102/gears/200/git?p=git;a=commitdiff; > > > h=81870913e3ad19840ce3caead095787f9a132ff4 > > > > > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > > > всегда есть в hasher. > > > Зачем удаляете /usr/share/info? > > > > Удалил потому, что возникает ошибка со сборкой netmask.info.xz > > > > > %autoreconf вместо ./autogen не справится? > > > > С данным пакетом разбираюсь, %autoreconf - справляется. > > > > Исправил https://git.altlinux.org/tasks/356102/ > > /usr/share/info пакуется. %_infodir/%name.info.xz Не надо указывать явно xz, метод сжатия может и поменяться. Поэтому: %_infodir/%name.info.* И, как тут выше писали, уберите BuildRequires: gcc BuildRequires: automake Так как их вытягивает rpm-build. Раз всё равно переделывать, укажите причину отключения проверки в комментарии над %set_verify_info_method relaxed
(Ответ для Антон Мидюков на комментарий #50) > (Ответ для Pavel Shilov на комментарий #49) > > (Ответ для Pavel Shilov на комментарий #46) > > > (Ответ для Антон Мидюков на комментарий #42) > > > > (Ответ для Pavel Shilov на комментарий #41) > > > > > Поторопился, был не прав. Исправил ошибку с опакечиванием каталогов. > > > > > Убрал зависимость от bash-completion > > > > > https://git.altlinux.org/tasks/355829/ > > > > > > > > Одобрил. Идём дальше. > > > > > > > > https://git.altlinux.org/tasks/356102/gears/200/git?p=git;a=commitdiff; > > > > h=81870913e3ad19840ce3caead095787f9a132ff4 > > > > > > > > Нет необходимости добавлять gcc и automake в BuildRequires, так как они > > > > всегда есть в hasher. > > > > Зачем удаляете /usr/share/info? > > > > > > Удалил потому, что возникает ошибка со сборкой netmask.info.xz > > > > > > > %autoreconf вместо ./autogen не справится? > > > > > > С данным пакетом разбираюсь, %autoreconf - справляется. > > > > > > > Исправил https://git.altlinux.org/tasks/356102/ > > > > /usr/share/info пакуется. > > %_infodir/%name.info.xz > > Не надо указывать явно xz, метод сжатия может и поменяться. Поэтому: > %_infodir/%name.info.* > > И, как тут выше писали, уберите > BuildRequires: gcc > BuildRequires: automake > > Так как их вытягивает rpm-build. > > Раз всё равно переделывать, укажите причину отключения проверки в > комментарии над > %set_verify_info_method relaxed Готово https://git.altlinux.org/tasks/356102/
(Ответ для Pavel Shilov на комментарий #51) [...] > Готово https://git.altlinux.org/tasks/356102/ Запись в чейнджлог не совпадает с сутью нового релиза alt2 (к тому же допустили опечатку fot вместо for). Вы написали: - change verify method fot info file Но в этом релизе вы делаете другие вещи (verify method назначили ещё в alt1): - Убрали точку из summary - убрали излишние BuldRequires - убрали явное указание xz при упаковке info файла Предлагаю не делать release alt2, а всё сделать в alt1. Чтобы это всё малозначительное в changelog не писать. Только дату в changelog обновите на текущую.
(Ответ для Антон Мидюков на комментарий #52) > (Ответ для Pavel Shilov на комментарий #51) > [...] > > Готово https://git.altlinux.org/tasks/356102/ > > Запись в чейнджлог не совпадает с сутью нового релиза alt2 (к тому же > допустили опечатку fot вместо for). > Вы написали: > - change verify method fot info file > > Но в этом релизе вы делаете другие вещи (verify method назначили ещё в alt1): > - Убрали точку из summary > - убрали излишние BuldRequires > - убрали явное указание xz при упаковке info файла > > Предлагаю не делать release alt2, а всё сделать в alt1. Чтобы это всё > малозначительное в changelog не писать. Только дату в changelog обновите на > текущую. Извиняюсь, не подумал. Все поправил и пересобрал. https://git.altlinux.org/tasks/356102/
Пересобрал исходя из вышеуказанных комментариев https://git.altlinux.org/tasks/357636/ * systeroid * dasel
(Ответ для Pavel Shilov на комментарий #54) > Пересобрал исходя из вышеуказанных комментариев > https://git.altlinux.org/tasks/357636/ > * systeroid > * dasel Поставили точку в Summary. Но да ладно. Напомню, что не стоит так делать. Тем не менее одобрил, так как это мелочь.
Идём дальше. https://git.altlinux.org/tasks/353656/gears/300/git?p=git;a=commitdiff;h=a8b582a53109b21296aee597d40b0da0ddfe577c Тут замечания такие: 1. В BuildRequires не требуется указывать gettext и libtool, так как это зависимости rpn-build 2. У вас прописано %autoreconf -ivf Но эти опции '-ifv' и так выставляются макросом %autoreconf: $ rpm --eval %autoreconf autoreconf -fisv Не надо их указывать.
(Ответ для Антон Мидюков на комментарий #56) > Идём дальше. > https://git.altlinux.org/tasks/353656/gears/300/git?p=git;a=commitdiff; > h=a8b582a53109b21296aee597d40b0da0ddfe577c > > Тут замечания такие: > 1. В BuildRequires не требуется указывать gettext и libtool, так как это > зависимости rpn-build > 2. У вас прописано > %autoreconf -ivf > Но эти опции '-ifv' и так выставляются макросом %autoreconf: > $ rpm --eval %autoreconf > autoreconf -fisv > Не надо их указывать. Принято к сведению, пересобрал с учетом замечаний: https://git.altlinux.org/tasks/353656/
(Ответ для Pavel Shilov на комментарий #57) > Принято к сведению, пересобрал с учетом замечаний: > https://git.altlinux.org/tasks/353656/ Одобрил. Идём дальше. Смотрю https://git.altlinux.org/tasks/352936/gears/400/git?p=git;a=blob;f=.gear/dsd-fme.spec;h=c9a58b2a5a69efbcaea1fca9d196293abd62d741;hb=d96971d6c5b73a9a08e640a696b98d3d293c57f5 Зачем потребовалось делать export CFLAGS="%optflags" перед %cmake? Через параметр -DCMAKE_C_FLAGS:STRING='%optflags' они не передаются (смотреть значение макроса %cmake)?
(Ответ для Pavel Shilov на комментарий #57) > Принято к сведению Не совсем. Антон уже указал, что макросы перед использованием полезно раскрывать, чтоб хоть посмотреть, что они делают на самом деле. $ rpm --eval %cmake
(Ответ для Антон Мидюков на комментарий #58) > (Ответ для Pavel Shilov на комментарий #57) > > Принято к сведению, пересобрал с учетом замечаний: > > https://git.altlinux.org/tasks/353656/ > > Одобрил. Идём дальше. > Смотрю > https://git.altlinux.org/tasks/352936/gears/400/git?p=git;a=blob;f=.gear/dsd- > fme.spec;h=c9a58b2a5a69efbcaea1fca9d196293abd62d741; > hb=d96971d6c5b73a9a08e640a696b98d3d293c57f5 > > Зачем потребовалось делать > export CFLAGS="%optflags" > перед %cmake? > Через параметр -DCMAKE_C_FLAGS:STRING='%optflags' они не передаются > (смотреть значение макроса %cmake)? Исправлено https://git.altlinux.org/tasks/352936/ $ rpm --eval %cmake mkdir -p x86_64-alt-linux-gnu; cmake \ -DCMAKE_SKIP_INSTALL_RPATH:BOOL=yes \ -DCMAKE_C_FLAGS:STRING='-O2 -g' \ -DCMAKE_CXX_FLAGS:STRING='-O2 -g' \ -DCMAKE_Fortran_FLAGS:STRING='-O2 -g' \ -DCMAKE_INSTALL_PREFIX=/usr \ -DINCLUDE_INSTALL_DIR:PATH=/usr/include \ -DLIB_INSTALL_DIR:PATH=/usr/lib64 \ -DSYSCONF_INSTALL_DIR:PATH=/etc \ -DSHARE_INSTALL_PREFIX:PATH=/usr/share \ -DLIB_DESTINATION=lib64 \ -DLIB_SUFFIX="64" \ -S . -B "x86_64-alt-linux-gnu"
(Ответ для Pavel Shilov на комментарий #60) > (Ответ для Антон Мидюков на комментарий #58) > > (Ответ для Pavel Shilov на комментарий #57) > > > Принято к сведению, пересобрал с учетом замечаний: > > > https://git.altlinux.org/tasks/353656/ > > > > Одобрил. Идём дальше. > > Смотрю > > https://git.altlinux.org/tasks/352936/gears/400/git?p=git;a=blob;f=.gear/dsd- > > fme.spec;h=c9a58b2a5a69efbcaea1fca9d196293abd62d741; > > hb=d96971d6c5b73a9a08e640a696b98d3d293c57f5 > > > > Зачем потребовалось делать > > export CFLAGS="%optflags" > > перед %cmake? > > Через параметр -DCMAKE_C_FLAGS:STRING='%optflags' они не передаются > > (смотреть значение макроса %cmake)? > > Исправлено https://git.altlinux.org/tasks/352936/ > > $ rpm --eval %cmake > > mkdir -p x86_64-alt-linux-gnu; > cmake \ > -DCMAKE_SKIP_INSTALL_RPATH:BOOL=yes \ > -DCMAKE_C_FLAGS:STRING='-O2 -g' \ > -DCMAKE_CXX_FLAGS:STRING='-O2 -g' \ > -DCMAKE_Fortran_FLAGS:STRING='-O2 -g' \ > -DCMAKE_INSTALL_PREFIX=/usr \ > -DINCLUDE_INSTALL_DIR:PATH=/usr/include \ > -DLIB_INSTALL_DIR:PATH=/usr/lib64 \ > -DSYSCONF_INSTALL_DIR:PATH=/etc \ > -DSHARE_INSTALL_PREFIX:PATH=/usr/share \ > -DLIB_DESTINATION=lib64 \ > -DLIB_SUFFIX="64" \ > -S . -B "x86_64-alt-linux-gnu" Смотрю на https://git.altlinux.org/tasks/352936/gears/500/git?p=git;a=commitdiff;h=643fc59130dfbb253027234a33d0987a6cb48773 Что-то вы странное сделали. Зачем добавили -DCMAKE_C_FLAGS:STRING='%optflags', когда он уже был (скрывался в макросе %cmake)? И вы не отразили изменение в Changelog спека.
(Ответ для Антон Мидюков на комментарий #61) > (Ответ для Pavel Shilov на комментарий #60) > > (Ответ для Антон Мидюков на комментарий #58) > > > (Ответ для Pavel Shilov на комментарий #57) > > > > Принято к сведению, пересобрал с учетом замечаний: > > > > https://git.altlinux.org/tasks/353656/ > > > > > > Одобрил. Идём дальше. > > > Смотрю > > > https://git.altlinux.org/tasks/352936/gears/400/git?p=git;a=blob;f=.gear/dsd- > > > fme.spec;h=c9a58b2a5a69efbcaea1fca9d196293abd62d741; > > > hb=d96971d6c5b73a9a08e640a696b98d3d293c57f5 > > > > > > Зачем потребовалось делать > > > export CFLAGS="%optflags" > > > перед %cmake? > > > Через параметр -DCMAKE_C_FLAGS:STRING='%optflags' они не передаются > > > (смотреть значение макроса %cmake)? > > > > Исправлено https://git.altlinux.org/tasks/352936/ > > > > $ rpm --eval %cmake > > > > mkdir -p x86_64-alt-linux-gnu; > > cmake \ > > -DCMAKE_SKIP_INSTALL_RPATH:BOOL=yes \ > > -DCMAKE_C_FLAGS:STRING='-O2 -g' \ > > -DCMAKE_CXX_FLAGS:STRING='-O2 -g' \ > > -DCMAKE_Fortran_FLAGS:STRING='-O2 -g' \ > > -DCMAKE_INSTALL_PREFIX=/usr \ > > -DINCLUDE_INSTALL_DIR:PATH=/usr/include \ > > -DLIB_INSTALL_DIR:PATH=/usr/lib64 \ > > -DSYSCONF_INSTALL_DIR:PATH=/etc \ > > -DSHARE_INSTALL_PREFIX:PATH=/usr/share \ > > -DLIB_DESTINATION=lib64 \ > > -DLIB_SUFFIX="64" \ > > -S . -B "x86_64-alt-linux-gnu" > > Смотрю на > https://git.altlinux.org/tasks/352936/gears/500/git?p=git;a=commitdiff; > h=643fc59130dfbb253027234a33d0987a6cb48773 > > Что-то вы странное сделали. Зачем добавили > -DCMAKE_C_FLAGS:STRING='%optflags', когда он уже был (скрывался в макросе > %cmake)? > И вы не отразили изменение в Changelog спека. Виноват, не разобрался с флагами компиляции. Поправил changelog и удалил экспорт.
Считаю, что кандидат теперь доучился и готов к самостоятельной сборке пакетов.
Пользователь добавлен в группу мейнтейнеров. Желаю удачного мейнтейнерства!
(Ответ для Gleb F-Malinovskiy на комментарий #64) > Пользователь добавлен в группу мейнтейнеров. > > Желаю удачного мейнтейнерства! Спасибо
