Your README mentions the MIT license, but the LICENSE file is Apache

rocksavagetech / dynamicfifo

A highly configurable FIFO or FIFO controller with dynamic status flags.

Apache License 2.0

2 stars 1 forks source link

Your README mentions the MIT license, but the LICENSE file is Apache #3

Closed parker-research closed 1 month ago

parker-research commented 3 months ago

Huge fan of this work!

Just wanted to note that you have inconsistent licensing.

Have you considered using a license designed specifically for hardware; for example, the CERN Open Hardware Licence Version 2 - Permissive (CERN OHL v2 Permissive) license?

twsavage59 commented 3 months ago

Yes, I have to fix that. I polled a bunch of people I respect, including our lawyer and the consensus was Apache2. I had planned all along for MIT, but was convinced at the last minute to change. But I will look into that CERN license, I hadn't heard of it!

twsavage59 commented 2 months ago

Fixed, thanks for catching that. Also, there was a mention of dependencies which was also outdated information.

parker-research commented 2 months ago

No worries, happy to help! :)

maliberty commented 1 month ago

close?

twsavage59 commented 1 month ago

Yes, fixed with cdf7f00 commit back in June.

twsavage59 commented 1 month ago

Issue closed.