Greetings,

currently, the test suite for CPANPLUS::Dist::Build is failing with:

  t/02_CPANPLUS-Dist-Build..............ok 13/0Warning: this distribution
contains XS files, but Module::Build is not configured with C_support at
/opt/lib/perl5/site_perl/5.8.8/Module/Build/Base.pm line 1128.
 Warning: this distribution contains XS files, but Module::Build is not
configured with C_support at
/opt/lib/perl5/site_perl/5.8.8/Module/Build/Base.pm line 1128.

It'd be nice to put these tests in SKIP blocks when C_support is not
available. Looking up the
line of the code that emits this error, i see it's using a private method:

  if (keys %$xs_files && !$self->_mb_feature('C_support')) {
    $self->log_warn("Warning: this distribution contains XS files, ".
            "but Module::Build is not configured with C_support");
  }

I've attached a patch for CPANPLUS::Dist::Build that makes tests pass
again by detecting C support using the above mechanism. It also
removes a silly warning emitted during tests. Feel free to apply and
release if this is the proper way to do things, or adjust as you see fit.

Thanks,

Jos

Attachment: C-D-Build-c_support.patch
Description: Binary data

Reply via email to