Repository navigation
Add a --without-ssl node-test-commit job #643
Description
See nodejs/node#11734, it would be useful to have some testing to catch the non-default build cases (there are probably more than just --without-ssl).
{{ message }}
See nodejs/node#11734, it would be useful to have some testing to catch the non-default build cases (there are probably more than just --without-ssl).
Activity
richardlau commentedon Mar 8, 2017
Related #419 (
--without-intl)sam-github commentedon Mar 8, 2017
Related nodejs/node#11747
gibfahn commentedon Mar 8, 2017
@jbergstroem @mhdawson could one of you clone one of the build jobs (node-test-commit-linux I guess) so I can try to get this set up?
mhdawson commentedon Mar 8, 2017
I assume you will cut it down to run on a single linux variant only, as we don't want to double the load and the one would probably be enough to catch compile problems.
Cloned one for you to work on here: https://ci.nodejs.org/view/All/job/node-test-commit-linux-gibson/
rvagg commentedon Apr 23, 2018
This came up the other day but I don't recall where. @mhdawson & @danbev, where was that, IRC?
Anyway, I tried one out, the config is all done and waiting to be enabled (by ticking the label "" to the list of node labels in the job) but it has a snag, perhaps the first of many? https://ci.nodejs.org/job/node-test-commit-linux-containered/3916/nodes=ubuntu1604_sharedlibs_withoutssl_x64/console (you'll have to scroll up a bit, above the addons)
make test-ciinvolves npm which tries to do an update check but fails because that loads cryptoBefore it can be turned on, we're going to need to skip this step for this type of build.
maclover7 commentedon May 24, 2018
This is a thing now: https://ci.nodejs.org/job/node-test-commit-linux-containered/nodes=ubuntu1604_sharedlibs_withoutintl_x64/
danbev commentedon May 25, 2018
@maclover7 Awesome, thanks for that!