Comment 2 for bug 684517

Revision history for this message
Colin Watson (cjwatson) wrote :

https://code.launchpad.net/bzr-builder would be the place to try to start with this, and also https://code.launchpad.net/git-build-recipe which exists now and should be kept up to date with any changes to bzr-builder. Thanks for offering to help, as we're working mostly to capacity at the moment and it's unlikely that this sort of bug will be addressed otherwise.

Most-recent-commit of any of the various branches that form part of the recipe is probably an OK metric to use, although it would be nice if we could also take into account the last modification time of the recipe itself. That latter is more work. I think the most elegant way would be (1) have bzr-builder/git-build-recipe get that modification time from the mtime of the recipe file passed to them as input; (2) have lp:launchpad-buildd lpbuildd/sourcepackagerecipe.py set that mtime from an option passed to it in its build arguments; (3) have lp:launchpad lib/lp/code/model/recipebuilder.py set that argument from self.build.recipe.date_last_modified.

In the first pass, I'd recommend tackling only bzr-builder/git-build-recipe, and proving that the change works by way of tests. If you take my suggested approach of taking the recipe's mtime into consideration, then it won't do anything useful initially in a Launchpad context because the recipe file will always be current, but the other bits can be stitched in afterwards.

Feel free to discuss this with us as noted in https://dev.launchpad.net/Help.