search

florianschanda / miss_hit

MATLAB Independent, Small & Safe, High Integrity Tools - code formatter and more
GNU General Public License v3.0
158 stars 21 forks source link

Update m_language_builtins to use the latest MATLAB release #240

Open PeterTillema opened 2 years ago

PeterTillema commented 2 years ago

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

florianschanda commented 2 years ago

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.

krishnakumarg1984 commented 2 years ago

Mini bump

florianschanda commented 1 year ago

@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 commented 1 year ago

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.