Build nix-store with Meson, adjust Perl bindings deps to integrate with other meson builds#10908
Build nix-store with Meson, adjust Perl bindings deps to integrate with other meson builds#10908Ericson2314 merged 2 commits intomasterfrom
nix-store with Meson, adjust Perl bindings deps to integrate with other meson builds#10908Conversation
Ericson2314
changed the title
nix-store1 with Meson, adjust Perl bindings depsnix-store with Meson, adjust Perl bindings deps to integrate with other meson builds
Ericson2314
force-pushed
the
meson-libstore
branch
from
9093c42 to
1bcc5db
Compare
|
@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. |
|
I'm looking and I'm seeing a CI error, but it doesn't have anything to do with generating SQL.
|
|
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. |
|
@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. |
This is more robust, and match's Nixpkgs policy to force enable flags statically by default (a common distro thing).
Special thanks to everyone that has worked on a Meson port so far, @p01arst0rm and @Qyriad in particular. Co-Authored-By: p01arst0rm <[email protected]> Co-Authored-By: Artemis Tosini <[email protected]> Co-Authored-By: Artemis Tosini <[email protected]> Co-Authored-By: Felix Uhl <[email protected]> Co-Authored-By: Jade Lovelace <[email protected]> Co-Authored-By: Lunaphied <[email protected]> Co-Authored-By: Maximilian Bosch <[email protected]> Co-Authored-By: Pierre Bourdon <[email protected]> Co-Authored-By: Qyriad <[email protected]> Co-Authored-By: Rebecca Turner <[email protected]> Co-Authored-By: Winter <[email protected]> Co-Authored-By: eldritch horrors <[email protected]> Co-Authored-By: jade <[email protected]> Co-Authored-By: julia <[email protected]> Co-Authored-By: rebecca “wiggles” turner <[email protected]> Co-Authored-By: wiggles dog <[email protected]> Co-Authored-By: fricklerhandwerk <[email protected]> Co-authored-by: Eli Schwartz <[email protected]>
Ericson2314
force-pushed
the
meson-libstore
branch
from
1bcc5db to
81004a0
Compare
|
(#10855 was approved with this a part of it. I pulled it out just for purposes of debugging CI failures.) |
|
(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...) |
| # TODO rename, because it will conflict with downstream projects | ||
| configdata.set_quoted('PACKAGE_VERSION', meson.project_version()) | ||
|
|
||
| configdata.set_quoted('SYSTEM', host_machine.system()) |
There was a problem hiding this comment.
This is probably wrong. host_machine.system() is expected to have e.g. 'linux' in other places.
I don't know how to get the right value, reliably, and we've defined it to be based on autoconf, which I don't think we want to continue doing, or do we? Alternatively, we could leave it to the packager, and pass stdenv.hostPlatform.system in our own packaging?
3 participants
Motivation
More progress on #2503
Context
Following up to #10855
Priorities and Process
Add 👍 to pull requests you find important.
The Nix maintainer team uses a GitHub project board to schedule and track reviews.