On Tue, Jul 13, 2021 at 10:46 AM Atharva Raykar <raykar.ath@xxxxxxxxx> wrote: > > On 13-Jul-2021, at 13:48, Kaartic Sivaraam <kaartic.sivaraam@xxxxxxxxx> wrote: > > ... Alternatively, you could explore how other sub-commands > > handle recursing into submodules. To me it looks like they handle it by spawning > > a sub-process is likely the easiest approach for achieving recursion. That would > > solve the super-prefix problem as you have observed. > > Yes, I was hoping I would not have to spawn a subprocess for recursing on > update, and it does look theoretically possible--it would require changing > a lot of of existing code to use functions taking a repository objects > rather than assuming 'the_repository'. But since that was outside the scope > of my project, I did it like the other implementations that spawned a new > process for the recursion of 'update'. Yeah, it's definitely outside the scope of your project. So it's ok to spawn a new process.
