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

Provide homeManagerModule as a flake output #662

Merged
merged 2 commits into from
Oct 1, 2021
Merged

Conversation

ilkecan
Copy link
Contributor

@ilkecan ilkecan commented Sep 30, 2021

Main change is the added home-manager module, which can be used like

    neuron = {
      enable = true;
      notesDirectory = "/home/${name}/zettelkasten";
      prettyUrls = true;
    };

See divnix/digga#106 (comment) for pointers about how to add a home-manager module from a flake to imports.

I also removed the unnecessary app definitions, since by default nix run will try the packages with the same name. App definitions are mainly useful if there are multiple binaries or you want to name them differently.

@ilkecan
Copy link
Contributor Author

ilkecan commented Sep 30, 2021

Also nix flake show currently produces an error like

error: a 'aarch64-linux' with features {} is required to build '/nix/store/0a5kf39y7vpnbgcnis3gp3spbdpndvij-cabal2nix-neuron.drv', but I am a 'x86_64-linux' with features {benchmark, big-parallel, kvm, nixos-test}

But I couldn't fix this easily. In general, I think the nix code is unnecessarily complex. You said nix-thunk should be removed in favour of flake inputs in the previous PR. Does that mean https://github.com/srid/neuron/blob/master/nixpkgs.nix should be removed? I can try to work on these in a separate PR.

@ilkecan ilkecan marked this pull request as ready for review September 30, 2021 23:37
@ilkecan ilkecan changed the title Improve flake Provide homeManagerModule as a flake output Oct 1, 2021
@srid
Copy link
Owner

srid commented Oct 1, 2021

You said we nix-thunk should be removed in favour of flake inputs in the previous PR. Does that mean https://github.com/srid/neuron/blob/master/nixpkgs.nix should be removed? I can try to work on these in a separate PR.

Yes, that would be awesome. You can use https://github.com/srid/emanote for inspiration (by the way, emanote is the successor project to neuron).

flake.nix Show resolved Hide resolved
@srid srid merged commit 23d0f0e into srid:master Oct 1, 2021
@ilkecan ilkecan deleted the improve-flake branch October 1, 2021 00:42
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