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

Ubuntu Dependencies #50

Closed
wants to merge 1 commit into from

Conversation

Student007
Copy link
Member

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

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
Copy link
Contributor

@Student007 thanks for putting this together. There are actually a slew of dependencies needed; see #42. Perhaps we should just have a section in the README that lists all of them?

@Student007
Copy link
Member Author

@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
Copy link
Contributor

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 added a commit to Student007/openwrt-tessel that referenced this pull request Apr 7, 2016
Adds instructions to setup ubuntu:

### 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
```
Replaces tessel#50 as discussed.
@Student007
Copy link
Member Author

ok - closed due to new PR #51

@Student007 Student007 closed this Apr 7, 2016
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.

2 participants