HomePhorge

Save face loader state in a different, more robust way.
60ebc03e01e5Unpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

Save face loader state in a different, more robust way.

The previous version mostly worked, but wound up showing an odd failure mode
under Ruby 1.9.3 where it didn't seem to restore state. My best guess is that
something caused the older value to be closed over...

In any case this is an equally horrible, but more effective, way to resolve
the same problem. I don't mind *like* either of them, but they work.

Really, the whole thing should be rethought - it may not actually be worth the
level of pain required to test the loader in the unit tests.

Signed-off-by: Daniel Pittman <daniel@puppetlabs.com>

Details

Provenance
Daniel Pittman <daniel@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPUb5847f4b2d6a: Merge branch 'feature/13970/remove-dynamic-scoping' from zaphod42
Branches
Unknown
Tags
Unknown

Event Timeline

Daniel Pittman <daniel@puppetlabs.com> committed rPU60ebc03e01e5: Save face loader state in a different, more robust way. (authored by Daniel Pittman <daniel@puppetlabs.com>).Apr 29 2012, 4:14 AM