[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: update-copyright local hook
From: |
Jim Meyering |
Subject: |
Re: update-copyright local hook |
Date: |
Fri, 14 Aug 2009 21:11:10 +0200 |
Eric Blake wrote:
> Jim Meyering <jim <at> meyering.net> writes:
>> +# If you have an additional project-specific rule,
>> +# define it in cfg.mk and set this variable to its name.
>> +update-copyright-local ?=
>> +
>> .PHONY: update-copyright
>> -update-copyright:
>> +update-copyright: $(update-copyright-local)
>
> I'm wondering if making this a user-overridable variable is a bit overkill.
> After all, GNU make dependencies are additive (as long as only one rule
> exists), such that:
>
> maint.mk
> ========
> foo:
> rule
>
> cfg.mk
> ======
> foo: prereq
>
> works to run prereq before rule during 'make foo'.
>
>> # Run this rule once per year (usually early in January)
>> # to update all FSF copyright year lists in your project.
>> update-copyright-exclude-regexp ?= (^|/)COPYING$$
> ...
>> grep -l -w Copyright $$($(VC_LIST_EXCEPT)) \
>> | grep -v -E '$(update-copyright-exclude-regexp)' \
>
> Hmm. VC_LIST_EXCEPT excludes ChangeLog by default, but this is one case where
> we WANT ChangeLog's copyright updated along with everything else. Also, I
> think we should be exempting all forms of COPYING (such as COPYING.LIB). And
> the use of .x-update-copyright seems cleaner than trying to write a sane
> regexp
> replacement in update-copyright-exclude-regexp (after all, I just got that
> wrong in autoconf[1]). Would it be better to just do the following patch?
>
> [1]
> http://thread.gmane.org/gmane.comp.sysutils.autoconf.patches/6772/focus=6773
Yes. Good ideas, all.
Thanks.
> From: Eric Blake <address@hidden>
> Date: Fri, 14 Aug 2009 09:40:53 -0600
> Subject: [PATCH] maint.mk: simplify update-copyright rule
>
> * top/maint.mk (update-copyright-local): Delete, and document how
> to do it in cfg.mk instead.
> (update-copyright-exclude-regexp): Delete, and document how to do
> it in .x-update-copyright instead.
> (update-copyright): Simplify, thanks to VC_LIST_EXCEPT. Don't
> exclude ChangeLog.
- update-copyright local hook, Joel E. Denny, 2009/08/06
- Re: update-copyright local hook, Jim Meyering, 2009/08/06
- Re: update-copyright local hook, Joel E. Denny, 2009/08/06
- Re: update-copyright local hook, Eric Blake, 2009/08/14
- Re: update-copyright local hook,
Jim Meyering <=
- Re: update-copyright local hook, Eric Blake, 2009/08/14
- Re: update-copyright local hook, Jim Meyering, 2009/08/15
- Re: update-copyright local hook, Eric Blake, 2009/08/15
- Re: update-copyright local hook, Jim Meyering, 2009/08/15
- Re: update-copyright local hook, Eric Blake, 2009/08/15
- Re: update-copyright local hook, Joel E. Denny, 2009/08/17
- Re: update-copyright local hook, Jim Meyering, 2009/08/18