Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

improve the README #1

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

improve the README #1

wants to merge 6 commits into from

Conversation

jchris
Copy link
Contributor

@jchris jchris commented May 8, 2012

These two commits make the README a little bit more idiot-proof so folks like me can build from source without needing to use their brains.

jchris added 2 commits May 8, 2012 10:46

Verified

This commit was signed with the committer’s verified signature.
driesvints Dries Vints
…r names avoids a symlinking error in the `make install` step.
@ns-buildbot
Copy link

Can you submit these to gerrit.

(preferably after fixing the commit message on the first one, since it's too long)

On May 8, 2012, at 10:52 AM, Chris Anderson wrote:

These two commits make the README a little bit more idiot-proof so folks like me can build from source without needing to use their brains.

You can merge this Pull Request by running:

git pull https://github.com/jchris/manifest master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Update README to avoid short names for brew packages. Using the longer names avoids a symlinking error in the make install step.
  • Add a hint to the README about the correct argument for -m <branch_name>

-- File Changes --

M README.markdown (7)

-- Patch Links --

https://github.com/membase/manifest/pull/1.patch
https://github.com/membase/manifest/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

dustin sallings

@ns-buildbot
Copy link

Chris,

Can we send this change through gerrit

Farshid

On May 8, 2012, at 10:52 AM, "Chris Anderson" reply@reply.github.com wrote:

These two commits make the README a little bit more idiot-proof so folks like me can build from source without needing to use their brains.

You can merge this Pull Request by running:

git pull https://github.com/jchris/manifest master

Or you can view, comment on it, or merge it online at:

#1

-- Commit Summary --

  • Update README to avoid short names for brew packages. Using the longer names avoids a symlinking error in the make install step.
  • Add a hint to the README about the correct argument for -m <branch_name>

-- File Changes --

M README.markdown (7)

-- Patch Links --

https://github.com/membase/manifest/pull/1.patch
https://github.com/membase/manifest/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

@adamtajti
Copy link

Could we close down this pull request and the other two? They are going back to 2012-2013. @ingenthr

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants