Code review comment for lp:~robru/phablet-tools/citrain-on-rtm

hmm, this perl command prints nothing on a devel-proposed install ...

phablet@ubuntu-phablet:~$ system-image-cli -i
current build number: 244
device name: mako
channel: ubuntu-touch/devel-proposed
alias: ubuntu-touch/utopic-proposed
last update: 2014-09-17 09:24:18
version version: 244
version ubuntu: 20140917
version device: 20140917
version custom: mako-0.5
phablet@ubuntu-phablet:~$ system-image-cli -i | perl -lne 'm|^channel.*/(.*)/.*$| && print "$1"'
phablet@ubuntu-phablet:~$

how about instead something like:

DISTRO=""
if system-image-cli -i | grep ^channel: |grep -q rtm; then
    DISTRO="ubuntu-rtm/"
fi

phablet-config writable-image --ppa $PPA/${DISTRO}${SILO}

(also note that this phablet-config command will fail if you dont hand over the sudo password with the -r or --remotepassword options, you should add a citrain option to hand this through to the phablet-config command)