RotherOSS / otobo

OTOBO is one of the most flexible web-based ticketing systems used for Customer Service, Help Desk, IT Service Management. https://otobo.io/
GNU General Public License v3.0
250 stars 72 forks source link

Do not use legacy version of the ENV command #3553

Closed bschmalhofer closed 3 months ago

bschmalhofer commented 3 months ago

Doing a build with Docker version 27.0.2 I noticed warnings like:

2 warnings found (use --debug to expand):

  • LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 25)
  • LegacyKeyValueFormat: "ENV key=value" should be used instead of legacy "ENV key value" format (line 26)

These were due to the lines:

grep -n ENV otobo.elasticsearch.dockerfile 25:ENV LC_ALL C.UTF-8 26:ENV LANG C.UTF-8

Note that there are no = sign between key and value.

The explanation of the warnings is at https://docs.docker.com/reference/build-checks/legacy-key-value-format/ . The documentation of the ENV command confirms that. See https://docs.docker.com/reference/dockerfile/#env .

Note that e.g. otobo.nginx-kerberos.dockerfile already used the = signs.

ENV SPNEGO_AUTH_COMMIT_ID=v1.1.1 ENV SPNEGO_AUTH_COMMIT_ID_FILE=1.1.1

TODO:

bschmalhofer commented 3 months ago

Done. The Docker image builds in the branches were fine. A run of the test suite in rel-10_0 was fine too. Closing this issue.