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

Update nixpkgs, services-flake and crane #141

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

shivaraj-bh
Copy link
Member

@shivaraj-bh shivaraj-bh commented Nov 19, 2024

  • nixpkgs update is for redis: 7.0 -> 7.2
  • crane had to be updated to avoid error: output file /build/hw9n9iy0pniajp7hz9glfhjcxcvh9fvf-source/target/release/deps/libopenssl_sys-4a50bfca2de70625.rmeta is not writeable -- check its permissions, which was a result of the nixpkgs update.
  • services-flake updated to import the devShell output (see). Aside from the update, the PR also fixes logic to make apache-kafka service more reliable.

@shivaraj-bh
Copy link
Member Author

shivaraj-bh commented Nov 19, 2024

@khuzema786 can you test if the services in this PR integrate well with the application?

Copy link
Member

@srid srid left a comment

Choose a reason for hiding this comment

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

Can merge, if we can fix this hardcode stuff.

At some point in future, this repo should switch over to using rust-flake instead of using crane directly.

nix/services.nix Outdated Show resolved Hide resolved
nixpkgs update is to get latest redis and crane had to be updated
because otherwise we get:
```sh
> error: output file
/build/hw9n9iy0pniajp7hz9glfhjcxcvh9fvf-source/target/release/deps/libopenssl_sys-4a50bfca2de70625.rmeta
is not writeable -- check its permissions       >       > error: could
not compile `openssl-sys` (lib) due to previous error
```
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