Merge pull request #559 from DavidS/fm-3802-fix-ensure_resource-test

(FM-3802) make ensure_resource test of packages

(FM-3802) make ensure_resource test of packages

This ensures that the test passes independently of changes to rubygems.

Merge pull request #558 from puppetlabs/4.10.x

4.10.x mergeback

Merge pull request #557 from jbondpdx/4.10.x

edits to README

more carriage returns

more carriage returns, no

removing mid-line carriage returns

edits to README

Merge pull request #556 from HelenCampbell/4.10.0_prep

Changelog and versionbump for 4.10.0

Changelog and versionbump for 4.10.0

Merge pull request #554 from kjetilho/ticket/2886-seeded_rand

(#2886) seeded_rand: new function

Merge pull request #555 from puppetlabs/4.9.x

4.9.x Mergeback

(#2886) seeded_rand: new function

seeded_rand is needed for repeatable randomness across nodes in a cluster

Merge pull request #546 from jdevesa/validate_ip

Add validator for any IP address

Add validator for any IP address

Provide a validator for IP addresses, regardless they are IPv4 or IPv6,
and its documentation.

Merge pull request #550 from tphoney/release_4.9.1

4.9.1 release prep

4.9.1 release prep

Merge pull request #551 from mattbostock/fix_ipv4_doc

Fix reference to validate_bool in IP4 function

Fix reference to validate_bool in IP4 function

The documentation in `validate_ipv4_address` references `validate_bool`,
but I believe this should read `validate_ipv4_address` instead, which
makes more sense.

Merge pull request #548 from bmjen/ruby187

Fix Gemfile to work with ruby 1.8.7

Fix Gemfile to work with ruby 1.8.7

Merge pull request #543 from ghoneycutt/patch-2

Use absolute class name in example

Merge pull request #547 from reidmv/fm_3773

(FM-3773) Fix root_home fact on AIX 5.x

(FM-3773) Fix root_home fact on AIX 5.x

The -C (capital C) flag to lsuser is incorrect. It should be -c
(lowercase).

this commit updates the aix root_home fact to use `lsuser -c`, rather
than `lsuser -C`.

Merge pull request #544 from HelenCampbell/readmeaddition

pick_default addition to readme

pick_default addition to readme

Use absolute class name in example

Merge pull request #542 from greg0ire/patch-1

use properly encoded characters

use properly encoded characters

This is more severe than it sounds. These characters make puppet fail with the following message :

> Error 400 on SERVER: "\xC3" on US-ASCII

Merge pull request #541 from mattflaschen/patch-1

Fix capitalize docs

Fix capitalize docs

Capitalize lower-cases the remaining characters (due to the Ruby function having this behavior); document this, and make minor wording tweaks.

Merge pull request #540 from marrero984/update_str2bool_readme

(#2183) updated str2bool readme wording

(#2183) updated str2bool readme wording

Merge pull request #538 from mmckinst/bool2str_enhance

add functionality to bool2str function

add functionality to bool2str to return strings of your choice for a boolean

Merge pull request #539 from mentat/MODULES-2696

Add check to ensure regex does not throw for none type.

Add check to ensure regex does not throw for none type.

Add a quick check to ensure puppetversion value is not nil and supporting test.

Merge pull request #537 from cmurphy/fix_load_module_metadata

Fix load module metadata

Let load_module_metadata succeed on empty file

Some modules or module versions don't have a metadata.json file, but we
might still want to use the load_module_metadata function on them. The
lack of a file can still give us important information. For example, it
might tell us that the version of the module installed is "very old"
even if we can't read the version number directly. This patch adds a
parameter to let the user specify if an empty file is acceptable. To
preserve backwards compatibility it does not change the current default
behavior, which is to raise an error if metadata.json does not exist.

Rename load_module_metadata test path

`rake spec` only finds test files that end in _spec.rb, so this test
was not being run. Correct the path name so that the test runs
properly.

Merge pull request #536 from DavidS/improve-file_line-docs

(MODULES-2421) improve description of file_line

(MODULES-2421) improve description of file_line

This mostly needed extraction of the existing doc strings from
the type.

Merge pull request #535 from martinpfeifer/package_provider_fact

prevent deprecation warning about the allow_virtual parameter

prevent deprecation warning about the allow_virtual parameter

Merge pull request #534 from asasfu/feature/package_provider_fact

Add package_provider fact

Merge pull request #533 from HelenCampbell/MODULES-2614-Improved

Modules 2614 improved numeric value handling on empty function

Added acceptance test and updated readme

Check for numeric values as empty fails on those

Merge pull request #532 from DavidS/fix-is_a-readme

(FM-3701) Update README for is_a

(FM-3701) Update README for is_a

Merge pull request #531 from asasfu/improvement/service_provider_fact

fixup-PR#506 Speed improvements in facter resolution

fixup-PR#506 Speed improvements in facter resolution

This is to improve speed on Facter resolution of service_provider fact
that was just introduced in PR# 506.  The improvements go from 280ms
resolution time approx. down to 2ms resolution time approx. by adding
requires statements.

Add package_provider fact

This adds a package_provider fact for situations where we need to be
able to know the client's package provider in a simple way.  Situations
such as: package { 'name': install_options => [] }  As those tend to be
package provider specific options.

Merge pull request #506 from binford2k/feature/service_provider_fact

Add a service_provider fact

Merge pull request #530 from HelenCampbell/MODULES-2614

Adding update to empty function readme

Adding update to empty function readme

Merge pull request #528 from mhaskel/is_a_fixes

Update is_a acceptance tests to only run on puppet4

Update is_a acceptance tests to only run on puppet4

Merge pull request #527 from mhaskel/511_compatibility

Fix backwards compatibility from #511

Fix backwards compatibility from #511

Maintain the old behavior in the case where the optional second
parameter isn't passed. Also, adding arity is backwards incompatible since
stdlib still supports 2.7, so remove that.

Merge pull request #526 from DavidS/improve-validate_re-docs

(MAINT) validate_re: Clarify docs and error message

(MAINT) validate_re: Clarify docs and error message

Merge pull request #524 from binford2k/docs/intersection_truthiness

Clarify what an empty intersection looks like.

Clarify what an empty intersection looks like.

Merge pull request #523 from DavidS/modules-2516-is_a

(MODULES-2561) add is_a function

Merge pull request #511 from dmitryilyin/parseyaml_default

[MODULES-2462] Improve parseyaml function

(MODULES-2516) Adds an is_a() function

The data type system is very hard to understand. Many people don't
understand why

    type_of([1,2,3]) == Array

will fail, but

    type_of([1,2,3]) <= Array

passes. This does a simpler validation that doesn't rely on explicit
data types. Instead, use

    $foo = [1,2,3]
    if $foo.is_a(Array) {
      notify { 'This is an array': }
    }

This is based on code by Ben Ford <ben.ford@puppetlabs.com>.

  * Added acceptance tests
  * Added dispatch
  * Improved unit tests
  * Added docs to README

Merge pull request #518 from logicminds/case_insensitive

accept any case of boolean strings

Merge pull request #519 from dmitryilyin/fetch

[MAINT] Improve 'try_get_value' readme

[MAINT] Improve 'try_get_value' readme

accept any case of boolean strings

  * previously the str2bool function did not accept 'TRUE' as a bool
    type.  This causes the function to now accept TRUE, FALSE strings
    as a boolean type in order to be converted to a proper boolean.
  * This would also cause Y,N, YES, NO to be accepted as boolean types
    as well.

Release Prep 4.9.0

Merge pull request #517 from DavidS/try_get_value-acceptance

(MAINT) fix up try_get_value acceptance test

(MAINT) fix up try_get_value acceptance test

Merge pull request #513 from dmitryilyin/fetch

Add a new function "try_get_value"

Add a new function "try_get_value"

* Extracts a value from a deeply-nested data structure
* Returns default if a value could not be extracted

Merge pull request #515 from jfautley/ticket/MODULES-2478-support_root_home_fact_on_AIX

(MODULES-2478) Make root_home fact work on AIX using native lsuser command

(MODULES-2478) Support root_home fact on AIX through "lsuser" command

Squashed, and amended test for comment lines.

Merge pull request #514 from DavidS/add-convert_base

Adds a convert_base function, which can convert numbers between bases

Adds a convert_base function, which can convert numbers between bases

Squashed, improved docs, updated error handling and unit tests by David S.

[MODULES-2462] Improve parseyaml function

* Add default value support
  Second argument will be returned if yaml cannot be parsed
  instead of false value
* Update tests

Merge pull request #512 from Jetroid/consistentreadme

Consistent Readme

Add consistent *Type* information

Remove trailing whitespace

Two functions had not been given any *Type* information. This commit fixes that.

Merge pull request #510 from DavidS/base64-unit-tests

(MAINT) improve base64 unit tests

(MAINT) improve base64 unit tests

Merge pull request #507 from Jetroid/mod2456

(MODULES-2456) Modify union to accept more than two arrays

(MODULE-2456) Modify union to accept more than two arrays

Add spec tests to test the new functionality:
 *Case for 3 arrays.
 *Case for 4 arrays.
Modify README to note new functionality.

This is for issue MODULE-2456, follow the precedent of MODULE-444.

This change allows union to be much more useful, unioning many arrays
in one line rather than in n lines. Additionally, as this is only added
functionality, and does not affect the 2 array case that all modules
currently using array are using, it should not affect any existing
modules utilizing union.

This is now useful, for example, for merging many arrays of resources
(eg: packages.) to generate just one list with no duplicates, to avoid
duplicate resource declarations.

Add a service_provider fact

This returns the default provider Puppet will choose to manage services
on this system by instantiating a dummy service resource type and
returning the provider chosen.

Co-Authored-By: Simon Fraser University <asa188@sfu.ca>

Merge pull request #505 from gibbsoft/dos2unix

(MODULES-2410) Add new functions dos2unix and unix2dos

(MODULES-2410) Add new functions dos2unix and unix2dos

Merge pull request #499 from jearls/2370-use-match-for-ensure-absent

[MODULES-2370] allow `match` parameter to influence `ensure => absent` behavior.

Merge pull request #503 from puppetlabs/4.8.x

Mergeback 4.8.x

Add puppet_version back to spec_helper

Merge pull request #502 from hunner/fix_after

Sometimes this exits 1

Sometimes this exits 1

Merge pull request #501 from hunner/bug_helper

Fix extraneous end

Fix extraneous end

Merge pull request #500 from hunner/prep_4.8.0

Prep 4.8.0

Prep 4.8.0

allow `match` parameter to influence `ensure => absent` behavior.

Split the `destroy` method of the file_type::ruby provider into two
private methods:  `handle_destroy_line` which is the same as the previous
`destroy` method, and `handle_destroy_with_match` which will destroy any
line which matches the `match` parameter, raising an error if multiple
lines match and the `multiple` parameter is not `true`.  This new
behavior is only used if the new boolean parameter `match_for_absence`
is `true` (it defaults to `false`).