Thread (19 messages) flat view 19 messages, 6 authors, 2016-06-15

Re: [PATCHv3 2/3] cvsimport: fix the parsing of uppercase config options

From: Jonathan Nieder <hidden>
Date: 2016-06-15 22:50:08

Possibly related (same subject, not in this thread)

(+cc: Jeff, config parsing wizard)

Jakub Narebski wrote:
Jonathan Nieder [off-list ref] writes:
quoted
I somewhat like this idea.  So let's build a full table, shall
we?  The embedded dashes are meant for the command-line options
rather than the config file.
[...]
Good idea, though I'd rather we avoid new convention for multi word
names separated with '-' (multi-word), but rather use camel case
(multiWord).
No disagreement there. :)
Currently we have only unfortunate exception of `add.ignore-errors',
Maybe we can teach the config file parser to ignore dashes in addition
to case (except in the names of [genus "species"] headings)?  That
would be an incompatible change for third-party tools, though, so
maybe "git config" would have to take an explicit --strip-dashes
flag to do it.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help