ta: login shell, filename expansion enabled 53/2253/2
authorAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Sun, 23 Feb 2020 15:26:18 +0000 (16:26 +0100)
committerAlexandru Avadanii <Alexandru.Avadanii@enea.com>
Sun, 23 Feb 2020 16:04:22 +0000 (17:04 +0100)
commitbb1b3d84cda3e7c4bf51427c0d07ef5ba78688de
treed9fedabea45a52c1e566cf09fd6eef28af0076de
parenta50cd8cb04472e00985bb3271fa8dfaddda41d6f
ta: login shell, filename expansion enabled

Recent global-jjb bumps introduced changes to python-tools-install.sh
which now disables filename expansion (globbing).
Since TA RPM merge / ISO build jobs combine python-tools-install.sh and
other sh script files into a single build step (i.e. resulting a single
sh script), this breaks globbing for the TA RPM/ISO upload handling.

Also, since python-tools-install.sh does not run as a login shell (`bash
-l`), ~/.profile is not sourced and PATH might be missing certain paths
like ~/.local/bin, where `lftools` is located, resulting in build
failures due to missing/not found `lftools`.

To mitigate the above issues:
1. split the offending build step into 2 separate steps
to allow spawning the second step under a login shell (without modifying
python-tools-install.sh);
2. Run the second step under a login shell, so PATH contains `lftools`
installed by pip;
3. Add `set +f` to enable globbing (this is not critical after splitting
the build step above, since python-tools-install.sh disabling globbing
will no longer affect the second step; but it should make it more
obvious that the script *requires* globbing);

Change-Id: I4a5e6a4a88a296cb21d68338f8e5a5b219076308
Signed-off-by: Alexandru Avadanii <Alexandru.Avadanii@enea.com>
jjb/akraino-templates/akraino-ta-common-macros.yaml
jjb/shell/ta-iso-deploy.sh
jjb/shell/ta-rpm-deploy.sh