Build failed in Jenkins: cyrus-imapd-master #689

[Greg Banks]

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.