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

Unify conf file for sbt and mill. #40

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Readon
Copy link

@Readon Readon commented Jul 17, 2024

Example to support both sbt and mill in one project, and specified version in unified conf file.

@NikLeberg
Copy link

Realistically.. wouldnt someone that uses the template project want to use either sbt or mill and only rarely both? So this would only complicate things.

IMHO there is already a LOT to figure out when starting with scala and spinal, adding another file does not help much.

@Readon
Copy link
Author

Readon commented Jul 18, 2024

Realistically.. wouldnt someone that uses the template project want to use either sbt or mill and only rarely both? So this would only complicate things.

IMHO there is already a LOT to figure out when starting with scala and spinal, adding another file does not help much.

Yes, it could be a backup example only.

The history is that in SpinalHDL/SpinalHDL mill is imported, but the existing version control is to use a symlink for version.sc, which is too complicate to use under Windows.

I just find the solution, and think might ease the version bump each time spinalhdl's new version release.

@Dolu1990
Copy link
Member

I agree that we should keep it as simple as possible as this is a template.
Maybe we can merge this PR in a separated branch ?

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.

3 participants