Collective zc.buildout recipes

distutils recipes is not using correctly install-from-cache setting

Reported by aitzol on 2010-09-03
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
collective.buildout
Undecided
Unassigned

Bug Description

distutils recipe is getting self.install_from_cache setting from buildout['buildout'].get('install-from-cache'). buildout['buildout'] is a dict of strings, so values for keys are 'false','true','0'...

in line 110 the "if install_from_cache" is always true, because is never is an empty string. I don't know if it's posible to get the python boolean (not the string) for the settings, but if it's not posible the correct if statement is "if install_from_cache != 'false'"

I attach a patch file, but not sure about the format of the diff. This is the command I have used for generating the diff:

diff -u collective.recipe.distutils-0.1-py2.6.egg/collective/recipe/distutils/__init__.py collective.recipe.distutils-.1-py2.6.egg/collective/recipe/distutils/__init__.py.patched > __init__.py.patch

aitzol (anaberan) wrote :
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers