tree 3fb068803b9141172a154df14760a66199cd088c
parent 7fe19f7b49f4366fcba12b8f3cbef40913370ba9
author Serge Bazanski <serge@nexantic.com> 1621427256 +0200
committer Sergiusz Bazanski <serge@nexantic.com> 1621505336 +0000

scripts: refactor for reliability

This is a small refactor/rewrite pass to scripts/create_container.sh and
scripts/bin/bazel.

First of all, it moves the IntelliJ aspect overriding logic from parsing
command line flags in scripts/bin/bazel to replace the path that the
aspect is loaded from to crafting the build container in a way that the
aspect is already located where it's expected.

This allows us to vastly simplify the scripts/bin/bazel wrapper, and
most notably fixes an issue where it wasn't able to handle Bazel flags
that contained spaces and were wrapped in ""/'' delimited strings.
Instead of adding more and more flag replacing logic in this script, we
got rid of that logic entirely, and now the only thing that the wrapper
script does is set the output root and run Bazel within the container.

At the same time, we also fix discovering newer IntelliJ versions. On my
machine, it was still discovering my 2019 installation and using an old
version of the aspect.

We also generally dust off the scripts/create_container.sh script by
ensuring all variables are always used within double quotes (the
previous implementation could've accidentally removed some directory from
the host if the user homedir contained spaces) and moving all of the
logic into a function that uses local variables.

Tested on my local workstation, IntelliJ sync works.

Change-Id: I63ae042ffe0fc7f384b98768ed66b7426b3f5df4
Reviewed-on: https://review.monogon.dev/c/monogon/+/63
Reviewed-by: Leopold Schabel <leo@nexantic.com>