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

Add flake.nix for Nix users #47

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Add flake.nix for Nix users #47

wants to merge 1 commit into from

Conversation

alurm
Copy link
Contributor

@alurm alurm commented Jan 16, 2025

I removed server dependency on server/conf/server.pem, since that made Nix build.

To generate keys, make server/conf/server.pem still works.

@alurm
Copy link
Contributor Author

alurm commented Jan 16, 2025

I also added build: [[make "server"] ...] to cuekeeper.opam, I think it was required too.

To be clear, I don't imply that this should be merged (at least in the current form), but I thought that I should mention these changes here since they seem to be working and could be useful for someone.

@alurm
Copy link
Contributor Author

alurm commented Jan 16, 2025

Also, unrelated, I thought that it'd be good to have a button on the client (the one which doesn't require a server) to sync to an upstream but I don't really know how to implement it.

It would allow to both sync and have notes be local-first.

Maybe I'll send this as a separate pull request, but for now I don't even know where to start, so maybe I'll give up.

I think I found some piece of code for that in ck_model, maybe I'll try changing it.

CORS could be a problem, I suppose. I'll have to think about it.

…e Nix work)

To generate keys, run `nix develop` (optional) followed by `make server/conf/server.pem`.
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