opendev/system-config
Code Issues Proposed changes

Merge "Add final newline check for requirements"

Browse Source
This commit is contained in:
Jenkins 2013-08-24 15:35:08 +00:00 committed by Gerrit Code Review
commit 3cbfd13c56
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
modules/jenkins/files/slave_scripts/project-requirements-change.py
View File

@ -43,6 +43,9 @@ class RequirementsList(object):
if not os.path.exists(fn): if not os.path.exists(fn):
return return
for line in open(fn): 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: if '#' in line:
line = line[:line.find('#')] line = line[:line.find('#')]
line = line.strip() line = line.strip()