Skip to content

Decomission build-type: Make#11894

Open
Bodigrim wants to merge 2 commits into
masterfrom
decomission-build-type-make
Open

Decomission build-type: Make#11894
Bodigrim wants to merge 2 commits into
masterfrom
decomission-build-type-make

Conversation

@Bodigrim
Copy link
Copy Markdown
Collaborator

@Bodigrim Bodigrim commented May 26, 2026

Closes #11610.

build-type: Make has been broken at least since Cabal 3.4, which is 7 releases back, and no one ever complained. That's been enough of "deprecation" period ;)

(One of the reasons it was broken is that Distribution.Make.buildAction is from the epoch when Cabal packages didn't have any components and mandates noExtraFlags, while the rest of Cabal expects that buildAction should be able to take a component target and supplies one by default, even if a user didn't.)

The only Hackage package, whose latest release uses build-type: Make, is lvmrun: last released in 2014 and long broken because of the above.


Template Α: This PR modifies behaviour or interface

Include the following checklist in your PR:

@Bodigrim Bodigrim force-pushed the decomission-build-type-make branch 2 times, most recently from 212ab3b to c94f9b1 Compare May 26, 2026 23:40
Comment thread Cabal-syntax/src/Distribution/Types/BuildType.hs
Comment thread changelog.d/pr-11894.md Outdated
It's been broken at least since Cabal 3.4, which is 7 releases back,
and no one ever complained. That's been enough of "deprecation" period ;)

(One of the reasons it was broken is that `Distribution.Make.buildAction`
is from the epoch when Cabal packages didn't have any components
and mandates `noExtraFlags`, while the rest of Cabal expects that `buildAction`
should be able to take a component target and supplies one by default,
even if a user didn't.)

The only Hackage package, whose latest release uses `build-type: Make`,
is `lvmrun`: last released in 2014 and long broken because of the above.
@Bodigrim Bodigrim force-pushed the decomission-build-type-make branch from c94f9b1 to 1a3ed1b Compare May 27, 2026 21:22
Comment thread changelog.d/pr-11894.md Outdated
Co-authored-by: Artem Pelenitsyn <a.pelenitsyn@gmail.com>
@Bodigrim Bodigrim added the merge me Tell Mergify Bot to merge label May 28, 2026
@mergify mergify Bot added the ready and waiting Mergify is waiting out the cooldown period label May 28, 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.

Is it time to sunset build-type: Make?

3 participants