Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

Commit

Permalink
Node 6 with AppVeyor: don't use subst for testing (#1514)
Browse files Browse the repository at this point in the history
Node 6.x has trouble including modules
from MS-DOS drives created with subst.exe.

Since we use use subst only to set
symbols paths in the PDB file, let's
revert to the C: drive when doing tests.

Workaround for: nodejs/node#6500
  • Loading branch information
saper authored and xzyfer committed May 2, 2016
1 parent 0bc5da4 commit 15fe42e
Showing 1 changed file with 8 additions and 2 deletions.
10 changes: 8 additions & 2 deletions appveyor.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@
- nodejs_version: 3
- nodejs_version: 4
- nodejs_version: 5
- nodejs_version: 6

install:
- ps: Install-Product node $env:nodejs_version $env:platform
Expand All @@ -50,7 +51,9 @@
- git submodule update --init --recursive
- npm install --msvs_version=2013

test_script: npm test
test_script:
- ps: set-location -path c:\projects\node_modules\node-sass
- npm test

before_deploy:
# Save artifacts with full qualified names of binding.node and binding.pdb
Expand Down Expand Up @@ -105,6 +108,7 @@
- nodejs_version: 0.12
- nodejs_version: 4
- nodejs_version: 5
- nodejs_version: 6

install:
- ps: Install-Product node $env:nodejs_version $env:platform
Expand All @@ -113,4 +117,6 @@
- git submodule update --init --recursive
- npm install --msvs_version=2013

test_script: npm test
test_script:
- ps: set-location -path c:\projects\node_modules\node-sass
- npm test

0 comments on commit 15fe42e

Please sign in to comment.