Skip to content
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.

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

Docker: Refine the main dockerfile. v6.0.54 #3594

Merged
merged 3 commits into from
Jun 20, 2023

Conversation

winlinvip
Copy link
Member

@winlinvip winlinvip commented Jun 20, 2023

Occasionally, developers may wish to create the most recent SRS images or make changes to the SRS code and build images for distribution purposes.

This patch simplifies the SRS image building process for developers. Instead of using the previous command:

docker build -t srs -f trunk/Dockerfile .

Use the refined version:

docker build -t srs -f Dockerfile .

# Or
docker build -t srs .

Run SRS with the local image:

docker run --rm -it srs

Sometimes, developers may want to create the latest SRS image or modify SRS code and build an image for distribution.

Improving the Dockerfile can make it easier to compile the SRS image. The previous command was:

docker build -t srs -f trunk/Dockerfile .

This patch moves the Dockerfile to the root directory, so the command can be simpler:

docker build -t srs -f Dockerfile .

# Or
docker build -t srs .

After compiling the local image, you can run it directly.

docker run --rm -it srs

TRANS_BY_GPT3

@winlinvip winlinvip changed the title Docker: Refine the main dockerfile. Docker: Refine the main dockerfile. v6.0.54 Jun 20, 2023
@winlinvip winlinvip changed the title Docker: Refine the main dockerfile. v6.0.54 Docker: Refine the main dockerfile. v6.0.54 Jun 20, 2023
@winlinvip winlinvip merged commit 8c061fc into ossrs:develop Jun 20, 2023
@winlinvip winlinvip added the TransByAI Translated by AI/GPT. label Jul 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
TransByAI Translated by AI/GPT.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants