Issue #6753 has been updated by Chris Price.

Assignee deleted (Chris Price)


----------------------------------------
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-65075

Author: Markus Roberts
Status: Accepted
Priority: High
Assignee: 
Category: Faces
Target version: 3.x
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.

Reply via email to