[quagga-dev 9062] Re: Google changes to Quagga now available on code.google.com
joshb at google.com
Mon Feb 13 20:22:29 GMT 2012
Sorry about that - my fault. My intent here was to expedite the
availability of the code to the community and to the ISC for testing.
Clearly will need to come up with the right iterative approach to handle
large feature changes. Regarding the IS-IS changes specifically for
example. Would you be able to advise what areas we should prioritise for
repatching/commenting/for review? Idea of course is to come up with a
On Mon, 13 Feb 2012, David Lamparter wrote:
> Hi Subbaiah Venkata,
> On Wed, Feb 01, 2012 at 01:52:56AM -0800, Subbaiah Venkata wrote:
>> New ISIS branch is pushed to google public repository.
>> This branch has support for new features and bug fixes.
>> Please let me know if you have any questions.
> I've just taken a look at the branch. Honestly, WTF? It comes as 2
> patches with 518 / 130 hunks each, touching the entirety of isisd and
> some files outside of isis?!
> Please, please, split this into small reviewable patches. Patches that
> touch things outside isisd/ need to be specially separated and explained
> with proper comments.
> Also, please do not mix whitespace/indent changes with code changes; if
> you want to reindent code, that should be a separate patch that does
> nothing else.
More information about the Quagga-dev