Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Override default system umask with Exec umask parameter #70

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

Override default system umask with Exec umask parameter #70

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
f0089f3
Added umask param to set umask on extract exec
cbalfour Aug 10, 2016
23593e6
Quote default umask
cbalfour Aug 10, 2016
5922e98
json_pure requires Ruby version ~> 2.0
cbalfour Aug 10, 2016
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions .travis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ script: ["bundle exec rake validate", "bundle exec rake lint", "bundle exec rake
matrix:
fast_finish: true
include:
- rvm: 1.9.3
env: PUPPET_GEM_VERSION="~> 3.0"
# - rvm: 1.9.3
# env: PUPPET_GEM_VERSION="~> 3.0"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you commenting these lines?

Copy link
Author

@cbalfour cbalfour Aug 16, 2016 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For some reason travis-ci tests failed for ruby 1.9.3 with the error: json_pure requires Ruby version ~> 2.0. Not sure why.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

They're not failing on master, so I'd rather not comment that out.

- rvm: 2.0.0
env: PUPPET_GEM_VERSION="~> 3.0"
- rvm: 2.0.0
Expand Down
2 changes: 2 additions & 0 deletions manifests/extract.pp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,7 @@
'Solaris' => 'gtar',
default => 'tar',
},
$umask='022',
) {

if $root_dir {
Expand Down Expand Up @@ -82,6 +83,7 @@
timeout => $timeout,
user => $user,
path => $path,
umask => $umask,
}
}
'absent': {
Expand Down
1 change: 1 addition & 0 deletions manifests/init.pp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,7 @@
$purge_target=false,
$user=undef,
$tar_command=undef,
$umask='022',
) {

archive::download {"${name}.${extension}":
Expand Down