Skip to content

Conversation

@adrian-gierakowski
Copy link
Contributor

@adrian-gierakowski adrian-gierakowski commented Jan 6, 2026

update nixpkgs, nix version used and adapts nixPlugin to upstream changes

also updates cachix actions as the install action was causing following error (same as in #1282)

error:
       … while updating the lock file of flake 'git+file:///home/runner/work/noogle/noogle?ref=f7962110f6d4cc989936778a221c249c0521a52d&rev=f7962110f6d4cc989936778a221c249c0521a52d&shallow=1'

       error: unsupported tarball input attribute 'lastModified'

@adrian-gierakowski adrian-gierakowski changed the title feat: add CLI implementation plan chore: update nixpkgs and cachix actions Jan 6, 2026
google-labs-jules bot and others added 4 commits January 6, 2026 05:58
Update the C++ plugin code to use the new accessor methods for `nix::Value`
instead of accessing the private `payload` member directly. This resolves
build errors with Nix 2.31+.

Changes:
- Replace `value.payload.lambda` with `value.lambda()`
- Replace `value.payload.attrs` with `value.attrs()`
- Replace `value.payload.primOp` with `value.primOp()`
- Replace `value.payload.primOpApp` with `value.primOpApp()`
- Update dependent member accesses accordingly.
…-nix-2.31-13819925646931285426

Fix noogle-plugin build with Nix 2.31
`onig_sys` was failing to build with recent compiler updates due to strict C checks.
This change switches to using the system `oniguruma` library provided by `nixpkgs` instead of the bundled one, which resolves the build issue.
We also add `pkg-config` to native build inputs to ensure the library is found.
…830852308775

Fix pesto build by using system oniguruma
Copy link
Collaborator

@hsjobeki hsjobeki left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice job thanks for the fixes. 👍

@hsjobeki hsjobeki merged commit dc57ebe into nix-community:main Jan 11, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants