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

update license.txt and delete licensed-hardware.txt #355

Merged
merged 4 commits into from
Feb 13, 2024
Merged

Conversation

wickste
Copy link
Contributor

@wickste wickste commented Feb 7, 2024

PR checklist

  • Updated function header with a short description and version number
  • Added test case for bug fix or new feature
  • Validated on real hardware

Copy link

@williamelamie williamelamie left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MIT License update looks good

@jserv
Copy link

jserv commented Feb 12, 2024

At present, README.md claims

License terms for using ThreadX RTOS are defined in the LICENSE.txt file of this repo. Please refer to this file for all definitive licensing information. No additional license fees are required for deploying ThreadX RTOS on hardware defined in the LICENSED-HARDWARE.txt file. If you are using hardware not defined in the LICENSED-HARDWARE.txt file or have licensing questions in general, please contact Microsoft directly at https://aka.ms/azrtos-license.

The above seems to be outdated.

@wickste
Copy link
Contributor Author

wickste commented Feb 12, 2024 via email

Copy link

@billlamiework billlamiework left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to MIT license looks good.

Copy link

@billlamiework billlamiework left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change to MIT license looks good.

@wickste wickste merged commit 0b6f59d into master Feb 13, 2024
1 check passed
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.

5 participants