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

Feat/pub fq #114

Closed
Closed

Conversation

CPerezz
Copy link

@CPerezz CPerezz commented Sep 11, 2023

This makes avaliable the Fq struct so that we can integrate Bls12_381 into halo2curves by implementing the endomorphism trait CurveEndo for it.

CPerezz and others added 6 commits September 8, 2023 10:39
In order to solve:
`error: package `rayon-core v1.11.0` cannot be built because it requires rustc 1.59 or newer, while the currently active rustc version is 1.56.0`

This is due to the fact that:
```
❯ cargo tree -i rayon-core
rayon-core v1.11.0
└── rayon v1.7.0
    └── criterion v0.3.6
        [dev-dependencies]
        └── bls12_381 v0.8.0
```

As you can see here, `criterion` depends on `rayon_core` with `^0.3`. After `rayon` moved up MSRV without bumping minor version, everything broke.

This bumps the MSRV to 1.59 fixing this issue. See: zkcrypto#112
After bumping to `1.59`, I saw the following:
`error: package `regex-syntax v0.7.2` cannot be built because it requires rustc 1.60.0 or newer, while the currently active rustc version is 1.59.0`

```
cargo tree -i regex-syntax
regex-syntax v0.7.2
└── regex v1.8.4
    └── criterion v0.3.6
        [dev-dependencies]
        └── bls12_381 v0.8.0
```

It seems same thing happened as with `rayon_core`. Bumping to `1.60.0` solves all the issues.
It turns out that when `zeroize` and other features are set we get:
`package `memchr v2.6.3` cannot be built because it requires rustc 1.61
or newer`.

Therefore, this should be the last bump needed.
@CPerezz
Copy link
Author

CPerezz commented Sep 11, 2023

My bad. Opened in the wrong remote. Will open a PR later once #113 gets or not merged & also I discuss this with @str4d

@CPerezz CPerezz closed this Sep 11, 2023
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

Successfully merging this pull request may close these issues.

1 participant