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

</pre>
<p><a href="https://github.com/macports/macports-ports/commit/c97c7bdae9e90d22924637f003eefb7b568ad25b">https://github.com/macports/macports-ports/commit/c97c7bdae9e90d22924637f003eefb7b568ad25b</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit c97c7bdae9e90d22924637f003eefb7b568ad25b
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Sun Feb 26 08:09:48 2023 +1100

<span style='display:block; white-space:pre;color:#404040;'>    portgroups: use extract.rename unconditionally
</span>---
 _resources/port1.0/group/github-1.0.tcl | 46 ++++-----------------------------
 _resources/port1.0/group/gitlab-1.0.tcl |  4 +--
 _resources/port1.0/group/golang-1.0.tcl |  2 +-
 3 files changed, 7 insertions(+), 45 deletions(-)

<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/github-1.0.tcl b/_resources/port1.0/group/github-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index c5eacab3e5f..44346ac4f96 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/github-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/github-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -31,27 +31,19 @@ proc handle_tarball_from {option action args} {
</span>         switch ${args} {
             downloads {
                 github.master_sites https://github.com/downloads/${github.author}/${github.project}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    default extract.rename no
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                default extract.rename no
</span>             }
             releases {
                 github.master_sites ${github.homepage}/releases/download/${git.branch}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    default extract.rename no
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                default extract.rename no
</span>             }
             archive {
                 github.master_sites ${github.homepage}/archive/${git.branch}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    default extract.rename yes
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                default extract.rename yes
</span>             }
             tarball {
                 github.master_sites https://codeload.github.com/${github.author}/${github.project}/legacy.tar.gz/${git.branch}?dummy=
<span style='display:block; white-space:pre;background:#ffe0e0;'>-                if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                    default extract.rename yes
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+                default extract.rename yes
</span>             }
             tags {
                 return -code error "the value \"tags\" is deprecated for github.tarball_from. Please use \"tarball\" instead."
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -90,35 +82,7 @@ proc github.setup {gh_author gh_project gh_version {gh_tag_prefix ""} {gh_tag_su
</span>     default master_sites    {${github.master_sites}}
     distname                ${github.project}-${github.version}
 
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        default extract.rename yes
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    # This can be removed when extract.rename has been in a release for 2 weeks.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    post-extract {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # When fetching from a tag, the extracted directory name will contain a
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # truncated commit hash. So that the port author need not specify what
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # that hash is every time the version number changes, rename the
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # directory to the value of distname (not worksrcdir: ports may want to
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # set worksrcdir to a subdirectory of the extracted directory).
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # It is assumed that github.master_sites is a simple string, not a list.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        # Here be dragons.
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        if {![exists extract.rename] && ![file exists ${worksrcpath}] && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                ${github.tarball_from} eq "tarball" && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                ${fetch.type} eq "standard" && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                ${github.master_sites} in ${master_sites} && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                [llength ${extract.only}] > 0 && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                [llength [glob -nocomplain ${workpath}/*]] > 0} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            if {[file exists [glob -nocomplain ${workpath}/${github.author}-${github.project}-*]] && \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                [file isdirectory [glob -nocomplain ${workpath}/${github.author}-${github.project}-*]]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                move [glob ${workpath}/${github.author}-${github.project}-*] ${workpath}/${distname}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            } else {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                # tarball is not "${github.author}-${github.project}-*"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                ui_error "\n\ngithub PortGroup: Error: \${worksrcpath} does not exist after extracting distfiles. This might indicate that the author or project is different than set in the Portfile due to a rename at GitHub. Please examine the extracted directory in ${workpath} and try to correct the Portfile by either changing the author or project or adding the worksrcdir option with the correct directory name.\n"
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-                return -code error "Unexpected github tarball extract."
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-            }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    default extract.rename  yes
</span> 
     # If the version is composed entirely of hex characters, and is at least 7
     # characters long, and is not exactly 8 decimal digits (which might be a
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/gitlab-1.0.tcl b/_resources/port1.0/group/gitlab-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 25c59cc5b96..c34f8bcef84 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/gitlab-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/gitlab-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -51,9 +51,7 @@ proc gitlab.setup {gl_author gl_project gl_version {gl_tag_prefix ""} {gl_tag_su
</span> 
     # When fetching from a tag, the extracted directory name will contain a
     # truncated commit hash.
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[exists extract.rename]} {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-        default extract.rename yes
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-    }
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    default extract.rename yes
</span> 
     # If the version is composed entirely of hex characters, and is at least 7
     # characters long, and is not exactly 8 decimal digits (which might be a
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/_resources/port1.0/group/golang-1.0.tcl b/_resources/port1.0/group/golang-1.0.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index 9e129cd0766..2c81f4e7ca2 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/_resources/port1.0/group/golang-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/_resources/port1.0/group/golang-1.0.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -240,7 +240,7 @@ proc handle_set_go_vendors {vendors_str} {
</span>         set checksum_types $portchecksum::checksum_types
     }
     set num_tokens [llength ${vendors_str}]
<span style='display:block; white-space:pre;background:#ffe0e0;'>-    if {[exists extract.rename] && $num_tokens > 0} {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+    if {$num_tokens > 0} {
</span>         # portgroups like github may set this - can't be used with multiple distfiles
         extract.rename  no
     }
</pre><pre style='margin:0'>

</pre>