Request for review: my patch queue for 2.6

Bart Cerneels bart.cerneels at kde.org
Tue Dec 27 08:56:03 UTC 2011


On Sat, Dec 24, 2011 at 15:40, Matěj Laitl <matej at laitl.cz> wrote:
> On 20. 12. 2011 Bart Cerneels wrote:
>> Just make sure you squash the commits that belong together :)
>
> Do you mean some specific commits? E.g. there are 3 UpdateCapability removal-
> related commits, but each one stands on its own (with commit message
> describing the specific change) and everything is perfectly bisectable. (I must
> say I'm not a big fan of squashing related changes - I instead prefer the
> Linux-kernel way of thinking with git: make very granular commits so that it
> is `git bisect` who finds erroneous changes, not you)

One revertible commit, not a whole series that would make using git
bisect take way to long to be useful.

>
>> And don't forget to Changelog in the same commit!
>
> I guess you're alluding to my past commits. I've improved, I swear! :)
>
> So it seems nobody opposes merging the patch-set, so I'll do it once I find
> another bit of time in this holiday time..
>
>                Matěj
> _______________________________________________
> Amarok-devel mailing list
> Amarok-devel at kde.org
> https://mail.kde.org/mailman/listinfo/amarok-devel


More information about the Amarok-devel mailing list