Windows env fix#1309
Conversation
|
CC @MoLow |
|
@jasonLaster whats this? |
|
Hi! We're seeing a couple issues with windows contributors getting started.
Mind taking a look and helping?
…On Thu, Nov 24, 2016 at 3:42 PM Moshe Atlow ***@***.***> wrote:
@jasonLaster <https://github.com/jasonLaster> whats this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1309 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAPiYiSjfVqKBr9SFRwT3osokvTKrp82ks5rBfbIgaJpZM4K7zKj>
.
|
|
Hi, sorry for the delay in answering. In the meanwhile i tested on a computer with no admin priviligaes, so I could not install yarn, neither update node to > 6.9.0, so I tried running it directly, and this is the results: I'l try it later today on my computer using npm run/yarn start/yarn run. BTW, I havent tried yet, but this maybe a simple solution: https://www.npmjs.com/package/npm-run-path |
I believe with When you run |
|
Ok, and I just looked into this further. I think the first solution is to update our instructions. The error you're seeing in that screenshot is because you don't have the
Now when you run I am seeing a problem where you I don't believe the debugger server is running, looking into that now. |
|
@clarkbw well, the question is if this could be done automatically on npm install or on npm run firefox, since the exe is installed to node_modules\geckodriver, and it only needs to get into the path in runtime. |
Yes, sorry. What I have above shouldn't actually be the solution, just short term. On the Mac this is done automatically and it should be on Windows as well. |
|
@clarkbw perhaps look at npm run path issues for windows |
|
Oh, hmm. I'm seeing |
|
Or, actually |
|
The |
|
I've been trying out this solution, creating the driver manually, but still not having success: let options = new firefox.Options();
let service = new firefox.ServiceBuilder(__dirname + '\\..\\node_modules\\.bin\\geckodriver.cmd').build();
options.setProfile(firefoxProfile());
options.setBinary(firefoxBinary());
const driver = new firefox.Driver(options, service);It finds the local gecko driver but errors out elsewhere before running anything. |
|
Mabe the simplest solution is to add these lines to firefox-driver.js?:
is this a BAD idea? |
As I understand it, the I think you need to do something like this: if (isWindows) {
var path = require('path');
process.env.PATH = path.resolve(__dirname, '../node_modules/geckodriver/bin') + ';' + process.env.PATH;
} |




Associated Issue: #1248
Summary of Changes
*edited firefox-driver.js to run 'yarn start firefox'
Test Plan
Tell us a little a bit about how you tested your patch.
Example test plan:
*Navigate to debugger.html folder from command prompt from windows
*Execute "yarn start firefox" to start the server
*Open browser and go to localhost:8000
Output: debugger.html is up and running
Screenshots/Videos (OPTIONAL)
yarn start firefox vs yarn run firefox