Skip to content

Commit 32c7adb

Browse files
authored
Merge pull request #154 from kajinamit/syntax-fix
Fix lint errors
2 parents e107f33 + 376faee commit 32c7adb

File tree

4 files changed

+5
-11
lines changed

4 files changed

+5
-11
lines changed

manifests/get.pp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,6 @@
5353
$logfile = undef,
5454
$onlyif = undef,
5555
) {
56-
5756
if $keyfile {
5857
$mykeyfile = $keyfile
5958
} else {
@@ -155,7 +154,7 @@
155154

156155
exec { "rsync ${name}":
157156
command => "rsync -q ${rsync_options}",
158-
path => [ '/bin', '/usr/bin', '/usr/local/bin' ],
157+
path => ['/bin', '/usr/bin', '/usr/local/bin'],
159158
user => $execuser,
160159
# perform a dry-run to determine if anything needs to be updated
161160
# this ensures that we only actually create a Puppet event if something needs to

manifests/init.pp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,12 @@
22
#
33
# This module manages rsync
44
#
5-
class rsync(
5+
class rsync (
66
$package_ensure = 'installed',
77
$manage_package = true,
88
$puts = {},
99
$gets = {},
1010
) {
11-
1211
if $manage_package {
1312
package { 'rsync':
1413
ensure => $package_ensure,

manifests/put.pp

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@
3838
$timeout = '900',
3939
$options = '-a'
4040
) {
41-
4241
if $keyfile {
4342
$mykeyfile = $keyfile
4443
} else {
@@ -92,7 +91,7 @@
9291

9392
exec { "rsync ${name}":
9493
command => "rsync -q ${rsync_options}",
95-
path => [ '/bin', '/usr/bin' ],
94+
path => ['/bin', '/usr/bin'],
9695
# perform a dry-run to determine if anything needs to be updated
9796
# this ensures that we only actually create a Puppet event if something needs to
9897
# be updated

manifests/server.pp

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
# class xinetd if use_xinetd is set to true
77
# class rsync
88
#
9-
class rsync::server(
9+
class rsync::server (
1010
Boolean $use_xinetd = true,
1111
$address = '0.0.0.0',
1212
$motd_file = 'UNSET',
@@ -22,7 +22,6 @@
2222
Variant[Boolean, Enum['mask']] $service_enable = true,
2323
Boolean $manage_package = $rsync::manage_package,
2424
) inherits rsync {
25-
2625
if $use_xinetd {
2726
include xinetd
2827
xinetd::service { 'rsync':
@@ -33,14 +32,12 @@
3332
require => Package['rsync'],
3433
}
3534
} else {
36-
3735
# Manage the installation of the rsyncd package?
3836
if $manage_package {
39-
4037
# RHEL8 and newer (and their variants) have a separate package for rsyncd daemon. If the $package_name
4138
# variable is defined (the variable is defined in the hiera hierarchy), then install the package.
4239
if $package_name {
43-
package {$package_name:
40+
package { $package_name:
4441
ensure => $rsync::package_ensure,
4542
notify => Service[$servicename],
4643
}

0 commit comments

Comments
 (0)