Skip to content

feat: lake: use absolute paths#7822

Merged
tydeu merged 3 commits intoleanprover:masterfrom
tydeu:lake/abs-paths
Apr 5, 2025
Merged

feat: lake: use absolute paths#7822
tydeu merged 3 commits intoleanprover:masterfrom
tydeu:lake/abs-paths

Conversation

@tydeu
Copy link
Copy Markdown
Member

@tydeu tydeu commented Apr 4, 2025

This PR changes Lake to use normalized absolute paths for its various files and directories.

This is done by storing absolute paths for the workspace directory, package directories, and configuration files. These are then joined to relative paths (e.g., for source directories) using a custom join function that eliminates . paths.

Closes #7498. Closes #4042.

@tydeu tydeu added the changelog-lake Lake label Apr 4, 2025
@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 Apr 4, 2025
ghost pushed a commit to leanprover-community/batteries that referenced this pull request Apr 4, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 4, 2025
@tydeu tydeu marked this pull request as ready for review April 5, 2025 00:03
ghost pushed a commit to leanprover-community/batteries that referenced this pull request Apr 5, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 5, 2025
@ghost
Copy link
Copy Markdown

ghost commented Apr 5, 2025

Mathlib CI status (docs):

@ghost ghost added the builds-mathlib CI has verified that Mathlib builds against this PR label Apr 5, 2025
@tydeu tydeu added the release-ci Enable all CI checks for a PR, like is done for releases label Apr 5, 2025
ghost pushed a commit to leanprover-community/batteries that referenced this pull request Apr 5, 2025
ghost pushed a commit to leanprover-community/mathlib4 that referenced this pull request Apr 5, 2025
@tydeu tydeu added this pull request to the merge queue Apr 5, 2025
Merged via the queue into leanprover:master with commit 34385b8 Apr 5, 2025
25 checks passed
@tydeu tydeu deleted the lake/abs-paths branch April 5, 2025 14:45
@tydeu tydeu mentioned this pull request Apr 6, 2025
3 tasks
digama0 added a commit to leanprover-community/mathlib4 that referenced this pull request Apr 7, 2025
kmill added a commit to leanprover-community/mathlib4 that referenced this pull request Apr 19, 2025
* fixes

* fix test

* fix

* chore: adaptations for leanprover/lean4#7797

* stricter rfl check

* fix merge

* Trigger CI for leanprover/lean4#7797

* Trigger CI for leanprover/lean4#7797

* Trigger CI for leanprover/lean4#7797

* inverse construction

* Update lean-toolchain for testing leanprover/lean4#7791

* lake update

* Fix

* Fix

* Fix

* chore: bump to nightly-2025-04-03

* more fixes

* merge lean-pr-testing-6325

* bump proofwidgets

* Update lean-toolchain for testing leanprover/lean4#7812

* shake

* fix shake

* fix tests

* import mathlib.init

* Update lean-toolchain for testing leanprover/lean4#7816

* Trigger CI for leanprover/lean4#7816

* Update lean-toolchain for testing leanprover/lean4#7802

* Fix

* fixes

* Fix (pending upstream changes)

* Trigger CI for leanprover/lean4#7802

* fix

* chore: bump to nightly-2025-04-04

* Fix

* Fix

* Fix

* Fix

* Fix

* Fix

* Update lean-toolchain for testing leanprover/lean4#7818

* Fix

* Fix

* Fix

* Merge master into nightly-testing

* Fix

* Fix comments

* fixes

* Trigger CI for leanprover/lean4#7816

* fix

* fix

* missing doc-string?

* chore: bump to nightly-2025-04-05

* chore: bump to nightly-2025-04-06

* move Batteries to nightly-testing

* lake update

* lint

* fix

* chore: bump to nightly-2025-04-07

* sprinkle noncomputable

* fix: relativize file paths

fix for leanprover/lean4#7822

* chore: reset cache

* chore: bump to nightly-2025-04-07

* shake

* fix: relativize more

* Update lean-toolchain for testing leanprover/lean4#7856

* Update lean-toolchain for testing leanprover/lean4#7851

* Fix

* Fix

* Fix

* Fix

* chore: bump to nightly-2025-04-08

* Update lean-toolchain for testing leanprover/lean4#7870

* Trigger CI for leanprover/lean4#7870

* Trigger CI for leanprover/lean4#7870

* Trigger CI for leanprover/lean4#7870

* wip

* Trigger CI for leanprover/lean4#7870

* .

* chore: bump to nightly-2025-04-09

* Trigger CI for leanprover/lean4#7870

* .

* bump

* lake update

* fixes for leanprover/lean4#7873

* fixes

* deprecations

* chore: bump to nightly-2025-04-10

* chore: `Option.zipWith` -> `Option.merge`

* lake update

* toolchain

* fix

* fix

* Update lean-toolchain for testing leanprover/lean4#7897

* Trigger CI for leanprover/lean4#7897

* fix

* lake update and fix

* Fix

* Fix

* Trigger CI for leanprover/lean4#7897

* fix merge

* chore: adaptations for nightly-2025-04-10

* chore: bump to nightly-2025-04-14

* fix

* .

* annotate changed goal state

* fixes (adaptation notes)

* cleanup imports

* cleanup

* Update lean-toolchain for testing leanprover/lean4#7933

* Fix

* Fix

* Update lean-toolchain for testing leanprover/lean4#7975

* remove adaptation notes

* chore: bump to nightly-2025-04-16

---------

Co-authored-by: leanprover-community-mathlib4-bot <leanprover-community-mathlib4-bot@users.noreply.github.com>
Co-authored-by: Kim Morrison <kim@tqft.net>
Co-authored-by: Markus Himmel <markus@lean-fro.org>
Co-authored-by: Rob23oba <robin.arnez@web.de>
Co-authored-by: Mario Carneiro <di.gama@gmail.com>
Co-authored-by: Johan Commelin <johan@commelin.net>
Co-authored-by: github-actions <github-actions@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR changelog-lake Lake release-ci Enable all CI checks for a PR, like is done for releases 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.

lake: __dir__ contains relative path, that is cached and becomes wrong when using -d=.. flag lake puts relative paths in LD_LIBRARY_PATH

1 participant