diff options
| author | Luke Shumaker <lukeshu@lukeshu.com> | 2018-09-20 14:52:49 -0400 |
|---|---|---|
| committer | Luke Shumaker <lukeshu@lukeshu.com> | 2018-09-20 14:52:49 -0400 |
| commit | 793cd0dfabcc081251b347f2a4fb2d3e482a1b0c (patch) | |
| tree | bd8b664bf3c49b35984803b8b7c15b8d525a82c4 | |
| parent | d94e39da8c1423edce2a72bfaf991ef25cdd76a6 (diff) | |
librerelease: Run the HOOKs with `eval` rather than `bash -c`
This should give the hooks read-only access to whatever variables are set.
This includes both the configuration variables (like REPODEST), and local
instance variables (like REPODEST_port).
| -rwxr-xr-x | src/abslibre-tools/librerelease | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/src/abslibre-tools/librerelease b/src/abslibre-tools/librerelease index 3a987b2..be75ab7 100755 --- a/src/abslibre-tools/librerelease +++ b/src/abslibre-tools/librerelease @@ -243,8 +243,8 @@ clean() { release_packages() { if [[ -n $HOOKPRERELEASE ]]; then msg "Running HOOKPRERELEASE..." - plain '%s' "${HOOKPRERELEASE}" - bash -c "${HOOKPRERELEASE}" + plain '%s' "$HOOKPRERELEASE" + (eval -- "$HOOKPRERELEASE") fi lock 8 "${WORKDIR}/staging.lock" \ @@ -288,8 +288,8 @@ release_packages() { if [[ -n $HOOKPOSTRELEASE ]]; then msg "Running HOOKPOSTRELEASE..." - plain '%s' "${HOOKPOSTRELEASE}" - bash -c "${HOOKPOSTRELEASE}" + plain '%s' "$HOOKPOSTRELEASE" + (eval -- "$HOOKPOSTRELEASE") fi return $EXIT_SUCCESS |