Page MenuHomeFreeBSD

Mk/Uses/cabal.mk: Support for USE_CABAL in separate Makefile.cabal
ClosedPublic

Authored by alven on Aug 16 2025, 9:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 27 2025, 11:09 AM
Unknown Object (File)
Nov 21 2025, 4:20 AM
Unknown Object (File)
Nov 4 2025, 7:15 PM
Unknown Object (File)
Nov 4 2025, 7:13 PM
Unknown Object (File)
Nov 4 2025, 7:11 PM
Unknown Object (File)
Oct 29 2025, 6:41 AM
Unknown Object (File)
Oct 11 2025, 10:30 PM
Unknown Object (File)
Oct 11 2025, 10:30 PM
Subscribers
None

Details

Summary

Like we already do have in Mk/Uses/cargo.mk for Makefile.crates

  • Adopt www/hs-postgrest to this change in framework.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

alven requested review of this revision.Aug 16 2025, 9:52 PM
alven created this revision.

I find .modules part to be confusing. Rust names it .crates, so we probably should call it .cabal

Mk/Uses/cabal.mk
78
3) make make-use-cabal > Makefile.modules

I find .modules part to be confusing. Rust names it .crates, so we probably should call it .cabal

crate => crates
cabal => cabals

Cabal is a package manager name and a file format, so no, plural seems unfitting there.

Anyone from portmgr@: please come and help us with this simplest task... :)

alven retitled this revision from Mk/Uses/cabal.mk: support for USE_CABAL in separate Makefile.modules to Mk/Uses/cabal.mk: Support for USE_CABAL in separate Makefile.modules.Oct 7 2025, 3:15 PM

If you want an approval from me then I'd like to see the file named Makefile.cabal, not .modules.

alven retitled this revision from Mk/Uses/cabal.mk: Support for USE_CABAL in separate Makefile.modules to Mk/Uses/cabal.mk: Support for USE_CABAL in separate Makefile.cabal.Tue, Feb 17, 7:11 AM
alven added 1 blocking reviewer(s): arrowd.

Use filename 'Makefile.cabal'

Mk/Uses/cabal.mk
78

@alven What do you think about this comment? ^

Given that you verified that this actually works, LGTM.

You can also adapt www/hs-postgrest to make use of this feature.

Thank you! What now?

Mk/Uses/cabal.mk
78

Was just a mistake you've repeated after me? :)
I've fixed it to what you like now. It was all that simple, yes? :)

This revision was not accepted when it landed; it landed in state Needs Review.Tue, Feb 17, 8:16 AM
This revision was automatically updated to reflect the committed changes.
alven signed these changes with MFA.Tue, Feb 17, 8:20 AM

Thank you! :)