[newbies] orion

Vladimir D. Seleznev vseleznv на altlinux.org
Пн Янв 22 20:19:06 MSK 2018


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

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

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


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