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

DotNetTool Module: Add support for dotnet local tools #3209

Open
augustoproiete opened this issue Feb 27, 2021 · 1 comment · May be fixed by #3212
Open

DotNetTool Module: Add support for dotnet local tools #3209

augustoproiete opened this issue Feb 27, 2021 · 1 comment · May be fixed by #3212

Comments

@augustoproiete
Copy link
Member

Creating a new issue for tracking cake-contrib/Cake.DotNetTool.Module#18 created by @gitfool given that GitHub doesn't allow transferring issues from one organization to another.


@gitfool wrote:

Note: I'm not sure this is a good idea and I'm currently stuck determining what the implementation looks like given local tools seem to be managed by the dotnet CLI.

I'll create a work-in-progress PR with how far I've got, including fixes for some minor bugs I found.

@gitfool
Copy link
Contributor

gitfool commented Feb 28, 2021

This is a followup to the second part of #2685 (comment):

  • Integrating with dotnet local tools

Also, for this to integrate better with dotnet local tools, the #tool directive should use the local tool directly, presumably invoking dotnet <tool>, rather than extracting and copying what was previously a global tool into the Cake tools directory alongside addins and modules. (I'm hoping such a change would also avoid the quirk resulting from building the same directory in Windows and then WSL or Docker with a host mount, or vice versa, where you first need to explicitly delete any such copied dotnet tools which would be for the wrong platform.)

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

Successfully merging a pull request may close this issue.

2 participants