Bug 41493 - warning при отсутствии watch файла
Summary: warning при отсутствии watch файла
Status: CLOSED WONTFIX
Alias: None
Product: Sisyphus
Classification: Development
Component: sisyphus_check (show other bugs)
Version: unstable
Hardware: x86_64 Linux
: P5 normal
Assignee: Dmitry V. Levin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2021-12-02 19:26 MSK by Anton Farygin
Modified: 2021-12-03 09:59 MSK (History)
8 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Anton Farygin 2021-12-02 19:26:54 MSK
Уже достаточно давно Игорем Власенко разработан и работает сервис отслеживания обновлений http://watch.altlinux.org

Документация: https://www.altlinux.org/Watch

Предлагаю добавить warning, если в собранном src.rpm пакете не найден файл по маске *watch
Comment 1 Alexey Gladkov 2021-12-02 19:37:19 MSK
К сожалению у нас нагородили необходимость парсинга спека, чтобы понять для какого source этот watch-файл. Поэтому в sisyphus_check можно только порверить сам факт наличия watch-файлов.

Но если бы у нас в репозиториях были watch-файлы, то это было бы гораздо полезнее чем Url в спеках и комментарии в том же спеке, где берётся тарболл.

Вот бы watch-файл был самодостаточным.
Comment 2 viy 2021-12-02 20:19:18 MSK
может, не стоит пока? Ведь на сервисе watch внедряется проверка обновлений по git репозиторию, найденному по тегу VCS: к примеру, https://gitlab.com/name/foo
Для пакета с рабочим VCS: требование watch кажется избыточным.
когда все устаканится, можно будет написать полиси, регламентирующее VCS.
Вынести на обсуждение и принять.
А потом, уже на основании действующего полиси, добавлять 
проверки наличия VCS:, валидности VCS:, наличия watch,
в т.ч. только тогда можно будет думать и о warnings.
Comment 3 viy 2021-12-02 20:23:28 MSK
т.е. предлагаю вопрос отложить до завершения работ по VCS:
и обсуждения полиси для VCS:
Comment 4 Alexey Gladkov 2021-12-02 20:33:33 MSK
Этот FR стал результатом обсуждения с Антоном ошибок в

https://beta.packages.altlinux.org/ru/sisyphus/maintainers/legion/watch

который построен на основе watch.altlinux.org . В частности запись gnupg2 [2.2.32] [2.3.3] не является правильной т.к. у нас собираются LTS версии. Это невозможно понять из тегов VCS или Url. Угадав тут не будет работать никогда.

Единственным выходом это сделать правильно это указать watch-файл.
Comment 5 Alexey Gladkov 2021-12-02 20:34:07 MSK
(Ответ для viy на комментарий #3)
> т.е. предлагаю вопрос отложить до завершения работ по VCS:
> и обсуждения полиси для VCS:

Ок.
Comment 6 Anton Farygin 2021-12-03 08:54:30 MSK
(Ответ для viy на комментарий #2)
> может, не стоит пока? Ведь на сервисе watch внедряется проверка обновлений
> по git репозиторию, найденному по тегу VCS: к примеру,
> https://gitlab.com/name/foo
> Для пакета с рабочим VCS: требование watch кажется избыточным.
> когда все устаканится, можно будет написать полиси, регламентирующее VCS.
> Вынести на обсуждение и принять.
> А потом, уже на основании действующего полиси, добавлять 
> проверки наличия VCS:, валидности VCS:, наличия watch,
> в т.ч. только тогда можно будет думать и о warnings.

у меня ещё есть пакеты, для которых отстуствует публичный VCS.
watch файл всё равно полезен. Например, в нём можно описать какая именно версия нужна:
https://git.altlinux.org/gears/c/clickhouse.git?p=clickhouse.git;a=blob;f=.gear/clickhouse.watch;h=46e211cfb9dac8d02ab087d9f5746dc86af00618;hb=a4cf94de574c1daadad7c3197260ef46d2a04e5e

Придётся этот синтаксис тащить в тэг VCS.

Я бы сделат так - если есть VCS/homepage на git, но нет watch файла, то использовать его, а если же есть watch file, то все остальные методы отключать.

Ну или как-то это же делать опционально.
Comment 7 Gleb F-Malinovskiy 2021-12-03 09:57:03 MSK
А я правильно понимаю, что изначальная задумка в том, чтобы массово класть watch-файлы в src.rpm?
Потому что я не вижу никакого другого способа сделать его предметом проверки sisyphus_check -- он работает с rpm-пакетами.
Comment 8 Anton Farygin 2021-12-03 09:59:58 MSK
да, конечно - по аналогии с Debian массово паковать watch файлы в пакеты.
сейчас всего чуть больше 200 пакетов содержит watch файлы.