[Buildroot] [PATCH-NEXT v3 1/6] support/download/dl-wrapper: add concept of download post-processing
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Mon Oct 11 07:04:28 UTC 2021
Hello Christian,
Thanks for your work on this topic! I didn't review the patches in
details, but here are a couple of comments on the form:
- The initial series also had support for Cargo, I think we want to
retain that as the idea was to solve both Go and Cargo at the same
time, to make sure we have a solution that works for both.
- PATCH-NEXT doesn't make any sense right now, there is no "next"
branch.
- Your patch series lacks a cover letter with a description of the
changes between versions. So of your commits have a v1 -> v2
changelog, but we don't know what changed in v3.
Best regards,
Thomas
On Sun, 10 Oct 2021 16:46:50 -0700
Christian Stewart <christian at paral.in> wrote:
> From: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
>
> In order to support package managers such as Cargo (Rust) or Go, we
> want to run some custom logic after the main download, but before
> packing the tarball and checking the hash.
>
> To implement this, this commit introduces a concept of download
> post-processing: if -p <something> is passed to the dl-wrapper, then
> support/download/<something>-post-process will be called.
>
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni at bootlin.com>
> Signed-off-by: Christian Stewart <christian at paral.in>
> ---
> support/download/dl-wrapper | 9 ++++++++-
> 1 file changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/support/download/dl-wrapper b/support/download/dl-wrapper
> index 3315bd410e..2d74554213 100755
> --- a/support/download/dl-wrapper
> +++ b/support/download/dl-wrapper
> @@ -25,7 +25,7 @@ main() {
> local -a uris
>
> # Parse our options; anything after '--' is for the backend
> - while getopts ":c:d:D:o:n:N:H:rf:u:q" OPT; do
> + while getopts ":c:d:D:o:n:N:H:rf:u:qp:" OPT; do
> case "${OPT}" in
> c) cset="${OPTARG}";;
> d) dl_dir="${OPTARG}";;
> @@ -37,6 +37,7 @@ main() {
> r) recurse="-r";;
> f) filename="${OPTARG}";;
> u) uris+=( "${OPTARG}" );;
> + p) post_process="${OPTARG}";;
> q) quiet="-q";;
> :) error "option '%s' expects a mandatory argument\n" "${OPTARG}";;
> \?) error "unknown option '%s'\n" "${OPTARG}";;
> @@ -135,6 +136,12 @@ main() {
> continue
> fi
>
> + if [ -n "${post_process}" ] ; then
> + ${OLDPWD}/support/download/${post_process}-post-process \
> + -o "${tmpf}" \
> + -n "${raw_base_name}"
> + fi
> +
> # cd back to free the temp-dir, so we can remove it later
> cd "${OLDPWD}"
>
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
More information about the buildroot
mailing list