Re: pkg/51903

[David Holland <dholland-pbugs%netbsd.org@localhost>]

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

From: David Holland <dholland-pbugs%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/51903
Date: Tue, 24 Jan 2017 16:43:52 +0000

 On Mon, Jan 23, 2017 at 06:05:01PM +0000, David Holland wrote:
  >  Check for warnings during the build? Given the symptoms there must be
  >  something wrong with the Perl <-> C glue. I remember fixing this some
  >  in the past; it probably needs more.
 
 So, erm, when I fixed it in the past I apparently wrote this:
 
 date: 2011-10-15 13:53:51 -0400;  author: dholland;  state: Exp; lines: +3 -1;
 Fix build with perl 5.14... I think. Someone who actually knows how to
 write Perl bindings should double-check this.
 
 XXX: This will almost certainly not work on LP64 platforms as it casts
 XXX: Perl-provided pointers to int and back. However, fixing this
 XXX: requires changing the API of the parent Sablotron package, which
 XXX: doesn't seem like a great idea at the moment.
 
 
 Looks like textproc/sablotron itself was last updated in 2005, so it
 may be time to patch it. Or maybe just kill it off.
 
 There seems to be a slightly newer sablotron 1.0.3 on sourceforge
 here: https://sourceforge.net/projects/sablotron/files/
 
 but that site also seems to be an import some random person did and
 not from the original upstream, so whether it's any good (or even safe
 to use) is an open question.
 
 Is the package you're looking at using sablotron because it is also
 old, or because its upstream likes sablotron over other possible
 options? If the latter, that's a sign it might be worth working on it
 a bit...
 
 
 -- 
 David A. Holland
 dholland%netbsd.org@localhost