-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Prefer composable approach #53
Comments
Yeah, I agree with using overlays. I've been using them both at work and also in some personal projects and I'm very happy with them. I also agree with avoiding I didn't know about |
iirc, extend doesn't play nice with override.
…On Mon, Jul 2, 2018, 10:45 AM Gabriel Gonzalez ***@***.***> wrote:
Yeah, I agree with using overlays. I've been using them both at work and
also in some personal projects and I'm very happy with them.
I also agree with avoiding rec (in favor of chaining multiple overlays)
I didn't know about extend, though. Let me check that out and learn how
it works
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#53 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACNoMT5En73b8nBhArLmwb6_3jkETIivks5uClwsgaJpZM4U-5G0>
.
|
… On Mon, Jul 2, 2018, 10:48 AM Spiros Boosalis ***@***.***> wrote:
iirc, extend doesn't play nice with override.
On Mon, Jul 2, 2018, 10:45 AM Gabriel Gonzalez ***@***.***>
wrote:
> Yeah, I agree with using overlays. I've been using them both at work and
> also in some personal projects and I'm very happy with them.
>
> I also agree with avoiding rec (in favor of chaining multiple overlays)
>
> I didn't know about extend, though. Let me check that out and learn how
> it works
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#53 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACNoMT5En73b8nBhArLmwb6_3jkETIivks5uClwsgaJpZM4U-5G0>
> .
>
|
Adding two examples of Haskell overlays which you might find handy:
I'm still not sure if this is their final form as there's some clutter I would like to refactor. Feedback welcome - I will try to create a PR for this repository with the example when I'm sure about the approach. |
The examples use methods that cause issues when trying to extend the package set further. Using the right tools, which are no harder, we can avoid frustration down the line.
rec
keyword. People get used to it and will use it in overlay definitions, leading to surprises.haskellPackages.extend
instead of.override { overrides =
. It lets you add more layers of overrides which is more flexible. Attempting the same on the current code replaces the single layer of overrides with the new one, which is unexpected. As a bonus, it is syntactically simpler.The text was updated successfully, but these errors were encountered: