threads / patch / 12731

patchShorten and simplify sendemail.alias* configuration variables.

Subject: [PATCH] Shorten and simplify sendemail.alias* configuration variables.

## tl;dr

3 messages between Mar 17, 2008 and Mar 19, 2008. Diffs are folded; open one to read it.

replies: 2people: 2as markdown or json

Xavier Maillard· Mar 17, 2008, 22:53 UTC · lore

I find it more consistent and less error prone to have aliasfile instead of aliasesfile and aliastype instead of aliasfiletype.

Documentation is also updated to reflect the changes.
Signed-off-by: Xavier Maillard <xma@gnu.org>
---
 Documentation/git-send-email.txt |    8 ++++----
 git-send-email.perl              |    4 ++--
 2 files changed, 6 insertions(+), 6 deletions(-)
Show changes to 2 files +6 −6

Documentation/git-send-email.txt, git-send-email.perl

diff --git a/Documentation/git-send-email.txt b/Documentation/git-send-email.txt
index 336d797..33fd589 100644
--- a/Documentation/git-send-email.txt
+++ b/Documentation/git-send-email.txt
@@ -192,12 +192,12 @@ sendemail.identity::
 	identities and to hoist sensitive authentication information
 	out of the repository and into the global configuration file.
 
-sendemail.aliasesfile::
+sendemail.aliasfile::
 	To avoid typing long email addresses, point this to one or more
-	email aliases files.  You must also supply 'sendemail.aliasfiletype'.
+	email aliases files.  You must also supply 'sendemail.aliastype'.
 
-sendemail.aliasfiletype::
-	Format of the file(s) specified in sendemail.aliasesfile. Must be
+sendemail.aliastype::
+	Format of the file(s) specified in sendemail.aliasfile. Must be
 	one of 'mutt', 'mailrc', 'pine', or 'gnus'.
 
 sendemail.to::
diff --git a/git-send-email.perl b/git-send-email.perl
index be4a20d..1d8bb8e 100755
--- a/git-send-email.perl
+++ b/git-send-email.perl
@@ -203,9 +203,9 @@ my %config_settings = (
     "smtppass" => \$smtp_authpass,
     "to" => \@to,
     "cccmd" => \$cc_cmd,
-    "aliasfiletype" => \$aliasfiletype,
+    "aliastype" => \$aliasfiletype,
     "bcc" => \@bcclist,
-    "aliasesfile" => \@alias_files,
+    "aliasfile" => \@alias_files,
     "suppresscc" => \@suppress_cc,
 );
 
-- 
1.5.4.4.685.g64f14.dirty
Frank Lichtenheld· Mar 18, 2008, 08:13 UTC · re: Xavier Maillard · lore

Re: [PATCH] Shorten and simplify sendemail.alias* configuration variables.

On Mon, Mar 17, 2008 at 11:53:04PM +0100, Xavier Maillard wrote:
> I find it more consistent and less error prone to have aliasfile
> instead of aliasesfile and aliastype instead of aliasfiletype.

I don't think that justifies breaking existing configs (which I assume it does).

Gruesse,
-- 
Frank Lichtenheld <frank@lichtenheld.de>
www: http://www.djpig.de/
Xavier Maillard· Mar 19, 2008, 01:00 UTC · re: Frank Lichtenheld · lore

Re: [PATCH] Shorten and simplify sendemail.alias* configuration variables.

   On Mon, Mar 17, 2008 at 11:53:04PM +0100, Xavier Maillard wrote:
   > I find it more consistent and less error prone to have aliasfile
   > instead of aliasesfile and aliastype instead of aliasfiletype.
   I don't think that justifies breaking existing configs (which I assume
   it does).

Yes it is likely to break configs after this patch has been applied. The m;ore confusing part in the current config variables is the *aliases* (plural) of the _aliasesfile_ variable. Actually, I am pretty sure, many people did not notice this plural form at first, so did I :)

Anyway, this is just a proposition of changes. Feel free not to apply it at all.

Regards
	Xavier
-- 
http://www.gnu.org
http://www.april.org
http://www.lolica.org

← back to recent threads