Skip to content
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

fix import pmx Texture missing on MacOS or Other OS. #50

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lengyue524
Copy link

some Texture path include '' ,but MacOS didn't recognize it ,need replace by 'os.path.sep'
Tested on my Mac.
focus on line 137.
Other lines auto delete blank by my IDE。

@lordscales91
Copy link
Contributor

Sadly it seems sugiany has abandoned the project... Well, I'm not going to maintain it actively. But I've fixed a few bugs.

@ptrthomas
Copy link

FYI @powroupi is active and we collaborated on fixing paths for non-windows OS-es: powroupi@d9b895f

Things work fine on my Mac.

@lordscales91
Copy link
Contributor

@ptrthomas I'm very glad to hear that, I took a look at some commits and you have made a great work.

Probably what I fixed in lordscales91/blender_mmd_tools@b90431d you had already fixed it a long time ago. I will open an issue here to let the people know this project has been discontinued by sugiany and is being active developed by @powroupi

cnDengyu pushed a commit to cnDengyu/blender_mmd_tools that referenced this pull request Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants