Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace manual 'wget' with puppet module #243

Closed
jeff1evesque opened this issue Jan 4, 2016 · 3 comments
Closed

Replace manual 'wget' with puppet module #243

jeff1evesque opened this issue Jan 4, 2016 · 3 comments
Labels

Comments

@jeff1evesque
Copy link
Owner

We will replace the manual exec implementation of wget, with the equivalent puppet module, when the wget::fetch syntax question has been resolved. Specifically, we should have an implementation as follows within install_drush.pp:

...
## include puppet modules
include wget

## variables
$drush_version = '8.0.0-rc4'

## download drush: wget is a wrapper around the 'exec' function
#
#  @timeout, override the default 'exec' timeout
#  @verbose, verbose logging
wget::fetch {'download-drush':
  source      => "https://github.com/drush-ops/drush/releases/download/${drush_version}/drush.phar",
  destination => '/tmp/drush.phar',
  verbose     => false,
}
...
@jeff1evesque
Copy link
Owner Author

0e9993e: was incorrectly committed to this issue.

@jeff1evesque
Copy link
Owner Author

This issue may be related with not including the following within install_drush.pp:

## define $PATH for all execs
Exec {path => ['/usr/bin/']}
...

@jeff1evesque
Copy link
Owner Author

This issue is invalid, since we are no longer implementing the wget package.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant