puppet 8.5.0-universal-darwin → 8.6.0-universal-darwin

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (543) hide show
  1. checksums.yaml +4 -4
  2. data/Gemfile.lock +29 -26
  3. data/ext/windows/service/daemon.rb +49 -55
  4. data/lib/hiera/scope.rb +3 -3
  5. data/lib/hiera_puppet.rb +1 -1
  6. data/lib/puppet/agent/disabler.rb +1 -1
  7. data/lib/puppet/application/agent.rb +2 -2
  8. data/lib/puppet/application/apply.rb +2 -2
  9. data/lib/puppet/application/describe.rb +6 -6
  10. data/lib/puppet/application/device.rb +99 -100
  11. data/lib/puppet/application/doc.rb +1 -1
  12. data/lib/puppet/application/face_base.rb +7 -7
  13. data/lib/puppet/application/filebucket.rb +1 -1
  14. data/lib/puppet/application/lookup.rb +5 -5
  15. data/lib/puppet/application/resource.rb +10 -3
  16. data/lib/puppet/application/ssl.rb +3 -3
  17. data/lib/puppet/application.rb +16 -18
  18. data/lib/puppet/coercion.rb +1 -1
  19. data/lib/puppet/configurer/downloader.rb +9 -4
  20. data/lib/puppet/configurer/fact_handler.rb +12 -13
  21. data/lib/puppet/configurer.rb +16 -18
  22. data/lib/puppet/confine/feature.rb +1 -1
  23. data/lib/puppet/confine/variable.rb +1 -1
  24. data/lib/puppet/confine.rb +2 -2
  25. data/lib/puppet/confine_collection.rb +1 -1
  26. data/lib/puppet/confiner.rb +2 -2
  27. data/lib/puppet/daemon.rb +3 -3
  28. data/lib/puppet/datatypes.rb +5 -6
  29. data/lib/puppet/defaults.rb +14 -8
  30. data/lib/puppet/environments.rb +3 -7
  31. data/lib/puppet/error.rb +3 -3
  32. data/lib/puppet/external/dot.rb +42 -34
  33. data/lib/puppet/face/catalog/select.rb +1 -1
  34. data/lib/puppet/face/epp.rb +7 -9
  35. data/lib/puppet/face/help.rb +12 -12
  36. data/lib/puppet/face/module/changes.rb +1 -3
  37. data/lib/puppet/face/module/list.rb +37 -37
  38. data/lib/puppet/face/node/clean.rb +1 -1
  39. data/lib/puppet/face/parser.rb +6 -8
  40. data/lib/puppet/face/report.rb +5 -7
  41. data/lib/puppet/facter_impl.rb +1 -1
  42. data/lib/puppet/feature/base.rb +9 -11
  43. data/lib/puppet/feature/pe_license.rb +1 -1
  44. data/lib/puppet/feature/ssh.rb +1 -1
  45. data/lib/puppet/feature/telnet.rb +3 -5
  46. data/lib/puppet/feature/zlib.rb +1 -1
  47. data/lib/puppet/ffi/windows/constants.rb +1 -1
  48. data/lib/puppet/ffi/windows/functions.rb +2 -2
  49. data/lib/puppet/ffi/windows/structs.rb +3 -3
  50. data/lib/puppet/file_bucket/dipper.rb +4 -4
  51. data/lib/puppet/file_bucket/file.rb +4 -4
  52. data/lib/puppet/file_serving/base.rb +5 -5
  53. data/lib/puppet/file_serving/configuration/parser.rb +9 -9
  54. data/lib/puppet/file_serving/configuration.rb +3 -3
  55. data/lib/puppet/file_serving/fileset.rb +13 -13
  56. data/lib/puppet/file_serving/http_metadata.rb +1 -1
  57. data/lib/puppet/file_serving/metadata.rb +13 -9
  58. data/lib/puppet/file_serving/mount/file.rb +6 -6
  59. data/lib/puppet/file_serving/mount/locales.rb +2 -4
  60. data/lib/puppet/file_serving/mount/pluginfacts.rb +2 -4
  61. data/lib/puppet/file_serving/mount/plugins.rb +2 -4
  62. data/lib/puppet/file_serving/mount.rb +1 -1
  63. data/lib/puppet/file_system/file_impl.rb +3 -3
  64. data/lib/puppet/file_system/memory_impl.rb +1 -1
  65. data/lib/puppet/file_system/path_pattern.rb +1 -1
  66. data/lib/puppet/file_system/uniquefile.rb +17 -16
  67. data/lib/puppet/file_system/windows.rb +26 -18
  68. data/lib/puppet/forge/cache.rb +2 -2
  69. data/lib/puppet/forge/errors.rb +1 -1
  70. data/lib/puppet/forge/repository.rb +2 -2
  71. data/lib/puppet/forge.rb +13 -19
  72. data/lib/puppet/functions/abs.rb +2 -2
  73. data/lib/puppet/functions/camelcase.rb +1 -1
  74. data/lib/puppet/functions/compare.rb +2 -2
  75. data/lib/puppet/functions/contain.rb +1 -1
  76. data/lib/puppet/functions/defined.rb +2 -2
  77. data/lib/puppet/functions/dig.rb +1 -1
  78. data/lib/puppet/functions/eyaml_lookup_key.rb +11 -13
  79. data/lib/puppet/functions/get.rb +1 -1
  80. data/lib/puppet/functions/hocon_data.rb +3 -5
  81. data/lib/puppet/functions/import.rb +1 -1
  82. data/lib/puppet/functions/json_data.rb +4 -6
  83. data/lib/puppet/functions/max.rb +2 -2
  84. data/lib/puppet/functions/min.rb +2 -2
  85. data/lib/puppet/functions/new.rb +1 -1
  86. data/lib/puppet/functions/reduce.rb +6 -10
  87. data/lib/puppet/functions/regsubst.rb +2 -2
  88. data/lib/puppet/functions/require.rb +1 -1
  89. data/lib/puppet/functions/yaml_data.rb +11 -13
  90. data/lib/puppet/functions.rb +32 -37
  91. data/lib/puppet/generate/type.rb +2 -2
  92. data/lib/puppet/gettext/config.rb +9 -9
  93. data/lib/puppet/graph/rb_tree_map.rb +31 -23
  94. data/lib/puppet/graph/relationship_graph.rb +12 -12
  95. data/lib/puppet/graph/simple_graph.rb +8 -10
  96. data/lib/puppet/http/client.rb +1 -1
  97. data/lib/puppet/http/dns.rb +4 -4
  98. data/lib/puppet/http/external_client.rb +1 -1
  99. data/lib/puppet/http/factory.rb +1 -1
  100. data/lib/puppet/http/proxy.rb +15 -15
  101. data/lib/puppet/http/redirector.rb +6 -2
  102. data/lib/puppet/http/resolver.rb +2 -2
  103. data/lib/puppet/http/retry_after_handler.rb +2 -2
  104. data/lib/puppet/http/service/ca.rb +1 -1
  105. data/lib/puppet/http/service/compiler.rb +4 -4
  106. data/lib/puppet/http/service/file_server.rb +1 -1
  107. data/lib/puppet/http/service/report.rb +2 -2
  108. data/lib/puppet/http/service.rb +9 -13
  109. data/lib/puppet/http/session.rb +1 -1
  110. data/lib/puppet/http/site.rb +1 -1
  111. data/lib/puppet/indirector/catalog/compiler.rb +44 -45
  112. data/lib/puppet/indirector/catalog/rest.rb +7 -5
  113. data/lib/puppet/indirector/catalog/store_configs.rb +1 -1
  114. data/lib/puppet/indirector/exec.rb +2 -2
  115. data/lib/puppet/indirector/face.rb +7 -9
  116. data/lib/puppet/indirector/fact_search.rb +1 -1
  117. data/lib/puppet/indirector/facts/facter.rb +6 -6
  118. data/lib/puppet/indirector/facts/json.rb +1 -1
  119. data/lib/puppet/indirector/facts/store_configs.rb +1 -1
  120. data/lib/puppet/indirector/file_bucket_file/file.rb +17 -17
  121. data/lib/puppet/indirector/file_bucket_file/rest.rb +3 -3
  122. data/lib/puppet/indirector/file_content/rest.rb +1 -1
  123. data/lib/puppet/indirector/file_metadata/rest.rb +1 -1
  124. data/lib/puppet/indirector/hiera.rb +1 -1
  125. data/lib/puppet/indirector/indirection.rb +27 -29
  126. data/lib/puppet/indirector/json.rb +4 -4
  127. data/lib/puppet/indirector/memory.rb +2 -2
  128. data/lib/puppet/indirector/msgpack.rb +5 -5
  129. data/lib/puppet/indirector/node/store_configs.rb +1 -1
  130. data/lib/puppet/indirector/none.rb +1 -1
  131. data/lib/puppet/indirector/report/json.rb +1 -1
  132. data/lib/puppet/indirector/report/yaml.rb +1 -1
  133. data/lib/puppet/indirector/request.rb +4 -4
  134. data/lib/puppet/indirector/resource/ral.rb +2 -4
  135. data/lib/puppet/indirector/resource/store_configs.rb +1 -1
  136. data/lib/puppet/indirector/terminus.rb +4 -4
  137. data/lib/puppet/indirector/yaml.rb +3 -3
  138. data/lib/puppet/info_service/class_information_service.rb +5 -8
  139. data/lib/puppet/info_service/task_information_service.rb +6 -7
  140. data/lib/puppet/interface/action.rb +5 -5
  141. data/lib/puppet/interface/action_manager.rb +5 -5
  142. data/lib/puppet/interface/documentation.rb +5 -5
  143. data/lib/puppet/interface/face_collection.rb +7 -7
  144. data/lib/puppet/interface/option.rb +2 -2
  145. data/lib/puppet/interface/option_manager.rb +6 -6
  146. data/lib/puppet/interface.rb +4 -4
  147. data/lib/puppet/metatype/manager.rb +2 -2
  148. data/lib/puppet/module/plan.rb +10 -10
  149. data/lib/puppet/module/task.rb +8 -8
  150. data/lib/puppet/module.rb +39 -41
  151. data/lib/puppet/module_tool/applications/application.rb +10 -8
  152. data/lib/puppet/module_tool/applications/installer.rb +53 -50
  153. data/lib/puppet/module_tool/applications/unpacker.rb +6 -8
  154. data/lib/puppet/module_tool/applications/upgrader.rb +37 -34
  155. data/lib/puppet/module_tool/checksums.rb +2 -2
  156. data/lib/puppet/module_tool/dependency.rb +1 -1
  157. data/lib/puppet/module_tool/errors/base.rb +1 -1
  158. data/lib/puppet/module_tool/install_directory.rb +1 -1
  159. data/lib/puppet/module_tool/local_tarball.rb +3 -5
  160. data/lib/puppet/module_tool/metadata.rb +12 -8
  161. data/lib/puppet/module_tool/shared_behaviors.rb +27 -12
  162. data/lib/puppet/module_tool/tar/mini.rb +3 -3
  163. data/lib/puppet/module_tool.rb +4 -4
  164. data/lib/puppet/network/client_request.rb +5 -3
  165. data/lib/puppet/network/format.rb +2 -2
  166. data/lib/puppet/network/format_support.rb +1 -1
  167. data/lib/puppet/network/formats.rb +1 -1
  168. data/lib/puppet/network/http/api/indirected_routes.rb +24 -44
  169. data/lib/puppet/network/http/api/server/v3.rb +1 -1
  170. data/lib/puppet/network/http/connection.rb +1 -1
  171. data/lib/puppet/network/http/handler.rb +8 -12
  172. data/lib/puppet/network/http/request.rb +1 -1
  173. data/lib/puppet/network/http/route.rb +9 -9
  174. data/lib/puppet/node/environment.rb +21 -23
  175. data/lib/puppet/node/facts.rb +1 -1
  176. data/lib/puppet/node.rb +5 -5
  177. data/lib/puppet/pal/json_catalog_encoder.rb +1 -1
  178. data/lib/puppet/pal/pal_impl.rb +48 -50
  179. data/lib/puppet/pal/plan_signature.rb +1 -1
  180. data/lib/puppet/pal/task_signature.rb +1 -1
  181. data/lib/puppet/parameter/package_options.rb +1 -1
  182. data/lib/puppet/parameter/value.rb +2 -2
  183. data/lib/puppet/parameter/value_collection.rb +7 -7
  184. data/lib/puppet/parameter.rb +8 -9
  185. data/lib/puppet/parser/ast/block_expression.rb +2 -2
  186. data/lib/puppet/parser/ast/hostclass.rb +1 -1
  187. data/lib/puppet/parser/ast/pops_bridge.rb +3 -4
  188. data/lib/puppet/parser/ast/resource.rb +4 -4
  189. data/lib/puppet/parser/ast/resourceparam.rb +4 -4
  190. data/lib/puppet/parser/ast.rb +14 -15
  191. data/lib/puppet/parser/compiler.rb +26 -28
  192. data/lib/puppet/parser/functions/create_resources.rb +3 -3
  193. data/lib/puppet/parser/functions/fail.rb +1 -1
  194. data/lib/puppet/parser/functions/generate.rb +2 -2
  195. data/lib/puppet/parser/functions/tag.rb +1 -1
  196. data/lib/puppet/parser/functions.rb +2 -2
  197. data/lib/puppet/parser/relationship.rb +3 -1
  198. data/lib/puppet/parser/resource/param.rb +2 -2
  199. data/lib/puppet/parser/resource.rb +23 -23
  200. data/lib/puppet/parser/scope.rb +10 -10
  201. data/lib/puppet/parser/script_compiler.rb +1 -1
  202. data/lib/puppet/parser/templatewrapper.rb +1 -1
  203. data/lib/puppet/parser/type_loader.rb +10 -12
  204. data/lib/puppet/parser.rb +1 -1
  205. data/lib/puppet/pops/adaptable.rb +1 -1
  206. data/lib/puppet/pops/evaluator/access_operator.rb +8 -8
  207. data/lib/puppet/pops/evaluator/callable_signature.rb +4 -4
  208. data/lib/puppet/pops/evaluator/collector_transformer.rb +4 -4
  209. data/lib/puppet/pops/evaluator/collectors/abstract_collector.rb +7 -7
  210. data/lib/puppet/pops/evaluator/collectors/fixed_set_collector.rb +6 -6
  211. data/lib/puppet/pops/evaluator/compare_operator.rb +10 -10
  212. data/lib/puppet/pops/evaluator/deferred_resolver.rb +1 -1
  213. data/lib/puppet/pops/evaluator/epp_evaluator.rb +2 -2
  214. data/lib/puppet/pops/evaluator/evaluator_impl.rb +56 -60
  215. data/lib/puppet/pops/evaluator/literal_evaluator.rb +4 -0
  216. data/lib/puppet/pops/evaluator/relationship_operator.rb +2 -2
  217. data/lib/puppet/pops/evaluator/runtime3_converter.rb +3 -3
  218. data/lib/puppet/pops/evaluator/runtime3_support.rb +1 -1
  219. data/lib/puppet/pops/functions/dispatcher.rb +3 -3
  220. data/lib/puppet/pops/functions/function.rb +14 -16
  221. data/lib/puppet/pops/issue_reporter.rb +2 -2
  222. data/lib/puppet/pops/label_provider.rb +2 -2
  223. data/lib/puppet/pops/loader/dependency_loader.rb +3 -3
  224. data/lib/puppet/pops/loader/loader.rb +4 -4
  225. data/lib/puppet/pops/loader/loader_paths.rb +3 -3
  226. data/lib/puppet/pops/loader/module_loaders.rb +11 -11
  227. data/lib/puppet/pops/loader/ruby_data_type_instantiator.rb +1 -1
  228. data/lib/puppet/pops/loader/ruby_function_instantiator.rb +1 -1
  229. data/lib/puppet/pops/loader/ruby_legacy_function_instantiator.rb +5 -1
  230. data/lib/puppet/pops/loader/static_loader.rb +3 -3
  231. data/lib/puppet/pops/loaders.rb +3 -3
  232. data/lib/puppet/pops/lookup/data_dig_function_provider.rb +1 -1
  233. data/lib/puppet/pops/lookup/explainer.rb +2 -2
  234. data/lib/puppet/pops/lookup/global_data_provider.rb +1 -1
  235. data/lib/puppet/pops/lookup/hiera_config.rb +15 -11
  236. data/lib/puppet/pops/lookup/interpolation.rb +3 -3
  237. data/lib/puppet/pops/lookup/key_recorder.rb +1 -1
  238. data/lib/puppet/pops/lookup/location_resolver.rb +1 -1
  239. data/lib/puppet/pops/lookup/lookup_adapter.rb +7 -7
  240. data/lib/puppet/pops/lookup/sub_lookup.rb +1 -1
  241. data/lib/puppet/pops/merge_strategy.rb +6 -6
  242. data/lib/puppet/pops/migration/migration_checker.rb +1 -1
  243. data/lib/puppet/pops/model/ast_transformer.rb +9 -11
  244. data/lib/puppet/pops/model/factory.rb +4 -5
  245. data/lib/puppet/pops/parser/epp_parser.rb +2 -2
  246. data/lib/puppet/pops/parser/epp_support.rb +1 -1
  247. data/lib/puppet/pops/parser/evaluating_parser.rb +2 -2
  248. data/lib/puppet/pops/parser/heredoc_support.rb +4 -4
  249. data/lib/puppet/pops/parser/interpolation_support.rb +3 -3
  250. data/lib/puppet/pops/parser/lexer2.rb +20 -12
  251. data/lib/puppet/pops/parser/lexer_support.rb +1 -1
  252. data/lib/puppet/pops/parser/locator.rb +4 -4
  253. data/lib/puppet/pops/parser/parser_support.rb +12 -14
  254. data/lib/puppet/pops/parser/pn_parser.rb +3 -3
  255. data/lib/puppet/pops/parser/slurp_support.rb +8 -8
  256. data/lib/puppet/pops/patterns.rb +12 -12
  257. data/lib/puppet/pops/pcore.rb +1 -1
  258. data/lib/puppet/pops/pn.rb +2 -2
  259. data/lib/puppet/pops/puppet_stack.rb +1 -1
  260. data/lib/puppet/pops/resource/resource_type_impl.rb +1 -1
  261. data/lib/puppet/pops/serialization/abstract_reader.rb +2 -2
  262. data/lib/puppet/pops/serialization/abstract_writer.rb +3 -3
  263. data/lib/puppet/pops/time/timestamp.rb +1 -1
  264. data/lib/puppet/pops/types/class_loader.rb +4 -6
  265. data/lib/puppet/pops/types/iterable.rb +1 -1
  266. data/lib/puppet/pops/types/p_binary_type.rb +1 -1
  267. data/lib/puppet/pops/types/p_init_type.rb +1 -1
  268. data/lib/puppet/pops/types/p_object_type.rb +5 -5
  269. data/lib/puppet/pops/types/p_object_type_extension.rb +4 -6
  270. data/lib/puppet/pops/types/p_sem_ver_range_type.rb +1 -1
  271. data/lib/puppet/pops/types/p_sensitive_type.rb +1 -1
  272. data/lib/puppet/pops/types/p_timespan_type.rb +2 -2
  273. data/lib/puppet/pops/types/p_type_set_type.rb +10 -10
  274. data/lib/puppet/pops/types/p_uri_type.rb +4 -4
  275. data/lib/puppet/pops/types/recursion_guard.rb +4 -4
  276. data/lib/puppet/pops/types/ruby_generator.rb +2 -2
  277. data/lib/puppet/pops/types/string_converter.rb +12 -8
  278. data/lib/puppet/pops/types/tree_iterators.rb +4 -6
  279. data/lib/puppet/pops/types/type_calculator.rb +10 -10
  280. data/lib/puppet/pops/types/type_factory.rb +7 -3
  281. data/lib/puppet/pops/types/type_formatter.rb +22 -24
  282. data/lib/puppet/pops/types/type_mismatch_describer.rb +24 -24
  283. data/lib/puppet/pops/types/types.rb +28 -24
  284. data/lib/puppet/pops/utils.rb +38 -42
  285. data/lib/puppet/pops/validation/checker4_0.rb +10 -10
  286. data/lib/puppet/pops/validation/tasks_checker.rb +1 -1
  287. data/lib/puppet/pops/validation/validator_factory_4_0.rb +2 -0
  288. data/lib/puppet/pops/validation.rb +4 -4
  289. data/lib/puppet/pops/visitor.rb +2 -2
  290. data/lib/puppet/property/ensure.rb +12 -14
  291. data/lib/puppet/property/keyvalue.rb +4 -4
  292. data/lib/puppet/property/list.rb +4 -4
  293. data/lib/puppet/property/ordered_list.rb +1 -1
  294. data/lib/puppet/property.rb +31 -33
  295. data/lib/puppet/provider/exec.rb +2 -2
  296. data/lib/puppet/provider/file/posix.rb +19 -17
  297. data/lib/puppet/provider/file/windows.rb +13 -19
  298. data/lib/puppet/provider/group/directoryservice.rb +3 -2
  299. data/lib/puppet/provider/group/groupadd.rb +4 -4
  300. data/lib/puppet/provider/group/windows_adsi.rb +2 -2
  301. data/lib/puppet/provider/nameservice/directoryservice.rb +29 -29
  302. data/lib/puppet/provider/nameservice/objectadd.rb +2 -4
  303. data/lib/puppet/provider/nameservice/pw.rb +1 -2
  304. data/lib/puppet/provider/nameservice.rb +8 -8
  305. data/lib/puppet/provider/package/aix.rb +19 -19
  306. data/lib/puppet/provider/package/appdmg.rb +2 -2
  307. data/lib/puppet/provider/package/apple.rb +1 -1
  308. data/lib/puppet/provider/package/apt.rb +18 -18
  309. data/lib/puppet/provider/package/aptitude.rb +1 -3
  310. data/lib/puppet/provider/package/aptrpm.rb +9 -9
  311. data/lib/puppet/provider/package/blastwave.rb +10 -14
  312. data/lib/puppet/provider/package/dnfmodule.rb +3 -3
  313. data/lib/puppet/provider/package/dpkg.rb +11 -13
  314. data/lib/puppet/provider/package/fink.rb +14 -14
  315. data/lib/puppet/provider/package/freebsd.rb +1 -1
  316. data/lib/puppet/provider/package/gem.rb +12 -14
  317. data/lib/puppet/provider/package/macports.rb +2 -2
  318. data/lib/puppet/provider/package/nim.rb +10 -10
  319. data/lib/puppet/provider/package/openbsd.rb +22 -22
  320. data/lib/puppet/provider/package/opkg.rb +6 -6
  321. data/lib/puppet/provider/package/pacman.rb +35 -29
  322. data/lib/puppet/provider/package/pip.rb +25 -24
  323. data/lib/puppet/provider/package/pkg.rb +15 -15
  324. data/lib/puppet/provider/package/pkgdmg.rb +10 -10
  325. data/lib/puppet/provider/package/pkgin.rb +2 -2
  326. data/lib/puppet/provider/package/pkgng.rb +5 -5
  327. data/lib/puppet/provider/package/pkgutil.rb +17 -21
  328. data/lib/puppet/provider/package/portage.rb +59 -61
  329. data/lib/puppet/provider/package/ports.rb +12 -11
  330. data/lib/puppet/provider/package/portupgrade.rb +35 -35
  331. data/lib/puppet/provider/package/puppetserver_gem.rb +10 -9
  332. data/lib/puppet/provider/package/rpm.rb +26 -26
  333. data/lib/puppet/provider/package/rug.rb +7 -9
  334. data/lib/puppet/provider/package/sun.rb +4 -4
  335. data/lib/puppet/provider/package/up2date.rb +5 -7
  336. data/lib/puppet/provider/package/urpmi.rb +6 -6
  337. data/lib/puppet/provider/package/windows/exe_package.rb +7 -7
  338. data/lib/puppet/provider/package/windows/msi_package.rb +3 -3
  339. data/lib/puppet/provider/package/windows/package.rb +2 -2
  340. data/lib/puppet/provider/package/windows.rb +3 -5
  341. data/lib/puppet/provider/package/yum.rb +33 -37
  342. data/lib/puppet/provider/package/zypper.rb +12 -14
  343. data/lib/puppet/provider/package.rb +1 -1
  344. data/lib/puppet/provider/package_targetable.rb +1 -1
  345. data/lib/puppet/provider/parsedfile.rb +9 -9
  346. data/lib/puppet/provider/service/base.rb +16 -21
  347. data/lib/puppet/provider/service/bsd.rb +4 -4
  348. data/lib/puppet/provider/service/daemontools.rb +31 -31
  349. data/lib/puppet/provider/service/debian.rb +6 -6
  350. data/lib/puppet/provider/service/freebsd.rb +36 -36
  351. data/lib/puppet/provider/service/gentoo.rb +6 -6
  352. data/lib/puppet/provider/service/init.rb +19 -19
  353. data/lib/puppet/provider/service/launchd.rb +22 -22
  354. data/lib/puppet/provider/service/openbsd.rb +10 -10
  355. data/lib/puppet/provider/service/openrc.rb +1 -1
  356. data/lib/puppet/provider/service/openwrt.rb +3 -3
  357. data/lib/puppet/provider/service/rcng.rb +3 -3
  358. data/lib/puppet/provider/service/redhat.rb +4 -4
  359. data/lib/puppet/provider/service/runit.rb +6 -6
  360. data/lib/puppet/provider/service/service.rb +5 -7
  361. data/lib/puppet/provider/service/smf.rb +28 -27
  362. data/lib/puppet/provider/service/src.rb +7 -7
  363. data/lib/puppet/provider/service/systemd.rb +31 -37
  364. data/lib/puppet/provider/service/upstart.rb +17 -17
  365. data/lib/puppet/provider/service/windows.rb +7 -7
  366. data/lib/puppet/provider/user/aix.rb +4 -2
  367. data/lib/puppet/provider/user/directoryservice.rb +34 -36
  368. data/lib/puppet/provider/user/hpux.rb +10 -14
  369. data/lib/puppet/provider/user/pw.rb +1 -1
  370. data/lib/puppet/provider/user/user_role_add.rb +16 -18
  371. data/lib/puppet/provider/user/useradd.rb +7 -7
  372. data/lib/puppet/provider/user/windows_adsi.rb +1 -1
  373. data/lib/puppet/provider.rb +8 -8
  374. data/lib/puppet/reference/indirection.rb +3 -0
  375. data/lib/puppet/reference/metaparameter.rb +1 -1
  376. data/lib/puppet/reference/providers.rb +3 -3
  377. data/lib/puppet/reference/type.rb +1 -1
  378. data/lib/puppet/relationship.rb +4 -5
  379. data/lib/puppet/reports/http.rb +1 -1
  380. data/lib/puppet/reports/log.rb +2 -2
  381. data/lib/puppet/reports/store.rb +4 -4
  382. data/lib/puppet/reports.rb +2 -2
  383. data/lib/puppet/resource/catalog.rb +16 -16
  384. data/lib/puppet/resource/status.rb +3 -3
  385. data/lib/puppet/resource/type.rb +8 -8
  386. data/lib/puppet/resource/type_collection.rb +1 -1
  387. data/lib/puppet/resource.rb +41 -41
  388. data/lib/puppet/scheduler/scheduler.rb +1 -1
  389. data/lib/puppet/settings/alias_setting.rb +3 -5
  390. data/lib/puppet/settings/base_setting.rb +11 -11
  391. data/lib/puppet/settings/boolean_setting.rb +2 -2
  392. data/lib/puppet/settings/config_file.rb +5 -4
  393. data/lib/puppet/settings/duration_setting.rb +2 -2
  394. data/lib/puppet/settings/environment_conf.rb +3 -3
  395. data/lib/puppet/settings/file_or_directory_setting.rb +2 -2
  396. data/lib/puppet/settings/file_setting.rb +11 -11
  397. data/lib/puppet/settings/ini_file.rb +3 -3
  398. data/lib/puppet/settings/port_setting.rb +1 -1
  399. data/lib/puppet/settings/priority_setting.rb +2 -2
  400. data/lib/puppet/settings/ttl_setting.rb +2 -2
  401. data/lib/puppet/settings/value_translator.rb +8 -8
  402. data/lib/puppet/settings.rb +65 -70
  403. data/lib/puppet/ssl/base.rb +2 -4
  404. data/lib/puppet/ssl/certificate_request.rb +18 -22
  405. data/lib/puppet/ssl/certificate_request_attributes.rb +3 -3
  406. data/lib/puppet/ssl/oids.rb +2 -2
  407. data/lib/puppet/ssl/openssl_loader.rb +4 -4
  408. data/lib/puppet/syntax_checkers/base64.rb +3 -3
  409. data/lib/puppet/syntax_checkers/epp.rb +3 -3
  410. data/lib/puppet/syntax_checkers/json.rb +3 -3
  411. data/lib/puppet/syntax_checkers/pp.rb +3 -3
  412. data/lib/puppet/transaction/additional_resource_generator.rb +1 -1
  413. data/lib/puppet/transaction/event.rb +2 -2
  414. data/lib/puppet/transaction/event_manager.rb +5 -5
  415. data/lib/puppet/transaction/persistence.rb +12 -14
  416. data/lib/puppet/transaction/report.rb +4 -4
  417. data/lib/puppet/transaction/resource_harness.rb +2 -2
  418. data/lib/puppet/transaction.rb +14 -18
  419. data/lib/puppet/type/component.rb +3 -3
  420. data/lib/puppet/type/exec.rb +43 -45
  421. data/lib/puppet/type/file/checksum_value.rb +1 -1
  422. data/lib/puppet/type/file/content.rb +6 -6
  423. data/lib/puppet/type/file/ctime.rb +2 -2
  424. data/lib/puppet/type/file/data_sync.rb +3 -3
  425. data/lib/puppet/type/file/ensure.rb +16 -15
  426. data/lib/puppet/type/file/mode.rb +9 -9
  427. data/lib/puppet/type/file/mtime.rb +2 -2
  428. data/lib/puppet/type/file/selcontext.rb +14 -14
  429. data/lib/puppet/type/file/source.rb +19 -21
  430. data/lib/puppet/type/file/target.rb +11 -11
  431. data/lib/puppet/type/file.rb +21 -21
  432. data/lib/puppet/type/filebucket.rb +3 -3
  433. data/lib/puppet/type/group.rb +3 -3
  434. data/lib/puppet/type/notify.rb +2 -2
  435. data/lib/puppet/type/package.rb +4 -4
  436. data/lib/puppet/type/resources.rb +6 -6
  437. data/lib/puppet/type/schedule.rb +9 -9
  438. data/lib/puppet/type/service.rb +6 -8
  439. data/lib/puppet/type/tidy.rb +22 -24
  440. data/lib/puppet/type/user.rb +13 -13
  441. data/lib/puppet/type.rb +76 -88
  442. data/lib/puppet/util/at_fork/solaris.rb +33 -37
  443. data/lib/puppet/util/autoload.rb +2 -2
  444. data/lib/puppet/util/backups.rb +9 -9
  445. data/lib/puppet/util/character_encoding.rb +7 -6
  446. data/lib/puppet/util/checksums.rb +2 -2
  447. data/lib/puppet/util/classgen.rb +8 -8
  448. data/lib/puppet/util/colors.rb +1 -1
  449. data/lib/puppet/util/command_line/puppet_option_parser.rb +1 -1
  450. data/lib/puppet/util/command_line/trollop.rb +42 -46
  451. data/lib/puppet/util/command_line.rb +2 -2
  452. data/lib/puppet/util/constant_inflector.rb +2 -2
  453. data/lib/puppet/util/diff.rb +19 -21
  454. data/lib/puppet/util/docs.rb +2 -2
  455. data/lib/puppet/util/errors.rb +9 -9
  456. data/lib/puppet/util/execution.rb +32 -9
  457. data/lib/puppet/util/feature.rb +1 -1
  458. data/lib/puppet/util/fileparsing.rb +12 -16
  459. data/lib/puppet/util/filetype.rb +36 -40
  460. data/lib/puppet/util/inifile.rb +6 -12
  461. data/lib/puppet/util/ldap/connection.rb +1 -1
  462. data/lib/puppet/util/ldap/manager.rb +10 -12
  463. data/lib/puppet/util/lockfile.rb +6 -8
  464. data/lib/puppet/util/log/destination.rb +2 -2
  465. data/lib/puppet/util/log/destinations.rb +4 -4
  466. data/lib/puppet/util/log.rb +12 -9
  467. data/lib/puppet/util/logging.rb +11 -11
  468. data/lib/puppet/util/metric.rb +3 -3
  469. data/lib/puppet/util/monkey_patches.rb +5 -7
  470. data/lib/puppet/util/network_device/config.rb +3 -3
  471. data/lib/puppet/util/package/version/debian.rb +27 -28
  472. data/lib/puppet/util/package/version/pip.rb +5 -5
  473. data/lib/puppet/util/package/version/range/eq.rb +1 -1
  474. data/lib/puppet/util/package/version/rpm.rb +1 -1
  475. data/lib/puppet/util/package.rb +2 -2
  476. data/lib/puppet/util/pidlock.rb +2 -2
  477. data/lib/puppet/util/platform.rb +1 -1
  478. data/lib/puppet/util/plist.rb +8 -10
  479. data/lib/puppet/util/posix.rb +17 -17
  480. data/lib/puppet/util/profiler/aggregate.rb +1 -1
  481. data/lib/puppet/util/provider_features.rb +3 -3
  482. data/lib/puppet/util/rdoc/code_objects.rb +3 -3
  483. data/lib/puppet/util/rdoc/generators/puppet_generator.rb +63 -64
  484. data/lib/puppet/util/rdoc/generators/template/puppet/puppet.rb +23 -23
  485. data/lib/puppet/util/rdoc/parser/puppet_parser_core.rb +12 -12
  486. data/lib/puppet/util/rdoc/parser/puppet_parser_rdoc2.rb +1 -1
  487. data/lib/puppet/util/rdoc/parser.rb +1 -1
  488. data/lib/puppet/util/reference.rb +13 -9
  489. data/lib/puppet/util/retry_action.rb +1 -1
  490. data/lib/puppet/util/rpm_compare.rb +17 -18
  491. data/lib/puppet/util/rubygems.rb +1 -1
  492. data/lib/puppet/util/selinux.rb +7 -7
  493. data/lib/puppet/util/storage.rb +10 -12
  494. data/lib/puppet/util/suidmanager.rb +18 -19
  495. data/lib/puppet/util/symbolic_file_mode.rb +71 -75
  496. data/lib/puppet/util/tag_set.rb +2 -2
  497. data/lib/puppet/util/tagging.rb +8 -10
  498. data/lib/puppet/util/terminal.rb +4 -4
  499. data/lib/puppet/util/user_attr.rb +7 -7
  500. data/lib/puppet/util/watcher.rb +3 -5
  501. data/lib/puppet/util/windows/access_control_list.rb +1 -1
  502. data/lib/puppet/util/windows/adsi.rb +42 -42
  503. data/lib/puppet/util/windows/daemon.rb +64 -68
  504. data/lib/puppet/util/windows/error.rb +2 -2
  505. data/lib/puppet/util/windows/file.rb +18 -25
  506. data/lib/puppet/util/windows/monkey_patches/process.rb +1 -1
  507. data/lib/puppet/util/windows/principal.rb +8 -8
  508. data/lib/puppet/util/windows/process.rb +16 -28
  509. data/lib/puppet/util/windows/registry.rb +16 -15
  510. data/lib/puppet/util/windows/root_certs.rb +2 -2
  511. data/lib/puppet/util/windows/security.rb +31 -31
  512. data/lib/puppet/util/windows/service.rb +11 -11
  513. data/lib/puppet/util/windows/sid.rb +8 -8
  514. data/lib/puppet/util/windows/user.rb +18 -20
  515. data/lib/puppet/util/yaml.rb +1 -1
  516. data/lib/puppet/util.rb +55 -46
  517. data/lib/puppet/version.rb +2 -2
  518. data/lib/puppet/x509/pem_store.rb +1 -1
  519. data/lib/puppet.rb +3 -3
  520. data/man/man5/puppet.conf.5 +10 -2
  521. data/man/man8/puppet-agent.8 +1 -1
  522. data/man/man8/puppet-apply.8 +1 -1
  523. data/man/man8/puppet-catalog.8 +1 -1
  524. data/man/man8/puppet-config.8 +1 -1
  525. data/man/man8/puppet-describe.8 +1 -1
  526. data/man/man8/puppet-device.8 +1 -1
  527. data/man/man8/puppet-doc.8 +1 -1
  528. data/man/man8/puppet-epp.8 +1 -1
  529. data/man/man8/puppet-facts.8 +1 -1
  530. data/man/man8/puppet-filebucket.8 +1 -1
  531. data/man/man8/puppet-generate.8 +1 -1
  532. data/man/man8/puppet-help.8 +1 -1
  533. data/man/man8/puppet-lookup.8 +1 -1
  534. data/man/man8/puppet-module.8 +1 -1
  535. data/man/man8/puppet-node.8 +1 -1
  536. data/man/man8/puppet-parser.8 +1 -1
  537. data/man/man8/puppet-plugin.8 +1 -1
  538. data/man/man8/puppet-report.8 +1 -1
  539. data/man/man8/puppet-resource.8 +5 -1
  540. data/man/man8/puppet-script.8 +1 -1
  541. data/man/man8/puppet-ssl.8 +1 -1
  542. data/man/man8/puppet.8 +2 -2
  543. metadata +2 -2
