Issue #4938 has been updated by Charlie Sharpsteen.

Assignee set to Charlie Sharpsteen

----------------------------------------
Feature #4938: Deprecate confusing symlink specification in 'ensure' for File 
resources
https://projects.puppetlabs.com/issues/4938#change-87324

* Author: Nigel Kersten
* Status: Accepted
* Priority: High
* Assignee: Charlie Sharpsteen
* Category: 
* Target version: 3.x
* Affected Puppet version: 
* Keywords: usabililty
* Branch: 
----------------------------------------
I generally think we've overloaded 'ensure' too much, and in the case of the 
File resource, it's confusing.

Which of these do you think reads better? The latter works now, but only 
because anything not "file" or "directory" or "absent" or "present" falls down 
to a symlink.

<pre>
file { "/tmp/foo":
  ensure => "/tmp/bar",
}

file { "/tmp/foo":
  ensure => symlink,
  target   => "/tmp/bar",
}
</pre>

I propose we deprecate the current usage of ensure to overload symlinks, and 
make "symlink" a special word, just like "file" and "directory".


-- 
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 unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/puppet-bugs?hl=en.
For more options, visit https://groups.google.com/groups/opt_out.


Reply via email to