Skip to content

Add flake#13

Open
AshleyYakeley wants to merge 5 commits intoobsidiansystems:nixpkgs-24.05from
AshleyYakeley:flake
Open

Add flake#13
AshleyYakeley wants to merge 5 commits intoobsidiansystems:nixpkgs-24.05from
AshleyYakeley:flake

Conversation

@AshleyYakeley
Copy link
Copy Markdown

Flake with Home Manager module (and packages). The SDK version is configurable in the HM module.

@AshleyYakeley
Copy link
Copy Markdown
Author

This is what I'm using for myself; I don't have a clear idea yet as to what else would be helpful.

defaultStuff = getStuff { };
in
{
packages."x86_64-linux" = {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

DAML SDK does support mac, can you add this?

Something like:

outputs = inputs: let
  lib = inputs.nixpkgs.lib;
  supportedSystems = lib.genAttrs [ "x86_64-linux" "x86_64-darwin" ];
in {
  packages = supportedSystems (system: let
    pkgs = inputs.nixpkgs.legacyPackages."${system}";
  in ... 
  )
}

Same thing for devShells as well

};
};

homeModule =
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I'm hesitant to include the home-manager module, as generally it'd be better to make this be "per-project configuration", instead of "per-system configuration"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

this can be merged with the home-manager module included in any case

};
};

canton-free = import ./canton.nix {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

can you split canton into a singular attribute under packages? along with the vscode extension and the sdk proper

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