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

[opam] upper bound on cohttp #96

Closed
jrochel opened this issue Feb 22, 2021 · 4 comments
Closed

[opam] upper bound on cohttp #96

jrochel opened this issue Feb 22, 2021 · 4 comments

Comments

@jrochel
Copy link

jrochel commented Feb 22, 2021

In the opam repository of this package (https://opam.ocaml.org/packages/aws/) there is an upper version limit (implemented as a conflict) on the cohttp package:

conflicts: [
  "cohttp" {< "0.17.0"}
  "cohttp" {>="0.99"}
]

Is there any good reason for this? Everything compiles just fine for me with cohttp.2.5.4.

@kit-ty-kate
Copy link

Added details from ocaml/opam-repository#18212 (comment): the latest available version does not compile with cohttp + lwt + ssl so the constraints are correct.

I would encourage the developers here to make a new release as the master branch seems to be alright.

@tmcgilchrist
Copy link
Collaborator

Thanks for the report @jrochel I'm working on putting a new version onto opam.
This issue #83 tracks the outstanding tasks.

Which AWS services are you interested in using?

@jrochel
Copy link
Author

jrochel commented Feb 24, 2021

We are currently using a whole bunch: S3, cloudformation, sns, sqs, rds, cloudwatch, Elasticsearch, Lambda, Ses, Mediaconvert, ssm. But we are using our own library, which is hand-coded and only very partial, and ugly, too ugly for publishing.

I'm currently working on adding DynamoDB to the mix and I would very much like to use your package aws for it. If everything works out fine, I'll probably work on porting our bs-aws (https://github.com/besport/bs-aws) package to aws, this adding some functorial interfaces and abstractions on top.

I'll probably make a PR (adding DynamoDB to aws) soon, however I'm having some troubles with mutual recursion in the botocore specifications. I'll work a bit on a solution, which I'd like to discuss with you guys.

@tmcgilchrist
Copy link
Collaborator

This should be resolved in the latest 1.2 release on OPAM https://opam.ocaml.org/packages/aws/
Please reopen if the problem still exists.

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

3 participants