Some post-rc1 fixes - Githubissues

mighty-gerbils / gerbil

Gerbil Scheme

https://cons.io

GNU Lesser General Public License v2.1

1.15k stars 110 forks source link

Some post-rc1 fixes #987

Closed vyzo closed 11 months ago

vyzo commented 11 months ago

Fixes clown shoes with double v in configured gerbil version
kills obsolete release.ss script; it is not useful any more
adds a runtime initialized indicator and sanity checks it when trying to load the expander

netlify[bot] commented 11 months ago

Deploy Preview for elastic-ritchie-8f47f9 ready!

Name	Link
Latest commit	c4aefc613c275096d228c5dc75314428c387fb2a
Latest deploy log	https://app.netlify.com/sites/elastic-ritchie-8f47f9/deploys/651ff88b8ba4a900080f1c7d
Deploy Preview	https://deploy-preview-987--elastic-ritchie-8f47f9.netlify.app
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

vyzo commented 11 months ago

Because libgerbil doesnt know in advance what nodes will be builtin. And ultimately it shouldn't, the only thing you cannot do without gerbil-runtime-init! is load dynamic modules, and loading them in the initializer of a module is undefined behaviour because the program has not been fully initialized yet.

So yeah, it does the right thing right now.

mighty-gerbils / gerbil

Some post-rc1 fixes #987

✅ Deploy Preview for elastic-ritchie-8f47f9 ready!

Deploy Preview for elastic-ritchie-8f47f9 ready!