[libvirt] [PATCH] Skip the 'copyright-check' on -maint branches
Eric Blake
eblake at redhat.com
Mon Feb 10 21:57:57 UTC 2014
On 02/04/2014 06:51 AM, Daniel P. Berrange wrote:
> The maint branches will often have out of date copyright headers
> so we must skip the 'sc_copyright_check' rule there.
>
> Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
> ---
> cfg.mk | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> Perhaps there's a better way to detect -maint branches than the
> logic I tried.
>
Based on reading __git_ps1 (thanks, bash completion functions!) maybe
you want to try:
git describe --contains --all HEAD
which directly gives you the current branch name that you are based off
of, even when in the middle of a rebase on that branch.
> diff --git a/cfg.mk b/cfg.mk
> index 207dfeb..47c7798 100644
> --- a/cfg.mk
> +++ b/cfg.mk
> @@ -88,6 +88,17 @@ else
> distdir: sc_vulnerable_makefile_CVE-2012-3386.z
> endif
>
> +GIT_BRANCH := $(shell if test -d $(srcdir)/.git; then \
> + cd $(srcdir) && \
> + (git status -s -b | grep '\#\#' | \
> + sed -e 's/.*-maint/-maint/'); fi)
Even if we go with 'git status -s -b', you can shave a process by using
sed -n '/##/ s/.*-maint/-maint/p' and skipping the grep (anyone using
'grep | sed' is almost always wasting a process).
> +
> +# We fully expect -maint branches to have out of date
> +# copyright dates, so we must skip that check
> +ifeq ($(GIT_BRANCH),-maint)
> + local-checks-to-skip += sc_copyright_check
> +endif
> +
> # Files that should never cause syntax check failures.
> VC_LIST_ALWAYS_EXCLUDE_REGEX = \
> (^(HACKING|docs/(news\.html\.in|.*\.patch))|\.po)$$
>
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 604 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20140210/c06176fd/attachment-0001.sig>
More information about the libvir-list
mailing list