[OT-DOCS] border router build updates#203
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the OTBR build documentation to clone the repository recursively. The reviewer suggests retaining the --depth=1 flag alongside --recursive to perform a shallow clone, preventing slow and resource-intensive downloads of the entire commit history.
| 1. Clone the OTBR repository: | ||
| ``` | ||
| $ git clone --depth=1 https://github.com/openthread/ot-br-posix | ||
| $ git clone --recursive https://github.com/openthread/ot-br-posix |
There was a problem hiding this comment.
Cloning the repository recursively without --depth=1 will download the entire commit history of the repository and all of its submodules, which can be very slow and resource-intensive. It is recommended to keep --depth=1 to perform a shallow clone for faster downloads.
| $ git clone --recursive https://github.com/openthread/ot-br-posix | |
| $ git clone --recursive --depth=1 https://github.com/openthread/ot-br-posix |
There was a problem hiding this comment.
I think this suggestion makes sense, @jrhodie please make the update here and in the other file
|
|
||
| ``` | ||
| $ git clone --depth=1 https://github.com/openthread/ot-br-posix | ||
| $ git clone --recursive https://github.com/openthread/ot-br-posix |
There was a problem hiding this comment.
Cloning the repository recursively without --depth=1 will download the entire commit history of the repository and all of its submodules, which can be very slow and resource-intensive. It is recommended to keep --depth=1 to perform a shallow clone for faster downloads.
| $ git clone --recursive https://github.com/openthread/ot-br-posix | |
| $ git clone --recursive --depth=1 https://github.com/openthread/ot-br-posix |
| 1. Clone the OTBR repository: | ||
| ``` | ||
| $ git clone --depth=1 https://github.com/openthread/ot-br-posix | ||
| $ git clone --recursive https://github.com/openthread/ot-br-posix |
There was a problem hiding this comment.
I think this suggestion makes sense, @jrhodie please make the update here and in the other file
2 participants
No description provided.