Skip to content

Make 'cabal clean' more robust: retry after getting 'Unsatisfied constraints (directory is non empty)' error#11938

Open
Bodigrim wants to merge 2 commits into
masterfrom
more-robust-cmdclean
Open

Make 'cabal clean' more robust: retry after getting 'Unsatisfied constraints (directory is non empty)' error#11938
Bodigrim wants to merge 2 commits into
masterfrom
more-robust-cmdclean

Conversation

@Bodigrim

@Bodigrim Bodigrim commented Jun 5, 2026

Copy link
Copy Markdown
Collaborator

I've been testing Cabal HEAD manually and uncovered another case when retrying cabal clean is helpful. This is a follow up to #11604.


Template Α: This PR modifies behaviour or interface

Include the following checklist in your PR:

@Bodigrim Bodigrim force-pushed the more-robust-cmdclean branch 2 times, most recently from 854b26e to 81ce9df Compare June 5, 2026 22:13

@ffaf1 ffaf1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Very good.

If this does not make the 3.18 cut, it will have to be rebased.

@Bodigrim Bodigrim force-pushed the more-robust-cmdclean branch from 81ce9df to 79e93c6 Compare June 6, 2026 09:33
@Bodigrim

Bodigrim commented Jun 6, 2026

Copy link
Copy Markdown
Collaborator Author

This still needs one more approval please.

@ulysses4ever ulysses4ever left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I must confess this picking and choosing seemingly random error types to retry doesn't look pretty to me. But if it helps the cause, I can live with it ...

@Bodigrim Bodigrim added merge me Tell Mergify Bot to merge and removed attention: needs-review labels Jun 6, 2026
@mergify mergify Bot added ready and waiting Mergify is waiting out the cooldown period merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days queued labels Jun 6, 2026
@mergify

mergify Bot commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Merge Queue Status

This pull request spent 1 hour 38 minutes 29 seconds in the queue, including 1 hour 28 minutes 3 seconds running CI.

Waiting for any of
  • check-neutral = Validate post job
  • check-skipped = Validate post job
  • check-success = Validate post job
All conditions
  • any of [🛡 GitHub branch protection]:
    • check-neutral = Validate post job
    • check-skipped = Validate post job
    • check-success = Validate post job
  • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = Doctest Cabal
    • check-neutral = Doctest Cabal
    • check-skipped = Doctest Cabal
  • any of [🛡 GitHub branch protection]:
    • check-success = Meta checks
    • check-neutral = Meta checks
    • check-skipped = Meta checks
  • any of [🛡 GitHub branch protection]:
    • check-success = docs/readthedocs.org:cabal
    • check-neutral = docs/readthedocs.org:cabal
    • check-skipped = docs/readthedocs.org:cabal
  • any of [🛡 GitHub branch protection]:
    • check-success = fourmolu
    • check-neutral = fourmolu
    • check-skipped = fourmolu
  • any of [🛡 GitHub branch protection]:
    • check-success = hlint
    • check-neutral = hlint
    • check-skipped = hlint
  • any of [🛡 GitHub branch protection]:
    • check-success = Bootstrap post job
    • check-neutral = Bootstrap post job
    • check-skipped = Bootstrap post job
  • any of [🛡 GitHub branch protection]:
    • check-success = whitespace
    • check-neutral = whitespace
    • check-skipped = whitespace
  • any of [🛡 GitHub branch protection]:
    • check-success = Check sdist post job
    • check-neutral = Check sdist post job
    • check-skipped = Check sdist post job
  • any of [🛡 GitHub branch protection]:
    • check-success = Changelogs
    • check-neutral = Changelogs
    • check-skipped = Changelogs

Reason

The merge conditions cannot be satisfied due to failing checks

Failing checks:

Hint

You may have to fix your CI before adding the pull request to the queue again.
If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio queue comment.

mergify Bot added a commit that referenced this pull request Jun 8, 2026
@mergify mergify Bot added dequeued and removed queued labels Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dequeued merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days merge me Tell Mergify Bot to merge ready and waiting Mergify is waiting out the cooldown period

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants