Thanks, John. I've applied your patch (in a slightly modified form)
and released a new beta, 0.13_01. It would be great if you (and/or a
couple other people on the vmsperl list) could try it out, and then I
can release 0.14 when it checks out.
-Ken
p.s. - I wish all "novices" were as capable as you. ;-)
On Sep 10, 2005, at 11:24 PM, John E. Malmberg wrote:
John E. Malmberg wrote:
Ken Williams wrote:
Hi John,
After my last email I reworked the patch so that all the
modifications are in Platform/VMS.pm. Could you confirm that it
still works for you and I haven't broken it?
Hello Ken,
As I previously wrote to you, both my patch and this patch do not work.
While my patch fixed the ParseXS/t/basic that I was working on, it
broke three other tests.
I have requested that Change 25364 be regressed from blead.
I have rerun all the lib/Extutils/ tests with this latest patched
version of CBuilder/Platform/VMS.pm and the original CBuilder/base.pm.
In this patch:
sub need_prelink is left to return 0, because the VMS specific sub
do_link has to set up the pre-link option file when needed.
In sub arg_include_dirs, I think I have fixed the it to properly
extract the include directories from the $self->{config}{ccflags}.
I am still quite a novice in Perl programming.
Not only is the '/inc' case insensitive as you noted, it can be any
thing from: '/inc=[]' to '/include_directory=([],[foo])'.
Next is that I needed to create a VMS specific sub lib_file and put
the Dynaloader::mod2fname() code into it to set the name of the shared
image.
The VMS specific sub _do_link that you provided needed to have the
name changing code removed from it and put into lib_file as I
mentioned above, so that test CBuilder/t/02-link will pass.
I also had to make the addition of the link option files conditional
on if you are building a loadable image.
