From 50d7dd1a8412cd4baaf93118924b0945432cbb63 Mon Sep 17 00:00:00 2001
From: James Le Cuirot <jlecuirot@microsoft.com>
Date: Wed, 16 Apr 2025 12:11:57 +0100
Subject: [PATCH 1/2] Drop all references to a toolchain-specific binpkg host

This evidently hasn't been used in a very long time, if ever, and it's
just adding confusion.

Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
---
 build_library/dev_container_util.sh |  3 +--
 build_library/toolchain_util.sh     | 23 +++--------------------
 build_packages                      |  7 -------
 setup_board                         |  6 +-----
 4 files changed, 5 insertions(+), 34 deletions(-)

diff --git a/build_library/dev_container_util.sh b/build_library/dev_container_util.sh
index f87bd1235a..1643a61f00 100755
--- a/build_library/dev_container_util.sh
+++ b/build_library/dev_container_util.sh
@@ -38,8 +38,7 @@ CHOST=$(get_board_chost $BOARD)
 DISTDIR="/var/lib/portage/distfiles"
 PKGDIR="/var/lib/portage/pkgs"
 PORT_LOGDIR="/var/log/portage"
-PORTAGE_BINHOST="$(get_binhost_url "${binhost}" "${update_group}" 'pkgs')
-$(get_binhost_url "${binhost}" "${update_group}" 'toolchain')"
+PORTAGE_BINHOST="$(get_binhost_url "${binhost}" "${update_group}" 'pkgs')"
 EOF
 
     sudo_clobber "${root_fs_dir}/etc/portage/repos.conf/portage-stable.conf" <<EOF
diff --git a/build_library/toolchain_util.sh b/build_library/toolchain_util.sh
index 8960a37483..2b064ecbef 100644
--- a/build_library/toolchain_util.sh
+++ b/build_library/toolchain_util.sh
@@ -130,37 +130,24 @@ get_board_profile() {
     done
 }
 
