Re: additionally saving history for each directory?

[Greg Klanderman <gak@xxxxxxxxxxxxxx>]

>>>>> On February 9, 2021 Bart Schaefer <schaefer@xxxxxxxxxxxxxxxx>
>>>>> wrote:

> On Mon, Feb 8, 2021 at 5:26 PM Greg Klanderman <gak@xxxxxxxxxxxxxx>
> wrote:
>>  Figured I'd follow up with this on workers since I never got a
>> response on whether
>>  save_per_directory_history () { setopt localoptions
>> incappendhistory fc -p -a .zsh_local_history 1000 print -sr --
>> ${1%%$'\n'}
>> }
>>  should work in zshaddhistory_functions for saving local history.

> It doesn't work because you added the -a option to the fc command.

>      A hook function may call 'fc -p ...' to switch the history
>      context so that the history is saved in a different file from
>      the that in the global HISTFILE parameter.  This is handled
>      specially: the history context is automatically restored after
>      the processing of the history line is finished.

Thanks for looking into this Bart..

I think you are saying that it is just an unintended side effect of
how the history hook magic works that makes what I did not work?
But based on the docs, it seems like what I tried should work?

So does this mean it is not possible to write the history line to
more than 2 files?

> The history file isn't actually written until the hooks have all
> finished running, because the exit status of the hook function can
> control whether the event gets written at all.  "fc -p -a" closes
> the alternate history when the function returns, which is before the
> decision to write has been made, so nothing gets written.

Of course, but I'm not trying to change where the default history line
is saved.. I'm trying to temporarily switch to a different history,
write the line there, and have the default history restored.

When I do

           zshaddhistory() {
             print -sr -- ${1%%$'\n'}
             fc -p .zsh_local_history
           }

does the print not print to the default history file immediately, or
is it being delayed pending the return value of all the hooks for
whether to actually write?

> The other non-obvious (and not really documented) thing about using
> "fc -p" in the zshaddhistory hook is that it has to be the last line
> in the function.  It appears from some prodding that you can follow
> it with another history hook function in zshaddhistory_functions,
> but if anything follows it in the same function, it doesn't work.  I
> suspect this is unintentional, but it seems to be the reason that
> the documentation has the example in the order that it does:

>           zshaddhistory() {
>             print -sr -- ${1%%$'\n'}
>             fc -p .zsh_local_history
>           }

> Further, even if you put the "fc -p" before the "print -s", the hook
> magic causes the line to be added to the original internal history
> rather than to a new list created by "fc -p", so "fc -AI" or similar
> have nothing to write into .zsh_local_history.

Interesting.  If you happened to dig deep enough to have paged in
where all this magic is happening, could you give me a few pointers to
how it is working?

> The only way to do it is to use "fc -p" without "-a" and as the last
> command in the function.

> And, I suspect although I got a bit lost
> trying all the permutations, incappendhistory has to be a global
> setopt rather than a local one, again because the local scope has
> already ended before the write decision is made.

Sure, I would expect that for the regular automatic history write
which happens at the end of the zshaddhistory_functions, but not an
explicit print -s in that local scope.  Unless that print -s is also
being delayed until the end of zshaddhistory_functions..

Anyway, I have incappendhistory on globally so at least that should
not be an issue!

cheers,
Greg