What's up with "No targets specified and no makefile found"?

2009-12-21 Thread Joshua ben Jore
The CPAN smokers regularly spit back UNKNOWN results where the text of the error is: Output from '/usr/bin/make': make: *** No targets specified and no makefile found. Stop. One example is http://www.nntp.perl.org/group/perl.cpan.testers/2009/07/msg4616138.html but there are many, many

Re: What's up with "No targets specified and no makefile found"?

2009-12-21 Thread David Golden
You need to be asking on cpan-testers-discuss. Copying that list now. -- David On Mon, Dec 21, 2009 at 4:02 PM, Joshua ben Jore wrote: > The CPAN smokers regularly spit back UNKNOWN results where the text of > the error is: > >    Output from '/usr/bin/make': > >    make: *** No targets specifi

Re: What's up with "No targets specified and no makefile found"?

2009-12-21 Thread Chris 'BinGOs' Williams
On Mon, Dec 21, 2009 at 04:15:22PM -0500, David Golden wrote: > You need to be asking on cpan-testers-discuss. Copying that list now. > > -- David > > On Mon, Dec 21, 2009 at 4:02 PM, Joshua ben Jore wrote: > > The CPAN smokers regularly spit back UNKNOWN results where the text of > > the error

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Joshua ben Jore
On Mon, Dec 21, 2009 at 11:19 PM, Chris 'BinGOs' Williams wrote: > On Mon, Dec 21, 2009 at 04:15:22PM -0500, David Golden wrote: >> You need to be asking on cpan-testers-discuss.  Copying that list now. >> >> -- David >> >> On Mon, Dec 21, 2009 at 4:02 PM, Joshua ben Jore wrote: >> > The CPAN smo

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Nicholas Clark
On Tue, Dec 22, 2009 at 07:03:49AM -0800, Joshua ben Jore wrote: > Where is EU::MM's version control? I'd like to submit a patch adding a bunch > of: > > or croak("Can't write '$fn': $!"); Good plan. (At least, in my opinion) http://search.cpan.org/~mschwern/ExtUtils-MakeMaker-6.56/ says

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Michael G Schwern
Joshua ben Jore wrote: The just-released EU::MM 6.56 repeats this pattern frequently: open my($fh), '>', ... or croak("Can't open ... for writing: $!"); ... print $fh ...; # <<< no error checking close $fh ...; # <<< no error checking Grepping for code: egrep -rnH '

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Joshua ben Jore
On Tue, Dec 22, 2009 at 2:23 PM, Michael G Schwern wrote: > Joshua ben Jore wrote: >> >> The just-released EU::MM 6.56 repeats this pattern frequently: >> >>    open my($fh), '>', ... >>        or croak("Can't open ... for writing: $!"); >>    ... >>    print $fh ...; # <<< no error checking >>  

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Michael G Schwern
Joshua ben Jore wrote: Fortunately there is one key location where MakeMaker does almost all its writing to disk, ExtUtils::MakeMaker->flush. A simple patch to check that would be lovely. If you want to write and insert a safe _print() wrapper method and scatter that around that's fine, too.

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread chromatic
On Tuesday 22 December 2009 at 14:23, Michael G wrote: > Putting in error checking at some key locations is fine, but the prospect of > adding `or die "Can't print $foo: $!"` to every print and close makes me > want to claw my eyeballs out. Its unfortunate MakeMaker can't rely on > autodie and

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Joshua ben Jore
On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: > A little experimentation with a small disk image shows that close() will > error if there's no disk left.  No need to check every print.  And a close() You have to check every print. Most prints will just extend the buffer but some print

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Joshua ben Jore
On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: > A little experimentation with a small disk image shows that close() will > error if there's no disk left.  No need to check every print.  And a close() > wrapper is trivial.  It does mean there needs to be a close() for every > open() whi

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Joshua ben Jore
On Tue, Dec 22, 2009 at 9:29 PM, Marvin Humphrey wrote: > On Tue, Dec 22, 2009 at 09:03:42PM -0800, Joshua ben Jore wrote: >> On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: >> > A little experimentation with a small disk image shows that close() will >> > error if there's no disk left.

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Michael G Schwern
Marvin Humphrey wrote: On Tue, Dec 22, 2009 at 09:03:42PM -0800, Joshua ben Jore wrote: On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: A little experimentation with a small disk image shows that close() will error if there's no disk left. No need to check every print. And a close(

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Michael G Schwern
Joshua ben Jore wrote: On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: A little experimentation with a small disk image shows that close() will error if there's no disk left. No need to check every print. And a close() wrapper is trivial. It does mean there needs to be a close() fo

Re: What's up with "No targets specified and no makefile found"?

2009-12-22 Thread Marvin Humphrey
On Tue, Dec 22, 2009 at 09:03:42PM -0800, Joshua ben Jore wrote: > On Tue, Dec 22, 2009 at 4:29 PM, Michael G Schwern wrote: > > A little experimentation with a small disk image shows that close() will > > error if there's no disk left.  No need to check every print.  And a close() > > You have t

Re: What's up with "No targets specified and no makefile found"?

2009-12-24 Thread Paul Johnson
On Tue, Dec 22, 2009 at 04:29:27PM -0800, Michael G Schwern wrote: > A little experimentation with a small disk image shows that close() will > error if there's no disk left. No need to check every print. And a > close() wrapper is trivial. It does mean there needs to be a close() for > ever