From: Michael Olbrich <m.olbrich@pengutronix.de>
To: ptxdist@pengutronix.de
Subject: Re: [ptxdist] [PATCH 1/2] ptxdist: add licensecheck command
Date: Fri, 29 May 2020 07:59:14 +0200 [thread overview]
Message-ID: <20200529055914.GA31789@pengutronix.de> (raw)
In-Reply-To: <20200525092820.818-1-rhi@pengutronix.de>
On Mon, May 25, 2020 at 11:28:19AM +0200, Roland Hieber wrote:
> Make it possible to check all MD5 sums of license files by building all
> report stages without also building the license report (which produces
> tons of LaTeX output).
>
> Signed-off-by: Roland Hieber <rhi@pengutronix.de>
> ---
> bin/ptxdist | 6 ++++++
> doc/ref_parameter.inc | 3 +++
> rules/post/ptxd_make_license_report.make | 2 ++
> 3 files changed, 11 insertions(+)
>
> diff --git a/bin/ptxdist b/bin/ptxdist
> index 1e10c08d47f2..4a372ff6392c 100755
> --- a/bin/ptxdist
> +++ b/bin/ptxdist
> @@ -752,6 +752,7 @@ Misc:
> it is known by "make"
> printnext <var> assumes that the contents of <var> is another
> variable and print the contents of this variable
> + licensecheck check md5sums of license files for all packages
> list-packages print a list of all selected packages
> local-src <pkg> [<directory>] overwrite a package source with a locally provided
> directory containing the sourcecode.
> @@ -1776,6 +1777,11 @@ EOF
> ptxd_make_log "${images[@]}"
> exit
> ;;
> + licensecheck)
> + check_premake_compiler &&
> + ptxd_make_log report
> + exit
> + ;;
> list-packages)
> check_config &&
> check_deps || return
> diff --git a/doc/ref_parameter.inc b/doc/ref_parameter.inc
> index 74689b9d3cff..aaac8956f147 100644
> --- a/doc/ref_parameter.inc
> +++ b/doc/ref_parameter.inc
> @@ -234,6 +234,9 @@ Misc Actions
> contents of this variable. Shell variables are currently not checked here.
> All other rules for ``print`` apply.
>
> +``licensecheck``
> + For all selected packages, check the MD5 sums of license files.
> +
> ``list-packages``
> print a list of all selected packages. This list does not include the
> host and cross tools.
> diff --git a/rules/post/ptxd_make_license_report.make b/rules/post/ptxd_make_license_report.make
> index 47dc176668d0..1a13c3ad4964 100644
> --- a/rules/post/ptxd_make_license_report.make
> +++ b/rules/post/ptxd_make_license_report.make
> @@ -15,6 +15,8 @@ PTX_PACKAGES_TARGET := \
> $(PTX_PACKAGES_INSTALL) \
> $(EXTRA_PACKAGES)
>
> +report: $(addprefix $(STATEDIR)/,$(addsuffix .report,$(PTX_PACKAGES_SELECTED)))
Hmm, 'report' is a bit too generic for my taste. Just use 'licensecheck'
like the command. Also, You need to add it to PHONY.
mol
> +
> PHONY += license-report
>
> license-report: \
> --
> 2.26.2
>
>
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
>
--
Pengutronix e.K. | |
Steuerwalder Str. 21 | http://www.pengutronix.de/ |
31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
next prev parent reply other threads:[~2020-05-29 5:59 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-25 9:28 Roland Hieber
2020-05-25 9:28 ` [ptxdist] [PATCH 2/2] ptxd_lib_dgen: optimize ordering only when packages are being fully built Roland Hieber
2020-05-25 11:46 ` Roland Hieber
2020-05-25 12:47 ` Michael Olbrich
2020-05-25 20:22 ` [ptxdist] [PATCH v2 2/2] ptxd_lib_dgen: optimize ordering only when dependencies are needed Roland Hieber
2020-06-05 5:59 ` [ptxdist] [APPLIED] " Michael Olbrich
2020-05-29 5:59 ` Michael Olbrich [this message]
2020-05-29 9:13 ` [ptxdist] [PATCH v2 1/2] ptxdist: add licensecheck command Roland Hieber
2020-06-05 5:59 ` [ptxdist] [APPLIED] " Michael Olbrich
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20200529055914.GA31789@pengutronix.de \
--to=m.olbrich@pengutronix.de \
--cc=ptxdist@pengutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox