Skip to content

Update Dockerfile#3743

Closed
s22527119-star wants to merge 1 commit into
openimsdk:mainfrom
s22527119-star:patch-2
Closed

Update Dockerfile#3743
s22527119-star wants to merge 1 commit into
openimsdk:mainfrom
s22527119-star:patch-2

Conversation

@s22527119-star

Copy link
Copy Markdown

🅰 Please add the issue ID after "Fixes #"

Fixes #

@pull-request-size pull-request-size Bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label Jun 16, 2026
@github-actions

Copy link
Copy Markdown
Contributor

💕 Thank you for your contribution and please kindly read and sign our CLA. CLA Docs


I have read the CLA Document and I hereby sign the CLA


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@withchao

Copy link
Copy Markdown
Contributor

Thanks for the contribution.

We cannot accept this PR because the current build/images/openim-server/Dockerfile is designed as a shared, parameterized Dockerfile for multiple OpenIM services. CMD_PATH and BINARY_NAME are provided by docker-compose.build.yml and build/build.sh so the same Dockerfile can build each service image.

This change hardcodes a single build target and output name, which would break the multi-service image build flow. It also introduces invalid Docker/shell behavior in the non-release branch.

If there is a specific build issue you are trying to fix, please open a new PR with a clear description, reproduction steps, and a change that preserves the existing CMD_PATH / BINARY_NAME behavior.

Closing this PR for now.

@withchao withchao closed this Jun 26, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants