Skip to content
This repository was archived by the owner on Feb 17, 2025. It is now read-only.

CMake files updated for usage with crypto3 #88 #89

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

CMake files updated for usage with crypto3 #88 #89

wants to merge 1 commit into from

Conversation

@ETatuzova
Copy link
Contributor

@ETatuzova ETatuzova commented Jul 11, 2024

No description provided.

@ETatuzova ETatuzova requested a review from AndreyMlashkin July 11, 2024 11:11
@ETatuzova ETatuzova linked an issue Jul 11, 2024 that may be closed by this pull request
Convenient build with crypto3 #88
Open
@AndreyMlashkin
Copy link
Contributor

AndreyMlashkin commented Jul 11, 2024

It's fine to have it in a branch, but it shouldn't go to master. Current crypto3 uses ::all target

@ETatuzova
Copy link
Contributor Author

ETatuzova commented Jul 16, 2024 •
edited
Loading

It's fine to have it in a branch, but it shouldn't go to master. Current crypto3 uses ::all target

@AndreyMlashkin, are you sure?
I tried it with latest master and got an error:

CMake Error at libs/transpiler/CMakeLists.txt:47 (target_link_libraries):
  The link interface of target "crypto3_transpiler" contains:

    crypto3::all

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@AndreyMlashkin AndreyMlashkin Awaiting requested review from AndreyMlashkin

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Convenient build with crypto3

3 participants

@ETatuzova @AndreyMlashkin