ebuild for gentoo: install docs for plugins if USE flag is declared.

[shdown]

Why don’t you modify the existing ebuilds? Why do you only install docs for plugins? Also, I think you should run docompress -x /usr/share/doc/${PF}/plugins/${plugin} for each plugin.

[itspec-ru]

@shdown

1. Why don’t you modify the existing ebuilds? На это есть свои причины. При обновлении системы, например так emerge --update --deep --newuse --verbose @world пакеты с версией 9999 не обновятся. А если присваивать версии r1,r2,r3 ..., то - обновятся. В текущем официальном дереве portage такая практика тоже применяется, например dev-vcs/git-9999.ebuild dev-vcs/git-9999-r1.ebuild dev-vcs/git-9999-r2.ebuild dev-vcs/git-9999-r3.ebuild Добавив ebuild с новой версией я поступил так же как seamus-45 https://github.com/shdown/luastatus/commit/5f40d80cc5a9edf127020d7fcb8ec1cc447aecb0 Так как Вы приняли его pull request, то я посчитал что такой подход предпочтителен. Хотя в дистрибутиве действительно можно оставить только один единственный вариант с версией 9999. Вам решать, я бы оставил только одну обновленную версию 9999, даже удалил бы luastatus-0.1.0-r1.ebuild

2. Why do you only install docs for plugins? в src_install() вызывается default, а она выполняет стандартные действия, в том числе и действия по установке документации. Поэтому в любом случае (даже без USE флага 'doc') в /usr/share/doc/luastyatus-${PF} будет установлен README.md. файл LICENCE.txt не нужен. Тип лицензии прописан в самом ebuild'е, а текст лицензии содержитя в /usr/portage/licenses/LGPL-3. Что касается содержимого каталога DOCS, то это всё для разработчиков а не для пользователя. По этому считаю что будет лишним устанавливать это. Я упустил из вида доки из каталога barlibs. Исправлю

3. I think you should run docompress -x /usr/share/doc/${PF}/plugins/${plugin} for each plugin. В portage текстовые файлы, в том числе и .md упаковываются bz2, это общее правило. Файлы .lua я решил не упаковывать, так как это всё же программы хотя и примеры.

[shdown]

На это есть свои причины.

Хорошо, не знал. А то, что seamus-45 переименовал файл, я и не заметил. Вам стоит сделать так же — старые версии всё равно хранятся в git’е.

Я упустил из вида доки из каталога barlibs. Исправлю

Да, я именно это и имел в виду.

В portage текстовые файлы, в том числе и .md упаковываются bz2, это общее правило. Файлы .lua я решил не упаковывать, так как это всё же программы хотя и примеры.

Хорошо.

[itspec-ru]

Не понял на счет версий "Вам стоит сделать так же". В смысле присвоить новый номер версии?

[shdown]

Не понял на счет версий "Вам стоит сделать так же". В смысле присвоить новый номер версии?

Удалить старый ebuild (-r1) а этот (-r2) оставить, как есть (в смысле имени).

[itspec-ru]

Я тут попутно наткнулся на одну проблемку. Файлы плагинов .so компилятся и устанавливаются в систему все независимо от USE флагов, т. е. опции -DBUILDPLUGIN судя по всему игнорируются. Если это так задумано, то USE флаги ${PLUGINS} не нужны, в противном случае нужно разбираться.

[shdown]

Файлы плагинов .so компилятся и устанавливаются в систему все независимо от USE флагов, т. е. опции -DBUILDPLUGIN судя по всему игнорируются.

Нет, так не задумано. И судя по всему, так быть просто не должно. Можно добавить echo ${mycmakeargs[*]}, или что там для дебага в ebuild’ах принято, в src_configure().

[itspec-ru]

Всё нормально, забыл про плюсики в переменной PLUGINS. Т. е. все флаги luastatusplugins по умолчанию вкючены, чтобы что то не устанавливалось нужно флаг явно выключить. Не типичное поведение, хотя в данном случае абсолютно правильное. Я по привычке проверял включением нужных флагов, для ${BARLIBS} всё как и ожидалось, а ${PLUGINS} можно только выключать.

[shdown]

Merged!