(In reply to Tom Ritter [:tjr] from comment #7) > When you vendor through mach vendor there should be a second separate commit that basically squashes all the local patches into one big one, right? > > Updatebot _could_ be changed to submit them as separate commits - but then you'd need to go through and r+ 34 revisions instead of two... Storing changes as a series of patch files is what I don't like. I think it's justified in the case of Cairo, for the reasons given in comment 5.
Bug 2035541 Comment 8 Edit History
Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.
(In reply to Tom Ritter [:tjr] from comment #7) > When you vendor through mach vendor there should be a second separate commit that basically squashes all the local patches into one big one, right? > > Updatebot _could_ be changed to submit them as separate commits - but then you'd need to go through and r+ 34 revisions instead of two... This isn't the issue. Storing changes as a series of patch files is what I don't like. I think it's justified in the case of Cairo, for the reasons given in comment 5.