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

ERROR: Condition "!p_keep_state && has_instances" is true. Returning: ERR_ALREADY_IN_USE #97138

Closed
donn-xx opened this issue Sep 18, 2024 · 3 comments · Fixed by #97168
Closed

Comments

@donn-xx
Copy link

donn-xx commented Sep 18, 2024

Tested versions

Does not happen in 4.3 stable
Does happen in 4.4-dev (commit 48403b5)

System information

Godot v4.4.dev (48403b5) - Ubuntu 22.04.5 LTS 22.04 - X11 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 2060 (nvidia; 535.183.01) - Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz (6 Threads)

Issue description

Developing a plugin. Trying to save any changes to my gdscript files causes the error:

ERROR: Condition "!p_keep_state && has_instances" is true. Returning: ERR_ALREADY_IN_USE

This does not happen in 4.3 stable.

Steps to reproduce

Try the MRP. Very simple addon. See notes in bug_test.gd

Sample output on run:

Godot Engine v4.4.dev.custom_build.48403b535 (2024-09-16 23:05:57 UTC) - https://godotengine.org
Vulkan 1.3.242 - Forward+ - Using Device #0: NVIDIA - NVIDIA GeForce RTX 2060

YES:true
bug ready!
1. ERROR: Condition "!p_keep_state && has_instances" is true. Returning: ERR_ALREADY_IN_USE
   Any change here produces that error ONCE (so far).
2. After that, no changes in the code are reflected in the execution of this script.
3. Restarting the project gets you back to work, but goto 2.
YES:false <-- clicked the fail control a few times
YES:true
YES:false
YES:true <-- after this made a code change and used Ctrl+S

ERROR: Condition "!p_keep_state && has_instances" is true. Returning: ERR_ALREADY_IN_USE
   at: reload (modules/gdscript/gdscript.cpp:747)

Minimal reproduction project (MRP)

MRP

@donn-xx
Copy link
Author

donn-xx commented Sep 18, 2024

MRP added.

@KoBeWi
Copy link
Member

KoBeWi commented Sep 18, 2024

Changing this line to reload(true) (similar to what #92177 did)


fixes it. But again, it's suspicious.

@ryevdokimov
Copy link
Contributor

Bisected to: 9fd431f

@Hilderin

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

Successfully merging a pull request may close this issue.

4 participants