error updating Postfix

[Wesley Shields]

On Tue, Jul 21, 2009 at 12:00:47AM -0400, Sahil Tandon wrote:
> On Tue, 21 Jul 2009, Per olof Ljungmark wrote:
> 
> > Wesley Shields wrote:
> >> On Tue, Jul 21, 2009 at 12:23:16AM +0200, Per olof Ljungmark wrote:
> >>> cc -DDEF_CONFIG_DIR=\"/usr/local/etc/postfix\"  
> >>> -DDEF_DAEMON_DIR=\"/usr/local/libexec/postfix\"  
> >>> -DDEF_COMMAND_DIR=\"/usr/local/sbin\"  
> >>> -DDEF_SENDMAIL_PATH=\"/usr/local/sbin/sendmail\"  
> >>> -DDEF_NEWALIAS_PATH=\"/usr/local/bin/newaliases\"  
> >>> -DDEF_MAILQ_PATH=\"/usr/local/bin/mailq\"  
> >>> -DDEF_MANPAGE_DIR=\"/usr/local/man\"  
> >>> -DDEF_README_DIR=\"/usr/local/share/doc/postfix\"  
> >>> -DDEF_HTML_DIR=\"/usr/local/share/doc/postfix\"  
> >>> -DDEF_QUEUE_DIR=\"/var/spool/postfix\"  
> >>> -DDEF_DATA_DIR=\"/var/db/postfix\" -DDEF_MAIL_OWNER=\"postfix\"  
> >>> -DDEF_SGID_GROUP=\"maildrop\" -DHAS_PCRE -I/usr/local/include  
> >>> -DUSE_SASL_AUTH -DUSE_CYRUS_SASL -I/usr/local/include  
> >>> -I/usr/local/include/sasl -DUSE_TLS -I/usr/include  -O2  
> >>> -fno-strict-aliasing -pipe -I. -DFREEBSD7 -c attr_clnt.c
> >>> In file included from attr_clnt.c:77:
> >>> /usr/include/unistd.h:329: error: conflicting types for 'closefrom'
> >>> ./sys_defs.h:1404: error: previous declaration of 'closefrom' was here
> >>> *** Error code 1
> >>
> >> It was fixed on 2009/07/09. Make sure you have the latest ports tree.
> >
> > Hm, I updated an hour ago... from cvsup.FreeBSD.org, the makefile is  
> > dated July 17th when the system was also updated.
> >
> > 7.2-STABLE FreeBSD 7.2-STABLE #0: Fri Jul 17 17:27:43 CEST 2009
> >
> > Just for the sake of it I cvsupped again just now to be sure but no  
> > change. What was fixed in the 9th.? I obviously need to check if  
> > something went wrong when I updated the system.
> 
> On 9 Jul 2009, ports/136599 fixed build on 8.0 systems.  Now that the
> closefrom() system call has been MFC'd to -STABLE, a similar patch is
> required for mail/postfix.  This was broached on cvs-ports@ on 17 Jul 2009
> and keramida@ was testing a patch over the weekend.  I've copied him on
> this message.  For context (and an example of all that is really needed in
> ${WRKSRC}/src/util/sys_defs.h), see:

Ah, I didn't realize the fix had been MFC'ed and that the machine in
question was RELENG_7. That explains it. Hopefully keramida@ has time to
commit it.

Sorry for the noise.

-- WXS