Skip to content
This repository has been archived by the owner on Jul 5, 2021. It is now read-only.

Revise installation docs to reflect the latest changes #98

Merged
merged 4 commits into from
Feb 24, 2016

Conversation

dignifiedquire
Copy link
Contributor

Closes #95

@dignifiedquire
Copy link
Contributor Author

@RichardLitt @whyrusleeping @diasdavid please check


You can install the [Go IPFS implementation](https://github.com/ipfs/go-ipfs)
from source or from prebuilt packages.
We recommend to install IPFS from a prebuilt package. You can obtain those from [dist.ipfs.io](http://dist.ipfs.io/#go-ipfs).
Copy link
Contributor

Choose a reason for hiding this comment

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

We recommend installing IPFS

@RichardLitt
Copy link
Contributor

Shouldn't we add a note saying that go get and previous other methods no longer work?

@dignifiedquire
Copy link
Contributor Author

Shouldn't we add a note saying that go get and previous other methods no longer work?

I'd rather not, to avoid confusion of mentioning it, but on the other hand it might be good to say so.

@ghost
Copy link

ghost commented Feb 17, 2016

Shouldn't we add a note saying that go get and previous other methods no longer work?

I'd rather not, to avoid confusion of mentioning it, but on the other hand it might be good to say so.

I feel like the number one reason for confusion right now is that go get doesn't work anymore ;) We should say so, especially since it has been the preferred installation method for a year or longer.

@whyrusleeping
Copy link
Contributor

agreed, lets mention that go get is no longer a correct way of installing


Download a binary for your platform (more coming):
After downloading, untar the archive, and move `ipfs` somewhere in your `$PATH`:
Copy link

Choose a reason for hiding this comment

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

Should also mention dist.ipfs.io at least once, so that people know where to download from

Copy link
Contributor

Choose a reason for hiding this comment

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

let's do:

"and move the ipfs binary somewhere in your executables $PATH":

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, I think in the packages need to come with an "install.sh" that is double-click friendly (this was the case with gobuilder, not sure if it remains so right now). that way users may not need to cd into the right place

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jbenet there is still an install.sh available, I'm guessing that works for linxus + os x?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@lgierth why, dist.ipfs.io is mentioned at the beginning on where to download things.


For convenience here are the latest versions (`0.3.11`) linked directly
Copy link
Contributor

Choose a reason for hiding this comment

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

No, let's not have versions here. it creates yet another thing to update.

@dignifiedquire
Copy link
Contributor Author

@jbenet @RichardLitt @lgierth updated

jbenet added a commit that referenced this pull request Feb 24, 2016
Revise installation docs to reflect the latest changes
@jbenet jbenet merged commit bdb9096 into master Feb 24, 2016
@jbenet jbenet deleted the fix/install-instructions branch February 24, 2016 10:19
@jbenet
Copy link
Contributor

jbenet commented Feb 24, 2016

thanks @dignifiedquire

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants