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

Bug Fix: Find correct binary on new install #27

Merged
merged 1 commit into from
Dec 8, 2017

Conversation

khornberg
Copy link
Contributor

On a new install of aws-fuzzy-finder, I got a go stack trace and
aws-fuzzy-finder whould fail to work.

Changing the setup to include the correct version of fzf resolved the
problem.

On a new install of aws-fuzzy-finder, I got a go stack trace and
aws-fuzzy-finder whould fail to work.

Changing the setup to include the correct version of fzf resolved the
problem.
@khornberg
Copy link
Contributor Author

To reproduce, I'm on High Sierra in a python 3.6 virtualenv.

Install aws-fuzzy-finder and then try running aws-fuzzy.
I got a go stack trace at this point

If I pip install -e . from within the source of aws-fuzzy-finder with the change in this PR, aws-fuzzy works as expected.

@rszalski rszalski added the bug label Dec 8, 2017
@rszalski rszalski self-requested a review December 8, 2017 10:29
@rszalski
Copy link
Collaborator

rszalski commented Dec 8, 2017

Right, I upgraded the libs, changed settings.py but forgot about setup.py :(

Thanks for the fix!

@rszalski rszalski merged commit 0ad5bc1 into pmazurek:master Dec 8, 2017
@khornberg khornberg deleted the bug/new-install branch December 8, 2017 13:53
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 this pull request may close these issues.

2 participants