10 years, 1 month ago
(2010-11-24 05:36:19 UTC)
#1
Added -e, and removed context lines.
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
Verified. LGTM, thanks for the quick fix.
P.S: we may also need to add checking of return value by the 'sed -i' in
chromeos-factoryinstall ebuild.
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
> P.S: we may also need to add checking of return value by the 'sed -i' in
> chromeos-factoryinstall ebuild.
Yeah, I though of that too, however sed doesn't seem to provide meaningful
return values. I think adding the patches may be the way to go there as well.
Hung-Te
"grep -qs blah && sed -i command file" may be a simple solution
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
Issue 5332004: Fix factory patching
(Closed)
Created 10 years, 1 month ago by Nick Sanders
Modified 9 years, 7 months ago
Reviewers: Hung-Te
Base URL: ssh://git@gitrw.chromium.org:9222/crosutils.git
Comments: 0