-
-
Notifications
You must be signed in to change notification settings - Fork 48
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
Support GHC 9.0 #423
Comments
Please <3 |
Issues this one dependent on: |
Stack is now built with GHC 9.2.4 and Stan does not currently support GHC >= 9.0. See kowainik/stan#423
uf, can be tricky, not only modules was rearranged, but also api was changed in ghc package... |
I recommend switching to |
I've brought stan up to date all the way to 9.41, including updating dependencies across the entire Kowainik ecosystem. It's not clear to me whether stan is actively being maintained. If not I may make a fork. https://github.com/kowainik/stan/compare/main...tomjaguarpaw:stan:9.4-compat?expand=1 Footnotes
|
Have I done anything wrong here? Using 9.2.4 and 9.4-compat branch.
|
Oops, I pushed the wrong branch. Please fetch the branch |
Errors gone, just warnings and test failures now:
I can probably at least use this by skipping tests now. |
The warnings are expected, the test failures are not. I will have a look. Possibly it's a difference between 9.2.1 (which I used) and 9.2.4 (which you used). If so, that's terrible, but I will try to fix it. |
It's possible that it's all related to -Weverything (almost) or -Werror based stuff? Thanks! |
I can confirm my branch works on
I can confirm it is broken on
I suspect we might just have to forbid stan on 9.2.4. 9.2.4's |
How very strange. For now I'll skip, and upgrade to 9.4.2 when cabal's and others' deps support it. Thanks for your effort! |
I see that this also updates to use clay 0.14, nice one! I'd be happy if the devs of this package merged your branch. I don't know if there're any more issues? |
I don't recall any blocking issues. I think my branch is all that's required to get Stan up to 9.4. (The CPP is a bit hokey and could do with improvement though.) |
I had to make some changes to the
Anyway: thank you for this update! Definitely subscribing to this issue. Would love to see this or something like it merged soon :D |
Ah thanks. I don't use Stack so I didn't know about that. I can add it to my branch at some point, or feel free to submit a PR. |
No worries! I honestly don't know if this is a good approach -- more of a temporary hack, really. Just thought I'd share in case it helps someone else. The It's especially tricky since stan needs to be built with the same GHC version as a given project. I honestly don't know what the "right" solution is here. Maybe to just use cabal :) (I'm pretty new to both haskell and stack, so... grain of salt and all that. I'm sure the stan maintainers can whip up a better solution.) If I do get it working in a "real" way, though, I'll be happy to open a PR! |
@tomjaguarpaw Great stuff with this branch. So, can you create PR or there are some other problems with it? |
No problem as far as I know, although the version-dependent stuff should probably be done a different way. My attempt was an experiment which ended up quite ugly. I don't have the bandwidth to shepherd a PR for it myself right now but if someone else wants to they are welcome to. |
@tomjaguarpaw You mean bunch of |
No, I mean the way of choosing modules for different GHC versions. |
It might be fine. I think it's usually done a slightly different way, I'm not sure because I didn't check :) |
By the way, some of the dependencies still don't build. (I updated the branch to take into account the ones that do.) |
( |
@tomjaguarpaw Yeah, trial seem to not update base. |
Thanks to Hackage revisions made by @andreasabel the vendored dependencies are no longer required! My branch builds against Hackage. |
Well, I think it's time to make my branch an actual PR: #506 |
Stan plugin was removed from HLS, so issue is not relevant to me anymore. |
Supported up to 9.4: https://hackage.haskell.org/package/stan-0.1.0.0 |
I see you became co-maintainer here @tomjaguarpaw , that's very good news! |
No, just |
No description provided.
The text was updated successfully, but these errors were encountered: