Skip to content
This repository has been archived by the owner on Dec 18, 2017. It is now read-only.

Alias -r is case sensitive #449

Closed
glennc opened this issue Sep 22, 2015 · 6 comments · Fixed by #469
Closed

Alias -r is case sensitive #449

glennc opened this issue Sep 22, 2015 · 6 comments · Fixed by #469
Assignees
Milestone

Comments

@glennc
Copy link
Member

glennc commented Sep 22, 2015

dnvm alias default -r CoreCLR will create an alias but then not work correctly due to the case.

It should either fail and tell me that the casing is wrong or actually work. Ideally it would work, perhaps lowering all the runtime names since they are all lower case in the file names.

@muratg muratg added the bug label Sep 23, 2015
@muratg muratg added this to the 1.0.0-rc1 milestone Sep 23, 2015
@bjorkstromm
Copy link
Contributor

Up for grabs? I could have a look at it.

@muratg
Copy link

muratg commented Oct 5, 2015

@mholo65 Sure, thanks!

@bjorkstromm
Copy link
Contributor

@glennc Isn't version missing from your description? I.e. shouldn't the command be e.g. dnvm alias default 1.0.0-beta7 -r CoreCLR

@bjorkstromm
Copy link
Contributor

How about other commands? E.g. dnvm install latest -r CoreCLR will also fail?

@glennc
Copy link
Member Author

glennc commented Oct 14, 2015

dnvm install default -r coreclr will take the version and other information from the default alias.

This is a useful feature that often gets overlooked :).

Yes, seems likely it is a problem everywhere and should be fixed everywhere.

@muratg
Copy link

muratg commented Oct 15, 2015

👍 It would be great if we could be 100% consistent with this

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

Successfully merging a pull request may close this issue.

4 participants