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

4 packages from gitlab.com/nomadic-labs/data-encoding/-/archive/v1.0.1/data-encoding-v1.0.1.tar.gz #24678

Conversation

raphael-proust
Copy link
Collaborator

This pull-request concerns:
-data-encoding.1.0.1: Library of JSON and binary encoding combinators
-json-data-encoding.1.0.1: Type-safe encoding to and decoding from JSON
-json-data-encoding-browser.1.0.1: Type-safe encoding to and decoding from JSON (browser support)
-json-data-encoding-bson.1.0.1: Type-safe encoding to and decoding from JSON (bson support)



🐫 Pull-request generated by opam-publish v2.1.0

@raphael-proust
Copy link
Collaborator Author

Turns out there's a missing constraint on zarith. I'm adding this.

@raphael-proust
Copy link
Collaborator Author

janestreet/zarith_stubs_js#10 ← this is meant to fix the underlying issue.

@avsm
Copy link
Member

avsm commented Nov 1, 2023

Still tests failing here...

@raphael-proust
Copy link
Collaborator Author

After #24755 is merged this here MR should be updated with new constraints.

@raphael-proust raphael-proust force-pushed the opam-publish-data-encoding-json-data-encoding-json-data-encoding-browser-json-data-encoding-bson.1.0.1 branch from df9c11f to 1d78feb Compare January 15, 2024 08:38
@raphael-proust
Copy link
Collaborator Author

Updated with zarith stubs constraints

@raphael-proust
Copy link
Collaborator Author

CI errors are:

  • can't generate .exe on 32bit+ocaml5 plateforms (no rule for whatever.exe)
  • some system dependencies and other packages
  • what seems to be an actual issue on s390x-ocaml-5.1

I don't care too much about the s390x/5.1 issue because it is quite a specific combination.

@mseri
Copy link
Member

mseri commented Jan 24, 2024

what seems to be an actual issue on s390x-ocaml-5.1

Should we make it unavailable there? It may be something to do with little or big endianness.

@raphael-proust
Copy link
Collaborator Author

I've added the availability field.

I don't think it's an endianness issue because it works on s390x-ocaml.4.14. So it's probably a combination of multiple factors, endianness being only one of them?

@avsm
Copy link
Member

avsm commented Jan 26, 2024

Possibly ocaml/ocaml#12919 for on s390x

@mseri
Copy link
Member

mseri commented Mar 14, 2024

What is the status? Can we merge this?

@raphael-proust
Copy link
Collaborator Author

raphael-proust commented Mar 20, 2024

@mseri I guess technically the package is fine but the s390x+ocaml.5.1 combination is broken. I'm ok merging with the current availability field. The main issue is that when a new version of ocaml is released with the issue fixed we should then fix the available field to be more accurate.

Still, I think we can merge.

@mseri
Copy link
Member

mseri commented Mar 21, 2024

Please send an update to the availability field once a fixed ocaml is released

@mseri mseri merged commit a874cb8 into ocaml:master Mar 21, 2024
1 of 2 checks passed
tezoslibrarian pushed a commit to tezos/tezos-mirror that referenced this pull request Apr 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants