On Wed, Sep 20, 2006 at 07:52:03AM +0200, Ralf S. Engelschall wrote: > On Tue, Sep 19, 2006, Adam D. Morley wrote: > > > [...] > > Solaris is apparently not enjoying building OSS this week: > > > > openpkg rpm -bb sasl.spec > > ... > > Making all in sample > > if /opt/openpkg/bin/cc -DHAVE_CONFIG_H -I. -I. -I.. -I../include > > -I/opt/openpkg/include -I/opt/openpkg/include -I/opt/openpkg/include > > -Wall -W -O2 -pipe -I/opt/openpkg/include -MT client.o -MD -MP -MF > > ".deps/client.Tpo" \ > > -c -o client.o `test -f 'client.c' || echo './'`client.c; \ > > then mv ".deps/client.Tpo" ".deps/client.Po"; \ > > else rm -f ".deps/client.Tpo"; exit 1; \ > > fi > > client.c: In function 'getrealm': > > client.c:81: warning: unused parameter 'context' > > client.c: In function 'simple': > > client.c:106: warning: unused parameter 'context' > > client.c: At top level: > > client.c:139: error: static declaration of 'getpassphrase' follows > > non-static declaration > > /usr/include/stdlib.h:206: error: previous declaration of > > 'getpassphrase' was here > > Yes, SASL is broken here. It Autoconf-like wrapped a getpassphrase(3) > function stub with "#ifndef HAVE_GETPASSPHRASE" but the whole > SASL doesn't have any Autoconf checks for this function. > This obviously breaks under platforms like Solaris where > the system provides a getpassphrase(3) function. Now fixed: > http://cvs.openpkg.org/chngview?cn=30150 > Thanks for the hint.
Hi, Awesome. Thanks. Should I send the patch upstream to the cyrus group, or will you notify them that their autoconf needs another check? Also, will it show up in -CURRENT, or should I just grab the patchset off the web? -- adam ______________________________________________________________________ The OpenPKG Project www.openpkg.org User Communication List [email protected]
