Hey Peter,
        That looks right. I'm going to check in the newest upstream versions to
see if that's the case or not, before I apply the patch. I know there
are some newer versions and all.

                --jay

On Mon, 2005-01-24 at 21:27 +0100, Peter Palfrader wrote:
> Package: libmodule-build-perl
> Version: 0.26-1
> Severity: normal
> 
> When I try to build Class::Methodmaker 2.04:
> 
> [..]
> | cc 
> -I/home/weasel/projects/debian/debian/pkg-libclass-methodmaker-perl/build-tree/debian/tmp/usr/lib/perl/5.8/CORE
>  -fPIC -c -D_REENTRANT -D_GNU_SOURCE -DTHREADS_HAVE_PIDS -DDEBIAN 
> -fno-strict-aliasing -I/usr/local/include -D_LARGEFILE_SOURCE 
> -D_FILE_OFFSET_BITS=64 -O2 -o lib/Class/MethodMaker/Engine.o 
> lib/Class/MethodMaker/Engine.c
> | lib/Class/MethodMaker/Engine.xs:1:20: EXTERN.h: No such file or directory
> | lib/Class/MethodMaker/Engine.xs:2:18: perl.h: No such file or directory
> | lib/Class/MethodMaker/Engine.xs:3:18: XSUB.h: No such file or directory
> 
> As you can see, the call to the c compiler (from process_xs_files(),
> which called process_xs() which did compile_c()), uses -I for a
> directory, which in my case is the installarchlib directory.
> 
> Module::Build should use archlib, not installarchlib here.
> 
> --- /usr/share/perl5/Module/Build/Base.pm.orig        2005-01-24 
> 21:22:31.947433374 +0100
> +++ /usr/share/perl5/Module/Build/Base.pm     2005-01-24 21:26:26.899866876 
> +0100
> @@ -2442,7 +2442,7 @@
>    return $obj_file if $self->up_to_date($file, $obj_file);
>    
>    my @include_dirs = map {"-I$_"} (@{$p->{include_dirs}},
> -                                File::Spec->catdir($cf->{installarchlib}, 
> 'CORE'));
> +                                File::Spec->catdir($cf->{archlib}, 'CORE'));
>    
>    my @extra_compiler_flags = 
> $self->split_like_shell($p->{extra_compiler_flags});
>    my @cccdlflags = $self->split_like_shell($cf->{cccdlflags});
> 
> 
-- 
Jay Bonci    | <[EMAIL PROTECTED]>
GPG: E0B8B2DE| 562B 35DC BE8D 7802 DB31  6423 64D8 790F E0B8 B2DE



Attachment: signature.asc
Description: This is a digitally signed message part

Reply via email to