From 365faa110fa0decfba7d21372caab5e58d8fb1e7 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Fri, 5 Jul 2019 16:32:38 +0100 Subject: [PATCH 01/14] buildsystem: support absolute paths in PKG_PATCH_DIRS --- scripts/unpack | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/scripts/unpack b/scripts/unpack index d3aa480329..673fc4f08f 100755 --- a/scripts/unpack +++ b/scripts/unpack @@ -122,9 +122,14 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ]; then PATCH_DIRS_PRJ="" if [ -n "${PKG_PATCH_DIRS}" ]; then for patch_dir in ${PKG_PATCH_DIRS}; do - [ -d ${PKG_DIR}/patches/${patch_dir} ] && PATCH_DIRS_PKG+=" ${PKG_DIR}/patches/${patch_dir}/*.patch" - [ -d ${PROJECT_DIR}/${PROJECT}/patches/${PKG_NAME}/${patch_dir} ] && PATCH_DIRS_PRJ+=" ${PROJECT_DIR}/${PROJECT}/patches/${PKG_NAME}/${patch_dir}/*.patch" - [ -d ${PROJECT_DIR}/${PROJECT}/devices/${DEVICE}/patches/${PKG_NAME}/${patch_dir} ] && PATCH_DIRS_PRJ+=" ${PROJECT_DIR}/${PROJECT}/devices/${DEVICE}/patches/${PKG_NAME}/${patch_dir}/*.patch" + if [[ ${patch_dir} =~ ^/ ]]; then + [ -f ${patch_dir} ] && PATCH_DIRS_PKG+=" ${patch_dir}" + [ -d ${patch_dir} ] && PATCH_DIRS_PKG+=" ${patch_dir}/*.patch" + else + [ -d ${PKG_DIR}/patches/${patch_dir} ] && PATCH_DIRS_PKG+=" ${PKG_DIR}/patches/${patch_dir}/*.patch" + [ -d ${PROJECT_DIR}/${PROJECT}/patches/${PKG_NAME}/${patch_dir} ] && PATCH_DIRS_PRJ+=" ${PROJECT_DIR}/${PROJECT}/patches/${PKG_NAME}/${patch_dir}/*.patch" + [ -d ${PROJECT_DIR}/${PROJECT}/devices/${DEVICE}/patches/${PKG_NAME}/${patch_dir} ] && PATCH_DIRS_PRJ+=" ${PROJECT_DIR}/${PROJECT}/devices/${DEVICE}/patches/${PKG_NAME}/${patch_dir}/*.patch" + fi done fi @@ -165,7 +170,7 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ]; then elif [[ "${thisdir}" =~ ^${PROJECT_DIR}/.* ]]; then PATCH_DESC="(project - $(basename "${thisdir}"))" else - PATCH_DESC="(unknown - $(basename "${thisdir}"))" + PATCH_DESC="(absolute - ${i})" fi fi From adbe82298a1ad8d2b5466d9df980d46de4adddc1 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Fri, 5 Jul 2019 21:17:41 +0100 Subject: [PATCH 02/14] scripts/unpack: avoid sub-process during custom unpack() --- scripts/unpack | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/scripts/unpack b/scripts/unpack index 673fc4f08f..1c5876c862 100755 --- a/scripts/unpack +++ b/scripts/unpack @@ -66,10 +66,10 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ]; then pkg_call_exists pre_unpack && pkg_call pre_unpack if pkg_call_exists unpack; then - ( - PKG_BUILD="${PKG_UNPACK_DIR}/${PKG_NAME}-${PKG_VERSION}" - pkg_call unpack - ) + PKG_BUILD_ORIG="${PKG_BUILD}" + PKG_BUILD="${PKG_UNPACK_DIR}/${PKG_NAME}-${PKG_VERSION}" + pkg_call unpack + PKG_BUILD="${PKG_BUILD_ORIG}" else if [ -n "${PKG_URL}" ]; then ${SCRIPTS}/extract "${PKG_NAME}" "${PKG_UNPACK_DIR}" From 02601a25003c743fd7cf7c7dea3f137f25948b3d Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Fri, 5 Jul 2019 23:18:41 +0100 Subject: [PATCH 03/14] crazycat & dvb-latest: fix misconfiguration --- packages/linux-driver-addons/dvb/crazycat/package.mk | 2 +- packages/linux-driver-addons/dvb/dvb-latest/package.mk | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/linux-driver-addons/dvb/crazycat/package.mk b/packages/linux-driver-addons/dvb/crazycat/package.mk index 8889791b6e..ee9ec8253d 100644 --- a/packages/linux-driver-addons/dvb/crazycat/package.mk +++ b/packages/linux-driver-addons/dvb/crazycat/package.mk @@ -8,7 +8,7 @@ PKG_LICENSE="GPL" PKG_SITE="https://bitbucket.org/CrazyCat/media_build" PKG_URL="https://bitbucket.org/CrazyCat/media_build/get/$PKG_VERSION.tar.gz" PKG_DEPENDS_TARGET="toolchain linux media_tree_cc" -PKG_NEED_UNPACK="$LINUX_DEPENDS media_tree_cc" +PKG_NEED_UNPACK="$LINUX_DEPENDS $(get_pkg_directory media_tree_cc)" PKG_SECTION="driver.dvb" PKG_LONGDESC="DVB driver for TBS cards with CrazyCats additions" diff --git a/packages/linux-driver-addons/dvb/dvb-latest/package.mk b/packages/linux-driver-addons/dvb/dvb-latest/package.mk index d8b8b8997f..9685def240 100644 --- a/packages/linux-driver-addons/dvb/dvb-latest/package.mk +++ b/packages/linux-driver-addons/dvb/dvb-latest/package.mk @@ -8,7 +8,7 @@ PKG_LICENSE="GPL" PKG_SITE="http://git.linuxtv.org/media_build.git" PKG_URL="https://git.linuxtv.org/media_build.git/snapshot/${PKG_VERSION}.tar.gz" PKG_DEPENDS_TARGET="toolchain linux media_tree" -PKG_NEED_UNPACK="$LINUX_DEPENDS media_tree" +PKG_NEED_UNPACK="$LINUX_DEPENDS $(get_pkg_directory media_tree)" PKG_SECTION="driver.dvb" PKG_LONGDESC="DVB drivers from the latest kernel (media_build)" From 341aa35c3c4a77420b3fa4fe85cfcc76a26b53c4 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sat, 6 Jul 2019 08:53:38 +0100 Subject: [PATCH 04/14] buildsystem: perform unpack when custom function provided --- scripts/unpack | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/scripts/unpack b/scripts/unpack index 1c5876c862..81668f0115 100755 --- a/scripts/unpack +++ b/scripts/unpack @@ -55,7 +55,7 @@ pkg_lock_status "ACTIVE" "${PKG_NAME}" "unpack" ${SCRIPTS}/get "${PKG_NAME}" -if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ]; then +if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ] || pkg_call_exists unpack; then build_msg "CLR_UNPACK" "UNPACK" "${PKG_NAME}" "indent" # unpack into a unique location as unpacking into a single ${BUILD} directory is not thread-safe From 79b31b45fd8e7dad47fcee106489d4cef5263f3d Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sat, 6 Jul 2019 08:23:03 +0100 Subject: [PATCH 05/14] package inheritance: cleanup redundant files --- .../chrome-depends/chrome-libXcomposite/sources/trigger_unpack | 0 .../chrome-depends/chrome-libXdamage/sources/trigger_unpack | 0 .../chrome-depends/chrome-libXfixes/sources/trigger_unpack | 0 .../chrome-depends/chrome-libXi/sources/trigger_unpack | 0 .../chrome-depends/chrome-libXrender/sources/trigger_unpack | 0 .../chrome-depends/chrome-libXtst/sources/trigger_unpack | 0 .../chrome-depends/chrome-libxcb/sources/trigger_unpack | 0 .../dotnet-runtime-depends/curl3/sources/trigger_unpack | 0 8 files changed, 0 insertions(+), 0 deletions(-) delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXcomposite/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXdamage/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXfixes/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXi/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXrender/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libXtst/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/chrome-depends/chrome-libxcb/sources/trigger_unpack delete mode 100644 packages/addons/addon-depends/dotnet-runtime-depends/curl3/sources/trigger_unpack diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXcomposite/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXcomposite/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXdamage/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXdamage/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXfixes/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXfixes/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXi/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXi/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXrender/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXrender/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libXtst/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libXtst/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/chrome-depends/chrome-libxcb/sources/trigger_unpack b/packages/addons/addon-depends/chrome-depends/chrome-libxcb/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 diff --git a/packages/addons/addon-depends/dotnet-runtime-depends/curl3/sources/trigger_unpack b/packages/addons/addon-depends/dotnet-runtime-depends/curl3/sources/trigger_unpack deleted file mode 100644 index e69de29bb2..0000000000 From e0dc24ea42a8c80446bc6bb32da786cc37b58f0c Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sat, 6 Jul 2019 09:19:38 +0100 Subject: [PATCH 06/14] scripts/unpack: ensure PKG_BUILD is unpack dir for all unpack functions The unpack folder will be a temporary (isolated) unpack folder for pre_unpack() and unpack(), but by the time of post_unpack() the folder will have been moved into the main ${BUILD} "public" folder. This change ensures that PKG_BUILD always references the correct folder at all times. --- scripts/unpack | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-) diff --git a/scripts/unpack b/scripts/unpack index 81668f0115..3af850a6de 100755 --- a/scripts/unpack +++ b/scripts/unpack @@ -63,13 +63,15 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ] || pkg_call_exists rm -rf "${PKG_UNPACK_DIR}" mkdir -p "${PKG_UNPACK_DIR}" + # Save PKG_BUILD and point at our private unpack directory so that any + # modifications to the content of ${PKG_BUILD} will be "safe". + PKG_BUILD_ORIG="${PKG_BUILD}" + PKG_BUILD="${PKG_UNPACK_DIR}/${PKG_NAME}-${PKG_VERSION}" + pkg_call_exists pre_unpack && pkg_call pre_unpack if pkg_call_exists unpack; then - PKG_BUILD_ORIG="${PKG_BUILD}" - PKG_BUILD="${PKG_UNPACK_DIR}/${PKG_NAME}-${PKG_VERSION}" pkg_call unpack - PKG_BUILD="${PKG_BUILD_ORIG}" else if [ -n "${PKG_URL}" ]; then ${SCRIPTS}/extract "${PKG_NAME}" "${PKG_UNPACK_DIR}" @@ -88,8 +90,6 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ] || pkg_call_exists # fallback mv "${BUILD}"/${PKG_SOURCE_DIR} "${PKG_UNPACK_DIR}/.intermediate" fi - elif [ -d "${PKG_BUILD}"* ]; then - mv "${PKG_BUILD}"* "${PKG_UNPACK_DIR}/.intermediate" fi fi @@ -99,8 +99,11 @@ if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ] || pkg_call_exists cp -PRf "${PKG_DIR}/sources/"* "${PKG_UNPACK_DIR}/.intermediate" fi + # Add a tag to the unpacked folder before transferring into the shared build folder echo "INFO_PKG_NAME=\"${PKG_NAME}\"" > "${PKG_UNPACK_DIR}/.intermediate/.libreelec-package" + # Restore original PKG_BUILD, and transfer the unpacked folder + PKG_BUILD="${PKG_BUILD_ORIG}" rm -fr "${PKG_BUILD}" mv "${PKG_UNPACK_DIR}/.intermediate" "${PKG_BUILD}" From cff7ea1ab8932a11b435295307a8eff5bd29a142 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sat, 6 Jul 2019 23:18:47 +0100 Subject: [PATCH 07/14] scripts/unpack: always download missing tarballs - fixes PKG_DEPENDS_UNPACK --- scripts/unpack | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/scripts/unpack b/scripts/unpack index 3af850a6de..c88bf0769d 100755 --- a/scripts/unpack +++ b/scripts/unpack @@ -17,6 +17,8 @@ PARENT_PKG="${2:-${PKG_NAME}}" pkg_lock "${PKG_NAME}" "unpack" "${PARENT_PKG}" +${SCRIPTS}/get "${PKG_NAME}" + if [ -n "${PKG_DEPENDS_UNPACK}" ]; then for p in ${PKG_DEPENDS_UNPACK}; do ${SCRIPTS}/unpack "${p}" "${PARENT_PKG}" @@ -53,8 +55,6 @@ fi pkg_lock_status "ACTIVE" "${PKG_NAME}" "unpack" -${SCRIPTS}/get "${PKG_NAME}" - if [ -d "${SOURCES}/${PKG_NAME}" -o -d "${PKG_DIR}/sources" ] || pkg_call_exists unpack; then build_msg "CLR_UNPACK" "UNPACK" "${PKG_NAME}" "indent" From 7d84fbe6fa11a1f055081b574530052411841a05 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sat, 6 Jul 2019 23:26:36 +0100 Subject: [PATCH 08/14] buildsystem: add GETPKG history event --- scripts/get_archive | 2 ++ scripts/get_file | 2 ++ scripts/get_git | 2 ++ tools/mtstats.py | 4 ++-- 4 files changed, 8 insertions(+), 2 deletions(-) diff --git a/scripts/get_archive b/scripts/get_archive index eb912015a5..85ec6c545f 100755 --- a/scripts/get_archive +++ b/scripts/get_archive @@ -18,6 +18,8 @@ _get_file_already_downloaded && exit 0 # At this point, we need to download something... build_msg "CLR_GET" "GET" "${1} (archive)" "indent" +pkg_lock_status "GETPKG" "${PKG_NAME}" "unpack" "downloading package..." + PACKAGE_MIRROR="${DISTRO_MIRROR}/${PKG_NAME}/${PKG_SOURCE_NAME}" [ "${VERBOSE}" != "yes" ] && WGET_OPT=-q WGET_CMD="wget --output-file=- --timeout=30 --tries=3 --passive-ftp --no-check-certificate -c ${WGET_OPT} --progress=bar:force --show-progress -O ${PACKAGE}" diff --git a/scripts/get_file b/scripts/get_file index 2a34dd545b..02c0021df4 100755 --- a/scripts/get_file +++ b/scripts/get_file @@ -2,3 +2,5 @@ # Copyright (C) 2018-present Team LibreELEC (https://libreelec.tv) build_msg "CLR_GET" "GET" "${1} (file)" "indent" + +pkg_lock_status "GETPKG" "${PKG_NAME}" "unpack" "processing package file..." diff --git a/scripts/get_git b/scripts/get_git index 68bf92023a..dd8486bf34 100755 --- a/scripts/get_git +++ b/scripts/get_git @@ -44,6 +44,8 @@ _get_repo_already_downloaded && exit 0 # At this point, we need to download something... build_msg "CLR_GET" "GET" "${1} (git)" "indent" +pkg_lock_status "GETPKG" "${PKG_NAME}" "unpack" "processing package repository..." + rm -f "${STAMP_URL}" "${STAMP_SHA}" GIT_CLONE_PARAMS="" diff --git a/tools/mtstats.py b/tools/mtstats.py index f9fd9d47c4..8fdc4ff9b2 100755 --- a/tools/mtstats.py +++ b/tools/mtstats.py @@ -63,7 +63,7 @@ for event in events: "start": 0.0, "total": 0.0}) - if event.status in ["ACTIVE", "MUTEX"]: + if event.status in ["ACTIVE", "MUTEX", "GETPKG"]: if slot["active"] == False: active += 1 concurrent = concurrency.get(active, {"start": 0.0, "total": 0.0}) @@ -98,7 +98,7 @@ print("Total Build Time: %s\n" % secs_to_hms(elapsed, blankzero=False)) print("Peak concurrency: %d out of %d slots\n" % (peak, len(slots))) -print("Slot usage (time in ACTIVE state): | Concurrency breakdown:\n") +print("Slot usage (time in an \"active\" state): | Concurrency breakdown:\n") print("#Rank Slot Usage ( Pct ) # of Slots Usage ( Pct )") lines = [] From ff8c463b31690488333008ce77d2c8c1ed57779d Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sun, 7 Jul 2019 00:11:44 +0100 Subject: [PATCH 09/14] buildsystem: bootstrapping parallel must not use locks --- config/multithread | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/multithread b/config/multithread index ca389cddbe..18b24f959d 100644 --- a/config/multithread +++ b/config/multithread @@ -150,7 +150,7 @@ start_multithread_build() { [ "${THREADCOUNT}" = "0" ] && THREADCOUNT=1 # Bootstrap GNU parallel - $SCRIPTS/build parallel:host 2>&1 || die "Unable to bootstrap parallel package" + MTWITHLOCKS=no $SCRIPTS/build parallel:host 2>&1 || die "Unable to bootstrap parallel package" # if number of detected slots is 1 then don't bother using inter-process locks as this is a sequential build [ $(seq 1 32 | ${TOOLCHAIN}/bin/parallel --plain --no-notice --max-procs ${THREADCOUNT} echo {%} | sort -n | tail -1) -eq 1 ] && singlethread=yes || singlethread=no From 8380877cf571710d741d7a800e6a481665f44c01 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sun, 7 Jul 2019 22:57:37 +0100 Subject: [PATCH 10/14] buildsystem: support 6 digit process ids --- config/functions | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/functions b/config/functions index 7270b16d56..fbf3b7d811 100644 --- a/config/functions +++ b/config/functions @@ -1293,7 +1293,7 @@ pkg_lock_status() { ( flock --exclusive 94 - printf -v line "%s: <%05d> [%02d/%0*d] %-7s %-7s %-35s" \ + printf -v line "%s: <%06d> [%02d/%0*d] %-7s %-7s %-35s" \ "$(date +%Y-%m-%d\ %H:%M:%S.%N)" $$ ${this_job} ${#MTMAXJOBS} ${PARALLEL_SEQ:-0} "${status}" "${task}" "${pkg}" [ -n "${msg}" ] && line+=" (${msg})" From fc97f456dc2542b3d491bb3bcbd603becdc3e273 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Sun, 7 Jul 2019 23:37:31 +0100 Subject: [PATCH 11/14] buildsystem: HACK: fix current status after initramfs initramfs is currently built from within linux:target which is a problem yet to be solved as it should be built as separately scheduled processes just like everything else. As we build initramfs from within linux:target, once initramfs is finished it emits an "UNLOCK" event which is the last event for the linux:target package until it too finishes, perhaps 5 or 10 minutes later. During this 5-10 minute period the dashboard gives the impression that initramfs is "stuck" in an UNLOCKed state when in fact processing continues within linux:target, which is now the "ACTIVE" package for the process slot. Until the initramfs scheduling problem is resolved, emit an extra "ACTIVE" event upon returning to linux:target correcting the dashboard (and also tools/mtstats.py elapsed time accumulation). --- packages/linux/package.mk | 1 + 1 file changed, 1 insertion(+) diff --git a/packages/linux/package.mk b/packages/linux/package.mk index 5cccf973df..e42a17e8f9 100644 --- a/packages/linux/package.mk +++ b/packages/linux/package.mk @@ -189,6 +189,7 @@ make_target() { rm -rf $BUILD/initramfs $SCRIPTS/install initramfs ) + pkg_lock_status "ACTIVE" "linux:target" "build" # arm64 target does not support creating uImage. # Build Image first, then wrap it using u-boot's mkimage. From eaf81e7277e9585dee784ca19ed73fb0ca45a9e0 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Tue, 9 Jul 2019 04:40:04 +0100 Subject: [PATCH 12/14] buildsystem: fix and relocate comment --- config/functions | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/config/functions b/config/functions index fbf3b7d811..cced5cdb50 100644 --- a/config/functions +++ b/config/functions @@ -1239,6 +1239,8 @@ enable_service() { ### MULTI-THREADED FUNCTION HELPERS ### +# Test MTWITHLOCKS so that these functions are a no-op during non-multithreaded builds. + # Prevent concurrent modifications to a package (unpack) or # package:target (install/build). # @@ -1362,7 +1364,6 @@ update_dashboard() { # Thread concurrency helpers to avoid concurrency issues with some code, # eg. when Python installs directly into $TOOLCHAIN. -# Test MTJOBID so that these functions are a no-op during non-multithreaded builds. acquire_exclusive_lock() { [ "${MTWITHLOCKS}" != "yes" ] && return 0 From 82f70d28c9970439b3516b727787a8b2ef4b9f01 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Tue, 9 Jul 2019 10:37:21 +0100 Subject: [PATCH 13/14] buildsystem: no need to source package for PKG_NAME --- config/functions | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/config/functions b/config/functions index cced5cdb50..702346190f 100644 --- a/config/functions +++ b/config/functions @@ -714,7 +714,7 @@ get_pkg_variable() { # get package's build dir get_build_dir() { - local _PKG_NAME="$(get_pkg_variable "$1" PKG_NAME)" _PKG_VERSION="$(get_pkg_version "$1")" + local _PKG_NAME="${1%:*}" _PKG_VERSION="$(get_pkg_version "$1")" if [ -n "$_PKG_NAME" -a -n "$_PKG_VERSION" ]; then echo $BUILD/${_PKG_NAME}-${_PKG_VERSION} fi From 8fc217337c1dc13f222fd6bed887434bcb202af3 Mon Sep 17 00:00:00 2001 From: MilhouseVH Date: Wed, 10 Jul 2019 05:23:39 +0100 Subject: [PATCH 14/14] buildsystem: include BUILD_SUFFIX in dashboard --- config/functions | 1 + 1 file changed, 1 insertion(+) diff --git a/config/functions b/config/functions index 702346190f..376d50dc01 100644 --- a/config/functions +++ b/config/functions @@ -1336,6 +1336,7 @@ update_dashboard() { projdevarch="${PROJECT}/" [ -n "${DEVICE}" ] && projdevarch+="${DEVICE}/" projdevarch+="${TARGET_ARCH}" + [ -n "${BUILD_SUFFIX}" ] && projdevarch+=", ${BUILD_SUFFIX}" TZ=UTC0 printf -v elapsed "%(%H:%M:%S)T" $(($(date +%s) - MTBUILDSTART)) printf -v preamble "%s Dashboard (%s) - %d of %d jobs completed, %s elapsed" "${DISTRONAME}" "${projdevarch}" $((num + 1)) ${MTMAXJOBS} "${elapsed}" printf -v preamble "%b%-105s %s" "\e[2J\e[0;0H" "${preamble//\//\\/}" "$(date "+%Y-%m-%d %H:%M:%S")"