Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the %run hook function for running binaries. #90

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jpco
Copy link
Collaborator

@jpco jpco commented Feb 7, 2024

Resolves #80.

I'm actually not sure if it's better to go with this "use the primitive if %run is unset" fallback, or to just throw an error if %run happens to be unset. The former is more useful but the latter is more consistent.

@jpco
Copy link
Collaborator Author

jpco commented Jul 8, 2024

Well, I was coming back to switch this to the more-consistent behavior (throw an error if %run isn't set) but it seems like there's a memory bug in this PR that causes es to crash with -DGCDEBUG=1 with the commands

fn-%run = ()
exit

EDIT: Actually, I was wrong, I think this might have been one of the recent exception-related bug fixes that haven't been merged into my development branch. It works at head.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Should %run be the built-in way binaries are run?
1 participant