openimsdk / openim-docker

openim-docker configuration for deploying OpenIM. Provides a build solution for a stable distribution, as well as a docker compose deployment strategy
https://openim.io
Apache License 2.0
43 stars 54 forks source link

Bug: openim/openim-docker/scripts/lib/color.sh: line 23: conditional binary operator expected #32

Closed echo-music closed 6 months ago

echo-music commented 1 year ago

What happened?

docker 部署 在mac 终端执行 make init 报错

openim/openim-docker/scripts/lib/color.sh: line 23: conditional binary operator expected

What did you expect to happen?

openim/openim-docker/scripts/lib/color.sh: line 23: conditional binary operator expected

How can we reproduce it (as minimally and precisely as possible)?

openim/openim-docker/scripts/lib/color.sh: line 23: conditional binary operator expected

Anything else we need to know?

No response

version

```console $ {name} version # paste output here ```

Cloud provider

OS version

```console # On Linux: $ cat /etc/os-release # paste output here $ uname -a # paste output here # On Windows: C:\> wmic os get Caption, Version, BuildNumber, OSArchitecture # paste output here ```

Install tools

kubbot commented 1 year ago

Hello! Thank you for filing an issue.

If this is a bug report, please include relevant logs to help us debug the problem.

Join slack 🤖 to connect and communicate with our developers.

cubxxw commented 1 year ago

Maybe you can skip make init and just change your configuration file

You can check your Bash version by running the following command:

bash --version

-v is a unary operator introduced in bash 4.2 and later that checks whether a variable is set. In this code, if [[! -v COLOR_OPEN]]; The then line checks whether the COLOR_OPEN variable is set. If it is not set, then the line COLOR_OPEN=1 is run.

# shellcheck disable=SC2034
if [[ ! -v COLOR_OPEN ]]; then
    COLOR_OPEN=1
fi

https://github.com/openimsdk/open-im-server/blob/b72b72f74f20ba193635e9c5606215436552f69a/scripts/lib/color.sh#L22C1-L25C3

kubbot commented 10 months ago

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

cubxxw commented 10 months ago

The issue has been resolved now.

kubbot commented 8 months ago

This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

skiffer-git commented 6 months ago

The Docker deployment plan has been fully upgraded. Please refer to the README for the new scheme. The new scheme is simpler to use and supports Linux, Windows, and Mac.