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.

Sign up for GitHub

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

Jump to bottom

A new $tmp_dir option for handling '/tmp' with noexec, and a bug fix. #105

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

A new $tmp_dir option for handling '/tmp' with noexec, and a bug fix. #105

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
cc02e0b
[bug] escape strings when installing gems
martyn Mar 3, 2014
d995eb3
[bug] try using sudo instead of su to install gem as non-root user
martyn Mar 3, 2014
111fec0
[feature] allow for override of TMPDIR
martyn Sep 8, 2014
9e3b531
[fix] sudo was quoting the list() command incorrectly for some versions
martyn Sep 8, 2014
b851271
[refactor] move cmd to top of gem execution block
martyn Sep 8, 2014
de06431
[custom revert] switch back to su from sudo to prevent tty error
martyn Sep 8, 2014
cadc9ae
[bug] remove sudo command, use su instead
martyn Sep 8, 2014
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
3 changes: 2 additions & 1 deletion manifests/compile.pp
View file
Open in desktop
Expand Up @@ -12,6 +12,7 @@
$keep = false,
$configure_opts = '--disable-install-doc',
$bundler = present,
$tmp_dir = "/tmp"
) {

# Workaround http://projects.puppetlabs.com/issues/9848
Expand Down Expand Up @@ -72,7 +73,7 @@
user => $user,
group => $group,
cwd => $home_path,
environment => [ "HOME=${home_path}", "CONFIGURE_OPTS=${configure_opts}" ],
environment => [ "HOME=${home_path}", "CONFIGURE_OPTS=${configure_opts}", "TMPDIR=$tmp_dir" ],
creates => "${versions}/${ruby}",
path => $path,
logoutput => 'on_failure',
Expand Down