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

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.

@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" [email protected] 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

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.

3 participants