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
selenium-server-standalone: restore htmlunit-driver support #21868
Conversation
@hsenag, thanks for your PR! By analyzing the history of the files in this pull request, we identified @offlinehacker, @domenkozar and @utdemir to be potential reviewers. |
homepage = https://github.com/SeleniumHQ/htmlunit-driver; | ||
description = "A WebDriver server for running Selenium tests on the HtmlUnit headless browser"; | ||
platforms = platforms.all; | ||
license = licenses.asl20; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it probably make sense to reuse the maintainer of selenium here.
|
||
unpackPhase = "true"; | ||
|
||
buildInputs = [ ]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line can be removed.
|
||
installPhase = '' | ||
mkdir -p $out/share/lib/${name} | ||
cp $src $out/share/lib/${name}/${name}.jar |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
installPhase = "install -D $src $out/share/lib/${name}/${name}.jar";
It was moved to a separate project between versions 2.45 and 2.53: SeleniumHQ/selenium@2d3150b
5276b09
to
f0c230c
Compare
@Mic92 thanks, I've updated the commit with your suggestions. |
It was moved to a separate project between versions 2.45 and 2.53:
SeleniumHQ/selenium@2d3150b
Motivation for this change
This restores the previous behaviour of selenium-server-standalone, namely having the HtmlUnit driver available. They've been split into separate projects upstream, but it still makes sense for end users to have the whole package. selenium-server-standalone already depended on chromedriver in a similar way.
Things done
(nix.useSandbox on NixOS,
or option
build-use-sandbox
innix.conf
on non-NixOS)
nix-shell -p nox --run "nox-review wip"
./result/bin/
)