[Buildroot] [PATCH vRFC 2/3] package/depot-tools: new package
Yann E. MORIN
yann.morin.1998 at free.fr
Sun Jul 23 19:52:56 UTC 2023
Adam, All,
On 2023-07-20 16:36 -0700, Adam Duskett spake thusly:
> Chromium and Chromium OS use a package of scripts called
> depot_tools to manage checkouts and code reviews. This package
> also includes the gclient utility.
>
> gclient is a Python script to manage a workspace of modular dependencies that
> are each checked out independently from different subversion or git
> repositories. Features include:
>
> - Dependencies can be specified on a per-OS basis.
> - Dependencies can be specified relative to their parent dependency.
> - Variables can be used to abstract concepts.
> - Hooks can be specified to be run after a checkout.
> - .gclient and DEPS are Python scripts. You can hack in easily or add
> additional configuration data.
>
> .gclient file: It's the primary file. It is, in fact, a Python script. It
> specifies the following variables:
>
> - solutions: an array of dictionaries specifying the projects that will be
> fetched.
> - hooks: additional hooks to be run when this meta checkout is synced.
> - target_os: an optional array of (target) operating systems to fetch
> OS-specific dependencies for.
> - cache_dir: Primarily for bots, multiple working sets use a single git
> cache.
>
> gclient is necessary for checking out the flutter-engine source code, as the
> release tarballs provided on the flutter-engine github are in no state to
> compile. Google expects the use of gclient to download a source directory
> structure suitable to build the Flutter engine.
Thanks for this very detailed commit log.
> Signed-off-by: Adam Duskett <adam.duskett at amarulasolutions.com>
[--SNIP--]
> diff --git a/package/depot-tools/0001-Disable-ninjalog-upload.patch b/package/depot-tools/0001-Disable-ninjalog-upload.patch
> new file mode 100644
> index 0000000000..55fd78247f
> --- /dev/null
> +++ b/package/depot-tools/0001-Disable-ninjalog-upload.patch
> @@ -0,0 +1,35 @@
> +From eeea6ad620b88a81a4b05d49f32a375b4bbcef36 Mon Sep 17 00:00:00 2001
> +From: Adam Duskett <aduskett at gmail.com>
> +Date: Thu, 13 Jul 2023 15:50:25 -0700
> +Subject: [PATCH] Disable ninjalog upload
> +
> +Google doesn't need the ninja logs.
> +
> +Upstream: buildroot-specific.
> +Signed-off-by: Adam Duskett <aduskett at gmail.com>
> +---
> + autoninja | 4 ++--
> + 1 file changed, 2 insertions(+), 2 deletions(-)
> +
> +diff --git a/autoninja b/autoninja
> +index 06d45d1..354a2f1 100755
> +--- a/autoninja
> ++++ b/autoninja
> +@@ -30,12 +30,12 @@ if eval "$command"; then
> + fi
> +
> + # Collect ninjalog from googler.
> +- python3 "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
> ++ # python3 "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
Don't comment lines out, just remove them (with the associated comment
while at it).
> + exit
> + fi
> +
> + # Collect ninjalog from googler.
> +-python3 "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
> ++# python3 "$(dirname -- "$0")/ninjalog_uploader_wrapper.py" --cmd $command
Ditto.
[--SNIP--]
> diff --git a/package/depot-tools/depot-tools.mk b/package/depot-tools/depot-tools.mk
> new file mode 100644
> index 0000000000..906340016c
> --- /dev/null
> +++ b/package/depot-tools/depot-tools.mk
> @@ -0,0 +1,33 @@
> +################################################################################
> +#
> +# depot-tools
> +#
> +################################################################################
> +
> +DEPOT_TOOLS_VERSION = 4e87f5bfe244e903f712408ea68dc3c3a6fe2d00
> +DEPOT_TOOLS_SITE = https://chromium.googlesource.com/chromium/tools/depot_tools
> +DEPOT_TOOLS_SITE_METHOD = git
> +DEPOT_TOOLS_LICENSE = Chromium
The license in the LICENSE file looks suspiciously like it is in fact a
BSD-3-Clause:
https://spdx.org/licenses/BSD-3-Clause.html
> +DEPOT_TOOLS_LICENSE_FILES = LICENSE
> +DEPOT_TOOLS_CPE_ID_VENDOR = google
> +HOST_DEPOT_TOOLS_DEPENDENCIES = \
> + host-python3 \
> + host-python-httplib2 \
> + host-python-pyparsing \
> + host-python-six
> +
> +define HOST_DEPOT_TOOLS_INSTALL_CMDS
> + mkdir -p $(HOST_DIR)/share/depot_tools
> + cp -dprf $(@D)/* $(HOST_DIR)/share/depot_tools/
> +endef
> +
> +# Helper wrapper to make using gclient easier.
> +DEPOT_TOOLS_GCLIENT=\
Spaces on both sides of the equal sign.
> + PATH=$(BR_PATH):$(HOST_DIR)/share/depot_tools \
> + LD_LIBRARY_PATH=$(HOST_DIR)/lib \
Ick. Why do we need to set LD_LIBRARY_PATH?
> + PYTHONPATH=$(HOST_DIR)/lib/python$(PYTHON3_VERSION_MAJOR) \
PYTHONPATH=$(PYTHON3_PATH)
Why do we need to force it?
> + DEPOT_TOOLS_UPDATE=0 \
> + PYTHONDONTWRITEBYTECODE=1 \
Why do we need PYTHONDONTWRITEBYTECODE=1? I have it in my environment to
avoid littering repositories, indeed, but why do we need to force it for
gclient?
Regards,
Yann E. MORIN.
> + $(HOST_DIR)/share/depot_tools/gclient.py
> +
> +$(eval $(host-generic-package))
> --
> 2.41.0
>
> _______________________________________________
> buildroot mailing list
> buildroot at buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot
--
.-----------------.--------------------.------------------.--------------------.
| Yann E. MORIN | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software Designer | \ / CAMPAIGN | ___ |
| +33 561 099 427 `------------.-------: X AGAINST | \e/ There is no |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL | v conspiracy. |
'------------------------------^-------^------------------^--------------------'
More information about the buildroot
mailing list