search

aspnet / dnvm

OBSOLETE - see readme
Other
174 stars 61 forks source link

Alias -r is case sensitive #449

Closed glennc closed 9 years ago

glennc commented 9 years ago

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.

bjorkstromm commented 9 years ago

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

muratg commented 9 years ago

@mholo65 Sure, thanks!

bjorkstromm commented 9 years ago

@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 commented 9 years ago

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

glennc commented 9 years ago

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 commented 9 years ago

:+1: It would be great if we could be 100% consistent with this