-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
mdl2obj Linux and macOS support #15
Comments
Hello! @PeachIceTea
Thank you for notice. |
The new version does build and run under Linux. Can't test Mac right now, but I believe its safe to assume that pkg for Mac will work as well. Personally I think I would prefer having source files and using my distros version of Node, but I understand the advantages of shipping something that will just work. |
Linux user here, Ubuntu 20.04.6 LTS x86_64. |
@Lachrymogenic there is a new commit in the mdl2obj repo that fixes that issue. Just run |
Seems to be some sort of issue with your code, too. wouldnt run unless I converted Seems to work completely fine after changing that one line now. Imports into blender just fine. |
I little bit rushed during fix, yes :D |
Currently the code for mdl2obj is shipped as a Windows executable mdl2obj.exe.
I suppose it makes sense to not ask users on Windows to go install Node.js. For better support of Linux and macOS I would advocate to ship the raw Javascript and ask those users to simply install Node.
I would be happy to create a Pull Request with those changes if you think that is good enough of a solution.
As a side note the mdl2obj repo seems to be missing the source-mdl source code that is required to make it work.
The text was updated successfully, but these errors were encountered: