Re: master 18b680cfd1: Fix bug#52467 by adding a new custom variable 'display-comint-buffer-action'

[Juri Linkov]

>> --- a/lisp/progmodes/project.el
>> +++ b/lisp/progmodes/project.el
>> @@ -1015,7 +1015,7 @@ if one already exists."
>> -        (pop-to-buffer-same-window shell-buffer)
>> +        (pop-to-buffer shell-buffer display-comint-buffer-action)
>
> Please add boundp fallbacks: project.el should retain compatibility with
> older Emacs releases.

Or better (bound-and-true-p display-comint-buffer-action)

> +(defcustom display-comint-buffer-action 'display-buffer-same-window

This default value is not what pop-to-buffer-same-window uses:

  (pop-to-buffer buffer display-buffer--same-window-action norecord)

So for backward-compatibility the default value needs to be the same:

  (defcustom display-comint-buffer-action display-buffer--same-window-action

that requires a different type:

  :type 'display-buffer--action-custom-type