search

Metaxal / quickscript

Easy scripting for DrRacket
Other
17 stars 6 forks source link

Disabled Scripts with syntax errors cause popup on DrRacket startup #48

Closed rfindler closed 4 years ago

rfindler commented 4 years ago

One of my scripts, bars.rkt, has a free identifier in it. It is disabled via the Manage Scripts | Disable Scripts... menu:

Screen Shot 2020-08-09 at 10 34 32 AM

When I start up DrRacket, however, I see this message:

Screen Shot 2020-08-09 at 10 35 55 AM

Probably it should be ignored, since it is disabled?

rfindler commented 4 years ago

This message actually appears each time I open a new DrRacket window (but not new tab).

Metaxal commented 4 years ago

Thanks for the report! Indeed that's the current behaviour and it needs to change.

The reason behind this is 'shadow scripts'. Since the shadowed script is disabled, it still needs to be compiled if the shadowing script is compiled. I thought that compile-zos does not recompile dependencies, so I chose to recompile everything, but that's not ideal.

Ideally, the dependencies of scripts should be recompiled too (the shadowed script is a dependency of the shadowing script), but I'm not sure what's the right way to do it.

So my question is: What's the correct way to recompile dependencies?

Metaxal commented 4 years ago

Matthew just confirmed that compile-zos doesn't compile dependencies, but compile-directory-zos does, so I'll try to switch to the latter.

Metaxal commented 4 years ago

The last commit should resolve your problem. Can you confirm it works as expected?