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

Support RocketChip build Chisel from source #3507

Merged
merged 3 commits into from
Nov 15, 2023
Merged

Support RocketChip build Chisel from source #3507

merged 3 commits into from
Nov 15, 2023

Conversation

sequencer
Copy link
Member

@sequencer sequencer commented Oct 8, 2023

This PR support RocketChip build chisel from source, this flow is not for users, only for devs to test new version of Chisel and check some backport feature is working when they are not released to maven.

Related issue:

Type of change: bug report | feature request | other enhancement

Impact: no functional change | API addition (no impact on existing code) | API modification

Development Phase: proposal | implementation

Release Notes

@sequencer sequencer changed the base branch from master to dev October 8, 2023 00:28
@jerryz123
Copy link
Contributor

I am generally cautious about relying on unreleased versions of dependencies.....

IMO default build should always build from released versions, which developer-only flag to build from source.

@sequencer
Copy link
Member Author

I am generally cautious about relying on unreleased versions of dependencies.....

RocketChip won't depend on the unreleased version of Chisel, this is only for debug use.

Default build will only build from the release version of Chisel

@sequencer sequencer force-pushed the chisel_source branch 2 times, most recently from 5132a13 to 0e59371 Compare October 17, 2023 10:24
@sequencer
Copy link
Member Author

@jerryz123 I think this is safe to get in, just provide an additional source cross target, no influence to our release flow.

@sequencer sequencer merged commit 11f0119 into dev Nov 15, 2023
2 of 25 checks passed
@sequencer sequencer deleted the chisel_source branch November 15, 2023 06:40
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