Inaccuracies in install instructions

dominikh commented 8 years ago

A list of current inaccuracies and inconsistencies I've found in the README's install instructions:

[ ] go get -v sourcegraph.com/sourcegraph/srclib-go doesn't work because of the use of Godep and imports that haven't had their import paths rewritten. This should probably be changed to godep get?
[ ] Apparently, the src tool was renamed to srclib on srclib's master, but the latest pre-compiled binary is still called src. This duality makes it a bit hard to fix the instructions at the moment
[ ] src toolchain add sourcegraph.com/sourcegraph/srclib-go doesn't seem to be a valid command anymore, at least with srclib from current master. Instead, one has to create a symlink manually. Was the removal of that function intentional? Are developers expected to mess around with symlinks?

Should the README on master be correct for the development version of srclib, or for the current release? That is, should references to src be rewritten to srclib already?

dominikh commented 8 years ago

For what it's worth, the src vs srclib issue also affects the other srclib-* repositories.

beyang commented 8 years ago

Yes, all references to src should be rewritten as srclib. The new src command runs the entire Sourcegraph application (and includes srclib as a subcommand, but it is a separate thing).

The latest srclib binary is available here: https://s3-us-west-2.amazonaws.com/srclib-release/srclib/0.2.0/linux-amd64/srclib.gz.

beyang commented 8 years ago

And yes, the install instructions should mention godep. Thanks for working through these issues!

guilhermebr commented 8 years ago

:+1:

sourcegraph / srclib-go

Inaccuracies in install instructions #69