Skip Menu |

This queue is for tickets about the Image-Info CPAN distribution.

Report information
The Basics
Id: 18136
Status: resolved
Worked: 20 min
Priority: 0/
Queue: Image-Info
People
Owner: TELS [...] cpan.org
Requestors: DON [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: Critical
Broken in: 1.19
Fixed in: 1.20
Attachments
signature.asc

History Show all quoted text
  Sun Mar 12 20:51:23 2006 DON [...] cpan.org - Ticket created
Subject: perl Makefile.PL fails
$ perl Makefile.PL String found where operator expected at Makefile.PL line 89, near "name 'Image-Info'" (Do you need to predeclare name?) String found where operator expected at Makefile.PL line 94, near "requires 'IO::String'" (Do you need to predeclare requires?) String found where operator expected at Makefile.PL line 96, near "recommends 'Image::Xpm'" (Do you need to predeclare recommends?) String found where operator expected at Makefile.PL line 100, near "build_requires 'Test::More'" (Do you need to predeclare build_requires?) String found where operator expected at Makefile.PL line 102, near "license 'perl'" (Do you need to predeclare license?) String found where operator expected at Makefile.PL line 104, near "author 'Tels <nospam-abuse@bloodgate.com>'" (Do you need to predeclare author?) syntax error at Makefile.PL line 89, near "name 'Image-Info'" syntax error at Makefile.PL line 94, near "requires 'IO::String'" Bareword "auto_install" not allowed while "strict subs" in use at Makefile.PL line 110. Bareword "WriteAll" not allowed while "strict subs" in use at Makefile.PL line 113. Execution of Makefile.PL aborted due to compilation errors. Also, I don't see why you've changed the build system from Extutils::MakeMaker to Module::Install, and then proceeded to make the builds of the perl module different depending on what operating system the perl module is on. [It would be better to just document the difference, and have the perl module figure this out on its own so code can be moved seamlessly between operating systems.]
  Mon Mar 13 02:08:53 2006 nospam-abuse [...] bloodgate.com - Correspondence added
Subject: Re: [rt.cpan.org #18136] perl Makefile.PL fails
Date: Mon, 13 Mar 2006 08:08:18 +0100
To: bug-Image-Info [...] rt.cpan.org
From: Tels <nospam-abuse [...] bloodgate.com>
Moin, On Monday 13 March 2006 02:51, via RT wrote: Show quoted text
> Sun Mar 12 20:51:23 2006: Request 18136 was acted upon. > Transaction: Ticket created by DON > Queue: Image-Info > Subject: perl Makefile.PL fails > Owner: Nobody > Requestors: DON@cpan.org > Status: new > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=18136 > > > > $ perl Makefile.PL > String found where operator expected at Makefile.PL line 89, near "name > 'Image-Info'" > (Do you need to predeclare name?)
This is a bug in the bundled Module::Install. It will be fixed by the next version. Sorry for that. Show quoted text
> Also, I don't see why you've changed the build system from > Extutils::MakeMaker to Module::Install,
Thats a good question. Module::Install seemed to offer a few things over MakeMaker (and uses it behind the scene, anyway). It should also totally transparent to the user, except features like offering to install missing modules for the users and makes life for me as author easier by f.i. writing a better META.yml file without me mucking in Makefile.PL. However, so far it has created a few niggles that I need to either fix, or drop it again. Show quoted text
> and then proceeded to make the > builds of the perl module different depending on what operating system > the perl module is on. [It would be better to just document the > difference, and have the perl module figure this out on its own so code > can be moved seamlessly between operating systems.]
This code was there in from the beginning and was not changed by me at all. And given the fragile state Image::Info is in, I will likely not change this now, because this would just introduce new bugs and problems while not really solve anything (how often do you "compile" a package on one system, then move it over to a completely different one?) Best wishes, Tels -- Signed on Mon Mar 13 08:00:59 2006 with key 0x93B84C15. Visit my photo gallery at http://bloodgate.com/photos/ PGP key on http://bloodgate.com/tels.asc or per email. "What you have to have faith in is human beings being able to work together." -- Larry Sanger
Download (untitled)
application/pgp-signature 481b

Message body not shown because it is not plain text.

  Mon Mar 13 02:08:55 2006 The RT System itself - Status changed from 'new' to 'open'
  Mon Mar 13 02:26:36 2006 don [...] donarmstrong.com - Correspondence added
Subject: Re: [rt.cpan.org #18136] perl Makefile.PL fails
Date: Sun, 12 Mar 2006 23:25:46 -0800
To: bug-Image-Info [...] rt.cpan.org
From: Don Armstrong <don [...] donarmstrong.com>

Message body is not shown because it is too large.

Download signature.asc
application/pgp-signature 189b

Message body not shown because it is not plain text.

  Mon Mar 13 15:58:22 2006 nospam-abuse [...] bloodgate.com - Correspondence added
Subject: Re: [rt.cpan.org #18136] perl Makefile.PL fails
Date: Mon, 13 Mar 2006 21:58:01 +0100
To: bug-Image-Info [...] rt.cpan.org
From: Tels <nospam-abuse [...] bloodgate.com>
Moin, On Monday 13 March 2006 08:26, Don Armstrong via RT wrote: Show quoted text
> Queue: Image-Info > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=18136 > >
> > This is a bug in the bundled Module::Install. It will be fixed by > > the next version. Sorry for that.
> > No worries.
I just released Image::Info 1.20, which should fix the Module::Install bug, as well as the test failings in the TIFF testsuite. About build vs. run-time: Show quoted text
> > This code was there in from the beginning and was not changed by me > > at all.
> > You're right; I noticed that after I sent the bug report. >
> > how often do you "compile" a package on one system, then move it > > over to a completely different one?
> > Surprisingly often, actually... but that's because I package > Image::Info for Debian.
Oh. Sorry, please disregard my comment then. So, if it would make your life much easier, then I will fix that later. Please file a new bug report about the building of lib/Image/Info.pm from Info.pm.tmpl. (Actually, I kinda dislike that way, too, because I lost quite a few things due to editing the generated Info.pm file accidentily :D Thanx for your report, Tels -- Signed on Mon Mar 13 21:55:50 2006 with key 0x93B84C15. Visit my photo gallery at http://bloodgate.com/photos/ PGP key on http://bloodgate.com/tels.asc or per email. "Proctor & Gamble unveiled a new soap this week. Although it looks normal, the soap is actually hollow, which eliminates those little pieces that are always left at the end." -- SNL
Download (untitled)
application/pgp-signature 481b

Message body not shown because it is not plain text.

  Tue Mar 14 12:02:40 2006 TELS [...] cpan.org - TimeWorked changed from (no value) to '20'
  Tue Mar 14 12:02:41 2006 TELS [...] cpan.org - Status changed from 'open' to 'resolved'
  Tue Mar 14 12:02:42 2006 TELS [...] cpan.org - Severity Important changed to Critical
  Tue Mar 14 12:02:42 2006 TELS [...] cpan.org - Fixed in 1.20 added
  Sat Apr 29 12:39:08 2006 TELS [...] cpan.org - Taken