Subject: Re: bug#32564: 27.0.50; cedet/ede/pconf.el

From: Noam Postavsky <[email-masked]>
Date: Fri, 31 Aug 2018 22:16:30 -0400

Pierre Lorenzon <[email-masked]> writes:

> Bug in pconf.el has not been fixed yet. Here is a diff
> indicating what has to be done. I have to do it each time I
> upgrade my git copy of emacs otherwise C-c . C in ede mode
> crashes.

> @@ -135,7 +135,7 @@
>  	  (with-current-buffer "*compilation*"
>  	    (goto-char (point-max))
>  
> -	    (when (not (string= mode-line-process ":exit [0]"))
> +	    (when (not (string= (car mode-line-process) ":exit [0]"))
>  	      (error "Configure failed!"))
>  
>  	    ;; The Makefile is now recreated by configure?
>
> Diff finished.  Wed Aug 29 05:09:19 2018
>
>
>>>>  -- End Diff 
>
>
> The problem is that this code do not take care of the fact that
> type of variable `mode-line-process' has been changed from
> string to list.

Wouldn't it be better to, e.g., add a function to
compilation-finish-functions so that pconf would stop depending on the
exact modeline format used by compile.el?