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

installation_path parameter problems #371

Closed
bdovaz opened this issue Nov 1, 2019 · 4 comments · Fixed by #373
Closed

installation_path parameter problems #371

bdovaz opened this issue Nov 1, 2019 · 4 comments · Fixed by #373
Assignees
Labels

Comments

@bdovaz
Copy link

bdovaz commented Nov 1, 2019

This command works correctly:

u3d install 2019.2.11f1 --installation_path "E:\Unity2019.2.11f1"

If I run the same command again it reinstalls it instead of saying it is already installed.

Also if I want to install a module in the same path later it is impossible for me.

u3d install 2019.2.11f1 -p Android --installation_path "E:\Unity2019.2.11f1"

I get the following error:

Please install Unity before any of its packages

I don't know if it's related to this other bug:

#370

@niezbop niezbop self-assigned this Nov 4, 2019
@niezbop niezbop added the bug label Nov 4, 2019
@niezbop
Copy link
Member

niezbop commented Nov 4, 2019

Hey @bdovaz ! Thanks for raising this issue to our attention!

This is indeed due to #370, I'll answer on the other issue!

@niezbop
Copy link
Member

niezbop commented Nov 6, 2019

Hey @bdovaz! Just to let you know that we started tackling this issue, and we're trying to design a good solution for this issue.

My current proposition is to introduce an environment variable to specify the additional paths, but we're pondering whether this is a good idea.

We'd love your input if you have some time to discuss this issue regarding the best way to tackle that, so feel free to leave a comment on #373 !

@bdovaz
Copy link
Author

bdovaz commented Nov 6, 2019

Thanks @niezbop !

@niezbop
Copy link
Member

niezbop commented Nov 7, 2019

Heyo @bdovaz ! The solution to this will be in the next release! Thanks a lot for your feedback

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

Successfully merging a pull request may close this issue.

2 participants