[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: How to block tests on hydra
From: |
Noam Postavsky |
Subject: |
Re: How to block tests on hydra |
Date: |
Tue, 11 Jul 2017 07:45:27 -0400 |
On Tue, Jul 11, 2017 at 3:20 AM, Michael Albinus <address@hidden> wrote:
>>
>> I'm surprised it does anything at all actually, isn't the (add-to-list
>> 'buffers ...) call a bug? Should be using `push' I think.
>>
>> - (add-to-list 'buffers (generate-new-buffer "foo")))
>> + (push (generate-new-buffer "foo") buffers))
>
> Why that? `add-to-list' is as good as `push' in this case. I haven't seen a
> problem with that. If you run in edebug, you'll see that `buffers' keeps
> all process buffers.
But you have lexical-binding:t set at the top of the file, and
`buffer' is a let-bound variable, whereas `add-to-list' operates on
dynamic variables. Hmm, looking at the disassembly it seems that
add-to-list's compiler macro fixed up the problem, although I thought
it should give a compile warning in this situation.
>
>> I noticed you added a with-timeout on that test, but it doesn't seem
>> to be working.
>
> The timeout is a self-defense. And it doesn't trigger at least for me,
> because (I believe) the test case is working properly now, and finishes
> in time.
What about this one:
http://hydra.nixos.org/eval/1373949
https://nix-cache.s3.amazonaws.com/log/v7ndmrhdhjw76v9mzghjyijnsmw2npl8-emacs-coverage-unknown.drv
>
>> By the way, I hit the "`tramp-test36-asynchronous-requests' timed out"
>> message when running locally in an -O0 build, although it succeeds
>> with an -O2 build. Maybe I just have a weak CPU.
>
> How does it make a difference? Is an asynchronous process to be intended
> to run in another thread, on another processor?
I don't know why it makes a difference.
- How to block tests on hydra, Michael Albinus, 2017/07/01
- Re: How to block tests on hydra, Noam Postavsky, 2017/07/10
- Re: How to block tests on hydra, Michael Albinus, 2017/07/11
- Re: How to block tests on hydra,
Noam Postavsky <=
- Re: How to block tests on hydra, Michael Albinus, 2017/07/12
- Re: How to block tests on hydra, Noam Postavsky, 2017/07/13
- Re: How to block tests on hydra, Michael Albinus, 2017/07/14
- Re: How to block tests on hydra, Noam Postavsky, 2017/07/14
- Re: How to block tests on hydra, Michael Albinus, 2017/07/15