On Fri, May 02, 2014 at 09:09:44AM +0100, Lee Jones wrote:
On Thu, 01 May 2014, Mark Brown wrote:
Sure, but I didn't see an urgent need for that immediately so I just went ahead - the ack usually means that it's OK to apply.
Hmm... I'm inclined to disagree. The Ack is a marker that I'm okay with the changes, but the note after makes it pretty clear that the preference would be for the patch to be handled via an IB.
Right, and it's sitting on a separate branch which is (as we've seen) very easy to add a tag to if it turns out to be needed. This is what all the small topic branches are there for, it makes it really easy to cross merge, and obviously applying the patch on a branch is always going to be the first step in creating an immutable branch. It certainly doesn't seem worth getting worried about.
Really, if you're not happy for something to be applied you probably don't want to be acking it or you at least want to be really explicit about things.