Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Side by Side Diff: modules/nodejs/manifests/package.pp

Issue 29483578: #2007 - Introduce nodejs module (Closed)
Patch Set: Created July 9, 2017, 1:30 a.m.
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View unified diff | Download patch
OLDNEW
(Empty)
1 # == Type: nodejs::package
2 #
3 # Manage nodejs packages.
4 #
5 # === Parameters:
6 #
7 # [*options*]
8 # A list of zero or more options to install the package.
9 #
10 define nodejs::package (
mathias 2017/07/09 07:26:50 Shouldn't there be some $ensure parameter in order
11 $options = [],
12 ) {
13
14 $install_command = [
15 "npm", "install",
16 $options,
17 $title,
18 ]
19
20 exec {"install_$title":
21 path => ["/usr/bin"],
22 command => shellquote($install_command),
23 require => Package['nodejs'],
24 onlyif => "test ! -x /usr/bin/${title}",
mathias 2017/07/09 07:26:50 Please use the 'creates' parameter instead.
25 }
26 }
OLDNEW

Powered by Google App Engine
This is Rietveld