Code review comment for lp:~wgrant/bzr-builder/bug-1242796

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

On Wed, Aug 06, 2014 at 12:17:23PM -0000, William Grant wrote:
> William Grant has proposed merging lp:~wgrant/bzr-builder/bug-1242796 into lp:bzr-builder with lp:~wgrant/bzr-builder/bug-1350430 as a prerequisite.
>
> Commit message:
> Fix --no-build to leave a PACKAGE-VERSION directory rather than a TEMPLATE directory, returning to the 0.7.2 behaviour.
>
> Requested reviews:
> bzr-builder developers (bzr-builder-devs)
> Related bugs:
> Bug #1242796 in bzr-builder: "Fix destdir for recipe"
> https://bugs.launchpad.net/bzr-builder/+bug/1242796
>
> For more details, see:
> https://code.launchpad.net/~wgrant/bzr-builder/bug-1242796/+merge/229772
>
> Fix --no-build to leave a PACKAGE-VERSION directory rather than a TEMPLATE directory. This regressed in r158. It technically breaks compatibility with 0.7.3 by returning to the 0.7.2 behaviour, but I doubt anything actually wants the broken 0.7.3 behaviour
>
> 21:51:45 < jelmer> oh, right
> 21:51:50 < jelmer> yeah, that's not intentional

Fix LGTM.

Jelmer

« Back to merge proposal