Skip to content
Snippets Groups Projects
Unverified Commit 460375d7 authored by mergify[bot]'s avatar mergify[bot] Committed by GitHub
Browse files

fix `cabal install --program-suffix/prefix` (fix #10290 and #10476) (#10483) (#10510)


* fix cabal install --program-suffix/prefix (fix #10290 and #10476)

When checking for existing installations, cabal would not account for
an affix (suffix or prefix). So, if you had a `hello` binary installed, installing a
second one with a non-empty affix (a perfectly legal operation) would
fail.

The reason seemed to be a typo in
09c04e9a, which passed the arguments to
the Symlink structure in a wrong order.

When failing to install a binary because of an existing one, cabal would
report suffix-less existing target even if a suffix was set.

* Add regression tests for overwrite policies and porgram-affixes

Add regression tests for the `program-prefix` and `program-suffix` flags
combined with the overwrite-policy.
In short, the overwrite-policy needs to take potential program affixes
into account when deciding whether it will need to overwrite a program
path during installation.

---------

Co-authored-by: default avatarFendor <fendor@posteo.de>
(cherry picked from commit ee3c3139)

Co-authored-by: default avatarArtem Pelenitsyn <a.pelenitsyn@gmail.com>
parent a3a1dfbe
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment