On 26 Jul 2016, at 3:59, Martin Winter wrote:

On 21 Jul 2016, at 6:54, Paul Jakma wrote:

On Wed, 20 Jul 2016, Lou Berger wrote:

Id say give them / someone a chance to  provide a fix...

Well, if Martin wants to run that through his torture testing, we can advance master if there's no bad breakage.

There is at least one issue left - the final commit fails on the DejaGNU unit tests

(see here: https://ci1.netdef.org/browse/QUAGGA-QMASTER20-CI001BUILD-1 )

I’ve started (yet again), going through all the commits one by one… It will take some time. (Expect approx 1 commit per 15 min in best case)

Up to commit 4ab273b (“lib: Replace lists with arrays to store read and write threads”) all is fine for my basic tests.

Commit b9ac2f3 (“lib: Consolidate VIEW_NODE to be ENABLE_NODE as well”) breaks “make check”
(Dejagnu libzebra test “testcli”)
See https://ci1.netdef.org/browse/QUAGGA-QMASTER20-30/test
or for the full “make test” output here:
https://ci1.netdef.org/browse/QUAGGA-QMASTER20-30/artifact/CI001BUILD/ErrorLog/log_dejagnu.txt

Somehow a later commit fixes this again, but I have not yet identified the fixing commit.
(Tested up to Git 4933f23, approx 8 commits later)

The final commit in the proposed branch fails various BGP dejagnu tests, but has the above
“testcli” fixed again.
See https://ci1.netdef.org/browse/QUAGGA-QMASTER20-1/test

- Martin

_______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

Reply via email to