-
Notifications
You must be signed in to change notification settings - Fork 47
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
Why is findExecutables private? #14
Comments
@evancz: For the case of a |
Ah, I see that now. Is there something that will help me? It looks like it's plausible to do what I want on UNIX, maybe using |
Following our (seemingly accepted) new principle of just getting things done: I've pushed a commit that exposes |
I don’t want to tar and feather you, but given that this does not actually help with @evancz issue, and I’m not sure what other uses cases there are, I don’t think there is much point in exporting this function. It will just cause confusions for users, thinking “what is that for?” when they read through the docs. |
As I see it:
The argument of "could confuse people" implies that we should never add functions that some users may not want. |
Looks like you found a use case (although maybe a bit hypothetical), so I concur ;–) |
well, the "could confuse people" part could be addressed by improving the documentation... :) |
On Windows (specifically non-Cygwin), you should inspect the |
Closing since the original issue has been resolved. |
This reverts commit 0dd40ad.
Context: I am working on a tool like
git
such thatgit hats
actually routes the user to an executable namedgit-hats
. One nice feature provided bygit
is that they will list all of the commands available on your system, even if they are not distributed with git. This means figuring everything on the user's PATH and filtering it down to things that matchgit-*
.My case described here is pretty much exactly the same, and I need the functionality of
findExecutables
to make it happen.Question: Is there a strong reason
findExecutables
(implemented here) is not exposed? If so, would you recommend duplicating the code? If so, what is going on with themingw32_HOST_OS
macros? Do I need to do any special configuration to use macros like that?The text was updated successfully, but these errors were encountered: