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

refactor: change postUpdate? config to a decl #2723

Merged
merged 1 commit into from
Oct 21, 2023

Conversation

tydeu
Copy link
Member

@tydeu tydeu commented Oct 20, 2023

A follow-up to #2603.

The primary motivation is to restore PackageConfig ABI compatibility with rc1 to avoid some nasty lake clean errors with incompatabile oleans while we try to diagnose why old oleans are used.

However, it also helps to simplify the code for post update hooks on the user end as Lake can now magically provide the package object to user code.

@tydeu tydeu marked this pull request as ready for review October 20, 2023 22:11
@tydeu tydeu force-pushed the lake-post-update-touchup branch from 917d125 to 2ec2dac Compare October 20, 2023 23:34
@github-actions github-actions bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label Oct 21, 2023
leanprover-community-mathlib4-bot added a commit to leanprover-community/mathlib4 that referenced this pull request Oct 21, 2023
@tydeu tydeu merged commit 6c20673 into leanprover:master Oct 21, 2023
@tydeu tydeu deleted the lake-post-update-touchup branch October 21, 2023 01:38
@github-actions
Copy link
Contributor

The backport to releases/v4.2.0 failed:

The process '/usr/bin/git' failed with exit code 1

To backport manually, run these commands in your terminal:

# Fetch latest updates from GitHub
git fetch
# Create a new working tree
git worktree add .worktrees/backport-releases/v4.2.0 releases/v4.2.0
# Navigate to the new working tree
cd .worktrees/backport-releases/v4.2.0
# Create a new branch
git switch --create backport-2723-to-releases/v4.2.0
# Cherry-pick the merged commit of this pull request and resolve the conflicts
git cherry-pick -x --mainline 1 6c206737370bc41139bb60100d0e69437d39c84d
# Push it to GitHub
git push --set-upstream origin backport-2723-to-releases/v4.2.0
# Go back to the original working tree
cd ../..
# Delete the working tree
git worktree remove .worktrees/backport-releases/v4.2.0

Then, create a pull request where the base branch is releases/v4.2.0 and the compare/head branch is backport-2723-to-releases/v4.2.0.

@leanprover-community-mathlib4-bot leanprover-community-mathlib4-bot added the builds-mathlib CI has verified that Mathlib builds against this PR label Oct 21, 2023
@leanprover-community-mathlib4-bot
Copy link
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport releases/v4.2.0 builds-mathlib CI has verified that Mathlib builds against this PR toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants