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 /pbuilder-buildpackage | |
| 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 'pbuilder-buildpackage')
| -rwxr-xr-x | pbuilder-buildpackage | 10 |
1 files changed, 4 insertions, 6 deletions
diff --git a/pbuilder-buildpackage b/pbuilder-buildpackage index f8cb5ae..45b1177 100755 --- a/pbuilder-buildpackage +++ b/pbuilder-buildpackage @@ -171,12 +171,10 @@ if [ -d "${BUILDRESULT}" ]; then cp -p ${FILE} "${BUILDRESULT}" || true fi done - if [ -n "${ADDITIONAL_BUILDRESULTS}" ]; then - echo "Saving additional results ${ADDITIONAL_BUILDRESULTS[*]}" - for FILE in "${ADDITIONAL_BUILDRESULTS[@]}"; do - cp -a "${BUILDPLACE}/tmp/buildd/"*"/${FILE}" "${BUILDRESULT}" || true - done - fi + for FILE in "${ADDITIONAL_BUILDRESULTS[@]}"; do + log "I: Trying to save additional result ${FILE}" + cp -a "${BUILDPLACE}/tmp/buildd/"*"/${FILE}" "${BUILDRESULT}" || true + done else log "E: BUILDRESULT=[$BUILDRESULT] is not a directory." fi |