NixOS / ofborg

@ofborg tooling automation https://monitoring.ofborg.org/dashboard/db/ofborg
https://ofborg.org
MIT License
232 stars 166 forks source link

Update nixpks used in flakes to a current version #670

Open SuperSandro2000 opened 8 months ago

SuperSandro2000 commented 8 months ago

Ofborg currently uses the one year old 22.11 and 22.05 release branches. It should be updated to 23.11.

cole-h commented 8 months ago

Just a note that ofborg does not use the flake in this repo for evaluating / etc nixpkgs. It uses the checkout of the PR it's operating on (and/or the master nixpkgs at the time of operation). The flake only affects building the ofborg software, nothing else.

zeuner commented 8 months ago

The flake only affects building the ofborg software, nothing else.

No doubt on that (apart from maybe also running ofborg, interfacing to GitHub etc.), but since 22.05 doesn't build anymore (see e.g. https://github.com/NixOS/nixpkgs/pull/277064, https://github.com/NixOS/nixpkgs/pull/277158) and can't be updated (see https://github.com/NixOS/nixpkgs/pull/277064#issuecomment-1870605406), this blocks building ofborg. If there is no need for up-to-date packages for building, having an LTS release (see https://discourse.nixos.org/t/interest-in-a-nixos-lts/4705) might reduce ofborg maintenance costs, but with the current ecosystem, updating seems to be the most straight-forward solution.

zeuner commented 8 months ago

This (https://github.com/NixOS/ofborg/pull/648) PR might be a step towards resolving the issue for the Rust part.

SuperSandro2000 commented 8 months ago

Can we at least merge that PR? It is a really bad sign that our CI is in such a bad state.