From 8fbee0933a091c0e229c9efd22b521cc8eef71e8 Mon Sep 17 00:00:00 2001
From: Todd Derr <salty@bandcamp.com>
Date: Wed, 13 Oct 2021 14:05:47 -0400
Subject: [PATCH] Add an environment variable `ICDIFF_DEFAULTS` to allow
 setting default flags.

I'm not 100% happy with this because there's no way to override boolean
options using `optparse`.  i.e. if `ICDIFF_DEFAULTS=-N` there is no way
to override that on the command line.

Switching to `argparse` would allow that via `BooleanOptionalAction` but
that requires python 3.9.

With `optparse, I could also add `no-XXX` flags for all the existing
boolean flags - LMK.
---
 README.md | 5 +++++
 icdiff    | 8 +++++++-
 2 files changed, 12 insertions(+), 1 deletion(-)

diff --git a/README.md b/README.md
index 430f233..d7b5db0 100644
--- a/README.md
+++ b/README.md
@@ -80,6 +80,11 @@ Show differences between files in a two column view.
                         '--color-map=separator:white,description:cyan
 ```
 
+The environment variable `ICDIFF_DEFAULTS` can be set to a
+space delimited list of default flag values. For example:
+
+  `export ICDIFF_DEFAULTS='-N -U3 --color-map=description:magenta_bold,separator:magenta_bold'`
+
 ## Using with Git
 
 To see what it looks like, try:
diff --git a/icdiff b/icdiff
index 4d92058..abd4d95 100755
--- a/icdiff
+++ b/icdiff
@@ -601,12 +601,18 @@ def set_cols_option(options):
 def validate_has_two_arguments(parser, args):
     if len(args) != 2:
         parser.print_help()
+        print("\nThe environment variable ICDIFF_DEFAULTS can be set to a\n"
+              "space delimited list of default flag values.")
         sys.exit()
 
 
 def start():
     parser = create_option_parser()
-    options, args = parser.parse_args()
+    arguments = sys.argv[1:]
+    defaults = os.getenv("ICDIFF_DEFAULTS")
+    if defaults != None:
+        arguments = defaults.split() + arguments
+    options, args = parser.parse_args(arguments)
     validate_has_two_arguments(parser, args)
     if not options.cols:
         set_cols_option(options)