Skip to content

Conversation

@Ericson2314
Copy link
Contributor

No description provided.

@inclyc
Copy link
Member

inclyc commented Apr 9, 2025

These files are created/edited but not formatted:
libnixt/include/nixt/Deserialize.h
libnixt/include/nixt/Visitor.h
nixd/lib/Eval/AttrSetProvider.cpp

Copy link
Member

@inclyc inclyc left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Waiting for a nix's stable release? Also pay attention to those formatting nits 💯 .

nlohmann_json
];

env.CXXFLAGS = "-include ${nix.dev}/include/nix/config.h";
Copy link
Member

Choose a reason for hiding this comment

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

Confirm that this is not needed anymore?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, config file cleanup is the big milestone in 2.28!

Hydra has not yet caught up with Nix 2.28.

Flake lock file updates:

• Updated input 'nixpkgs':
    'github:NixOS/nixpkgs/4989a246d7a390a859852baddb1013f825435cee?narHash=sha256-kMBQ5PRiFLagltK0sH%2B08aiNt3zGERC2297iB6vrvlU%3D' (2024-12-17)
  → 'github:NixOS/nixpkgs/ef3a956f697525883b77192cbe208233ea0f8f79?narHash=sha256-oxI9TLgnQbQ/WL0tIwVSIooLbXq4PW1QUhf5aQmXFgk%3D' (2025-04-09)
#include "PtrPool.h"

#include <nix/expr/nixexpr.hh>
#include <nix/input-accessor.hh>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

input-accessor.hh doesn't exist anymore, yet I built everything. Is this file dead code?

Copy link
Member

Choose a reason for hiding this comment

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

Yes. Related issue: #574

@Ericson2314
Copy link
Contributor Author

Formatted now

@inclyc inclyc linked an issue Apr 12, 2025 that may be closed by this pull request
@inclyc inclyc merged commit 3aa27fd into nix-community:main Apr 12, 2025
19 checks passed
@Ericson2314 Ericson2314 deleted the nix-2.28 branch April 12, 2025 04:46
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.

evaluation failure when using newest unstable nixpkgs version

2 participants