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

describe how to use local spinal clone as dependency #261

Merged
merged 1 commit into from
Jul 19, 2024

Conversation

NikLeberg
Copy link
Contributor

@NikLeberg NikLeberg commented Jul 17, 2024

Lately I have been bitten by countless SCOPE VIOLATION or HIERARCHY VIOLATION errors that I did not know where they came from. It drove me crazy.

Thanks to SpinalHDL/SpinalHDL#1387 I noticed that I simply set up my sbt build wrong. I was using Spinal from a local git clone (actually my fork of it) and didnt notice the missing scalac options. (I kinda dislike sbt, I think it is very complicated and slow...)

Nonetheless, I wanted to spare others from pain and try to document it a bit.

Todo: the mill side of it is missing as I don't use mill. Does someone have a working configuration for it?

@Dolu1990
Copy link
Member

Nice thanks :D

i don't know about Mill either.

@Readon
Copy link
Collaborator

Readon commented Jul 17, 2024

I have written mill support script on Readon/FormalTutorial/build.sc file. Wish that helps.

@NikLeberg NikLeberg force-pushed the how_use_local_spinal_dep branch from a98a0c7 to 001fc21 Compare July 18, 2024 10:55
@NikLeberg
Copy link
Contributor Author

Figured out how to do it with mill.
Mill is quite awesome I have to say.

@Readon
Copy link
Collaborator

Readon commented Jul 18, 2024

I also tried to unify the version info for sbt and mill in one conf file @ SpinalHDL/SpinalTemplateSbt#40

@Dolu1990
Copy link
Member

Thanks ^^

@Dolu1990 Dolu1990 merged commit a0eb055 into SpinalHDL:master Jul 19, 2024
3 checks passed
@NikLeberg NikLeberg deleted the how_use_local_spinal_dep branch July 19, 2024 08:25
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