On Wed Aug 27, 2025 at 9:29 AM JST, John Hubbard wrote: > On 8/25/25 9:07 PM, Alexandre Courbot wrote: > ... >> The base of this series is today's nova-next, but there a few more >> unmerged dependencies required: >> >> - BorrowedPage, AsPageIter and VmallocPageIter [2] >> - Rust infrastructure for sg_table and scatterlist [3] >> - FromBytes [4] >> - The Alignment series [5] > > Hi Alex! > > For future patchsets that have complex and/or lengthy prerequisite > patchsets, let's please please make it easier for people to > locally apply the series. Very important. > > The very nicest experience is by simply providing a link to your git > branch that exactly matches the patchset you just posted. > > In this particular case, it would also have worked to just say, "this > series can be applied by first applying FromBytes [4], to the base commit." > > Because that works, and is simpler than hunting down the other three > items, which are not actually necessary for git am (although they > might be for actual testing). > > But the point is that the admin can be made simpler for the reviewers--even > those of us who know exactly what you're up to. And we should keep > that in mind, especially because there are many more of these situations > coming soon.
Right, b4 is supposed to be able to help with this as well, but indeed a link to a git tree should be the standard with patchsets requiring such dependencies. :) Will try to keep this in mind.
