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

[Andrey Savchenko <bircoph@altlinux.org>]

[-- 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 --]