From: Adam Brewster Date: Sun, 06 Jul 2008 02:49:31 GMT Subject: Re: [PATCH v5] Message-ID: In-Reply-To: <7vskunpyqz.fsf@gitster.siamese.dyndns.org> > > Yeah, it is simply a bad taste to use helper in builtin-A from builtin-B. > More library-ish files such as revision.c are better home for utility > functions to be shared between builtins and commands. > > Here is what I queued. > Thank you. > By the way did you compile test your fix before sending? > I ran make and test, but I didn't notice the warnings that prompted the question. I also forgot to re-check it after re-working the commits to put everything in order. > -- >8 -- Thank you again for your help and patience in dealing with my multiple failed attempts to get this right. Adam