NixOS / nix

Nix, the purely functional package manager
https://nixos.org/
GNU Lesser General Public License v2.1
11.47k stars 1.44k forks source link

Build `nix-store` with Meson, adjust Perl bindings deps to integrate with other meson builds #10908

Closed Ericson2314 closed 2 weeks ago

Ericson2314 commented 2 weeks ago

Motivation

More progress on #2503

Context

Following up to #10855

Priorities and Process

Add :+1: to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

Ericson2314 commented 2 weeks ago

@eli-schwartz Any idea why it might be hanging in the generate SQL steps on Linux only? It didn't happen for me doing similar cross builds locally.

eli-schwartz commented 2 weeks ago

I'm looking and I'm seeing a CI error, but it doesn't have anything to do with generating SQL.

The hosted runner: GitHub Actions 5 lost communication with the server. Anything in your workflow that terminates the runner process, starves it for CPU/Memory, or blocks its network access can cause this error.

eli-schwartz commented 2 weeks ago

It says it failed in 47 minutes but the job stages with logs are only a couple seconds long total. This is unquestionably a GitHub-level bug in GitHub's own infrastructure, because something broke on their end mid-run.

Restarting probably helps.

Ericson2314 commented 2 weeks ago

@eli-schwartz Sorry. Yes there is a bug that makes the logs go away. Very frustrating. But I did restart it with the logs kept open after a few times of this to see that it was consistently stalling when building the SQL-processing steps.

I don't think there is much interesting in the logs but I'll leave the tab open after restarting to then paste it here.

Ericson2314 commented 2 weeks ago

(https://github.com/NixOS/nix/pull/10855 was approved with this a part of it. I pulled it out just for purposes of debugging CI failures.)

Ericson2314 commented 2 weeks ago

(I went to restart a timed out macOS job, and then that briefly caused all checks to pass, which resulted in an auto merge. Yay GitHub Action...)