* StGit: stgit mail configuration options
@ 2008-11-06 10:21 David Kågedal
2008-11-10 10:19 ` Karl Hasselström
0 siblings, 1 reply; 2+ messages in thread
From: David Kågedal @ 2008-11-06 10:21 UTC (permalink / raw)
To: Catalin Marinas; +Cc: Git Mailing List
The "stg mail" command uses a couple of configuration options that are
completely undocumented, and don't fit well together.
"stgit.autobcc" seems to give a default value for the --bcc option.
"stgit.mail.prefix seems to give a default value for the --prefix
option.
Shouldn't these both be stgit.* or stgit.mail.*? And shouldn't they be
documented? Or maybe removed?
And I propse that the autobcc option should be a boolean which means
that user.email should be used.
--
David Kågedal
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: StGit: stgit mail configuration options
2008-11-06 10:21 StGit: stgit mail configuration options David Kågedal
@ 2008-11-10 10:19 ` Karl Hasselström
0 siblings, 0 replies; 2+ messages in thread
From: Karl Hasselström @ 2008-11-10 10:19 UTC (permalink / raw)
To: David Kågedal; +Cc: Catalin Marinas, Git Mailing List
On 2008-11-06 11:21:20 +0100, David Kågedal wrote:
> The "stg mail" command uses a couple of configuration options that
> are completely undocumented, and don't fit well together.
I agree that they definitely need documenting.
As for renaming them and/or changing their types, that's probably a
good idea too. And once they are documented, we could e.g. rename and
only mention the new name in the docs, but keep the old name as well
for backward compatibility.
One interesting twist would be to fix StGit's config parser so that it
can automatically generate a man page listing all the config options
it recognizes. It needs updating anyway for other reasons (I believe
it still calls git-config several times instead of just once), and
this could be done at the same time.
--
Karl Hasselström, kha@treskal.com
www.treskal.com/kalle
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2008-11-10 10:22 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-11-06 10:21 StGit: stgit mail configuration options David Kågedal
2008-11-10 10:19 ` Karl Hasselström
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox