-
Notifications
You must be signed in to change notification settings - Fork 2
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
Add List of Projects #6
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- Chisel Testers (https://github.com/freechipsproject/chisel-testers) | ||
- Treadle (https://github.com/freechipsproject/treadle) | ||
- Bootcamp (https://github.com/freechipsproject/chisel-bootcamp) | ||
- Template (https://github.com/freechipsproject/chisel-template) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this one already public domain?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's currently unlicensed but it was included in the relicensing emails so will be relicensed to Apache 2.0.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or rather I mean it should be public domain by principle, not by accident.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The template should not be Apache 2.0 but public domain to facilitate adoption.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, good point. It should be left that way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Public domain is not a thing in many jurisdictions. It should always have a fallback license like Apache-2.0
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It already has the http://unlicense.org/ applied to it which is essentially a 'legal' version of public domain. Apache 2.0 is NOT public domain.
What about ip-contributions? |
Great point, @chick we may need to check about relicensing this one. We also ought to own the website repository: https://github.com/freechipsproject/www.chisel-lang.org Since this PR is approved I think a follow on PR would be appropriate |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This list is good, but raises a question to me. I understand why it might not make sense to put firrtl-interpreter not being on this list, but it's also a dependency of most of the things on the list. Maybe there should be a deprecated project list, and we can start with firrtl-interpreter on it?
I second @grebe idea, let's deprecate firrtl-interpreter for 3.4 |
Alright so changes to be made:
Assuming this sounds good I'll rebase with these changes and dismiss all reviews. |
This is the proposed initial list of projects in the Chisel Working Group. Each project has been approved for relicensing to Apache 2.0 (FIRRTL and Treadle already have been). They will later be moved to the CHIPS Alliance Github project: https://github.com/chipsalliance/.