launchpad-reviewers team mailing list archive

[Steve Kowalik <stevenk@xxxxxxxxxx>]

The proposal to merge lp:~stevenk/launchpad/buildout-change-yui into lp:launchpad has been updated.

Description changed to:

I have hit this bug a few times during the Thunderdome. What happens is that the yui top-level directory is removed and the yui version that is specified in versions.cfg is unpacked. The last thing the buildout recipe does is symlink icing/yui to yui/<unpacked yui directory>. The problem is if icing/yui already exists, ln will refuse to replace it, and buildout doesn't show the error.

This causes another errors further on in the build chain since the icing/yui symlink has not been updated and now points to an unpacked yui directory that doesn't exist.

For more details, see:
https://code.launchpad.net/~stevenk/launchpad/buildout-change-yui/+merge/88457
-- 
https://code.launchpad.net/~stevenk/launchpad/buildout-change-yui/+merge/88457
Your team Launchpad code reviewers is requested to review the proposed merge of lp:~stevenk/launchpad/buildout-change-yui into lp:launchpad.