Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make dysnomia releasable on nixpkgs_unstable #11

Draft
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

ibizaman
Copy link

@ibizaman ibizaman commented Apr 14, 2023

This PR is my ongoing effort to make dysnomia compatible with nixpkgs_unstable.

I can run nix-build release.nix -A release --arg systems '["i686-linux" "x86_64-linux"]' with the two commented out tests. I can't build for the other systems.

The most important change was that now builtins.currentSystem does not
exist anymore. This means we need to have one system and import
nixpkgs with it and release does not contain all systems anymore. I
have no idea about the implications here, I chose the easiest way IMO
to get the ball rolling, but I'm happy to go back to the drawing board
if needed.
The name attribute is required now.
Importing `nixpkgs` without specifying `system` does not work with
flakes as `currentSystem` attribute does not exist. So instead, we
need to give nixpkgs and a system. I chose to actually give pkgs
directly. Same for python-testing, I chose to give it instead of
importing in the test files.
@ibizaman ibizaman marked this pull request as draft April 14, 2023 07:30
@ibizaman ibizaman changed the title Make dysnomia a flake and update to nixpkgs_unstable Make dysnomia releasable on nixpkgs_unstable Apr 15, 2023
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.

1 participant