svn commit: r547197 - head/math/faiss

Piotr Kubaj pkubaj at anongoth.pl
Mon Aug 31 17:29:01 UTC 2020


This commit is wrong.

1. openblas works also on powerpc and powerpc64 - probably others as
well.
2. On architectures where openblas doesn't build, this port will not even start
building to due missing dependency, so setting ONLY_FOR_ARCHS is
pointless.

If you insist on setting ONLY_FOR_ARCHS, could you at least take a list
of architectures where openblas builds from
https://www.freshports.org/math/openblas and set it properly?

On 20-08-31 16:09:03, Yuri Victorovich wrote:
>Author: yuri
>Date: Mon Aug 31 16:09:03 2020
>New Revision: 547197
>URL: https://svnweb.freebsd.org/changeset/ports/547197
>
>Log:
>  math/faiss: Add ONLY_FOR_ARCHS
>
>  Reported by:	linimon
>
>Modified:
>  head/math/faiss/Makefile
>
>Modified: head/math/faiss/Makefile
>==============================================================================
>--- head/math/faiss/Makefile	Mon Aug 31 16:01:10 2020	(r547196)
>+++ head/math/faiss/Makefile	Mon Aug 31 16:09:03 2020	(r547197)
>@@ -12,6 +12,9 @@ COMMENT=	Library for efficient similarity search & clu
> LICENSE=	MIT
> LICENSE_FILE=	${WRKSRC}/LICENSE
>
>+ONLY_FOR_ARCHS=	amd64 i386
>+ONLY_FOR_ARCHS_REASON=	Links with libopenblasp-rN.N.N.so which is installed by math/openblas only on amd64,i386 architectures
>+
> LIB_DEPENDS=	libopenblasp-r0.3.9.so:math/openblas
>
> USES=		cmake compiler:c++11-lang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 898 bytes
Desc: not available
URL: <http://lists.freebsd.org/pipermail/svn-ports-head/attachments/20200831/fc65a4c4/attachment.sig>


More information about the svn-ports-head mailing list