Dear Dave,
your patch looks great except that it introduces tabs. Please no tabs in ASDF. At least SBCL hates them.
Also, if you can contrive a test case, maybe as an add-on to the image-op test in test-program.script, that would be great.
—♯ƒ • François-René ÐVB Rideau •Reflection&Cybernethics• http://fare.tunes.org drug, n: A substance that, injected into a rat, produces a scientific paper.
On Sun, Aug 10, 2014 at 4:06 PM, Dave Cooper david.cooper@genworks.com wrote:
On Sat, Aug 9, 2014 at 5:16 PM, Faré fahree@gmail.com wrote:
Sorry, but doesn't it work to set *immutable-systems* AFTER you load everything?
No, it doesn't, because we never actually load anything via ASDF. Everything was preloaded into the built image, using monolithic-compile-bundles, sans actual ASDF.
I admit there not yet a test for this functionality in the asdf test suite; it may be that the functionality is buggy and/or its API suboptimal, and you're welcome to suggest patches and/or suggestions.
Ok, here is a patch (against 3.1.3.2 of master) which:
o Makes it check for sysdef-preloaded-system-search as well as system-registered-p, when doing sysdef-immutable-system-search and when checking for *immutable-systems* in find-system. (previously it was only checking system-registered-p).
o Adds a function asdf/find-system:register-immutable-system which adds the requested system to the *immutable-systems* as well as calls register-preloaded-system with it.
-- My Best,
Dave Cooper, Genworks Support