openvox 9.0.0.pre.beta1 → 9.0.0.pre.beta2

data/lib/puppet/parser/functions/versioncmp.rb

@@ -23,7 +23,7 @@ Example:
         notice('2.6-1 is > than 2.4.5')
     }
 
-This function uses the same version comparison algorithm used by Puppet's
+This function uses the same version comparison algorithm used by OpenVox's
 `package` type.
 
 ") do |args|

data/lib/puppet/parser/functions/with.rb

@@ -5,7 +5,7 @@ Puppet::Parser::Functions.newfunction(
   :type => :rvalue,
   :arity => -1,
   :doc => <<~DOC
-    Call a [lambda](https://puppet.com/docs/puppet/latest/lang_lambdas.html)
+    Call a [lambda](https://docs.openvoxproject.org/openvox/latest/lang_lambdas.html)
     with the given arguments and return the result. Since a lambda's scope is
     local to the lambda, you can use the `with` function to create private blocks
     of code within a class using variables whose values cannot be accessed outside

data/lib/puppet/parser/functions.rb

@@ -117,15 +117,15 @@ module Puppet::Parser::Functions
   #
   # **The {newfunction} method provides a public API.**
   #
-  # This method is used both internally inside of Puppet to define parser
+  # This method is used both internally inside of OpenVox to define parser
   # functions.  For example, template() is defined in
   # {file:lib/puppet/parser/functions/template.rb template.rb} using the
   # {newfunction} method.  Third party Puppet modules such as
   # [stdlib](https://forge.puppetlabs.com/puppetlabs/stdlib) use this method to
-  # extend the behavior and functionality of Puppet.
+  # extend the behavior and functionality of OpenVox.
   #
   # See also [Docs: Custom
-  # Functions](https://puppet.com/docs/puppet/5.5/lang_write_functions_in_puppet.html)
+  # Functions](https://docs.openvoxproject.org/openvox/latest/lang_write_functions_in_puppet.html)
   #
   # @example Define a new Puppet DSL Function
   #     >> Puppet::Parser::Functions.newfunction(:double, :arity => 1,

data/lib/puppet/parser/scope.rb

@@ -8,7 +8,7 @@ require_relative '../../puppet/parser'
 require_relative '../../puppet/parser/templatewrapper'
 require_relative '../../puppet/parser/resource'
 
-# This class is part of the internal parser/evaluator/compiler functionality of Puppet.
+# This class is part of the internal parser/evaluator/compiler functionality of OpenVox.
 # It is passed between the various classes that participate in evaluation.
 # None of its methods are API except those that are clearly marked as such.
 #

data/lib/puppet/plugins/syntax_checkers.rb

@@ -7,7 +7,7 @@ module SyntaxCheckers
   # @api public
   SYNTAX_CHECKERS_KEY = :'puppet::syntaxcheckers'
 
-  # SyntaxChecker is a Puppet Extension Point for the purpose of extending Puppet with syntax checkers.
+  # SyntaxChecker is an OpenVox Extension Point for the purpose of extending OpenVox with syntax checkers.
   # The intended use is to create a class derived from this class and then register it with the Puppet context
   #
   # Creating the Extension Class
@@ -17,7 +17,7 @@ module SyntaxCheckers
   # e.g. 'Puppetx::Exampleorg::XmlData::XmlChecker" and
   # be located in `lib/puppetx/exampleorg/xml_data/xml_checker.rb`. The Puppet Binder will auto-load this file when it
   # has a binding to the class `Puppetx::Exampleorg::XmlData::XmlChecker'
-  # The Ruby Module `Puppetx` is created by Puppet, the remaining modules should be created by the loaded logic - e.g.:
+  # The Ruby Module `Puppetx` is created by OpenVox, the remaining modules should be created by the loaded logic - e.g.:
   #
   # @example Defining an XmlChecker
   #   module Puppetx::Exampleorg

data/lib/puppet/plugins.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 # The Puppet Plugins module defines extension points where plugins can be configured
-# to add or modify puppet's behavior. See the respective classes in this module for more
+# to add or modify OpenVox's behavior. See the respective classes in this module for more
 # details.
 #
 # @api public

data/lib/puppet/pops/evaluator/collectors/exported_collector.rb

@@ -27,7 +27,7 @@ class Puppet::Pops::Evaluator::Collectors::ExportedCollector < Puppet::Pops::Eva
   end
 
   # Collect exported resources as defined by an exported
-  # collection. Used with PuppetDB
+  # collection. Used with OpenVoxDB
   def collect
     resources = []
 

data/lib/puppet/pops/evaluator/deferred_resolver.rb

@@ -48,7 +48,10 @@ class DeferredResolver
     #    # Server facts are always about the local node's version etc.
     #    @scope.set_server_facts(node.server_facts)
 
-    resolver.resolve_futures(catalog)
+    # Must be wrapped in with_context_overrides to ensure global_scope is valid for Puppet-language functions
+    compiler.with_context_overrides do
+      resolver.resolve_futures(catalog)
+    end
     nil
   end
 

data/lib/puppet/pops/issues.rb

@@ -392,7 +392,7 @@ module Issues
   # Issues when expressions that are not implemented or activated in the current version are used.
   #
   UNSUPPORTED_EXPRESSION = issue :UNSUPPORTED_EXPRESSION do
-    _("Expressions of type %{expression} are not supported in this version of Puppet") % { expression: label.a_an(semantic) }
+    _("Expressions of type %{expression} are not supported in this version of OpenVox") % { expression: label.a_an(semantic) }
   end
 
   ILLEGAL_RELATIONSHIP_OPERAND_TYPE = issue :ILLEGAL_RELATIONSHIP_OPERAND_TYPE, :operand do
@@ -859,11 +859,11 @@ module Issues
   end
 
   HIERA_OPTION_RESERVED_BY_PUPPET = hard_issue :HIERA_OPTION_RESERVED_BY_PUPPET, :key, :name do
-    _("Option key '%{key}' used in hierarchy '%{name}' is reserved by Puppet") % { key: key, name: name }
+    _("Option key '%{key}' used in hierarchy '%{name}' is reserved by OpenVox") % { key: key, name: name }
   end
 
   HIERA_DEFAULT_OPTION_RESERVED_BY_PUPPET = hard_issue :HIERA_DEFAULT_OPTION_RESERVED_BY_PUPPET, :key do
-    _("Option key '%{key}' used in defaults is reserved by Puppet") % { key: key }
+    _("Option key '%{key}' used in defaults is reserved by OpenVox") % { key: key }
   end
 
   HIERA_DATA_PROVIDER_FUNCTION_NOT_FOUND = hard_issue :HIERA_DATA_PROVIDER_FUNCTION_NOT_FOUND, :function_type, :function_name do

data/lib/puppet/pops/loader/module_loaders.rb

@@ -38,10 +38,10 @@ module ModuleLoaders
   end
 
   def self.system_loader_from(parent_loader, loaders)
-    # Puppet system may be installed in a fixed location via RPM, installed as a Gem, via source etc.
-    # The only way to find this across the different ways puppet can be installed is
-    # to search up the path from this source file's __FILE__ location until it finds the base of
-    # puppet.
+    # OpenVox may be installed in a fixed location via RPM, installed as a Gem, via source etc.
+    # The only way to find the OpenVox code across these different installation methods is
+    # to search up the path from this source file's __FILE__ location until it finds the root
+    # of the OpenVox codebase.
     #
     puppet_lib = File.realpath(File.join(File.dirname(__FILE__), '../../..'))
     LibRootedFileBased.new(parent_loader,

data/lib/puppet/pops/loader/ruby_legacy_function_instantiator.rb

@@ -99,7 +99,7 @@ class Puppet::Pops::Loader::RubyLegacyFunctionInstantiator
         method_name: mname,
         source_ref: source_ref,
         line: mline,
-        url: "https://puppet.com/docs/puppet/latest/functions_refactor_legacy.html"
+        url: "https://docs.openvoxproject.org/openvox/latest/functions_refactor_legacy.html"
       }
     end
     x.each { |v| walk(v, source_ref, result) }

data/lib/puppet/pops/loaders.rb

@@ -56,7 +56,7 @@ class Loaders
     # 2. Cache loader(optional) - i.e. what puppet stores on disk via pluginsync; gate behind the for_agent flag.
     # 3. Environment loader - i.e. what is bound across the environment, may change for each setup
     #    TODO: loaders need to work when also running in an agent doing catalog application. There is no
-    #    concept of environment the same way as when running as a master (except when doing apply).
+    #    concept of environment the same way as when running as a server (except when doing apply).
     #    The creation mechanisms should probably differ between the two.
     @private_environment_loader =
       if for_agent
@@ -274,11 +274,11 @@ class Loaders
   #
   # There are two sources that can be used for the initial parse:
   #
-  #   1. The value of `Puppet[:code]`: Puppet can take a string from
+  #   1. The value of `Puppet[:code]`: OpenVox can take a string from
   #     its settings and parse that as a manifest. This is used by various
-  #     Puppet applications to read in a manifest and pass it to the
+  #     OpenVox applications to read in a manifest and pass it to the
   #     environment as a side effect. This is attempted first.
-  #   2. The contents of the environment's +manifest+ attribute: Puppet will
+  #   2. The contents of the environment's +manifest+ attribute: OpenVox will
   #     try to load the environment manifest. The manifest must be a file.
   #
   # @return [Model::Program] The manifest parsed into a model object

data/lib/puppet/pops/lookup/lookup_adapter.rb

@@ -124,27 +124,9 @@ class LookupAdapter < DataAdapter
   end
 
   def lookup_global(key, lookup_invocation, merge_strategy)
-    # hiera_xxx will always use global_provider regardless of data_binding_terminus setting
-    terminus = lookup_invocation.hiera_xxx_call? ? :hiera : Puppet[:data_binding_terminus]
-    case terminus
-    when :hiera, 'hiera'
-      provider = global_provider(lookup_invocation)
-      throw :no_such_key if provider.nil?
-      provider.key_lookup(key, lookup_invocation, merge_strategy)
-    when :none, 'none', '', nil
-      # If global lookup is disabled, immediately report as not found
-      lookup_invocation.report_not_found(key)
-      throw :no_such_key
-    else
-      lookup_invocation.with(:global, terminus) do
-        catch(:no_such_key) do
-          return lookup_invocation.report_found(key, Puppet::DataBinding.indirection.find(key.root_key,
-                                                                                          { :environment => environment, :variables => lookup_invocation.scope, :merge => merge_strategy }))
-        end
-        lookup_invocation.report_not_found(key)
-        throw :no_such_key
-      end
-    end
+    provider = global_provider(lookup_invocation)
+    throw :no_such_key if provider.nil?
+    provider.key_lookup(key, lookup_invocation, merge_strategy)
   rescue Puppet::DataBinding::LookupError => detail
     raise detail unless detail.issue_code.nil?
 
@@ -468,55 +450,14 @@ class LookupAdapter < DataAdapter
     env_conf = environment.configuration
     return nil if env_conf.nil? || env_conf.path_to_env.nil?
 
-    # Get the name of the data provider from the environment's configuration
-    provider_name = env_conf.environment_data_provider
     env_path = Pathname(env_conf.path_to_env)
     config_path = env_path + HieraConfig::CONFIG_FILE_NAME
 
-    ep = nil
-    if config_path.exist?
-      ep = EnvironmentDataProvider.new
-      # A version 5 hiera.yaml trumps any data provider setting in the environment.conf
-      ep_config = ep.config(lookup_invocation)
-      if ep_config.nil?
-        ep = nil
-      elsif ep_config.version >= 5
-        unless provider_name.nil? || Puppet[:strict] == :off
-          Puppet.warn_once('deprecations', 'environment.conf#data_provider',
-                           _("Defining environment_data_provider='%{provider_name}' in environment.conf is deprecated") % { provider_name: provider_name }, env_path + 'environment.conf')
-
-          unless provider_name == 'hiera'
-            Puppet.warn_once('deprecations', 'environment.conf#data_provider_overridden',
-                             _("The environment_data_provider='%{provider_name}' setting is ignored since '%{config_path}' version >= 5") % { provider_name: provider_name, config_path: config_path }, env_path + 'environment.conf')
-          end
-        end
-        provider_name = nil
-      end
-    end
-
-    if provider_name.nil?
-      ep
-    else
-      unless Puppet[:strict] == :off
-        msg = _("Defining environment_data_provider='%{provider_name}' in environment.conf is deprecated.") % { provider_name: provider_name }
-        msg += " " + _("A '%{hiera_config}' file should be used instead") % { hiera_config: HieraConfig::CONFIG_FILE_NAME } if ep.nil?
-        Puppet.warn_once('deprecations', 'environment.conf#data_provider', msg, env_path + 'environment.conf')
-      end
+    return nil unless config_path.exist?
 
-      case provider_name
-      when 'none'
-        nil
-      when 'hiera'
-        # Use hiera.yaml or default settings if it is missing
-        ep || EnvironmentDataProvider.new
-      when 'function'
-        ep = EnvironmentDataProvider.new
-        ep.config = HieraConfigV5.v4_function_config(env_path, 'environment::data', ep)
-        ep
-      else
-        raise Puppet::Error, _("Environment '%{env}', cannot find environment_data_provider '%{provider}'") % { env: environment.name, provider: provider_name }
-      end
-    end
+    ep = EnvironmentDataProvider.new
+    ep_config = ep.config(lookup_invocation)
+    ep_config.nil? ? nil : ep
   end
 
   # @return [Puppet::Node::Environment] the environment of the compiler that this adapter is associated with

data/lib/puppet/pops/model/factory.rb

@@ -532,15 +532,15 @@ class Factory
 
   def and(r)    f_build_binary(AndExpression, self, r);         end
 
-  def not();    f_build_unary(NotExpression, self);             end
+  def not;    f_build_unary(NotExpression, self);             end
 
-  def minus();  f_build_unary(UnaryMinusExpression, self);      end
+  def minus;  f_build_unary(UnaryMinusExpression, self);      end
 
-  def unfold(); f_build_unary(UnfoldExpression, self);          end
+  def unfold; f_build_unary(UnfoldExpression, self);          end
 
-  def text();   f_build_unary(TextExpression, self);            end
+  def text;   f_build_unary(TextExpression, self);            end
 
-  def var();    f_build_unary(VariableExpression, self);        end
+  def var;    f_build_unary(VariableExpression, self);        end
 
   def access(r); f_build_binary(AccessExpression, self, r);     end
 

data/lib/puppet/pops/resource/resource_type_impl.rb

@@ -267,7 +267,7 @@ class ResourceTypeImpl
   # * meta parameters
   #
   def allattrs
-    raise NotImplementedError, "allattrs() - return all attribute names in order - probably not used master side"
+    raise NotImplementedError, "allattrs() - return all attribute names in order - probably not used on the server side"
     # key_attributes | (parameters & [:provider]) | properties.collect { |property| property.name } | parameters | metaparams
   end
 

data/lib/puppet/pops/types/iterable.rb

@@ -136,7 +136,7 @@ module Puppet::Pops::Types
       end
     end
 
-    # Answers the question if there is an end to the iteration. Puppet does not currently provide any unbounded
+    # Answers the question if there is an end to the iteration. OpenVox does not currently provide any unbounded
     # iterables.
     #
     # @return [Boolean] `true` if the iteration is unbounded

data/lib/puppet/property/keyvalue.rb

@@ -47,7 +47,7 @@ module Puppet
       end
 
       def hashify_should
-        # Puppet casts all should values to arrays. Thus, if the user
+        # OpenVox casts all should values to arrays. Thus, if the user
         # passed in a hash for our property's should value, the should_value
         # parameter will be a single element array so we just extract our value
         # directly.
@@ -79,8 +79,8 @@ module Puppet
         # shared keys will get overwritten by members
         should_value = current.merge(members)
 
-        # Figure out the keys that will actually change in our Puppet run.
-        # This lets us reduce the verbosity of Puppet's logging for instances
+        # Figure out the keys that will actually change in our OpenVox run.
+        # This lets us reduce the verbosity of OpenVox's logging for instances
         # of this class when we want to.
         #
         # NOTE: We use ||= here because we only need to compute the

data/lib/puppet/provider/group/aix.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Group Puppet provider for AIX. It uses standard commands to manage groups:
+# Group OpenVox provider for AIX. It uses standard commands to manage groups:
 #  mkgroup, rmgroup, lsgroup, chgroup
 require_relative '../../../puppet/provider/aix_object'
 
@@ -94,7 +94,7 @@ Puppet::Type.type(:group).provide :aix, :parent => Puppet::Provider::AixObject d
   # to do it this way so that we do not accidentally break something.
   # This is ok for now, since we do plan on moving this and the
   # auth_membership management over to the property class in a future
-  # Puppet release.
+  # OpenVox release.
   def members_insync?(current, should)
     current.sort == @resource.parameter(:members).actual_should(current, should)
   end

data/lib/puppet/provider/nameservice/directoryservice.rb

@@ -263,7 +263,7 @@ class Puppet::Provider::NameService::DirectoryService < Puppet::Provider::NameSe
         # converted_hash_plist['SALTED-SHA512'] is a Base64 encoded
         # string. The password_hash provided as a resource attribute is a
         # hex value. We need to convert the Base64 encoded string to a
-        # hex value and provide it back to Puppet.
+        # hex value and provide it back to OpenVox.
         converted_hash_plist['SALTED-SHA512'].unpack1("H*")
 
       end
@@ -393,7 +393,7 @@ class Puppet::Provider::NameService::DirectoryService < Puppet::Provider::NameSe
     # of the password hash if required
     # Shelling out sucks, but for a single use case it doesn't seem worth
     # requiring people install a UUID library that doesn't come with the system.
-    # This should be revisited if Puppet starts managing UUIDs for other platform
+    # This should be revisited if OpenVox starts managing UUIDs for other platform
     # user records.
     guid = %x(/usr/bin/uuidgen).chomp
 

data/lib/puppet/provider/package/aix.rb

@@ -6,7 +6,7 @@ require_relative '../../../puppet/util/package'
 Puppet::Type.type(:package).provide :aix, :parent => Puppet::Provider::Package do
   desc "Installation from an AIX software directory, using the AIX `installp`
        command.  The `source` parameter is required for this provider, and should
-       be set to the absolute path (on the puppet agent machine) of a directory
+       be set to the absolute path (on the OpenVox agent machine) of a directory
        containing one or more BFF package files.
 
        The `installp` command will generate a table of contents file (named `.toc`)

data/lib/puppet/provider/package/apt.rb

@@ -149,9 +149,9 @@ Puppet::Type.type(:package).provide :apt, :parent => :dpkg, :source => :dpkg do
     when true, false, Symbol
       # pass
     else
-      # Add the package version and --force-yes option
+      # Add the package version and --allow-downgrades option (it was --force-yes before)
       str += "=#{should}"
-      cmd << "--force-yes"
+      cmd << "--allow-downgrades"
     end
 
     cmd += install_options if @resource[:install_options]

data/lib/puppet/provider/package/aptitude.rb

@@ -14,7 +14,6 @@ Puppet::Type.type(:package).provide :aptitude, :parent => :apt, :source => :dpkg
     args.flatten!
     # Apparently aptitude hasn't always supported a -q flag.
     args.delete("-q") if args.include?("-q")
-    args.delete("--force-yes") if args.include?("--force-yes")
     output = aptitude(*args)
 
     # Yay, stupid aptitude doesn't throw an error when the package is missing.

data/lib/puppet/provider/package/dnfmodule.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# dnfmodule - A puppet package provider for DNF modules
+# dnfmodule - An OpenVox package provider for DNF modules
 #
 # Installing a module:
 #  package { 'postgresql':

data/lib/puppet/provider/package/nim.rb

@@ -6,7 +6,7 @@ require_relative '../../../puppet/util/package'
 Puppet::Type.type(:package).provide :nim, :parent => :aix, :source => :aix do
   desc "Installation from an AIX NIM LPP source.  The `source` parameter is required
       for this provider, and should specify the name of a NIM `lpp_source` resource
-      that is visible to the puppet agent machine.  This provider supports the
+      that is visible to the OpenVox agent machine.  This provider supports the
       management of both BFF/installp and RPM packages.
 
       Note that package downgrades are *not* supported; if your resource specifies

data/lib/puppet/provider/package/pacman.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_relative '../../../puppet/provider/package'
-require 'set' # rubocop:disable Lint/RedundantRequireStatement
 require 'uri'
 
 Puppet::Type.type(:package).provide :pacman, :parent => Puppet::Provider::Package do
@@ -248,7 +247,7 @@ Puppet::Type.type(:package).provide :pacman, :parent => Puppet::Provider::Packag
              else
                fail _("Source %{source} is not supported by pacman") % { source: source }
              end
-    pacman "--noconfirm", "--noprogressbar", "--update", source
+    pacman "--noconfirm", "--noprogressbar", "--upgrade", source
   end
 
   def install_from_repo

data/lib/puppet/provider/package/pip.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Puppet package provider for Python's `pip` package management frontend.
+# OpenVox package provider for Python's `pip` package management frontend.
 # <http://pip.pypa.io/>
 
 require_relative '../../../puppet/util/package/version/pip'

data/lib/puppet/provider/package/pip2.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Puppet package provider for Python's `pip2` package management frontend.
+# OpenVox package provider for Python's `pip2` package management frontend.
 # <http://pip.pypa.io/>
 
 Puppet::Type.type(:package).provide :pip2,

data/lib/puppet/provider/package/pip3.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-# Puppet package provider for Python's `pip3` package management frontend.
+# OpenVox package provider for Python's `pip3` package management frontend.
 # <http://pip.pypa.io/>
 
 Puppet::Type.type(:package).provide :pip3,

data/lib/puppet/provider/package/pkgdmg.rb

@@ -19,7 +19,7 @@ Puppet::Type.type(:package).provide :pkgdmg, :parent => Puppet::Provider::Packag
 
     This provider works by checking the contents of a DMG image for Apple pkg or
     mpkg files. Any number of pkg or mpkg files may exist in the root directory
-    of the DMG file system, and Puppet will install all of them. Subdirectories
+    of the DMG file system, and OpenVox will install all of them. Subdirectories
     are not checked for packages.
 
     This provider can also accept plain .pkg (but not .mpkg) files in addition
@@ -32,7 +32,7 @@ Puppet::Type.type(:package).provide :pkgdmg, :parent => Puppet::Provider::Packag
     * The `name` of the resource must be the filename (without path) of the DMG file.
     * When installing the packages from a DMG, this provider writes a file to
       disk at `/var/db/.puppet_pkgdmg_installed_NAME`. If that file is present,
-      Puppet assumes all packages from that DMG are already installed.
+      OpenVox assumes all packages from that DMG are already installed.
     * This provider is not versionable and uses DMG filenames to determine
       whether a package has been installed. Thus, to install new a version of a
       package, you must create a new DMG with a different filename."

data/lib/puppet/provider/package/puppet_gem.rb

@@ -1,8 +1,8 @@
 # frozen_string_literal: true
 
 Puppet::Type.type(:package).provide :puppet_gem, :parent => :gem do
-  desc "Puppet Ruby Gem support. This provider is useful for managing
-        gems needed by the ruby provided in the puppet-agent package."
+  desc "OpenVox Ruby Gem support. This provider is useful for managing
+        gems needed by the ruby provided in the openvox-agent package."
 
   has_feature :versionable, :install_options, :uninstall_options
 

data/lib/puppet/provider/package/puppetserver_gem.rb

@@ -9,7 +9,7 @@ require 'uri'
 
 # Ruby gems support.
 Puppet::Type.type(:package).provide :puppetserver_gem, :parent => :gem do
-  desc "Puppet Server Ruby Gem support. If a URL is passed via `source`, then
+  desc "OpenVox Server Ruby Gem support. If a URL is passed via `source`, then
     that URL is appended to the list of remote gem repositories which by default
     contains rubygems.org; To ensure that only the specified source is used also
     pass `--clear-sources` in via `install_options`; if a source is present but

data/lib/puppet/provider/package/windows.rb

@@ -19,7 +19,7 @@ Puppet::Type.type(:package).provide(:windows, :parent => Puppet::Provider::Packa
 
     If the executable requires special arguments to perform a silent install or
     uninstall, then the appropriate arguments should be specified using the
-    `install_options` or `uninstall_options` attributes, respectively.  Puppet
+    `install_options` or `uninstall_options` attributes, respectively.  OpenVox
     will automatically quote any option that contains spaces."
 
   confine    'os.name' => :windows

data/lib/puppet/provider/service/freebsd.rb

@@ -86,7 +86,7 @@ Puppet::Type.type(:service).provide :freebsd, :parent => :init do
 
   # Add a new setting to the rc files
   def rc_add(service, rcvar, yesno)
-    append = "\# Added by Puppet\n#{rcvar}_enable=\"#{yesno}\"\n"
+    append = "\# Added by OpenVox\n#{rcvar}_enable=\"#{yesno}\"\n"
     # First, try the one-file-per-service style
     if Puppet::FileSystem.exist?(rcconf_dir)
       File.open(rcconf_dir + "/#{service}", File::WRONLY | File::APPEND | File::CREAT, 0o644) { |f|

data/lib/puppet/provider/service/init.rb

@@ -52,21 +52,21 @@ Puppet::Type.type(:service).provide :init, :parent => :base do
     excludes += %w[wait-for-state portmap-wait]
     # these excludes were found with grep -r -L start /etc/init.d
     excludes += %w[rcS module-init-tools]
-    # Prevent puppet failing on unsafe scripts from Yocto Linux
+    # Prevent OpenVox failing on unsafe scripts from Yocto Linux
     if Puppet.runtime[:facter].value('os.family') == "cisco-wrlinux"
       excludes += %w[banner.sh bootmisc.sh checkroot.sh devpts.sh dmesg.sh
                      hostname.sh mountall.sh mountnfs.sh populate-volatile.sh
                      rmnologin.sh save-rtc.sh sendsigs sysfs.sh umountfs
                      umountnfs.sh]
     end
-    # Prevent puppet failing to get status of the new service introduced
+    # Prevent OpenVox failing to get status of the new service introduced
     # by the fix for this (bug https://bugs.launchpad.net/ubuntu/+source/lightdm/+bug/982889)
-    # due to puppet's inability to deal with upstart services with instances.
+    # due to OpenVox's inability to deal with upstart services with instances.
     excludes += %w[plymouth-ready]
-    # Prevent puppet failing to get status of these services, which need parameters
+    # Prevent OpenVox failing to get status of these services, which need parameters
     # passed in (see https://bugs.launchpad.net/ubuntu/+source/puppet/+bug/1276766).
     excludes += %w[idmapd-mounting startpar-bridge]
-    # Prevent puppet failing to get status of these services, additional upstart
+    # Prevent OpenVox failing to get status of these services, additional upstart
     # service with instances
     excludes += %w[cryptdisks-udev]
     excludes += %w[statd-mounting]

data/lib/puppet/provider/service/launchd.rb

@@ -26,7 +26,7 @@ Puppet::Type.type(:service).provide :launchd, :parent => :base do
     * status
     * restart
 
-    Here is how the Puppet states correspond to `launchd` states:
+    Here is how the OpenVox states correspond to `launchd` states:
 
     * stopped --- job unloaded
     * started --- job loaded
@@ -148,7 +148,7 @@ Puppet::Type.type(:service).provide :launchd, :parent => :base do
           @label_to_path_map[job["Label"]] = filepath
         else
           # TRANSLATORS 'plist' and label' should not be translated
-          Puppet.debug(_("The %{file} plist does not contain a 'label' key; Puppet is skipping it") % { file: filepath })
+          Puppet.debug(_("The %{file} plist does not contain a 'label' key; OpenVox is skipping it") % { file: filepath })
           next
         end
       end

data/lib/puppet/provider/service/smf.rb

@@ -33,7 +33,7 @@ Puppet::Type.type(:service).provide :smf, :parent => :base do
   def self.instances
     service_instances = svcs("-H", "-o", "state,fmri").split("\n")
 
-    # Puppet does not manage services in the legacy_run state, so filter those out.
+    # OpenVox does not manage services in the legacy_run state, so filter those out.
     service_instances.reject! { |line| line =~ /^legacy_run/ }
 
     service_instances.collect! do |line|

data/lib/puppet/provider/service/upstart.rb

@@ -36,7 +36,7 @@ Puppet::Type.type(:service).provide :upstart, :parent => :debian do
     # Puppet::Util::Execution.execute does not currently work on jRuby.
     # Unfortunately, since this confine is invoked whenever we check for
     # provider suitability and since provider suitability is still checked
-    # on the master, this confine will still be invoked on the master. Thus
+    # on the server, this confine will still be invoked on the server. Thus
     # to avoid raising an exception, we do an early return if we're running
     # on jRuby.
     return false if Puppet::Util::Platform.jruby?
@@ -60,7 +60,7 @@ Puppet::Type.type(:service).provide :upstart, :parent => :debian do
   def self.excludes
     excludes = super
     if Puppet.runtime[:facter].value('os.family') == 'RedHat'
-      # Puppet cannot deal with services that have instances, so we have to
+      # OpenVox cannot deal with services that have instances, so we have to
       # ignore these services using instances on redhat based systems.
       excludes += %w[serial tty]
     end

data/lib/puppet/provider/user/aix.rb

@@ -1,15 +1,15 @@
 # frozen_string_literal: true
 
-# User Puppet provider for AIX. It uses standard commands to manage users:
+# User OpenVox provider for AIX. It uses standard commands to manage users:
 #  mkuser, rmuser, lsuser, chuser
 #
 # Notes:
 # - AIX users can have expiry date defined with minute granularity,
-#   but Puppet does not allow it. There is a ticket open for that (#5431)
+#   but OpenVox does not allow it. There is a ticket open for that (#5431)
 #
 # - AIX maximum password age is in WEEKs, not days
 #
-# See https://puppet.com/docs/puppet/latest/provider_development.html
+# See https://docs.openvoxproject.org/openvox/latest/provider_development.html
 # for more information
 require_relative '../../../puppet/provider/aix_object'
 require_relative '../../../puppet/util/posix'