Add a binary_indicator parameter for the usage of $digest_string.

The hash commands on OpenBSD don't distinguish between binary and text files, so they don't handle the '*' in front of the file name to be checked. additionally update metadata.json for more current OpenBSD versions
camptocamp · Aug 30, 2016 · d75c797 · d75c797
1 parent 82fc829
commit d75c797
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 2 deletions.
diff --git a/manifests/download.pp b/manifests/download.pp
@@ -93,7 +93,7 @@
           'present': {
             file {"${src_target}/${name}.${digest_type}":
               ensure  => $ensure,
-              content => "${digest_string} *${name}",
+              content => "${digest_string} ${::archive::params::binary_indicator}${name}",
               owner   => $user,
               notify  => Exec["download archive ${name} and check sum"],
             }

diff --git a/manifests/extract.pp b/manifests/extract.pp
@@ -47,6 +47,8 @@
   },
 ) {
 
+  require ::archive::params
+
   if $root_dir {
     $extract_dir = "${target}/${root_dir}"
   } else {

diff --git a/manifests/params.pp b/manifests/params.pp
@@ -12,6 +12,7 @@
         'sha256' => 'sha256',
         'sha512' => 'sha512',
       }
+      $binary_indicator = undef
     }
     default: {
       $tarcmd = 'tar'
@@ -23,6 +24,7 @@
         'sha384' => 'sha384sum',
         'sha512' => 'sha512sum',
       }
+      $binary_indicator = '*'
     }
   }
 }
diff --git a/metadata.json b/metadata.json
@@ -26,7 +26,10 @@
     {
       "operatingsystem": "OpenBSD",
       "operatingsystemrelease": [
-        "5.7"
+        "5.7",
+        "5.8",
+        "5.9",
+        "6.0"
       ]
     },
     {
-Original file line number
+Diff line change
@@ Expand Up / @@ -26,7 +26,10 @@ @@
         {
           "operatingsystem": "OpenBSD",
           "operatingsystemrelease": [
-            "5.7"
+            "5.7",
+            "5.8",
+            "5.9",
+            "6.0"
           ]
         },
         {
@@ Expand Down @@