00:39 <cr1901_modern> http://ix.io/1pgT This should fix the build failure... any feedback?

00:39 <whitequark> did you add the roach workaround?

00:42 <cr1901_modern> That's been there since the original commit of the unittest

00:42 <whitequark> ok, lgtm

00:43 <cr1901_modern> Cool, making PR now and _hopefully_ build will be good :)

00:49 <cr1901_modern> https://github.com/m-labs/migen/pull/164 Hope this works

00:49 <whitequark> force build --branch=pull/164/merge migen

00:50 <whitequark> bb-m-labs: force build --branch=pull/164/merge migen

00:50 <bb-m-labs> I'll give a shout when the build finishes

00:50 <bb-m-labs> build forced [ETA 5m29s]

00:53 <bb-m-labs> build #341 of migen is complete: Failure [failed python_unittest] Build details are at http://buildbot.m-labs.hk/builders/migen/builds/341

00:53 <cr1901_modern> hmmm

00:54 <cr1901_modern> I'll test on a machine w/o diamond or ISE

01:00 <cr1901_modern> Oh. Right. Toolchains aren't strings. They're objects.

02:07 <cr1901_modern> bb-m-labs: force build --branch=pull/164/merge migen

02:07 <bb-m-labs> build forced [ETA 5m29s]

02:07 <bb-m-labs> I'll give a shout when the build finishes

02:09 <bb-m-labs> build #342 of migen is complete: Failure [failed python_unittest] Build details are at http://buildbot.m-labs.hk/builders/migen/builds/342

02:10 <cr1901_modern> What the hell?

02:14 <cr1901_modern> whitequark: buildbot doesn't appear to be using the correct commit... there is no "mkdir_and_build(plat, name)" on line 69

06:44 <whitequark> hm

06:53 <sb0> more conda problems?

06:54 <sb0> and yes, ISE is installed on buildbot

07:01 <whitequark> that doesn't seem like a conda problem

07:01 <whitequark> for one, the unittest step doesn't use conda.

07:02 <whitequark> bb-m-labs: force build --branch=pull/164/head migen

07:02 <bb-m-labs> build forced [ETA 5m29s]

07:02 <bb-m-labs> I'll give a shout when the build finishes

07:03 <GitHub-m-labs> [artiq] sbourdeauducq pushed 1 new commit to release-3: https://github.com/m-labs/artiq/commit/f2eafa89fb77ebbf6a8eff95454d1dcb3db472bf

07:03 <GitHub-m-labs> artiq/release-3 f2eafa8 Robert Jordens: manual: deprecate old release...

07:08 <bb-m-labs> build #343 of migen is complete: Success [build successful] Build details are at http://buildbot.m-labs.hk/builders/migen/builds/343

07:42 <bb-m-labs> build #2077 of artiq-board is complete: Failure [failed conda_build] Build details are at http://buildbot.m-labs.hk/builders/artiq-board/builds/2077 blamelist: Robert Jordens <rj@m-labs.hk>

07:42 <bb-m-labs> build #2719 of artiq is complete: Failure [failed] Build details are at http://buildbot.m-labs.hk/builders/artiq/builds/2719 blamelist: Robert Jordens <rj@m-labs.hk>

08:34 <cr1901_modern> whitequark: Maybe I just needed to wait a bit longer for conda to update

08:35 <cr1901_modern> sb0: https://github.com/m-labs/migen/pull/164#discussion_r235261067 Hah, this was my original plan. But I suspected such a method would be too special-purpose to put into a class.

08:36 <whitequark> cr1901_modern: no, I used /head instead of /merge

08:36 <whitequark> not sure why that even matters

08:36 <cr1901_modern> Oh, so you did

08:36 <cr1901_modern> and if you don't know why it matters, I won't either :)

16:38 <GitHub-m-labs> [artiq] hermitdemschoenenleben commented on issue #1195: thank you, now it works!... https://github.com/m-labs/artiq/issues/1195#issuecomment-440731319

16:42 <GitHub-m-labs> [artiq] sbourdeauducq commented on issue #1142: One option for DDS "injection": the dashboard submits an experiment into the scheduler that simply sets the frequency.... https://github.com/m-labs/artiq/issues/1142#issuecomment-440732750

16:43 <GitHub-m-labs> [artiq] sbourdeauducq commented on issue #1142: One option for DDS "injection": the dashboard submits an Experiment into the scheduler that simply sets the frequency.... https://github.com/m-labs/artiq/issues/1142#issuecomment-440732750