<pre style='margin:0'>
Joshua Root (jmroot) pushed a commit to branch master
in repository mpbb.
</pre>
<p><a href="https://github.com/macports/mpbb/commit/9a735472646136ff02cffc9399fabe2cf8e6fa94">https://github.com/macports/mpbb/commit/9a735472646136ff02cffc9399fabe2cf8e6fa94</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 9a73547 dependencies.tcl: attempt to mitigate 503 errors
</span>9a73547 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 9a735472646136ff02cffc9399fabe2cf8e6fa94
</span>Author: Joshua Root <jmr@macports.org>
AuthorDate: Fri Oct 27 18:09:06 2023 +1100
<span style='display:block; white-space:pre;color:#404040;'> dependencies.tcl: attempt to mitigate 503 errors
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Ensure that archivefetch is not attempted at all in install_dep_source,
</span><span style='display:block; white-space:pre;color:#404040;'> since we only get there if it already failed. This should prevent
</span><span style='display:block; white-space:pre;color:#404040;'> failure from archivefetch getting a 503 for a signature again.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> This is at best a poor and partial workaround, since archivefetch
</span><span style='display:block; white-space:pre;color:#404040;'> failing for ports that have an archive available in the first place
</span><span style='display:block; white-space:pre;color:#404040;'> means that those ports are unnecessarily built from source, and
</span><span style='display:block; white-space:pre;color:#404040;'> furthermore, fetching the distfiles might fail the same way.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> See: https://trac.macports.org/ticket/68563
</span>---
tools/dependencies.tcl | 4 ++++
1 file changed, 4 insertions(+)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/tools/dependencies.tcl b/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;color:#808080;'>index dcc46ce..d8318c1 100755
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/tools/dependencies.tcl
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -497,6 +497,10 @@ proc install_dep_source {portinfo_list} {
</span> close_open_mports
array unset ::mportinfo_array
set ditem [lindex [open_port $depinfo(name)] 0]
<span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Ensure archivefetch is not attempted at all
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set workername [ditem_key $ditem workername]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ $workername eval [list set portutil::archive_available_result 0]
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ $workername eval [list archive_sites]
</span>
# deactivate ports not needed for this dep
if {[catch {deactivate_unneeded depinfo} result]} {
</pre><pre style='margin:0'>
</pre>