Skip Menu |

This queue is for tickets about the Text-CSV_XS CPAN distribution.

Report information
The Basics
Id: 65654
Status: resolved
Priority: 0/
Queue: Text-CSV_XS

People
Owner: Nobody in particular
Requestors: zinser [...] zinser.no-ip.info
Cc:
AdminCc:

Bug Information
Severity: Normal
Broken in: 0.32
Fixed in: 0.81



Subject: Build fails on OpenVMS
Hello Merijn, I've tried to build Text::CSV_XS in the following environment: Text::CSV_XS 0.80 Perl 5.12.2 OpenVMS 8.3-1H1 Running Makefile.PL works fine, but the generated makefile (descrip.mms on OpenVMS) can not be parsed successfully and therefore the module does not build. This is caused by the following line cover test_cover: Multiple targets on one line have to be separated by a , in OpenVMS. This is not a problem with MakeMaker, but gets inserted directly by Makefile.PL The attached patch fixes the problem (conditionally adds a coma in case the script runs on OpenVMS, also adds a space before the colon needed in list mode). This should not have any negative impact on other platforms. It has been successfully cross tested on Linux (openSUSE 11.3). I'd be happy if your would consider this change for inclusion in the next version. Greetings, Martin
Subject: Makefile.PL.patch
*** Makefile.PL 2011-02-11 14:28:09.690636822 -0600 --- Makefile.PL.orig 2010-08-11 02:07:07.000000000 -0500 *************** *** 51,57 **** sub postamble { - my $make_sep = ($^O eq 'VMS') ? "," : ""; my $valgrind = join " ", qw( PERL_DESTRUCT_LEVEL=2 PERL_DL_NONLAZY=1 valgrind --- 51,56 ---- *************** *** 73,79 **** '' : ""; join "\n" => ! "cover $make_sep test_cover :", ' ccache -C', ' -@rm -f *.gc??', ' cover -test', --- 72,78 ---- '' : ""; join "\n" => ! 'cover test_cover:', ' ccache -C', ' -@rm -f *.gc??', ' cover -test',
Comiited and pushed. Thanks