facter 3.11.4.cfacter.20180821 → 3.11.5.cfacter.20181022

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: bcbeadda3a08eb363825fbaa0dbc28aa0ea7d722
-  data.tar.gz: 0ceba75967ddc26ded11b2349024518dfe99ab55
+  metadata.gz: a4a4e2fb925fbcdc4a6df9f5059c352c21537d00
+  data.tar.gz: e2fa7aed4bc3b820555b69ec88c4fc1732b72932
 SHA512:
-  metadata.gz: be5fb3cf4c2f375c26cc4a0442a03aaa51459295c8851d8155328bcb5a287ec2fb2323a6267d175b4ec087619510932f53a65eb916c5178ea006ac0e1445b73a
-  data.tar.gz: f43786a9d453221a5bca2808ceb375aacca08044a0bcc487602cdc0f9c8b52b5d35f670d1161ea1d66f7afed5ffbc6873855ee629a18d39f5d95683983d7641d
+  metadata.gz: ab739721a51ceeed3d9e91be204b99cc028b7f1cb412714b216fc082d3cd5ec7b6110776586e45fd6a30bbe9cdd897cff3f448282896743ac712739d9d917b7a
+  data.tar.gz: 4c1ff6d9e83af0f0b7a0f346f2a3e95f5adb5885f8f56f7d137d16fe7e00398b23b8ddac455e942586d71f20ce339cce9413e8ed16be44e8b906a264671ebe31

data/ext/facter/facter/CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.2.2)
-project(FACTER VERSION 3.11.4)
+project(FACTER VERSION 3.11.5)
 
 # Set this early, so it's available. AIX gets weird, man.
 if("${CMAKE_SYSTEM_NAME}" MATCHES "AIX")

data/ext/facter/facter/Gemfile