@@ -146,7 +146,7 @@ Puppet::Functions.create_function(:max) do
146
146
  assert_arg_count(args)
147
147
 
148
148
  args.max do |a, b|
149
- if a.to_s =~ %r{\A^-?\d+([._eE]\d+)?\z} && b.to_s =~ %r{\A-?\d+([._eE]\d+)?\z}
149
+ if a.to_s =~ /\A^-?\d+([._eE]\d+)?\z/ && b.to_s =~ /\A-?\d+([._eE]\d+)?\z/
150
150
  Puppet.warn_once('deprecations', 'max_function_numeric_coerce_string',
151
151
  _("The max() function's auto conversion of String to Numeric is deprecated - change to convert input before calling, or use lambda"))
152
152
  a.to_f <=> b.to_f
@@ -232,7 +232,7 @@ Puppet::Functions.create_function(:max) do
232
232
  args.max do |a, b|
233
233
  as = a.to_s
234
234
  bs = b.to_s
235
- if as =~ %r{\A^-?\d+([._eE]\d+)?\z} && bs =~ %r{\A-?\d+([._eE]\d+)?\z}
235
+ if as =~ /\A^-?\d+([._eE]\d+)?\z/ && bs =~ /\A-?\d+([._eE]\d+)?\z/
236
236
  Puppet.warn_once('deprecations', 'max_function_numeric_coerce_string',
237
237
  _("The max() function's auto conversion of String to Numeric is deprecated - change to convert input before calling, or use lambda"))
