Skip to content

Update nixpkgs and CMake simplifcations#954

Merged
stv0g merged 11 commits into
masterfrom
update-nixpkgs
Sep 9, 2025
Merged

Update nixpkgs and CMake simplifcations#954
stv0g merged 11 commits into
masterfrom
update-nixpkgs

Conversation

@stv0g
Copy link
Copy Markdown
Contributor

@stv0g stv0g commented Sep 1, 2025

No description provided.

@stv0g stv0g requested a review from n-eiling as a code owner September 1, 2025 11:01
@stv0g stv0g changed the title Update nixpkgs Update nixpkgs and CMake simplifcations Sep 1, 2025
@stv0g stv0g force-pushed the update-nixpkgs branch 3 times, most recently from 89b16ed to 421fa1d Compare September 5, 2025 08:51
@stv0g stv0g requested a review from pjungkamp September 5, 2025 08:52
@stv0g
Copy link
Copy Markdown
Contributor Author

stv0g commented Sep 5, 2025

@pjungkamp I would be interested in your review on this PR :)

@pjungkamp
Copy link
Copy Markdown
Contributor

I like the CMake cleanup. Neither villas-common nor villas-fpga are considered as a standalone projects anymore which allows us to strip even more unnecessary boilerplate from the build system.

The only nitpicks I've got is that I'd like to see some formatting improvements on the json_pack calls when you're already touching that code. You took the time to decipher the format string so make it easier for the next person that has to touch that.

Comment thread lib/api/requests/status.cpp
Comment thread packaging/nix/reverse-struct-group.patch
Comment thread README.md Outdated
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
steffen-vogel-opalrt and others added 2 commits September 8, 2025 12:03
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Philipp Jungkamp <philipp.jungkamp@rwth-aachen.de>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
n-eiling
n-eiling previously approved these changes Sep 8, 2025
Signed-off-by: Steffen Vogel <steffen.vogel@opal-rt.com>
@stv0g stv0g merged commit 836c7ea into master Sep 9, 2025
3 checks passed
@stv0g stv0g deleted the update-nixpkgs branch September 9, 2025 12:52
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.

4 participants