[PATCH] remove 'appendhistory' from zsh-newuser-install

[PATCH] remove 'appendhistory' from zsh-newuser-install

[Andrey Butirsky]

[-- Attachment #1: Type: text/plain, Size: 949 bytes --]

Hello,
could we remove 'appendhistory' option from zsh-newuser-install, since 
it actually 'on' by default and new users unlikely would like to change it?

---
  Functions/Newuser/zsh-newuser-install | 1 -
  1 file changed, 1 deletion(-)

diff --git a/Functions/Newuser/zsh-newuser-install b/Functions/Newuser/zsh-newuser-install
index e4028fd50..60ac16b13 100644
--- a/Functions/Newuser/zsh-newuser-install
+++ b/Functions/Newuser/zsh-newuser-install
@@ -899,7 +899,6 @@ __zni_options_config() {
    # The "no" prefix is used to indicate options on by default.
    __zni_apply_defaults -o autocd '' "Change directory given just path." \
      extendedglob '' "Use additional pattern matching features." \
-    appendhistory '' "Append new history lines instead of overwriting." \
      '!nomatch' '' "Unmatched patterns cause an error." \
      '!beep' '' "Beep on errors." \
      notify '' "Immediately report changes in background job status."

Re: [PATCH] remove 'appendhistory' from zsh-newuser-install

[Daniel Shahaf]

Andrey Butirsky wrote on Sat, 04 Jan 2020 22:08 +00:00:
> Hello,
> could we remove 'appendhistory' option from zsh-newuser-install, since 
> it actually 'on' by default and new users unlikely would like to change it?
> 
> ---

Yes, I agree.  Applied.

The diff did not apply cleanly, presumably because it was sent with
format=flowed.  To prevent leading/trailing whitespace from being munged
you can either send patches as attachments, disable format=flowed in
Thunderbird (https://wiki.openstack.org/wiki/MailingListEtiquette#Thunderbird),
or mark the unidiff as "preformatted text" in Thunderbird's editor.

Thanks for the patch,

Daniel