From a9f7eb4bab7110e9addd27dbe5639686e6b5cd2c Mon Sep 17 00:00:00 2001 From: Monty Taylor Date: Thu, 1 May 2014 13:41:34 -0700 Subject: [PATCH] Stop giving -f and -e lines a pass in requirements We had exclusions in the testing to allow for legacy issues and a migration path. That's not cutting it any more and has allowed a new -f line to slip in. Be more strict. Change-Id: I7c1debdd9307135efc68b402bf830a44c914d3fe --- .../files/slave_scripts/project-requirements-change.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/modules/openstack_project/files/slave_scripts/project-requirements-change.py b/modules/openstack_project/files/slave_scripts/project-requirements-change.py index b014152847..00691a5a90 100755 --- a/modules/openstack_project/files/slave_scripts/project-requirements-change.py +++ b/modules/openstack_project/files/slave_scripts/project-requirements-change.py @@ -99,9 +99,7 @@ class RequirementsList(object): line = line[:line.find('#')] line = line.strip() if (not line or - line.startswith('http://tarballs.openstack.org/') or - line.startswith('-e') or - line.startswith('-f')): + line.startswith('http://tarballs.openstack.org/')): continue req = pkg_resources.Requirement.parse(line) if (not ignore_dups and strict and req.project_name.lower()