Page MenuHomeDevCentral

Fix entrypoint.sh creating one folder named {plugins,embedded-db}
ClosedPublic

Authored by dereckson on Oct 14 2020, 02:21.
Tags
None
Referenced Files
F12277914: D2372.id.diff
Fri, Oct 17, 09:00
F12277291: D2372.id.diff
Fri, Oct 17, 04:18
Unknown Object (File)
Thu, Oct 16, 14:58
Unknown Object (File)
Thu, Oct 16, 13:23
Unknown Object (File)
Thu, Oct 16, 03:15
Unknown Object (File)
Fri, Oct 10, 11:02
Unknown Object (File)
Tue, Oct 7, 01:00
Unknown Object (File)
Fri, Oct 3, 22:45
Subscribers
None

Details

Summary

To allow spaces in variables like OPENFIRE_DATA_DIR, they have been enclosed
in quotes.

As the expression uses GNU coreutils mkdir {} syntax, put the {} in quotes
has broken the directory creation: instead to create /path/to/openfire/plugins
and /path/to/openfire/embedded-db, it created the exact quoted content.

That breaks new installations when directories don't exist.

This change fixes that regression. This is a follow-up for e76b3cfd9f47.

Test Plan
$ export OPENFIRE_DATA_DIR=/tmp/openfire
$ mkdir -p "${OPENFIRE_DATA_DIR}"/{plugins,embedded-db}
$ ls $OPENFIRE_DATA_DIR
embedded-db  plugins

Diff Detail

Repository
rDO Docker image for Openfire
Lint
Lint Not Applicable
Unit
Tests Not Applicable