OmniSharp / omnisharp-atom

Omnisharp support for the atom editor
https://atom.io/packages/omnisharp-atom
MIT License
318 stars 61 forks source link

omnisharp looks for default alias runtime #415

Open cataquil opened 9 years ago

cataquil commented 9 years ago

Installed Atom from site. Installed omnisharp-atom from package install in atom (apt works find, no need for choco install)

ran powershell script from asp.net/home dnvm install 1.0.0-beta dnvm alias 1.0.0-beta4 default

roslyn global.json had 1.0.0-beta4 specified, but still omnisharp-atom looks for default

the persistent switch on 'dnvm use' didn't work

This was on windows 8, yeah probably use vs2015 but this is interesting anyhow. So perhaps this could be in the docs for other folks.

nosami commented 9 years ago

The default runtime is used by omnisharp itself. Any runtime can be used by your code.

david-driscoll commented 9 years ago

Odd, I thought we booted off of our own runtime (the one packaged with the server) or is this some other runtime?

cataquil commented 9 years ago

I installed dnvm as per instructions. No default was set. It will be helpful for other newbies to add an instruction to setup a default alias

david-driscoll commented 9 years ago

@nosami shouldn't the sdk.runtime override this?

"sdk": {
    "version": "1.0.0-beta5"
}

Wonder if we can update omnisharp-roslyn to try the first remaining framework if it can't find the default alias?

nosami commented 9 years ago

The default alias is the one used to power omnisharp itself, not the application being developed. It's by design, the same way that VS works.

On Sat, Jul 11, 2015 at 9:13 PM, David Driscoll notifications@github.com wrote:

@nosami https://github.com/nosami shouldn't the sdk.runtime override this?

"sdk": { "version": "1.0.0-beta5" }

Wonder if we can update omnisharp-roslyn to try the first remaining framework if it can't find the default alias?

— Reply to this email directly or view it on GitHub https://github.com/OmniSharp/omnisharp-atom/issues/415#issuecomment-120658126 .

david-driscoll commented 9 years ago

Okay cool.

There is a configuration for it, we just need to expose that.

On Sat, Jul 11, 2015 at 6:06 PM, Jason Imison notifications@github.com wrote:

The default alias is the one used to power omnisharp itself, not the application being developed. It's by design, the same way that VS works.

On Sat, Jul 11, 2015 at 9:13 PM, David Driscoll notifications@github.com wrote:

@nosami https://github.com/nosami shouldn't the sdk.runtime override this?

"sdk": { "version": "1.0.0-beta5" }

Wonder if we can update omnisharp-roslyn to try the first remaining framework if it can't find the default alias?

— Reply to this email directly or view it on GitHub < https://github.com/OmniSharp/omnisharp-atom/issues/415#issuecomment-120658126

.

— Reply to this email directly or view it on GitHub https://github.com/OmniSharp/omnisharp-atom/issues/415#issuecomment-120665310 .