Merge lp:~sylvain-pineau/checkbox/fix-1475539 into lp:checkbox

Proposed by Sylvain Pineau
Status: Merged
Approved by: Zygmunt Krynicki
Approved revision: 4203
Merged at revision: 4208
Proposed branch: lp:~sylvain-pineau/checkbox/fix-1475539
Merge into: lp:checkbox
Diff against target: 13 lines (+2/-2)
1 file modified
providers/plainbox-provider-sru/units/sru.pxu (+2/-2)
To merge this branch: bzr merge lp:~sylvain-pineau/checkbox/fix-1475539
Reviewer Review Type Date Requested Status
Zygmunt Krynicki (community) Approve
Review via email: mp+285191@code.launchpad.net

Description of the change

Fixes the linked bug

To post a comment you must log in.
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

+1, Whee for having after :)

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'providers/plainbox-provider-sru/units/sru.pxu'
2--- providers/plainbox-provider-sru/units/sru.pxu 2016-01-27 03:13:03 +0000
3+++ providers/plainbox-provider-sru/units/sru.pxu 2016-02-05 14:10:15 +0000
4@@ -16,8 +16,8 @@
5 plugin: attachment
6 category_id: 2013.com.canonical.plainbox::graphics
7 id: screenshot_opencv_validation.jpg
8-depends: graphics/screenshot_opencv_validation
9-command: base64 ${PLAINBOX_SESSION_SHARE}/screenshot_opencv_validation.jpg
10+after: graphics/screenshot_opencv_validation
11+command: [ -f ${PLAINBOX_SESSION_SHARE}/screenshot_opencv_validation.jpg ] && base64 ${PLAINBOX_SESSION_SHARE}/screenshot_opencv_validation.jpg
12 _description: Attaches the screenshot captured in graphics/screenshot_opencv_validation.
13 _summary: Attach the screenshot captured for the automatically validated screenshot test
14

Subscribers

People subscribed via source and target branches