Missing tdebase application handbooks #293

Basic information

TDE version: R14.1.0 [DEVELOPMENT]
Distribution: Gentoo, portage-3.0.38.1-r3, trinity-official overlay ~ 2022-11-04
Hardware: amd64

Description

Handbooks for the tdebase applications (e.g. konqueror, konsole) do not get installed. I tried playing around with handbook use flag and modified ebuilds with TRINITY_HANDBOOK="optional" but couldn't get anything to work in the short amount of time I played around with it.

Unfortunately the English language documentation is not collected separately for each package, since the CMakeList files have not been rewritten for this. As a solution to the problem, add another tdebase-doc package, with the following content:

# Copyright 2023 The Trinity Desktop Project
# Distributed under the terms of the GNU General Public License v2

EAPI="8"

TRINITY_MODULE_NAME="tdebase"
TRINITY_MODULE_TYPE="core"
inherit trinity-meta-2

DESCRIPTION="Documentation for tdebase-derived packages"
if [[ ${PV} != *9999* ]] ; then
	KEYWORDS="~amd64 ~x86"
fi

RDEPEND="
	~trinity-base/khelpcenter-${PV}
"

src_prepare() {
	sed 's/tde_conditional_add_project_docs( BUILD_DOC )/tde_conditional_add_subdirectory( BUILD_DOC doc )/' -i "${S}/CMakeLists.txt"
	trinity-meta-2_src_prepare
}

src_install() {
	trinity-meta-2_src_install
	rm -rf "${D}/usr/trinity/14/bin"
	rm -rf "${D}//usr/trinity/14/share/autostart"
}

Unfortunately the English language documentation is not collected separately for each package, since the CMakeList files have not been rewritten for this.

Actually, it is possible: the directories in doc/* subdir are added auto-magically by cmake, so if you remove (don't extract) the docs for all packages but current and pass -DBUILD_DOC to cmake it will build documentation only for the current package.

Will require some tweaks in eclass, but it shouldn't be too hard to implement...

Also as for now all tdebase documentation is already installed by trinity-base/khelpcenter, so the trinity-base/tdebase-doc will fail due to file collisions.