Issue #16189 has been updated by Henrik Lindberg.

*Solution  "pushing handling of --mode to be made earlier":*

There is one obvious way of fixing the problem; by making --mode a global 
option. 

This however clashes with the doc application which uses `--mode (-m)` for 
documentation mode (text, pdf, rdoc). Either rename the doc option, or rename 
the run mode option to `--runmode (-r)` (currently it is `--mode (-r)`).
 
A downside is that it is possible to set a stupid run mode e.g. `master --mode 
user' and vice versa.

If this solution is chosen, the actual work can be done in 
`Settings.parse_global_options` + method for respective option called from 
`Settings.handlearg`.

As a consequence, the actual run mode to use would then need to be picked up 
from Settings. This can be done the way it is done now, but instead of picking 
`:user` as a default in `Application.class.run:mode`, it would pick up the 
default from `Settings`. Settings in turn would naturally return `:user` if no 
mode option was used on the command line.

 
----------------------------------------
Bug #16189: puppet config print and --configprint disagree
https://projects.puppetlabs.com/issues/16189#change-71003

Author: Patrick Carlisle
Status: Accepted
Priority: Normal
Assignee: 
Category: 
Target version: 3.0.0
Affected Puppet version: 3.0.0-rc5
Keywords: 
Branch: 


    thinky:puppet:% puppet master --configprint ssldir                          
                                                                                
                            <3.x>
    /etc/puppet/ssl
    thinky:puppet:% puppet config print ssldir --mode master                    
                                                                                
                            <3.x>
    /home/patrick/.puppet/ssl


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