aboutsummaryrefslogtreecommitdiff
path: root/reproduce/analysis/bash/download-multi-try
diff options
context:
space:
mode:
authorMohammad Akhlaghi <mohammad@akhlaghi.org>2019-05-09 15:54:56 +0100
committerMohammad Akhlaghi <mohammad@akhlaghi.org>2019-05-09 15:54:56 +0100
commit0dd3569305596284ab8c168874e3c25faad0ef05 (patch)
tree59b1b60677c780d778046b43a45b79383039cc62 /reproduce/analysis/bash/download-multi-try
parent1b3be9f63e735902d3eef24f5ecac5ae71012602 (diff)
Metastore Git hooks properly written after its installation
Until now, in the recipe to build Metastore, we would store the current directory in a `current_dir' Bash variable before running `gbuild' to build Metastore. However, in this Makefile, we aren't using `.ONESHELL'. As a result, each un-quoted new-line character creates a new shell and the `current_dir' variable that we used afterwards was empty! This happened because until recently, Metastore was being built in `high-level.mk' (where `.ONESHELL' is activated, because we are using GNU Make to call that Makefile). But in `basic.mk', we are using the host's Make, which may not be GNU Make, so we can't use any GNU-only features. Also, this error was hard to notice, because we weren't using `&&' before writing the final target of Metastore! With this commit, both this issues are addressed: all the lines in the recipe are now quoted in the end with a backslash, and the steps before making the target are separated by `&&', not `;' (which doesn't cause a crash if the command fails). This fixes bug #56295.
Diffstat (limited to 'reproduce/analysis/bash/download-multi-try')
0 files changed, 0 insertions, 0 deletions