Bug 40829 - [3.6] join sav@
Summary: [3.6] join sav@
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:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-08-30 18:22 MSK by Aleksey Saprunov
Modified: 2023-12-05 19:26 MSK (History)
6 users (show)

See Also:


Attachments
Публичный ключ (3.00 KB, text/plain)
2021-08-30 18:22 MSK, Aleksey Saprunov
no flags Details
Публичный (97 bytes, application/vnd.ms-publisher)
2021-08-30 18:23 MSK, Aleksey Saprunov
no flags Details
Ключ с новой почтой sav@altlinux.org (82 bytes, application/x-iwork-keynote-sffkey)
2021-10-06 13:41 MSK, Aleksey Saprunov
no flags Details
Ключ с новым ником sav (3.00 KB, application/pgp-encrypted)
2021-10-06 13:42 MSK, Aleksey Saprunov
no flags Details
Новый ключ sav@altlinux.org (98 bytes, application/vnd.ms-publisher)
2023-01-17 14:51 MSK, Aleksey Saprunov
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aleksey Saprunov 2021-08-30 18:22:36 MSK
Created attachment 9628 [details]
Публичный ключ

псевдоним - SA
форвард почты с sa@altlunux.org на workernight@yandex.ru
ментор - nir@
Поддержка пакетов связанных с групповыми политиками
Comment 1 Aleksey Saprunov 2021-08-30 18:23:09 MSK
Created attachment 9629 [details]
Публичный
Comment 2 Igor Chudov 2021-08-30 18:31:26 MSK
Менторство подтверждаю, кандидат будет собирать ПО для ПЛК: modbus-utils.
Comment 3 Aleksey Saprunov 2021-09-01 11:05:06 MSK
Добрый день. Собран пакет, modbus-utils 
форк - https://github.com/AlexSP0/modbus-utils-1  

