Correcting syntax when using if constructs in links_files().
[live-boot-grml.git] / scripts / live
index 08d6d74..fad7107 100755 (executable)
@@ -1485,7 +1485,8 @@ setup_unionfs ()
        then
                if [ -n "${PERSISTENT_READONLY}" ]
                then
-                       root_backing="/${rootmnt}/live/persistent/$(basename ${cowdevice})-root"
+                       mount -t tmpfs -o rw,noatime,mode=755 tmpfs "/cow"
+                       root_backing="${rootmnt}/live/persistent/$(basename ${cowdevice})-root"
                        mkdir -p ${root_backing}
                else
                        root_backing="/cow"
@@ -1505,6 +1506,7 @@ setup_unionfs ()
 
        rofscount=$(echo ${rofslist} |wc -w)
 
+       # XXX: we now ensure that there can only be one read-only filesystem. Should this be inside the EXPOSED_ROOT if?
        if [ ${rofscount} -ne 1 ]
        then
                panic "only one RO file system supported with exposedroot: ${rofslist}"
@@ -1532,21 +1534,25 @@ setup_unionfs ()
                #panic "unionmount does not support subunions (${cow_dirs})."
        fi
 
-       unionmountopts=""
-       unionmountpoint=""
-
        for dir in ${cow_dirs}; do
-               unionrw="/cow${dir}"
-               unionro="${rofs}${dir}"
                unionmountpoint="${rootmnt}${dir}"
-               mkdir -p ${unionrw}
+               mkdir -p ${unionmountpoint}
                if [ "${UNIONTYPE}" = "unionmount" ]
                then
                        # FIXME: handle PERSISTENT_READONLY
                        unionmountopts="-t ${cow_fstype} -o noatime,union,${cow_mountopt} ${cowdevice}"
                        mount_full $unionmountopts "${unionmountpoint}"
                else
-                       do_union ${root_backing} ${unionrw} ${unionro} ${unionmountpoint}
+                       cow_dir="/cow${dir}"
+                       rofs_dir="${rofs}${dir}"
+                       mkdir -p ${cow_dir}
+                       if [ -n "${PERSISTENT_READONLY}" ] && [ "${cowdevice}" != "tmpfs" ]
+                       then
+                               #mount -t tmpfs -o rw,noatime,mode=755 tmpfs "${cow_dir}"
+                               do_union ${unionmountpoint} ${cow_dir} ${root_backing} ${rofs_dir}
+                       else
+                               do_union ${unionmountpoint} ${cow_dir} ${rofs_dir}
+                       fi
                fi || panic "mount ${UNIONTYPE} on ${unionmountpoint} failed with option ${unionmountopts}"
        done
 
@@ -1558,6 +1564,34 @@ setup_unionfs ()
        mkdir -p "${rootmnt}/live"
        mount -t tmpfs tmpfs ${rootmnt}/live
 
+       live_rofs_list=""
+       # SHOWMOUNTS is necessary for custom mounts with the union option
+       if [ -n "${SHOWMOUNTS}" ] || ( [ -n "${PERSISTENT}" ] && [ -z "${NOPERSISTENT}" ] )
+       then
+               # XXX: is the for loop really necessary? rofslist can only contain one item (see above XXX about EXPOSEDROOT) and this is also assumed elsewhere above (see use of $rofs above).
+               for d in ${rofslist}
+               do
+                       live_rofs="${rootmnt}/live/${d##*/}"
+                       live_rofs_list="${live_rofs_list} ${live_rofs}"
+                       mkdir -p "${live_rofs}"
+                       case d in
+                               *.dir)
+                                       # do nothing # mount -o bind "${d}" "${live_rofs}"
+                                       ;;
+                               *)
+                                       case "${UNIONTYPE}" in
+                                               unionfs-fuse)
+                                                       mount -o bind "${d}" "${live_rofs}"
+                                                       ;;
+                                               *)
+                                                       mount -o move "${d}" "${live_rofs}"
+                                                       ;;
+                                       esac
+                                       ;;
+                       esac
+               done
+       fi
+
        # Adding other custom mounts
        if [ -n "${PERSISTENT}" ] && [ -z "${NOPERSISTENT}" ]
        then
@@ -1597,9 +1631,9 @@ setup_unionfs ()
                        [ "${DEBUG}" = "Yes" ] && cp ${include_list} ${persistent_backing}/${persistence_list}.${device_name}
                        while read source dest options # < ${include_list}
                        do
-                               if echo ${source} | grep -qe "^[[:space:]]*#"
+                               if echo ${source} | grep -qe "^[[:space:]]*\(#.*\)\?$"
                                then
-                                       # skipping commented line
+                                       # skipping empty or commented lines
                                        continue
                                fi
 
@@ -1622,7 +1656,7 @@ setup_unionfs ()
                                for opt in $(echo ${options} | tr ',' ' ');
                                do
                                        case "${opt}" in
-                                               "linkfiles")
+                                               linkfiles|union)
                                                        ;;
                                                *)
                                                        log_warning_msg "Skipping custom mount with unkown option: ${opt}"
@@ -1671,12 +1705,16 @@ setup_unionfs ()
                while read source dest options  # < ${custom_mounts}
                do
                        local opt_linkfiles=""
+                       local opt_union=""
                        for opt in $(echo ${options} | tr ',' ' ');
                        do
                                 case "${opt}" in
-                                       "linkfiles")
+                                       linkfiles)
                                                opt_linkfiles="yes"
                                                ;;
+                                       union)
+                                               opt_union="yes"
+                                               ;;
                                esac
                        done
 
@@ -1710,8 +1748,6 @@ setup_unionfs ()
                                fi
                        fi
 
-                       # FIXME: could we instead only save the aufs-diff in the persistent media? implications? What about when there's changes in the live image?
-
                        # if ${source} doesn't exist on our persistent media we
                        # bootstrap it with $dest from the live filesystem.
                        # this both makes sense and is critical if we're
@@ -1721,18 +1757,38 @@ setup_unionfs ()
                                if [ -n "${PERSISTENT_READONLY}" ] || [ -n "${opt_linkfiles}" ]
                                then
                                        continue
+                               elif [ -n "${opt_union}" ]
+                               then
+                                       # union's don't need to be bootstrapped
+                                       mkdir "${source}"
+                               else
+                                       # ensure that $dest is not copied *into* $source
+                                       mkdir -p "$(dirname ${source})"
+                                       cp -a "${dest}" "${source}"
                                fi
-                               # ensure that $dest is not copied *into* $source
-                               mkdir -p "$(dirname ${source})"
-                               cp -a "${dest}" "${source}"
                        fi
 
+                       rofs_dest_backing=""
+                       for d in ${live_rofs_list}
+                       do
+                               rofs_dest_backing="${d}/$(echo ${dest} | sed -e "s|${rootmnt}||")"
+                               if [ -d "${rofs_dest_backing}" ]
+                               then
+                                       break
+                               else
+                                       rofs_dest_backing=""
+                               fi
+                       done
+
                        if [ -z "${PERSISTENT_READONLY}" ]
                        then
                                if [ -n "${opt_linkfiles}" ]
                                then
                                        links_source="${source}"
                                        links_dest="${dest}"
+                               elif [ -n "${opt_union}" ]
+                               then
+                                       do_union ${dest} ${source} ${rofs_dest_backing}
                                else
                                        mount --bind "${source}" "${dest}"
                                fi
@@ -1743,10 +1799,9 @@ setup_unionfs ()
                                        dest="$(mktemp -d ${persistent_backing}/links_source-XXXXXX)"
                                        links_source="${dest}"
                                fi
-                               unionrw="$(echo ${dest} | sed -e "s|${rootmnt}|/cow/|")"
-                               mkdir -p ${unionrw}
-                               unionmountopts="noatime,${noxino_opt}dirs=${unionrw}=rw:${source}=${roopt}"
-                               do_union ${root_backing} ${unionrw} ${source} ${dest}
+                               cow_dir="$(echo ${dest} | sed -e "s|${rootmnt}|/cow/|")"
+                               mkdir -p ${cow_dir}
+                               do_union ${dest} ${cow_dir} ${source} ${rofs_dest_backing}
                        fi
 
                        if [ -n "${opt_linkfiles}" ]
@@ -1757,7 +1812,7 @@ setup_unionfs ()
                        PERSISTENCE_IS_ON="1"
                        export PERSISTENCE_IS_ON
                done < ${custom_mounts}
-               rm -f ${bindings}
+               rm -f ${custom_mounts}
 
                # Look for other snapshots to copy in
                try_snap "${root_snapdata}" "${rootmnt}" "ROOT"
@@ -1765,32 +1820,6 @@ setup_unionfs ()
                try_snap "${home_snapdata}" "${rootmnt}" "HOME" "/home"
        fi
 
-       if [ -n "${SHOWMOUNTS}" ]
-       then
-               for d in ${rofslist}
-               do
-                       mkdir -p "${rootmnt}/live/${d##*/}"
-
-                       case d in
-                               *.dir)
-                                       # do nothing # mount -o bind "${d}" "${rootmnt}/live/${d##*/}"
-                                       ;;
-
-                               *)
-                                       case "${UNIONTYPE}" in
-                                               unionfs-fuse)
-                                                       mount -o bind "${d}" "${rootmnt}/live/${d##*/}"
-                                                       ;;
-
-                                               *)
-                                                       mount -o move "${d}" "${rootmnt}/live/${d##*/}"
-                                                       ;;
-                                       esac
-                                       ;;
-                       esac
-               done
-       fi
-
        # shows cow fs on /cow for use by live-snapshot
        mkdir -p "${rootmnt}/live/cow"
        mount -o move /cow "${rootmnt}/live/cow" >/dev/null 2>&1 || mount -o bind /cow "${rootmnt}/live/cow" || log_warning_msg "Unable to move or bind /cow to ${rootmnt}/live/cow"