Merge pull request #130 from Tyriar/windows_demo

Get demo working on Windows
This commit is contained in:
Antonis Kalipetis
2016-06-23 11:52:16 +03:00
committed by GitHub
5 changed files with 6 additions and 14 deletions
+1 -1
View File
@@ -19,7 +19,7 @@ npm install
npm start
```
Then open http://0.0.0.0:3000 in a web browser.
Then open http://0.0.0.0:3000 in a web browser (use http://127.0.0.1:3000 is running under Windows).
## Addons
-4
View File
@@ -1,4 +0,0 @@
#! /usr/bin/env sh
# Development server for xterm.js
node demo/app.js
-5
View File
@@ -1,5 +0,0 @@
#! /bin/bash
#
# Testing script for xterm.js
node_modules/.bin/mocha $@
+3 -2
View File
@@ -1,6 +1,7 @@
var express = require('express');
var app = express();
var expressWs = require('express-ws')(app);
var os = require('os');
var pty = require('pty.js');
app.use('/src', express.static(__dirname + '/../src'));
@@ -22,7 +23,7 @@ app.ws('/bash', function(ws, req) {
/**
* Open bash terminal and attach it
*/
var term = pty.spawn('bash', [], {
var term = pty.spawn(process.platform === 'win32' ? 'cmd.exe' : 'bash', [], {
name: 'xterm-color',
cols: 80,
rows: 24,
@@ -46,7 +47,7 @@ app.ws('/bash', function(ws, req) {
});
var port = process.env.PORT || 3000,
host = '0.0.0.0';
host = os.platform() === 'win32' ? '127.0.0.1' : '0.0.0.0';
console.log('App listening to http://' + host + ':' + port);
app.listen(port, host);
+2 -2
View File
@@ -15,8 +15,8 @@
"docdash": "0.4.0"
},
"scripts": {
"start": "bash bin/server",
"test": "bash bin/test --recursive",
"start": "node demo/app",
"test": "mocha --recursive",
"build:docs": "node_modules/.bin/jsdoc -c jsdoc.json"
}
}