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

[new release] bam (3 packages) (0.2) #25864

Merged
merged 2 commits into from
May 16, 2024
Merged

Conversation

francoisthire
Copy link

A property-based testing library with internal shrinking

CHANGES:

@francoisthire francoisthire force-pushed the release-bam-v0.2 branch 8 times, most recently from 61a8bcb to 2313a21 Compare May 12, 2024 22:35
@francoisthire
Copy link
Author

The current issue seems to concern 32 bits architectures. I am not sure where it comes from. I am using dune to generate the opam files. I did not find a way to specify the architecture via the dune-project file.

@raphael-proust
Copy link
Collaborator

Not all fields of the opam file are available in dune. The missing ones can be added via the use of a separate file. See https://dune.readthedocs.io/en/stable/reference/packages.html in which you find

If a pkg.opam.template file is found, the fields found in this opam file are added to the generated file, overriding the generated ones in case of collision. This can be used as an escape hatch to add arbitrary opam fields.

So I think you need this template file to contain

conflicts: [
  "ocaml-option-bytecode-only"
]

@raphael-proust
Copy link
Collaborator

The actual addition you want is probably

available: arch != "arm32" & arch != "x86_32"

Sorry for the confusion, I copied the wrong fix.

CHANGES:

- francoisthire/bam#2,francoisthire/bam#5: Add a PPX for Bam derving generators from a type or a module type

- francoisthire/bam#1: Fix the `Bam.Std.oneof` generator
packages/bam/bam.0.2/opam Outdated Show resolved Hide resolved
@francoisthire
Copy link
Author

The CI was green before, and not it is pending/with failurs. Is it expected?

@mseri
Copy link
Member

mseri commented May 16, 2024

The issues are on the CI now, this looks good. Thanks

@mseri mseri merged commit 691a55a into ocaml:master May 16, 2024
1 of 2 checks passed
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

4 participants