-# Usage: get_board_binhost [-t] board [version...]
-# -t: toolchain only, full rebuilds re-using toolchain pkgs
+# Usage: get_board_binhost board [version...]
 # If no versions are specified the current and SDK versions are used.
 get_board_binhost() {
-    local toolchain_only=0 board ver
-    if [[ "$1" == "-t" ]]; then
-        toolchain_only=1
-        shift
-    fi
+    local board ver
     board="$1"
     shift
 
-    local pkgs_include_toolchain=0
     if [[ $# -eq 0 ]]; then
         if [[ "${FLATCAR_BUILD_ID}" =~ ^nightly-.*$ ]] ; then
             # containerised nightly build; this uses [VERSION]-[BUILD_ID] for binpkg url
-            #  and toolchain packages are at the same location as OS image ones
             set -- "${FLATCAR_VERSION_ID}+${FLATCAR_BUILD_ID}"
-            pkgs_include_toolchain=1
         else
             set -- "${FLATCAR_VERSION_ID}"
         fi
     fi
 
     for ver in "$@"; do
-        if [[ $toolchain_only -eq 0 ]]; then
-            echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/pkgs/"
-        fi
-        if [[ $pkgs_include_toolchain -eq 0 ]]; then
-            echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/toolchain/"
-        fi
+        echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/pkgs/"
     done
 }
 
@@ -196,13 +183,9 @@ get_sdk_binhost() {
         FLATCAR_DEV_BUILDS_SDK="${FLATCAR_DEV_BUILDS_SDK-${SETTING_BINPKG_SERVER_PROD}/sdk}"
     fi
     for ver in "$@"; do
-        # Usually only crossdev needs to be fetched from /toolchain/ in the setup_board step.
         # The entry for /pkgs/ is there if something needs to be reinstalled in the SDK
         # but normally it is not needed because everything is already part of the tarball.
-        # To install the crossdev Rust package, /toolchain-arm64/ is derived from /toolchain/
-        # when necessary in install_cross_toolchain().
         if curl -Ifs -o /dev/null "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/pkgs/"; then
-            echo "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/toolchain/"
             echo "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/pkgs/"
         fi
     done
diff --git a/build_packages b/build_packages
index 77c7742b7b..b3c0186c39 100755
--- a/build_packages
+++ b/build_packages
@@ -24,8 +24,6 @@ DEFINE_boolean getbinpkg "${FLAGS_TRUE}" \
   "Download binary packages from remote repository."
 DEFINE_string getbinpkgver "" \
   "Use binary packages from a specific version."
-DEFINE_boolean toolchainpkgonly "${FLAGS_FALSE}" \
-  "Use binary packages only for the board toolchain."
 DEFINE_boolean workon "${FLAGS_TRUE}" \
   "Automatically rebuild updated flatcar-workon packages."
 DEFINE_boolean fetchonly "${FLAGS_FALSE}" \
@@ -100,11 +98,6 @@ if [ "${FLAGS_usepkg}" -eq "${FLAGS_TRUE}" ]; then
   else
     UPDATE_ARGS+=( --nogetbinpkg )
   fi
-  if [[ "${FLAGS_toolchainpkgonly}" -eq "${FLAGS_TRUE}" ]]; then
-    UPDATE_ARGS+=( --toolchainpkgonly )
-  else
-    UPDATE_ARGS+=( --notoolchainpkgonly )
-  fi
   if [[ -n "${FLAGS_getbinpkgver}" ]]; then
     UPDATE_ARGS+=( --getbinpkgver="${FLAGS_getbinpkgver}" )
   fi
diff --git a/setup_board b/setup_board
index f2c45188fe..c1acdc1f25 100755
--- a/setup_board
+++ b/setup_board
@@ -31,8 +31,6 @@ DEFINE_string pkgdir "" \
   "Use binary packages from a custom directory instead of /build/[ARCH]/var/lib/portage/pkgs/."
 DEFINE_string binhost "" \
   "Use binary packages from a specific location instead of $FLATCAR_DEV_BUILDS/... "
-DEFINE_boolean toolchainpkgonly "${FLAGS_FALSE}" \
-  "Use binary packages only for the board toolchain."
 DEFINE_boolean skip_toolchain_update "${FLAGS_FALSE}" \
   "Don't update toolchain automatically."
 DEFINE_boolean skip_chroot_upgrade "${FLAGS_FALSE}" \
@@ -136,13 +134,11 @@ EOF
 }
 
 generate_binhost_list() {
-  local t
-  [[ "${FLAGS_toolchainpkgonly}" -eq "${FLAGS_TRUE}" ]] && t="-t"
   FLAGS_getbinpkgver="${FLAGS_getbinpkgver/current/${FLATCAR_VERSION_ID}}"
   FLAGS_getbinpkgver="${FLAGS_getbinpkgver/latest/${FLATCAR_VERSION_ID}}"
   FLAGS_getbinpkgver="${FLAGS_getbinpkgver/sdk/${FLATCAR_SDK_VERSION}}"
 
-  get_board_binhost $t "${BOARD}" ${FLAGS_getbinpkgver}
+  get_board_binhost "${BOARD}" ${FLAGS_getbinpkgver}
 }
 
 # Parse command line flags

From e57080daf3d3f5b1397405419b6fe1f015d3d2f1 Mon Sep 17 00:00:00 2001
From: James Le Cuirot <jlecuirot@microsoft.com>
Date: Wed, 16 Apr 2025 12:29:51 +0100
Subject: [PATCH 2/2] Drop obsolete SETTING_BINPKG_SERVER_DEV variable

This isn't used anywhere, and the referenced bucket is empty.

Signed-off-by: James Le Cuirot <jlecuirot@microsoft.com>
---
 set_version  | 3 +--
 settings.env | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/set_version b/set_version
index 240002a14d..833a182440 100755
--- a/set_version
+++ b/set_version
@@ -43,8 +43,7 @@ Usage: $0 FLAGS...
   --file FILE:                                     Modify another file than ${FILE}, useful if run
                                                    outside of the SDK chroot. If /dev/stdout or
                                                    /dev/stderr is used, only new values are printed.
-  --binhost                                        Use a custom binhost (defaults to '${DEFAULT_BASE_URL}'),
-                                                   e.g., '${SETTING_BINPKG_SERVER_DEV}'.
+  --binhost                                        Use a custom binhost (defaults to '${DEFAULT_BASE_URL}').
                                                    This will update BOARD and SDK URLs accordingly.
 "
   exit 1
diff --git a/settings.env b/settings.env
index a0b0cd537d..3961d5d4ce 100644
--- a/settings.env
+++ b/settings.env
@@ -4,5 +4,4 @@
 SETTING_BINPKG_SERVER_PROD="https://mirror.release.flatcar-linux.net"
 
 # development servers / bin caches.
-SETTING_BINPKG_SERVER_DEV="https://bucket.release.flatcar-linux.net/flatcar-jenkins"
 SETTING_BINPKG_SERVER_DEV_CONTAINERISED="https://bincache.flatcar-linux.net"