Skip Menu |

This queue is for tickets about the Module-Build CPAN distribution.

Report information
The Basics
Id: 113682
Status: open
Priority: 0/
Queue: Module-Build
People
Owner: Nobody in particular
Requestors: ribasushi [...] leporine.io
Cc:
AdminCc:
Bug Information
Severity: Normal
Broken in: (no value)
Fixed in: (no value)

History Show all quoted text
  Mon Apr 11 10:12:43 2016 ribasushi [...] leporine.io - Ticket created
Subject: Fails tests if cc in $PATH is a random 0-returning program ( all fine if the program returns > 0)
See topic and attached session log.
Subject: mb_argh.txt

Message body is not shown because it is too large.

  Mon Apr 11 13:21:55 2016 kwilliams [...] cpan.org - Correspondence added
Not sure I understand the problem. If you replace CC with something that claims to be succeeding but doesn't do what CC does, and Module::Build tries to use it, isn't failure expected? A compiler isn't required for M::B, but if it's there, apparently "working", and non-functional, failure seems the right way to go.
  Mon Apr 11 13:21:55 2016 The RT System itself - Status changed from 'new' to 'open'
  Wed Apr 13 12:06:43 2016 fawaka [...] gmail.com - Correspondence added
Subject: Re: [rt.cpan.org #113682] Fails tests if cc in $PATH is a random 0-returning program ( all fine if the program returns > 0)
Date: Wed, 13 Apr 2016 18:06:13 +0200
To: bug-Module-Build [...] rt.cpan.org
From: Leon Timmermans <fawaka [...] gmail.com>
On Mon, Apr 11, 2016 at 4:12 PM, Peter Rabbitson via RT < bug-Module-Build@rt.cpan.org> wrote: Show quoted text
> See topic and attached session log. >
That test almost does the right thing but ignores to check the result of the executable that has been build, fixing that was surprisingly easy (see attachment). A different test, based on checking if it can compile a loadable object is probably more reliable though. Leon

Message body is not shown because sender requested not to inline it.

  Thu May 19 12:39:21 2016 ribasushi [...] leporine.io - Correspondence added
On Wed Apr 13 18:06:43 2016, fawaka@gmail.com wrote: Show quoted text
> > That test almost does the right thing but ignores to check the result of > the executable that has been build, fixing that was surprisingly easy (see > attachment).
There are couple more spots like this, see https://github.com/Perl-Toolchain-Gang/Module-Build/commit/45fb63cccad7bb089ec6ba6a87683eb530a1d96c#commitcomment-17543692
  Sat Aug 06 17:58:34 2016 LEONT [...] cpan.org - Severity Normal added