[newbies] orion

[newbies] orion

[Vladimir D. Seleznev]

Доброго времени суток!

Поревьюил этот репозиторий [1]. Пара замечаний/вопросов.

1. Почему решили собирать версию 1.5.1? В репозитории есть 1.6.1, не
лучше ли собрать её?
2. Шапка текущего спека намешана, лучше привести его к виду как здесь [2]
(поле Packager добавлять не обязательно, и в общем случае не нужно).
3. В текущем спеке используются лишние конструкции, рекомендую прогнать
его с помощью cleanup_spec из пакета rpm-utils.
4. Здесь в поле Source используется путь до тарбола, находящимся на
внешнем ресурсе. Мы так не делаем, когда собираем пакеты из
gear-репозитория: всё-равно мы пакуем свои тарболы, сгенерённые из тега.
По этой же причине мы пакуем тарболы не в tar.gz, а в tar: чтобы
избежать коллизий с апстримными тарболами, плюс смысла в этом большого
нет, т.к. sourcerpm и так сжимается гораздо более сильным сжатием. Лучше
переделать схему сжатия на tar, и убрать url из поля Source.

-- 
[1] http://git.altlinux.org/people/mvoronov/packages/orion.git
[2] https://www.altlinux.org/SampleSpecs/empty
[3] http://git.altlinux.org/people/mvoronov/packages/?p=orion.git;a=blob;f=orion.spec;h=76ff892338659b42490c191be9e2a6430bd6168f;hb=27ecc271e8ee6ec28b39cbcb7b9e096e8eff25fd#l8

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

Re: [newbies] orion

[Michael Shigorin]

On Mon, Jan 22, 2018 at 08:19:06PM +0300, Vladimir D. Seleznev wrote:
> 4. Здесь в поле Source используется путь до тарбола,
> находящимся на внешнем ресурсе. Мы так не делаем,

Делаем, делаем -- хотя бы видно, где тарбол взять можно.

> когда собираем пакеты из gear-репозитория:

Ну вот разве что, и то я бы не рассматривал как догму.
Просто если рядом нет какого git url, сбить с толку может.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [newbies] orion

[Vladimir D. Seleznev]

On Mon, Jan 22, 2018 at 08:48:41PM +0300, Michael Shigorin wrote:
> On Mon, Jan 22, 2018 at 08:19:06PM +0300, Vladimir D. Seleznev wrote:
> > 4. Здесь в поле Source используется путь до тарбола,
> > находящимся на внешнем ресурсе. Мы так не делаем,
> 
> Делаем, делаем -- хотя бы видно, где тарбол взять можно.

Вот так:

    Source: https://github.com/alamminsalo/orion/archive/v%{version}/%{name}-%{version}.tar.gz

при сборке из gear'а крайне нежелательно писать. Это наводит на мысли,
что в сорцпакете лежит тарбол, что по ссылке, что неверно. Тем более,
когда тарбол генерится из тега.

Лучше писать

    # https://github.com/alamminsalo/orion/
    Source: %name-%version.tar

и то, в этом случае лучше пакетировать непосредственно из тега с
версией, предоставляемого апстримом:

    tar: v@version@:.

а не в текущем виде:

    tar.gz: .

> > когда собираем пакеты из gear-репозитория:
> 
> Ну вот разве что, и то я бы не рассматривал как догму.
> Просто если рядом нет какого git url, сбить с толку может.

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

Re: [newbies] orion

[Michael Shigorin]

On Mon, Jan 22, 2018 at 09:05:04PM +0300, Vladimir D. Seleznev wrote:
> Лучше писать
> 
>     # https://github.com/alamminsalo/orion/
>     Source: %name-%version.tar
> 
> и то, в этом случае лучше пакетировать непосредственно из тега с
> версией, предоставляемого апстримом:
> 
>     tar: v@version@:.
> 
> а не в текущем виде:
> 
>     tar.gz: .

А вот это однозначно да.

-- 
 ---- WBR, Michael Shigorin / http://altlinux.org
  ------ http://opennet.ru / http://anna-news.info

Re: [newbies] orion

[Vladimir D. Seleznev]

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.
> > Да.

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

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

Re: [newbies] orion

[Vladimir D. Seleznev]

On Mon, Jan 29, 2018 at 09:10:10AM +0300, Sergey V Turchin wrote:
> On Saturday, 27 January 2018 11:47:19 MSK Vladimir D wrote:
> 
> [...]
> > Если спек был взят явно из другого проекта, то надо бы оставить как
> > минимум упоминание об этом с указанием откуда. Но это на усмотрение
> > ментора.
> Я показал местонахожденние рыбы уже после создания спека. ;-)

Тем не менее, используемые в пакете патчи подозрительно похожи на те,
что в Fedora, как по именам, так и по содержанию :)

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