Building nightly node 12 on alpine is broken since 2018-12-06 (alpine linux) #25095
Comments
|
I think
Line 984 in a6f69eb cc @nodejs/build-files @nodejs/v8-update |
richardlau
added
build
|
Should I be taking extra steps to prepare the source to build, then? maybe removing the requirement to build Or maybe the nightly source file is not being built properly? |
|
Since it's in Line 27 in a6f69eb |
|
@lsmoura It's a gap in our process -- The nightly builds are built from a git checkout, not the source tarball. I'm not aware that we have any builds that build from the source tarball -- cc @nodejs/build ? |
|
well, 50% of my motivation to make these builds are to help the nodejs team with something. Even if it is "I've found these errors and I don't know if you're aware of them". Mission accomplished, somehow (= I'll automate this process soon enough and make logfiles available... Thanks for the tips. I'll give it a try. |
|
I'm preparing a PR to fix this |
|
This fixed temporarily on my build script, after unpacking the file: |
- Less files to maintain. - We don't use them. - We build with GN to run V8 tests. Fixes: nodejs#25095 Refs: nodejs#25010
|
By the end of the build process, I got this one other missing piece: |
|
So, what might a test worker for src tarballs look like? We can probably only afford to do it on a single platform, and that'll likely be one of the Ubuntu 16.04 or 18.04 Docker containers we have in the mix for these kinds of miscellaneous things. make tar
tar zxvf <tarball> -C ${NODE_TEST_DIR}
cd ${NODE_TEST_DIR}/<expected path>
./configure <with normal test options>
make <with normal test options>
.... ?Would we copy in the tests and run a full test suite on the compiled build? Is that necessary or just confirming that it compiles is enough? |
We shouldn't need to copy in any tests (Node.js tests are included in the source tarball). The things that don't get into the source tarball are tests for deps (which we don't run on the regular CI anyway) and things from tools which I think prevents linting. |
- Less files to maintain. - We don't use them. - We build with GN to run V8 tests. Fixes: nodejs#25095 Refs: nodejs#25010 PR-URL: nodejs#25097 Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Daniel Bevenius <daniel.bevenius@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Daijiro Wachi <daijiro.wachi@gmail.com> Reviewed-By: Ujjwal Sharma <usharma1998@gmail.com>
|
I've just encountered this with the v12.11.0 tarball. Almost identical error: Regression maybe? |
I've been building node12 nightly to create alpine linux docker images since December 1st using the source packages available at the nodejs website.
Ever since release
nightly20181207cbf0e5a1f4, the build fails with the same error:I'm building the code using the base alpine linux docker image for version 3.8.1, with no custom packages (only official ones required to build the image).
Here's how I'm building from the source:
Every nightly on december before that is build properly.
You can check lsmoura/node:12nightly if you want to see other nightly builds working.
Thanks for all the work from the nodejs developers!
The text was updated successfully, but these errors were encountered: