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