238
238
  a.to_f <=> b.to_f
@@ -145,7 +145,7 @@ Puppet::Functions.create_function(:min) do
145
145
  assert_arg_count(args)
146
146
 
147
147
  args.min do |a, b|
148
- if a.to_s =~ %r{\A^-?\d+([._eE]\d+)?\z} && b.to_s =~ %r{\A-?\d+([._eE]\d+)?\z}
148
+ if a.to_s =~ /\A^-?\d+([._eE]\d+)?\z/ && b.to_s =~ /\A-?\d+([._eE]\d+)?\z/
149
149
  Puppet.warn_once('deprecations', 'min_function_numeric_coerce_string',
150
150
  _("The min() function's auto conversion of String to Numeric is deprecated - change to convert input before calling, or use lambda"))
151
151
  a.to_f <=> b.to_f
@@ -231,7 +231,7 @@ Puppet::Functions.create_function(:min) do
231
231
  args.min do |a, b|
232
232
  as = a.to_s
233
233
  bs = b.to_s
234
- if as =~ %r{\A^-?\d+([._eE]\d+)?\z} && bs =~ %r{\A-?\d+([._eE]\d+)?\z}
234
+ if as =~ /\A^-?\d+([._eE]\d+)?\z/ && bs =~ /\A-?\d+([._eE]\d+)?\z/
235
235
  Puppet.warn_once('deprecations', 'min_function_numeric_coerce_string',
236
236
  _("The min() function's auto conversion of String to Numeric is deprecated - change to convert input before calling, or use lambda"))
