Add ECA.#1
Merged
Merged
Conversation
- Abandon nix-update approach (cannot handle multi-platform packages) - Port update scripts from old-update-method branch - Convert all packages to use lib.importJSON instead of builtins.fromJSON - Create shared ci/lib/github-release-update.sh for common update logic - Add multi-platform support for eca-bin (4 platforms with different hashes) - Switch eca-bin from fetchzip to fetchurl to use published .sha256 files - Update ci/update.sh to run all package update scripts - Reduce update script size by 46% (24 lines -> 13 lines for simple packages) All packages now have: - sources.json with version, url, and hash - update.sh script that generates sources.json - default.nix that reads from sources.json Benefits: - Works from any platform (no cross-evaluation issues) - Multi-platform packages supported (eca-bin has 4 different hashes) - Flake-friendly (uses pure lib.importJSON) - Consistent and maintainable - Easy to add new packages
- Change from macos-latest to ubuntu-latest (update script downloads all platforms) - Simplify from multi-job multi-platform workflow to single job - Fix commit to add sources.json instead of default.nix - Remove unused commitAndForcePush and commitAndPush functions - CI workflow unchanged (still runs flake check on both Linux and macOS)
- Document sources.json pattern - Add instructions for adding new packages - Add instructions for updating packages - Fix typo: github actions -> GitHub Actions
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.