search

tessel / openwrt-tessel

OpenWrt build scripts and package feed for Tessel 2
Other
32 stars 14 forks source link

Ubuntu Dependencies #50

Closed Student007 closed 8 years ago

Student007 commented 8 years ago

Ubuntu requires zlib1g-dev libncurses5-dev which is not clear from native error reporting. For that reason this information will save some peoples time

johnnyman727 commented 8 years ago

@Student007 thanks for putting this together. There are actually a slew of dependencies needed; see https://github.com/tessel/openwrt-tessel/issues/42. Perhaps we should just have a section in the README that lists all of them?

Student007 commented 8 years ago

@johnnyman727 good point - didn't know about #42 Should I create a new PR and replace the suggested command with yours from #42 ? About the text - here is a suggestion for a new one (or should we remove the error part and create an issue?):

Dependencies

If you get an error like this (Ubuntu):

Build dependency: Please install ncurses. (Missing libncurses.so or ncurses.h)

You are required to install some source dependencies like libncurses5-dev and libncursesw5-dev!

In general we suggest to prepare a build-server with the following command:

sudo apt-get install make gcc g++ subversion unzip libncurses5-dev libncursesw5-dev ncurses-doc zlib1g-dev libssl-dev gettext --yes

Alternative text without the error (in the case we have an issue):

Setup Ubuntu

In general we suggest to prepare a build-server with the following command:

sudo apt-get install make gcc g++ subversion unzip libncurses5-dev libncursesw5-dev ncurses-doc zlib1g-dev libssl-dev gettext --yes
johnnyman727 commented 8 years ago

Should I create a new PR and replace the suggested command with yours from #42 ?

You can either make a new PR or create a new commit and rebase so that only that commit is in the history - your choice.

I think your alternative suggestion is better.

Student007 commented 8 years ago

ok - closed due to new PR https://github.com/tessel/openwrt-tessel/pull/51