Skip to content

Update shell.nix dependencies #1488

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

Merged
merged 1 commit into from
May 15, 2025
Merged

Conversation

chimnayajith
Copy link
Contributor

This PR updates the dependencies in the Nix environment file to fix build issues.

Changes made:

  • Updated epoxy package name to libepoxy
  • Upgraded jdk11 to jdk17 as required by Android Gradle
  • Added nodejs dependency (required for the icon font generation script)

Environment details

Tested and working on NixOS 24.11

- Renamed epoxy to libepoxy to align with current Nixpkgs.
- Upgraded jdk11 to jdk17 due to Android Gradle's requirements.
- Added nodejs for compatibility with the icon tool using npm.

Tested on NixOS version 24.11
@gnprice
Copy link
Member

gnprice commented May 15, 2025

Thanks for the update! WFM too; and I think working for two of us is as much validation as we need for this file. Merging.

I've edited the commit message's summary line:

-    Update shell.nix with new package versions and dependencies
+    nixos: Update to use libepoxy, jdk17, nodejs

to (a) have a prefix following our style (namely "nixos:"), and (b) include more specifics about what the change is doing.

@gnprice gnprice force-pushed the update-shell-nix branch from 3f3bb6c to 710ebd9 Compare May 15, 2025 00:54
@gnprice gnprice merged commit 710ebd9 into zulip:main May 15, 2025
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