Re: pkg/49782 (textproc/mdocml won't build on Solaris)

[christos%zoulas.com@localhost (Christos Zoulas)]

The following reply was made to PR pkg/49782; it has been noted by GNATS.

From: christos%zoulas.com@localhost (Christos Zoulas)
To: gnats-bugs%NetBSD.org@localhost, solaris-pkg-people%NetBSD.org@localhost, 
	gnats-admin%netbsd.org@localhost, pkgsrc-bugs%netbsd.org@localhost, bnjf+netbsd%bnjf.id.au@localhost
Cc: 
Subject: Re: pkg/49782 (textproc/mdocml won't build on Solaris)
Date: Tue, 7 Apr 2015 21:17:07 -0400

 On Apr 8, 12:25am, bnjf%bnjf.id.au@localhost (Brad Forschinger) wrote:
 -- Subject: Re: pkg/49782 (textproc/mdocml won't build on Solaris)
 
 | The following reply was made to PR pkg/49782; it has been noted by GNATS.
 | 
 | From: Brad Forschinger <bnjf%bnjf.id.au@localhost>
 | To: gnats-bugs%netbsd.org@localhost
 | Cc: solaris-pkg-people%netbsd.org@localhost, pkg-manager%netbsd.org@localhost, 
 | 	pkgsrc-bugs%netbsd.org@localhost, gnats-admin%netbsd.org@localhost, wiz%netbsd.org@localhost, 
 | 	bnjf+netbsd%bnjf.id.au@localhost
 | Subject: Re: pkg/49782 (textproc/mdocml won't build on Solaris)
 | Date: Wed, 8 Apr 2015 10:24:33 +1000
 | 
 |  No good.  The patch is looking for __sun__, not __sun.
 |  
 |  +#ifdef __sun__
 |  
 |  should be
 |  
 |  +#ifdef __sun
 |  
 |  in patch-compat_fts.c.
 
 This is compiler-dependent. You should check for either like:
 
     #if defined(__sun) || defined(__sun__)
 
 to be safe...
 
 christos