devel@ where you _can_ ask
 help / color / mirror / Atom feed
* [newbies] orion
@ 2018-01-22 17:19 Vladimir D. Seleznev
  2018-01-22 17:48 ` Michael Shigorin
    0 siblings, 2 replies; 6+ messages in thread
From: Vladimir D. Seleznev @ 2018-01-22 17:19 UTC (permalink / raw)
  To: devel-newbies, mvoronov, zerg

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

Поревьюил этот репозиторий [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

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [newbies] orion
  2018-01-22 17:19 [newbies] orion Vladimir D. Seleznev
@ 2018-01-22 17:48 ` Michael Shigorin
  2018-01-22 18:05   ` Vladimir D. Seleznev
    1 sibling, 1 reply; 6+ messages in thread
From: Michael Shigorin @ 2018-01-22 17:48 UTC (permalink / raw)
  To: devel-newbies

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [newbies] orion
  2018-01-22 17:48 ` Michael Shigorin
@ 2018-01-22 18:05   ` Vladimir D. Seleznev
  2018-01-22 18:05     ` Michael Shigorin
  0 siblings, 1 reply; 6+ messages in thread
From: Vladimir D. Seleznev @ 2018-01-22 18:05 UTC (permalink / raw)
  To: devel@ where you _can_ ask

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, сбить с толку может.

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [newbies] orion
  2018-01-22 18:05   ` Vladimir D. Seleznev
@ 2018-01-22 18:05     ` Michael Shigorin
  0 siblings, 0 replies; 6+ messages in thread
From: Michael Shigorin @ 2018-01-22 18:05 UTC (permalink / raw)
  To: devel-newbies

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [newbies] orion
  @ 2018-01-27  8:47     ` Vladimir D. Seleznev
    0 siblings, 1 reply; 6+ messages in thread
From: Vladimir D. Seleznev @ 2018-01-27  8:47 UTC (permalink / raw)
  To: Maxim Voronov; +Cc: darktemplar, mvoronov, devel-newbies, Sergey V Turchin

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

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

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [newbies] orion
  @ 2018-01-31 17:34         ` Vladimir D. Seleznev
  0 siblings, 0 replies; 6+ messages in thread
From: Vladimir D. Seleznev @ 2018-01-31 17:34 UTC (permalink / raw)
  To: Sergey V Turchin; +Cc: mvoronov, darktemplar, devel-newbies

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, как по именам, так и по содержанию :)

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


^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2018-01-31 17:34 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-01-22 17:19 [newbies] orion Vladimir D. Seleznev
2018-01-22 17:48 ` Michael Shigorin
2018-01-22 18:05   ` Vladimir D. Seleznev
2018-01-22 18:05     ` Michael Shigorin
2018-01-27  8:47     ` Vladimir D. Seleznev
2018-01-31 17:34         ` Vladimir D. Seleznev

devel@ where you _can_ ask

This inbox may be cloned and mirrored by anyone:

	git clone --mirror http://lore.altlinux.org/devel-newbies/0 devel-newbies/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-newbies devel-newbies/ http://lore.altlinux.org/devel-newbies \
		devel-newbies@lists.altlinux.org devel-newbies@lists.altlinux.ru devel-newbies@lists.altlinux.com
	public-inbox-index devel-newbies

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


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