Создан и принят пулл реквест https://github.com/Krzysztow/modbus-utils.
Comment 4 Igor Chudov 2021-09-02 16:41:21 MSK
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 не имею.
Comment 5 Gleb F-Malinovskiy 2021-09-02 17:14:32 MSK
(In reply to Aleksey Saprunov from comment #0)
> Created attachment 9628 [details]
> Публичный ключ
> 
> псевдоним - SA

К сожалению, этот ник уже использовался другим членом Team, к тому же нынешняя процедура предполагает использование ников не менее 3 символов.
Comment 6 Aleksey Saprunov 2021-09-03 16:27:18 MSK
(Ответ для Gleb F-Malinovskiy на комментарий #5)
> (In reply to Aleksey Saprunov from comment #0)
> > Created attachment 9628 [details] [подробности] [details]
> > Публичный ключ
> > 
> > псевдоним - SA
> 
> К сожалению, этот ник уже использовался другим членом Team, к тому же
> нынешняя процедура предполагает использование ников не менее 3 символов.

Добрый вечер! Ник sav свободен?
Comment 7 Gleb F-Malinovskiy 2021-09-03 18:32:48 MSK
(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 свободен?

Насколько я вижу, да.
Comment 8 Aleksey Saprunov 2021-09-06 13:35:58 MSK
Добрый день. Собран пакет qtl866
форк - https://github.com/AlexSP0/qtl866/tree/alt_release

создан, но пока висит пуллреквест: https://github.com/wd5gnr/qtl866/pull/14
Comment 9 Igor Chudov 2021-09-07 14:52:49 MSK
Добрый день.

Я проверил и особых замечаний не имею, кроме того, что стоит последний тегированый коммит сделать с помощью gear-commit, чтобы была соответствующая запись в истории. В целом, в hasher оно собирается и устанавливается у меня вполне корректно.
Comment 10 Aleksey Saprunov 2021-09-07 15:04:43 MSK
(Ответ для Igor Chudov на комментарий #9)
> Добрый день.
> 
> Я проверил и особых замечаний не имею, кроме того, что стоит последний
> тегированый коммит сделать с помощью gear-commit, чтобы была соответствующая
> запись в истории. В целом, в hasher оно собирается и устанавливается у меня
> вполне корректно.

поправил.
Comment 11 Igor Chudov 2021-09-07 16:36:01 MSK
Добрый день.

Проверил. С моей стороны замечаний нет. Предлагаю продвинуть задачу дальше.
Comment 12 Aleksey Saprunov 2021-10-06 13:41:42 MSK
Created attachment 9774 [details]
Ключ с новой почтой sav@altlinux.org
Comment 13 Aleksey Saprunov 2021-10-06 13:42:49 MSK
Created attachment 9775 [details]
Ключ с новым ником sav
Comment 14 Aleksey Saprunov 2021-10-06 13:43:32 MSK
Обновил ключи для sav@altlinux.org
Comment 15 Gleb F-Malinovskiy 2021-10-18 16:47:58 MSK
(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.
Comment 16 Igor Chudov 2021-11-22 14:15:30 MSK
Добрый день.

Давайте продвинем кандидата дальше по процедуре.
Comment 17 Gleb F-Malinovskiy 2021-11-29 15:26:25 MSK
Увы, почта так и не была подтверждена.
Comment 18 Aleksey Saprunov 2021-12-01 10:57:38 MSK
псевдоним - sav
форвард почты с sa@altlunux.org на spbmbm@gmail.com
ментор - nir@

Добрый день! Исправил почту на spbmbm@gmail.com
Comment 19 Igor Chudov 2021-12-29 16:33:15 MSK
Добрый день.

На данный момент форвард почты не настроен. Я предлагаю зафиксировать реквизиты для отправки в неизменном виде.

Глеб, какой адрес почты должен подтвердить кандидат? Что от него требуется?
Comment 20 Gleb F-Malinovskiy 2022-01-17 16:46:53 MSK
ssh ключ на gitery.alt зарегистрирован.
Адрес для пересылки создан.

T/J/S -> 2.3.
Comment 21 Igor Chudov 2022-03-01 15:52:57 MSK
Добрый день, коллеги.

Я вроде бы уже проверял пакет Алексея и на тот момент он соответствовал требованиям. specfile был оформлен корректно, сборка производилась в Hasher. Предлагаю продвинуть Алексея по процедуре и дать ему доступ к сборке. Базовые действия он изучил, остальное аккуратно освоит.
Comment 22 Igor Chudov 2023-01-11 13:26:23 MSK
Обновляю статус коллеги, так как считаю, что он достаточно квалифицирован для работы над пакетами.
Comment 23 Gleb F-Malinovskiy 2023-01-16 21:08:26 MSK
ssh ключ на gyle.alt зарегистрирован.
Пакет alt-gpgkeys обновлён.

T/J/S -> 3.5.
Comment 24 Aleksey Saprunov 2023-01-17 14:51:46 MSK
Created attachment 12317 [details]
Новый ключ  sav@altlinux.org
Comment 25 Aleksey Saprunov 2023-01-17 14:55:25 MSK
Прошу заменить ssh ключ(новый файл во вложении - https://bugzilla.altlinux.org/attachment.cgi?id=12317)  К сожалению, был приложен не тот файл. Заранее спасибо.
Comment 26 Gleb F-Malinovskiy 2023-01-17 15:08:30 MSK
(In reply to Aleksey Saprunov from comment #25)
> Прошу заменить ssh ключ(новый файл во вложении -
> https://bugzilla.altlinux.org/attachment.cgi?id=12317)  К сожалению, был
> приложен не тот файл. Заранее спасибо.

Вроде бы, этот же ключ и был с самого начала, у него только комментарий в процессе поменялся, но комментарий ни на что не влияет, конечно.  В промежуточном вложении только был хэш ключа («лучше, у меня есть картинка ключа»), но того же самого.
Comment 27 Igor Chudov 2023-01-23 13:17:10 MSK
Репозиторий https://git.altlinux.org/people/sav/packages/?p=libmodbus-utils.git;a=summary и спеку посмотрел.
Задачу 313724 посмотрел.

Замечаний не имею. Надо приглашать кросс-ревьювера.
Comment 28 Gleb F-Malinovskiy 2023-01-24 13:07:10 MSK
(In reply to Igor Chudov from comment #27)
> Репозиторий
> https://git.altlinux.org/people/sav/packages/?p=libmodbus-utils.git;
> a=summary и спеку посмотрел.
> Задачу 313724 посмотрел.

Напоминаю, что ментор может заапрувить задание когда считает, что оно готово для Сизифа.

> Замечаний не имею. Надо приглашать кросс-ревьювера.

Мне кажется, что любой рецензент скажет, что одного пакета мало.  Вы уверены, что этого пакета будет достаточно?
Comment 29 Aleksey Saprunov 2023-05-26 11:46:32 MSK
Собрал qtl866:
https://git.altlinux.org/people/sav/packages/?p=qtl866.git

Также собирал admc с некоторыми новыми фичами для ревью:
https://git.altlinux.org/people/sav/packages/?p=admc.git
Comment 30 Aleksey Saprunov 2023-05-26 11:48:58 MSK
Для qtl866 задача 317707, для admc задача 315344
Comment 31 Igor Chudov 2023-06-06 10:11:01 MSK
Задачи для qtl866 и modbus-utils ушли в Sisyphus. Это очень положительная новость, так как эти инструменты позволяют работать с оборудованием для промышленной автоматизации.

С ADMC у меня вопросы скорее к руководителям проекта. Нужна ли эта сборка сейчас или нет.
Comment 32 Igor Chudov 2023-06-06 12:11:15 MSK
rider@ согласился быть рецензентом, прошу секретаря подтвердить выбор кандидатуры. ping Gleb.
Comment 33 Gleb F-Malinovskiy 2023-09-10 13:24:58 MSK
Призван рецензент (rider@) для независимой оценки готовности кандидата.

T/J/S -> 4.2.
Comment 34 Anton Farygin 2023-09-11 08:48:05 MSK
(Ответ для Aleksey Saprunov на комментарий #29)
> Собрал qtl866:
> https://git.altlinux.org/people/sav/packages/?p=qtl866.git

а откуда взялась версия 1.0.0 ? не могу найти её в апстриме.
Comment 35 Anton Farygin 2023-09-11 08:53:43 MSK
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
Comment 36 Anton Farygin 2023-09-11 08:56:44 MSK
https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git;a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048

группа выбрана не самым удачным образом
секции build, install и files лучше разделять пустой строкой для облегчения читаемости.

Версии 1.0.0 не существует (или я не нашёл).
Comment 37 Anton Farygin 2023-09-11 08:59:54 MSK
И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/
 - смотрите что бы ACL позволял вносить исправления для everybody или nobody.
Comment 38 Aleksey Saprunov 2023-09-20 20:59:10 MSK
(Ответ для Anton Farygin на комментарий #34)
> (Ответ для Aleksey Saprunov на комментарий #29)
> > Собрал qtl866:
> > https://git.altlinux.org/people/sav/packages/?p=qtl866.git
> 
> а откуда взялась версия 1.0.0 ? не могу найти её в апстриме.

Прошу прощения за длительный ответ. Такой версии в апстриме нет. Исправил на 0.1.0
Comment 39 Aleksey Saprunov 2023-09-20 21:01:00 MSK
(Ответ для 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

Понял, в течение ближайщего времени исправлю.
Comment 40 Aleksey Saprunov 2023-09-20 21:02:15 MSK
(Ответ для Anton Farygin на комментарий #36)
> https://git.altlinux.org/people/sav/packages/?p=modbus-utils.git;
> a=commitdiff;h=18b90f864238d2ec46e74ab384db90d762e41048
> 
> группа выбрана не самым удачным образом
> секции build, install и files лучше разделять пустой строкой для облегчения
> читаемости.
> 
> Версии 1.0.0 не существует (или я не нашёл).

В ближайшее время исправлю
Comment 41 Aleksey Saprunov 2023-09-20 21:03:18 MSK
(Ответ для Anton Farygin на комментарий #37)
> И ещё просьба взять что-то ещё. Можно пройтись и выбрать что-то из этого
> списка https://git.altlinux.org/beehive/logs/Sisyphus-x86_64/latest/error/
>  - смотрите что бы ACL позволял вносить исправления для everybody или nobody.

Хорошо, после исправления выбранных пакетов, соберу.
Comment 42 Aleksey Saprunov 2023-09-20 22:09:06 MSK
(Ответ для 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
Comment 43 Anton Farygin 2023-09-21 05:58:14 MSK
(Ответ для 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

А эта откуда взялась ? можно мне ссылку ?
Comment 44 Anton Farygin 2023-09-21 06:40:20 MSK
(Ответ для 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
Comment 45 Aleksey Saprunov 2023-09-21 08:10:50 MSK
(Ответ для 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

Понял, спасибо.
Comment 46 Aleksey Saprunov 2023-09-21 08:17:25 MSK
(Ответ для 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
Comment 47 Anton Farygin 2023-09-21 10:29:29 MSK
Если апстрим не выпускал версии, то используйте версию 0
Comment 48 Aleksey Saprunov 2023-09-21 12:26:23 MSK
(Ответ для Anton Farygin на комментарий #47)
> Если апстрим не выпускал версии, то используйте версию 0

Попробовал собрать с версией 0. Однако, ментор протолкнул пакет в сизиф с версией 1.0:

https://git.altlinux.org/tasks/322172/logs/events.3.1.log

Предполагаю, что надо поднять версию до 1.0.0-alt2?
Comment 49 Aleksey Saprunov 2023-09-21 12:29:00 MSK
(Ответ для 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.
Comment 50 Aleksey Saprunov 2023-09-21 14:11:18 MSK
(Ответ для Anton Farygin на комментарий #47)
> Если апстрим не выпускал версии, то используйте версию 0

Пересобрал с версией 0. Задача 322174.
Comment 51 Anton Farygin 2023-09-21 14:39:58 MSK
Важно при такой версии обязательно указывать git commit в release. Почитайте на www.altlinux.org как это делать, или посмотрите в других пакетах.
Comment 52 Aleksey Saprunov 2023-09-21 16:19:04 MSK
(Ответ для Anton Farygin на комментарий #51)
> Важно при такой версии обязательно указывать git commit в release. Почитайте
> на www.altlinux.org как это делать, или посмотрите в других пакетах.

Поправил. Задача 322174
Comment 53 Aleksey Saprunov 2023-09-21 16:44:51 MSK

(Ответ для 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
Comment 54 Anton Farygin 2023-09-22 07:33:31 MSK
(Ответ для 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
Comment 55 Anton Farygin 2023-09-22 07:41:23 MSK
(Ответ для 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.
Comment 56 Aleksey Saprunov 2023-09-22 13:46:52 MSK
(Ответ для 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

Принято, поправлю.
Comment 57 Aleksey Saprunov 2023-09-22 13:47:19 MSK
(Ответ для 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.

Хорошо, сделаю.
Comment 58 Aleksey Saprunov 2023-09-25 15:23:13 MSK
(Ответ для 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.
Comment 59 Aleksey Saprunov 2023-09-25 15:24:06 MSK
(Ответ для 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.
Comment 60 Anton Farygin 2023-09-25 15:29:18 MSK
(Ответ для 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

Как быть с ним ?
Comment 61 Aleksey Saprunov 2023-09-25 16:07:49 MSK
(Ответ для 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, пересобрал.
Comment 62 Anton Farygin 2023-09-25 17:50:14 MSK
В репозиторий собран пакет qtl866 а в задании qtl866-utils
Это разные пакеты. Какой из них называется неправильно ?
Comment 63 Aleksey Saprunov 2023-09-25 23:20:48 MSK
(Ответ для Anton Farygin на комментарий #62)
> В репозиторий собран пакет qtl866 а в задании qtl866-utils
> Это разные пакеты. Какой из них называется неправильно ?

Правильное название qtl866. Так как пакет в сизифе уже с версией 1.0.0-alt1, установил версию 1.0.0-alt2.git1173c3e0
Comment 64 Anton Farygin 2023-09-26 08:35:22 MSK
нет, используйте Epoch
Comment 65 Aleksey Saprunov 2023-09-26 13:34:18 MSK
(Ответ для Anton Farygin на комментарий #64)
> нет, используйте Epoch

Пересобрал с epoch=1. Задача 322174.
Comment 66 Anton Farygin 2023-09-26 14:04:05 MSK
В 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
Comment 67 Aleksey Saprunov 2023-09-27 14:01:50 MSK
(Ответ для 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.
Comment 68 Anton Farygin 2023-09-27 15:13:30 MSK
Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане точки в конце предложения, начинающегося с заглавной буквы.
Comment 69 Aleksey Saprunov 2023-09-27 16:26:57 MSK
(Ответ для Anton Farygin на комментарий #68)
> Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане
> точки в конце предложения, начинающегося с заглавной буквы.

Поправил. Номер задачи прежний.
Comment 70 Anton Farygin 2023-09-28 15:24:24 MSK
(Ответ для Aleksey Saprunov на комментарий #69)
> (Ответ для Anton Farygin на комментарий #68)
> > Пожалуйста, прочитайте ещё раз правила написания changelog. Особенно в плане
> > точки в конце предложения, начинающегося с заглавной буквы.
> 
> Поправил. Номер задачи прежний.

ok. approve выдал несмотря на то, что в changelog не раскрыты все изменения в specfile, а есть информация только про лицензию.

Но содержимое spec и git меня уже устраивает.

Предлагаю пройтись по другим пакетам из замечаний.
Comment 71 Aleksey Saprunov 2023-10-09 12:45:36 MSK
(Ответ для 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
Comment 72 Anton Farygin 2023-10-10 05:27:41 MSK
(Ответ для 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 выглядит так, как будто изменения в проекте сделал ментейнер.
Но реальные изменения делал апстрим, надо бы про это явно написать.
Comment 73 Aleksey Saprunov 2023-10-10 11:44:51 MSK
(Ответ для 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. Поправил версию, указал мейнтейнера.
Comment 74 Aleksey Saprunov 2023-10-10 11:45:12 MSK
задача 331282
Comment 75 Anton Farygin 2023-10-26 10:43:59 MSK
(Ответ для 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, а лучше бы написал что применены изменения, сделанные апстримом (и их список как раз приведён).
Comment 76 Aleksey Saprunov 2023-10-30 15:02:02 MSK
(Ответ для 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.
Comment 77 Anton Farygin 2023-10-30 15:07:31 MSK
в 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
Comment 78 Anton Farygin 2023-11-09 13:32:42 MSK
Нет ответа от кандидата, возвращаю его на предыдущую стадию.
Comment 79 Aleksey Saprunov 2023-11-09 13:40:34 MSK
(Ответ для Anton Farygin на комментарий #78)
> Нет ответа от кандидата, возвращаю его на предыдущую стадию.

Виноват. Прошу не возвращать на предыдущую стадию. Задача 331282 собрана с исправлениями
Comment 80 Anton Farygin 2023-11-09 13:48:09 MSK
В соседней задаче с kaa@ мне был предъявлен квотум недоверия и обвинили в предвзятости. 
Поэтому предлагаю секретарю поискать другого ревьювера. Так будет лучше, иначе действительно выглядит так, как будто я предвзято отношусь к коллегам из Саратова.
Comment 81 Gleb F-Malinovskiy 2023-12-05 19:24:24 MSK
Адрес подписан на devel@, теперь это делается раньше -- в пункте 3.6.