openstack/requirements
Code Issues Proposed changes

Avoid duplicate pep8 generation

Browse Source 
pep8 requirements were always written twice to the
updated requirements. Simply preserve the original pep8
line, like the source code comment mentioned.

Change-Id: I6b23f756594feeea0a5dbe14c58be981501bc5a4
This commit is contained in:
Dirk Mueller 2013-05-31 14:15:20 +02:00
parent 1e5f2fae72
commit 7c62ae57b7
1 changed files with 5 additions and 3 deletions

8
update.py

@ -72,13 +72,15 @@ def _copy_requires(req, source_path, dest_dir):
new_reqs.write("# This file is managed by openstack-depends\n")
for old_require in dest_keys:
# Special cases:
# versions of our stuff from tarballs.openstack.org are ok
# projects need to align pep8 version on their own time
if "pep8" in old_require:
new_reqs.write("%s\n" % dest_reqs[old_require])
continue
# versions of our stuff from tarballs.openstack.org are ok
if old_require in source_reqs or \
"http://tarballs.openstack.org" in old_require:
new_reqs.write("%s\n" % source_reqs[old_require])
if "pep8" in old_require:
new_reqs.write("%s\n" % dest_reqs[old_require])
def main(argv):