From: Junio C Hamano <gitster@pobox.com>
To: Michael Haggerty <mhagger@alum.mit.edu>
Cc: git discussion list <git@vger.kernel.org>
Subject: Re: RFC GSoC idea: new "git config" features
Date: Fri, 28 Feb 2014 12:00:48 -0800 [thread overview]
Message-ID: <xmqqwqgft3bj.fsf@gitster.dls.corp.google.com> (raw)
In-Reply-To: <53108650.2020708@alum.mit.edu> (Michael Haggerty's message of "Fri, 28 Feb 2014 13:51:28 +0100")
Michael Haggerty <mhagger@alum.mit.edu> writes:
> I just wrote up another double-idea that has been stewing in my head for
> a while:
>
> * Allow configuration values to be unset via a config file
> * Fix "git config --unset" to clean up detritus from sections that are
> left empty.
The former is *way* too large for a GSoC project. Most
configuration variables are meant to be read sequencially and affect
in-core variables directly, like
/* file-scope global */
static int frotz = -1; /* unset */
static int parse_config_frotz(const char *key, const char *value, void *cb)
{
if (!strcmp(key, "core.frotz"))
frotz = git_config_int(value);
return 0;
}
... and somewhere ...
git_config(parse_config_frotz, NULL);
The config parsers are distributed and there is no single registry
that knows how in-core variables owned by each subsystem represent
an "unset" value. In the above example, -1 is such a sentinel
value, but in some other contexts, the subsystem may choose to use
INT_MAX. The only way to allow "resetting to previous" is to
(1) come up with a way to pass "this key is being reset to
'unspecified'" to existing git_config() callback functions
(like parse_config_frotz() in the above illustration), which
may or may not involve changing the function signature of the
callbacks;
(2) go through all the git_config() callback functions and make
them understand the new "reset to 'unspecified'" convention.
which may not sound too bad at the first glance (especially, the
first one is almost trivial).
But the side effects these callbacks may cause are not limited to
setting a simple scaler variable (like 'frotz' in the illustration)
but would include things that are hard to undo once done
(e.g. calling a set-up function with a lot of side effects).
The latter, on the other hand, should be a change that is of a
fairly limited scope, and would be a good fit for a GSoC project
(incidentally, it has been one of the items on my leftover-bits list
http://git-blame.blogspot.com/p/leftover-bits.html for quite some
time).
Thanks.
next prev parent reply other threads:[~2014-02-28 20:01 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-02-28 12:51 RFC GSoC idea: new "git config" features Michael Haggerty
2014-02-28 20:00 ` Junio C Hamano [this message]
2014-03-01 0:19 ` Michael Haggerty
2014-03-01 7:52 ` Jeff King
2014-03-01 11:01 ` Matthieu Moy
2014-03-14 4:43 ` Jeff King
2014-03-14 21:00 ` Junio C Hamano
2014-03-03 20:07 ` Junio C Hamano
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
List information: http://vger.kernel.org/majordomo-info.html
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=xmqqwqgft3bj.fsf@gitster.dls.corp.google.com \
--to=gitster@pobox.com \
--cc=git@vger.kernel.org \
--cc=mhagger@alum.mit.edu \
/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.
Code repositories for project(s) associated with this public inbox
https://80x24.org/mirrors/git.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).