Skip to content

Commit 50d7dd1

Browse files
committed
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 <[email protected]>
1 parent 8fae246 commit 50d7dd1

File tree

4 files changed

+5
-34
lines changed

4 files changed

+5
-34
lines changed

build_library/dev_container_util.sh

+1-2
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,7 @@ CHOST=$(get_board_chost $BOARD)
3838
DISTDIR="/var/lib/portage/distfiles"
3939
PKGDIR="/var/lib/portage/pkgs"
4040
PORT_LOGDIR="/var/log/portage"
41-
PORTAGE_BINHOST="$(get_binhost_url "${binhost}" "${update_group}" 'pkgs')
42-
$(get_binhost_url "${binhost}" "${update_group}" 'toolchain')"
41+
PORTAGE_BINHOST="$(get_binhost_url "${binhost}" "${update_group}" 'pkgs')"
4342
EOF
4443

4544
sudo_clobber "${root_fs_dir}/etc/portage/repos.conf/portage-stable.conf" <<EOF

build_library/toolchain_util.sh

+3-20
Original file line numberDiff line numberDiff line change
@@ -130,37 +130,24 @@ get_board_profile() {
130130
done
131131
}
132132

133-
# Usage: get_board_binhost [-t] board [version...]
134-
# -t: toolchain only, full rebuilds re-using toolchain pkgs
133+
# Usage: get_board_binhost board [version...]
135134
# If no versions are specified the current and SDK versions are used.
136135
get_board_binhost() {
137-
local toolchain_only=0 board ver
138-
if [[ "$1" == "-t" ]]; then
139-
toolchain_only=1
140-
shift
141-
fi
136+
local board ver
142137
board="$1"
143138
shift
144139

145-
local pkgs_include_toolchain=0
146140
if [[ $# -eq 0 ]]; then
147141
if [[ "${FLATCAR_BUILD_ID}" =~ ^nightly-.*$ ]] ; then
148142
# containerised nightly build; this uses [VERSION]-[BUILD_ID] for binpkg url
149-
# and toolchain packages are at the same location as OS image ones
150143
set -- "${FLATCAR_VERSION_ID}+${FLATCAR_BUILD_ID}"
151-
pkgs_include_toolchain=1
152144
else
153145
set -- "${FLATCAR_VERSION_ID}"
154146
fi
155147
fi
156148

157149
for ver in "$@"; do
158-
if [[ $toolchain_only -eq 0 ]]; then
159-
echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/pkgs/"
160-
fi
161-
if [[ $pkgs_include_toolchain -eq 0 ]]; then
162-
echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/toolchain/"
163-
fi
150+
echo "${FLATCAR_DEV_BUILDS}/boards/${board}/${ver}/pkgs/"
164151
done
165152
}
166153

@@ -196,13 +183,9 @@ get_sdk_binhost() {
196183
FLATCAR_DEV_BUILDS_SDK="${FLATCAR_DEV_BUILDS_SDK-${SETTING_BINPKG_SERVER_PROD}/sdk}"
197184
fi
198185
for ver in "$@"; do
199-
# Usually only crossdev needs to be fetched from /toolchain/ in the setup_board step.
200186
# The entry for /pkgs/ is there if something needs to be reinstalled in the SDK
201187
# but normally it is not needed because everything is already part of the tarball.
202-
# To install the crossdev Rust package, /toolchain-arm64/ is derived from /toolchain/
203-
# when necessary in install_cross_toolchain().
204188
if curl -Ifs -o /dev/null "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/pkgs/"; then
205-
echo "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/toolchain/"
206189
echo "${FLATCAR_DEV_BUILDS_SDK}/${arch}/${ver}/pkgs/"
207190
fi
208191
done

build_packages

-7
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,6 @@ DEFINE_boolean getbinpkg "${FLAGS_TRUE}" \
2424
"Download binary packages from remote repository."
2525
DEFINE_string getbinpkgver "" \
2626
"Use binary packages from a specific version."
27-
DEFINE_boolean toolchainpkgonly "${FLAGS_FALSE}" \
28-
"Use binary packages only for the board toolchain."
2927
DEFINE_boolean workon "${FLAGS_TRUE}" \
3028
"Automatically rebuild updated flatcar-workon packages."
3129
DEFINE_boolean fetchonly "${FLAGS_FALSE}" \
@@ -100,11 +98,6 @@ if [ "${FLAGS_usepkg}" -eq "${FLAGS_TRUE}" ]; then
10098
else
10199
UPDATE_ARGS+=( --nogetbinpkg )
102100
fi
103-
if [[ "${FLAGS_toolchainpkgonly}" -eq "${FLAGS_TRUE}" ]]; then
104-
UPDATE_ARGS+=( --toolchainpkgonly )
105-
else
106-
UPDATE_ARGS+=( --notoolchainpkgonly )
107-
fi
108101
if [[ -n "${FLAGS_getbinpkgver}" ]]; then
109102
UPDATE_ARGS+=( --getbinpkgver="${FLAGS_getbinpkgver}" )
110103
fi

setup_board

+1-5
Original file line numberDiff line numberDiff line change
@@ -31,8 +31,6 @@ DEFINE_string pkgdir "" \
3131
"Use binary packages from a custom directory instead of /build/[ARCH]/var/lib/portage/pkgs/."
3232
DEFINE_string binhost "" \
3333
"Use binary packages from a specific location instead of $FLATCAR_DEV_BUILDS/... "
34-
DEFINE_boolean toolchainpkgonly "${FLAGS_FALSE}" \
35-
"Use binary packages only for the board toolchain."
3634
DEFINE_boolean skip_toolchain_update "${FLAGS_FALSE}" \
3735
"Don't update toolchain automatically."
3836
DEFINE_boolean skip_chroot_upgrade "${FLAGS_FALSE}" \
@@ -136,13 +134,11 @@ EOF
136134
}
137135

138136
generate_binhost_list() {
139-
local t
140-
[[ "${FLAGS_toolchainpkgonly}" -eq "${FLAGS_TRUE}" ]] && t="-t"
141137
FLAGS_getbinpkgver="${FLAGS_getbinpkgver/current/${FLATCAR_VERSION_ID}}"
142138
FLAGS_getbinpkgver="${FLAGS_getbinpkgver/latest/${FLATCAR_VERSION_ID}}"
143139
FLAGS_getbinpkgver="${FLAGS_getbinpkgver/sdk/${FLATCAR_SDK_VERSION}}"
144140

145-
get_board_binhost $t "${BOARD}" ${FLAGS_getbinpkgver}
141+
get_board_binhost "${BOARD}" ${FLAGS_getbinpkgver}
146142
}
147143

148144
# Parse command line flags

0 commit comments

Comments
 (0)