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

fix: sync packages if "hermit install" fails to find a package #405

Merged
merged 1 commit into from
May 15, 2024

Conversation

jvmakine
Copy link
Collaborator

@jvmakine jvmakine commented May 15, 2024

This occasionally breaks enabling the hermit IntelliJ plugin that installs dependencies automatically at startup.

We also skipt the test if we had synchronized previously if we are forcing a sync. This is needed as "hermit install" does a non forced sync at the beginning, marking source as synced, and this prevents the later forced sync from running when the package is not found.

Fixes #363

This occasionally breaks enabling the hermit IntelliJ plugin that installs dependencies automatically at startup.

We also skipt the test if we had synchronized previously if we are forcing a sync.
This is needed as "hermit install" does a non forced sync at the beginning, marking source as synced, and this prevents the later forced sync from running when the package is not found.
@jvmakine jvmakine marked this pull request as draft May 15, 2024 04:19
@jvmakine jvmakine marked this pull request as ready for review May 15, 2024 04:26
@jvmakine jvmakine merged commit 991aa47 into master May 15, 2024
6 checks passed
@jvmakine jvmakine deleted the juho/fix-sync-on-missing-packages branch May 15, 2024 21:57
spicykay pushed a commit to spicykay/hermit that referenced this pull request Aug 23, 2024
Co-authored-by: Liam Nattrass <[email protected]>
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.

Hermit should sync the manifests whenever it comes across a package version not in the local manifests
2 participants