mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
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

  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