237
237
  a.to_f <=> b.to_f
@@ -994,7 +994,7 @@ Puppet::Functions.create_function(:new, Puppet::Functions::InternalFunction) do
994
994
  return args[0] if args.size == 1 && !t.is_a?(Puppet::Pops::Types::PInitType) && t.instance?(args[0])
995
995
 
996
996
  result = assert_type(t, new_function_for_type(t).call(scope, *args))
997
- return block_given? ? yield(result) : result
997
+ block_given? ? yield(result) : result
998
998
  end
999
999
 
1000
1000
  def new_function_for_type(t)
@@ -142,22 +142,18 @@ Puppet::Functions.create_function(:reduce) do
142
142
  def reduce_without_memo(enumerable)
143
143
  enum = Puppet::Pops::Types::Iterable.asserted_iterable(self, enumerable)
144
144
  enum.reduce do |memo, x|
145
- begin
146
- yield(memo, x)
147
- rescue StopIteration
148
- return memo
149
- end
145
+ yield(memo, x)
146
+ rescue StopIteration
147
+ return memo
150
148
  end
151
149
  end
152
150
 
153
151
  def reduce_with_memo(enumerable, given_memo)
154
152
  enum = Puppet::Pops::Types::Iterable.asserted_iterable(self, enumerable)
