--MP_XvatTZb9zxlnUE4/dpT5SsL
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

On Tue, 11 Sep 2007 16:57:20 -0500 (CDT)
"Jeremy C. Reed" <reed@reedmedia.net> wrote:

> On Tue, 11 Sep 2007, Steven M. Bellovin wrote:
> 
> > > Ah, yes; here it is:
> > > http://www.netbsd.org/cgi-bin/query-pr-single.pl?number=36178
> > > 
> > 
> > OK, the patch works for me with modular xorg; thanks.  The question
> > is whether this is a patch that should be fed back upstream to the
> > xfig folks or to the xorg folks, or if it should be in our xorg
> > patch directory.
> 
> This patch for xc/lib/Xaw3d/SmeLine.c is not for X.org. As far as I
> can tell, it doesn't match up with X.org's lib/libXaw/src/SmeLine.c.

I've attached the patch I used (via localpatches) to recompile
x11/Xaw3d, which in turn makes xfig work.  (I thought that that
directory was part of xorg, but pkgdepgraph suggests not -- all that
uses it on my system are gv and xfig.)
> 
> The X.org FTP server only provides Xaw3d via its contrib directory.
> As far as I know, the original Xaw3d author/developer doesn't work on
> it for several years and, sadly, it seems the more recent maintainer
> passed away. The last release was four years ago.
> 
> Please just add that patch to /pkgsrc/x11/Xaw3d/patches/.

Will do.
> 
> By the why, what about building pkgsrc/graphics/xfig with: 
> XAW_TYPE=standard or XAW_TYPE=xpm?
> 
My X11 clue is minimal at best; I not only didn't know I could do that,
I have no idea what those options mean...


		--Steve Bellovin, http://www.cs.columbia.edu/~smb

--MP_XvatTZb9zxlnUE4/dpT5SsL
Content-Type: application/octet-stream; name=patch-za
Content-Transfer-Encoding: base64
Content-Disposition: attachment; filename=patch-za

LS0tIFNtZUxpbmUuYy5vcmlnCTIwMDAtMTEtMjcgMDg6MTk6MjguMDAwMDAwMDAwIC0wNTAwCisr
KyBTbWVMaW5lLmMJMjAwNy0wOS0xMSAxNzoyMTozOS4wMDAwMDAwMDAgLTA0MDAKQEAgLTIxNyw3
ICsyMTcsNyBAQAogICAgIFNtZUxpbmVPYmplY3QgZW50cnkgPSAoU21lTGluZU9iamVjdCkgdzsK
ICAgICBTaW1wbGVNZW51V2lkZ2V0IHNtdyA9IChTaW1wbGVNZW51V2lkZ2V0KSBYdFBhcmVudCAo
dyk7CiAgICAgVGhyZWVEV2lkZ2V0IHRkdyA9IChUaHJlZURXaWRnZXQpIHNtdy0+c2ltcGxlX21l
bnUudGhyZWVEOwotICAgIERpbWVuc2lvbiBzID0gdGR3LT50aHJlZUQuc2hhZG93X3dpZHRoOwor
ICAgIERpbWVuc2lvbiBzID0gdGR3ID8gdGR3LT50aHJlZUQuc2hhZG93X3dpZHRoIDogTlVMTDsK
ICAgICBpbnQgeSA9IGVudHJ5LT5yZWN0YW5nbGUueSArIAogCSAgICAoaW50KShlbnRyeS0+cmVj
dGFuZ2xlLmhlaWdodCAtIGVudHJ5LT5zbWVfbGluZS5saW5lX3dpZHRoKSAvIDI7CiAK

--MP_XvatTZb9zxlnUE4/dpT5SsL--