@stamourv a new weird difference: https://plt.eecs.northwestern.edu/release-pkg-build/server/built/deps/dssl2.txt is only happening on the release build

also, css passed on HEAD, still failed on the release

@samth: Re dssl2: I see that @tov made changes to the deps between the pre-release pkg-build run and the HEAD pkg-build run. Let’s see what the next pre-release pkg-build says.

ah ok

@samth: Re css: Wargrey got back to me, looking more into it is on my to-do list.

also, I also ran the turnstile tests, they passed, I took it off the checklist

If you’d like to investigate too, I can forward you the info.

I won’t have time

But I ran some tests and checked them off

@stamourv indeed I added deps yesterday. Did I do something wrong?

@tov: I don’t think so? A pre-release pkg-build (see link from samth above) that happened before your changes had some deps errors. Your changes may have fixed them.

But we’ll have to wait for the next pre-release pkg-build to know for sure.

Oh, okay, I think I fixed them. Cheers!

:)

Would it be bad to spawn a thread in a plumber flush callback that frees some resource and not wait on the thread before removing the flush callback registration? Could racket exit after the plumber is flushed but before the background thread finishes?