[Dovecot] MANAGESIEVE patch v5 for dovecot 1.0.2

Stephan Bosch stephan at rename-it.nl
Wed Aug 15 19:33:45 EEST 2007


Robin Breathe schreef:
> Stephan Bosch wrote:
>   
>> Have fun testing the patch. Notify me when there are problems.
>>     
>
> Stephan,
>
> There's a small problem with your patch as it stands: it depends on a
> number of GCCisms, and fails to compile with, for example, Sun CC under
> Solaris 10.
> Removing all of your "__attribute__((unused))" declarations goes some
> way, but the build then fails with the following:
>
>  /opt/SUNWspro/bin/cc -DHAVE_CONFIG_H -I. -I../.. -I../../src/lib
> -I../../src/li                                             b-storage
> -I../../src/lib-mail -I../../src/lib-sievestorage -I/app/openssl/0.9.7
>                                            m/include -g -c
> sieve-implementation.c  -KPIC -DPIC -o .libs/sieve-implementatio
>                                      n.o
> "sieve-implementation.c", line 193: void function cannot return value
> cc: acomp failed for sieve-implementation.c
>
> A reasonable error given that sieve_runenv_mark_duplicate() is a void
> function with a return :) Removing the "return" leads to a clean build,
> but it's not clear what implications that might have.
>
> NB: this is applied against dovecot-1.0.3, though only one of the hunks
> is off by 1 line
I'll fix these issues tonight and make a release for 1.0.3. Thanks for 
the headsup! :)

Regards,

Stephan.


More information about the dovecot mailing list