[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: maint.mk sync
From: |
Jim Meyering |
Subject: |
Re: maint.mk sync |
Date: |
Fri, 17 Apr 2009 09:40:12 +0200 |
Simon Josefsson wrote:
> I'm beginning to think that incremental syncing of maint.mk will never
> finish, the sync targets are moving too quickly.
>
> I propose to just import the coreutils maint.mk into gnulib, and then
> move forward from there on. The patch below takes coreutils' maint.mk,
> and keeps the current coverage+gettext+indent target, and use that in
> gnulib. I've verified that importing it into at least one of my
> projects appears to work. This could break some packages that use
> maintainer-makefile. I think the people who are using that have deep
> understanding of these issues anyway, so they should be able to adapt.
>
> Ok to push?
I've just looked through again and see several rules that people may
find annoying: how many people care if you spell it "filesystem" or
"file system"? ;-) It's easy to put up with that if the code already
conforms, and even easier just to disable the check, but if a rule like
that is turned _on_ by default, it'll just turn people off.
Also, the diagnostic-checking rules are insufficiently general. At the
very least, they should take an optional cfg.mk-specified regexp to
identify the functions used to emit diagnostics.
One option is to maintain in maint.mk a list of the names of
less-generally-useful rules, and disable those by default.
Then we can leave most of the rules in the same file.
> I know there are some coreutils specific stuff below, but they don't
> seem to cause harm. The intention is that this can be fixed later.
>
> When this has been pushed into gnulib, I'll send a patch to coreutils
> that makes it use maintainer-makefile from gnulib. It should be quite
> short.
You're welcome to push that, as far as I'm concerned.
> After that, I can propose a patch that moves some of the coreutils
> specific stuff in maint.mk to coreutils cfg.mk.
No hurry on that part.
I suspect I'll want to maintain sync manually for a little
while before making the leap.
- maint.mk sync, Simon Josefsson, 2009/04/17
- Re: maint.mk sync,
Jim Meyering <=