Skip to content

build: show cc outputs when version detection failed#61700

Closed
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:configure-cc-vers
Closed

build: show cc outputs when version detection failed#61700
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:configure-cc-vers

Conversation

@legendecas
Copy link
Member

@legendecas legendecas commented Feb 5, 2026 •
edited
Loading

There are chances that ./configure fails on GitHub Actions.
Print the output of the command to see if there is any way to
improve it.

Example: https://github.com/nodejs/node/actions/runs/21730375777/job/62683227640?pr=61699

make: Entering directory '/home/runner/work/node/node/node'
python3 ./configure --verbose --error-on-warn --v8-enable-temporal-support
sccache: error: Timed out waiting for server startup. Maybe the remote service is unreachable?
Run with SCCACHE_LOG=debug SCCACHE_NO_DAEMON=1 to get more information
Node.js configure: Found Python 3.14.2...
Traceback (most recent call last):
  File "/home/runner/work/node/node/node/./configure", line 27, in <module>
    import configure
  File "/home/runner/work/node/node/node/configure.py", line 2587, in <module>
    check_compiler(output)
    ~~~~~~~~~~~~~~^^^^^^^^
  File "/home/runner/work/node/node/node/configure.py", line 1510, in check_compiler
    ok, is_clang, clang_version, gcc_version, is_apple = try_check_compiler(CXX, 'c++')
                                                         ~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
  File "/home/runner/work/node/node/node/configure.py", line 1291, in try_check_compiler
    is_apple = values[7] == '1'
               ~~~~~~^^^
IndexError: list index out of range
make: *** [Makefile:626: build-ci] Error 1
make: Leaving directory '/home/runner/work/node/node/node'

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Feb 5, 2026
@richardlau
Copy link
Member

richardlau commented Feb 5, 2026

There are chances that ./configure fails on GitHub Actions. Print the output of the command to see if there is any way to improve it.

Example: https://github.com/nodejs/node/actions/runs/21730375777/job/62683227640?pr=61699

make: Entering directory '/home/runner/work/node/node/node'
python3 ./configure --verbose --error-on-warn --v8-enable-temporal-support
sccache: error: Timed out waiting for server startup. Maybe the remote service is unreachable?
Run with SCCACHE_LOG=debug SCCACHE_NO_DAEMON=1 to get more information
Node.js configure: Found Python 3.14.2...
Traceback (most recent call last):
  File "/home/runner/work/node/node/node/./configure", line 27, in <module>
    import configure
  File "/home/runner/work/node/node/node/configure.py", line 2587, in <module>
    check_compiler(output)
    ~~~~~~~~~~~~~~^^^^^^^^
  File "/home/runner/work/node/node/node/configure.py", line 1510, in check_compiler
    ok, is_clang, clang_version, gcc_version, is_apple = try_check_compiler(CXX, 'c++')
                                                         ~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
  File "/home/runner/work/node/node/node/configure.py", line 1291, in try_check_compiler
    is_apple = values[7] == '1'
               ~~~~~~^^^
IndexError: list index out of range
make: *** [Makefile:626: build-ci] Error 1
make: Leaving directory '/home/runner/work/node/node/node'

I think the failure is due to the sccache error in that output?

@legendecas
Copy link
Member Author

legendecas commented Feb 6, 2026

I think the failure is due to the sccache error in that output?

That's the reason we need this patch to see what's the output. :D

@legendecas legendecas added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Feb 7, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Feb 7, 2026
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 7, 2026

Commit Queue failed
- Loading data for nodejs/node/pull/61700
✔  Done loading data for nodejs/node/pull/61700
----------------------------------- PR info ------------------------------------
Title      build: show cc outputs when version detection failed (#61700)
   ⚠  Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch     legendecas:configure-cc-vers -> nodejs:main
Labels     build, author ready, needs-ci
Commits    1
 - build: show cc outputs when version detection failed
Committers 1
 - Chengzhong Wu <cwu631@bloomberg.net>
PR-URL: https://github.com/nodejs/node/pull/61700
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/61700
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 05 Feb 2026 22:26:04 GMT
   ✔  Approvals: 3
   ✔  - Juan José Arboleda (@juanarbol): https://github.com/nodejs/node/pull/61700#pullrequestreview-3759818707
   ✔  - Richard Lau (@richardlau) (TSC): https://github.com/nodejs/node/pull/61700#pullrequestreview-3759959688
   ✔  - Colin Ihrig (@cjihrig): https://github.com/nodejs/node/pull/61700#pullrequestreview-3763058885
   ✔  Last GitHub CI successful
   ✘  No Jenkins CI runs detected
--------------------------------------------------------------------------------
   ✔  Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/21788063510

@nodejs-github-bot nodejs-github-bot added the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. label Feb 7, 2026
@richardlau richardlau added request-ci Add this label to start a Jenkins CI on a PR. and removed commit-queue-failed An error occurred while landing this pull request using GitHub Actions. labels Feb 7, 2026
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Feb 7, 2026
@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 7, 2026

CI: https://ci.nodejs.org/job/node-test-pull-request/71235/

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 8, 2026

CI: https://ci.nodejs.org/job/node-test-pull-request/71239/

@nodejs-github-bot
Copy link
Collaborator

nodejs-github-bot commented Feb 8, 2026

CI: https://ci.nodejs.org/job/node-test-pull-request/71258/

@legendecas legendecas added commit-queue Add this label to land a pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Feb 8, 2026
legendecas added a commit that referenced this pull request Feb 8, 2026
@legendecas
build: show cc outputs when version detection failed
286828b 
PR-URL: #61700
Reviewed-By: Juan José Arboleda <soyjuanarbol@gmail.com>
Reviewed-By: Richard Lau <richard.lau@ibm.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@legendecas
Copy link
Member Author

legendecas commented Feb 8, 2026

Landed in 286828b

@legendecas legendecas closed this Feb 8, 2026
@legendecas legendecas deleted the configure-cc-vers branch February 8, 2026 21:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cjihrig cjihrig cjihrig approved these changes

@richardlau richardlau richardlau approved these changes

@juanarbol juanarbol juanarbol approved these changes

Assignees

No one assigned

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@legendecas @richardlau @nodejs-github-bot @cjihrig @juanarbol