niryariv / opentaba-server

BSD 3-Clause "New" or "Revised" License
4 stars 13 forks source link

add missing instructions for ubuntu issues on installation (missing package) #95

Closed noamoss closed 9 years ago

noamoss commented 9 years ago

adding the missing package: libmemcached-dev

niryariv commented 9 years ago

Not merging:

  1. This breaks compatibility for systems without apt-get installed (ie non-linux)
  2. This isn't really part of the installation, in the same way that installing Python isn't in the README.

A better approach would be to add a line saying "If you get an error like ... you might need to install libmemcache (for example apt get ... in Ubuntu")

noamoss commented 9 years ago

is it better?

niryariv commented 9 years ago

screen shot 2015-05-16 at 9 39 01 pm ???

niryariv commented 9 years ago

did you see the issue there? you've mistakenly added a bunch of unrelated files to the repo - we can go over this together on wednesday

noamoss commented 9 years ago

It was the virtualenv files. I thought it was needed and realized you don't want it. On May 17, 2015 7:54 AM, "Nir Yariv" notifications@github.com wrote:

did you see the issue there? you've mistakenly added a bunch of unrelated files to the repo - we can go over this together on wednesday

— Reply to this email directly or view it on GitHub https://github.com/niryariv/opentaba-server/pull/95#issuecomment-102747439 .