Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 09 Jun 2017 11:50:56 +0000
From:      bugzilla-noreply@freebsd.org
To:        freebsd-ports-bugs@FreeBSD.org
Subject:   [Bug 219609] [NEW PORT] science/tensorflow: Computation using data flow graphs for scalable machine learning
Message-ID:  <bug-219609-13-TlsewvoKyz@https.bugs.freebsd.org/bugzilla/>
In-Reply-To: <bug-219609-13@https.bugs.freebsd.org/bugzilla/>
References:  <bug-219609-13@https.bugs.freebsd.org/bugzilla/>

next in thread | previous in thread | raw e-mail | index | archive | help
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=3D219609

--- Comment #13 from Mathieu Arnold <mat@FreeBSD.org> ---
The python_CMD definition is not needed, remove it.

post-patch and post-configure: Never call SED directly, we have REINPLACE_C=
MD
for that.

The procfs mount must be removed. Packages are build as regular user, no
regular user can ever mount anything. Also, they are built in jails where
mounting is not allowed, so even if root was building, it could not mount
either.

Please merge do-configure and post-configure, there is no reason to have two
targets when only one is needed.

In do-configure, you seem to be setting enviroment variables without using
SETENV, please add it.

do-build, same, use SETENV.

The do-install target is a bit strange, it looks like you are "installing"
files by moving them with MV. you should instead be copying them with
COPYTREE_SHARE.

--=20
You are receiving this mail because:
You are the assignee for the bug.=



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