dev-libs/beignet: fix broken prefix support
authorMarek Szuba <marecki@gentoo.org>
Fri, 31 Aug 2018 13:07:36 +0000 (14:07 +0100)
committerMarek Szuba <marecki@gentoo.org>
Fri, 31 Aug 2018 13:07:55 +0000 (14:07 +0100)
With VENDOR_DIR as set in the ebuild beginning with a slash,
${EPREFIX%/}/${VENDOR_DIR} ends up containing // regardless of the
value of EPREFIX. On a less serious note, it isn't necessary to strip
the trailing slash from EPREFIX because according to specifications
there shouldn't be one; it seems the author of the patch confused it
with e.g. ED.

Closes: https://bugs.gentoo.org/663818
Package-Manager: Portage-2.3.40, Repoman-2.3.9

dev-libs/beignet/beignet-1.3.2-r1.ebuild

index 7975dc31d3ce0e10e1b18fb95c2b9825ad40bff7..aabc386a74b16fda4b27011836dfa5044652cb1e 100644 (file)
@@ -82,7 +82,7 @@ multilib_src_configure() {
        VENDOR_DIR="/usr/$(get_libdir)/OpenCL/vendors/${PN}"
 
        local mycmakeargs=(
-               -DCMAKE_INSTALL_PREFIX="${EPREFIX%/}/${VENDOR_DIR}"
+               -DCMAKE_INSTALL_PREFIX="${EPREFIX}${VENDOR_DIR}"
                -DOCLICD_COMPAT=$(usex ocl-icd)
                $(usex ocl20 "" "-DENABLE_OPENCL_20=OFF")
        )
@@ -96,7 +96,7 @@ multilib_src_install() {
        cmake-utils_src_install
 
        insinto /etc/OpenCL/vendors/
-       echo "${EPREFIX%/}/${VENDOR_DIR}/lib/${PN}/libcl.so" > "${PN}-${ABI}.icd" || die "Failed to generate ICD file"
+       echo "${EPREFIX}${VENDOR_DIR}/lib/${PN}/libcl.so" > "${PN}-${ABI}.icd" || die "Failed to generate ICD file"
        doins "${PN}-${ABI}.icd"
 
        dosym "lib/${PN}/libcl.so" "${VENDOR_DIR}"/libOpenCL.so.1