Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG][0.12.1] Unable to install on mac10.13 #1535

Closed
roblav96 opened this issue Mar 25, 2020 · 8 comments
Closed

[BUG][0.12.1] Unable to install on mac10.13 #1535

roblav96 opened this issue Mar 25, 2020 · 8 comments

Comments

@roblav96
Copy link

Context:

  • Playwright Version: v0.12.1
  • Operating System: Mac
  • Extra: macOS High Sierra 10.13.6

Code Snippet
npm install playwright-core playwright-chromium

~/project on  master [!?] 
$ npm install playwright-core playwright-chromium

> [email protected] install /Users/admin/project/node_modules/playwright-chromium
> node install.js

(node:54634) UnhandledPromiseRejectionWarning: TypeError: path.join is not iterable (cannot read property Symbol(Symbol.iterator))
    at Object.executablePath (/Users/admin/project/node_modules/playwright-core/lib/server/browserFetcher.js:125:17)
    at downloadOptionsFromENV (/Users/admin/project/node_modules/playwright-core/download-browser.js:44:36)
    at /Users/admin/project/node_modules/playwright-chromium/install.js:21:40
    at Object.<anonymous> (/Users/admin/project/node_modules/playwright-chromium/install.js:22:3)
    at Module._compile (internal/modules/cjs/loader.js:1147:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:1167:10)
    at Module.load (internal/modules/cjs/loader.js:996:32)
    at Function.Module._load (internal/modules/cjs/loader.js:896:14)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:71:12)
    at internal/main/run_main_module.js:17:47

Describe the bug

Can't npm install playwright-chromium, throws at lib/server/browserFetcher.js:125:17<executablePath> due to mac10.13 not being mapped on the RELATIVE_EXECUTABLE_PATHS.

@roblav96 roblav96 changed the title [BUG] Unable to install on mac10.13 [BUG][v0.12.1] Unable to install on mac10.13 Mar 25, 2020
@roblav96 roblav96 changed the title [BUG][v0.12.1] Unable to install on mac10.13 [BUG][0.12.1] Unable to install on mac10.13 Mar 25, 2020
@mxschmitt
Copy link
Member

Which Node.js version are you using?

@roblav96
Copy link
Author

@mxschmitt

$ node -v
v13.11.0
$ npm -v
6.14.3

@roblav96
Copy link
Author

The sw_vers -productVersion resulting in mac10.13 need to be mapped on the variables DOWNLOAD_URLS and RELATIVE_EXECUTABLE_PATHS

@mxschmitt
Copy link
Member

mxschmitt commented Mar 25, 2020

In theory we could probably just fire up a Pull Request to add the mapping there. But not sure if Playwright will support Mac OS 10.13 (was released in September 2017).
Also the error handling could be improved if an unsupported OS/Version will be used.

cc @JoelEinbinder

@dgozman
Copy link
Contributor

dgozman commented Mar 26, 2020

We are currently only building bundling WebKit and Firefox for 10.14 and 10.15. @aslushnikov, do you think 10.13 is viable?

@aslushnikov
Copy link
Contributor

@roblav96
Copy link
Author

We are currently only building bundling WebKit and Firefox for 10.14 and 10.15. @aslushnikov, do you think 10.13 is viable?

I believe 10.13 is still viable as long as homebrew continues to publish a :high_sierra bottle for the node formula.

aslushnikov pushed a commit that referenced this issue Mar 30, 2020
)

I don't have a macos 10.13 build to test on, and we aren't set up to compile it for WebKit. However there is a good chance this will work for Chromium and Firefox.

I also improved the error message received when on an unsupported platform.

#1535
@aslushnikov
Copy link
Contributor

Fixed with #1549

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants