-
-
Notifications
You must be signed in to change notification settings - Fork 22k
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
Blender importer: allow Blender 4 executables #82211
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Notice any breakage?
@@ -322,7 +322,7 @@ static bool _test_blender_path(const String &p_path, String *r_err = nullptr) { | |||
} | |||
return false; | |||
} | |||
if (version > 3) { | |||
if (version > 4) { | |||
if (r_err) { | |||
*r_err = TTR("This Blender installation is too new for this importer (not 3.x)."); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This message needs to be updated
#81194 needs to be merged first, or this won't work. |
#81194 actually removes the upper-bound version check entirely. |
Superseded by #81194. |
Blender 4 is currently scheduled to release in ~6 weeks, so it'd be nice to be able to already test the importer with the new version.