-
-
Notifications
You must be signed in to change notification settings - Fork 21.1k
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 error when exporting files without import parameters #94405
Conversation
ydeltastar
commented
Jul 15, 2024
- Fixes: Compute shader file gives error when exporting project #94400
@@ -1230,7 +1230,9 @@ Error EditorExportPlatform::export_project_files(const Ref<EditorExportPreset> & | |||
|
|||
// Erase useless sections. | |||
config->erase_section("deps"); |
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.
Should this also have a check just in case?
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.
I think "deps" always has elements in it since it stores the path of the source file and the imported file. So we shouldn't need to check it
@@ -1295,7 +1297,9 @@ Error EditorExportPlatform::export_project_files(const Ref<EditorExportPreset> & | |||
|
|||
// Erase useless sections. | |||
config->erase_section("deps"); |
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.
Same as above.
ad05978
to
326e517
Compare
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.
Makes sense to me.
It seems like the [params]
section is ignored when the config is loaded because it never contains any items.
Additionally, the only other place that config->erase_section("params");
is called, checks for the existence of "params" first
Thanks! |