Skip to content

Commit a6725ae

Browse files
committed
Merge remote-tracking branch 'avtar/GPII-1814'
* avtar/GPII-1814: GPII-1814: Documents the restart step required for interactive VM usage GPII-1814: Moves Universal related tasks to separate batch file GPII-1814: Replaces gpii-universal Grunt task with batch file GPII-1814: Moves npm command to related script GPII-1814: Fixes CI build
2 parents 8d7b3f4 + c803f13 commit a6725ae

File tree

6 files changed

+18
-7
lines changed

6 files changed

+18
-7
lines changed

README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,11 @@ will be withdrawn soon. You can operate these as follows:
4141

4242
It is possible to provision a Windows VM for testing purposes. Please ensure you have met [these VM requirements](https://github.com/GPII/qi-development-environments/#requirements) before proceeding. After that you can use the ``vagrant up`` command to create an instance of a [Windows 10 Evaluation VM](https://github.com/idi-ops/packer-windows) which will boot an instance of the Windows 10 VM, pull in the GPII Framework's npm dependencies, and then build it.
4343

44-
If this is your first time creating this VM a 6.5GB download will take place. The downloaded image will be valid for 90 days after which the Windows installation will no longer be useable. To remove an expired image you can use the ``vagrant box remove "inclusivedesign/windows10-eval"`` command.
44+
If this is your first time creating this VM an 8 GB download will take place. The downloaded image will be valid for 90 days after which the Windows installation will no longer be useable. To remove an expired image you can use the ``vagrant box remove "inclusivedesign/windows10-eval"`` command.
4545

46-
Once the VM has finished booting up you can open a command prompt window and use the following commands to test the framework:
46+
Once the VM has finished booting up you will need to type the ``vagrant reload`` command to cause it to restart. This is required so changes made to the Windows VM's ``PATH`` environment variable as part of the provisioning process are available in terminal sessions. This step is a temporary workaround and will be removed in the near future.
47+
48+
Now you can open a command prompt window and use the following commands to test the framework:
4749

4850
```
4951
cd c:\vagrant

Vagrantfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,5 +37,6 @@ Vagrant.configure(2) do |config|
3737
config.vm.provision "shell", path: "provisioning/chocolatey-packages.bat"
3838
config.vm.provision "shell", path: "provisioning/npm-packages.bat"
3939
config.vm.provision "shell", path: "provisioning/build.bat"
40+
config.vm.provision "shell", path: "provisioning/universal.bat"
4041

4142
end

provisioning/build.bat

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
cd c:\vagrant
2-
npm install --ignore-scripts=true
3-
grunt build
2+
npm install
3+
Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
choco install nodejs -version 4.4.3 --forcex86 -y
2-
choco install nodejs.install -version 4.4.3 --forcex86 -y
3-
setx /M PATH "%PATH%;c:\programdata\chocolatey\lib\nodejs.commandline\tools;c:\users\vagrant\appdata\roaming\npm"
1+
choco install nodejs.install -version 4.4.3 -y
2+
choco install python2 -y
3+
setx /M PATH "%PATH%;C:\Program Files\nodejs;C:\tools\python2"

provisioning/npm-packages.bat

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,3 @@
1+
call npm config set msvs_version 2015 --global
12
call npm install grunt-cli -g
23
call npm install testem -g

provisioning/universal.bat

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
mkdir c:\node_modules
2+
cd c:\node_modules
3+
git clone --depth 1 https://github.com/GPII/universal.git
4+
cd universal
5+
npm install
6+
npm install dedupe-infusion
7+
node -e "require('dedupe-infusion')()"

0 commit comments

Comments
 (0)