From 265588e4f42fa04923476b118334db80b78a3c8c Mon Sep 17 00:00:00 2001 From: David James Date: Fri, 30 Jul 2010 10:58:40 -0700 Subject: [PATCH] Add newline after CROS_WORKON_COMMIT="" line. Currently, cros_mark_as_stable.py does not add a newline after its CROS_WORKON_COMMIT="" line. This works fine for 9999 ebuilds that have a newline after the EAPI="" line, but doesn't work for ebuilds that don't do this. This should fix the preflight queue build. TEST=Ran cros_mark_as_stable.py on the update engine and verified that the CROS_WORKON_COMMIT="" line was not joined together with the next line. BUG=none Review URL: http://codereview.chromium.org/3036029 --- cros_mark_as_stable.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cros_mark_as_stable.py b/cros_mark_as_stable.py index e71813b022..2d9bd9e3af 100755 --- a/cros_mark_as_stable.py +++ b/cros_mark_as_stable.py @@ -282,7 +282,7 @@ class EBuildStableMarker(object): elif line.startswith('EAPI'): # Always add new commit_id after EAPI definition. redirect_file.write(line) - redirect_file.write('CROS_WORKON_COMMIT="%s"' % commit_id) + redirect_file.write('CROS_WORKON_COMMIT="%s"\n' % commit_id) elif not line.startswith('CROS_WORKON_COMMIT'): # Skip old CROS_WORKON_COMMIT definition. redirect_file.write(line)