With new ruby (1.9.3) and puppet (3.2) the module does not work

[onyxmaster]

Attaching patch that fixed it for me: The first change is harmless, the second one is backwards incompatible.

--- lib/puppet/provider/net_share/net_share.rb  2013-05-23 15:40:04.000000000 +0400
+++ lib/puppet/provider/net_share/net_share.rb  2013-05-23 15:47:07.000000000 +0400
@@ -73,7 +73,7 @@

     properties[:ensure] = :present

-    output.each do |line|
+    output.each_line do |line|
       break if line.rstrip.length == 0

       last_name = name
--- lib/puppet/type/net_share.rb    2013-05-23 15:40:04.000000000 +0400
+++ lib/puppet/type/net_share.rb    2013-05-23 15:47:07.000000000 +0400
@@ -51,10 +51,8 @@
     desc "An array of permissions. Example: ['computer\\user,full', 'computer\\user2,change', 'computer\\user3,read']"

     munge do |value|
-      value.collect do |item|
-        user, access = item.split(',', 2)
-        "#{user.strip},#{access.strip.downcase}"
-      end
+      user, access = value.split(',', 2)
+      "#{user.strip},#{access.strip.downcase}"
     end
   end

[nibalizer]

@simondean Have you reviewed the patch? Confirm/denied the bug?

@onyxmaster Maybe submit the patch as a github pull request?

[Cylindric]

Aah, thank you! I was getting errors until I found this patch :)

Munging failed for value "computername\\username,full" in class permissions: undefined method `collect' for "computername\\username,full":String

[TerribleDev]

👍

[onyxmaster]

Looks like this is never going to be fixed in this repo, well there are other forks =)