Hey David, hi MB fans,

Accordingly with the documentation, ExtUtils::CBuilder compile method
support 'include_dirs' as

    Specifies any additional directories in which to search for header
    files. May be given as a string indicating a single directory, or
    as a list reference indicating multiple directories.

But in latest version a string is not allowed.

Patch fixes that.

Cheers
Alberto

-- 
Alberto Simões
Only in ExtUtils-CBuilder-mine/: Makefile
Only in ExtUtils-CBuilder-mine/: blib
diff -rc ExtUtils-CBuilder-0.27/lib/ExtUtils/CBuilder/Base.pm 
ExtUtils-CBuilder-mine/lib/ExtUtils/CBuilder/Base.pm
*** ExtUtils-CBuilder-0.27/lib/ExtUtils/CBuilder/Base.pm        2009-10-30 
01:30:17.000000000 +0000
--- ExtUtils-CBuilder-mine/lib/ExtUtils/CBuilder/Base.pm        2010-02-13 
20:18:35.000000000 +0000
***************
*** 89,95 ****
    my $cf = $self->{config}; # For convenience
  
    $args{object_file} ||= $self->object_file($args{source});
!   
    my @include_dirs = $self->arg_include_dirs
      (@{$args{include_dirs} || []},
       $self->perl_inc());
--- 89,97 ----
    my $cf = $self->{config}; # For convenience
  
    $args{object_file} ||= $self->object_file($args{source});
! 
!   $args{include_dirs} = [ $args{include_dirs} ] if 
exists($args{include_dirs}) &&
!                                                    ref($args{include_dirs}) 
ne "ARRAY";
    my @include_dirs = $self->arg_include_dirs
      (@{$args{include_dirs} || []},
       $self->perl_inc());
Only in ExtUtils-CBuilder-mine/: pm_to_blib

Reply via email to