@@ -0,0 +1,13 @@
+source ENV['GEM_SOURCE'] || 'https://artifactory.delivery.puppetlabs.net/artifactory/api/gems/rubygems/'
+
+def location_for(place, fake_version = nil)
+  if place =~ /^(git[:@][^#]*)#(.*)/
+    [fake_version, { :git => $1, :branch => $2, :require => false }].compact
+  elsif place =~ /^file:\/\/(.*)/
+    ['>= 0', { :path => File.expand_path($1), :require => false }]
+  else
+    [place, { :require => false }]
+  end
+end
+
+gem 'packaging', *location_for(ENV['PACKAGING_LOCATION'] || '~> 0.99')

data/ext/facter/facter/Rakefile

@@ -6,3 +6,12 @@ Dir['tasks/**/*.rake'].each { |t| load t }
 
 require 'packaging'
 Pkg::Util::RakeUtils.load_packaging_tasks
+
+namespace :package do
+  task :bootstrap do
+    puts 'Bootstrap is no longer needed, using packaging-as-a-gem'
+  end
+  task :implode do
+    puts 'Implode is no longer needed, using packaging-as-a-gem'
+  end
+end

data/ext/facter/facter/acceptance/lib/facter/acceptance/base_fact_utils.rb

@@ -523,11 +523,13 @@ module Facter
         elsif agent['platform'] =~ /2008/
           os_version = '2008 R2'
         elsif agent['platform'] =~ /2012/
-          os_version = '2012 R2'
+          os_version = agent['platform'] =~ /R2/ ? '2012 R2' : '2012'
         elsif agent['platform'] =~ /-10/
           os_version = '10'
         elsif agent['platform'] =~ /2016/
           os_version = '2016'
+        elsif agent['platform'] =~ /2019/
+          os_version = '2019'
         else
           fail_test "Unknown Windows version #{agent['platform']}"
         end

data/ext/facter/facter/acceptance/tests/facts/dmi.rb

@@ -15,13 +15,13 @@ test_name "C96148: verify dmi facts" do
     expected_facts = {
         'dmi.manufacturer' => /\w+/,
         'dmi.product.name' => /\w+/,
+        'dmi.product.uuid' => /[-0-9A-Fa-f]+/,
     }
     unless agent['platform'] =~ /windows/
       expected_facts.merge!({'dmi.bios.release_date' => /\d+\/\d+\/\d+/,
                              'dmi.bios.vendor'       => /\w+/,
                              'dmi.bios.version'      => /\d+/,
                              'dmi.chassis.type'      => /\w+/,
-                             'dmi.product.uuid'      => /[-0-9A-Fa-f]+/,
                             })
     end
     unless agent['platform'] =~ /windows|cisco|aarch64|el-/

data/ext/facter/facter/appveyor.yml

@@ -1,27 +1,21 @@
 version: 3.1.0.{build}
 clone_depth: 10
 environment:
-  LEATHERMAN_VERSION: 0.99.0
-  CPPHOCON_VERSION: 0.1.4
+  LEATHERMAN_VERSION: 1.4.4
+  CPPHOCON_VERSION: 0.1.8
 
 init:
   - |
-      choco install mingw-w64 -y -Version 4.8.3 -source https://www.myget.org/F/puppetlabs
+      choco install -y mingw-w64 -Version 5.2.0 -source https://www.myget.org/F/puppetlabs
+      choco install -y cmake -Version 3.2.2 -source https://www.myget.org/F/puppetlabs
       choco install -y gettext -Version 0.19.6 -source https://www.myget.org/F/puppetlabs
+      choco install -y pl-toolchain-x64 -Version 2015.12.01.1 -source https://www.myget.org/F/puppetlabs
+      choco install -y pl-boost-x64 -Version 1.58.0.2 -source https://www.myget.org/F/puppetlabs
+      choco install -y pl-openssl-x64 -Version 1.0.24.1 -source https://www.myget.org/F/puppetlabs
+      choco install -y pl-curl-x64 -Version 7.46.0.1 -source https://www.myget.org/F/puppetlabs
+      choco install -y pl-yaml-cpp-x64 -Version 0.5.1.2 -source https://www.myget.org/F/puppetlabs
       cmake --version
   - ps: |
-      $env:PATH = $env:PATH.Replace("Git\bin", "Git\cmd")
-      $env:PATH = $env:PATH.Replace("Git\usr\bin", "Git\cmd")
-
-      wget 'https://s3.amazonaws.com/kylo-pl-bucket/boost_1_57_0-x86_64_mingw-w64_4.8.3_win32_seh.7z' -OutFile "$env:temp\boost.7z"
-      7z.exe x $env:temp\boost.7z -oC:\tools | FIND /V "ing  "
-
-      wget 'https://s3.amazonaws.com/kylo-pl-bucket/yaml-cpp-0.5.1-x86_64_mingw-w64_4.8.3_win32_seh.7z' -OutFile "$env:temp\yaml-cpp.7z"
-      7z.exe x $env:temp\yaml-cpp.7z -oC:\tools | FIND /V "ing  "
-
-      wget 'https://s3.amazonaws.com/kylo-pl-bucket/curl-7.42.1-x86_64_mingw-w64_4.8.3_win32_seh.7z' -OutFile "$env:temp\curl-7.42.1-x86_64_mingw-w64_4.8.3_win32_seh.7z"
-      7z.exe x "$env:temp\curl-7.42.1-x86_64_mingw-w64_4.8.3_win32_seh.7z" -oC:\tools | FIND /V "ing "
-
       wget "https://github.com/puppetlabs/leatherman/releases/download/$env:LEATHERMAN_VERSION/leatherman.7z" -OutFile "$env:temp\leatherman.7z"
       7z.exe x $env:temp\leatherman.7z -oC:\tools | FIND /V "ing "
 
@@ -29,12 +23,15 @@ init:
       7z.exe x $env:temp\cpp-hocon.7z -oC:\tools | FIND /V "ing "
 
 install:
-  - SET PATH=C:\Ruby21-x64\bin;C:\tools\mingw64\bin;C:\Program Files\gettext-iconv;%PATH%
+    # Minimize environment polution; previously we were linking against the wrong OpenSSL DLLs.
+    # Include Ruby and Powershell for unit tests.
+  - SET PATH=C:\tools\pl-build-tools\bin;C:\tools\mingw64\bin;C:\ProgramData\chocolatey\bin;C:\Ruby22-x64\bin;C:\Program Files\7-Zip;C:\Windows\system32;C:\Windows;C:\Windows\System32\WindowsPowerShell\v1.0;C:\Program Files\gettext-iconv
+  - ps: rm -r C:\OpenSSL-Win64
   - bundle install --jobs 4 --retry 2 --gemfile=lib/Gemfile --quiet
 
 build_script:
   - ps: |
-      cmake -G "MinGW Makefiles" -DBOOST_ROOT="C:\tools\boost_1_57_0-x86_64_mingw-w64_4.8.3_win32_seh" -DYAMLCPP_ROOT="C:\tools\yaml-cpp-0.5.1-x86_64_mingw-w64_4.8.3_win32_seh" -DBOOST_STATIC=ON -DCURL_STATIC=ON -DCMAKE_INSTALL_PREFIX="C:\Program Files\FACTER" -DCMAKE_PREFIX_PATH="C:\tools\leatherman;C:\tools\curl-7.42.1-x86_64_mingw-w64_4.8.3_win32_seh;C:\tools\cpp-hocon" .
+      cmake -G "MinGW Makefiles" -DCMAKE_TOOLCHAIN_FILE="C:\tools\pl-build-tools\pl-build-toolchain.cmake" -DCMAKE_PREFIX_PATH="C:\tools\leatherman;C:\tools\cpp-hocon" -DCMAKE_INSTALL_PREFIX="C:\Program Files\FACTER" -DBOOST_STATIC=ON .
       mingw32-make -j2
 
 test_script:

data/ext/facter/facter/exe/facter.cc

@@ -332,7 +332,8 @@ int main(int argc, char **argv)
             if (vm.count("custom-dir")) {
                 custom_directories = vm["custom-dir"].as<vector<string>>();
             }
-            facter::ruby::load_custom_facts(facts, vm.count("puppet"), custom_directories);
+            bool redirect_ruby_stdout = vm.count("json") || vm.count("yaml");
+            facter::ruby::load_custom_facts(facts, vm.count("puppet"), redirect_ruby_stdout, custom_directories);
         }
 
         if (!vm["no-external-facts"].as<bool>()) {

data/ext/facter/facter/lib/Doxyfile

@@ -38,7 +38,7 @@ PROJECT_NAME           = facter
 # could be handy for archiving the generated documentation or if some version
 # control system is used.
 
-PROJECT_NUMBER         = 3.11.4
+PROJECT_NUMBER         = 3.11.5
 
 # Using the PROJECT_BRIEF tag one can provide an optional one line description
 # for a project that appears at the top of each page and should give viewer a

data/ext/facter/facter/lib/inc/facter/ruby/ruby.hpp

@@ -21,6 +21,17 @@ namespace facter { namespace ruby {
      */
     LIBFACTER_EXPORT bool initialize(bool include_stack_trace = false);
 
+    /**
+     * Loads custom facts into the given collection.
+     * Important: this function should be called from main().
+     * Calling this function from an arbitrary stack depth may result in segfaults during Ruby GC.
+     * @param facts The collection to populate with custom facts.
+     * @param initialize_puppet Whether puppet should be loaded to find additional facts.
+     * @param redirect_stdout Whether Ruby's stdout should be redirected to stderr
+     * @param paths The paths to search for custom facts.
+     */
+    LIBFACTER_EXPORT void load_custom_facts(facter::facts::collection& facts, bool initialize_puppet, bool redirect_stdout, std::vector<std::string> const& paths = {});
+
     /**
      * Loads custom facts into the given collection.
      * Important: this function should be called from main().

data/ext/facter/facter/lib/inc/internal/facts/linux/os_osrelease.hpp

@@ -0,0 +1,98 @@
+/**
+ * @file
+ * Declares the Linux operating system query helper based on /etc/os-release.
+ */
+#pragma once
+
+#include <internal/facts/linux/os_linux.hpp>
+#include <facter/facts/os.hpp>
+#include <facter/facts/os_family.hpp>
+#include <boost/regex.hpp>
+#include <iostream>
+using namespace std;
+
+namespace facter { namespace facts { namespace linux {
+
+    /**
+     * Responsible for determining the name/family/release of Freedesktop-compliant operating systems.
+     */
+    struct os_osrelease : os_linux
+    {
+        /**
+         * Constructs os_release based on details from /etc/os-release.
+         */
+        os_osrelease() : os_linux({"ID", "VERSION_ID"}) {}
+
+        /**
+         * Returns the release name based on the ID field from /etc/os-release
+         * (which has fewer variations to check for than the NAME field)
+         * @param distro_id Unused.
+         * @return Returns the OS name.
+         */
+        virtual std::string get_name(std::string const& distro_id) const override
+        {
+            auto val = _release_info.find("ID");
+            if (val != _release_info.end()) {
+                auto& id = val->second;
+
+                if (id == "coreos") {
+                    return os::coreos;
+                } else if (id == "cumulus-linux") {
+                    return os::cumulus;
+                } else if (id == "opensuse" || id == "opensuse-leap") {
+                    return os::open_suse;
+                } else if (id == "sled") {
+                    return os::suse_enterprise_desktop;
+                } else if (id == "sles") {
+                    return os::suse_enterprise_server;
+                }
+            }
+            return std::string();
+        }
+
+        /**
+         * Returns the release family based on the ID field from /etc/os-release
+         * (which has fewer variations to check for than the NAME field)
+         * @param name Unused.
+         * @return Returns the OS family.
+         */
+        virtual std::string get_family(std::string const& name) const override
+        {
+            auto val = _release_info.find("ID");
+            if (val != _release_info.end()) {
+                auto& id = val->second;
+
+                if (id == "coreos") {
+                    return os_family::coreos;
+                } else if (id == "cumulus-linux") {
+                    return os_family::debian;
+                } else if (id == "opensuse" || id == "opensuse-leap" || id == "sled" || id == "sles") {
+                    return os_family::suse;
+                }
+            }
+            return std::string();
+        }
+
+        /**
+         * Returns the OS release version based on the VERSION_ID field from /etc/os-release.
+         * @param name Unused.
+         * @param distro_release Unused.
+         * @return Returns the release version.
+         */
+        virtual std::string get_release(std::string const& name, std::string const& distro_release) const override
+        {
+            auto val = _release_info.find("VERSION_ID");
+            if (val != _release_info.end()) {
+                if (boost::regex_match(val->second, boost::regex("^\\d+$"))) {
+                    // FACT-1880: when VERSION_ID doesn't specify a point-release,
+                    // return the major version with ".0" appended so that
+                    // os.release.minor always returns a value.
+                    return val->second + ".0";
+                }
+                return val->second;
+            }
+            return std::string();
+        }
+    };
+
+}}}  // namespace facter::facts::linux

data/ext/facter/facter/lib/src/facts/collection.cc

@@ -309,6 +309,8 @@ namespace facter { namespace facts {
     {
         resolve_facts();
 
+        // We intentionally are using find_if with no return value as a "map until" construct.
+        // cppcheck-suppress ignoredReturnValue
         find_if(begin(_facts), end(_facts), [&func](map<string, unique_ptr<value>>::value_type const& it) {
             return !func(it.first, it.second.get());
         });

data/ext/facter/facter/lib/src/facts/linux/operating_system_resolver.cc

@@ -2,8 +2,7 @@
 #include <internal/facts/linux/release_file.hpp>
 #include <internal/facts/linux/os_linux.hpp>
 #include <internal/facts/linux/os_cisco.hpp>
-#include <internal/facts/linux/os_coreos.hpp>
-#include <internal/facts/linux/os_cumulus.hpp>
+#include <internal/facts/linux/os_osrelease.hpp>
 #include <facter/facts/os.hpp>
 #include <facter/facts/scalar_value.hpp>
 #include <facter/facts/map_value.hpp>
@@ -29,13 +28,11 @@ namespace facter { namespace facts { namespace linux {
 
     static unique_ptr<os_linux> get_os()
     {
-        auto release_info = os_linux::key_value_file(release_file::os, {"NAME", "CISCO_RELEASE_INFO", "ID"});
-        auto const& name = release_info["NAME"];
+        auto release_info = os_linux::key_value_file(release_file::os, {"ID", "CISCO_RELEASE_INFO"});
         auto const& id = release_info["ID"];
-        if (name == "Cumulus Linux") {
-            return unique_ptr<os_linux>(new os_cumulus());
-        } else if (name == "CoreOS" || id == "coreos") {
-            return unique_ptr<os_linux>(new os_coreos());
+        if (id == "coreos" || id == "cumulus-linux" || id == "opensuse" ||
+            id == "opensuse-leap" || id== "sled" || id == "sles") {
+            return unique_ptr<os_linux>(new os_osrelease());
         } else {
             auto const& cisco = release_info["CISCO_RELEASE_INFO"];
             boost::system::error_code ec;

data/ext/facter/facter/lib/src/facts/windows/dmi_resolver.cc

@@ -17,11 +17,12 @@ namespace facter { namespace facts { namespace windows {
     {
         data result;
 
-        auto vals = _wmi->query(wmi::computersystemproduct, {wmi::name});
+        auto vals = _wmi->query(wmi::computersystemproduct, {wmi::name, wmi::uuid});
         if (vals.empty()) {
-            LOG_DEBUG("WMI query returned no results for {1} with value {2}.", wmi::computersystemproduct, wmi::name);
+            LOG_DEBUG("WMI query returned no results for {1} with values {2} and {3}.", wmi::computersystemproduct, wmi::name, wmi::uuid);
         } else {
             result.product_name = wmi::get(vals, wmi::name);
+            result.uuid = wmi::get(vals, wmi::uuid);
         }
 
         vals = _wmi->query(wmi::bios, {wmi::manufacturer, wmi::serialnumber});

data/ext/facter/facter/lib/src/facts/windows/kernel_resolver.cc

@@ -1,60 +1,42 @@
+#include <leatherman/dynamic_library/dynamic_library.hpp>
 #include <internal/facts/windows/kernel_resolver.hpp>
-#include <leatherman/windows/system_error.hpp>
-#include <leatherman/windows/windows.hpp>
 #include <facter/facts/os.hpp>
 #include <leatherman/logging/logging.hpp>
+
 #include <boost/optional.hpp>
-#include <boost/algorithm/string/trim.hpp>
 #include <boost/format.hpp>
-#include <boost/nowide/convert.hpp>
+#include <windows.h>
+#include <ntstatus.h>
 
 using namespace std;
-using namespace leatherman::windows;
+using namespace leatherman::dynamic_library;
+using RtlGetVersionPtr = NTSTATUS (WINAPI *)(PRTL_OSVERSIONINFOW);
 
 namespace facter { namespace facts { namespace windows {
 
     static boost::optional<string> get_release()
     {
-        // GetVersionEx requires the manifest is correct, and is essentially deprecated.
-        // Another method of getting the OS version can be found at
-        // http://msdn.microsoft.com/en-us/library/windows/desktop/ms724429(v=vs.85).aspx
-        // and is what we use here.
-        auto fileName = L"Kernel32.dll";
-        auto fileVerSize = GetFileVersionInfoSizeW(fileName, nullptr);
-        if (fileVerSize == 0) {
-            return boost::none;
+        dynamic_library ntoskrnl;
+        if (! ntoskrnl.load("ntoskrnl.exe")) {
+          return boost::none;
         }
 
-        vector<wchar_t> buffer(fileVerSize);
-        if (!GetFileVersionInfoW(fileName, 0, fileVerSize, buffer.data())) {
-            return boost::none;
+        auto rtlGetVersion = reinterpret_cast<RtlGetVersionPtr>(
+            ntoskrnl.find_symbol("RtlGetVersion"));
+        if (! rtlGetVersion) {
+          return boost::none;
         }
 
-        struct LANGANDCODEPAGE {
-            WORD wLanguage, wCodePage;
-        } *lpTranslate;
-        UINT cbTranslate;
-
-        if (!VerQueryValueW(buffer.data(), L"\\VarFileinfo\\Translation",
-            reinterpret_cast<LPVOID*>(&lpTranslate), &cbTranslate)) {
-            return boost::none;
-        }
-
-        // Use the 1st language found, as ProductVersion should be language-independent.
-        wstring subBlock = str(boost::wformat(L"\\StringFileInfo\\%04x%04x\\ProductVersion")
-            % lpTranslate->wLanguage % lpTranslate->wCodePage);
-
-        wchar_t *version;
-        UINT versionLen;
-        if (!VerQueryValueW(buffer.data(), subBlock.c_str(), reinterpret_cast<LPVOID*>(&version), &versionLen)) {
-            return boost::none;
+        OSVERSIONINFOW versionInfo;
+        if (rtlGetVersion(&versionInfo) != STATUS_SUCCESS) {
+          LOG_DEBUG("failed to get the OS version information from RtlGetVersion");
+          return boost::none;
         }
 
-        // Strip the last (file version) token to get just the OS version.
-        wstring versionStrW(version, versionLen);
-        auto versionStr = boost::nowide::narrow(versionStrW);
-        boost::trim_right_if(versionStr, [](char c) { return c != '.'; });  // Remove everything after '.'
-        boost::trim_right_if(versionStr, [](char c) { return c == '.'; });  // Remove '.'
+        auto versionStr = (boost::format("%1%.%2%.%3%")
+            % versionInfo.dwMajorVersion
+            % versionInfo.dwMinorVersion
+            % versionInfo.dwBuildNumber).str();
 
         return versionStr;
     }
@@ -67,8 +49,6 @@ namespace facter { namespace facts { namespace windows {
         if (release) {
             result.release = move(*release);
             result.version = result.release;
-        } else {
-            LOG_DEBUG("failed to retrieve kernel facts: {1}", leatherman::windows::system_error());
         }
 
         result.name = os::windows;

data/ext/facter/facter/lib/src/facts/windows/operating_system_resolver.cc

@@ -2,7 +2,9 @@
 #include <leatherman/windows/system_error.hpp>
 #include <leatherman/windows/wmi.hpp>
 #include <leatherman/windows/windows.hpp>
+#include <facter/facts/fact.hpp>
 #include <facter/facts/collection.hpp>
+#include <facter/facts/scalar_value.hpp>
 #include <facter/facts/os_family.hpp>
 #include <leatherman/logging/logging.hpp>
 #include <leatherman/util/regex.hpp>
@@ -10,6 +12,7 @@
 #include <winnt.h>
 #include <Shlobj.h>
 #include <map>
+#include <string>
 #include <boost/filesystem.hpp>
 
 using namespace std;
@@ -101,7 +104,26 @@ namespace facter { namespace facts { namespace windows {
         auto version = result.release.substr(0, lastDot);
         bool consumerrel = (wmi::get(vals, wmi::producttype) == "1");
         if (version == "10.0") {
-            result.release = consumerrel ? "10" : "2016";
+            // Calculate the build number to distinguish between
+            // Windows Server 2016 and 2019. Note that the kernel
+            // version is written as <major>.<minor>.<build_number>
+            auto kernel_version_fact = facts.get<string_value>(fact::kernel_version);
+            if (! kernel_version_fact) {
+                LOG_DEBUG("Could not resolve the OS release and OS major version facts from the kernel version fact");
+                return result;
+            }
+            auto kernel_version = kernel_version_fact->value();
+            auto build_number_as_str = kernel_version.substr(
+                kernel_version.find_last_of('.') + 1);
+            auto build_number = stol(build_number_as_str);
+
+            if (consumerrel) {
+              result.release = "10";
+            } else if (build_number >= 17623L) {
+              result.release = "2019";
+            } else {
+              result.release = "2016";
+            }
         } else if (version == "6.3") {
             result.release = consumerrel ? "8.1" : "2012 R2";
         } else if (version == "6.2") {

data/ext/facter/facter/lib/src/ruby/ruby.cc

@@ -30,6 +30,27 @@ static const char load_puppet[] =
 "  end\n"
 "end\n";
 
+// This struct redirects stdout to stderr in the ruby runtime for the
+// duration of its lifetime. We use this to ensure that any custom
+// facts writing to stdout during their initialization or execution
+// won't corrupt json/yaml output from the facter executable.
+struct RbStdoutGuard {
+    VALUE old_stdout;
+    api& ruby;
+
+    RbStdoutGuard(api& ruby) :ruby(ruby) {
+        LOG_DEBUG("Redirecting ruby's stdout to stderr");
+        auto rb_stderr = ruby.rb_gv_get("$stderr");
+        old_stdout = ruby.rb_gv_get("$stdout");
+        ruby.rb_gv_set("$stdout", rb_stderr);
+    }
+
+    ~RbStdoutGuard() {
+        LOG_DEBUG("Restoring Ruby's stdout");
+        ruby.rb_gv_set("$stdout", old_stdout);
+    }
+};
+
 namespace facter { namespace ruby {
 
     bool initialize(bool include_stack_trace)
@@ -48,7 +69,7 @@ namespace facter { namespace ruby {
         return true;
     }
 
-    void load_custom_facts(collection& facts, bool initialize_puppet, vector<string> const& paths)
+    void load_custom_facts(collection& facts, bool initialize_puppet, bool redirect_stdout, vector<string> const& paths)
     {
 #ifdef _WIN32
         // Initialize WSA before resolving custom facts. The Ruby runtime does this only when running
@@ -67,12 +88,23 @@ namespace facter { namespace ruby {
             }
         }
         mod.search(paths);
-        mod.resolve_facts();
+        if (redirect_stdout) {
+            // Redirect stdout->stderr for custom facts.
+            RbStdoutGuard stdout_guard{ruby};
+            mod.resolve_facts();
+        } else {
+            mod.resolve_facts();
+        }
     }
 
     void load_custom_facts(collection& facts, vector<string> const& paths)
     {
-         load_custom_facts(facts, false, paths);
+        load_custom_facts(facts, false, false, paths);
+    }
+
+    void load_custom_facts(collection& facts, bool initialize_puppet, vector<string> const& paths)
+    {
+        load_custom_facts(facts, initialize_puppet, false, paths);
     }
 
     value const* lookup(value const* value, vector<string>::iterator segment, vector<string>::iterator end) {

data/ext/facter/facter/lib/tests/CMakeLists.txt

@@ -40,6 +40,7 @@ set(LIBFACTER_TESTS_COMMON_SOURCES
     "logging/logging.cc"
     "log_capture.cc"
     "main.cc"
+    "mock_server.cc"
     "util/string.cc"
     "fixtures.cc"
     "collection_fixture.cc"
@@ -127,37 +128,20 @@ include_directories(
     ${CPPHOCON_INCLUDE_DIRS}
 )
 
-# On EL 4, we run into a linking error when trying to create libraries or
-# executables that link in a static library with code using threads. As I
-# described in https://gcc.gnu.org/ml/gcc-help/2015-08/msg00035.html, we get
-# the error undefined reference to symbol '__tls_get_addr@@GLIBC_2.3'.
-# Build mock_server as a separate shared library to avoid this error.
 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-deprecated-declarations")
-add_library(mock-server SHARED mock_server.cc)
-target_link_libraries(mock-server PRIVATE
-    ${Boost_THREAD_LIBRARY}
-    ${Boost_SYSTEM_LIBRARY}
-    ${LIBFACTER_TESTS_PLATFORM_LIBRARIES})
+if (WIN32)
+    # On Windows with GCC 5.2, Boost.System emits warnings that aren't correctly
+    # suppressed by pragmas. Explicitly skip them.
+	set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unused-variable")
+endif()
 
 add_executable(libfacter_test $<TARGET_OBJECTS:libfactersrc>
     ${LIBFACTER_TESTS_COMMON_SOURCES}
     ${LIBFACTER_TESTS_PLATFORM_SOURCES}
     ${LIBFACTER_TESTS_CATEGORY_SOURCES})
-# On Windows, mock-server comes after Boost libraries to avoid double
-# definition of boost::system::system_category() on Windows. On Linux, it
-# comes before to avoid picking up incomplete Boost.Asio symbols included
-# by Boost.Log in Leatherman logging.
-if (WIN32)
-    target_link_libraries(libfacter_test
-        ${LIBS}
-        ${LIBFACTER_TESTS_PLATFORM_LIBRARIES}
-        mock-server)
-else()
-    target_link_libraries(libfacter_test
-        mock-server
-        ${LIBS}
-        ${LIBFACTER_TESTS_PLATFORM_LIBRARIES})
-endif()
+target_link_libraries(libfacter_test
+    ${LIBS}
+    ${LIBFACTER_TESTS_PLATFORM_LIBRARIES})
 
 if (${CMAKE_SYSTEM_NAME} MATCHES "Darwin" AND BOOST_STATIC AND LEATHERMAN_USE_LOCALES)
     target_link_libraries(libfacter_test iconv)

data/ext/facter/facter/lib/tests/facts/collection.cc

@@ -67,7 +67,7 @@ struct multi_resolver : facter::facts::resolver
 
 struct temp_variable
 {
-    temp_variable(string name, string const& value) :
+    temp_variable(string&& name, string const& value) :
         _name(move(name))
     {
         environment::set(_name, value);

data/ext/facter/facter/locales/FACTER.pot

@@ -6,7 +6,7 @@
 #, fuzzy
 msgid ""
 msgstr ""
-"Project-Id-Version: FACTER 3.11.4\n"
+"Project-Id-Version: FACTER 3.11.5\n"
 "Report-Msgid-Bugs-To: docs@puppet.com\n"
 "POT-Creation-Date: \n"
 "PO-Revision-Date: YEAR-MO-DA HO:MI+ZONE\n"
@@ -1225,11 +1225,6 @@ msgstr ""
 msgid "failed adding platform facts that require WMI: {1}"
 msgstr ""
 
-#. debug
-#: lib/src/facts/windows/dmi_resolver.cc
-msgid "WMI query returned no results for {1} with value {2}."
-msgstr ""
-
 #. debug
 #: lib/src/facts/windows/dmi_resolver.cc
 msgid "WMI query returned no results for {1} with values {2} and {3}."
@@ -1242,7 +1237,11 @@ msgstr ""
 
 #. debug
 #: lib/src/facts/windows/kernel_resolver.cc
-msgid "failed to retrieve kernel facts: {1}"
+msgid "failed to get the OS version information from RtlGetVersion"
+msgstr ""
+
+#: lib/src/facts/windows/kernel_resolver.cc
+msgid "%1%.%2%.%3%"
 msgstr ""
 
 #. debug
@@ -1290,6 +1289,13 @@ msgstr ""
 msgid "error finding SYSTEMROOT: {1}"
 msgstr ""
 
+#. debug
+#: lib/src/facts/windows/operating_system_resolver.cc
+msgid ""
+"Could not resolve the OS release and OS major version facts from the kernel "
+"version fact"
+msgstr ""
+
 #. debug
 #: lib/src/facts/windows/processor_resolver.cc
 msgid "WMI processor Name, Architecture query returned no results."
@@ -1525,6 +1531,16 @@ msgstr ""
 msgid "timeout= is not supported for custom facts and will be ignored."
 msgstr ""
 
+#. debug
+#: lib/src/ruby/ruby.cc
+msgid "Redirecting ruby's stdout to stderr"
+msgstr ""
+
+#. debug
+#: lib/src/ruby/ruby.cc
+msgid "Restoring Ruby's stdout"
+msgstr ""
+
 #. warning
 #: lib/src/ruby/ruby.cc
 msgid "Could not load puppet; some facts may be unavailable: {1}"

data/ext/facter/leatherman/CMakeLists.txt

@@ -1,5 +1,5 @@
 cmake_minimum_required(VERSION 3.2.2)
-project(leatherman VERSION 1.4.2)
+project(leatherman VERSION 1.4.4)
 
 if (WIN32)
     link_libraries("-Wl,--nxcompat -Wl,--dynamicbase")

data/ext/facter/leatherman/ruby/inc/leatherman/ruby/api.hpp

@@ -207,6 +207,10 @@ namespace leatherman {  namespace ruby {
          * See MRI documentation.
          */
         VALUE (* const rb_gv_get)(char const*);
+        /**
+         * See MRI documentation.
+         */
+        VALUE (* const rb_gv_set)(char const*, VALUE);
         /**
          * See MRI documentation.
          */

data/ext/facter/leatherman/ruby/src/api.cc

@@ -51,6 +51,7 @@ namespace leatherman { namespace ruby {
         LOAD_SYMBOL(rb_define_singleton_method),
         LOAD_SYMBOL(rb_class_new_instance),
         LOAD_SYMBOL(rb_gv_get),
+        LOAD_SYMBOL(rb_gv_set),
         LOAD_SYMBOL(rb_eval_string),
         LOAD_SYMBOL(rb_funcall),
         LOAD_ALIASED_SYMBOL(rb_funcallv, rb_funcall2),

data/ext/facter/leatherman/windows/inc/leatherman/windows/wmi.hpp

@@ -43,6 +43,11 @@ namespace leatherman { namespace windows {
          */
         constexpr static char const* computersystemproduct = "Win32_ComputerSystemProduct";
 
+        /**
+         * Identifier for the WMI property UUID
+         */
+        constexpr static char const* uuid = "UUID";
+
         /**
          * Identifier for the WMI class Win32_OperatingSystem
          */

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: facter
 version: !ruby/object:Gem::Version
-  version: 3.11.4.cfacter.20180821
+  version: 3.11.5.cfacter.20181022
 platform: ruby
 authors:
 - Puppet
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-08-21 00:00:00.000000000 Z
+date: 2018-10-22 00:00:00.000000000 Z
 dependencies: []
 description: 
 email: info@puppet.com
@@ -181,6 +181,7 @@ files:
 - ext/facter/facter/CMakeLists.txt
 - ext/facter/facter/CONTRIBUTING.md
 - ext/facter/facter/Extensibility.md
+- ext/facter/facter/Gemfile
 - ext/facter/facter/LICENSE
 - ext/facter/facter/MAINTAINERS
 - ext/facter/facter/README.md
@@ -393,9 +394,8 @@ files:
 - ext/facter/facter/lib/inc/internal/facts/linux/networking_resolver.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/operating_system_resolver.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/os_cisco.hpp
-- ext/facter/facter/lib/inc/internal/facts/linux/os_coreos.hpp
-- ext/facter/facter/lib/inc/internal/facts/linux/os_cumulus.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/os_linux.hpp
+- ext/facter/facter/lib/inc/internal/facts/linux/os_osrelease.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/processor_resolver.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/release_file.hpp
 - ext/facter/facter/lib/inc/internal/facts/linux/uptime_resolver.hpp

data/ext/facter/facter/lib/inc/internal/facts/linux/os_coreos.hpp

@@ -1,56 +0,0 @@
-/**
- * @file
- * Declares the CoreOS Linux operating system query helper.
- */
-#pragma once
-
-#include <internal/facts/linux/os_linux.hpp>
-#include <facter/facts/os.hpp>
-#include <facter/facts/os_family.hpp>
-
-namespace facter { namespace facts { namespace linux {
-
-    /**
-     * Responsible for determining the name/family/release of CoreOS operating systems.
-     */
-    struct os_coreos : os_linux
-    {
-        /**
-         * Constructs the os_cumulus and reads /etc/os-release to gather relevant details.
-         */
-        os_coreos() : os_linux({"VERSION_ID"}) {}
-
-        /**
-         * Returns the release name.
-         * @param distro_id Unused.
-         * @return Returns "CoreOS".
-         */
-        virtual std::string get_name(std::string const& distro_id) const override
-        {
-            return os::coreos;
-        }
-
-        /**
-         * Returns the release family.
-         * @param name Unused.
-         * @return Returns "CoreOS".
-         */
-        virtual std::string get_family(std::string const& name) const override
-        {
-            return os_family::coreos;
-        }
-
-        /**
-         * Finds VERSION_ID from the release file contents and returns it as the release.
-         * @param name Unused.
-         * @param distro_release Unused.
-         * @return Returns the release version.
-         */
-        virtual std::string get_release(std::string const& name, std::string const& distro_release) const override
-        {
-            auto val = _release_info.find("VERSION_ID");
-            return (val != _release_info.end()) ? val->second : std::string();
-        }
-    };
-
-}}}  // namespace facter::facts::linux

data/ext/facter/facter/lib/inc/internal/facts/linux/os_cumulus.hpp

@@ -1,56 +0,0 @@
-/**
- * @file
- * Declares the Cumulus Linux operating system query helper.
- */
-#pragma once
-
-#include <internal/facts/linux/os_linux.hpp>
-#include <facter/facts/os.hpp>
-#include <facter/facts/os_family.hpp>
-
-namespace facter { namespace facts { namespace linux {
-
-    /**
-     * Responsible for determining the name/family/release of Cumulus operating systems.
-     */
-    struct os_cumulus : os_linux
-    {
-        /**
-         * Constructs the os_cumulus and reads /etc/os-release to gather relevant details.
-         */
-        os_cumulus() : os_linux({"VERSION_ID"}) {}
-
-        /**
-         * Returns the release name.
-         * @param distro_id Unused.
-         * @return Returns "CumulusLinux".
-         */
-        virtual std::string get_name(std::string const& distro_id) const override
-        {
-            return os::cumulus;
-        }
-
-        /**
-         * Returns the release family.
-         * @param name Unused.
-         * @return Returns "Debian".
-         */
-        virtual std::string get_family(std::string const& name) const override
-        {
-            return os_family::debian;
-        }
-
-        /**
-         * Finds VERSION_ID from the release file contents and returns it as the release.
-         * @param name Unused.
-         * @param distro_release Unused.
-         * @return Returns the release version.
-         */
-        virtual std::string get_release(std::string const& name, std::string const& distro_release) const override
-        {
-            auto val = _release_info.find("VERSION_ID");
-            return (val != _release_info.end()) ? val->second : std::string();
-        }
-    };
-
-}}}  // namespace facter::facts::linux