Skip to content

chore: Update bender version + use default remotes in Bender.yml#305

Merged
colluca merged 6 commits intopulp-platform:mainfrom
fischeti:update-bender
Mar 4, 2026
Merged

chore: Update bender version + use default remotes in Bender.yml#305
colluca merged 6 commits intopulp-platform:mainfrom
fischeti:update-bender

Conversation

@fischeti
Copy link
Contributor

@fischeti fischeti commented Feb 19, 2026

Changes related to bender version update

  • bender now emits an error if source file do not exist (suppressible with --suppress E31). I added a small fix in Makefile to make sure that sources are generated before calling bender script verilator
  • bender now allows to specify use default remotes to shorten the description of versioned dependencies.
  • Moves hw/generated include dir as previously tried in Bender.yml: Move generated include dir into source group #304 and reverted back.

TODOs

  • Wait for bender 0.30.1 0.31.0 for some fixes
  • Check internal Gitlab CI

Copy link
Collaborator

@colluca colluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @fischeti!

I suppose the default remote can't be used for non-versioned dependencies, is that correct?

@fischeti
Copy link
Contributor Author

fischeti commented Mar 4, 2026

I suppose the default remote can't be used for non-versioned dependencies, is that correct?

Yes, this is intentional design because we want to nudge people into using versioned releases as much as possible😉

The PR is ready again now, a new Bender version was released yesterday with some fixes, and I also fixed #304 again in this PR. Gitlab CI passes as well this time.

Copy link
Collaborator

@colluca colluca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM :)

@colluca colluca merged commit 8320dc0 into pulp-platform:main Mar 4, 2026
11 of 12 checks passed
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