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

Failed to generate GodotSharp solution. Error: 47 #12466

Closed
ghost opened this issue Oct 28, 2017 · 6 comments
Closed

Failed to generate GodotSharp solution. Error: 47 #12466

ghost opened this issue Oct 28, 2017 · 6 comments

Comments

@ghost
Copy link

ghost commented Oct 28, 2017

Using current master, all godot projects using mono fail to run.

The CLI errors are:

   At: editor/doc/doc_data.cpp:830.                                                                                                                                                                                                                                            
ERROR: _generate_cs_type: Condition ' !setter && !getter ' is true. returned: ERR_BUG                                                                                                                                                                                          
   At: modules/mono/editor/bindings_generator.cpp:665. 

This started happening after I re-generated the mono glue with the current master & recompiled.

@neikeq
Copy link
Contributor

neikeq commented Oct 29, 2017

This was caused by a doc regression, but tagging as a bug in the mono module since we should get those from ClassDB instead of DocData.

@neikeq
Copy link
Contributor

neikeq commented Oct 29, 2017

@akien-mga I won't re-write this in time for tomorrow, so we must make sure this regression is fixed before alpha2.

@akien-mga
Copy link
Member

Doctool will fix it, that PR was bogus.

akien-mga added a commit that referenced this issue Oct 29, 2017
Fixes regression from #12396 which caused #12466.
[ci skip]
@neikeq
Copy link
Contributor

neikeq commented Nov 23, 2017

I guess this can be closed, right?

@ghost
Copy link
Author

ghost commented Nov 23, 2017

we should get those from ClassDB instead of DocData.

If this is done or not needed anymore, sure

@neikeq
Copy link
Contributor

neikeq commented Jan 23, 2018

Fixed by #15136

@neikeq neikeq closed this as completed Jan 23, 2018
@akien-mga akien-mga added this to the 3.0 milestone Jan 23, 2018
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