Merge pull request #612 from smortex/freebsd

Add support for FreeBSD
voxpupuli · May 5, 2021 · 3c31b13 · 3c31b13
2 parents 91c32e2 + 237d7a0
commit 3c31b13
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 7 deletions.
diff --git a/manifests/install.pp b/manifests/install.pp
@@ -15,11 +15,12 @@
   }
 
   $pythondev = $facts['os']['family'] ? {
-    'AIX'    => "${python}-devel",
-    'RedHat' => "${python}-devel",
-    'Debian' => "${python}-dev",
-    'Suse'   => "${python}-devel",
-    'Gentoo' => undef,
+    'AIX'     => "${python}-devel",
+    'Debian'  => "${python}-dev",
+    'FreeBSD' => undef,
+    'Gentoo'  => undef,
+    'RedHat'  => "${python}-devel",
+    'Suse'    => "${python}-devel",
   }
 
   $pip_ensure = $python::pip ? {
@@ -225,6 +226,10 @@
         $pip_category = undef
         $pip_package  = 'python2-pip'
         $pip_provider = pip2
+      } elsif $facts['os']['family'] == 'FreeBSD' {
+        $pip_category = undef
+        $pip_package  = "py${python::version}-pip"
+        $pip_provider = 'pip'
       } elsif $facts['os']['family'] == 'Gentoo' {
         $pip_category = 'dev-python'
         $pip_package  = 'pip'

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -5,7 +5,7 @@
 #
 class python::params {
   # Module compatibility check
-  unless $facts['os']['family'] in ['Debian', 'RedHat', 'Suse', 'Gentoo', 'AIX',] {
+  unless $facts['os']['family'] in ['AIX', 'Debian', 'FreeBSD', 'Gentoo', 'RedHat', 'Suse'] {
     fail("Module is not compatible with ${facts['os']['name']}")
   }
 

diff --git a/metadata.json b/metadata.json
@@ -36,6 +36,14 @@
         "10"
       ]
     },
+    {
+      "operatingsystem": "FreeBSD",
+      "operatingsystemrelease": [
+        "11",
+        "12",
+        "13"
+      ]
+    },
     {
       "operatingsystem": "Gentoo"
     },

diff --git a/spec/defines/requirements_spec.rb b/spec/defines/requirements_spec.rb
@@ -53,7 +53,7 @@
         it { is_expected.to contain_package('gunicorn') }
         it { is_expected.to contain_file('/requirements.txt').with_owner('root').with_group('root') }
 
-        if facts[:os]['name'] == 'Gentoo'
+        if %w[FreeBSD Gentoo].include?(facts[:os]['name'])
           it { is_expected.not_to contain_package('python-dev') }
         else
           it { is_expected.to contain_package('python-dev') }
-Original file line number
+Diff line change
@@ Expand Up / @@ -36,6 +36,14 @@ @@
             "10"
           ]
         },
+        {
+          "operatingsystem": "FreeBSD",
+          "operatingsystemrelease": [
+            "11",
+            "12",
+            "13"
+          ]
+        },
         {
           "operatingsystem": "Gentoo"
         },
@@ Expand Down @@