Skip to content

Commit

Permalink
Add the < version check for nodejs/node-v0.x-archive#2685, now that i…
Browse files Browse the repository at this point in the history
…ts been merged.
  • Loading branch information
TooTallNate committed Feb 6, 2012
1 parent 1c4f2e9 commit b182b3a
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions lib/configure.js
Original file line number Diff line number Diff line change
Expand Up @@ -39,10 +39,13 @@ function configure (gyp, argv, callback) {
, gyp_addon = path.join(devDir, 'tools', 'gyp_addon')

if (win) {
// add a <= version check when joyent/node#2685 gets merged
argv.push('-Dnode_root_dir=' + devDir)
argv.push('-I')
argv.push(path.join(devDir, 'tools', 'patch.gypi'))
if (version < 0.8) {
// if < 0.8, we need to manually apply the patch at joyent/node#2685,
// since it got merged somewhere in 0.7.x.
argv.push('-Dnode_root_dir=' + devDir)
argv.push('-I')
argv.push(path.join(devDir, 'tools', 'patch.gypi'))
}
} else {
// Force the 'make' target for non-Windows
argv.unshift('.gyp')
Expand Down

0 comments on commit b182b3a

Please sign in to comment.