> On Thu, Jun 29, 2023 at 9:33 AM Emily Shaffer <nasamuffin@xxxxxxxxxx> wrote: Hi, thanks for replying! > > Yeah, it's almost certainly stale in MyFirstObjectWalk - there was > > very recently a patch to clean up some headers which probably were > > implicitly including trace.h when I wrote this walkthrough. Patches > > totally welcome - and if you were working from the reference code in > > https://github.com/nasamuffin/git/tree/myfirstrevwalk > > bah, wrong link, the tutorial points to branch `revwalk` instead of > `myfirstrevwalk`, but the offer stands :) > > > and it's on your > > way to rebase and fix that too, I'm happy to update my branch > > accordingly too. (If you weren't, don't worry about doing the extra > > work, though.) Sure will! But do you mean open a PR on your fork? I have the patch ready, and would be very happy to do so, if it is accepted! Thanks a lot! Vinayak