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

Issue 3997001: common.sh: switch out echo -ne for printf and fix incorrect offset (Closed)

Created:
10 years, 2 months ago by Will Drewry
Modified:
9 years, 7 months ago
Reviewers:
Nick Sanders, adlr
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

common.sh: switch out echo -ne for printf and fix incorrect offset echo -ne isn't portable but printf with octals is so this makes the switch. In addition, the current offset is off by 3 and is updating the UUID space. This was from style cleanup and all functional test passed - of course. This change fixes that too. TEST=built and installed and checked ro enforcement works BUG=chromium-os:7972 Change-Id: Ifb3cb2c6f3219af819baff5750453439e1ba6edf

Patch Set 1 #

Patch Set 2 : ugh and fix a last minute bug! #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -4 lines) Patch
M common.sh View 1 2 chunks +4 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Will Drewry
10 years, 2 months ago (2010-10-20 21:54:32 UTC) #1
Will Drewry
Realized I introduced a last minute bug when I shuffled around the prior change. Doesn't ...
10 years, 2 months ago (2010-10-20 22:38:54 UTC) #2
adlr
10 years, 2 months ago (2010-10-20 22:41:07 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698