Re: [RFC PATCH] userdiff: ship built-in driver config file

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Jun 19, 2019 at 08:30:25AM +0200, Johannes Sixt wrote:

> >> Why place this file in .git? To have per-repository diff drivers, we can
> >> already specify them via 'git config'. This file should be installed in
> >> the system.
> > 
> > I think it _could_ actually just be part of the system /etc/gitconfig,
> > though it is kind of big, and Git has a tendency to parse the config
> > more than necessary. I wonder if would add a noticeable slowdown.
> 
> But /etc/gitconfig would be the wrong place, because it would not be
> updated when a new version ships with new patterns.

I was thinking it would be, but I guess there is a merging problem if
the admin has made their own changes.

> I would suggest to install the file as $prefix/share/git-core/userdiff
> although the name "userdiff" sounds like an accident. How about
> .../filetypes?

Does it need to be specific to userdiff or filetypes? Could this be a
generic fourth level of config: repo, user, system, builtin? We
effectively already have that, except the "builtin" ones are truly baked
into the binary, which means they are not visible. So right now you
cannot say "git config diff.tex.xfuncname" and get any useful
information, even though we clearly are going to respect it.

On the other hand, this would make "git config --list" quite a bit
longer. And any solution that involves putting it into the generic
config paths may suffer from the bloating/slowdown problem I mentioned.

But without that, I have to wonder what problem we are really solving.
Now it's baked into the binary. Later it will be baked into the
distribution, but we still don't want anybody to touch it because their
changes will be overwritten. I guess it's a little easier for somebody
to find .../share/git-core/userdiff and use it as a template than it is
to find the definitions in the source. But it's not exactly easy.

-Peff



[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]

  Powered by Linux