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

Remove the words "open source" from the description of Coldcard #4111

Conversation

ai-oleynikov
Copy link
Contributor

The "Commons Clause" License, used by Coldcard's firmware, doesn't meet the criteria of an open-source license. Even the authors of the license themselves don't claim it to be open source.

https://commonsclause.com/#faq

Is this “Open Source”?
No.

So we shouldn't call Coldcard open source.

@crwatkins
Copy link
Contributor

I agree in principal. The criteria for listing include “Source code is public” and that has been the recent emphasis. @nvk Do you have a recommended alternative description for Coldcard here?

@nvk
Copy link
Contributor

nvk commented Oct 11, 2023

If it's just a question of words. "Verifiable Source Code" is acceptable. As long as the indicating icon stays green.

This is not a reference to the person crating this PR, but I find the motivation for this change often from bad faith as ppl try to paint the device as if the source is not available. Let a alone the endless wars over this term being unsettled.

@crwatkins
Copy link
Contributor

@ai-oleynikov How about this using the words suggest by @nvk?

Coldcard is an easy to use, ultra-secure and affordable hardware wallet with verifiable source code that is easy to back up via an encrypted microSD card.

@ai-oleynikov
Copy link
Contributor Author

@crwatkins fine by me

@crwatkins
Copy link
Contributor

crwatkins commented Apr 16, 2024

@ai-oleynikov Would you like to update the PR? You don’t have to make changes to all the language files as the Transfix team can do that.

@crwatkins
Copy link
Contributor

Hey @ai-oleynikov Would you like to update your PR?

@crwatkins
Copy link
Contributor

crwatkins commented Jun 21, 2024

I've created a new PR #4235 to implement the change discussed above.

Thanks to @ai-oleynikov for proposing this.

@crwatkins crwatkins closed this Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants