Bug #55753 for Text-Sass: dist

Sat Mar 20 21:05:52 2010 miyagawa [...] gmail.com - Ticket created

Subject:	dist_version in Build.PL
Date:	Sat, 20 Mar 2010 18:05:02 -0700
To:	bug-Text-Sass [...] rt.cpan.org
From:	Tatsuhiko Miyagawa <miyagawa [...] gmail.com>

You're doing this in Build.PL -- dist_version => do { require 'lib/Text/Sass.pm'; $Text::Sass::VERSION; }, Don't do it, because when installers (CPAN.pm/PLUS/minus) runs Build.PL, dependencies required in your Text/Sass.pm such as Readonly.pm are not satisfied yet and your Build.PL just dies. -- Tatsuhiko Miyagawa

Tue Mar 23 12:43:00 2010 dolmen [...] cpan.org - Correspondence added

Le Sam 20 Mar 2010 21:05:52, miyagawa@gmail.com a écrit : Show quoted text

> You're doing this in Build.PL -- > > dist_version => do { require 'lib/Text/Sass.pm'; > $Text::Sass::VERSION; }, > > Don't do it, because when installers (CPAN.pm/PLUS/minus) runs > Build.PL, dependencies required in your Text/Sass.pm such as > Readonly.pm are not satisfied yet and your Build.PL just dies.

I agree. The fix: - module_name => 'Text-Sass', + module_name => 'Text::Sass', - dist_version => do { require 'lib/Text/Sass.pm'; $Text::Sass::VERSION; }, dist_abstract => q[Text-Sass - Perl implementation of http://sass-lang.com/], -- Olivier Mengué - http://o.mengue.free.fr/

Tue Mar 23 12:43:01 2010 The RT System itself - Status changed from 'new' to 'open'

Tue Mar 23 13:05:53 2010 rmp [...] psyphi.net - Correspondence added

0.5 with improved Build.PL going into CPAN now.

Tue Mar 23 13:05:54 2010 rmp [...] psyphi.net - Status changed from 'open' to 'resolved'

Bug #55753 for Text-Sass: dist_version in Build.PL