Skip site navigation (1)Skip section navigation (2)
Date:      Sun, 14 Feb 2021 14:40:30 +0100
From:      Christoph Moench-Tegeder <cmt@burggraben.net>
To:        "Jason W. Bacon" <jwb@freebsd.org>
Cc:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   Re: svn commit: r565224 - in head/lang/pocl: . files
Message-ID:  <YCkoTsUm1QeWBzO8@elch.exwg.net>
In-Reply-To: <202102141304.11ED4BTl046406@repo.freebsd.org>
References:  <202102141304.11ED4BTl046406@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
## Jason W. Bacon (jwb@FreeBSD.org):

> New Revision: 565224
> URL: https://svnweb.freebsd.org/changeset/ports/565224
> 
> Log:
>   lang/pocl: Update hwloc dependency to hwloc2

The intention is great, but the execution... has problems.

> -PORTNAME=			pocl
> +PORTNAME=		pocl

First, while I'm all in favor of consistent white space, mixing this
with the stated (per commit message) change makes the diff harder
to read and "svn annotate" (and similar tools) harder to use later on.

>  BUILD_DEPENDS=	llvm${LLVM_VERSION}>=0:devel/llvm${LLVM_VERSION} \
> -				opencl>=2.2:devel/opencl
> -LIB_DEPENDS=	libhwloc.so:devel/hwloc \
> -				libOpenCL.so:devel/ocl-icd
> -RUN_DEPENDS=	${BUILD_DEPENDS}
> +			opencl>=2.2:devel/opencl
> +LIB_DEPENDS=	libhwloc.so.15:devel/hwloc2 libOpenCL.so:devel/ocl-icd
> +RUN_DEPENDS=	libhwloc.so.15:devel/hwloc2 libOpenCL.so:devel/ocl-icd

And this is plain wrong. libhwloc.so.15 and libOpenCL.so are not
"executables or files this port depends on" - they're libraries and
belong into LIB_DEPENDS only. At the same time, you dropped that
llvm from RUN_DEPENDS - now that I look at it closely, I have a hunch
that the previous "RUN_DEPENDS= ${BUILD_DEPENDS}" wasn't right either
(would this really need sphinx, cmake and ninja as RUN_DEPENDS?),
but none of that is even mentioned in the commit message and I
cannot tell if this was intentional or an accident.

Anyways, the build bombs out in run-depends "pocl-1.6 depends on
executable: libhwloc.so.15 - not found", which is why I looked at this
in the first place.

Regards,
Christoph

-- 
Spare Space



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?YCkoTsUm1QeWBzO8>