GNU bug report logs - #76955
30.1; php-ts-mode-php-executable default path may not match remote path

Previous Next

Package: emacs;

Reported by: Morgan Willcock <morgan <at> ice9.digital>

Date: Tue, 11 Mar 2025 20:11:02 UTC

Severity: normal

Found in version 30.1

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Morgan Willcock <morgan <at> ice9.digital>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: v.pupillo <at> gmail.com, 76955 <at> debbugs.gnu.org
Subject: bug#76955: 30.1; php-ts-mode-php-executable default path may not match remote path
Date: Tue, 18 Mar 2025 16:00:57 +0000
Eli Zaretskii <eliz <at> gnu.org> writes:

>> From: Morgan Willcock <morgan <at> ice9.digital>
>> Cc: Eli Zaretskii <eliz <at> gnu.org>,  76955 <at> debbugs.gnu.org
>> Date: Mon, 17 Mar 2025 22:30:03 +0000
>> 
>> Vincenzo Pupillo <v.pupillo <at> gmail.com> writes:
>> 
>> > Ciao Eli, this is the new patch.
>> 
>> Hi Vincenzo,
>> 
>> The php binaries which I use are installed with pkgsrc, which supports
>> installing multiple versions of PHP, installed as bin/php${PHP_VERSION}.
>> One of those versions would be available as bin/php.
>> 
>> If I have two remote versions installed, and one of them is found by
>> executable-find searching for "php", I cannot see how I would select the
>> other version once the patch is applied.
>
> How do you do this with the current version of php-ts-mode?

Since the value of php-ts-mode-php-executable is used locally and
remotely, I just set the buffer local value to the correct name.

  (connection-local-set-profile-variables
   'with-php84
   '((php-ts-mode-php-executable . "php84")))

  (connection-local-set-profiles
   '(:application tramp :machine "remotename")
   'with-php84)

>> Shouldn't the search name "php" be configurable to allow the name to be
>> overridden?  Something like this:
>> 
>>   (executable-find php-program t)
>> 
>> where php-program defaults to being "php".
>
> It's possible to add this, but it would be a separate feature.

The variable php-ts-mode-php-executable is effectively already providing
this feature, but the suggested change appears to remove it.

-- 
Morgan Willcock




This bug report was last modified 52 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.