FRRouting / topotests

Moved to frrouting/frr

22 stars 26 forks source link

topotest: fix a regression in version_cmp #45

Closed rzalamena closed 6 years ago

rzalamena commented 6 years ago

It was found a regression on an edge case when the second number in the comparison was (at least) 2 numbers longer the comparison would fail with a wrong return value. It succeeded for some cases because the first comparison in the exception was correct, but not the second.

NetDEF-CI commented 6 years ago

Continuous Integration Result: SUCCESSFUL

Congratulations, this patch passed basic tests

Tested-by: NetDEF / OpenSourceRouting.org CI System

CI System Testrun URL: https://ci1.netdef.org/browse/FRR-TOPOPR-57/

This is a comment from an EXPERIMENTAL automated CI system. For questions and feedback in regards to this CI system, please feel free to email Martin Winter - mwinter (at) opensourcerouting.org.