fetchnews.c: invalid operands to binary

[Raymond Sundland]

I'm so C expert, but shouldn't it be =-?  As in ... stamp =- timezone?



-----Original Message-----
From: info-cyrus-bounces at lists.andrew.cmu.edu
[mailto:info-cyrus-bounces at lists.andrew.cmu.edu] On Behalf Of Ken Murchison
Sent: Monday, April 03, 2006 1:38 PM
To: Antoine Jacoutot
Cc: info-cyrus at lists.andrew.cmu.edu
Subject: Re: fetchnews.c: invalid operands to binary

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
----
Cyrus Home Page: http://asg.web.cmu.edu/cyrus
Cyrus Wiki/FAQ: http://cyruswiki.andrew.cmu.edu
List Archives/Info: http://asg.web.cmu.edu/cyrus/mailing-list.html