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

Build Synlig as standalone binary #2538

Merged
merged 3 commits into from
Sep 5, 2024
Merged

Conversation

kamilrakoczy
Copy link
Collaborator

@kamilrakoczy kamilrakoczy commented Sep 4, 2024

Synlig is no longer a yosys plugin, it is a standalone tool now, it uses yosys as a framework for synthesis.

It is still possible to build synlig as a plugin, but it might be removed in the future.

Requires: #2539

@kamilrakoczy kamilrakoczy force-pushed the 64605-standalone-binary branch 2 times, most recently from 59a6cf6 to 33b3c50 Compare September 4, 2024 12:51
@tgorochowik tgorochowik self-assigned this Sep 4, 2024
Copy link

github-actions bot commented Sep 4, 2024

@alaindargelas
Copy link
Collaborator

alaindargelas commented Sep 4, 2024

@kamilrakoczy, Please do not remove the CMake-based plugin build system ever as we need a way to build synlig as a plugin.
Being completely hard wired to a particular version of Yosys in not a good thing when one as a Yosys fork as we do.
The Makefile plugin build may go away.

@kamilrakoczy
Copy link
Collaborator Author

This PR changes the default build configuration for synlig, it does not remove the CMake-based build system.

We encourage external users to use synlig as a standalone binary, as it makes sure that synlig will work correctly without any requirements for a specific yosys version.

This workaround addresses an issue with Surelog cache that sometimes gets corrupted.
@alaindargelas
Copy link
Collaborator

Sure, that works for regular end users who want the default latest Yosys, it does not necessarily work for the Yosys-mingling ones

@tgorochowik tgorochowik merged commit 13705ab into main Sep 5, 2024
30 checks passed
@tgorochowik tgorochowik deleted the 64605-standalone-binary branch September 5, 2024 19:42
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.

4 participants