#56 Try to fix arguments to this new send_fedmsg function.
Closed 4 years ago by ausil. Opened 4 years ago by ralph.
ralph/releng master  into  master

file modified
+2 -2
@@ -55,8 +55,8 @@ 

      echo "$(date --utc) build${DIST}: ${message}" | tee -a "${logfile}" >&2

  }

  

- fedmsg_json_start=$(printf '{"log": "start", "branch": "%s", "arch": "%s"}' "$BRANCHED" "$ARCH")

- fedmsg_json_done=$(printf '{"log": "done", "branch": "%s", "arch": "%s"}' "$BRANCHED" "$ARCH")

+ fedmsg_json_start=\'$(printf '{"log": "start", "branch": "%s", "arch": "%s"}' "$BRANCHED" "$ARCH")\'

+ fedmsg_json_done=\'$(printf '{"log": "done", "branch": "%s", "arch": "%s"}' "$BRANCHED" "$ARCH")\'

  

  FEDMSG_MODNAME="compose"

  FEDMSG_CERTPREFIX="bodhi"

file modified
+2 -2
@@ -24,8 +24,8 @@ 

  

  FEDMSG_MODNAME="compose"

  FEDMSG_CERTPREFIX="bodhi"

- fedmsg_json_start=$(printf '{"log": "start", "branch": "%s"}' "${BRANCH}")

- fedmsg_json_done=$(printf '{"log": "done", "branch": "%s"}' "${BRANCH}")

+ fedmsg_json_start=\'$(printf '{"log": "start", "branch": "%s"}' "${BRANCH}")\'

+ fedmsg_json_done=\'$(printf '{"log": "done", "branch": "%s"}' "${BRANCH}")\'

  . ./scripts/fedmsg-functions.sh

  

  FULLPATH=$(pwd)

no initial comment

There are apparently still more bugs in the way this all got reorganized into scripts/fedmsg-functions.sh.

See https://lists.fedoraproject.org/archives/list/releng-cron%40lists.fedoraproject.org/message/W4QSEES43EM5LQGZYGMVC7WSAOCV35RB/ for an example of a failure.

This is my best bet at fixing it.

This can not be merged as the commit is not signed off on