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 more explicit install instructions #81

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

unusualevent
Copy link

not sure if this is even working correctly.

I'm still getting errors when trying to build a package.

@unusualevent
Copy link
Author

unusualevent commented Jul 28, 2023

might as well as libscrypt-devel as well.... although I can't get that to build.

@@ -15,6 +15,28 @@ Features:
See info below, the [API docs](http://godoc.org/github.com/cretz/bine), and the [examples](examples). The project is
MIT licensed. The Tor docs/specs and https://github.com/yawning/bulb were great helps when building this.

## Installing
Bine requires tor-static, which cannot be installed by go's package manager.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bine does not require tor-static. That's only if you want to link it statically (i.e. you opt in by importing github.com/cretz/bine/process/embedded). This is mentioned under "Example".

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can't get it to work without tor-static existing, so I guess the example doesn't work.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please be more specific about the error you are encountering when running the example. Are you sure you are following all the steps such as "assuming tor is on the PATH"?

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.

None yet

2 participants