aboutsummaryrefslogtreecommitdiff
path: root/pdebuild
diff options
context:
space:
mode:
authorJunichi Uekawa <dancer@netfort.gr.jp>2012-03-13 17:17:47 +0900
committerJunichi Uekawa <dancer@netfort.gr.jp>2012-03-13 17:25:12 +0900
commite990894f68de3c31ef8371fcf4baa301cfe7144a (patch)
treef2c017efcde3a2d4ce085dd3ad7c603d90a10aad /pdebuild
parent2ab7c50ef2a39961c0a037b8ae07bb7df2593693 (diff)
downloadpbuilder-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--pdebuild10
1 files changed, 4 insertions, 6 deletions
diff --git a/pdebuild b/pdebuild
index ed8deb4..971a273 100644
--- a/pdebuild
+++ b/pdebuild
@@ -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