On Tue, Jan 13, 2026 at 10:57:37AM +0000, Stuart Henderson wrote:
> why not patch the command line to use tar-compatible flags instead? 
> --no-same-owner is the default for our tar, and --directory is just the 
> same as -C
> 
good catch, diff follows.
 Cheers
  Giovanni

Index: Makefile
===================================================================
RCS file: /cvs/ports/sysutils/ruby-puppet/8/Makefile,v
diff -u -p -r1.9 Makefile
--- Makefile    22 Sep 2025 12:35:15 -0000      1.9
+++ Makefile    13 Jan 2026 13:56:04 -0000
@@ -1,7 +1,7 @@
 PORTROACH=             limit:^8
 
 VERSION=               8.10.0
-REVISION=              3
+REVISION=              4
 
 RUN_DEPENDS+=          converters/ruby-multi_json,${MODRUBY_FLAVOR}>=1.13,<2 \
                        devel/ruby-concurrent-ruby,${MODRUBY_FLAVOR}>=1,<2 \
Index: patches/patch-lib_puppet_module_tool_tar_gnu_rb
===================================================================
RCS file: patches/patch-lib_puppet_module_tool_tar_gnu_rb
diff -N patches/patch-lib_puppet_module_tool_tar_gnu_rb
--- /dev/null   1 Jan 1970 00:00:00 -0000
+++ patches/patch-lib_puppet_module_tool_tar_gnu_rb     13 Jan 2026 13:56:04 
-0000
@@ -0,0 +1,12 @@
+Index: lib/puppet/module_tool/tar/gnu.rb
+--- lib/puppet/module_tool/tar/gnu.rb.orig
++++ lib/puppet/module_tool/tar/gnu.rb
+@@ -8,7 +8,7 @@ class Puppet::ModuleTool::Tar::Gnu
+     destdir = File.expand_path(destdir)
+     safe_destdir = Shellwords.shellescape(destdir)
+ 
+-    Puppet::Util::Execution.execute("gzip -dc #{safe_sourcefile} | tar 
--extract --no-same-owner --directory #{safe_destdir} --file -")
++    Puppet::Util::Execution.execute("gzip -dc #{safe_sourcefile} | tar 
--extract -C #{safe_destdir} --file -")
+     Puppet::Util::Execution.execute(['find', destdir, '-type', 'd', '-exec', 
'chmod', '755', '{}', '+'])
+     Puppet::Util::Execution.execute(['find', destdir, '-type', 'f', '-exec', 
'chmod', 'u+rw,g+r,a-st', '{}', '+'])
+     Puppet::Util::Execution.execute(['chown', '-R', owner, destdir])

Attachment: signature.asc
Description: PGP signature

Reply via email to