Skip to content
This repository was archived by the owner on Apr 3, 2018. It is now read-only.

Comments

doc: developer: add some introductory developer docs#643

Merged
sboeuf merged 2 commits intocontainers:masterfrom
grahamwhaley:20180227_add_devdocs
Mar 2, 2018
Merged

doc: developer: add some introductory developer docs#643
sboeuf merged 2 commits intocontainers:masterfrom
grahamwhaley:20180227_add_devdocs

Conversation

@grahamwhaley
Copy link
Contributor

There was little info n how to get started with a build/install/run of virtcontainers.
Add some minimal introductory docs.

Graham whaley added 2 commits February 27, 2018 18:16
There was little information on how to get started with
vc development - any prerequisites, how to build/install/test.
Add some docs to start detailing this.

Fixes: containers#639

Signed-off-by: Graham whaley <graham.whaley@intel.com>
The setup script has some prereqs that it does not check for:
- GOPATH set
- docker and git installed and in the path

Signed-off-by: Graham whaley <graham.whaley@intel.com>
@grahamwhaley
Copy link
Contributor Author

This is somewhat minimal, and feedback welcome if there is stuff we should add now.
I also added a few extra checks to the setup script, as they were easy and it bombed if you don't have them.

@sameo
Copy link
Collaborator

sameo commented Feb 27, 2018

LGTM

Approved with PullApprove Approved with PullApprove

@sboeuf
Copy link
Collaborator

sboeuf commented Mar 2, 2018

LGTM

@sboeuf sboeuf merged commit f4e3d70 into containers:master Mar 2, 2018
@sboeuf sboeuf removed the review label Mar 2, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants