Merge ~sylvain-pineau/checkbox-core-snap:fix_wrapper_common_runtime_var into checkbox-core-snap:master

Proposed by Sylvain Pineau
Status: Merged
Approved by: Jonathan Cave
Approved revision: 2c8f4121716da8ec1582b62f99dc59219da4b497
Merged at revision: c6f0bf985947e4d1492bb928b59e1836913aea13
Proposed branch: ~sylvain-pineau/checkbox-core-snap:fix_wrapper_common_runtime_var
Merge into: checkbox-core-snap:master
Diff against target: 10 lines (+1/-1)
1 file modified
config/wrapper_common (+1/-1)
Reviewer Review Type Date Requested Status
Jonathan Cave (community) Approve
Review via email: mp+381978@code.launchpad.net

Description of the change

just the addition of $() around the command

To post a comment you must log in.
Revision history for this message
Jonathan Cave (jocave) wrote :

+1, thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/config/wrapper_common b/config/wrapper_common
2index 5427f30..cc48466 100644
3--- a/config/wrapper_common
4+++ b/config/wrapper_common
5@@ -35,4 +35,4 @@ fi
6
7 # make sure we always know where the content snap is especially for classic
8 # checkbox snaps where the checkbox-runtime bind mount does not exist
9-export CHECKBOX_RUNTIME=findmnt -oTARGET -n $(findmnt -oSOURCE -n $RUNTIME) | grep -P "^\/snap\/checkbox(|\d{2})\/"
10+export CHECKBOX_RUNTIME=$(findmnt -oTARGET -n $(findmnt -oSOURCE -n $RUNTIME) | grep -P "^\/snap\/checkbox(|\d{2})\/")

Subscribers

People subscribed via source and target branches