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

Editor crash while importing a material exported from material maker #85214

Closed
yahiaetman opened this issue Nov 22, 2023 · 3 comments
Closed

Comments

@yahiaetman
Copy link

Godot version

v4.2-rc1_mono_win64

System information

Window 11 - Godot v4.2-rc1 mono win64 - Forward+

Issue description

While loading a certain material exported from material maker, the editor always crashes.

The material in question is included in the reproduction project.

The same material works fine in Godot 4.1.1 (mono - win64).

Steps to reproduce

  1. Create a new project in Godot v4.2-rc1_mono_win64.
  2. Drag and drop the material (included in the reproduction project) into the project.
  3. The editor crashes.

Minimal reproduction project

GoldImportBug.zip

@yahiaetman yahiaetman changed the title Editor crash while importing a material maker output Editor crash while importing a material exported from material maker Nov 22, 2023
@akien-mga akien-mga added this to the 4.2 milestone Nov 22, 2023
@akien-mga
Copy link
Member

I can't seem to reproduce the issue on Linux, but it may be a Windows specific bug.

I have a hunch it might be threading related and thus similar to #85105.

Could you test the build linked in #85039 (comment) and confirm if that solves the issue?

@yahiaetman
Copy link
Author

Yes, it works fine in the build you've linked. The material was imported without any problems.

@akien-mga
Copy link
Member

Great! Closing as duplicate of #85105 then, the next 4.2-rc2 build will have that fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants