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.

Sign up for GitHub

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

Jump to bottom

Error compiling the noirc_frontend crate #403

Closed
ilitteri opened this issue Oct 24, 2022 · 1 comment
Closed

Error compiling the noirc_frontend crate #403

ilitteri opened this issue Oct 24, 2022 · 1 comment
Labels
bug Something isn't working

Comments

@ilitteri
Copy link
Contributor

Description

Aim

To compile noirc_frontend crate.

Expected behavior

It should compile successfully.

Bug

There is no way to specify which acvm feature to use.

acvm has two features but none of them is specified in the Cargo.toml nor could be specified by the user in the terminal. Also acvm has no default feature.

I think that the default feature of acvm should be bn254 as it is in noir_fields and noirc_frontend should inherit both features as well.

To reproduce

For some crates you have two different features for compilation both related to the elliptic curves used (bn254, bls12381). In those crates there is no default feature set and you should specify which to use or you are getting a compilation error (saying that you should use a feature).

  1. Clone the repo.
  2. cd crates/noirc_frontend.
  3. cargo c or cargo c --features bn254 or cargo c --features bls12_381.

Environment

  • OS: macOS Monterey 12.6
@ilitteri ilitteri added the bug Something isn't working label Oct 24, 2022
ilitteri added a commit to lambdaclass/noir that referenced this issue Oct 24, 2022
@ilitteri
Update Cargo.toml
59faf5c 
This commit fixes noir-lang#403.

No compilation features were provided so the user wasn't able to compile this crate successfully.

Now the user could compile with the bn254 feature or the bls12_381 feature.
This was referenced Oct 24, 2022
Closed
Closed
Closed
@ilitteri
Copy link
Contributor Author

ilitteri commented Dec 9, 2022

See #404 closing comment.

@ilitteri ilitteri closed this as completed Dec 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix error compiling the noirc_frontend crate lambdaclass/noir
1 participant
@ilitteri