Please review pull request #649: (maint) Remove unnecessary step from puppet module build help opened by (jeffmccune)

Description:

Without this patch the puppet help module build command says to make
sure the module directory is in the form of -.
This is an unnecessary step because it isn't actually necessary anymore.
It's a problem because we want to make things appear to be as simple as
they actually are, not more complex than they need to be.

I just tested this with a module existing at /vagrant/modules/jenkins
(no symlinks) using the command: puppet module build .

Everything worked out fine. Puppet correctly reads the Modulefile and
figures out the author name.

This is a documentation only change.

  • Opened: Mon Apr 09 23:26:15 UTC 2012
  • Based on: puppetlabs:2.7.x (aa267eff69dd5ce6e840bc2aa9592592ea7681d8)
  • Requested merge: jeffmccune:maint/2.7.x/puppet_module_build_help (130661156198bef0cd3f4e93427918cfc6f1f97c)

Diff follows:

diff --git a/lib/puppet/face/module/build.rb b/lib/puppet/face/module/build.rb
index f1a0ef4..eec482e 100644
--- a/lib/puppet/face/module/build.rb
+++ b/lib/puppet/face/module/build.rb
@@ -3,9 +3,7 @@
     summary "Build a module release package."
     description <<-EOT
       Prepares a local module for release on the Puppet Forge by building a
-      ready-to-upload archive file. Before using this action, make sure that
-      the module directory's name is in the standard <username>-<module>
-      format.
+      ready-to-upload archive file.
 
       This action uses the Modulefile in the module directory to set metadata
       used by the Forge. See <http://links.puppetlabs.com/modulefile> for more

    

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