Issue #15202 has been updated by Ken Barber. Status changed from Needs More Information to Tests Insufficient Assignee changed from Ken Barber to Jeff McCune
Hi Jeff ... Actually after looking at this I notice there are already acceptance tests for this condition that were now failing with this patch. Sorry for not running the acceptance tests as part of the review initially, I'll try to do this all in one go next time. Here is a sample fix for your patch that makes them pass. Feel free to use it, or base something off of it: <https://github.com/kbarber/puppet/commit/cfa98eced27f87833bd3232a94eb921178f70362> ---------------------------------------- Bug #15202: Puppet should create directories when installing modules https://projects.puppetlabs.com/issues/15202#change-65804 Author: Jeff McCune Status: Tests Insufficient Priority: Normal Assignee: Jeff McCune Category: module tool Target version: 3.0.0 Affected Puppet version: 3.0.0rc3 Keywords: module tool install mkdir modules Branch: # Overview # This needs to be fixed. (Run as a normal user account on my Mac with Puppet FOSS) Chris Price mentioned he has a patch in a topic branch but I couldn't find it. <pre> $ puppet module install puppetlabs-cloud_provisioner Preparing to install into /Users/jeff/.puppet/modules ... Error: Could not install module 'puppetlabs-cloud_provisioner' (latest) Directory /Users/jeff/.puppet/modules does not exist </pre> -- 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.
