* bug#50279: 28.0.50; ffap-file-finder customization no longer works
@ 2021-08-30 17:07 Bob Rogers
2021-08-31 1:59 ` Lars Ingebrigtsen
0 siblings, 1 reply; 13+ messages in thread
From: Bob Rogers @ 2021-08-30 17:07 UTC (permalink / raw)
To: 50279
I have a long-standing customization to ffap-file-finder which offers
external applications depending on file extension that is broken by the
following commit:
commit 253b36bd40d1b342cdda09b0bee183788120f67e
Author: dickmao <none>
Date: Tue Aug 3 12:02:48 2021 -0400
ffap ido accommodation
Now that ffap-file-finder can be ido-find-file, the
noninteractive portion of find-file-at-point cannot
assume ffap-file-finder always takes an argument
(ido-find-file does not).
* lisp/ffap.el (find-file-at-point): Do not call ffap-file-finder.
* test/lisp/ffap-tests.el (ffap-ido-mode): Test it.
The essence of the change is that it does find-file at the critical
point instead of funcalling ffap-file-finder. I assume that breaking
such extensions was not the intent, as it is an incompatible change to a
defcustom variable, and there is no associated NEWS entry.
-- Bob Rogers
http://www.rgrjr.com/
^ permalink raw reply [flat|nested] 13+ messages in thread* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2021-08-30 17:07 bug#50279: 28.0.50; ffap-file-finder customization no longer works Bob Rogers @ 2021-08-31 1:59 ` Lars Ingebrigtsen [not found] ` <24877.41211.907036.418118@orion.rgrjr.com> 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2021-08-31 1:59 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279, dick.r.chiang Bob Rogers <rogers@rgrjr.com> writes: > I have a long-standing customization to ffap-file-finder which offers > external applications depending on file extension that is broken by the > following commit: > > commit 253b36bd40d1b342cdda09b0bee183788120f67e > Author: dickmao <none> > Date: Tue Aug 3 12:02:48 2021 -0400 > > ffap ido accommodation Do you have a test case that demonstrates the breakage? -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
[parent not found: <24877.41211.907036.418118@orion.rgrjr.com>]
* bug#50279: 28.0.50; ffap-file-finder customization no longer works [not found] ` <24877.41211.907036.418118@orion.rgrjr.com> @ 2021-08-31 3:43 ` Lars Ingebrigtsen 2021-09-29 16:02 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2021-08-31 3:43 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 (Re-sending the mail to the bug tracker.) Bob Rogers <rogers@rgrjr.com> writes: > From: Lars Ingebrigtsen <larsi@gnus.org> > Date: Tue, 31 Aug 2021 03:59:34 +0200 > > Do you have a test case that demonstrates the breakage? > > I can reproduce this using my full customizations, but not in emacs -Q > with an excerpt of my code. More precisely, the problem reproduces, but > not the desired behavior after reverting the change. And indeed, > looking closer at the code, I don't understand why reverting this > particular change should make a difference. But it worked when I built > emacs on 31-Jul (d3d6ea50d2) and didn't when I next built it on 11-Aug > (3b5f8ab0d0), and I don't see any other change to ffap.el that might > explain the difference. > > I'll be out of town tomorrow, so I'll have to give this another look > on Wednesday. > > -- Bob ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2021-08-31 3:43 ` Lars Ingebrigtsen @ 2021-09-29 16:02 ` Lars Ingebrigtsen 2021-09-29 16:11 ` Bob Rogers 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2021-09-29 16:02 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Lars Ingebrigtsen <larsi@gnus.org> writes: >> I can reproduce this using my full customizations, but not in emacs -Q >> with an excerpt of my code. More precisely, the problem reproduces, but >> not the desired behavior after reverting the change. And indeed, >> looking closer at the code, I don't understand why reverting this >> particular change should make a difference. But it worked when I built >> emacs on 31-Jul (d3d6ea50d2) and didn't when I next built it on 11-Aug >> (3b5f8ab0d0), and I don't see any other change to ffap.el that might >> explain the difference. >> >> I'll be out of town tomorrow, so I'll have to give this another look >> on Wednesday. This was a month ago -- did you make any further progress in debugging this? -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2021-09-29 16:02 ` Lars Ingebrigtsen @ 2021-09-29 16:11 ` Bob Rogers 2021-09-30 6:09 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Bob Rogers @ 2021-09-29 16:11 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 50279 From: Lars Ingebrigtsen <larsi@gnus.org> Date: Wed, 29 Sep 2021 18:02:52 +0200 Lars Ingebrigtsen <larsi@gnus.org> writes: >> I can reproduce this using my full customizations, but not in emacs -Q >> with an excerpt of my code. More precisely, the problem reproduces, but >> not the desired behavior after reverting the change. And indeed, >> looking closer at the code, I don't understand why reverting this >> particular change should make a difference. But it worked when I built >> emacs on 31-Jul (d3d6ea50d2) and didn't when I next built it on 11-Aug >> (3b5f8ab0d0), and I don't see any other change to ffap.el that might >> explain the difference. >> >> I'll be out of town tomorrow, so I'll have to give this another look >> on Wednesday. This was a month ago -- did you make any further progress in debugging this? It definitely has to do with my initializations, but I've been swamped for the last three weeks and unable to get any further. I should be able to get back to it this week. -- Bob ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2021-09-29 16:11 ` Bob Rogers @ 2021-09-30 6:09 ` Lars Ingebrigtsen 2022-08-22 18:52 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2021-09-30 6:09 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Bob Rogers <rogers@rgrjr.com> writes: > It definitely has to do with my initializations, but I've been swamped > for the last three weeks and unable to get any further. I should be > able to get back to it this week. Great; there's no hurry. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2021-09-30 6:09 ` Lars Ingebrigtsen @ 2022-08-22 18:52 ` Lars Ingebrigtsen 2022-08-22 19:08 ` Bob Rogers 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2022-08-22 18:52 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Lars Ingebrigtsen <larsi@gnus.org> writes: >> It definitely has to do with my initializations, but I've been swamped >> for the last three weeks and unable to get any further. I should be >> able to get back to it this week. > > Great; there's no hurry. This was almost a year ago. Any progress? ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-08-22 18:52 ` Lars Ingebrigtsen @ 2022-08-22 19:08 ` Bob Rogers 2022-08-23 10:00 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Bob Rogers @ 2022-08-22 19:08 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 50279 From: Lars Ingebrigtsen <larsi@gnus.org> Date: Mon, 22 Aug 2022 20:52:30 +0200 Lars Ingebrigtsen <larsi@gnus.org> writes: >> It definitely has to do with my initializations, but I've been swamped >> for the last three weeks and unable to get any further. I should be >> able to get back to it this week. > > Great; there's no hurry. This was almost a year ago. Any progress? I spent a while trying bifurcation last winter with no result. I really do want to resolve this issue, but I just moved my family across the continent, and have barely had time to think for months now. Our furniture delivery has been delayed, so I've started to clean up some software loose ends (and catch up on emacs-devel); with any luck I may have time to look at this again. For once I have something to blame other than my world-class procrastination skills . . . -- Bob ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-08-22 19:08 ` Bob Rogers @ 2022-08-23 10:00 ` Lars Ingebrigtsen 2022-09-06 3:10 ` Bob Rogers 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2022-08-23 10:00 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Bob Rogers <rogers@rgrjr.com> writes: > Our furniture delivery has been delayed, so I've started to clean up > some software loose ends (and catch up on emacs-devel); with any luck > I may have time to look at this again. Great! > For once I have something to blame other than my world-class > procrastination skills . . . 😀 ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-08-23 10:00 ` Lars Ingebrigtsen @ 2022-09-06 3:10 ` Bob Rogers 2022-09-06 10:25 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Bob Rogers @ 2022-09-06 3:10 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 50279 From: Lars Ingebrigtsen <larsi@gnus.org> Date: Tue, 23 Aug 2022 12:00:51 +0200 Bob Rogers <rogers@rgrjr.com> writes: > Our furniture delivery has been delayed, so I've started to clean up > some software loose ends (and catch up on emacs-devel); with any luck > I may have time to look at this again. Great! Having finally gotten around to investigating this properly, I find that the solution should have been obvious: dickmao removed the feature in the following commit: commit 253b36bd40d1b342cdda09b0bee183788120f67e Author: dickmao <none> Date: Tue Aug 3 12:02:48 2021 -0400 ffap ido accommodation Now that ffap-file-finder can be ido-find-file, the noninteractive portion of find-file-at-point cannot assume ffap-file-finder always takes an argument (ido-find-file does not). * lisp/ffap.el (find-file-at-point): Do not call ffap-file-finder. * test/lisp/ffap-tests.el (ffap-ido-mode): Test it. I no longer understand why we thought it could not be reproduced in "emacs -Q", as I can do so with the following code: (defun rgr-ffap-file-finder (file-name &optional wildcards) (interactive "FFind file: \np") (message "[got %S]" file-name) (find-file file-name wildcards)) (setq ffap-file-finder 'rgr-ffap-file-finder) and this recipe: 1. "emacs -Q" 2. Load the code above. 3. Locate a suitable file in dired (I used a JPEG). 4. Position point on the file name, and do "M-x ffap". The JPEG will appear in emacs but the "[got ...]" message will not appear. Additionally, find-file-at-point can be instrumented with the patch below. That will identify the exact find-file call being used, and is how I traced it to this commit. But of course resolving my issue without breaking dickmao's issue is beyond my pay grade. ;-} Though I do notice that ffap.el already has (memq ffap-file-finder '(find-file find-alternate-file)) as a way of testing for arg-handling capability. But maybe that's enough of a code smell. -- Bob ------------------------------------------------------------------------ diff --git a/lisp/ffap.el b/lisp/ffap.el index 88b4bce9fd..a4dec5e792 100644 --- a/lisp/ffap.el +++ b/lisp/ffap.el @@ -1623,6 +1623,7 @@ find-file-at-point ((or (not ffap-newfile-prompt) (file-exists-p filename) (y-or-n-p "File does not exist, create buffer? ")) + (message "ffap calling find-file on %S directly" filename) (find-file ;; expand-file-name fixes "~/~/.emacs" bug (expand-file-name filename))) ^ permalink raw reply related [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-09-06 3:10 ` Bob Rogers @ 2022-09-06 10:25 ` Lars Ingebrigtsen 2022-09-06 17:03 ` Bob Rogers 0 siblings, 1 reply; 13+ messages in thread From: Lars Ingebrigtsen @ 2022-09-06 10:25 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Bob Rogers <rogers@rgrjr.com> writes: > But of course resolving my issue without breaking dickmao's issue is > beyond my pay grade. ;-} Though I do notice that ffap.el already has > > (memq ffap-file-finder '(find-file find-alternate-file)) > > as a way of testing for arg-handling capability. But maybe that's > enough of a code smell. Yeah, the ffap code is kinda gross. I think I've now fixed this in Emacs 29 by restoring the old code in ffap and adjusting the hooks in ido instead. Can you check whether this fixed the problem for you? ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-09-06 10:25 ` Lars Ingebrigtsen @ 2022-09-06 17:03 ` Bob Rogers 2022-09-06 20:14 ` Lars Ingebrigtsen 0 siblings, 1 reply; 13+ messages in thread From: Bob Rogers @ 2022-09-06 17:03 UTC (permalink / raw) To: Lars Ingebrigtsen; +Cc: 50279 From: Lars Ingebrigtsen <larsi@gnus.org> Date: Tue, 06 Sep 2022 12:25:31 +0200 Bob Rogers <rogers@rgrjr.com> writes: > But of course resolving my issue without breaking dickmao's issue is > beyond my pay grade. ;-} Though I do notice that ffap.el already has > > (memq ffap-file-finder '(find-file find-alternate-file)) > > as a way of testing for arg-handling capability. But maybe that's > enough of a code smell. Yeah, the ffap code is kinda gross. No argument from this quarter. I think I've now fixed this in Emacs 29 by restoring the old code in ffap and adjusting the hooks in ido instead. Can you check whether this fixed the problem for you? Works like a charm. Another thorn in my side removed; thanks! -- Bob ^ permalink raw reply [flat|nested] 13+ messages in thread
* bug#50279: 28.0.50; ffap-file-finder customization no longer works 2022-09-06 17:03 ` Bob Rogers @ 2022-09-06 20:14 ` Lars Ingebrigtsen 0 siblings, 0 replies; 13+ messages in thread From: Lars Ingebrigtsen @ 2022-09-06 20:14 UTC (permalink / raw) To: Bob Rogers; +Cc: 50279 Bob Rogers <rogers@rgrjr.com> writes: > Works like a charm. Another thorn in my side removed; thanks! Thanks for checking; I'm closing this bug report, then. ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2022-09-06 20:14 UTC | newest]
Thread overview: 13+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-08-30 17:07 bug#50279: 28.0.50; ffap-file-finder customization no longer works Bob Rogers
2021-08-31 1:59 ` Lars Ingebrigtsen
[not found] ` <24877.41211.907036.418118@orion.rgrjr.com>
2021-08-31 3:43 ` Lars Ingebrigtsen
2021-09-29 16:02 ` Lars Ingebrigtsen
2021-09-29 16:11 ` Bob Rogers
2021-09-30 6:09 ` Lars Ingebrigtsen
2022-08-22 18:52 ` Lars Ingebrigtsen
2022-08-22 19:08 ` Bob Rogers
2022-08-23 10:00 ` Lars Ingebrigtsen
2022-09-06 3:10 ` Bob Rogers
2022-09-06 10:25 ` Lars Ingebrigtsen
2022-09-06 17:03 ` Bob Rogers
2022-09-06 20:14 ` Lars Ingebrigtsen
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).