miss_hit icon indicating copy to clipboard operation
miss_hit copied to clipboard

Update m_language_builtins to use the latest MATLAB release

Open PeterTillema opened this issue 3 years ago • 4 comments

Also, besides this, it would be nice if Windows is supported as well, where MATLAB is named matlab.exe, not matlab :)

PeterTillema avatar Nov 08 '21 09:11 PeterTillema

Also, besides this, it would be nice if Windows is supported as well, where MATLAB is named matlab.exe, not matlab :)

Development environment for MH will stay Linux, it's just not worth it to deal with Windows issues. Unless I get a sudden influx of windows contributors :) But of course the tool itself always will support any platform that Python runs on!

First, thank you for this PR.

Before I apply this I want to quickly have a think about language support in general. Right now my real life work uses this tool and they are on a certain MATLAB version; so uplifting this would mean I need to fork.

But what I really want actually is to have a language option beyond "octave = true/false"; and this PR is a good reminder that I should do that sooner rather than later.

I've been a bit busy and non-responsive and for that I apologise, but again thank you for the PR; I will do something about it possibly on the weekend.

florianschanda avatar Nov 10 '21 10:11 florianschanda

Mini bump

krishnakumarg1984 avatar May 20 '22 15:05 krishnakumarg1984

@krishnakumarg1984 and @PeterTillema I have not forgotten about this. I got another ticket now with a similar root issue (#258) and I will now say that this is the next big thing to be happening for MH (the detailed config of language dialect).

florianschanda avatar Aug 13 '22 13:08 florianschanda

OK, so this is coming to the next release. I will push a massive change soon that allows you to say matlab: 2017b or octave: 4.4 in your config file.

While it doesn't do immediately what is asked for in this PR; it enables it.

florianschanda avatar Aug 21 '22 13:08 florianschanda