VS task runner uses hardcoded paths

[shrayasr]

/u/opium_tm says (permalink):

VS runner is implemented completely wrong way. Paths to VS are hardcoded in lib. For VS before 2017 you should get installation paths from well known registry branch. For VS 2017 you should use documented COM interfaces to enumerate (possibly side by side) VS 2017 installations.

Fair point. Everything in the project is in a "works-for-me" kind of situation.

Basically we're currently at:

• Make it work
• Make it right
• Make it fast

[SpartanX1]

Hello I am interested in solving this issue. Are you still active in this project?

[shrayasr]

Oh yes. I use it everyday :) Please go ahead.

[SpartanX1]

Hey you did not mention how to and where to place the config file?

[shrayasr]

Which config file @SpartanX1? The base iko config is placed at ~/iko.toml

[SpartanX1]

Do i need to install toml or something ? or just create a file with this name

[SpartanX1]

I have made changes for vs17 , but i need to check how it works.
Here's the syntax. I hope there is a space between path and @params?

var path = GetVisualStudioInstallationFolder();

Process.Start("cmd", $@"/c start "+path+" "[email protected]);

[SpartanX1]

I am not clear on how to run and test the application.
Could you please provide the steps? @shrayasr

[shrayasr]

Do i need to install toml or something ? or just create a file with this name

Not at all. TOML is a plain text file in a certain format. Just creating a file with the name is enough.

I am not clear on how to run and test the application.
Could you please provide the steps?

• Build it
• dotnet \path\to\iko.dll <task name>

This assumes that you have already defined the tasks in the TOML file. In your case, the TOML file could be:

[ikosln]
cmd = 'vs17'
path = 'c:\users\spartanx1\projects\iko\iko.sln'

and run it as

dotnet \path\to\iko.dll ikosln

[SpartanX1]

Thanks I will test it out

[shrayasr]

You're welcome. Thank you for working on this 🎉

[SpartanX1]

Hello could you review my pull request @shrayasr

[shrayasr]

Will do it soon, @SpartanX1. Held up at work at the moment

[SpartanX1]

Sure thanks:)

[aintabb]

@shrayasr Is this issue still open?