X-Git-Url: https://review.fuel-infra.org/gitweb?a=blobdiff_plain;f=manifests%2Fpin.pp;h=bd8b470b1710160879bbeb7e3e9c7bd9e99d1720;hb=refs%2Fheads%2Frelease-prep;hp=8145b7d4b6575c024c1435e803beb5ebda6ea334;hpb=d6069a619ce1b30f19014932f81e83f445c846e6;p=puppet-modules%2Fpuppetlabs-apt.git

diff --git a/manifests/pin.pp b/manifests/pin.pp
index 8145b7d..bd8b470 100644
--- a/manifests/pin.pp
+++ b/manifests/pin.pp
@@ -33,22 +33,30 @@
 # @param label
 #   Names the label of the packages in the directory tree of the Release file.
 #
-define apt::pin(
-  Optional[Enum['file', 'present', 'absent']] $ensure = present,
-  Optional[String] $explanation                       = undef,
-  Variant[Integer] $order                             = 50,
-  Variant[String, Array] $packages                    = '*',
-  Variant[Numeric, String] $priority                  = 0,
-  Optional[String] $release                           = '', # a=
-  Optional[String] $origin                            = '',
-  Optional[String] $version                           = '',
-  Optional[String] $codename                          = '', # n=
-  Optional[String] $release_version                   = '', # v=
-  Optional[String] $component                         = '', # c=
-  Optional[String] $originator                        = '', # o=
-  Optional[String] $label                             = '',  # l=
+# @param origin
+#   The package origin
+#
+# @param version
+#   The version of the package
+#
+# @param codename
+#   The codename of the package
+#
+define apt::pin (
+  Enum['file', 'present', 'absent'] $ensure = present,
+  Optional[String] $explanation             = undef,
+  Variant[Integer] $order                   = 50,
+  Variant[String, Array] $packages          = '*',
+  Variant[Numeric, String] $priority        = 0,
+  Optional[String] $release                 = undef, # a=
+  Optional[String] $origin                  = undef,
+  Optional[String] $version                 = undef,
+  Optional[String] $codename                = undef, # n=
+  Optional[String] $release_version         = undef, # v=
+  Optional[String] $component               = undef, # c=
+  Optional[String] $originator              = undef, # o=
+  Optional[String] $label                   = undef,  # l=
 ) {
-
   if $explanation {
     $_explanation = $explanation
   } else {
@@ -79,16 +87,16 @@ define apt::pin(
   }
 
   if $packages_string != '*' { # specific form
-    if ( $pin_release != '' and ( $origin != '' or $version != '' )) or
-      ( $version != '' and ( $pin_release != '' or $origin != '' )) {
-      fail(translate('parameters release, origin, and version are mutually exclusive'))
+    if ( $pin_release != '' and ( $origin or $version )) or
+    ( $version and ( $pin_release != '' or $origin )) {
+      fail('parameters release, origin, and version are mutually exclusive')
     }
   } else { # general form
-    if $version != '' {
-      fail(translate('parameter version cannot be used in general form'))
+    if $version {
+      fail('parameter version cannot be used in general form')
     }
-    if ( $pin_release != '' and $origin != '' ) {
-      fail(translate('parameters release and origin are mutually exclusive'))
+    if ( $pin_release != '' and $origin ) {
+      fail('parameters release and origin are mutually exclusive')
     }
   }