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

Hackage has version that has package constraints #1

Closed
msooseth opened this issue Oct 6, 2022 · 5 comments
Closed

Hackage has version that has package constraints #1

msooseth opened this issue Oct 6, 2022 · 5 comments

Comments

@msooseth
Copy link

msooseth commented Oct 6, 2022

Hi,

We are planning on using your tool in our system, however, the Hackage version of it unfortunately misses your commit 5abbc3e, i.e. "remove package constraints" This means that we can't add it as a dependency, as then we have issues with dependencies that cannot be met.

Maybe you have forgotten to upload the new version to Hackage? Or is there a reason why the new version is not on Hackage? I'd love to use your tool directly from Hackage. Would it be possible to push the new version, without the package constraints, to Hackage?

Thanks a lot in advance,

Mate

@crvdgc
Copy link
Collaborator

crvdgc commented Oct 7, 2022

Greetings. I indeed didn't upstream the commit to Hackage. On Hackage, a more strict semver policy is enforced (though in practice, it seems to be only encouraged). The constraint removal commit is only a hacky one to quickly fix another project that depends on this.

I will add a more precise dependency and upstream it to Hackage ASAP.

@msooseth
Copy link
Author

msooseth commented Oct 7, 2022

Thank you so much! That would help us a ton. We are using it to parse the response from SMT solvers for our symbolic execution framework, in progress, here: https://github.com/ethereum/hevm (open source, of course)

Thank you again and looking forward to the fix on Hackage,

Mate

@crvdgc
Copy link
Collaborator

crvdgc commented Oct 8, 2022

I relaxed the version constraint and tested it with stackage lts-19.27 and made it available at hackage. Let me know if this solves your problem.

Thank you for wanting to use it, but a word of warning though: only the function definition part is tested thoroughly in another project and I was told parsec may not be efficient enough for performance critical use cases.

@msooseth
Copy link
Author

msooseth commented Oct 10, 2022

Yaaaaay, that fixed it! Thank you so much! I am the developer of CryptoMiniSat and ApproxMC and work on STP -- if you ever need anything, just ping me at @msoos

Cheers,

Mate

@crvdgc
Copy link
Collaborator

crvdgc commented Oct 10, 2022

Glad to help.

@crvdgc crvdgc closed this as completed Oct 10, 2022
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