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

--stdin not working? #85

Closed
MichaelChirico opened this issue Jun 20, 2021 · 7 comments · Fixed by #86
Closed

--stdin not working? #85

MichaelChirico opened this issue Jun 20, 2021 · 7 comments · Fixed by #86
Labels
bug Something isn't working

Comments

@MichaelChirico
Copy link

MichaelChirico commented Jun 20, 2021

I tried the README example for running woke on stdin and it didn't work:

echo "This has whitelist from stdin" | woke --stdin
No violations found. Stay woke ✊

I just tried installing with:

curl -sSfL https://git.io/getwoke | \
  bash -s -- -b /usr/local/bin

My version:

woke --version
# woke version 0.9.0

The same issue if I try the go get route as well.

@MichaelChirico MichaelChirico added the bug Something isn't working label Jun 20, 2021
@MichaelChirico
Copy link
Author

I am trying to git bisect to when things changed but not having any luck installing an older version -- I tried running git checkout v0.1.5 && sh install.sh but it just installs the latest release. I am not familiar enough with go to go any further than that I'm afraid :\

@caitlinelfring
Copy link
Member

Thanks for the bug report! This appears to have been introduced in https://github.com/get-woke/woke/releases/tag/v0.8.1 (specifically #78). I'll work on a fix

@caitlinelfring
Copy link
Member

A side note, to install a different release using the install script, you just need to supply the version at the end. I'll update this in the readme to be clearer:

curl -sSfL https://git.io/getwoke | \
  bash -s -- -b /usr/local/bin v0.8.0

@MichaelChirico
Copy link
Author

thanks! does it have to be a release or can a commit hash work as well? (sorry I'm afk & can't test myself)

@caitlinelfring
Copy link
Member

thanks! does it have to be a release or can a commit hash work as well? (sorry I'm afk & can't test myself)

The install script only works for releases. If you want to run a specific commit hash, you'll need to use the go toolchain (go get will only install HEAD)

@caitlinelfring
Copy link
Member

caitlinelfring commented Jun 20, 2021

Fix has been released in v0.9.1

@MichaelChirico
Copy link
Author

Thanks! Works now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants