Closed ynott closed 2 years ago
Thank you for contribution!
But, I can't agree to copy includes
and gst-plugins
from host Jetson directories.
Because it is not always installed DeepStream on the host in case of using Docker, and also it will not depends on the host libraries to run NGC containers for L4T in the nearly future release JetPack 5.x.
I want to use deepstream-l4t:6.0.1-samples
which is including that two directories rather than 6.0.1-base
.
Although it is increased the size of container using for building image, I will add some commands for removing unnecessary files sometime...
What do you think about this change?
Because it is not always installed DeepStream on the host in case of using Docker,
That is nice point of view and it's make sense. It's ok to include sources to this repository.
I will add some commands for removing unnecessary files sometime... What do you think about this change?
It is not a problem.
Do I cherry-pick from this branch and merge it with the feature-add-dockerfile branch?
Terrible. Image size has increased to 4.3 GB.
@takutoi
Aside from the large Docker image size, it is now buildable. Try the test.
OK, I confirmed to build and run app without problem. Thanks for your working. As a side note, it has 2.57GB image size on my TX2 env.
feature-add-dockerfile branch contained
includes
andgst-plugins
of Deepstream-6.0 source code and was double managed.Therefore, I added a build script called
docker-build.sh
that creates Docker images. This shell script pre-copies the required source code of Deepstream-6.0 before the Docker build.Also, the setup.sh could not be executed because the sudo command was not present in the Docker image, so it was run without sudo when running with root privileges.
And add a sample text
docker-build-run.txt
to exec commandbuild
andrun