search

Metacello / metacello

Metacello is a package management system for Smalltalk
MIT License
87 stars 43 forks source link

Interesting behavior for Metacello load using cacheRepository #400

Open dalehenrich opened 8 years ago

dalehenrich commented 8 years ago

I don't think this is a bug, since the cacheRepository is really intended for use when doing pure fetches and not for doing loads ... with that said, there is nothing to prevent one from attempting to do a load when you've set the cacheRepository ... however as the tests associated with this bug prove, the results will not be what one wants ...

dalehenrich commented 8 years ago

This discovery came while setting up tests for Issue #399 ... and one of the test MetacelloCypressIssue399PackageCacheTestCase>>testIssue400A showed unexpected behavior in that the package was not loaded ... there may be other odd behavior worth exploring some day involving the following:

neither of these methods schedule a fetch and thus nothing gets added to the directives and nothing is loaded ...

I think the ultimate resolution here will be to disallow a load when a cacheRepository is specified ...

pdebruic commented 5 years ago

I just ran into this for the past weekend. Didn't know about the distinction between fetch and load. Can we add a warning if cacheRepository: is set when doing a load? Proceeding unsets the cacheRepository or something.