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

PBD renaming functionality assumes file is present when it shouldn't #96350

Closed
ryevdokimov opened this issue Aug 30, 2024 · 3 comments · Fixed by #96383
Closed

PBD renaming functionality assumes file is present when it shouldn't #96350

ryevdokimov opened this issue Aug 30, 2024 · 3 comments · Fixed by #96383

Comments

@ryevdokimov
Copy link
Contributor

Tested versions

Since: e73f40e

System information

N/A

Issue description

In short, #87117 makes the assumption mentioned in the title resulting in error like this, which is now possible after this PR #93972:
image

See comments here for details: #93972 (comment)

image

Steps to reproduce

Build: f0ee0bd
Add a plugin like Godot-Jolt that has an entry for PBD in the header (may have to build this for 4.4)
Make sure .godot folder is deleted
Open the project

Minimal reproduction project (MRP)

N/A

@ghost
Copy link

ghost commented Aug 30, 2024

Having the same issue in 4.3 stable.
Screenshot 2024-08-30 141400

tried reinstalling Jolt but no luck. Also tried renaming the dll to pdb for kicks and that just produced more errors.

@mihe
Copy link
Contributor

mihe commented Aug 31, 2024

If the errors bother you then you can (until this is fixed) just download the debug symbols for Godot Jolt here and extract them alongside the DLLs.

@ghost
Copy link

ghost commented Aug 31, 2024

If the errors bother you then you can (until this is fixed) just download the debug symbols for Godot Jolt here and extract them alongside the DLLs.

It does bother me a bit, so I'll go ahead and do this. Thank you!

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

Successfully merging a pull request may close this issue.

3 participants