All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] udev-cache: Remove unnecessary tar read from stdin
@ 2015-04-21 15:35 Ken Sharp
  2015-04-21 15:35 ` [PATCH 2/2] udev-cache: improve error handling Ken Sharp
  0 siblings, 1 reply; 2+ messages in thread
From: Ken Sharp @ 2015-04-21 15:35 UTC (permalink / raw)
  To: openembedded-core; +Cc: Ken Sharp

xargs already formats the inputs to tar correctly, so the
'-T -' argument to tar is unnecessary.

Signed-off-by: Ken Sharp <ken.sharp@ni.com>
---
 meta/recipes-core/udev/udev/udev-cache | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-core/udev/udev/udev-cache b/meta/recipes-core/udev/udev/udev-cache
index df97000..12f68fb 100644
--- a/meta/recipes-core/udev/udev/udev-cache
+++ b/meta/recipes-core/udev/udev/udev-cache
@@ -57,7 +57,7 @@ echo "Populating dev cache"
 	udevadm control --stop-exec-queue
 	sysconf_cmd > "$SYSCONF_TMP"
 	find /dev -xdev \( -type b -o -type c -o -type l \) | cut -c 2- \
-		| xargs tar cf "${DEVCACHE_TMP}" -T-
+		| xargs tar cf "${DEVCACHE_TMP}"
 	gzip < "${DEVCACHE_TMP}" > "$DEVCACHE"
 	rm -f "${DEVCACHE_TMP}"
 	mv "$SYSCONF_TMP" "$SYSCONF_CACHED"
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread

* [PATCH 2/2] udev-cache: improve error handling
  2015-04-21 15:35 [PATCH 1/2] udev-cache: Remove unnecessary tar read from stdin Ken Sharp
@ 2015-04-21 15:35 ` Ken Sharp
  0 siblings, 0 replies; 2+ messages in thread
From: Ken Sharp @ 2015-04-21 15:35 UTC (permalink / raw)
  To: openembedded-core; +Cc: Ken Sharp

If an error occurs while the udev cache is being populated, the system
is left in a state where udev is stopped.  Remedy this with a clean up
function to restart udev and remove any intermediate files.

Signed-off-by: Ken Sharp <ken.sharp@ni.com>
Reviewed-by: Ben Shelton <ben.shelton@ni.com>
---
 meta/recipes-core/udev/udev/udev-cache | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/meta/recipes-core/udev/udev/udev-cache b/meta/recipes-core/udev/udev/udev-cache
index 12f68fb..dcfff1c 100644
--- a/meta/recipes-core/udev/udev/udev-cache
+++ b/meta/recipes-core/udev/udev/udev-cache
@@ -51,9 +51,15 @@ fi
 [ "${VERBOSE}" == "no" ] || echo "found."
 echo "Populating dev cache"
 
+err_cleanup () {
+        echo "udev-cache: update failed!"
+        udevadm control --start-exec-queue
+	rm -f -- "$SYSCONF_TMP" "$DEVCACHE_TMP" "$DEVCACHE" "$SYSCONF_CACHED"
+}
+
 (
 	set -e
-	trap 'echo "udev-cache: update failed!"' EXIT
+	trap 'err_cleanup' EXIT
 	udevadm control --stop-exec-queue
 	sysconf_cmd > "$SYSCONF_TMP"
 	find /dev -xdev \( -type b -o -type c -o -type l \) | cut -c 2- \
-- 
1.9.1



^ permalink raw reply related	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2015-04-21 15:35 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-04-21 15:35 [PATCH 1/2] udev-cache: Remove unnecessary tar read from stdin Ken Sharp
2015-04-21 15:35 ` [PATCH 2/2] udev-cache: improve error handling Ken Sharp

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.