Skip to content
This repository has been archived by the owner on Aug 19, 2024. It is now read-only.

Allow patched versions of Verilator #582

Merged

Conversation

stevenmburns
Copy link

We want to be able to use the same version of Verilator for chiseltest that we use for cocotb testing. Currently we need to use a patched version of verilator for cocotb and this breaks the version parsing mechanism in chiseltest. This PR just allows more version names, specifically it allows the name 5.002.p (actually anything with the format D.DDD or D.DDD.anystring).

Copy link
Collaborator

@ekiwi ekiwi left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM

@ekiwi ekiwi modified the milestones: 0.6.0, 0.5.x Dec 7, 2022
@ekiwi ekiwi added the Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI. label Dec 7, 2022
@ekiwi ekiwi force-pushed the fix/allow_patched_verilator_versions branch from 714b61e to 3a0d271 Compare December 7, 2022 19:55
@ekiwi ekiwi force-pushed the fix/allow_patched_verilator_versions branch from 3a0d271 to 65bbf11 Compare December 7, 2022 21:05
@mergify mergify bot merged commit 9a9d347 into ucb-bar:main Dec 7, 2022
mergify bot pushed a commit that referenced this pull request Dec 7, 2022
@mergify mergify bot added the Backported This PR has been backported to marked stable branch label Dec 7, 2022
mergify bot added a commit that referenced this pull request Dec 7, 2022
(cherry picked from commit 9a9d347)

Co-authored-by: Steve Burns <steven.m.burns@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Backported This PR has been backported to marked stable branch Please Merge Accepted PRs that are ready to be merged. Useful when waiting on CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants