]> code.ossystems Code Review - openembedded-core.git/commitdiff
populate-volatile.sh: don't spawn background process at rootfs time
authorChen Qi <Qi.Chen@windriver.com>
Sat, 27 Jul 2013 14:21:57 +0000 (22:21 +0800)
committerRichard Purdie <richard.purdie@linuxfoundation.org>
Fri, 16 Aug 2013 10:14:14 +0000 (11:14 +0100)
If we're building a read-only rootfs, we'll get the following error now
and then.

    tar: .: file changed as we read it

The root cause is that we spawn background process at rootfs time.
When the tar command is running, it's possible that files under rootfs
are changed by background processes, thus this error.

[YOCTO #4937]

Signed-off-by: Chen Qi <Qi.Chen@windriver.com>
Signed-off-by: Saul Wold <sgw@linux.intel.com>
meta/recipes-core/initscripts/initscripts-1.0/populate-volatile.sh

index 6b3830883b136403b09cd5f7f4f697ffed45f0f8..0ff0e1af7f20e430d38ae96ec636c055b532f8c7 100755 (executable)
@@ -42,7 +42,7 @@ create_file() {
                        # but these failures should not be logged to make sure the do_rootfs
                        # process doesn't fail. This does no harm, as this script will
                        # run on target to set up the correct files and directories.
-                       eval $EXEC > /dev/null 2>&1 &
+                       eval $EXEC > /dev/null 2>&1
                fi
        }
 }
@@ -87,7 +87,7 @@ link_file() {
        else
                # For the same reason with create_file(), failures should
                # not be logged.
-               eval $EXEC > /dev/null 2>&1 &
+               eval $EXEC > /dev/null 2>&1
        fi
 }
 
@@ -156,7 +156,7 @@ apply_cfgfile() {
                [ "${TTYPE}" = "l" ] && {
                        TSOURCE="$TLTARGET"
                        [ "${VERBOSE}" != "no" ] && echo "Creating link -${TNAME}- pointing to -${TSOURCE}-."
-                       link_file "${TSOURCE}" "${TNAME}" &
+                       link_file "${TSOURCE}" "${TNAME}"
                        continue
                }