search

membase / manifest

repo manifest
http://techzone.couchbase.com/wiki/display/membase/Contributing+changes
44 stars 117 forks source link

improve the README #1

Open jchris opened 12 years ago

jchris commented 12 years ago

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 commented 12 years ago 
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:

https://github.com/membase/manifest/pull/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

-- 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: https://github.com/membase/manifest/pull/1

dustin sallings

ns-buildbot commented 12 years ago

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:

https://github.com/membase/manifest/pull/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

-- 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: https://github.com/membase/manifest/pull/1

adamtajti commented 7 years ago

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