8 +package_extract_file("ubuntu_deploy.sh", "/tmp/ubuntu_deploy.sh"); 9 +set_perm(0, 0, 0777, "/tmp/ubuntu_deploy.sh");
Do we really have /tmp/ access at recovery? Never tested, downloading the tarball to test.
55 +cat > ${WORKDIR}/ubuntu_deploy.sh << 'EOF' 56 +#!/sbin/sh
Any reason for it not to be a separated script? If separated, also make sure to add a header with copyright and such.
61 +phablet_home_bak=/data/phablet_home +network_settings_bak=/data/nm_connections
I'd prefer to use a common directory for backups, like /data/ubuntu_bak or similar, so we know where to find the files in case of a failure when flashing the image.
Rest looks fine, creating an image to test it.
« Back to merge proposal
8 +package_ extract_ file("ubuntu_ deploy. sh", "/tmp/ubuntu_ deploy. sh"); deploy. sh");
9 +set_perm(0, 0, 0777, "/tmp/ubuntu_
Do we really have /tmp/ access at recovery? Never tested, downloading the tarball to test.
55 +cat > ${WORKDIR} /ubuntu_ deploy. sh << 'EOF'
56 +#!/sbin/sh
Any reason for it not to be a separated script? If separated, also make sure to add a header with copyright and such.
61 +phablet_ home_bak= /data/phablet_ home settings_ bak=/data/ nm_connections
+network_
I'd prefer to use a common directory for backups, like /data/ubuntu_bak or similar, so we know where to find the files in case of a failure when flashing the image.
Rest looks fine, creating an image to test it.