Merge lp://qastaging/~kalikiana/u1db-qt/hackingDocs into lp://qastaging/u1db-qt

Proposed by Cris Dywan
Status: Merged
Approved by: Cris Dywan
Approved revision: 115
Merged at revision: 127
Proposed branch: lp://qastaging/~kalikiana/u1db-qt/hackingDocs
Merge into: lp://qastaging/u1db-qt
Diff against target: 59 lines (+35/-18)
1 file modified
HACKING (+35/-18)
To merge this branch: bzr merge lp://qastaging/~kalikiana/u1db-qt/hackingDocs
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve
Nekhelesh Ramananthan (community) Approve
Benjamin Zeller Pending
Review via email: mp+205426@code.qastaging.launchpad.net

Commit message

Mention BUILD_DOC flag in HACKING file

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

28 +Build dependencies to have:
29 + ubuntu-sdk-libs-dev, dbus-test-runner, qtdeclarative5-test-plugin

Perhaps we could add a command here, sudo apt-get build-dep u1db-qt. This should automatically install all required dependencies.

The other sections such as retrieving source, building it, and testing by hand works as expected.

I haven't reviewed others such as Build Docs and Unit tests since I am not too familiar with them.

115. By Cris Dywan

Mention apt-get build-dep for installing dependencies

Revision history for this message
Nekhelesh Ramananthan (nik90) wrote :

yup that looks good.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: