[newbies] orion

Vladimir D. Seleznev vseleznv на altlinux.org
Сб Янв 27 11:47:19 MSK 2018


On Tue, Jan 23, 2018 at 11:38:13AM +0300, Maxim Voronov wrote:
> Обновил репозиторий согласно рекомендациям.
> 
> On 23 Jan 2018 9:20 am, "Sergey V Turchin" <zerg на altlinux.org> wrote:
> 
> > On Monday, 22 January 2018 20:19:06 MSK Vladimir D wrote:
> > > Доброго времени суток!
> > >
> > > Поревьюил этот репозиторий [1]. Пара замечаний/вопросов.
> > >
> > > 1. Почему решили собирать версию 1.5.1? В репозитории есть 1.6.1, не
> > > лучше ли собрать её?
> > Это моя идея. После отправки в p8 в Сизиф соберем новую, т.к. зависит на
> > версию Qt.
> >
> > > 2. Шапка текущего спека намешана, лучше привести его к виду как здесь [2]
> > > (поле Packager добавлять не обязательно, и в общем случае не нужно).
> > Или можно на мои спеки смотреть. Там похоже.
> >
> > > 3. В текущем спеке используются лишние конструкции, рекомендую прогнать
> > > его с помощью cleanup_spec из пакета rpm-utils.
> > Так же cleanup_spec полезно делать сразу, когда берете spec-рыбу из другого
> > дистрибутива.
> >
> > > 4. Здесь в поле Source используется путь до тарбола, находящимся на
> > > внешнем ресурсе. Мы так не делаем, когда собираем пакеты из
> > > gear-репозитория: всё-равно мы пакуем свои тарболы, сгенерённые из тега.
> > Да. Интернет-путь -- анахронизм.
> >
> > > По этой же причине мы пакуем тарболы не в tar.gz, а в tar: чтобы
> > > избежать коллизий с апстримными тарболами, плюс смысла в этом большого
> > > нет, т.к. sourcerpm и так сжимается гораздо более сильным сжатием.
> > Скорее, второе. Еще и жать сжатое всё равно будет.
> >
> > > Лучше переделать схему сжатия на tar, и убрать url из поля Source.
> > Да.

Если спек был взят явно из другого проекта, то надо бы оставить как
минимум упоминание об этом с указанием откуда. Но это на усмотрение
ментора.

-- 
   С уважением,
   Владимир Селезнев


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