Skip to content

Fix "fusesoc tool list" with edalize 0.6.2+#769

Open
jmerdich wants to merge 1 commit intoolofk:mainfrom
jmerdich:patch-1
Open

Fix "fusesoc tool list" with edalize 0.6.2+#769
jmerdich wants to merge 1 commit intoolofk:mainfrom
jmerdich:patch-1

Conversation

@jmerdich
Copy link
Copy Markdown

Edalize removed the methods previously used to list the tools (walk_tool_packages). Use the new methods when present.

Edalize removed the methods previously used to list the tools (`walk_tool_packages`). Use the new methods when present.
jmerdich added a commit to jmerdich/edalize that referenced this pull request Mar 20, 2026
Found when making olofk/fusesoc#769. 

Edatool is not itself a tool, just a base class. Don't list it when enumerating tools.
@jmerdich
Copy link
Copy Markdown
Author

Fixes #755

olofk pushed a commit to olofk/edalize that referenced this pull request Mar 23, 2026
Found when making olofk/fusesoc#769. 

Edatool is not itself a tool, just a base class. Don't list it when enumerating tools.
@olofk
Copy link
Copy Markdown
Owner

olofk commented Mar 24, 2026

Thanks for this. Functionality looks fine, but the code doesn't pass the formatting check. Make sure you run black on the code to format this properly. Easiest way is to set it up as a pre-commit hook by running pre-commit install in your local repo. This will do a formatting check every time you commit.

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.

2 participants