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

Add optional GetToolExecutableNames that takes tool settings. #2679

Closed
devlead opened this issue Dec 3, 2019 · 0 comments · Fixed by #2680
Closed

Add optional GetToolExecutableNames that takes tool settings. #2679

devlead opened this issue Dec 3, 2019 · 0 comments · Fixed by #2680
Assignees
Labels
Milestone

Comments

@devlead
Copy link
Member

@devlead devlead commented Dec 3, 2019

Currently there's no way to access tool settings when supplying a tool executable name, which might be needed if i.e. different processor architectures have different executables.

protected abstract IEnumerable<string> GetToolExecutableNames();

@devlead devlead added the Improvement label Dec 3, 2019
@devlead devlead added this to the v0.36.0 milestone Dec 3, 2019
@devlead devlead self-assigned this Dec 3, 2019
devlead added a commit to devlead/cake that referenced this issue Dec 3, 2019
… Tool

* fixes cake-build#2679
patriksvensson added a commit that referenced this issue Dec 3, 2019
GH2679: Add GetToolExecutableNames(TSettings settings) to Tool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.