Add devcontainer build env#3277
Conversation
| @@ -0,0 +1,100 @@ | |||
| # Building SRT Using Devcontainer or Docker | |||
There was a problem hiding this comment.
i confirmed and verified the all following procedure.
just FYI, when we open the devcontainer via vscode 1st time, it will need to build to container image. so it takes a minutes to build the docker image for devcontainer. but from the 2nd time to open the devcontainer, it just binds the environment to the already-built image, so i does not take any time to start.
| ## Build Instructions | ||
|
|
||
| [Linux (Ubuntu/CentOS)](./docs/build/build-linux.md) | [Windows](./docs/build/build-win.md) | [macOS](./docs/build/build-macOS.md) | [iOS](./docs/build/build-iOS.md) | [Android](./docs/build/build-android.md) | [Package Managers](./docs/build/package-managers.md) | ||
| [Linux (Ubuntu/CentOS)](./docs/build/build-linux.md) | [Windows](./docs/build/build-win.md) | [macOS](./docs/build/build-macOS.md) | [iOS](./docs/build/build-iOS.md) | [Android](./docs/build/build-android.md) | [Package Managers](./docs/build/package-managers.md) | [Devcontainer/Docker](./docs/build/build-devcontainer.md) |
There was a problem hiding this comment.
since we talked about this is optional procedure, i added this procedure in the last.
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
fujitatomoya
force-pushed
the
add-devcontainer-build-env
branch
from
96d70e3 to
3627352
Compare
|
Just a question: what are requirements for the file location of the devcontainer configuration and the docker? Can they be freely moved elsewhere? |
| @@ -0,0 +1,19 @@ | |||
| { | |||
| "name": "SRT Development", | |||
| "dockerFile": "../docker/Dockerfile", | |||
There was a problem hiding this comment.
@ethouris .devcontainer/devcontainer.json needs to be statically there, and docker file is linked as relative path. to answer your question, no we cannot move those files freely. and i am not quite sure why this is a hard requirement... if that is we can never use the devcontainer or codespace...
|
@ethouris after the discussion with you, i changed the title on this issue. let's keep this to enable devcontainer environment for developers. i think this also makes a lot of sense for the dev env in these days to welcome developers and engineers. |
ethouris
added
Type: Enhancement
|
@ethouris friendly ping! is there anything needs to be done on my side? if that is so, please do not hesitate to do that 😄 |
|
Alright, just note that it is merged to "dev" branch. The 1.5.5 release is not out yet and more stability tests will have to be conducted on the "dev" branch before - after the release of course - it will become the new "master". |
2 participants
part of #3265, and
replaces #3275this still requires codespell script that is expected to be issued by user.