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

Issue 5332004: Fix factory patching (Closed)

Created:
10 years, 1 month ago by Nick Sanders
Modified:
9 years, 7 months ago
Reviewers:
Hung-Te
CC:
chromium-os-reviews_chromium.org, Mandeep Singh Baines, anush, sosa
Visibility:
Public.

Description

Fix factory patching BUG=7535 TEST=build factory, no errors, verify patches. Change-Id: I480633cc7a0824cd2dfab82acd6a82f952cd7daf

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+5 lines, -21 lines) Patch
M mod_for_factory_scripts/200patchInitScript View 2 chunks +4 lines, -20 lines 0 comments Download
M mod_for_factory_scripts/factory_setup.sh View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 4 (0 generated)
Nick Sanders
Added -e, and removed context lines.
10 years, 1 month ago (2010-11-24 05:36:19 UTC) #1
Hung-Te
Verified. LGTM, thanks for the quick fix. P.S: we may also need to add checking ...
10 years, 1 month ago (2010-11-24 05:44:07 UTC) #2
Nick Sanders
> P.S: we may also need to add checking of return value by the 'sed ...
10 years, 1 month ago (2010-11-24 06:04:08 UTC) #3
Hung-Te
10 years, 1 month ago (2010-11-24 06:20:16 UTC) #4
"grep -qs blah && sed -i command file" may be a simple solution

Powered by Google App Engine
This is Rietveld 408576698