Skip to content

Remove cabal flags while releasing#3088

Closed
adithyaov wants to merge 4 commits intomasterfrom
flag-removal
Closed

Remove cabal flags while releasing#3088
adithyaov wants to merge 4 commits intomasterfrom
flag-removal

Conversation

@adithyaov
Copy link
Copy Markdown
Member

No description provided.

Copy link
Copy Markdown
Member

@harendra-kumar harendra-kumar left a comment

Choose a reason for hiding this comment

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

Instead of keeping it in streamly repo, we can put it in a separate repo and host a static binary in that repo, download and use it from there.

Comment thread .github/workflows/haskell.yml Outdated
@adithyaov
Copy link
Copy Markdown
Member Author

This is making the maintenance process unnecessarily complex. Ensuring the build works with the new generated cabal file involves building all tests and benchmarks, as well as verifying that existing flags are functional.

There is no easy way to make sure that the newer cabal file is the same as the stripped version of the older cabal file.

Instead, it may be easier to prefix the flags with INTERNAL_. This makes it explicit and cautions the user, albeit it is a little ugly.

Comment on lines +3 to +10
ifdef(`DEV_MODE',
`define(`FLAG_DEV', `flag(dev)')',
`define(`FLAG_DEV', `false')'
)
ifdef(`DEV_MODE',
`define(`FLAG_USE_UNLIFTIO', `flag(use-unliftio)')',
`define(`FLAG_USE_UNLIFTIO', `false')'
)
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

In DEV_MODE, we can enable and disable flags on demand.
In non-DEV_MODE, flags are set to false.

@adithyaov
Copy link
Copy Markdown
Member Author

Too complex for something so simple :-)

@adithyaov adithyaov closed this Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants