Skip to content
This repository was archived by the owner on Mar 24, 2022. It is now read-only.

Conversation

@acfoltzer
Copy link
Contributor

We do not need a nightly compiler until #254 lands (unless wg-ffi-unwind succeeds before then). This reverts us to a stable compiler for now, and adds a note to the Vmctx interface warning folks about the safety issues with guest callbacks.

While I was at it, I also split off some of the bitrot avoidance bits of the tests into a separate make target, making local development a bit less painful. It still hurts a bit due to all the lucetc compiles that have to happen under the hood when running various tests, but improving that will have to come later.

The benchmark and fuzz targets in particular take quite a long time to run, and are usually not the
focus of development. They're still worth running in CI, though, to catch bitrot.
Copy link
Member

@data-pup data-pup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@acfoltzer acfoltzer merged commit 1889b11 into master Jan 6, 2020
@acfoltzer acfoltzer deleted the acf/gardening branch January 6, 2020 22:52
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants