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

Xenko 3.1 Cannot resolve paths with spaces. #385

Closed
FailMelon opened this issue Feb 18, 2019 · 7 comments
Closed

Xenko 3.1 Cannot resolve paths with spaces. #385

FailMelon opened this issue Feb 18, 2019 · 7 comments
Labels
bug Something isn't working

Comments

@FailMelon
Copy link

FailMelon commented Feb 18, 2019

Release Type: Official Release/GitHub (please choose appropriate option)
Official Release - Beta

Version: Version number and/or git branch
3.1.0.1-beta01-0441

Platform(s): Does the problem occur on Windows, Android...?
Windows 10

Describe the bug
A clear and concise description of what the bug is.
When creating a new project xenko cannot find files it needs, i believe this is caused because there is a space in my windows user.

To Reproduce
Steps to reproduce the behavior:

  1. Create a windows account with a space
  2. Try to create a project under 3.1 build.

Expected behavior
A clear and concise description of what you expected to happen.
Xenko should be able to resolve paths with spaces.

Screenshots
If applicable, add screenshots to help explain your problem (use GitHub drag & drop)
image

@FailMelon FailMelon added the bug Something isn't working label Feb 18, 2019
@tebjan
Copy link
Member

tebjan commented Feb 18, 2019

sounds like this was already fixed in latest master, just not yet published in new beta: #381

@FailMelon
Copy link
Author

Ah i missed this pull request that would probably fix it.

@tebjan
Copy link
Member

tebjan commented Feb 18, 2019

let's hope for a new beta soon then.

@Kryptos-FR
Copy link
Member

Kryptos-FR commented Feb 18, 2019

@tebjan I don't think #381 will do anything for that issue.

And in general having spaces in username is bad and I expect other software to break as well. Looking on the Internet (in your favourite search engine) with a search query such as "Windows username space" brings a lot of results with various software, program or tools failing.

Even Microsoft recommends against doing so: https://docs.microsoft.com/en-us/previous-versions/windows/it-pro/windows-2000-server/bb726984(v=technet.10)

Logon names can contain all other special characters, including spaces, periods, dashes, and underscores. But it's generally not a good idea to use spaces in account names.

@FailMelon
Copy link
Author

@Kryptos-FR I'm not even gonna begin to start at how ignorant that post is, if #381 doesn't fix it then it needs to be resolved still.

@Kryptos-FR
Copy link
Member

@FailMelon I'm not even gonna begin to start how disrespectful your post was. #381 was fixing a different issue, regarding building the sources.

@xen2
Copy link
Member

xen2 commented Feb 19, 2019

@FailMelon Thanks for the report.

This specific issue you had is an issue in the T4 template generator lib we use.
I have pushed a fix in Xenko for it already: 4372725 (and also a fix on the T4 library side: mono/t4#38).

Please wait for the next beta release.
Closing since it is already fixed.

@xen2 xen2 closed this as completed Feb 19, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants