zip: restore extended attributes when under MacOS

Homebrew · Oct 8, 2018 · 9e43899 · 9e43899
1 parent 3fd75bb
commit 9e43899
Showing 1 changed file with 15 additions and 4 deletions.
diff --git a/Library/Homebrew/extend/os/mac/unpack_strategy/zip.rb b/Library/Homebrew/extend/os/mac/unpack_strategy/zip.rb
@@ -2,10 +2,21 @@ module UnpackStrategy
   class Zip
     prepend Module.new {
       def extract_to_dir(unpack_dir, basename:, verbose:)
-        volumes = super.stderr.chomp
-                       .split("\n")
-                       .map { |l| l[/\A   skipping: (.+)  volume label\Z/, 1] }
-                       .compact
+        result = super
+
+        volumes = result.stderr.chomp
+                        .split("\n")
+                        .map { |l| l[/\A   skipping: (.+)  volume label\Z/, 1] }
+                        .compact
+
+        if result.stderr.lines.any? { |line| line.start_with?("._") }
+          # Merge ._ files back into extended attributes.
+          # ._ files inside volumes are automatically merged by ditto.
+          system_command!("dot_clean",
+                          args: ["-mv", "--keep=dotbar", unpack_dir],
+                          verbose: verbose,
+                          print_stderr: false)
+        end
 
         return if volumes.empty?