unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#3418: Issue with compile.el and compilation-parse-errors-filename-function
@ 2009-05-29 14:46 Gary Oberbrunner
  2016-01-26  5:21 ` Andrew Hyatt
  0 siblings, 1 reply; 10+ messages in thread
From: Gary Oberbrunner @ 2009-05-29 14:46 UTC (permalink / raw)
  To: bug-gnu-emacs

Hi emacs folks.  I submitted a patch to compilation-get-file-structure 
in compile.el in 2001, introducing this stanza:

	;; If compilation-parse-errors-filename-function is
	;; defined, use it to process the filename.
	(when compilation-parse-errors-filename-function
	  (setq filename
		(funcall
			 filename)))

At some point since then, the filename was changed to not always be 
absolute; there's now a variable spec-directory in that function.  This 
means that implementations of compilation-parse-errors-filename-function 
can't always work correctly since it doesn't know the full path of the file.

I'm happy to work on a fix, but I see a few issues.

Solution 1: add 2nd arg SPEC-DIRECTORY to 
compilation-parse-errors-filename-function.
Problem: existing implementations will get an incorrect number of args 
error and will have to change.

Solution 2: make filename absolute before passing to 
compilation-parse-errors-filename-function.
Problem: the rest of the code is pretty careful not to absolutize the 
filename; this would change the behavior in ways I don't completely 
understand.

Of course I am personally happy with solution 1, but since it affects 
compatibility I thought I should bring it up on this list.  I am not on 
the list, so please cc me with any replies, thanks!

-- 
. . . . . . . . . . . . . . . . . . . . . . . . .
Gary Oberbrunner                garyo@genarts.com
GenArts, Inc.                   Tel: 617-492-2888
955 Mass. Ave                   Fax: 617-492-2852
Cambridge, MA 02139 USA         www.genarts.com






^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2017-08-12 15:42 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-05-29 14:46 bug#3418: Issue with compile.el and compilation-parse-errors-filename-function Gary Oberbrunner
2016-01-26  5:21 ` Andrew Hyatt
2016-01-26 14:42   ` Eli Zaretskii
2016-01-26 15:15     ` Gary Oberbrunner
2016-01-26 16:08       ` Eli Zaretskii
2016-01-26 16:19         ` Gary Oberbrunner
2017-08-11  0:50           ` npostavs
2017-08-11  1:04             ` Gary Oberbrunner
2017-08-11  6:34             ` Eli Zaretskii
2017-08-12 15:42               ` npostavs

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).