Skip to content

Changed KNoT CLI commands to prevent lack of permissions in the Docker container #52

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.

Sign up for GitHub

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

Open
k-cabral wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Changed KNoT CLI commands to prevent lack of permissions in the Docker container #52

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
8ce4fd1
fixed docker instructions to prevent issues with permissions
k-cabral Jan 10, 2020
feefce4
made prompt format consistent with quickstart
k-cabral Jan 10, 2020
18dfe04
made section about compiling with docker clearer
k-cabral Jan 10, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions doc/quick-start/qs-guide.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -130,13 +130,13 @@ Compiling project

.. code-block:: bash

[user@container] $ knot make --board dk
[user@container] $ sudo -E knot make --board dk

- If using the `KNoT Dongle <https://docs.zephyrproject.org/latest/boards/arm/nrf52840_pca10059/doc/index.html>`_:

.. code-block:: bash

[user@container] $ knot make --board dongle
[user@container] $ sudo -E knot make --board dongle


Flashing board
Expand All @@ -146,7 +146,7 @@ Flashing board

.. code-block:: bash

[user@container] $ knot export output/
[user@container] $ sudo -E knot export output/

#. Install `nRF Connect <https://www.nordicsemi.com/Software-and-Tools/Development-Tools/nRF-Connect-for-desktop/Download>`_.

Expand Down
10 changes: 6 additions & 4 deletions doc/thing/thing-docker.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -42,9 +42,11 @@ by the tag you used.
Compile for your target board
-----------------------------

Inside the docker container, use the KNoT script to compile your project.

.. code-block:: bash

container> $ knot make -b {BOARD}
[user@container] $ sudo -E knot make -b {BOARD}

.. note:: Currently, KNoT support ``dk`` (nrf52840_pca10056) or ``dongle`` (nrf52840_pca10059) board. Replace {BOARD} with one of them.

Expand All @@ -64,7 +66,7 @@ Export the generated files to your project's directory

.. code-block:: bash

container> $ knot export /workdir/output
[user@container] $ sudo -E knot export /workdir/output

The generated files can now be flashed to your device by using the
`nRF Connect for Desktop <https://www.nordicsemi.com/?sc_itemid=%7B49D2264D-62FD-4C16-811F-88B477833C5D%7D>`_ and its
Expand Down Expand Up @@ -100,7 +102,7 @@ This will allow you to use the `--flash` flag to flash after building the projec

.. code-block:: bash

container> $ knot make -b {BOARD} --mcuboot
[user@container] $ sudo -E knot make -b {BOARD} --mcuboot

.. note:: Currently, KNoT support ``dk`` (nrf52840_pca10056) or ``dongle`` (nrf52840_pca10059) board. Replace {BOARD} with one of them.

Expand All @@ -115,6 +117,6 @@ To get a list of all available commands, run:

.. code-block:: bash

container> $ knot --help
[user@container] $ knot --help

More info is available at the `Thing CLI doc section <thing-cli.html>`_.