Re: [newbies] orion
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, как по именам, так и по содержанию :) -- С уважением, Владимир Селезнев ___ devel-newbies mailing list devel-newbies@lists.altlinux.org https://lists.altlinux.org/mailman/listinfo/devel-newbies
Re: [newbies] orion
On Tue, Jan 23, 2018 at 11:38:13AM +0300, Maxim Voronov wrote: > Обновил репозиторий согласно рекомендациям. > > On 23 Jan 2018 9:20 am, "Sergey V Turchin" 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 mailing list devel-newbies@lists.altlinux.org https://lists.altlinux.org/mailman/listinfo/devel-newbies
Re: [newbies] orion
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 ___ devel-newbies mailing list devel-newbies@lists.altlinux.org https://lists.altlinux.org/mailman/listinfo/devel-newbies
Re: [newbies] orion
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, сбить с толку может. -- С уважением, Владимир Селезнев ___ devel-newbies mailing list devel-newbies@lists.altlinux.org https://lists.altlinux.org/mailman/listinfo/devel-newbies
Re: [newbies] orion
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 ___ devel-newbies mailing list devel-newbies@lists.altlinux.org https://lists.altlinux.org/mailman/listinfo/devel-newbies