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

</pre>
<p><a href="https://github.com/macports/macports-infrastructure/commit/2653e51dcfcca9e3fda5c19046acf559c3591f09">https://github.com/macports/macports-infrastructure/commit/2653e51dcfcca9e3fda5c19046acf559c3591f09</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 2653e51dcfcca9e3fda5c19046acf559c3591f09
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Fri Mar 28 01:26:10 2025 +1100

<span style='display:block; white-space:pre;color:#404040;'>    deploy_archives: remove NEED_LOCK checks
</span><span style='display:block; white-space:pre;color:#404040;'>    
</span><span style='display:block; white-space:pre;color:#404040;'>    It's always true now.
</span>---
 buildbot/deploy_archives.sh | 57 ++++++++++++++++++---------------------------
 1 file changed, 23 insertions(+), 34 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/buildbot/deploy_archives.sh b/buildbot/deploy_archives.sh
</span><span style='display:block; white-space:pre;color:#808080;'>index d8c1152..063dea3 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/buildbot/deploy_archives.sh
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/buildbot/deploy_archives.sh
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -12,11 +12,6 @@ if [[ -z "$DLPATH" ]]; then
</span>     DLPATH="./deployed_archives"
 fi
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# Always lock, because multiple builders may be deploying files with
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# the same names at the same time, so unique subdirs are not enough.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# See: https://trac.macports.org/ticket/62977
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-NEED_LOCK=1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span> # path where archives get uploaded to buildmaster
 if [[ -z "$ULPATH" ]]; then
     # workaround for buildbot not accepting WithProperties in env
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -32,37 +27,35 @@ if [[ -z "$PRIVKEY" ]]; then
</span>     PRIVKEY=""
 fi
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-# buildbot apparently doesn't run jobs on the master in different dirs or
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-# prevent them from running simultaneously
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-if [[ -n "$NEED_LOCK" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if [[ -z "$LOCKFILE" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        LOCKFILE="./deploy.lock"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Buildbot apparently doesn't run jobs on the master in different dirs or
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# prevent them from running simultaneously.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# Always lock, because multiple builders may be deploying files with
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# the same names at the same time, so unique subdirs are not enough.
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+# See: https://trac.macports.org/ticket/62977
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if [[ -z "$LOCKFILE" ]]; then
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    LOCKFILE="./deploy.lock"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+fi
</span> 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    echo Acquiring lock...
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    if [[ "`uname -s`" = "Darwin" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        SLEPT=0
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        while ! shlock -f "$LOCKFILE" -p $$; do
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            sleep 1
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            let SLEPT="$SLEPT + 1"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            if [[ "$SLEPT" -gt 600 ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                echo Timeout acquiring lock, continuing anyway...
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                break
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            fi
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        done
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    else
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        if ! lockfile -1 -r 600 "$LOCKFILE"; then
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+echo Acquiring lock...
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+if [[ "`uname -s`" = "Darwin" ]]; then
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    SLEPT=0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    while ! shlock -f "$LOCKFILE" -p $$; do
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        sleep 1
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        let SLEPT="$SLEPT + 1"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        if [[ "$SLEPT" -gt 600 ]]; then
</span>             echo Timeout acquiring lock, continuing anyway...
<span style='display:block; white-space:pre;background:#e0ffe0;'>+            break
</span>         fi
<span style='display:block; white-space:pre;background:#e0ffe0;'>+    done
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+else
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if ! lockfile -1 -r 600 "$LOCKFILE"; then
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+        echo Timeout acquiring lock, continuing anyway...
</span>     fi
<span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span> fi
 
 if [[ ! -d $ULPATH ]]; then
     echo $ULPATH does not exist!
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if [[ -n "$NEED_LOCK" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        rm -f $LOCKFILE
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    rm -f "$LOCKFILE"
</span>     exit 1
 fi
 
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -77,9 +70,7 @@ if [[ -n "`ls ${ULPATH}`" ]]; then
</span>                 chmod a+r ${ULPATH}/${portname}/${aname}.rmd160
             else
                 rm -rf $ULPATH
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                if [[ -n "$NEED_LOCK" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    rm -f $LOCKFILE
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                rm -f "$LOCKFILE"
</span>                 exit 1
             fi
         fi
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -96,8 +87,6 @@ else
</span> fi
 
 # clean up after ourselves
<span style='display:block; white-space:pre;background:#ffe0e0;'>-if [[ -n "$NEED_LOCK" ]]; then
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    rm -f $LOCKFILE
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+rm -f "$LOCKFILE"
</span> rm -rf $ULPATH
 
</pre><pre style='margin:0'>

</pre>