Last Comment Bug 576105 - install-module.pl cannot install DateTime or DateTime::TimeZone on Windows
: install-module.pl cannot install DateTime or DateTime::TimeZone on Windows
Status: RESOLVED FIXED
:
Product: Bugzilla
Classification: Server Software
Component: Installation & Upgrading (show other bugs)
: 3.7
: All All
: -- normal (vote)
: Bugzilla 3.6
Assigned To: Max Kanat-Alexander
: default-qa
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2010-06-30 14:42 PDT by Max Kanat-Alexander
Modified: 2010-07-05 16:59 PDT (History)
0 users
mkanat: approval+
mkanat: blocking4.0+
mkanat: approval3.6+
See Also:
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
v1 (353 bytes, patch)
2010-07-05 16:56 PDT, Max Kanat-Alexander
mkanat: review+
Details | Diff | Splinter Review

Description Max Kanat-Alexander 2010-06-30 14:42:51 PDT
We set mbuild_install_build_command to "./Build", which isn't working on Windows, so modules that exclusively use Module::Build can't install. Right now, that means DateTime doesn't install properly.
Comment 1 Max Kanat-Alexander 2010-07-05 16:56:12 PDT
Created attachment 456119 [details] [diff] [review]
v1

Turns out we don't need to set ./Build at all--CPAN just knows that that's the right default, and because we install a newer CPAN during install-module, it's not a problem anyway. So we can remove this on both trunk and 3.6.
Comment 2 Max Kanat-Alexander 2010-07-05 16:59:32 PDT
Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/Install/CPAN.pm
Committed revision 7269.

Committing to: bzr+ssh://bzr.mozilla.org/bugzilla/3.6/
modified Bugzilla/Install/CPAN.pm
Committed revision 7121.

Note You need to log in before you can comment on or make changes to this bug.