* Re: [newbies] [Bug 39461] [3.4] join bne@ @ 2021-04-05 21:07 ` Andrey Savchenko 2021-12-08 23:00 ` Vladimir D. Seleznev 0 siblings, 2 replies; 5+ messages in thread From: Andrey Savchenko @ 2021-04-05 21:07 UTC (permalink / raw) To: Николай Бурыкин Cc: devel-newbies [-- Attachment #1: Type: text/plain, Size: 3634 bytes --] Добрый вечер! On Mon, 5 Apr 2021 13:16:47 +0300 Николай Бурыкин wrote: > 04.04.2021 14:35, Andrey Savchenko пишет: > > On Sun, 4 Apr 2021 13:49:03 +0300 Andrey Savchenko wrote: > >> On Sun, 4 Apr 2021 12:07:32 +0300 Николай Бурыкин wrote: > >>> Freelan успешно собрался > >>> http://git.altlinux.org/tasks/index/sisyphus/eperm/268926/ > > Выглядит хорошо. Пропустил. > > > > Есть небольшое косметическое замечание: перед %changelog желательно > > оставлять пустую строку; но это не обязательно, больше дело вкуса. > > > Спасибо. > По поводу %changelog досадная опечатка. Исправлю. > Успешно, без замечаний вчера собрался edbrowse > http://git.altlinux.org/tasks/index/sisyphus/eperm/268951/ > Если будет время, посмотрите тоже пожалуйста. Есть замечания по упаковке документации: 1) Не следует добавлять зависимость на основной пакет: Requires: %name = %version-%release Документацию вполне можно установить и читать и без него. Это не самое распространённое действие, но вполне допустимое: например, пользователь может захотеть ознакомиться с документацией к пакету перед его установкой, чтоб решить, нужно ли его вовсе устанавливать. 2) Документацию следует делать noarch: BuildArch: noarch 3) Примеры (*.ebrc) лучше установить в отдельный пакет examples. Он тоже должен быть noarch в данном случае. Так же смотрите рекомендации по упаковке документации и примеров: https://www.altlinux.org/Package_Splitting#Документация_и_примеры В общем-то, все вышеуказанные замечания там описаны. Из правила noarch в редких случаях возможны исключения: например, когда примеры архитектурно-зависимы. Кроме того у нас были случаи, когда документация на разных архитектурах генерировалась разная — но это очень редкая ситуация. Кроме того, есть замечание по sed: в аргументе подстановки лучше использовать макрос %_docdir, чем непосредственно указывать путь: sed -i "s|/usr/share/doc/%name|%_docdir/%name-%version|" CMakeLists.txt Результат сборки будет тот же, но на случай, если в будущем кому-то приспичит поменять /usr/share/doc на что-то ещё, будет проще на уровне дистрибутива всё это исправлять. В остальном всё хорошо. Для исправленного варианта тег пересоздайте без инкремента, т.к. эта версия в Сизиф ещё не попала. Best regards, Andrew Savchenko [-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <88b12e99-7752-18ba-43c9-3b20c1728188@basealt.ru>]
* Re: [newbies] [Bug 39461] [3.4] join bne@ @ 2021-04-09 16:20 ` Andrey Savchenko 2021-12-08 22:51 ` Aleksey Cheusov 1 sibling, 0 replies; 5+ messages in thread From: Andrey Savchenko @ 2021-04-09 16:20 UTC (permalink / raw) To: Николай Бурыкин Cc: devel-newbies [-- Attachment #1: Type: text/plain, Size: 374 bytes --] Добрый вечер! On Fri, 9 Apr 2021 15:18:04 +0300 Николай Бурыкин wrote: > Замечания вроде все исправил. Отправил в новую сборку > http://git.altlinux.org/tasks/index/sisyphus/eperm/269289/ Вроде бы всё хорошо. Пропустил задание. Best regards, Andrew Savchenko [-- Attachment #2: Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [newbies] [Bug 39461] [3.4] join bne@ 2021-04-09 16:20 ` Andrey Savchenko @ 2021-12-08 22:51 ` Aleksey Cheusov 1 sibling, 0 replies; 5+ messages in thread From: Aleksey Cheusov @ 2021-12-08 22:51 UTC (permalink / raw) To: Николай Бурыкин Cc: Andrey Savchenko, devel@ where you _can_ ask > Добрый день. >> Есть замечания по упаковке документации: >> >> 1) Не следует добавлять зависимость на основной пакет: >> Requires: %name = %version-%release Если это правило обязательно и принимается всеми разработчиками, было бы круто падать при попытке собрать такое. Через несколько итераций и волн стонов, весь дистр стал бы в этом смысле единообразным. > Интересно, мне казалось я перечитал все статьи, касающиеся сборки > пакетов на вики. > Тем не менее > https://www.altlinux.org/Package_Splitting#Документация_и_примеры > я как-то пропустил. Вообще на неё не натыкался. За неё отдельное > спасибо. Ты такой не один :-) ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [newbies] [Bug 39461] [3.4] join bne@ 2021-04-05 21:07 ` [newbies] [Bug 39461] [3.4] join bne@ Andrey Savchenko @ 2021-12-08 23:00 ` Vladimir D. Seleznev 2021-12-08 23:21 ` Dmitry V. Levin 1 sibling, 1 reply; 5+ messages in thread From: Vladimir D. Seleznev @ 2021-12-08 23:00 UTC (permalink / raw) To: devel@ where you _can_ ask On Tue, Apr 06, 2021 at 12:07:25AM +0300, Andrey Savchenko wrote: > Добрый вечер! > > On Mon, 5 Apr 2021 13:16:47 +0300 Николай Бурыкин wrote: > > 04.04.2021 14:35, Andrey Savchenko пишет: > > > On Sun, 4 Apr 2021 13:49:03 +0300 Andrey Savchenko wrote: > > >> On Sun, 4 Apr 2021 12:07:32 +0300 Николай Бурыкин wrote: > > >>> Freelan успешно собрался > > >>> http://git.altlinux.org/tasks/index/sisyphus/eperm/268926/ > > > Выглядит хорошо. Пропустил. > > > > > > Есть небольшое косметическое замечание: перед %changelog желательно > > > оставлять пустую строку; но это не обязательно, больше дело вкуса. > > > > > Спасибо. > > По поводу %changelog досадная опечатка. Исправлю. > > Успешно, без замечаний вчера собрался edbrowse > > http://git.altlinux.org/tasks/index/sisyphus/eperm/268951/ > > Если будет время, посмотрите тоже пожалуйста. > > Есть замечания по упаковке документации: > > 1) Не следует добавлять зависимость на основной пакет: > Requires: %name = %version-%release Ставить документацию отдельно от программы — распространённый use-case, поэтому так делать не надо. Кроме того, при указании зависимости на подпакеты в этом же пакете *не следует* указывать версию (релиз и т.д.), т.к. они вычислятся автоматически (и более строго). Их явная установка может помешать автовычислятору зависимостей. -- WBR, Vladimir D. Seleznev ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [newbies] [Bug 39461] [3.4] join bne@ 2021-12-08 23:00 ` Vladimir D. Seleznev @ 2021-12-08 23:21 ` Dmitry V. Levin 0 siblings, 0 replies; 5+ messages in thread From: Dmitry V. Levin @ 2021-12-08 23:21 UTC (permalink / raw) To: devel-newbies On Thu, Dec 09, 2021 at 02:00:23AM +0300, Vladimir D. Seleznev wrote: > On Tue, Apr 06, 2021 at 12:07:25AM +0300, Andrey Savchenko wrote: [...] > > Есть замечания по упаковке документации: > > > > 1) Не следует добавлять зависимость на основной пакет: > > Requires: %name = %version-%release > > Ставить документацию отдельно от программы — распространённый use-case, > поэтому так делать не надо. По этому вопросу нет общепринятой точки зрения. На мой взгляд, обеспечение совпадения версий подпакетов важнее, чем возможность установить подпакет с документацией отдельно, поэтому в своих пакетах я ставлю такую зависимость даже для документации, которую вроде бы можно было бы установить отдельно. Впрочем, есть и другие способы обеспечить совпадения версий, но они более громоздкие. Была идея на уровне rpm-build при упаковке нескольких подпакетов, не образующих по зависимостям односвязное множество, автоматически создавать пустой подпакет и добавлять во все остальные подпакеты жёсткую зависимость на него, но эта идея так и не была реализована. > Кроме того, при указании зависимости на подпакеты в этом же пакете *не > следует* указывать версию (релиз и т.д.), т.к. они вычислятся > автоматически (и более строго). Их явная установка может помешать > автовычислятору зависимостей. Их явная установка не может помешать автовычислятору, но последний появился уже так давно, что версионирование, указанное вручную, выглядит избыточным. -- ldv ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-12-08 23:21 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-04-05 21:07 ` [newbies] [Bug 39461] [3.4] join bne@ Andrey Savchenko 2021-04-09 16:20 ` Andrey Savchenko 2021-12-08 22:51 ` Aleksey Cheusov 2021-12-08 23:00 ` Vladimir D. Seleznev 2021-12-08 23:21 ` Dmitry V. Levin
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