HomePhorge

(#7690) Don't blow up when listing terminuses available for faces
2aa9f2fe390cUnpublished

Unpublished Commit ยท Learn More

Repository Importing: This repository is still importing.

Description

(#7690) Don't blow up when listing terminuses available for faces

Previously, in order to list the available terminuses for an indirected
face we loaded all the the terminuses in order to list them. This meant
that if a terminus like active_record didn't have the dependencies it
needed, the documentation would raise errors and not list terminuses.

<Puppet::Error: Could not autoload filename uninitialized constant Object::ActiveRecord>

Now we just list the terminuses available in the load path without
trying to load them. The terminus will still raise an error if you try
to use it without its dependencies being met.

Paired-with: Max Martin <max@puppetlabs.com>

Details

Provenance
Matt Robinson <matt@puppetlabs.com>Authored on
vanmeeuwenPushed on Jun 2 2015, 2:22 PM
Parents
rPU9447cb957b8c: maint: Dedup the loadpath so we don't have to walk it multiple times
Branches
Unknown
Tags
Unknown

Event Timeline

Matt Robinson <matt@puppetlabs.com> committed rPU2aa9f2fe390c: (#7690) Don't blow up when listing terminuses available for faces (authored by Matt Robinson <matt@puppetlabs.com>).May 27 2011, 1:08 AM