Skip to content
This repository was archived by the owner on Oct 30, 2018. It is now read-only.

Update Setup.md#316

Open
smorad wants to merge 1 commit intofacebookarchive:masterfrom
smorad:patch-2
Open

Update Setup.md#316
smorad wants to merge 1 commit intofacebookarchive:masterfrom
smorad:patch-2

Conversation

@smorad
Copy link
Copy Markdown
Contributor

@smorad smorad commented Jun 1, 2017

Talked a user through setting up atc. Turns out that pip will silently fail to install the atcd binary to /usr/local/bin if they don't have write permission. Add sudo to the guide so they aren't as confused.

@chantra
Copy link
Copy Markdown
Contributor

chantra commented Jun 1, 2017

originally, the documentation mentioned using a virtualenv, which should install the files where the user has permission. Was the virtualenv step missing?

@facebook-github-bot
Copy link
Copy Markdown

Thank you for your pull request. We require contributors to sign our Contributor License Agreement, and yours has expired.

Before we can review or merge your code, we need you to email cla@fb.com with your details so we can update your status.

Copy link
Copy Markdown
Contributor

@stasfilin stasfilin left a comment

Choose a reason for hiding this comment

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

@smorad you don't need sudo access for this, try to use virtualenv or pyenv

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants