<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository macports-base.

</pre>
<p><a href="https://github.com/macports/macports-base/commit/3f248bd82a7e689fd80ee658f9e981ac0912b1d0">https://github.com/macports/macports-base/commit/3f248bd82a7e689fd80ee658f9e981ac0912b1d0</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'>     new 3f248bd8 Put trace mode socket in a subdir
</span>3f248bd8 is described below

<span style='display:block; white-space:pre;color:#808000;'>commit 3f248bd82a7e689fd80ee658f9e981ac0912b1d0
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Wed Apr 14 03:05:28 2021 +1000

<span style='display:block; white-space:pre;color:#404040;'>    Put trace mode socket in a subdir
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    Closes: https://trac.macports.org/ticket/56180
</span>---
 src/port1.0/porttrace.tcl | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/port1.0/porttrace.tcl b/src/port1.0/porttrace.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index caeb8e15..e3beab91 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/port1.0/porttrace.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/port1.0/porttrace.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -45,10 +45,8 @@ namespace eval porttrace {
</span>     # The mktemp(3) template used to generate a filename for the fifo.
     # Note that Unix sockets are limited to 109 characters and that the
     # macports user must be able to connect to the socket (and in case of
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    # non-root installations, the current user, too). We're not prefixing the
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # path in /tmp with a separate macports-specific directory, because this
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # might not be writable by all users.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    variable fifo_mktemp_template "/tmp/macports-trace-XXXXXX"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    # non-root installations, the current user, too).
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    variable fifo_mktemp_template "/tmp/macports-trace/sock-XXXXXX"
</span> 
     ##
     # The Tcl thread that runs the server side of trace mode and deals with
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -146,6 +144,17 @@ namespace eval porttrace {
</span>         # processes being traced.
         set fifo [mktemp $fifo_mktemp_template]
 
<span style='display:block; white-space:pre;background:#e0ffe0;'>+        # Create enclosing directory with correct permissions, i.e. no
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # finding out what the socket is called by listing the directory,
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        # but it can be opened if you know its name.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        set fifo_dir [file dirname $fifo]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        file mkdir $fifo_dir
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        if {[geteuid] == 0} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            file attributes $fifo_dir -permissions 0311 -owner $macportsuser
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        } else {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+            file attributes $fifo_dir -permissions 0311
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span>         # Make sure the socket doesn't exist yet (this would cause errors
         # later)
         file delete -force $fifo
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -286,6 +295,7 @@ namespace eval porttrace {
</span>         tracelib clean
         # Delete the socket file
         file delete -force $fifo
<span style='display:block; white-space:pre;background:#e0ffe0;'>+        file delete -force [file dirname $fifo]
</span> 
         # Delete the slave.
         delete_slave
</pre><pre style='margin:0'>

</pre>