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

Exe find #147

Closed
wants to merge 9 commits into from
Closed

Exe find #147

wants to merge 9 commits into from

Conversation

CClouseau
Copy link
Contributor

@CClouseau CClouseau commented Apr 9, 2021

Hi @awaescher ,

It's been a while since I started thinking of a way to efficiently find executable paths... First I started a bit of rework of the exe search to add new applications more easily (Git Bash I remember).
I didn't find any reliable solution to search executables in "exotic" directories, so I ended up allowing to do it... in the configuration file !!
Sorry master, to go against your will of a zero config app !! ;)

So , it goes like this :
{
"AutoFetchMode": 0,
"PruneOnFetch": true,
"ExePaths": [{
"ApplicationName": "SourceTree",
"ApplicationFullPath": "C:\Users\######_01.001\AppData\Local\SourceTree\SourceTree.exe"
}, {
"ApplicationName": "GitBash",
"ApplicationFullPath": ""
}, {
"ApplicationName": "WindowsTerminal",
"ApplicationFullPath": ""
}, {
"ApplicationName": "VSCode",
"ApplicationFullPath": ""
}
]
}

=> if no path is provided the good ol' search does the job.

@CClouseau
Copy link
Contributor Author

@awaescher Hi again, I tried to pull the rework you did on "remotes" on my ExeFind branch: I had an issue calling ReadAllBranches because then, I had only remote branches in my checkout list. That's why I have deleted my latest commits.

@awaescher
Copy link
Owner

If I got this right, this pull request is not required anymore as we are merging #114.
Please feel free to reopen if I was wrong.

Thank you

@awaescher awaescher closed this Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants