Hey Guido,
> This looks great! The only thing left is to cleanup the patch series to
> not fix bugs introduced earlier in the _same_ patch series. E.g.
> c4f184b which fixes a bug introduced earlier, just merge that. Or
> 02f7a2a which removes parse_changelog_file which got introduced in
> 4f6809d
Hi Matthijs,
On Mon, Aug 30, 2010 at 10:09:52AM +0200, Matthijs Kooijman wrote:
> Hey Guido,
>
> > Yes. This looks really great in general, thanks alot! Two minor
> > nitpicks:
> Ok, nitpicks fixed. Instead of spamming this report with even more
> patches, I've just pushed them here:
>
> http
Hey Guido,
> Yes. This looks really great in general, thanks alot! Two minor
> nitpicks:
Ok, nitpicks fixed. Instead of spamming this report with even more
patches, I've just pushed them here:
http://git.stderr.nl/gitweb?p=matthijs/upstream/git-buildpackage.git
Pull from:
http://git.std
Hi Matthijs,
On Thu, Aug 19, 2010 at 02:26:01PM +0200, Matthijs Kooijman wrote:
> Hi Guido,
>
> I've rebased the patch against current git master, and also coded up a
> "read the changelog from the repository"-feature. The latter was a bit
> complicated, so I split it up in a few patches. There's
Hi Guido,
I've rebased the patch against current git master, and also coded up a
"read the changelog from the repository"-feature. The latter was a bit
complicated, so I split it up in a few patches. There's also an
unrelated variable name fix, that I came across while reading the code.
Is this w
5 matches
Mail list logo
