fetchnews.c: invalid operands to binary

[Ken Murchison]

Antoine Jacoutot wrote:
> Hi...
> 
> I'm trying to compile the 2.2.13 version of cyrus-imapd under OpenBSD.
> I've never had issues with earlier versions, but now, there seems to be 
> some changes in imap/fetchnews.c and I get the following error:
> 
> cc -c -I.. -I./../lib -I/usr/include -I./../sieve 
> -I/usr/local/include/db4 -I/usr/local/include -I/usr/include 
> -I/usr/include -I/usr/local/include -DHAVE_CONFIG_H -O2 -pipe -fPIC  
> fetchnews.c
> fetchnews.c: In function `main':
> fetchnews.c:436: error: invalid operands to binary -
> *** Error code 1
> 
>  If I change le line #436 from
> stamp -= timezone;
> to
> stamp = timezone;
> 
> then it compiles fine. BUT, I'm not a developper and I have no idea how 
> this change is good/dangerous... or not ???

This is definitely not what you want to do.  I don't know why OpenBSD 
has a problem with this.  It compiles fine on Linux and Solaris.  You 
could try changing #436 to:

stamp -= (time_t) timezone;



and/or try adding the following around line #66:

extern long timezone;


Let me know if one or both of these changes solve the problem.

-- 
Kenneth Murchison
Systems Programmer
Project Cyrus Developer/Maintainer
Carnegie Mellon University