On 6/9/2010 6:19 PM, Markus Roberts wrote:
I'd advocate a mkdir -p style solution, unless anyone can see problems
with that.


Well, I would say that we should either be localizing or not.  We're now
defaulting to a value that works great for the linux world but doesn't even
work in the non-linux world.

IMO, it's a far larger bug to mkdir -p /var/lib on, say, OS X and AIX than
to just have a case statement that localizes for a few basic OS choices.

I'd say that those are two distinct issues: what the default should be
and how we should go about creating the directory if it doesn't exist.
  I agree that we should make the default sane on the various targets,
but regardless of how the path is determined there's no reason to
botch the creation of it.

With my Win32 hat on, target-specific defaults get my +1. See also

http://github.com/DavidS/puppet/commit/2fbe66c50e63d9ee0a52cae1cc54d2f1d736317f


Regarding the don't-botch-directory-creation stuff, I seem to remember that (semi-)automatically managing parent dirs is a common feature request. Maybe it's time to implement this?



Best Regards, David
--
dasz.at OG              Tel: +43 (0)664 2602670     Web: http://dasz.at
Klosterneuburg                                         UID: ATU64260999

       FB-Nr.: FN 309285 g          FB-Gericht: LG Korneuburg

--
You received this message because you are subscribed to the Google Groups "Puppet 
Developers" 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-dev?hl=en.

Reply via email to