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

Fix 38745: Importing FBX of a non-Unicode encoded path causes Assimp throws error #39363

Merged
merged 1 commit into from
Jun 7, 2020

Conversation

MarcusElg
Copy link
Contributor

Fixes #38745 by converting the path to utf8

@RevoluPowered
Copy link
Contributor

Thanks for this fix, can you please update the commit message to be "import" there is a spelling error as you've written "inport".

Copy link
Contributor

@RevoluPowered RevoluPowered left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Commit message needs updated

@MarcusElg
Copy link
Contributor Author

@RevoluPowered fixed

@akien-mga akien-mga added bug cherrypick:3.x Considered for cherry-picking into a future 3.x release topic:import labels Jun 7, 2020
@akien-mga akien-mga added this to the 4.0 milestone Jun 7, 2020
@akien-mga akien-mga merged commit 46d8d60 into godotengine:master Jun 7, 2020
@akien-mga
Copy link
Member

Thanks!

@MarcusElg
Copy link
Contributor Author

@akien-mga this was my eleventh commit (source: https://github.com/godotengine/godot/commits?author=MCrafterzz) so will I be added to the contributors list?

@MarcusElg MarcusElg deleted the fix38745 branch June 7, 2020 13:53
@akien-mga
Copy link
Member

Yes, in my next update commit (at the start of July). Which name would like you me to put?

@MarcusElg
Copy link
Contributor Author

MarcusElg commented Jun 7, 2020

Yes, in my next update commit (at the start of July). Which name would like you me to put?

Marcus (MCrafterzz)

Anutrix added a commit to Anutrix/godot that referenced this pull request Jun 8, 2020
@akien-mga
Copy link
Member

Cherry-picked for 3.2.2.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Jun 10, 2020
akien-mga pushed a commit to akien-mga/godot that referenced this pull request Jun 10, 2020
…cene that became unused after the recent commit ec1bf96(godotengine#39363).

(cherry picked from commit 365c35f)
huhund pushed a commit to huhund/godot that referenced this pull request Nov 10, 2020
…cene that became unused after the recent commit ec1bf96(godotengine#39363).

(cherry picked from commit 365c35f)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Importing FBX of a non-Unicode encoded path causes Assimp throws error
3 participants