opendev/system-config
Code Issues Proposed changes

Merge "Fix line parsing in requirements check"

Browse Source
This commit is contained in:
Jenkins 2013-08-16 22:58:35 +00:00 committed by Gerrit Code Review
commit ba5ef099f8
1 changed files with 2 additions and 1 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,7 +43,8 @@ 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):
line = line[:line.find('#')] if '#' in line:
line = line[:line.find('#')]
line = line.strip() line = line.strip()
if (not line or if (not line or
line.startswith('http://tarballs.openstack.org/') or line.startswith('http://tarballs.openstack.org/') or