roc-lang / roc

A fast, friendly, functional language.
https://roc-lang.org
Universal Permissive License v1.0
3.86k stars 284 forks source link

Make `roc dev someFile.roc` forward exit status #6842

Closed JackoCoolio closed 5 days ago

JackoCoolio commented 1 week ago

Closes #6797 and also cleans up a few things I noticed (in separate commits).

The issue is that we get the "exit status" here... https://github.com/roc-lang/roc/blob/6a074adcf235beaa171c962bf71742325e98cf9c/crates/cli/src/lib.rs#L1245-L1249 ...but status isn't actually the exit status. Per waitpid(2), libc::WEXITSTATUS(status) is what we're looking for.

JackoCoolio commented 5 days ago

@lukewilliamboswell Are you able to merge this?