develooper Front page | perl.perl5.porters | Postings from May 2004

Re: [perl #29623] Patch for h2xs.t in Perl 5.8.4

From:
Rafael Garcia-Suarez
Date:
May 18, 2004 06:17
Subject:
Re: [perl #29623] Patch for h2xs.t in Perl 5.8.4
Message ID:
20040518151441.4d7734cb@localhost
mats@sm5sxl.net (via RT) wrote:
> The '$header' variable in the parameter list at line 120 in 
> lib/h2xs.t is incorrectly placed (it should be the last one). It will
> cause the h2xs test to fail as follows:
...
> Below is a patch to fix it.

I've applied your patches to the development version of perl, thanks
(change #22828).

What I don't explain, is, why does this test work for me ? I can see
that h2xs uses Getopt::Long with the configuration option pass_through,
so your patch looks fine, but it may only hide a deeper bug.

> *** h2xs.t.orig	Tue Aug 12 11:42:37 2003
> --- h2xs.t	Sun May 16 15:13:38 2004
> ***************
> *** 117,123 ****
>   Writing $name/MANIFEST
>   EONOXSFILES
>   
> ! "-f -n $name $header -b $thisversion", $], <<"EOXSFILES",
>   Writing $name/ppport.h
>   Writing $name/lib/$name.pm
>   Writing $name/$name.xs
> --- 117,123 ----
>   Writing $name/MANIFEST
>   EONOXSFILES
>   
> ! "-f -n $name -b $thisversion $header", $], <<"EOXSFILES",
>   Writing $name/ppport.h
>   Writing $name/lib/$name.pm
>   Writing $name/$name.xs



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About