Comment 8 for bug 144228

Revision history for this message
Kazuhiko (kazuhiko) wrote :

Petri:
I just looked at your code and it should be good to prevent this issue.
My proposal,"mode change to 0700 in buildout.rmtree", is required to save the case where the directory was *already* polluted by this issue, i.e. the the mode of the destination directory of a symlink was alreay changed to 0600 that raises exception when buildout tries to delete its contents.

So if we only care further buildout results that are created the buildout with this fix, it is not mandatory. But if we care existing buildout results that are created by buildout without this fix, it will make some people happy.