From 9d2d2455a85b60cd50e5aa3ee57d82762869150c Mon Sep 17 00:00:00 2001 From: "James E. Blair" Date: Fri, 16 Aug 2013 13:43:59 -0700 Subject: [PATCH] Add final newline check for requirements Ensure that every line has a newline; particularly the last one. Change-Id: I8cfa194461946fb78dc9ea0aa51cf6158d5bb81c --- .../jenkins/files/slave_scripts/project-requirements-change.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/modules/jenkins/files/slave_scripts/project-requirements-change.py b/modules/jenkins/files/slave_scripts/project-requirements-change.py index 25d1c9cf42..56750f081d 100755 --- a/modules/jenkins/files/slave_scripts/project-requirements-change.py +++ b/modules/jenkins/files/slave_scripts/project-requirements-change.py @@ -43,6 +43,9 @@ class RequirementsList(object): if not os.path.exists(fn): return for line in open(fn): + if '\n' not in line: + raise Exception("Requirements file %s does not " + "end with a newline." % fn) if '#' in line: line = line[:line.find('#')] line = line.strip()