[PATCH] EU::CBuilder compile supporting include_dirs as a string

Alberto Simões <[email protected]> Sat, 13 Feb 2010 20:23:25 +0000

Newsgroups	gmane.comp.lang.perl.modules.module-build
Organization	Departamento de Informática - Universi dade do Minho
Message-ID	<[email protected]>

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

EU-CB.diff (text/plain, 1.1 KB)

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