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

Link with libicuuc, upgrade deps, add CI caching #17

Merged
merged 5 commits into from
Oct 8, 2024
Merged

Link with libicuuc, upgrade deps, add CI caching #17

merged 5 commits into from
Oct 8, 2024

Conversation

anttih
Copy link
Contributor

@anttih anttih commented Oct 5, 2024

  • Links scheme with libicuuc so that latest purescm works
  • Adds CI caching. This shaves off 5 minutes from the build.

@anttih
Copy link
Contributor Author

anttih commented Oct 5, 2024

Still in CI we get:

Reading Spago workspace configuration...
⚠️ Your project contains a spago.lock file, but it cannot be decoded. Spago will generate a new one.
Error was: $.workspace.packages.build_plan.arrays: Could not decode Lockfile:
  Could not decode WorkspaceLock:
    Could not decode Dependencies:
      No value found

@anttih anttih changed the title Link with libicuuc, upgrade deps Link with libicuuc, upgrade deps, add CI caching Oct 5, 2024
@anttih
Copy link
Contributor Author

anttih commented Oct 5, 2024

Ah, I have an older spago locally so the spago.lock files don't match.

@anttih
Copy link
Contributor Author

anttih commented Oct 7, 2024

After updating the lock file with latest spago we get:

Reading Spago workspace configuration...
⚠️ Your project contains a spago.lock file, but it cannot be decoded. Spago will generate a new one.
Error was: $.workspace.packages.build_plan.arrays: Could not decode Lockfile:
  Could not decode WorkspaceLock:
    Could not decode Dependencies:
      No value found

@f-f
Copy link
Contributor

f-f commented Oct 8, 2024

After updating the lock file with latest spago we get

Fixed - we were not using the latest spago, I guess we should fix it in the nix shell

@f-f f-f merged commit f970d66 into master Oct 8, 2024
1 check passed
@f-f f-f deleted the fix-build branch October 8, 2024 18:31
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