[devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Aleksey Cheusov]

http://git.altlinux.org/tasks/252022/logs/events.1.1.log
/people/cheusov/packages/oksh.git

Буду признателен за review и разрешение на импорт

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Andrey Savchenko]

[-- Attachment #1: Type: text/plain, Size: 2819 bytes --]

Добрый день!

On Thu, 21 May 2020 00:44:08 +0300 Aleksey Cheusov wrote:
> http://git.altlinux.org/tasks/252022/logs/events.1.1.log
> /people/cheusov/packages/oksh.git
> 
> Буду признателен за review и разрешение на импорт

Серьёзных проблем нет, но могу отметить следующие замечания:

1. При сборке вижу проблему с неправильно указанной лицензией:
/.out/oksh-6.7-alt1.x86_64.rpm: license not found in '/usr/share/license' directory: Public
/.out/oksh-6.7-alt1.x86_64.rpm: license not found in '/usr/share/license' directory: Domain
/.out/oksh-6.7-alt1.src.rpm: license not found in '/usr/share/license' directory: Public
/.out/oksh-6.7-alt1.src.rpm: license not found in '/usr/share/license' directory: Domain

У нас принято указывать лицензии из списка /usr/share/licenses.
Если там подходящей нет, можно добавить в пакет common-licenses.

В данном случае подходящей лицензией на мой взгляд будет Unlicense:

This is free and unencumbered software released into the public domain.

Полный текст в /usr/share/license/Unlicense

Эта проверка у нас обязательной совсем недавно стала и, видимо, ещё
не была должным образом документирована.

2. %changelog немного странный для первой версии:
* Wed May 20 2020 Aleksey Cheusov <cheusov@altlinux.org> 6.7-alt1
- Minor clean-ups for spec

Я бы использовал git commit message от первого коммита:
- Import archive oksh-6.7

Удалять коммиты при этом не нужно. У нас есть любители делать
rebase -i на каждый чих, но я против правок истории кроме крайней
необходимости (ага, я читал 1984 Оруэла и знаю, чем чреваты правки
истории); исключение, на мой взгляд, допустимо, когда патчи
хранятся не в виде файлов, а в виде коммитов в куче бранчей и нужно
следить за их минимизацией. Но это не тот случай.

3. По-моему, было бы неплохо зарегистрировать шелл в /etc/shells по
подобию tcsh. Но это на твоё усмотрение. Возможно, есть причины, по
которым не хочется так делать.

Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Aleksey Cheusov]

21.05.2020, 11:58, "Andrey Savchenko" <bircoph@altlinux.org>:
> On Thu, 21 May 2020 00:44:08 +0300 Aleksey Cheusov wrote:
>>  http://git.altlinux.org/tasks/252022/logs/events.1.1.log
>>  /people/cheusov/packages/oksh.git
>>
>>  Буду признателен за review и разрешение на импорт
>
> Серьёзных проблем нет, но могу отметить следующие замечания:

Спасибо. task #252044. Там же.

> 1. При сборке вижу проблему с неправильно указанной лицензией:

Fixed. Как сделать в hasher-е эту ошибку фатальной?
Пользуюсь вариантом настройки со скриптом "compile".

> 2. %changelog немного странный для первой версии:

Даже не знаю, как так получилось, наверное, я во сне кнопки нажимал.
Fixed.

> 3. По-моему, было бы неплохо зарегистрировать шелл в /etc/shells по
> подобию tcsh. Но это на твоё усмотрение. Возможно, есть причины, по
> которым не хочется так делать.

Мне вот как раз не нравится как в tcsh. Чтобы сделать нормально, надо завести
rpm макросы и прописать их во всех shell-ах прямо сразу и одновременно.
Если таких макросов нет, пусть админ локал хоста руками разрешит,
не вижу в этом большого криминала. Тем более, что альтовский mksh 
тоже никуда не прописывается.

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Andrey Savchenko]

[-- Attachment #1: Type: text/plain, Size: 3351 bytes --]

On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
> 21.05.2020, 11:58, "Andrey Savchenko" <bircoph@altlinux.org>:
> > On Thu, 21 May 2020 00:44:08 +0300 Aleksey Cheusov wrote:
> >>  http://git.altlinux.org/tasks/252022/logs/events.1.1.log
> >>  /people/cheusov/packages/oksh.git
> >>
> >>  Буду признателен за review и разрешение на импорт
> >
> > Серьёзных проблем нет, но могу отметить следующие замечания:
> 
> Спасибо. task #252044. Там же.

Я больше проблем не вижу, но похоже, что моего аппрува
недостаточно. Я не особо понимаю, как работает этот механизм для
мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
нужен approve от mike или админов сборочницы.

> > 1. При сборке вижу проблему с неправильно указанной лицензией:
> 
> Fixed. Как сделать в hasher-е эту ошибку фатальной?
> Пользуюсь вариантом настройки со скриптом "compile".

Нужно в файле /etc/sisyphus_check/check.d/250-check-license
раскомментировать следующие строчки:

150     #[ "$bad_license" = 0 ] ||
151     #   rc=1

Насколько я понимаю, сейчас у многих пакетов неправильно указаны
лицензии (по крайней мере с точки зрения синтаксиса), кроме того,
некоторых вполне нормальных, но редких лицензий нет в пакете
common-licenses: многих людей сбивает с толку название и они
считают, что там должны быть только часто употребимые лицензии,
однако проверка построена так, что этот пакет содержит все
допустимые лицензии. Думаю, что пока эти все проблемы будут
исправлены, будет предупреждение вместо ошибки.

> > 3. По-моему, было бы неплохо зарегистрировать шелл в /etc/shells по
> > подобию tcsh. Но это на твоё усмотрение. Возможно, есть причины, по
> > которым не хочется так делать.
> 
> Мне вот как раз не нравится как в tcsh. Чтобы сделать нормально, надо завести
> rpm макросы и прописать их во всех shell-ах прямо сразу и одновременно.
> Если таких макросов нет, пусть админ локал хоста руками разрешит,
> не вижу в этом большого криминала. Тем более, что альтовский mksh 
> тоже никуда не прописывается.

Согласен, этот вопрос нужно решать глобально.

Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Aleksey Cheusov]

21.05.2020, 23:41, "Andrey Savchenko" <bircoph@altlinux.org>:
> On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
>>  Спасибо. task #252044. Там же.
>
> Я больше проблем не вижу, но похоже, что моего аппрува
> недостаточно. Я не особо понимаю, как работает этот механизм для
> мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
> нужен approve от mike или админов сборочницы.

ssh build.alt task run --commit 252044
прошел, пакет в сизифе.

И какой-то добрый человек или бот добавил меня в acl
$ ssh build.alt acl sisyphus oksh show
oksh	cheusov @everybody
$

>>  Как сделать в hasher-е эту ошибку фатальной?
>>  Пользуюсь вариантом настройки со скриптом "compile".
>
> Нужно в файле /etc/sisyphus_check/check.d/250-check-license
> раскомментировать следующие строчки:
>
> 150 #[ "$bad_license" = 0 ] ||
> 151 # rc=1

Не вижу такого файла и проверки лицензии вообще.

[root@ryzen ica]# rpm -q hasher
hasher-1.4.3-alt1.noarch
[root@ryzen ica]# ls /etc/sisyphus_check/check.d/
000-check-changelog  040-check-packager     080-check-group    120-check-libtool     170-check-subdirs    210-check-kernel    240-check-browser
010-check-gpg        050-check-printable    090-check-nvr      130-check-fhs         180-check-pkgconfig  211-check-firmware
020-check-buildhost  060-check-summary      100-check-deps     140-check-perms       190-check-dirlist    220-check-python
030-check-buildtime  070-check-description  110-check-content  150-check-intersects  200-check-noarch     230-check-bindir
[root@ryzen ica]# 

> Насколько я понимаю, сейчас у многих пакетов неправильно указаны
> лицензии... Думаю, что пока эти все проблемы будут
> исправлены, будет предупреждение вместо ошибки.

Если б я была царицей, я бы сделал эту проверку обязательной, и предоставил бы возможность
сделать ее ворнингом в особенных случаях. Далее объявил бы всеобщую мобилизацию,
Ибо нефиг расслабляться :-) Как ее сделать ворнингом -- отдельный вопрос,
может, на уровне спека, может, еще как... Тут вам виднее.

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Gleb Fotengauer-Malinovskiy]

[-- Attachment #1: Type: text/plain, Size: 1242 bytes --]

On Fri, May 22, 2020 at 04:57:09PM +0300, Aleksey Cheusov wrote:
> 21.05.2020, 23:41, "Andrey Savchenko" <bircoph@altlinux.org>:
> > On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
> >>  Спасибо. task #252044. Там же.
> >
> > Я больше проблем не вижу, но похоже, что моего аппрува
> > недостаточно. Я не особо понимаю, как работает этот механизм для
> > мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
> > нужен approve от mike или админов сборочницы.

Подойдёт аппрув от любого члена тим уже завершившего join.

> ssh build.alt task run --commit 252044
> прошел, пакет в сизифе.
> 
> И какой-то добрый человек или бот добавил меня в acl
> $ ssh build.alt acl sisyphus oksh show
> oksh	cheusov @everybody

Сборочница делает это автоматически для всех новых пакетов.  При этом ACL
для кандидатов не действует.

-- 
glebfm

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 801 bytes --]

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Andrey Savchenko]

[-- Attachment #1: Type: text/plain, Size: 1101 bytes --]

On Fri, 22 May 2020 17:35:33 +0300 Gleb Fotengauer-Malinovskiy
wrote:
> On Fri, May 22, 2020 at 04:57:09PM +0300, Aleksey Cheusov wrote:
> > 21.05.2020, 23:41, "Andrey Savchenko" <bircoph@altlinux.org>:
> > > On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
> > >>  Спасибо. task #252044. Там же.
> > >
> > > Я больше проблем не вижу, но похоже, что моего аппрува
> > > недостаточно. Я не особо понимаю, как работает этот механизм для
> > > мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
> > > нужен approve от mike или админов сборочницы.
> 
> Подойдёт аппрув от любого члена тим уже завершившего join.

Я тоже так думал, но после моего аппрува таск остался в EPERM,
а должен был перейти в TESTED (он был без --commit).
 
Best regards,
Andrew Savchenko

[-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --]

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Антон Мидюков]

22.05.2020 22:33, Andrey Savchenko пишет:
> On Fri, 22 May 2020 17:35:33 +0300 Gleb Fotengauer-Malinovskiy
> wrote:
>> On Fri, May 22, 2020 at 04:57:09PM +0300, Aleksey Cheusov wrote:
>>> 21.05.2020, 23:41, "Andrey Savchenko" <bircoph@altlinux.org>:
>>>> On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
>>>>>   Спасибо. task #252044. Там же.
>>>> Я больше проблем не вижу, но похоже, что моего аппрува
>>>> недостаточно. Я не особо понимаю, как работает этот механизм для
>>>> мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
>>>> нужен approve от mike или админов сборочницы.
>> Подойдёт аппрув от любого члена тим уже завершившего join.
> Я тоже так думал, но после моего аппрува таск остался в EPERM,
> а должен был перейти в TESTED (он был без --commit).

Я не замечал ни разу изменение статуса задания после аппрува.

Делаю повторный аппрув, чтобы получить в ответ, что аппрув уже выдан :-)

-- 
С уважением, Антон Мидюков <antohami@altlinux.org>

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Michael Shigorin]

On Fri, May 22, 2020 at 11:04:34PM +0700, Антон Мидюков wrote:
> >> Подойдёт аппрув от любого члена тим уже завершившего join.
> > Я тоже так думал, но после моего аппрува таск остался в EPERM,
> > а должен был перейти в TESTED (он был без --commit).
> Я не замечал ни разу изменение статуса задания после аппрува.
> Делаю повторный аппрув, чтобы получить в ответ, что аппрув уже выдан :-)

Статус задания (не считая -> NEW) меняют run, abort, rm,
успешная сборка или облом, насколько понимаю; approve
сам по себе лишь выставляет флажок на подзадание.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Michael Shigorin]

On Fri, May 22, 2020 at 05:35:33PM +0300, Gleb Fotengauer-Malinovskiy wrote:
> > И какой-то добрый человек или бот добавил меня в acl
> > $ ssh build.alt acl sisyphus oksh show
> > oksh	cheusov @everybody
> Сборочница делает это автоматически для всех новых пакетов.

Видишь, она местами добрая. :)

> При этом ACL для кандидатов не действует.

Давай всё-таки наконец закроем баг 2014 года успешно:
http://bugzilla.altlinux.org/30499

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))

[Ivan A. Melnikov]

On Fri, May 22, 2020 at 06:33:15PM +0300, Andrey Savchenko wrote:
> On Fri, 22 May 2020 17:35:33 +0300 Gleb Fotengauer-Malinovskiy
> wrote:
> > On Fri, May 22, 2020 at 04:57:09PM +0300, Aleksey Cheusov wrote:
> > > 21.05.2020, 23:41, "Andrey Savchenko" <bircoph@altlinux.org>:
> > > > On Thu, 21 May 2020 22:30:09 +0300 Aleksey Cheusov wrote:
> > > >>  Спасибо. task #252044. Там же.
> > > >
> > > > Я больше проблем не вижу, но похоже, что моего аппрува
> > > > недостаточно. Я не особо понимаю, как работает этот механизм для
> > > > мейнтенеров, которые ещё не разрешены на сборочнице. Возможно,
> > > > нужен approve от mike или админов сборочницы.
> > 
> > Подойдёт аппрув от любого члена тим уже завершившего join.
> 
> Я тоже так думал, но после моего аппрува таск остался в EPERM,
> а должен был перейти в TESTED (он был без --commit).

Аппрув не меняет состояние таски -- команда сама по себе
только записывает, что аппрув был. Его, кстати, можно увидеть
в task show. Реально же он будет проверен и применён
когда кто-нибудь запустит таску и она снова дойдёт до
проверки ACL. Такая реализация, например, позволяет
аппрувить таски "на лету", например непосредственно
во время сборки.

-- 
  wbr,
    iv m.