Merge lp:~canonical-platform-qa/qakit/change_previous_dir into lp:qakit

Proposed by Sergio Cazzolato
Status: Merged
Approved by: Max Brustkern
Approved revision: 195
Merged at revision: 195
Proposed branch: lp:~canonical-platform-qa/qakit/change_previous_dir
Merge into: lp:qakit
Diff against target: 14 lines (+2/-2)
1 file modified
disk_footprint/iso_footprint.sh (+2/-2)
To merge this branch: bzr merge lp:~canonical-platform-qa/qakit/change_previous_dir
Reviewer Review Type Date Requested Status
Max Brustkern (community) Approve
Review via email: mp+306807@code.launchpad.net

Commit message

Change to detect correctly the previous results dir for iso footprint check

To post a comment you must log in.
Revision history for this message
Max Brustkern (nuclearbob) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'disk_footprint/iso_footprint.sh'
2--- disk_footprint/iso_footprint.sh 2016-09-21 21:14:10 +0000
3+++ disk_footprint/iso_footprint.sh 2016-09-26 21:26:16 +0000
4@@ -87,8 +87,8 @@
5 PREVDIR=$OUTDIR
6 fi
7
8-BUILDID_PREV=$(basename $(ls -1 "$PREVDIR/"|grep -B1 -w "$BUILDID"|tail -2|head -1) .manifest)
9-MANIFEST_PREV="$PREVDIR/$BUILDID_PREV/${BUILDID_PREV}.manifest"
10+BUILDID_PREV=$(basename $PREVDIR)
11+MANIFEST_PREV="$PREVDIR/${BUILDID_PREV}.manifest"
12
13 if [ ! -f "$MANIFEST_PREV" ]; then
14 BUILDID_PREV=""

Subscribers

People subscribed via source and target branches

to all changes: