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

Support .rsp files for Scripting Project System bootstrapping #1024

Closed
filipw opened this issue Nov 11, 2017 · 4 comments
Closed

Support .rsp files for Scripting Project System bootstrapping #1024

filipw opened this issue Nov 11, 2017 · 4 comments

Comments

@filipw
Copy link
Member

filipw commented Nov 11, 2017

Spun off from dotnet/vscode-csharp#1856

It would be useful for people not using CSI or dotnet-script - especially those using custom CSX hosts - to be able to predefine using statements and assembly references via .rsp files. Those are standard files already used by the compiler itself and look like this: https://github.com/dotnet/roslyn/blob/version-2.4.0/src/Interactive/csi/csi.rsp

In our case, we'd pick up the references used there and use them as globally imported namespaces / references when constructing CSX project system. Configuration proposal - inside omnisharp.json, under script node, as rspPath property. For example:

Since omnisharp.json can be global, you could even - in one sweep - configure your entire DEV box if you wanted.

@filipw filipw changed the title Support .rsp files for Scripting Support .rsp files for Scripting Project System bootstrapping Nov 11, 2017
@filipw filipw self-assigned this Nov 11, 2017
@KarloX2
Copy link

KarloX2 commented Nov 12, 2017

This is a great idea!

Ideally, referencing assemblies with /r would optionally allow the usage of environment variables, maybe like so:

/r:C:\Some\Specific\Folder\MyLib.dll
/r:%MY_ENV_VAR%\Folder\MyOtherLib.dll

@KarloX2
Copy link

KarloX2 commented Nov 14, 2017

cc me: @KarloX2

@filipw
Copy link
Member Author

filipw commented Nov 27, 2017

OK I've made some good progress on this - will PR this soon.

@KarloX2
Copy link

KarloX2 commented Nov 28, 2017

Very good! I'm really curious to see it. Can you please post here once the update is available? Thanks!

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

No branches or pull requests

3 participants