Skip to content

Commit 0fa260c

Browse files
committed
init.pp: remove useless exec with default title
Per the docs, the special value `default` sets default attribute values for other resource bodies in the same expression. There are no resources in this expression so it doesn't do anything. This was probably intended to be a "legacy-style default, but the style guide says to avoid these anyway.
1 parent 063e230 commit 0fa260c

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

manifests/init.pp

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,6 @@
6262
Stdlib::Httpurl $anaconda_installer_url = 'https://repo.anaconda.com/archive/Anaconda3-5.2.0-Linux-x86_64.sh',
6363
Stdlib::Absolutepath $anaconda_install_path = '/opt/python',
6464
Boolean $manage_scl = true,
65-
Optional[Python::Umask] $umask = undef,
6665
) inherits python::params {
6766
$exec_prefix = $provider ? {
6867
'scl' => "/usr/bin/scl enable ${version} -- ",
@@ -76,11 +75,6 @@
7675
Class['python::install']
7776
-> Class['python::config']
7877

79-
# Set default umask.
80-
exec { default:
81-
umask => $umask,
82-
}
83-
8478
# Allow hiera configuration of python resources
8579
create_resources('python::pip', $python_pips)
8680
create_resources('python::pyvenv', $python_pyvenvs)

0 commit comments

Comments
 (0)