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

Adding support for Phantom 2.5.0 beta #689

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

phawxby
Copy link

@phawxby phawxby commented Apr 25, 2017

Also fixed the proxy env var to also use the standard env var.

#668

@phawxby
Copy link
Author

phawxby commented Apr 25, 2017

V2.1:
928f78fe-29b3-11e7-9f79-832af397bd0f

V2.5 beta:
homepage--en-us--desktop

lib/util.js Outdated
} else if (platform === 'linux' && arch == 'ia32') {
downloadUrl += 'linux-i686.tar.bz2'
checksum = '80e03cfeb22cc4dfe4e73b68ab81c9fdd7c78968cfd5358e6af33960464f15e3'
downloadUrl += 'beta-linux-ubuntu-trusty-x86_64.tar.gz'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this won't work, because not all linux distros are ubuntu trusty.

@phawxby
Copy link
Author

phawxby commented Apr 25, 2017 via email

@nicks
Copy link
Contributor

nicks commented Apr 25, 2017

That works in a world where no one has dependencies on anyone else. In the world we live in, there are transitive dependencies, and anyone who upgraded would break linux users downstream, which we would then have to support.

If you'd like to try 2.5 for your own purposes, you're free to create a new, separate npm package that just installs 2.5

@phawxby
Copy link
Author

phawxby commented Apr 25, 2017

@nicks Just testing an update to better handle Linux, would this be any better?

@phawxby
Copy link
Author

phawxby commented Apr 25, 2017

@nicks you'll have to forgive me on my poor knowledge of Linux. I've now got this working on Ubuntu however it has an OS dependency on libhyphen-dev libwebp-dev gcc-4.9 g++-4.9. What's the best way to signal this dependency to consumers or correct it during install?

@nicks
Copy link
Contributor

nicks commented Apr 25, 2017

ugh : (

There is no good way to signal these dependencies downstream. NPM doesn't have a good mechanism for this.

just to make sure we're on the same page, are you aware that PhantomJS 2.5 might never come out of beta?
https://groups.google.com/forum/#!topic/phantomjs/9aI5d-LDuNE

This is why we haven't pushed out a 2.5 installer - I don't want to push a bunch of compatibility/upgrade work onto downstream packages that ends up being a waste of their time.

@phawxby
Copy link
Author

phawxby commented Apr 25, 2017

Ugh 2, i had missed that announcement. It's a shame the repo is split between github, bitbucket and Google groups.

Well I'm not sure what to do with this PR then.

I'm guessing something like webdriver.io would be a the way forward for new projects or is there a direct integration for Casper into chrome headless?

@phawxby phawxby mentioned this pull request May 12, 2017
3 tasks
@tommedema
Copy link

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

Successfully merging this pull request may close these issues.

None yet

3 participants