ThrowTheSwitch / CException

Lightweight exception implementation for C
http://throwtheswitch.org
MIT License
326 stars 66 forks source link

CException is missing a license file #29

Closed swaldhoer closed 5 months ago

swaldhoer commented 8 months ago

All other ThrowTheSwitch projects have a dedicate license file

The license for CException is only stated at the end of the README file of the project.

Would it be possible to add a license file to the repository? I think it make sense as it makes the information better accessible and in a more common way.

mvandervoord commented 8 months ago

Great point. Thanks for noticing!

swaldhoer commented 8 months ago

@mvandervoord Would you be willing to also include an SPDX identifier to the sources? This would make the creation of an SBOM way simpler and is not really much effort to do that on source side (and on consumer side it creates effort).

Adding SPDX identifiers is simple for licenses like MIT, see https://spdx.dev/learn/handling-license-info.

mvandervoord commented 8 months ago

@swaldhoer -- I've corrected the license handling. It's now consistent with the other projects.

I looked into the SPDX identifiers. It's a simple process and I definitely see a place for it. The only downside is that the required Linux Foundation membership is rather costly. I'll leave this issue open and can hopefully do this in the future.

swaldhoer commented 8 months ago

@swaldhoer -- I've corrected the license handling. It's now consistent with the other projects.

perfect, thank you!

I looked into the SPDX identifiers. It's a simple process and I definitely see a place for it. The only downside is that the required Linux Foundation membership is rather costly.

Sorry, I don't understand this point. What is the relation between adding a SPDX identifier to the source code and a required Linux Foundation membership. Do you mean you need to be a member of the Linux Foundation in order to be allowed to add a SPD identifer to the source code? If so, could you point me to that source? I have never read or heard of that.

mvandervoord commented 8 months ago

My mistake. I see the important bits of information now. When I first went to the site, it had a popup talking about membership and I assumed that was necessary. Thanks for the followup!

swaldhoer commented 5 months ago

Fixed in 18d020cec7daca2c2af7c48555aa3df2e077591c and df9af17159b3807b81ee7b0387af8075a535391c.