should config options be treated as case-sensitive?
- From
Robert P. J. Day <rpjday@crashcourse.ca>
- Date
- May 22, 2018, 07:57 UTC
- Message-ID
- <alpine.LFD.2.21.1805220353370.989@localhost.localdomain>
in my wanderings, more oddities, such as this:
$ grep -ir blankboundary * builtin/blame.c: if (!strcmp(var, "blame.blankboundary")) { Documentation/config.txt:blame.blankBoundary:: Documentation/blame-options.txt: be controlled via the `blame.blankboundary` config option. Documentation/RelNotes/2.15.1.txt: * Description of blame.{showroot,blankboundary,showemail,date} Documentation/RelNotes/2.16.0.txt: * Description of blame.{showroot,blankboundary,showemail,date} $
where you can see the single instance of "blankBoundary" in Doc/config.txt (with the upper case 'B'), while the rest have no such thing.
for fun, i checked another of blame's config settings, with similar results:
builtin/blame.c: if (!strcmp(var, "blame.showemail")) { Documentation/config.txt:blame.showEmail:: Documentation/RelNotes/2.15.1.txt: * Description of blame.{showroot,blankboundary,showemail,date} Documentation/RelNotes/2.5.0.txt: * "git blame" learned blame.showEmail configuration variable. Documentation/RelNotes/2.16.0.txt: * Description of blame.{showroot,blankboundary,showemail,date} Documentation/git-blame.txt: This can also be controlled via the `blame.showEmail` config t/t8002-blame.sh:test_expect_success 'setup showEmail tests' ' t/t8002-blame.sh:test_expect_success 'blame with showemail options' ' t/t8002-blame.sh:test_expect_success 'blame with showEmail config false' ' t/t8002-blame.sh: git config blame.showEmail false && t/t8002-blame.sh:test_expect_success 'blame with showEmail config true' ' t/t8002-blame.sh: git config blame.showEmail true &&
thoughts?
rday
p.s. i am not *trying* to be annoyingly pedantic, i am merely succeeding.