ALT Linux Team development discussions
 help / color / mirror / Atom feed
From: Andrey Savchenko <bircoph@altlinux.org>
To: ALT Linux Team development discussions <devel@lists.altlinux.org>
Subject: Re: [devel] Новый пакет для сизифа: oksh (Portable OpenBSD ksh(1))
Date: Thu, 21 May 2020 23:41:46 +0300
Message-ID: <20200521234146.745c8e63a96efaa7845c1fd7@altlinux.org> (raw)
In-Reply-To: <399021590088952@mail.yandex.ru>

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

  reply	other threads:[~2020-05-21 20:41 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-05-20 21:44 Aleksey Cheusov
2020-05-21  8:58 ` Andrey Savchenko
2020-05-21 19:30   ` Aleksey Cheusov
2020-05-21 20:41     ` Andrey Savchenko [this message]
2020-05-22 13:57       ` Aleksey Cheusov
2020-05-22 14:35         ` Gleb Fotengauer-Malinovskiy
2020-05-22 15:33           ` Andrey Savchenko
2020-05-22 16:04             ` Антон Мидюков
2020-05-22 16:13               ` Michael Shigorin
2020-05-22 20:17             ` Ivan A. Melnikov
2020-05-22 16:15           ` Michael Shigorin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200521234146.745c8e63a96efaa7845c1fd7@altlinux.org \
    --to=bircoph@altlinux.org \
    --cc=devel@lists.altlinux.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

ALT Linux Team development discussions

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel/0 devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 devel devel/ http://lore.altlinux.org/devel \
		devel@altlinux.org devel@altlinux.ru devel@lists.altlinux.org devel@lists.altlinux.ru devel@linux.iplabs.ru mandrake-russian@linuxteam.iplabs.ru sisyphus@linuxteam.iplabs.ru
	public-inbox-index devel

Example config snippet for mirrors.
Newsgroup available over NNTP:
	nntp://lore.altlinux.org/org.altlinux.lists.devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git