Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(330)

Unified Diff: utility/dev_debug_vboot

Issue 6840005: Put the final output of dev_debug_vboot under /var/log/ (Closed) Base URL: ssh://gitrw.chromium.org:9222/vboot_reference.git@master
Patch Set: Created 9 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utility/dev_debug_vboot
diff --git a/utility/dev_debug_vboot b/utility/dev_debug_vboot
index b86aafbd1b3df2c9c2cd0b58c33f9dcafc4ce1d6..c098251010fd5049b61555bd92cf9bcd29886377 100755
--- a/utility/dev_debug_vboot
+++ b/utility/dev_debug_vboot
@@ -14,9 +14,7 @@ PATH=/bin:/sbin:/usr/bin:/usr/sbin
TMPDIR=$(mktemp -d /tmp/debug_vboot_XXXXXXXXX)
LOGFILE=noisy.log
-# The public file must live directly in /tmp, not in a subdirectory.
-# See http://crosbug.com/8947
-PUBLOGFILE=/tmp/debug_vboot_noisy.log
+PUBLOGFILE=/var/log/debug_vboot_noisy.log
# TODO(wfrichar): Need to support ARM. The hard disk path is likely different.
# We can use 'crossystem arch' to distinguish between x86 and ARM.
@@ -24,8 +22,7 @@ HD=/dev/sda
cleanup() {
if [ -z "${USE_EXISTING:-}" ]; then
- # See http://crosbug.com/8947
- cp --no-target-directory --remove-destination "${LOGFILE}" "${PUBLOGFILE}"
+ cp -f "${LOGFILE}" "${PUBLOGFILE}"
info "exporting log file as ${PUBLOGFILE}"
fi
if [ -n "${CLEANUP:-}" ]; then
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698