Issue #6753 has been updated by Chris Price.
An initial acceptance test that covers this (tangentially) is here: https://github.com/puppetlabs/puppet/pull/518 ---------------------------------------- Refactor #6753: Faces should be available as `puppet ${face}` without a stub `lib/puppet/application/${face}.rb` file. https://projects.puppetlabs.com/issues/6753#change-55328 Author: Markus Roberts Status: Accepted Priority: High Assignee: Chris Price Category: Faces Target version: Telly Affected Puppet version: Keywords: Branch: The plugin form of interfaces requires a small file for each indirection noun of the form: <pre> require 'puppet/application/indirection_base' class Puppet::Application::Catalog < Puppet::Application::IndirectionBase end </pre> These share a directory with 1) base class implementations, 2) the actual "interface" implementation. This has a number of unintended and confusing user-visible consequences (e.g. interface_base shows up in "puppet --help" as a command when it is not, etc.) and asking users to understand / maintain this structure imposes an unnecessary barrier to use / extension of the interface facility. Therefore, in the integrated version of interfaces: 1) interface_base & indirection_base should be moved to a more appropriate location 2) the "boilerplate" files should be auto-discovered / created by an appropriate hook in core 3) the interface command should be moved into the application directory with agent, etc. 4) this interface-specific application directory should be eliminated -- You have received this notification because you have either subscribed to it, or are involved in it. To change your notification preferences, please click here: http://projects.puppetlabs.com/my/account -- You received this message because you are subscribed to the Google Groups "Puppet Bugs" group. To post to this group, send email to [email protected]. To unsubscribe from this group, send email to [email protected]. For more options, visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
