Add path workaround for Linux#233
Conversation
|
I don't have a linux machine with Matlab to test it, but isn't Linux using a |
Linux uses `:` as path separator, not `;` like Windows.
|
Uh-oh, you're completely right. It was working in my minimal example to start Matlab as |
|
Nice, thanks for the patch! :) |
|
Thank you for the quick review, and for catching my mistake! :) |
|
Please use rebase and merge next time. |
|
Hello,
Sorry, the issue still seems to persist on Linux (Ubuntu 22.04) and Matlab 24.2.0.2712019 (R2024b) after the path workaround. Tested with clean Julia environments across 1.9, 1.10_lts and 1.11 versions. Also checked the pre--workaround tagged release version of MATLAB.jl(0.8.4) and by building the latest version (0.9.0) of the package locally. The issue seems to persist. Am I missing something? Please see the error from one of the configs I tested below (all other combinations seem to be identical): Could you please let me know if I am doing something wrong? I'd be happy to provide more info/ test the changes. |
|
Hey @sdamera95 |
|
I think I managed to reproduce the issue and fix it. On a fresh install, I created two environments. In the first one I installed I fixed it by rebuilding the package (in the environment with Then restarted Julia, and it worked. |
|
To be honest my first attempt at debugging this was: Installed Then I remembered that to change matlab version one has to trigger a rebuild with |
|
Fun (and somewhat interesting) fact: after rebuilding I am not competent enough to understand what is happening here. And the fact that two installations of |
|
That is very odd. |
|
In both environments (with the different versions of julia> MATLAB.matlab_libpath
"/usr/local/MATLAB/R2024b/bin/glnxa64"The content of # This file is automatically generated, do not edit.
function check_deps()
if libmx_size != filesize(Libdl.dlpath(joinpath(matlab_libpath, "libmx")))
error("MATLAB library has changed, re-run Pkg.build(\"MATLAB\")")
end
end
const matlab_libpath = "/usr/local/MATLAB/R2024b/bin/glnxa64"
const matlab_cmd = "exec /usr/local/MATLAB/R2024b/bin/matlab"
const libmx_size = 3600568I removed everything in |
Description
Exactly the same as #226, but for Linux. Tested and found out that it works too.
Related to:
Before
After