dovecot - Jan 2009 - Warnings compiling on OpenBSD 4.4 i386

Hi,
probably just nothing to worry about, but I get the following warnings 
when building 1.1.7 on OpenBSD 4.4 on i386. If I remove the 
--with-bsdauth config option, they all disappear. I won't use it anyway, 
but maybe there are others.

$ dmesg
OpenBSD 4.4 (GENERIC) #1021: Tue Aug 12 17:16:55 MDT 2008
     deraadt at i386.openbsd.org:/usr/src/sys/arch/i386/compile/GENERIC
cpu0: Intel(R) Xeon(R) CPU 3040 @ 1.86GHz ("GenuineIntel" 686-class) 
1.87 GHz
cpu0: 
FPU,V86,DE,PSE,TSC,MSR,PAE,MCE,CX8,APIC,SEP,MTRR,PGE,MCA,CMOV,PAT,PSE36,CFLUSH,DS,ACPI,MMX,FXSR,SSE,SSE2,SS,HTT,TM,SBF,SSE3,MWAIT,DS-CPL,VMX,EST,TM2,CX16,xTPR
real mem  = 1071788032 (1022MB)
avail mem = 1027936256 (980MB)
mainbus0 at root
bios0 at mainbus0: AT/286+ BIOS, date 12/31/99, BIOS32 rev. 0 @ 0xfd460, 
SMBIOS rev. 2.4 @ 0xdc010 (48 entries)
bios0: vendor HP version "O10" date 07/20/2007
bios0: HP ProLiant ML110 G4


$ ./configure --without-pop3d --with-deliver --with-passwd 
--with-passwd-file --with-bsdauth --without-ldap --sysconfdir=/etc 
--localstatedir=/var --sharedstatedir=/var



mail-index-map.c: In function `mail_index_map_ext_hdr_check':
mail-index-map.c:171: warning: comparison between signed and unsigned
mail-index-map.c: In function `mail_index_map_clone':
mail-index-map.c:1112: warning: comparison between signed and unsigned

mail-index-sync-update.c: In function `sync_header_update':
mail-index-sync-update.c:419: warning: comparison between signed and 
unsigned

mail-transaction-log-file.c: In function 
`log_file_track_mailbox_sync_offset_hdr':
mail-transaction-log-file.c:682: warning: comparison between signed and 
unsigned

passdb-bsdauth.c: In function `bsdauth_preinit':
passdb-bsdauth.c:59: warning: implicit declaration of function 
`auth_cache_parse_key'
passdb-bsdauth.c:60: warning: assignment makes pointer from integer 
without a cast


/Peter
-- 
Peter Lindgren
http://www.norrskenkonsult.com

On Jan 3, 2009, at 10:46 PM, Peter Lindgren wrote:
> mail-index-map.c: In function `mail_index_map_ext_hdr_check':
> mail-index-map.c:171: warning: comparison between signed and unsigned
> mail-index-map.c: In function `mail_index_map_clone':
> mail-index-map.c:1112: warning: comparison between signed and unsigned
>
> mail-index-sync-update.c: In function `sync_header_update':
> mail-index-sync-update.c:419: warning: comparison between signed and  
> unsigned
>
> mail-transaction-log-file.c: In function  
> `log_file_track_mailbox_sync_offset_hdr':
> mail-transaction-log-file.c:682: warning: comparison between signed  
> and unsigned
What gcc version? I think these are fixed in newer ones so I'd rather  
not make Dovecot's code uglier by adding unnecessary casts.
> passdb-bsdauth.c: In function `bsdauth_preinit':
> passdb-bsdauth.c:59: warning: implicit declaration of function  
> `auth_cache_parse_key'
> passdb-bsdauth.c:60: warning: assignment makes pointer from integer  
> without a cast
This should fix it: http://hg.dovecot.org/dovecot-1.1/rev/33b8b74a3140