* [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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox