mathworks-ref-arch / matlab-dockerfile

Create a docker container that contains a MATLAB install
Other
337 stars 96 forks source link

Path issues in the final build stage. #38

Closed bjacobs1 closed 2 years ago

bjacobs1 commented 2 years ago

In your Dockerfile there is a discrepancy which causes the symbolic link and the hint to where to put the license file to be incorrect. Either copy from /usr/local/MATLAB/Rxxxxx instead of from /usr/local/MATLAB when you copy your results to the final stage of the build process, or adjust the creation of the symbolic link and the hint to where to put the license file.

josmartin commented 2 years ago

@bjacobs1 Umm - in the Dockerfile (rather than Dockerfile.R2019b) we explicitly install into a location that no longer contains the release version of MATLAB in the path (see line 49 where we put the destination folder of the install). So the location of the binary that starts MATLAB is /usr/local/MATLAB/bin/matlab. Could you confirm that when you build the container this is where you see the MATLAB install? If this is the case then we shouldn't hint that anything needs to be in Rxxxxx since that is no longer a valid folder.

If I've mis-understood your request above please feel free to give more details on what you were expecting.

bjacobs1 commented 2 years ago

You're right, this is my mistake. I had a different build context than you prescribed. My apologies.

hopedrivendeveloper commented 2 years ago

@josmartin Despite line 49 -destinationFolder /usr/local/MATLAB while running docker build -t matlab-r2021b --build-arg LICENSE_SERVER=27000@MyServerName . Matlab indeed is installed into /usr/local/MATLAB/R2021b. Had to modify line 56 to COPY --from=middle-stage /usr/local/MATLAB/R2021b /usr/local/MATLAB in order to make it working

josmartin commented 2 years ago

Expecting this to be fixed by an up-coming change to the Dockerfile next week. Thanks @hopedrivendeveloper and @bjacobs1 for highlighting this.

prabhakk-mw commented 2 years ago

These Dockerfiles have now been deprecated in Favor of installing MATLAB using MPM.