Merge lp:~sergiusens/phablet-tools/shotspaces into lp:phablet-tools

Proposed by Sergio Schvezov
Status: Merged
Approved by: Ricardo Salveti
Approved revision: 266
Merged at revision: 266
Proposed branch: lp:~sergiusens/phablet-tools/shotspaces
Merge into: lp:phablet-tools
Diff against target: 29 lines (+3/-3)
1 file modified
phablet-screenshot (+3/-3)
To merge this branch: bzr merge lp:~sergiusens/phablet-tools/shotspaces
Reviewer Review Type Date Requested Status
Ricardo Salveti (community) Approve
Review via email: mp+215771@code.launchpad.net

Commit message

phablet-screenshot: allow spaces in file names and path

To post a comment you must log in.
Revision history for this message
Ricardo Salveti (rsalveti) wrote :

Works fine, thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'phablet-screenshot'
--- phablet-screenshot 2014-04-14 19:22:18 +0000
+++ phablet-screenshot 2014-04-14 22:22:15 +0000
@@ -113,7 +113,7 @@
113 adb $ADBOPTS pull /tmp/mir_screencast_$1.$sfx ${PICDIR}/fb113 adb $ADBOPTS pull /tmp/mir_screencast_$1.$sfx ${PICDIR}/fb
114114
115 [ ! -e "${PICDIR}/fb" ] && echo "E: Capture failed!" && return115 [ ! -e "${PICDIR}/fb" ] && echo "E: Capture failed!" && return
116 convert $CONVERTOPTS $sfx:${PICDIR}/fb[0] $DST116 convert $CONVERTOPTS $sfx:${PICDIR}/fb[0] "$DST"
117}117}
118118
119get_device() {119get_device() {
@@ -130,7 +130,7 @@
130 adb $ADBOPTS pull /tmp/$capfile ${PICDIR}/$capfile130 adb $ADBOPTS pull /tmp/$capfile ${PICDIR}/$capfile
131131
132 [ ! -e "${PICDIR}/${capfile}" ] && echo "E: Capture failed!" && return132 [ ! -e "${PICDIR}/${capfile}" ] && echo "E: Capture failed!" && return
133 convert $CONVERTOPTS ${PICDIR}/$capfile $DST133 convert $CONVERTOPTS ${PICDIR}/$capfile "$DST"
134}134}
135135
136SHORTOPTS="hdr:s:z:"136SHORTOPTS="hdr:s:z:"
@@ -162,7 +162,7 @@
162done162done
163163
164[ $# -eq 0 ] && usage164[ $# -eq 0 ] && usage
165DST=$1165DST="$1"
166166
167check_prerequisites167check_prerequisites
168168

Subscribers

People subscribed via source and target branches