Jeff King <peff@xxxxxxxx> writes: > On Sat, Jul 14, 2018 at 06:13:47PM +0000, brian m. carlson wrote: > >> On Tue, Jul 10, 2018 at 12:56:38PM -0400, Jeff King wrote: >> > On Tue, Jul 10, 2018 at 12:54:13PM -0400, Jeff King wrote: >> > >> > > Should we allow: >> > > >> > > [gpg "OpenPGP"] >> > > program = whatever >> > > >> > > given that we allow: >> > > >> > > [gpg] >> > > format = OpenPGP >> > > >> > > ? I think just using strcasecmp() here would be sufficient. But I wonder >> > > if it is a symptom of using the wrong tool (subsections) when we don't >> > > need it. >> > >> > I did just read the discussion in response to v1, where everybody told >> > you the opposite. ;) >> > >> > So I guess my question/points are more for brian and Junio. >> >> I'm okay with us forcing "openpgp". That seems sane enough for now, and >> if people scream loudly, we can loosen it. > > Well, I specifically meant "are you sure subsections like this are a > good idea". But it seems like people think so? I admit that I did not even consider that there may be better tool than using subsections to record this information. What are the possibilities you have in mind (if you have one)? > > If so, then I think the best route is just dictating that yes, > gpg.format is case-sensitive because it is referencing > gpg.<format>.program, which is itself case-sensitive (and "openpgp" is > the right spelling). > > -Peff
