Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
Whoops, that’s my error. bazel does respect that. Here’s a working Portfile for py-dm-tree that could be used to simplify the py-tensorflow Portfile in a few spots: > # -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; > c-basic-offset: 4 -*-

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Ryan Schmidt
And that's a bug that they could fix by respecting the value of the TMPDIR environment variable. > On Jul 7, 2020, at 17:21, Steven Smith wrote: > > “The wonderful bazel build system” expects to see ./tmp, not ./.tmp. > >> On Jul 7, 2020, at 3:45 PM, Ryan Schmidt wrote: >> >> On Jul 7, 2020,

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
“The wonderful bazel build system” expects to see ./tmp, not ./.tmp. > On Jul 7, 2020, at 3:45 PM, Ryan Schmidt wrote: > > On Jul 7, 2020, at 12:53, Steven Smith wrote: > >> Fixed the issue: requires creation of TMPDIR >> >> xinstall -d ${worksrcpath}/tmp > > MacPorts already sets the TMPDIR

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Ryan Schmidt
On Jul 7, 2020, at 14:15, Steven Smith wrote: > New issue `port destroot` tries to build with bazel again, and this breaks > because of permissions issues. > > Is there a simple way to just copy the stuff it already built into destroot? That's what one would expect "make install" (or

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Ryan Schmidt
On Jul 7, 2020, at 12:53, Steven Smith wrote: > Fixed the issue: requires creation of TMPDIR > > xinstall -d ${worksrcpath}/tmp MacPorts already sets the TMPDIR environment variable to ${workpath}/.tmp and creates that directory for you. Sounds like this build system needs to be fixed to use

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
Whoops, please disregard this comment—imports work with the correct python version. > On Jul 7, 2020, at 3:15 PM, Steven Smith wrote: > > Also, I’m not sure what’s causing this python import breakage of the bazel > build: smime.p7s Description: S/MIME cryptographic signature

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
New issue `port destroot` tries to build with bazel again, and this breaks because of permissions issues. Is there a simple way to just copy the stuff it already built into destroot? I’ve tried the standard destroot commands from the python port group and the pip commands used in

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
Fixed the issue: requires creation of TMPDIR xinstall -d ${worksrcpath}/tmp Thanks again. Steve > On Jul 7, 2020, at 7:31 AM, Steven Smith wrote: > > Thanks to your help, I’ve been able to edit setup.py akin to what you’ve done > in py-tensorflow and have solved the long timeout issue, but

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Ryan Schmidt
On Jul 7, 2020, at 06:17, Steven Smith wrote: > On Jul 6, 2020, at 11:54 AM, Ryan Schmidt wrote: > >> Usually the defaults of the github portgroup should be sufficient. But >> you're mixing it with the python portgroup, and it's possible that the >> python portgroup is overwriting the github

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
Thanks to your help, I’ve been able to edit setup.py akin to what you’ve done in py-tensorflow and have solved the long timeout issue, but not the underlying cause of the hang. The command: > sudo port build py37-dm-tree fails to build (with no reported errors, except that it fails). However,

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-07 Thread Steven Smith
> On Jul 6, 2020, at 11:54 AM, Ryan Schmidt wrote: > > I can't provide any advice on bazel, but I can give a few comments on the > portfile: > > >>> # -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; >>> c-basic-offset: 4 -*- vim:fenc=utf-8:ft=tcl:et:sw=4:ts=4:sts=4 >>>

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-06 Thread Steven Smith
Whoops yes it does. Thanks for the correction. I’ll go try the settings you’re using with “the wonderful bazel build system ...” Steve > On Jul 6, 2020, at 12:35, Christopher Jones wrote: > > yes, it does. See > >

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-06 Thread Christopher Jones
> On 6 Jul 2020, at 5:22 pm, Steven Smith wrote: > > I am not using --max_idle_secs=60. > > Also, neither is port py-tensorflow yes, it does. See https://github.com/macports/macports-ports/blob/master/python/py-tensorflow/Portfile#L187

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-06 Thread Steven Smith
I am not using --max_idle_secs=60. Also, neither is port py-tensorflow nor the repo tensorflow/tensorflow itself, nor any other MacPorts repo, and these all build fine without this option: https://github.com/macports/macports-ports/blob/master/python/py-tensorflow/Portfile

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-06 Thread Christopher Jones
Hi, Are you using this option bazel --max_idle_secs=60 when building with bazel ? Bazel builds start an underlying ’server’ process, that by default hangs around for 3600 secs (yes, 1hour) before disappearing. Without the above your build will hang around waiting for this to happen… Chris

Re: Deepmind Tree Bazel Build Portfile Help Request

2020-07-06 Thread Ryan Schmidt
I can't provide any advice on bazel, but I can give a few comments on the portfile: >> # -*- coding: utf-8; mode: tcl; tab-width: 4; indent-tabs-mode: nil; >> c-basic-offset: 4 -*- vim:fenc=utf-8:ft=tcl:et:sw=4:ts=4:sts=4 >> >> PortSystem 1.0 >> PortGroup compilers 1.0 Are

Deepmind Tree Bazel Build Portfile Help Request

2020-07-05 Thread Steven Smith
I’m preparing a PR for the Google Magenta project >, and am running into this issue for the “dm-tree” dependent package with the bazel build. The command `port -dv test py37-dm-tree` hangs during the bazel build. I can’t even