Skip to content
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

styx #1

Closed
jyp opened this issue Jan 31, 2017 · 2 comments
Closed

styx #1

jyp opened this issue Jan 31, 2017 · 2 comments

Comments

@jyp
Copy link

jyp commented Jan 31, 2017

You may be interested in this tool I've made: https://github.com/jyp/styx
If you have feedback on it I'd love to hear it!

@shajra
Copy link
Owner

shajra commented Jan 31, 2017

@jyp thanks a lot for reminding me about this project (I remember seeing a link fly by on Twitter, but I didn't follow up).

It's kind of funny. As I was reading your code, I was thinking "Is making the shell.nix as hard as I'm making it out to be? Maybe I should think about it fresh. And then I realized it could be as simple as swapping in a custom mkDerivation into the function generated by cabal2nix. And then I saw that that's exactly what you're doing with gatherDeps.

So I'm going to adjust this project to do a similar approach. After that, I won't feel like a "hack," just a reasonable solution.

As far as my review of styx, I don't think it's a bad approach at all. It introduces a tool to manage your "*.nix" files, so we don't have to think about Nix as much. My project is going down the other path -- trying to see how clean we can go just with normal Nix expressions before leaning on a tool like styx. I think there's some value in both approaches. You probably are missing some configuration option to allow people the full flexibility they'd get with raw Nix, and it's your call on whether to support those or not (maintenance isn't free). For instance, if someone wanted to turn on profiling, what's your recommended way of doing that with styx? There's two answers I can imagine: 1) just write Nix expressions that build upon the ones styx generates 2) adding more switches/configuration into styx's Main.hs source.

Thanks again for the pointer. Hopefully we can get more people using Nix where makes sense.

@jyp
Copy link
Author

jyp commented Feb 1, 2017 via email

@jyp jyp closed this as completed Feb 3, 2017
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

No branches or pull requests

2 participants