Skip to content

Conversation

@emillon
Copy link
Collaborator

@emillon emillon commented Mar 14, 2025

Repro for #11533

@emillon emillon added shared-cache Shared artefacts cache package management test Dune handling test labels Mar 14, 2025
Copy link
Collaborator

@Leonidas-from-XIV Leonidas-from-XIV left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I see. So the d that other expects in the second build is different, thus in the second build if will rebuild other and record it with the permissions that the restored d had and in the third build it will be able to restore other from the cache because it restores it with the right flags.

Sneaky! Good repro.

Signed-off-by: Etienne Millon <me@emillon.org>
@Leonidas-from-XIV Leonidas-from-XIV merged commit 0a80aa3 into ocaml:main Mar 18, 2025
24 of 25 checks passed
wmuth pushed a commit to wmuth/dune that referenced this pull request Mar 24, 2025
Signed-off-by: Etienne Millon <me@emillon.org>
anmonteiro pushed a commit to anmonteiro/dune that referenced this pull request Apr 22, 2025
Signed-off-by: Etienne Millon <me@emillon.org>
Sudha247 pushed a commit to Sudha247/dune that referenced this pull request Jul 23, 2025
Signed-off-by: Etienne Millon <me@emillon.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

package management shared-cache Shared artefacts cache test Dune handling test

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants