CI System is now partially updated (Missing part is in the timoteras-nhrpd branch)

It should now automatically build within an hour of any push to the trees

Status:

=====

volatile/patch-tracking/4/proposed/ff branch:
        https://ci1.netdef.org/browse/QUAGGA-QMASTER3

Current status:
Build passes on Ubuntu 12.04, 14.04, FreeBSD 8,9,10, NetBSD 6, Debian 8

Build fails on CentOS 6: (see https://ci1.netdef.org/browse/QUAGGA-QMASTER3-2/artifact/CI006BUILD/ErrorLog/log_make.txt )
        CC     sockopt.lo
        cc1: warnings being treated as errors
        sockopt.c: In function ‘setsockopt_ipv4_multicast’:
sockopt.c:236: error: dereferencing pointer ‘si’ does break strict-aliasing rules sockopt.c:232: error: dereferencing pointer ‘si’ does break strict-aliasing rules
        sockopt.c:230: note: initialized from here
        At top level:
        cc1: error: unrecognized command line option "-Wno-unused-result"

Build fails on CentOS 7: (see https://ci1.netdef.org/browse/QUAGGA-QMASTER3-2/artifact/CI005BUILD/ErrorLog/log_make_rpm.txt )
          CC       vty.lo
        vty.c: In function 'vty_read_config':
vty.c:2417:18: error: ignoring return value of 'getcwd', declared with attribute warn_unused_result [-Werror=unused-result]
                   getcwd (cwd, MAXPATHLEN);
                          ^
        vty.c: In function 'vty_use_backup_config':
vty.c:2379:11: error: ignoring return value of 'write', declared with attribute warn_unused_result [-Werror=unused-result]
             write (tmp, buffer, c);
                   ^
        vty.c: In function 'vty_log_fixed':
vty.c:2549:8: error: ignoring return value of 'writev', declared with attribute warn_unused_result [-Werror=unused-result]
          writev(vty->wfd, iov, 2);
                ^
        vty.c: In function 'vty_save_cwd':
vty.c:2997:13: error: ignoring return value of 'chdir', declared with attribute warn_unused_result [-Werror=unused-result]
               chdir (SYSCONFDIR);
                     ^
vty.c:2998:14: error: ignoring return value of 'getcwd', declared with attribute warn_unused_result [-Werror=unused-result]
               getcwd (cwd, MAXPATHLEN);
                      ^
        cc1: all warnings being treated as errors

Build fails on NetBSD 7: (see https://ci1.netdef.org/browse/QUAGGA-QMASTER3-2/artifact/CI012BUILD/ErrorLog/log_make.txt )
          CC       rt_socket.o
        rt_socket.c: In function 'kernel_rtm_ipv6_multipath':
rt_socket.c:282:7: error: variable 'error' set but not used [-Werror=unused-but-set-variable]
           int error;
               ^
        cc1: all warnings being treated as errors

=====

volatile/patch-tracking/4/proposed/timoteras-nhrpd branch:
        https://ci1.netdef.org/browse/QUAGGA-QMASTER5

Not yet all adapted as it requires the libc-ares library and I need to upgrade all my test systems to include it.
(I hope it’s available on all platforms…)

I’ve added it on the Ubuntu and the CentOS machines.

CentOS passes, but Ubuntu has a syntax error in the configure:
See https://ci1.netdef.org/browse/QUAGGA-QMASTER5-1/artifact/CI001BUILD/ErrorLog/log_configure.txt

checking for res_init in -lresolv... no
checking whether system has GNU regex... checking for regexec in -lc... yes
./configure: line 18818: syntax error near unexpected token `CARES,'
./configure: line 18818: `   PKG_CHECK_MODULES(CARES, libcares)'

(libcares is installed - this is a syntax error in configure and always fails)

=====

volatile/patch-tracking/4/proposed/netdef-solaris branch:
        https://ci1.netdef.org/browse/QUAGGA-QMASTER4

Compiles correctly, but fails in the basic IPv4 protocol tests (OSPF & BGP)
I’ll look into this one…  (Probably our own screwup)


- Martin


On 13 Oct 2015, at 9:43, Paul Jakma wrote:

As usual, proposed patches at:

http://git.savannah.gnu.org/cgit/quagga.git/log/?h=refs/heads/volatile/patch-tracking/4/proposed/ff


Carried over from previous round:


HACKING: Change format to MarkDown
http://patchwork.quagga.net/patch/1314/
I'm thinking that this is simply (more or less) a super-set of TeX and still rendered via TeX answers the objections?


Adding redhat init/service files to start pimd
http://patchwork.quagga.net/patch/1311/


distro: fix redhat/quagga.spec.in
http://patchwork.quagga.net/patch/1349/

Switchover -Werror to default setting
http://patchwork.quagga.net/patch/1293/
still need to address some issues?

bgpd: Quagga, BGP feature Best External
http://git.savannah.gnu.org/cgit/quagga.git/commit/?h=refs/heads/volatile/patch-tracking/4/proposed/ff&id=d30eee229f5f85205753c95c934993a7f8644176
Needs formatting fixes and review

Add support for run quagga in a FIB (Routing table) on FreeBSD
http://git.savannah.gnu.org/cgit/quagga.git/commit/?h=refs/heads/volatile/patch-tracking/4/proposed/ff&id=fae10e1658a5a4e0a694473e8fcd6e69ff55b5d6
Unsure of the status of this, I think there were some comments on nits (e.g. the define name), and there are also questions about the approach?



Newly proposed:

zebra: Set link-detect on by default
http://patchwork.quagga.net/patch/1351/


zebra: Add command to configure default for link-state, and make it sticky
http://patchwork.quagga.net/patch/1352/
Needs testing, and possibly more work.

git: add (generated) cscope files to .gitignore
http://patchwork.quagga.net/patch/1355/


lib: zclient.c remove extern struct thread_master *
http://patchwork.quagga.net/patch/1361/

bgpd: fix using of two pointers for struct thread_master
http://patchwork.quagga.net/patch/1360/
Maybe go the other way?


ospfd: add running flag
http://patchwork.quagga.net/patch/1362/
Discussions on list on approach


lib: Remove antiquated setting of socket options
http://patchwork.quagga.net/patch/1363/
FreeBSD status?

vtysh: Fix Quagga.conf file read in.
http://patchwork.quagga.net/patch/1367/

quagga: Additional centos 6 -enable-werror fixes
http://patchwork.quagga.net/patch/1369/

zebra: Add a new zebra message type ZEBRA_MESSAGE_ONLINK
Not queued - discussion on approach on list?

pimd: Do not display some default values as part of a show run
http://patchwork.quagga.net/patch/1373/

pimd: Cleanup zebra debugs to be protected by debug commands
http://patchwork.quagga.net/patch/1374/

pimd: Add knowledge of different packet types
http://patchwork.quagga.net/patch/1376/

pimd: Notice when we receive a packet type we can't handle yet
http://patchwork.quagga.net/patch/1377/

pimd: Cleanup interface startup
http://patchwork.quagga.net/patch/1379/

Quagga: Fix some more compile warnings
http://patchwork.quagga.net/patch/1380/

tests: Fix warnings from test-stream.c
http://patchwork.quagga.net/patch/1382/

isisd: Drop packet received on multiple interfaces due to the time gap in binding socket to an interface
http://patchwork.quagga.net/patch/1384/



Additionally, proposed to pull in the following trees:


The Cumulus next-hop tracking branch and Timo Teräs' NHRPd:

git://git.alpinelinux.org/user/tteras/quagga commit 172225b4eb
Note, may have another approach to onlink (see my last email to Christian) Also, a few conflicts against current master and non-conformist commit subjects.


The NetDEF Solaris work:

https://git-us.netdef.org/scm/osr/quagga-solaris.git commit 605172293128c4

regards,
--
Paul Jakma      [email protected]  @pjakma Key ID: 64A2FF6A
Fortune:
Telecommunications is upgrading._______________________________________________
Quagga-dev mailing list
[email protected]
https://lists.quagga.net/mailman/listinfo/quagga-dev

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

Reply via email to