Skip to content

Conversation

@Bodigrim
Copy link
Collaborator

Template B: This PR does not modify behaviour or interface

E.g. the PR only touches documentation or tests, does refactorings, etc.

Include the following checklist in your PR:

  • Patches conform to the coding conventions.
  • Is this a PR that fixes CI? If so, it will need to be backported to older cabal release branches (ask maintainers for directions).

@Bodigrim Bodigrim force-pushed the clean-up-Distribution.Compat.GetShortPathName branch from cd91dc2 to 6cd6b31 Compare January 16, 2026 21:56
Copy link
Member

@Mikolaj Mikolaj left a comment

Choose a reason for hiding this comment

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

I wonder who uses it and what for but at least we don't have to implement it now! :)

@geekosaur
Copy link
Collaborator

We may have to keep it around forever. Even if autoconf itself has been bulletproofed, there's no guarantee whatsoever that people's additions to their configure scripts are properly quoted.

@geekosaur
Copy link
Collaborator

And IIRC exhibit A there is ghc.

@Bodigrim Bodigrim added merge me Tell Mergify Bot to merge and removed attention: needs-review labels Jan 17, 2026
@mergify mergify bot added the ready and waiting Mergify is waiting out the cooldown period label Jan 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

4 participants