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

Andrey Savchenko bircoph на altlinux.org
Чт Май 21 11:58:08 MSK 2020


Добрый день!

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
----------- следующая часть -----------
Было удалено вложение не в текстовом формате...
Имя     : отсутствует
Тип     : application/pgp-signature
Размер  : 833 байтов
Описание: отсутствует
Url     : <http://lists.altlinux.org/pipermail/devel/attachments/20200521/eda383d9/attachment.bin>


Подробная информация о списке рассылки Devel