On Wed, Jul 4, 2012, at 05:03 PM, Jenkins wrote: > See <http://ci.cyrusimap.org/job/cyrus-imapd-master/689/changes> > > Changes: > > [git-dpa] update .gitignore to include config.(sub,guess) and install-sh > > [git-dpa] update .gitignore > > [git-dpa] config.h: add #define EXPORTED and HIDDEN > > [git-dpa] libcyrus_sieve: hide all internal symbols > > [git-dpa] imap/libcyrus_imap: hide symbol search_prefilter_messages() > > [git-dpa] fixup: libcyrus_sieve: hide all internal symbols > > [git-dpa] mark local variable as static > > [git-dpa] imap/libcyrus_imap: mark some internal variables as static > > [git-dpa] lib/libcyrus_min: hide function beautify_code > > [git-dpa] imap/libcyrus_imap: make even more variables static and hidden > > [...] > cunit/sieve.o: In function `test_comparator': > <http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:85: > undefined reference to `lookup_comp' > <http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:86: > undefined reference to `lookup_comp' > <http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:87: > undefined reference to `lookup_comp' > <http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:88: > undefined reference to `lookup_comp' > <http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:89: > undefined reference to `lookup_comp' > cunit/sieve.o:<http://ci.cyrusimap.org/job/cyrus-imapd-master/ws/imapd/./cunit/sieve.testc>:91: > more undefined references to `lookup_comp' follow > collect2: ld returned 1 exit status
This broke because Dilyan's linkage changes hid one too many symbols in libsieve. He's fixed it in commit http://git.cyrusimap.org/cyrus-imapd/commit/?id=7a94a84ff546ebb8d2ed4ef4255e32396b0c5afb It's great to see people actually responding to Jenkins build failures :) -- Greg.