155
153
  enum.reduce(given_memo) do |memo, x|
156
- begin
157
- yield(memo, x)
158
- rescue StopIteration
159
- return memo
160
- end
154
+ yield(memo, x)
155
+ rescue StopIteration
156
+ return memo
161
157
  end
162
158
  end
163
159
  end
@@ -78,7 +78,7 @@ Puppet::Functions.create_function(:regsubst) do
78
78
  def regsubst_string(target, pattern, replacement, flags = nil, encoding = nil)
79
79
  re_flags = 0
80
80
  operation = :sub
81
- if !flags.nil?
81
+ unless flags.nil?
82
82
  flags.split(//).each do |f|
83
83
  case f
84
84
  when 'G' then operation = :gsub
@@ -92,7 +92,7 @@ Puppet::Functions.create_function(:regsubst) do
92
92
  end
93
93
 
94
94
  def regsubst_regexp(target, pattern, replacement, flags = nil)
95
- pattern = (pattern.pattern || '') if pattern.is_a?(Puppet::Pops::Types::PRegexpType)
95
+ pattern = pattern.pattern || '' if pattern.is_a?(Puppet::Pops::Types::PRegexpType)
96
96
  inner_regsubst(target, pattern, replacement, flags == 'G' ? :gsub : :sub)
97
97
  end
98
98
 
@@ -70,7 +70,7 @@ Puppet::Functions.create_function(:require, Puppet::Functions::InternalFunction)
70
70
  classes.each do |klass|
71
71
  # lookup the class in the scopes
72
72
  klass = (classobj = krt.find_hostclass(klass)) ? classobj.name : nil
73
- raise Puppet::ParseError.new(_("Could not find class %{klass}") % { klass: klass }) unless klass
73
+ raise Puppet::ParseError, _("Could not find class %{klass}") % { klass: klass } unless klass
74
74
 
75
75
  ref = Puppet::Resource.new(:class, klass)
76
76
  resource = scope.resource
@@ -22,21 +22,19 @@ Puppet::Functions.create_function(:yaml_data) do
22
22
  def yaml_data(options, context)
23
23
  path = options['path']
24
24
  context.cached_file_data(path) do |content|
25
- begin
26
- data = Puppet::Util::Yaml.safe_load(content, [Symbol], path)
27
- if data.is_a?(Hash)
28
- Puppet::Pops::Lookup::HieraConfig.symkeys_to_string(data)
29
- else
30
- msg = _("%{path}: file does not contain a valid yaml hash" % { path: path })
31
- raise Puppet::DataBinding::LookupError, msg if Puppet[:strict] == :error && data != false
25
+ data = Puppet::Util::Yaml.safe_load(content, [Symbol], path)
26
+ if data.is_a?(Hash)
27
+ Puppet::Pops::Lookup::HieraConfig.symkeys_to_string(data)
28
+ else
29
+ msg = _("%{path}: file does not contain a valid yaml hash" % { path: path })
30
+ raise Puppet::DataBinding::LookupError, msg if Puppet[:strict] == :error && data != false
32
31
 
33
- Puppet.warning(msg)
34
- {}
35
- end
36
- rescue Puppet::Util::Yaml::YamlLoadError => ex
37
- # YamlLoadErrors include the absolute path to the file, so no need to add that
38
- raise Puppet::DataBinding::LookupError, _("Unable to parse %{message}") % { message: ex.message }
32
+ Puppet.warning(msg)
33
+ {}
39
34
  end
35
+ rescue Puppet::Util::Yaml::YamlLoadError => ex
36
+ # YamlLoadErrors include the absolute path to the file, so no need to add that
37
+ raise Puppet::DataBinding::LookupError, _("Unable to parse %{message}") % { message: ex.message }
40
38
  end
41
39
  end
42
40
 
@@ -188,12 +188,11 @@ module Puppet::Functions
188
188
  # and it will fail unless protected with an if defined? if the local
189
189
  # variable does not exist in the block's binder.
190
190
  #
191
- begin
192
- loader = block.binding.eval('loader_injected_arg if defined?(loader_injected_arg)')
193
- create_loaded_function(func_name, loader, function_base, &block)
194
- rescue StandardError => e
195
- raise ArgumentError, _("Function Load Error for function '%{function_name}': %{message}") % { function_name: func_name, message: e.message }
196
- end
191
+
192
+ loader = block.binding.eval('loader_injected_arg if defined?(loader_injected_arg)')
193
+ create_loaded_function(func_name, loader, function_base, &block)
194
+ rescue StandardError => e
195
+ raise ArgumentError, _("Function Load Error for function '%{function_name}': %{message}") % { function_name: func_name, message: e.message }
197
196
  end
198
197
 
199
198
  # Creates a function in, or in a local loader under the given loader.
@@ -570,7 +569,7 @@ module Puppet::Functions
570
569
  @block_name = nil
571
570
  @return_type = nil
572
571
  @argument_mismatch_hander = argument_mismatch_handler
573
- self.instance_eval(&block)
572
+ instance_eval(&block)
574
573
  callable_t = create_callable(@types, @block_type, @return_type, @min, @max)
575
574
  @dispatcher.add(Puppet::Pops::Functions::Dispatch.new(callable_t, meth_name, @names, @max == :default, @block_name, @injections, @weaving, @argument_mismatch_hander))
576
575
  end
@@ -591,14 +590,12 @@ module Puppet::Functions
591
590
  end
592
591
 
593
592
  def internal_type_parse(type_string, loader)
594
- begin
595
- Puppet::Pops::Types::TypeParser.singleton.parse(type_string, loader)
596
- rescue StandardError => e
597
- raise ArgumentError, _("Parsing of type string '\"%{type_string}\"' failed with message: <%{message}>.\n") % {
598
- type_string: type_string,
599
- message: e.message
600
- }
601
- end
593
+ Puppet::Pops::Types::TypeParser.singleton.parse(type_string, loader)
594
+ rescue StandardError => e
595
+ raise ArgumentError, _("Parsing of type string '\"%{type_string}\"' failed with message: <%{message}>.\n") % {
596
+ type_string: type_string,
597
+ message: e.message
598
+ }
602
599
  end
603
600
  private :internal_type_parse
604
601
  end
@@ -688,10 +685,10 @@ module Puppet::Functions
688
685
  PARAM_NAMES = [
689
686
  [],
690
687
  ['p0'].freeze,
691
- ['p0', 'p1'].freeze,
692
- ['p0', 'p1', 'p2'].freeze,
693
- ['p0', 'p1', 'p2', 'p3'].freeze,
694
- ['p0', 'p1', 'p2', 'p3', 'p4'].freeze,
688
+ %w[p0 p1].freeze,
689
+ %w[p0 p1 p2].freeze,
690
+ %w[p0 p1 p2 p3].freeze,
691
+ %w[p0 p1 p2 p3 p4].freeze
695
692
  ]
696
693
 
697
694
  # Creates an anonymous Function3x class that wraps a 3x function
@@ -746,25 +743,23 @@ module Puppet::Functions
746
743
  # used to support other features than calling.
747
744
  #
748
745
  def call(scope, *args, &block)
746
+ result = catch(:return) do
747
+ mapped_args = Puppet::Pops::Evaluator::Runtime3FunctionArgumentConverter.map_args(args, scope, '')
748
+ # this is the scope.function_xxx(...) call
749
+ return scope.send(self.class.method3x, mapped_args)
750
+ end
751
+ result.value
752
+ rescue Puppet::Pops::Evaluator::Next => jumper
753
+ begin
754
+ throw :next, jumper.value
755
+ rescue Puppet::Parser::Scope::UNCAUGHT_THROW_EXCEPTION
756
+ raise Puppet::ParseError.new("next() from context where this is illegal", jumper.file, jumper.line)
757
+ end
758
+ rescue Puppet::Pops::Evaluator::Return => jumper
749
759
  begin
750
- result = catch(:return) do
751
- mapped_args = Puppet::Pops::Evaluator::Runtime3FunctionArgumentConverter.map_args(args, scope, '')
752
- # this is the scope.function_xxx(...) call
753
- return scope.send(self.class.method3x, mapped_args)
754
- end
755
- return result.value
756
- rescue Puppet::Pops::Evaluator::Next => jumper
757
- begin
758
- throw :next, jumper.value
759
- rescue Puppet::Parser::Scope::UNCAUGHT_THROW_EXCEPTION
760
- raise Puppet::ParseError.new("next() from context where this is illegal", jumper.file, jumper.line)
761
- end
762
- rescue Puppet::Pops::Evaluator::Return => jumper
763
- begin
764
- throw :return, jumper
765
- rescue Puppet::Parser::Scope::UNCAUGHT_THROW_EXCEPTION
766
- raise Puppet::ParseError.new("return() from context where this is illegal", jumper.file, jumper.line)
767
- end
760
+ throw :return, jumper
761
+ rescue Puppet::Parser::Scope::UNCAUGHT_THROW_EXCEPTION
762
+ raise Puppet::ParseError.new("return() from context where this is illegal", jumper.file, jumper.line)
768
763
  end
769
764
  end
770
765
  end
@@ -135,7 +135,7 @@ module Puppet
135
135
  end
136
136
 
137
137
  # Sort the inputs by path
138
- inputs.sort_by! { |input| input.path }
138
+ inputs.sort_by!(&:path)
139
139
  end
140
140
 
141
141
  def self.bad_input?
@@ -155,7 +155,7 @@ module Puppet
155
155
  def self.generate(inputs, outputdir = nil, force = false)
156
156
  # remove files for non existing inputs
157
157
  unless outputdir.nil?
158
- filenames_to_keep = inputs.map { |i| i.output_name }
158
+ filenames_to_keep = inputs.map(&:output_name)
159
159
  existing_files = Puppet::FileSystem.children(outputdir).map { |f| Puppet::FileSystem.basename(f) }
160
160
  files_to_remove = existing_files - filenames_to_keep
161
161
  files_to_remove.each do |f|
@@ -43,9 +43,9 @@ module Puppet::GettextConfig
43
43
  # @return [String] the active locale
44
44
  def self.current_locale
45
45
  if gettext_loaded?
46
- return FastGettext.default_locale
46
+ FastGettext.default_locale
47
47
  else
48
- return 'en'
48
+ 'en'
49
49
  end
50
50
  end
51
51
 
@@ -55,7 +55,7 @@ module Puppet::GettextConfig
55
55
  def self.loaded_text_domains
56
56
  return [] if @gettext_disabled || !gettext_loaded?
57
57
 
58
- return FastGettext.translation_repositories.keys
58
+ FastGettext.translation_repositories.keys
59
59
  end
60
60
 
61
61
  # @api private
@@ -191,11 +191,11 @@ module Puppet::GettextConfig
191
191
  # @return [String] path to the config, or nil if not found
192
192
  def self.puppet_locale_path
193
193
  if Puppet::FileSystem.exist?(LOCAL_PATH)
194
- return LOCAL_PATH
194
+ LOCAL_PATH
195
195
  elsif Puppet::Util::Platform.windows? && Puppet::FileSystem.exist?(WINDOWS_PATH)
196
- return WINDOWS_PATH
196
+ WINDOWS_PATH
197
197
  elsif !Puppet::Util::Platform.windows? && Puppet::FileSystem.exist?(POSIX_PATH)
198
- return POSIX_PATH
198
+ POSIX_PATH
199
199
  else
200
200
  nil
201
201
  end
@@ -207,9 +207,9 @@ module Puppet::GettextConfig
207
207
  # @return [Symbol] :mo if in a package structure, :po otherwise
208
208
  def self.translation_mode(conf_path)
209
209
  if WINDOWS_PATH == conf_path || POSIX_PATH == conf_path
210
- return :mo
210
+ :mo
211
211
  else
212
- return :po
212
+ :po
213
213
  end
214
214
  end
215
215
 
@@ -239,7 +239,7 @@ module Puppet::GettextConfig
239
239
  end
240
240
 
241
241
  add_repository_to_domain(project_name, locale_dir, file_format, text_domain)
242
- return true
242
+ true
243
243
  end
244
244
 
245
245
  # @api private
@@ -236,33 +236,41 @@ class Puppet::Graph::RbTreeMap
236
236
 
237
237
  def rotate_left
238
238
  r = @right
239
- r_key, r_value = r.key, r.value
239
+ r_key = r.key
240
+ r_value = r.value
240
241
  b = r.left
241
242
  r.left = @left
242
243
  @left = r
243
244
  @right = r.right
244
245
  r.right = b
245
- r.color, r.key, r.value = :red, @key, @value
246
- @key, @value = r_key, r_value
246
+ r.color = :red
247
+ r.key = @key
248
+ r.value = @value
249
+ @key = r_key
250
+ @value = r_value
247
251
  self
248
252
  end
249
253
 
250
254
  def rotate_right
251
255
  l = @left
252
- l_key, l_value = l.key, l.value
256
+ l_key = l.key
257
+ l_value = l.value
253
258
  b = l.right
254
259
  l.right = @right
255
260
  @right = l
256
261
  @left = l.left
257
262
  l.left = b
258
- l.color, l.key, l.value = :red, @key, @value
259
- @key, @value = l_key, l_value
263
+ l.color = :red
264
+ l.key = @key
265
+ l.value = @value
266
+ @key = l_key
267
+ @value = l_value
260
268
  self
261
269
  end
262
270
 
263
271
  def move_red_left
264
272
  colorflip
265
- if (@right.left && @right.left.red?)
273
+ if @right.left && @right.left.red?
266
274
  @right.rotate_right
267
275
  rotate_left
268
276
  colorflip
@@ -272,7 +280,7 @@ class Puppet::Graph::RbTreeMap
272
280
 
273
281
  def move_red_right
274
282
  colorflip
275
- if (@left.left && @left.left.red?)
283
+ if @left.left && @left.left.red?
276
284
  rotate_right
277
285
  colorflip
278
286
  end
@@ -300,15 +308,15 @@ class Puppet::Graph::RbTreeMap
300
308
 
301
309
  def delete_recursive(node, key)
302
310
  if (key <=> node.key) == -1
303
- node.move_red_left if (!isred(node.left) && !isred(node.left.left))
311
+ node.move_red_left if !isred(node.left) && !isred(node.left.left)
304
312
  node.left, result = delete_recursive(node.left, key)
305
313
  else
306
314
  node.rotate_right if isred(node.left)
307
- if (((key <=> node.key) == 0) && node.right.nil?)
315
+ if ((key <=> node.key) == 0) && node.right.nil?
308
316
  return nil, node.value
309
317
  end
310
318
 
311
- if (!isred(node.right) && !isred(node.right.left))
319
+ if !isred(node.right) && !isred(node.right.left)
312
320
  node.move_red_right
313
321
  end
314
322
  if (key <=> node.key) == 0
@@ -321,7 +329,7 @@ class Puppet::Graph::RbTreeMap
321
329
  node.right, result = delete_recursive(node.right, key)
322
330
  end
323
331
  end
324
- return node.fixup, result
332
+ [node.fixup, result]
325
333
  end
326
334
 
327
335
  def delete_min_recursive(node)
@@ -329,35 +337,35 @@ class Puppet::Graph::RbTreeMap
329
337
  return nil, node.value
330
338
  end
331
339
 
332
- if (!isred(node.left) && !isred(node.left.left))
340
+ if !isred(node.left) && !isred(node.left.left)
333
341
  node.move_red_left
334
342
  end
335
343
  node.left, result = delete_min_recursive(node.left)
336
344
 
337
- return node.fixup, result
345
+ [node.fixup, result]
338
346
  end
339
347
 
340
348
  def delete_max_recursive(node)
341
- if (isred(node.left))
349
+ if isred(node.left)
342
350
  node = node.rotate_right
343
351
  end
344
352
  return nil, node.value if node.right.nil?
345
353
 
346
- if (!isred(node.right) && !isred(node.right.left))
354
+ if !isred(node.right) && !isred(node.right.left)
347
355
  node.move_red_right
348
356
  end
349
357
  node.right, result = delete_max_recursive(node.right)
350
358
 
351
- return node.fixup, result
359
+ [node.fixup, result]
352
360
  end
353
361
 
354
362
  def get_recursive(node, key)
355
363
  return nil if node.nil?
356
364
 
357
365
  case key <=> node.key
358
- when 0 then return node
359
- when -1 then return get_recursive(node.left, key)
360
- when 1 then return get_recursive(node.right, key)
366
+ when 0 then node
367
+ when -1 then get_recursive(node.left, key)
368
+ when 1 then get_recursive(node.right, key)
361
369
  end
362
370
  end
363
371
 
@@ -385,9 +393,9 @@ class Puppet::Graph::RbTreeMap
385
393
  when 1 then node.right = insert(node.right, key, value)
386
394
  end
387
395
 
388
- node.rotate_left if (node.right && node.right.red?)
389
- node.rotate_right if (node.left && node.left.red? && node.left.left && node.left.left.red?)
390
- node.colorflip if (node.left && node.left.red? && node.right && node.right.red?)
396
+ node.rotate_left if node.right && node.right.red?
397
+ node.rotate_right if node.left && node.left.red? && node.left.left && node.left.left.red?
398
+ node.colorflip if node.left && node.left.red? && node.right && node.right.red?
391
399
  node
392
400
  end
393
401
 
@@ -128,22 +128,22 @@ class Puppet::Graph::RelationshipGraph < Puppet::Graph::SimpleGraph
128
128
  deferred_resources << resource
129
129
  end
130
130
 
131
- if @ready.empty? and deferred_resources.any?
132
- if made_progress
133
- enqueue(*deferred_resources)
134
- else
135
- deferred_resources.each do |res|
136
- overly_deferred_resource_handler.call(res)
137
- finish(res)
138
- end
139
- end
131
+ next unless @ready.empty? and deferred_resources.any?
140
132
 
141
- made_progress = false
142
- deferred_resources = []
133
+ if made_progress
134
+ enqueue(*deferred_resources)
135
+ else
136
+ deferred_resources.each do |res|
137
+ overly_deferred_resource_handler.call(res)
138
+ finish(res)
139
+ end
143
140
  end
141
+
142
+ made_progress = false
143
+ deferred_resources = []
144
144
  end
145
145
 
146
- if !continue_while.call()
146
+ unless continue_while.call()
147
147
  while (resource = next_resource)
148
148
  canceled_resource_handler.call(resource)
149
149
  finish(resource)
@@ -106,7 +106,7 @@ class Puppet::Graph::SimpleGraph
106
106
  # decent Ruby stack.
107
107
  recur = [{ :node => root }]
108
108
 
109
- while not recur.empty? do
109
+ until recur.empty?
110
110
  frame = recur.last
111
111
  vertex = frame[:node]
112
112
 
@@ -171,7 +171,7 @@ class Puppet::Graph::SimpleGraph
171
171
 
172
172
  # we usually have a disconnected graph, must walk all possible roots
173
173
  vertices.each do |vertex|
174
- if !state[:index][vertex] then
174
+ unless state[:index][vertex] then
175
175
  tarjan vertex, state
176
176
  end
177
177
  end
@@ -186,9 +186,7 @@ class Puppet::Graph::SimpleGraph
186
186
  # time-scale.
187
187
  state[:scc].select do |component|
188
188
  multi_vertex_component?(component) || single_vertex_referring_to_self?(component)
189
- end.map do |component|
190
- component.sort
191
- end.sort
189
+ end.map(&:sort).sort
192
190
  end
193
191
 
194
192
  # Perform a BFS on the sub graph representing the cycle, with a view to
@@ -212,7 +210,7 @@ class Puppet::Graph::SimpleGraph
212
210
 
213
211
  # frame struct is vertex, [path]
214
212
  stack = [[cycle.first, []]]
215
- while frame = stack.shift do # rubocop:disable Lint/AssignmentInCondition
213
+ while frame = stack.shift # rubocop:disable Lint/AssignmentInCondition
216
214
  if frame[1].member?(frame[0]) then
217
215
  found << frame[1] + [frame[0]]
218
216
  break if found.length >= max_paths
@@ -223,7 +221,7 @@ class Puppet::Graph::SimpleGraph
223
221
  end
224
222
  end
225
223
 
226
- return found.sort
224
+ found.sort
227
225
  end
228
226
 
229
227
  # @return [Array] array of dependency cycles (arrays)
@@ -268,7 +266,7 @@ class Puppet::Graph::SimpleGraph
268
266
  filename = File.join(Puppet[:graphdir], "cycles.dot")
269
267
  # DOT files are assumed to be UTF-8 by default - http://www.graphviz.org/doc/info/lang.html
270
268
  File.open(filename, "w:UTF-8") { |f| f.puts graph }
271
- return filename
269
+ filename
272
270
  end
273
271
 
274
272
  # Add a new vertex to the graph.
@@ -330,7 +328,7 @@ class Puppet::Graph::SimpleGraph
330
328
  end
331
329
 
332
330
  def edges
333
- @in_to.values.collect { |x| x.values }.flatten
331
+ @in_to.values.collect(&:values).flatten
334
332
  end
335
333
 
336
334
  def each_edge
@@ -449,7 +447,7 @@ class Puppet::Graph::SimpleGraph
449
447
  # which match +dot+ properties will be used as well.
450
448
  def to_dot_graph(params = {})
451
449
  params['name'] ||= self.class.name.tr(':', '_')
452
- fontsize = params['fontsize'] ? params['fontsize'] : '8'
450
+ fontsize = params['fontsize'] || '8'
453
451
  graph = (directed? ? DOT::DOTDigraph : DOT::DOTSubgraph).new(params)
454
452
  edge_klass = directed? ? DOT::DOTDirectedEdge : DOT::DOTEdge
455
453
  vertices.each do |v|
@@ -365,7 +365,7 @@ class Puppet::HTTP::Client
365
365
  response = nil
366
366
  done = false
367
367
 
368
- while !done do
368
+ until done
369
369
  connect(request.uri, options: options) do |http|
370
370
  apply_auth(request, basic_auth) if redirects.zero?
371
371
 
@@ -41,7 +41,7 @@ module Puppet::HTTP
41
41
  # @param [Symbol] service_name the key of the service we are querying
42
42
  # @yields [String, Integer] server and port of selected record
43
43
  def each_srv_record(domain, service_name = :puppet, &block)
44
- if (domain.nil? or domain.empty?)
44
+ if domain.nil? or domain.empty?
45
45
  Puppet.debug "Domain not known; skipping SRV lookup"
46
46
  return
47
47
  end
@@ -118,7 +118,7 @@ module Puppet::HTTP
118
118
  # @param [String] service_name the service whose TTL we want
119
119
  # @return [Integer] the TTL for this service, in seconds
120
120
  def ttl(service_name)
121
- return @record_cache[service_name].ttl
121
+ @record_cache[service_name].ttl
122
122
  end
123
123
 
124
124
  # Checks if the cached entry for the given service has expired.
@@ -128,9 +128,9 @@ module Puppet::HTTP
128
128
  def expired?(service_name)
129
129
  entry = @record_cache[service_name]
130
130
  if entry
131
- return Time.now > (entry.resolution_time + entry.ttl)
131
+ Time.now > (entry.resolution_time + entry.ttl)
132
132
  else
133
- return true
133
+ true
134
134
  end
135
135
  end
136
136
 
@@ -39,7 +39,7 @@ class Puppet::HTTP::ExternalClient < Puppet::HTTP::Client
39
39
  # (see Puppet::HTTP::Client#post)
40
40
  # @api private
41
41
  def post(url, body, headers: {}, params: {}, options: {}, &block)
42
- raise ArgumentError.new("'post' requires a string 'body' argument") unless body.is_a?(String)
42
+ raise ArgumentError, "'post' requires a string 'body' argument" unless body.is_a?(String)
43
43
 
44
44
  url = encode_query(url, params)
45
45
 
@@ -17,7 +17,7 @@ class Puppet::HTTP::Factory
17
17
 
18
18
  def initialize
19
19
  # PUP-1411, make sure that openssl is initialized before we try to connect
20
- if !@@openssl_initialized
20
+ unless @@openssl_initialized
21
21
  OpenSSL::SSL::SSLContext.new
22
22
  @@openssl_initialized = true
23
23
  end