In case this helps, here's a quick way to check the tramp stuff if you have a second machine with MATLAB. I'll refer to the primary machine as the local machine and the secondary machine as the remote machine in the following.

1. Install emacs on the remote machine. Install matlab-mode on it also
   (ensure it is the same version as on the local machine).
2. Apply the patch on the installed matlab-mode on both the local &
   remote machines.
3. Ensure ssh server is running on the remote machine and is
   discoverable from the local machine.
4. Open a matlab file on the remote machine from an emacs session on
   the local machine through TRAMP.
5. Write some matlab code on the buffer.
6. Invoke matlab-shell from the tramp-buffer.
7. Check if matlab-shell-run-cell works.
8. Check, from the MATLAB REPL, which device it is running on. It
   should be running on the remote machine.

On 12/20/23 18:21, Uwe Brauer wrote:
"NNB" == Nidish Narayanaa Balaji<nidish.bal...@ila.uni-stuttgart.de>  writes:
No no - it is indeed backward compatible. I tested it out with emacs
28.1 also. font-lock-reference-face has been deprecated for a while
now. And font-lock-constant-face has been around since at least emacs
20.3 (according to documentation). So the patch is backward
compatible.
Ok thanks for checking.


Are there any other comments, @Eric, @John?

I'd prefer to wait a bit more till I push, I did not encounter any difficulties 
so far.
But I only checked the part concerning the fonts, not TRAMP stuff.

On 12/20/23 17:18, Uwe Brauer wrote:
"NNB" == Nidish Narayanaa Balaji<nidish.bal...@ila.uni-stuttgart.de>  writes:
Hello All,
I made a couple merge requests but Uwe told me that there are
apparently issues merging on sourceforge atm. Here are the links to
the two merge requests, just for reference:
+https://sourceforge.net/p/matlab-emacs/src/merge-requests/3/
+https://sourceforge.net/p/matlab-emacs/src/merge-requests/4/
The first one was a fix for the fact that emacs 29.1 has removed
font-lock-reference-face and we are to use font-lock-constant-face
instead.
It seems to me that your patch is not backward compatible, so user of
emacs <29 will run into difficulties, right?

--
Dr. Nidish Narayanaa Balaji

Humboldt Postdoctoral Fellow
Institute of Aircraft Propulsion Systems
Department of Aerospace Engineering and Geodesy
University of Stuttgart
Pfaffenwaldring 6
D-70569 Stuttgart

Tel.: +49 711 685 69433
e-mail:nidish.bal...@ila.uni-stuttgart.de
_______________________________________________
Matlab-emacs-discuss mailing list
Matlab-emacs-discuss@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/matlab-emacs-discuss

Reply via email to