Bug 40829
| Summary: | [3.6] join sav@ | ||||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Product: | Team Accounts | Reporter: | Aleksey Saprunov <sa> | ||||||||||||
| Component: | join | Assignee: | Gleb F-Malinovskiy <glebfm> | ||||||||||||
| Status: | ASSIGNED --- | QA Contact: | Andrey Cherepanov <cas> | ||||||||||||
| Severity: | normal | ||||||||||||||
| Priority: | P5 | CC: | enp, glebfm, iv, ldv, nir, nir, rider, savoptik | ||||||||||||
| Version: | unspecified | ||||||||||||||
| Hardware: | x86_64 | ||||||||||||||
| OS: | Linux | ||||||||||||||
| Attachments: |
|
||||||||||||||
Created attachment 9629 [details]
Публичный
Менторство подтверждаю, кандидат будет собирать ПО для ПЛК: modbus-utils. Добрый день. Собран пакет, modbus-utils форк - https://github.com/AlexSP0/modbus-utils-1 Создан и принят пулл реквест https://github.com/Krzysztow/modbus-utils. https://github.com/AlexSP0/modbus-utils-1/tree/1.0.0-alt1 - тег посмотрел,(Ответ для Aleksey Saprunov на комментарий #3) > Добрый день. Собран пакет, modbus-utils > форк - https://github.com/AlexSP0/modbus-utils-1 > > Создан и принят пулл реквест https://github.com/Krzysztow/modbus-utils. Проверил тег: https://github.com/AlexSP0/modbus-utils-1/tree/1.0.0-alt1 . Собирается в hasher. Претензий к spefile не имею. (In reply to Aleksey Saprunov from comment #0) > Created attachment 9628 [details] > Публичный ключ > > псевдоним - SA К сожалению, этот ник уже использовался другим членом Team, к тому же нынешняя процедура предполагает использование ников не менее 3 символов. (Ответ для Gleb F-Malinovskiy на комментарий #5) > (In reply to Aleksey Saprunov from comment #0) > > Created attachment 9628 [details] [подробности] [details] > > Публичный ключ > > > > псевдоним - SA > > К сожалению, этот ник уже использовался другим членом Team, к тому же > нынешняя процедура предполагает использование ников не менее 3 символов. Добрый вечер! Ник sav свободен? (In reply to Aleksey Saprunov from comment #6) > (Ответ для Gleb F-Malinovskiy на комментарий #5) > > (In reply to Aleksey Saprunov from comment #0) > > > Created attachment 9628 [details] [подробности] [details] > > > Публичный ключ > > > > > > псевдоним - SA > > > > К сожалению, этот ник уже использовался другим членом Team, к тому же > > нынешняя процедура предполагает использование ников не менее 3 символов. > > Добрый вечер! Ник sav свободен? Насколько я вижу, да. Добрый день. Собран пакет qtl866 форк - https://github.com/AlexSP0/qtl866/tree/alt_release создан, но пока висит пуллреквест: https://github.com/wd5gnr/qtl866/pull/14 Добрый день. Я проверил и особых замечаний не имею, кроме того, что стоит последний тегированый коммит сделать с помощью gear-commit, чтобы была соответствующая запись в истории. В целом, в hasher оно собирается и устанавливается у меня вполне корректно. (Ответ для Igor Chudov на комментарий #9) > Добрый день. > > Я проверил и особых замечаний не имею, кроме того, что стоит последний > тегированый коммит сделать с помощью gear-commit, чтобы была соответствующая > запись в истории. В целом, в hasher оно собирается и устанавливается у меня > вполне корректно. поправил. Добрый день. Проверил. С моей стороны замечаний нет. Предлагаю продвинуть задачу дальше. Created attachment 9774 [details] Ключ с новой почтой sav@altlinux.org Created attachment 9775 [details]
Ключ с новым ником sav
Обновил ключи для sav@altlinux.org (In reply to Aleksey Saprunov from comment #13) > Created attachment 9775 [details] > Ключ с новым ником sav Ok. (In reply to Aleksey Saprunov from comment #12) > Created attachment 9774 [details] > Ключ с новой почтой sav@altlinux.org "... лучше, у меня есть картинка ключа!" В этом файле -- хэш ключа, но поскольку это тот же ключ, что был раньше, а комментарий не очень важен. Ok. Добрый день. Давайте продвинем кандидата дальше по процедуре. Увы, почта так и не была подтверждена. псевдоним - sav форвард почты с sa@altlunux.org на spbmbm@gmail.com ментор - nir@ Добрый день! Исправил почту на spbmbm@gmail.com Добрый день. На данный момент форвард почты не настроен. Я предлагаю зафиксировать реквизиты для отправки в неизменном виде. Глеб, какой адрес почты должен подтвердить кандидат? Что от него требуется? ssh ключ на gitery.alt зарегистрирован. Адрес для пересылки создан. T/J/S -> 2.3. Добрый день, коллеги. Я вроде бы уже проверял пакет Алексея и на тот момент он соответствовал требованиям. specfile был оформлен корректно, сборка производилась в Hasher. Предлагаю продвинуть Алексея по процедуре и дать ему доступ к сборке. Базовые действия он изучил, остальное аккуратно освоит. Обновляю статус коллеги, так как считаю, что он достаточно квалифицирован для работы над пакетами. ssh ключ на gyle.alt зарегистрирован. Пакет alt-gpgkeys обновлён. T/J/S -> 3.5. Created attachment 12317 [details] Новый ключ sav@altlinux.org Прошу заменить ssh ключ(новый файл во вложении - https://bugzilla.altlinux.org/attachment.cgi?id=12317) К сожалению, был приложен не тот файл. Заранее спасибо. (In reply to Aleksey Saprunov from comment #25) > Прошу заменить ssh ключ(новый файл во вложении - > https://bugzilla.altlinux.org/attachment.cgi?id=12317) К сожалению, был > приложен не тот файл. Заранее спасибо. Вроде бы, этот же ключ и был с самого начала, у него только комментарий в процессе поменялся, но комментарий ни на что не влияет, конечно. В промежуточном вложении только был хэш ключа («лучше, у меня есть картинка ключа»), но того же самого. Репозиторий https://git.altlinux.org/people/sav/packages/?p=libmodbus-utils.git;a=summary и спеку посмотрел. Задачу 313724 посмотрел. Замечаний не имею. Надо приглашать кросс-ревьювера. (In reply to Igor Chudov from comment #27) > Репозиторий > https://git.altlinux.org/people/sav/packages/?p=libmodbus-utils.git; > a=summary и спеку посмотрел. > Задачу 313724 посмотрел. Напоминаю, что ментор может заапрувить задание когда считает, что оно готово для Сизифа. > Замечаний не имею. Надо приглашать кросс-ревьювера. Мне кажется, что любой рецензент скажет, что одного пакета мало. Вы уверены, что этого пакета будет достаточно? Собрал qtl866: https://git.altlinux.org/people/sav/packages/?p=qtl866.git Также собирал admc с некоторыми новыми фичами для ревью: https://git.altlinux.org/people/sav/packages/?p=admc.git Для qtl866 задача 317707, для admc задача 315344 Задачи для qtl866 и modbus-utils ушли в Sisyphus. Это очень положительная новость, так как эти инструменты позволяют работать с оборудованием для промышленной автоматизации. С ADMC у меня вопросы скорее к руководителям проекта. Нужна ли эта сборка сейчас или нет. rider@ согласился быть рецензентом, прошу секретаря подтвердить выбор кандидатуры. ping Gleb. Призван рецензент (rider@) для независимой оценки готовности кандидата. T/J/S -> 4.2. (Ответ для Aleksey Saprunov на комментарий #29) > Собрал qtl866: > https://git.altlinux.org/people/sav/packages/?p=qtl866.git а откуда взялась версия 1.0.0 ? не могу найти её в апстриме. https://git.altlinux.org/people/sav/packages/?p=alt-diagnostic-tool.git;a=blob;f=.gear/alt-diagnostic-tool.spec;h=cf2cb4e79bb36c79d26ad2e26e3d76bbb50d75af;hb=8faa7d84a978c6f2671b38a2e2a6de0af3f3b1cd Наша разработка, поэтому тут всё проще: - установку desktop файла лучше перенести в cmake. - по URL на github не выложен тэг 0.1.1 (или v0.1.1) и это может повлиять на поведение части скриптов, определяющих версии в апстримных проектах. Нет нормального description https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git;a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 группа выбрана не самым удачным образом секции build, install и files лучше разделять пустой строкой для облегчения читаемости. Версии 1.0.0 не существует (или я не нашёл). И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/ - смотрите что бы ACL позволял вносить исправления для everybody или nobody. (Ответ для Anton Farygin на комментарий #34) > (Ответ для Aleksey Saprunov на комментарий #29) > > Собрал qtl866: > > https://git.altlinux.org/people/sav/packages/?p=qtl866.git > > а откуда взялась версия 1.0.0 ? не могу найти её в апстриме. Прошу прощения за длительный ответ. Такой версии в апстриме нет. Исправил на 0.1.0 (Ответ для Anton Farygin на комментарий #36) > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > группа выбрана не самым удачным образом > секции build, install и files лучше разделять пустой строкой для облегчения > читаемости. > > Версии 1.0.0 не существует (или я не нашёл). (Ответ для Anton Farygin на комментарий #35) > https://git.altlinux.org/people/sav/packages/?p=alt-diagnostic-tool.git; > a=blob;f=.gear/alt-diagnostic-tool.spec; > h=cf2cb4e79bb36c79d26ad2e26e3d76bbb50d75af; > hb=8faa7d84a978c6f2671b38a2e2a6de0af3f3b1cd > > Наша разработка, поэтому тут всё проще: > - установку desktop файла лучше перенести в cmake. > - по URL на github не выложен тэг 0.1.1 (или v0.1.1) и это может повлиять на > поведение части скриптов, определяющих версии в апстримных проектах. > Нет нормального description Понял, в течение ближайщего времени исправлю. (Ответ для Anton Farygin на комментарий #36) > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > группа выбрана не самым удачным образом > секции build, install и files лучше разделять пустой строкой для облегчения > читаемости. > > Версии 1.0.0 не существует (или я не нашёл). В ближайшее время исправлю (Ответ для Anton Farygin на комментарий #37) > И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого > списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/ > - смотрите что бы ACL позволял вносить исправления для everybody или nobody. Хорошо, после исправления выбранных пакетов, соберу. (Ответ для Anton Farygin на комментарий #35) > https://git.altlinux.org/people/sav/packages/?p=alt-diagnostic-tool.git; > a=blob;f=.gear/alt-diagnostic-tool.spec; > h=cf2cb4e79bb36c79d26ad2e26e3d76bbb50d75af; > hb=8faa7d84a978c6f2671b38a2e2a6de0af3f3b1cd > > Наша разработка, поэтому тут всё проще: > - установку desktop файла лучше перенести в cmake. > - по URL на github не выложен тэг 0.1.1 (или v0.1.1) и это может повлиять на > поведение части скриптов, определяющих версии в апстримных проектах. > Нет нормального description Поправил. Номер задачи #323821 (Ответ для Aleksey Saprunov на комментарий #38) > (Ответ для Anton Farygin на комментарий #34) > > (Ответ для Aleksey Saprunov на комментарий #29) > > > Собрал qtl866: > > > https://git.altlinux.org/people/sav/packages/?p=qtl866.git > > > > а откуда взялась версия 1.0.0 ? не могу найти её в апстриме. > > Прошу прощения за длительный ответ. Такой версии в апстриме нет. Исправил на > 0.1.0 А эта откуда взялась ? можно мне ссылку ? (Ответ для Aleksey Saprunov на комментарий #42) > (Ответ для Anton Farygin на комментарий #35) > > https://git.altlinux.org/people/sav/packages/?p=alt-diagnostic-tool.git; > > a=blob;f=.gear/alt-diagnostic-tool.spec; > > h=cf2cb4e79bb36c79d26ad2e26e3d76bbb50d75af; > > hb=8faa7d84a978c6f2671b38a2e2a6de0af3f3b1cd > > > > Наша разработка, поэтому тут всё проще: > > - установку desktop файла лучше перенести в cmake. > > - по URL на github не выложен тэг 0.1.1 (или v0.1.1) и это может повлиять на > > поведение части скриптов, определяющих версии в апстримных проектах. > > Нет нормального description > > Поправил. Номер задачи #323821 ok. и ещё - лучше стараться не коммитить в репозиторий спек-файлы с оставленными временными комментариями. https://git.altlinux.org/tasks/323821/gears/1600/git?p=git;a=blob;f=alterator-module-executor.spec;h=8cfcddb12d0591905e80996111c9181339a23e81;hb=4a49081e6a557a7dfd65b33d255f6b6be9b5ace3 (Ответ для Anton Farygin на комментарий #44) > (Ответ для Aleksey Saprunov на комментарий #42) > > (Ответ для Anton Farygin на комментарий #35) > > > https://git.altlinux.org/people/sav/packages/?p=alt-diagnostic-tool.git; > > > a=blob;f=.gear/alt-diagnostic-tool.spec; > > > h=cf2cb4e79bb36c79d26ad2e26e3d76bbb50d75af; > > > hb=8faa7d84a978c6f2671b38a2e2a6de0af3f3b1cd > > > > > > Наша разработка, поэтому тут всё проще: > > > - установку desktop файла лучше перенести в cmake. > > > - по URL на github не выложен тэг 0.1.1 (или v0.1.1) и это может повлиять на > > > поведение части скриптов, определяющих версии в апстримных проектах. > > > Нет нормального description > > > > Поправил. Номер задачи #323821 > > ok. > > и ещё - лучше стараться не коммитить в репозиторий спек-файлы с оставленными > временными комментариями. > > https://git.altlinux.org/tasks/323821/gears/1600/git?p=git;a=blob; > f=alterator-module-executor.spec;h=8cfcddb12d0591905e80996111c9181339a23e81; > hb=4a49081e6a557a7dfd65b33d255f6b6be9b5ace3 Понял, спасибо. (Ответ для Anton Farygin на комментарий #43) > (Ответ для Aleksey Saprunov на комментарий #38) > > (Ответ для Anton Farygin на комментарий #34) > > > (Ответ для Aleksey Saprunov на комментарий #29) > > > > Собрал qtl866: > > > > https://git.altlinux.org/people/sav/packages/?p=qtl866.git > > > > > > а откуда взялась версия 1.0.0 ? не могу найти её в апстриме. > > > > Прошу прощения за длительный ответ. Такой версии в апстриме нет. Исправил на > > 0.1.0 > > А эта откуда взялась ? можно мне ссылку ? Тегов в апстриме нет, версию не нашел нигде. Но, есть коммит: https://github.com/wd5gnr/qtl866/commit/f3627d1e35ae5c9d6a2bc1f466e30ba61aeef3a0 В его описании указана версия 0.1, поэтому решил исправить версию на 0.1.0 Если апстрим не выпускал версии, то используйте версию 0 (Ответ для Anton Farygin на комментарий #47) > Если апстрим не выпускал версии, то используйте версию 0 Попробовал собрать с версией 0. Однако, ментор протолкнул пакет в сизиф с версией 1.0: https://git.altlinux.org/tasks/322172/logs/events.3.1.log Предполагаю, что надо поднять версию до 1.0.0-alt2? (Ответ для Aleksey Saprunov на комментарий #48) > (Ответ для Anton Farygin на комментарий #47) > > Если апстрим не выпускал версии, то используйте версию 0 > > Попробовал собрать с версией 0. Однако, ментор протолкнул пакет в сизиф с > версией 1.0: > > https://git.altlinux.org/tasks/322172/logs/events.3.1.log > > Предполагаю, что надо поднять версию до 1.0.0-alt2? Невнимателен. Это другой пакет с задачей 322172. Пересоберу задачу 322174 с версией 0. (Ответ для Anton Farygin на комментарий #47) > Если апстрим не выпускал версии, то используйте версию 0 Пересобрал с версией 0. Задача 322174. Важно при такой версии обязательно указывать git commit в release. Почитайте на www.altlinux.org как это делать, или посмотрите в других пакетах. (Ответ для Anton Farygin на комментарий #51) > Важно при такой версии обязательно указывать git commit в release. Почитайте > на www.altlinux.org как это делать, или посмотрите в других пакетах. Поправил. Задача 322174 (Ответ для Anton Farygin на комментарий #36) > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > группа выбрана не самым удачным образом > секции build, install и files лучше разделять пустой строкой для облегчения > читаемости. > > Версии 1.0.0 не существует (или я не нашёл). Поправил. Версия взята из файла CMakeLists.txt : https://github.com/Krzysztow/modbus-utils/blob/master/CMakeLists.txt (Ответ для Aleksey Saprunov на комментарий #52) > (Ответ для Anton Farygin на комментарий #51) > > Важно при такой версии обязательно указывать git commit в release. Почитайте > > на www.altlinux.org как это делать, или посмотрите в других пакетах. > > Поправил. Задача 322174 Тут в коммите случайно спек-файл заехал, надо разделить: https://git.altlinux.org/tasks/322174/gears/700/git?p=git;a=commitdiff;h=7aafb2795c43c2124519fe2b3dd8182f86787aff (Ответ для Aleksey Saprunov на комментарий #53) > > (Ответ для Anton Farygin на комментарий #36) > > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > > > группа выбрана не самым удачным образом > > секции build, install и files лучше разделять пустой строкой для облегчения > > читаемости. > > > > Версии 1.0.0 не существует (или я не нашёл). > > Поправил. Версия взята из файла CMakeLists.txt : > > https://github.com/Krzysztow/modbus-utils/blob/master/CMakeLists.txt Во многих проектах случается такая невыпущенная версия, которую просто написали в CMakeList. Но при этом проект не стоит на месте и идёт вперёд, не обновляя версию в Cmake. Предлагаю сейчас сделать версию alt2.git59991e07 и попросить апстрим повесить тэг на версию 1.0.0. (Ответ для Anton Farygin на комментарий #54) > (Ответ для Aleksey Saprunov на комментарий #52) > > (Ответ для Anton Farygin на комментарий #51) > > > Важно при такой версии обязательно указывать git commit в release. Почитайте > > > на www.altlinux.org как это делать, или посмотрите в других пакетах. > > > > Поправил. Задача 322174 > > Тут в коммите случайно спек-файл заехал, надо разделить: > https://git.altlinux.org/tasks/322174/gears/700/git?p=git;a=commitdiff; > h=7aafb2795c43c2124519fe2b3dd8182f86787aff Принято, поправлю. (Ответ для Anton Farygin на комментарий #55) > (Ответ для Aleksey Saprunov на комментарий #53) > > > > (Ответ для Anton Farygin на комментарий #36) > > > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > > > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > > > > > группа выбрана не самым удачным образом > > > секции build, install и files лучше разделять пустой строкой для облегчения > > > читаемости. > > > > > > Версии 1.0.0 не существует (или я не нашёл). > > > > Поправил. Версия взята из файла CMakeLists.txt : > > > > https://github.com/Krzysztow/modbus-utils/blob/master/CMakeLists.txt > > Во многих проектах случается такая невыпущенная версия, которую просто > написали в CMakeList. Но при этом проект не стоит на месте и идёт вперёд, не > обновляя версию в Cmake. > > Предлагаю сейчас сделать версию alt2.git59991e07 и попросить апстрим > повесить тэг на версию 1.0.0. Хорошо, сделаю. (Ответ для Anton Farygin на комментарий #54) > (Ответ для Aleksey Saprunov на комментарий #52) > > (Ответ для Anton Farygin на комментарий #51) > > > Важно при такой версии обязательно указывать git commit в release. Почитайте > > > на www.altlinux.org как это делать, или посмотрите в других пакетах. > > > > Поправил. Задача 322174 > > Тут в коммите случайно спек-файл заехал, надо разделить: > https://git.altlinux.org/tasks/322174/gears/700/git?p=git;a=commitdiff; > h=7aafb2795c43c2124519fe2b3dd8182f86787aff Коммиты разделил. Задача 322174. (Ответ для Anton Farygin на комментарий #55) > (Ответ для Aleksey Saprunov на комментарий #53) > > > > (Ответ для Anton Farygin на комментарий #36) > > > https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git; > > > a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048 > > > > > > группа выбрана не самым удачным образом > > > секции build, install и files лучше разделять пустой строкой для облегчения > > > читаемости. > > > > > > Версии 1.0.0 не существует (или я не нашёл). > > > > Поправил. Версия взята из файла CMakeLists.txt : > > > > https://github.com/Krzysztow/modbus-utils/blob/master/CMakeLists.txt > > Во многих проектах случается такая невыпущенная версия, которую просто > написали в CMakeList. Но при этом проект не стоит на месте и идёт вперёд, не > обновляя версию в Cmake. > > Предлагаю сейчас сделать версию alt2.git59991e07 и попросить апстрим > повесить тэг на версию 1.0.0. Исправил. Попросил апстрим повесить тег 1.0.0. (Ответ для Aleksey Saprunov на комментарий #58) > (Ответ для Anton Farygin на комментарий #54) > > (Ответ для Aleksey Saprunov на комментарий #52) > > > (Ответ для Anton Farygin на комментарий #51) > > > > Важно при такой версии обязательно указывать git commit в release. Почитайте > > > > на www.altlinux.org как это делать, или посмотрите в других пакетах. > > > > > > Поправил. Задача 322174 > > > > Тут в коммите случайно спек-файл заехал, надо разделить: > > https://git.altlinux.org/tasks/322174/gears/700/git?p=git;a=commitdiff; > > h=7aafb2795c43c2124519fe2b3dd8182f86787aff > > Коммиты разделил. Задача 322174. Всё хорошо, но в репозитории уже упакован https://packages.altlinux.org/ru/sisyphus/srpms/qtl866/2939441462893114955 Как быть с ним ? (Ответ для Anton Farygin на комментарий #60) > (Ответ для Aleksey Saprunov на комментарий #58) > > (Ответ для Anton Farygin на комментарий #54) > > > (Ответ для Aleksey Saprunov на комментарий #52) > > > > (Ответ для Anton Farygin на комментарий #51) > > > > > Важно при такой версии обязательно указывать git commit в release. Почитайте > > > > > на www.altlinux.org как это делать, или посмотрите в других пакетах. > > > > > > > > Поправил. Задача 322174 > > > > > > Тут в коммите случайно спек-файл заехал, надо разделить: > > > https://git.altlinux.org/tasks/322174/gears/700/git?p=git;a=commitdiff; > > > h=7aafb2795c43c2124519fe2b3dd8182f86787aff > > > > Коммиты разделил. Задача 322174. > > Всё хорошо, но в репозитории уже упакован > https://packages.altlinux.org/ru/sisyphus/srpms/qtl866/2939441462893114955 > > Как быть с ним ? Поднял релиз до alt2, пересобрал. В репозиторий собран пакет qtl866 а в задании qtl866-utils Это разные пакеты. Какой из них называется неправильно ? (Ответ для Anton Farygin на комментарий #62) > В репозиторий собран пакет qtl866 а в задании qtl866-utils > Это разные пакеты. Какой из них называется неправильно ? Правильное название qtl866. Так как пакет в сизифе уже с версией 1.0.0-alt1, установил версию 1.0.0-alt2.git1173c3e0 нет, используйте Epoch (Ответ для Anton Farygin на комментарий #64) > нет, используйте Epoch Пересобрал с epoch=1. Задача 322174. В changelog написано что-то не то и не соответствует правилам написания changelog, принятым в альте: https://www.altlinux.org/%D0%A0%D1%83%D0%BA%D0%BE%D0%B2%D0%BE%D0%B4%D1%81%D1%82%D0%B2%D0%BE_%D0%BF%D0%BE_%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D0%BD%D0%B8%D1%8E_changelog (Ответ для Anton Farygin на комментарий #66) > В changelog написано что-то не то и не соответствует правилам написания > changelog, принятым в альте: > https://www.altlinux.org/ > %D0%A0%D1%83%D0%BA%D0%BE%D0%B2%D0%BE%D0%B4%D1%81%D1%82%D0%B2%D0%BE_%D0%BF%D0% > BE_%D0%BD%D0%B0%D0%BF%D0%B8%D1%81%D0%B0%D0%BD%D0%B8%D1%8E_changelog Поправил changelog. Задача 322174. Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане точки в конце предложения, начинающегося с заглавной буквы. (Ответ для Anton Farygin на комментарий #68) > Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане > точки в конце предложения, начинающегося с заглавной буквы. Поправил. Номер задачи прежний. (Ответ для Aleksey Saprunov на комментарий #69) > (Ответ для Anton Farygin на комментарий #68) > > Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане > > точки в конце предложения, начинающегося с заглавной буквы. > > Поправил. Номер задачи прежний. ok. approve выдал несмотря на то, что в changelog не раскрыты все изменения в specfile, а есть информация только про лицензию. Но содержимое spec и git меня уже устраивает. Предлагаю пройтись по другим пакетам из замечаний. (Ответ для Anton Farygin на комментарий #37) > И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого > списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/ > - смотрите что бы ACL позволял вносить исправления для everybody или nobody. Добрый день. Выбрал из списка на сборку: https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/vpn-ws-0.2-alt4 Собрал в задаче 331282 (Ответ для Aleksey Saprunov на комментарий #71) > (Ответ для Anton Farygin на комментарий #37) > > И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого > > списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/ > > - смотрите что бы ACL позволял вносить исправления для everybody или nobody. > > Добрый день. Выбрал из списка на сборку: > https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/vpn-ws-0. > 2-alt4 > > Собрал в задаче 331282 Changelog выглядит так, как будто изменения в проекте сделал ментейнер. Но реальные изменения делал апстрим, надо бы про это явно написать. (Ответ для Anton Farygin на комментарий #72) > (Ответ для Aleksey Saprunov на комментарий #71) > > (Ответ для Anton Farygin на комментарий #37) > > > И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого > > > списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/ > > > - смотрите что бы ACL позволял вносить исправления для everybody или nobody. > > > > Добрый день. Выбрал из списка на сборку: > > https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/vpn-ws-0. > > 2-alt4 > > > > Собрал в задаче 331282 > > Changelog выглядит так, как будто изменения в проекте сделал ментейнер. > Но реальные изменения делал апстрим, надо бы про это явно написать. Исправил .spec файл в соответствии с NMU. Поправил версию, указал мейнтейнера. задача 331282 (Ответ для Aleksey Saprunov на комментарий #74) > задача 331282 https://git.altlinux.org/tasks/331282/gears/300/git?p=git;a=blob;f=altlinux/vpn-ws.spec;h=dec5871951e81bea18190f12d8f6a8509c113416;hb=c15e743ab88619cbe62815b103962527f3c75ceb Packager надо убирать из пакетов - это поле заполняется автоматически. Странно захардкожено имя устройства: https://git.altlinux.org/tasks/331282/gears/300/git?p=git;a=blob;f=altlinux/vpn-ws.service;h=214d0d5d50535f9212f60013a58af01fe7dca3aa;hb=c15e743ab88619cbe62815b103962527f3c75ceb И это обновление слишком круто для NMU. Надо призвать ментейнера на "посмотреть". Я бы убрал NMU из changelog, а лучше бы написал что применены изменения, сделанные апстримом (и их список как раз приведён). (Ответ для Anton Farygin на комментарий #75) > (Ответ для Aleksey Saprunov на комментарий #74) > > задача 331282 > > https://git.altlinux.org/tasks/331282/gears/300/git?p=git;a=blob;f=altlinux/ > vpn-ws.spec;h=dec5871951e81bea18190f12d8f6a8509c113416; > hb=c15e743ab88619cbe62815b103962527f3c75ceb > > Packager надо убирать из пакетов - это поле заполняется автоматически. > > Странно захардкожено имя устройства: > https://git.altlinux.org/tasks/331282/gears/300/git?p=git;a=blob;f=altlinux/ > vpn-ws.service;h=214d0d5d50535f9212f60013a58af01fe7dca3aa; > hb=c15e743ab88619cbe62815b103962527f3c75ceb > > И это обновление слишком круто для NMU. Надо призвать ментейнера на > "посмотреть". > Я бы убрал NMU из changelog, а лучше бы написал что применены изменения, > сделанные апстримом (и их список как раз приведён). Мейнтейнеру написал по электронной почте. NMU и Packager убрал. Задача 331282. в commit specfile плохо выглядит, как будто он переписан полностью, хотя за основу явно взят предыдущий: https://git.altlinux.org/tasks/331282/gears/700/git?p=git;a=commitdiff;h=7722b3e738702f20d34c9673df3fbd002cb76f2f А вот предыдущее дерево: https://git.altlinux.org/gears/v/vpn-ws.git?p=vpn-ws.git;a=history;f=altlinux;hb=dc20ae20e3995ccaba7d70abe85e5ac10bf514a3 Нет ответа от кандидата, возвращаю его на предыдущую стадию. (Ответ для Anton Farygin на комментарий #78) > Нет ответа от кандидата, возвращаю его на предыдущую стадию. Виноват. Прошу не возвращать на предыдущую стадию. Задача 331282 собрана с исправлениями В соседней задаче с kaa@ мне был предъявлен квотум недоверия и обвинили в предвзятости. Поэтому предлагаю секретарю поискать другого ревьювера. Так будет лучше, иначе действительно выглядит так, как будто я предвзято отношусь к коллегам из Саратова. Адрес подписан на devel@, теперь это делается раньше -- в пункте 3.6. Собраны задачи: 351740 - ADT для p11 351588 - новая версия ADT для Сизифа Собрана таска 350201 с alteratorctl, получено подтверждение для пропуска пакета с alteratorctl в Сизиф. (Ответ для Aleksey Saprunov на комментарий #82) > Собраны задачи: > 351740 - ADT для p11 > 351588 - новая версия ADT для Сизифа 351740 - получено подтверждение для отправки в Сизиф 351588 - получено подтверждение для отправки в Сизиф Здравствуйте, мой ментор Игорь Чудов не доступен. Прошу сменить ментора на Ивана Мельникова. (In reply to Aleksey Saprunov from comment #85) > Здравствуйте, мой ментор Игорь Чудов не доступен. Прошу сменить ментора на > Ивана Мельникова. Я не против. Игорь действительно не занимается сейчас вопросами, связаными с Team. (In reply to Aleksey Saprunov from comment #59) > (Ответ для Anton Farygin на комментарий #55) > > Во многих проектах случается такая невыпущенная версия, которую просто > > написали в CMakeList. Но при этом проект не стоит на месте и идёт вперёд, не > > обновляя версию в Cmake. > > > > Предлагаю сейчас сделать версию alt2.git59991e07 и попросить апстрим > > повесить тэг на версию 1.0.0. > > Исправил. Попросил апстрим повесить тег 1.0.0. > %changelog > -* Tue Aug 10 2021 Aleksey Saprunov <sav@altlinux.org> 1.0.0-alt1 > +* Tue Aug 10 2021 Aleksey Saprunov <sav@altlinux.org> 1.0.0-alt2.git7735e178 > - Initial release Не надо так делать. 1.0.0-alt1 уже в Сизифе и p11, так что changelog entry для него должен остаться. В целом, changelog должен отражать историю пакета в репозитории. Так что changelog для следующего релиза должен быть свой, и там стоит написать, что же поменялось. Заодно, кстати, посмотреть, не стоит ли взять более свежий снапшот. (In reply to Aleksey Saprunov from comment #65) > Пересобрал с epoch=1. Задача 322174. Почему %gpl3only? Я рандомно тыкнул в пару файлов, там "or (at your option) any later version." есть. На первый взгляд, главным изменением в этой задаче (и единственным изменением, которое здесь обсуждалось) оказался переход на новую схему версионирования. Однако ни сам переход, ни его мотивация, не отражены ни в changelog, ни в commit messages. Как так вышло? (Ответ для Ivan A. Melnikov на комментарий #87) > (In reply to Aleksey Saprunov from comment #59) > > (Ответ для Anton Farygin на комментарий #55) > > > Во многих проектах случается такая невыпущенная версия, которую просто > > > написали в CMakeList. Но при этом проект не стоит на месте и идёт вперёд, не > > > обновляя версию в Cmake. > > > > > > Предлагаю сейчас сделать версию alt2.git59991e07 и попросить апстрим > > > повесить тэг на версию 1.0.0. > > > > Исправил. Попросил апстрим повесить тег 1.0.0. > > > %changelog > > -* Tue Aug 10 2021 Aleksey Saprunov <sav@altlinux.org> 1.0.0-alt1 > > +* Tue Aug 10 2021 Aleksey Saprunov <sav@altlinux.org> 1.0.0-alt2.git7735e178 > > - Initial release > > Не надо так делать. 1.0.0-alt1 уже в Сизифе и p11, так что changelog entry > для него должен остаться. > > В целом, changelog должен отражать историю пакета в репозитории. > > Так что changelog для следующего релиза должен быть свой, и там стоит > написать, что же поменялось. Заодно, кстати, посмотреть, не стоит ли взять > более свежий снапшот. Поправил changelog, взял более свежий снапшот. Собрал в задаче 362874 для Сизифа. (In reply to Aleksey Saprunov from comment #89) > Поправил changelog, взял более свежий снапшот. Собрал в задаче 362874 для > Сизифа. Отвечу цитатой: (In reply to Anton Farygin from comment #72) > Changelog выглядит так, как будто изменения в проекте сделал ментейнер. > Но реальные изменения делал апстрим, надо бы про это явно написать. > Собрал в задаче 362874 для Сизифа. Стилистически, когда пакуется снапшот, да ещё и правилом "tar: .", корректнее называть тарбол @name@-@version@-@release@, а не просто @name@-@version@. Но это, думаю, мелочь, подзадачу 300 с исправленными замечаниями из comment #90 я проаппрувил. (In reply to Ivan A. Melnikov from comment #91) > Стилистически, когда пакуется снапшот, да ещё и правилом "tar: .", > корректнее называть тарбол @name@-@version@-@release@, а не просто > @name@-@version@. Я считаю, что если исходники в этом tarball могут отличаться от апстримных, то так делать даже обязательно. Понял, спасибо, берусь за второй пакет. (Ответ для Ivan A. Melnikov на комментарий #88) > (In reply to Aleksey Saprunov from comment #65) > > Пересобрал с epoch=1. Задача 322174. > > Почему %gpl3only? Я рандомно тыкнул в пару файлов, там "or (at your option) > any later version." есть. > > На первый взгляд, главным изменением в этой задаче (и единственным > изменением, которое здесь обсуждалось) оказался переход на новую схему > версионирования. Однако ни сам переход, ни его мотивация, не отражены ни в > changelog, ни в commit messages. Как так вышло? Пересобрал qtl866. Поставил нулевую версию, эпоху и релиз по git. Попробовал объяснить смену версии в changelog. Тарбол собрал по схеме @name@-@version@-@release@. Задача 322174 https://packages.altlinux.org/ru/tasks/322174/ (In reply to Aleksey Saprunov from comment #94) > Пересобрал qtl866. Поставил нулевую версию, эпоху и релиз по git. Попробовал > объяснить смену версии в changelog. Тарбол собрал по схеме > @name@-@version@-@release@. Задача 322174 > https://packages.altlinux.org/ru/tasks/322174/ Заапрувил subtask 2500. Сводка по пакету modbus-utils: * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c36 - замечание о том, что группа выбрана неправильно, секции build, install и files лучше разделять пустой строкой для облегчения читаемости. * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c36 - вопрос о том, откуда взялась версия 1.0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c53 - ссылка на CMakeLitst.txt, где указана версия 1.0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c51 - указание на необходимость в таких случах указывтаь версию git commit release * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c52 - исправлена версия на git commit release * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c54 - в коммит попал spec-файл, указано на необходимость разделения * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c55 - предложено сделать версию 1.0.0-alt2.git59991e07 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c58 - коммиты разделены * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c59 - версия исправлена Ментор сменился на Ивана Мельникова: * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c87 - замечание по содержанию changelog(не отражает историю пакета), предложено взять свежий snapshot * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c89 - исправление changelog и собран с более свежим snapshot * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c90 - замечания по содержанию changelog * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c91 - после исправления получен approve Пересобран в задаче 369750. Сводка по пакету qtl866: * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c34 - неправильно выбрана версия 1.0.0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c38 - версия исправлена на 0.1.0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c43 - вопрос о том, откуда взялась версия 0.1.0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c46 - версия взята из указанного коммита * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c47 - предложено взять версию 0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c50 - версия исправлена на 0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c60 - замечание о том, что ранее была заапрувлена неправильная версия * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c61 - версия изменена на alt2 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c62 - замечание о неправильно названии пакета * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c63 - исправление версии на 1.0.0-alt2.git1173c3e0 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c64 - замечание о том, что надо было использовать Epoch * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c65 - исправлено на Epoch=1 * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c66 - замечание по содержанию changelog * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c68 - замечание по содержанию changelog * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c69 - changelog Пакет протолкнут в Сизиф. Ментор сменился на Ивана Мельникова: * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c88 - замечание по неправильно выбранной лицензии, не отражающееся в changelog изменение версионирования. * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c94 - замечания исправлены * https://bugzilla.altlinux.org/show_bug.cgi?id=40829#c95 - получен approve В настоящие момент я сопровождаю следущие пакеты нашей разработки: https://packages.altlinux.org/ru/sisyphus/srpms/adt/ - ADT https://packages.altlinux.org/ru/sisyphus/srpms/alterator-explorer/ - alterator-explorer https://packages.altlinux.org/ru/sisyphus/srpms/alteratorctl/ - alteratorctl Так же для использования в наших продуктах (alteratorctl) была собрана (с учетом Shared Libs Policy - https://www.altlinux.org/Shared_Libs_Policy и Shared Libs Policy Comments https://www.altlinux.org/Shared_Libs_Policy_Comments) разделяемая библиотека libtomlc99: https://packages.altlinux.org/ru/sisyphus/srpms/libtomlc99/ https://packages.altlinux.org/ru/sisyphus/srpms/libtomlc99/specfiles/ нет, это не соответствует SharedLibsPolicy (Ответ для Anton Farygin на комментарий #100) > https://packages.altlinux.org/ru/sisyphus/srpms/libtomlc99/specfiles/ > > нет, это не соответствует SharedLibsPolicy Понял, исправлю. |
Description
Aleksey Saprunov 2021-08-30 18:22:36 MSK