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

Warn users that Cake.exe is missing when calling "Install Intellisense Support" #79

Open
MeikTranel opened this issue Dec 16, 2017 · 2 comments

Comments

@MeikTranel
Copy link

When checking out a repository clean, common sense would dictate that after executing the "Install intellisense support" Intellisense would work right off the bat (Given that troubleshooting steps have been taken (like making sure OmniSharp is installed, or that the Project has been loaded).

But what happens is, that for OmniSharp to work it is necessary that the regular bootstrapper has been called beforehand, because otherwise Cake.exe is not available.

Cake-VSCode should either:

  • spawn an Error Fly-In prompting the user to the lack of the Cake.Exe
  • call the generation of the cake.exe by itself
@gep13 gep13 added this to the 0.12.0 milestone Jul 1, 2018
@gep13 gep13 removed this from the v0.12.0 milestone Apr 9, 2019
@nils-a
Copy link
Member

nils-a commented Nov 13, 2020

Actually it seems /tools/Cake/Cake.Core.dll is required,

The error from OmniSharp is:

[fail]: OmniSharp.Cake.CakeProjectSystem
        d:\_dev\test-123\bob.cake will be ignored due to an following error
System.IO.FileNotFoundException: Cake is not installed. Path d:/_dev/test-123/tools/Cake/Cake.Core.dll does not exist.
   at OmniSharp.Cake.CakeProjectSystem.GetProject(CakeScript cakeScript, String filePath) in D:\a\1\s\src\OmniSharp.Cake\CakeProjectSystem.cs:line 280
   at OmniSharp.Cake.CakeProjectSystem.AddCakeFile(String cakeFilePath) in D:\a\1\s\src\OmniSharp.Cake\CakeProjectSystem.cs:line 124

@gep13 for a project/bootstrapper that will use the .NET global tool the "workaround" of calling the bootstrapper will no longer work. Should we ensure a version of Cake being downloaded into the tools folder (along with Cake.Bakery) or should we raise an issue upstram? (Or both...) No, my fault. cake-build/bakery#108 addressed that problem and my mistake was not to test with .NET global tool installed.

So we need to make sure that either the .NET global tool is installed globally or the local runner-of-choice is restored. We could try to detect packages.config|tool-manifest and start a restore if needed.

@ymartin59
Copy link

After https://github.com/cake-build/bakery/pull/109/files
issue may still happen with Chocolatey installation: cake-build/bakery#115

It is worth to notify user about issue to locate Cake.Core.dll as it requires quite some time to identify trouble in log to understand why IntelliSense does not work.

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

4 participants