Добрый день! 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 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