On Tue, Dec 7, 2010 at 4:17 PM, Nigel Kersten <[email protected]> wrote:
> On Tue, Dec 7, 2010 at 1:30 PM, Paul Berry <[email protected]> wrote: > > On Thu, Dec 2, 2010 at 8:46 AM, Nigel Kersten <[email protected]> > wrote: > >> > >> Signed-off-by: Nigel Kersten <[email protected]> > >> --- > >> conf/osx/createpackage.sh | 1 + > >> 1 files changed, 1 insertions(+), 0 deletions(-) > >> > >> diff --git a/conf/osx/createpackage.sh b/conf/osx/createpackage.sh > >> index 563b69c..24e12cf 100755 > >> --- a/conf/osx/createpackage.sh > >> +++ b/conf/osx/createpackage.sh > >> @@ -50,6 +50,7 @@ function find_puppet_root() { > >> > >> function install_puppet() { > >> echo "Installing Puppet to ${pkgroot}" > >> + mkdir -p "${pkgroot}/var/lib/puppet" > >> "${installer}" --destdir="${pkgroot}" --bindir="${BINDIR}" > >> --sbindir="${SBINDIR}" --sitelibdir="${SITELIBDIR}" > >> chown -R root:admin "${pkgroot}" > >> } > >> -- > >> 1.7.3.1 > >> > > > > Nigel-- > > Could we have a message in the commit (and in the bug report) explaining > why > > this change is beneficial? We'd be happy to accept it into the code base > > but we'd like to be able to trace the reason why it was added. > > Sure. I'll update it later. Can you mark the original bug as code > insufficient or something? I'm flitting on and off-line at the moment. > > Basically /var/lib doesn't exist on OS X at all, so /var/lib/puppet > doesn't get made. > > We should ultimately move to /var/db/puppet, but that change should be > left for a major version. > > No problem. The ticket's already updated. There's no hurry on this one AFAIK so feel free to work on it when you get a chance. -- 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.
