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

Bump mill from 0.11.5 to 0.12.5 #133

Merged
merged 6 commits into from
Jan 30, 2025
Merged

Conversation

Emin017
Copy link
Contributor

@Emin017 Emin017 commented Jan 28, 2025

  • Update the .mill-version file to specify the new mill version 0.12.5.
  • Rename the build.mill file to build.sc to match the updated mill version.
  • Adjust the SbtModule and SbtModuleTests imports to reflect the changes in the mill 0.12.5 API.
  • Add .mill-jvm-opts for Chisel test generation and output file handling
  • Update Java version requirement to JDK 11 in README.md

@jackkoenig
Copy link
Contributor

This needs .mill-jvm-ops containing -Dchisel.project.root=${PWD} as discussed in com-lihaoyi/mill#3840. It would also be good to include some explanation of why that is in a comment--probably in the build.mill file.

Since Mill 0.12 requires Java 11+, this also needs to update the README to no longer suggest Java 8 (possibly with a comment clarifying that Chisel works with Java 8 but our preferred build tool, Mill, requires Java 11).

* Add .mill-jvm-opts
* Add explanation comments in build.mill

Signed-off-by: Qiming Chu <[email protected]>
Update README to clarify that Mill 0.12+ requires Java 11 or newer,
while noting that Chisel itself is compatible with Java 8.

Signed-off-by: Qiming Chu <[email protected]>
Copy link
Contributor

@jackkoenig jackkoenig left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

@Emin017
Copy link
Contributor Author

Emin017 commented Jan 29, 2025

Sorry, I missed changing build.sc to build.mill in test.yml. Could we re-run the CI?

@jackkoenig
Copy link
Contributor

You might want to switch from using mill installed by a Github Action to using the Mill bootstrap script: https://mill-build.org/mill/cli/installation-ide.html#_bootstrap_scripts

@jackkoenig jackkoenig enabled auto-merge (squash) January 30, 2025 00:01
@jackkoenig jackkoenig merged commit 23d6c30 into chipsalliance:main Jan 30, 2025
1 check 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