Hi Bastian,

ok , so it a corner case as it is a very old perl lib :D.....

But I think both a) and b) just be done. First, the users should know what versions was tested for ,and secondly, to cope with the older versions - maybe having a #warning if PERL_EXIT_DESTRUCT_END is not defined will be better also.

thanks and regards,
bogdan


Bastian Friedrich wrote:

Hi,

On Thursday 14 December 2006 15:33, Bogdan-Andrei Iancu wrote:
I guess the <perl.h> file is correctly located and included.

Good.

perl -MExtUtils::Embed -e ccopts ->
-D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64
-I/usr/perl5/5.6.1/lib/sun4-solaris-64int/CORE
              ^^^^^
Whoops. :)

PERL_EXIT_DESTRUCT_END has been defined in the stable branch since Perl 5.8.0 (which has been out for almost 4 1/2 years).

Up to this point, I only have checked Perl 5.8.8.

There are at least two options, now:
a) State in README/docs that 5.8.0 is needed, and that the system has
  been tested with 5.8.8
b) Use a
#ifdef PERL_EXIT_DESTRUCT_END PL_exit_flags |= PERL_EXIT_DESTRUCT_END;
#else
PL_exit_flags |= PERL_EXIT_EXPECTED;
#endif
  wrapping or sth. similar

Please tell me what option you'd prefer (or just commit that yourself :))

As a side note: Perl 5.8.7 is available as a binary package for Solaris 9 from sunfreeware.com:
ftp://ftp.sunfreeware.com/pub/freeware/sparc/9/perl-5.8.7-sol9-sparc-local.gz

Best,
  Bastian



_______________________________________________
Devel mailing list
[email protected]
http://openser.org/cgi-bin/mailman/listinfo/devel

Reply via email to