diff options
| author |   Sam James <sam@gentoo.org> | 2023-06-26 10:54:06 +0100 |
|---|---|---|
| committer | Sam James <sam@gentoo.org> | 2023-07-02 14:23:39 +0100 |
| commit | b6e2235a3982cad4d74bb7eb49e858835344c3ba (patch) | |
| tree | ed6acf02d771d48044a2cdbe7cac21767808b37a /eclass | |
| parent | gui-libs/gtk: Stabilize 4.10.4 amd64, #909497 (diff) | |
| download | gentoo-b6e2235a3982cad4d74bb7eb49e858835344c3ba.tar.gz gentoo-b6e2235a3982cad4d74bb7eb49e858835344c3ba.tar.bz2 gentoo-b6e2235a3982cad4d74bb7eb49e858835344c3ba.zip | |
cmake.eclass: workaround S=${WORKDIR} creating builddir above ${WORKDIR}
When S=${WORKDIR}, cmake.eclass would create its build directory (if CMAKE_USE_DIR
is unset) above WORKDIR(!) as ${WORKDIR}_build. Creating directories above
WORKDIR is not legal.
Portage has its own bug (bug #889418) in that it doesn't clean up unknown directories
above WORKDIR in PORTAGE_TMPDIR, so combined, you get a problem where "ebuild ... clean" doesn't
actually clean things up at all, and you get very confusing issues if e.g. changing
CC between runs.
The explicit S=WORKDIR check isn't truly needed but it makes explicit our
intent here.
Bug: https://bugs.gentoo.org/889418
Closes: https://bugs.gentoo.org/889420
Signed-off-by: Sam James <sam@gentoo.org>
Diffstat (limited to 'eclass')
| -rw-r--r-- | eclass/cmake.eclass | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/eclass/cmake.eclass b/eclass/cmake.eclass index 1cdbc123a243..d70f2cbf1fac 100644 --- a/eclass/cmake.eclass +++ b/eclass/cmake.eclass @@ -293,6 +293,15 @@ _cmake_check_build_dir() { BUILD_DIR="${CMAKE_USE_DIR}" else : "${BUILD_DIR:=${CMAKE_USE_DIR}_build}" + + # Avoid creating ${WORKDIR}_build (which is above WORKDIR). + # TODO: For EAPI > 8, we should ban S=WORKDIR for CMake. + # See bug #889420. + if [[ ${S} == "${WORKDIR}" && ${BUILD_DIR} == "${WORKDIR}_build" ]] ; then + eqawarn "QA notice: S=WORKDIR is deprecated for cmake.eclass." + eqawarn "Please relocate the sources in src_unpack." + BUILD_DIR="${WORKDIR}"/${P}_build + fi fi einfo "Source directory (CMAKE_USE_DIR): \"${CMAKE_USE_DIR}\"" |