[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] Re: [BUG] [C-u C-u C-c C-o] open link with external program inva
From: |
Christopher Miles |
Subject: |
[PATCH] Re: [BUG] [C-u C-u C-c C-o] open link with external program invalid to open file |
Date: |
Thu, 21 Jan 2021 00:58:27 +0000 |
Maxim Nikulin <manikulin@gmail.com> writes:
> I have realized that :filter argument is not appropriate to arbitrary
> handler that could spam with various assert messages. Maybe :sentinel is
> better to report configuration problem to user when handler is called
> with inappropriate arguments.
Yes, use ~:sentinel~ is better for report problem when detect process failed or
configuration error. ~:filter~ usually used in handling process output.
I "=git am=" your first patch, but failed to apply your second patch. Seems not
continuous. So I modify code manually. I found I can't find the local variable
~shell-command~, and ~shell-command-switch~. Can your provide complete patch?
So I
can test it.
I update subject to prefix with [PATCH] so that reader knows this thread is on
patch discussion.
--
[ stardiviner ]
I try to make every word tell the meaning that I want to express.
Blog: https://stardiviner.github.io/
IRC(freenode): stardiviner, Matrix: stardiviner
GPG: F09F650D7D674819892591401B5DF1C95AE89AC3
signature.asc
Description: PGP signature