diff options
author | Junichi Uekawa <dancer@netfort.gr.jp> | 2012-03-13 17:17:47 +0900 |
---|---|---|
committer | Junichi Uekawa <dancer@netfort.gr.jp> | 2012-03-13 17:25:12 +0900 |
commit | e990894f68de3c31ef8371fcf4baa301cfe7144a (patch) | |
tree | f2c017efcde3a2d4ce085dd3ad7c603d90a10aad /pdebuild | |
parent | 2ab7c50ef2a39961c0a037b8ae07bb7df2593693 (diff) | |
download | pbuilder-e990894f68de3c31ef8371fcf4baa301cfe7144a.tar pbuilder-e990894f68de3c31ef8371fcf4baa301cfe7144a.tar.gz |
fix logging message and conditionality to make it less lines.
I think we don't need the if.
I'm not quite convinced ||true to kill the error message is a good idea
Diffstat (limited to 'pdebuild')
-rw-r--r-- | pdebuild | 10 |
1 files changed, 4 insertions, 6 deletions
@@ -63,12 +63,10 @@ if [ "${USE_PDEBUILD_INTERNAL}" = 'yes' ]; then conditional_cp_a ../"$files" "${BUILDRESULT}" done conditional_cp_a ../${CHANGES} "${BUILDRESULT}" - if [ -n "${ADDITIONAL_BUILDRESULTS}" ]; then - echo "Saving additional results ${ADDITIONAL_BUILDRESULTS[*]}" - for FILE in "${ADDITIONAL_BUILDRESULTS[@]}"; do - conditional_cp_a "${FILE}" "${BUILDRESULT}" || true - done - fi + for files in "${ADDITIONAL_BUILDRESULTS[@]}"; do + log "I: Trying to save additional result ${files}" + conditional_cp_a "${files}" "${BUILDRESULT}" || true + done else log "E: BUILDRESULT=[$BUILDRESULT] is not a directory." exit 1 |