[quagga-dev 5023] Re: patches needing review for CVS

Paul Jakma Paul.Jakma at Sun.COM
Mon Aug 6 14:56:36 BST 2007


On Mon, 30 Jul 2007, Paul Jakma wrote:

> - Denis' patch from bug #384:
>
> 	http://bugzilla.quagga.net/show_bug.cgi?id=384
>
>  - seems ok to me, and Denis has spent a month or two chasing down this
>    corner-case bug. He may be splitting it up into two patches, debug
>    and the actual bugfixes, hopefully.
>
> - BGP Capability cleanup
>  - Tests good for me, needs soak-time in CVS now
> - AS4 merge (i need to merge it with previous, but other than that)
>  - ditto
> - Quick implementation of AS_PATHLIMIT
>  - ditto
> - Jim's IS-IS/Solaris work
>  - self-contained, adds support where none existed before, seems
>    obvious to go in
> - Lorenzo's bgp_dump fixes
>  - seem sane
>
> comments?

Well, in lieu of objections, I think the above should go in.. Greg has 
committed Denis' patch, I've committed Lorenzo's. I'll go ahead and 
commit the remaining above unless there are screams really soon.

regards,
-- 
Paul Jakma,
Solaris Networking                       Sun Microsystems, Scotland
http://opensolaris.org/os/project/quagga tel: EMEA x73150 / +44 1506 673150



More information about the Quagga-dev mailing list