Skip to content

Remove LoRaWAN license from top-level directory #7468

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

Merged
merged 1 commit into from
Jul 18, 2018

Conversation

geky
Copy link
Contributor

@geky geky commented Jul 10, 2018

Description

After a discussion with @AnttiKauppila and @hasnainvirk, it looks like the BSD license in the top-level of mbed-os is misplaced. It was originally intended to cover contributions to the LoRaWAN directory, however this is currently covered under the BSD license here.

Having the BSD license at the top-level implies that mbed-os is licensed under BSD, which AFAIK mbed-os is still licensed under Apache 2.0.

Pull request type

[ ] Fix
[x] Refactor
[ ] New target
[ ] Feature
[ ] Breaking change

related #6087
cc @AnttiKauppila, @hasnainvirk, @jlovejoy

@cmonr cmonr requested review from jlovejoy, hasnainvirk, antiiv01 and a team July 11, 2018 02:29
@cmonr
Copy link
Contributor

cmonr commented Jul 11, 2018

Looks fine to me, but since this is licencing related, will wait until more ppl comment.

@cmonr
Copy link
Contributor

cmonr commented Jul 18, 2018

/morph build

@mbed-ci
Copy link

mbed-ci commented Jul 18, 2018

Build : SUCCESS

Build number : 2630
Build artifacts/logs : http://mbed-os.s3-website-eu-west-1.amazonaws.com/?prefix=builds/7468/

Triggering tests

/morph test
/morph uvisor-test
/morph export-build
/morph mbed2-build

@mbed-ci
Copy link

mbed-ci commented Jul 18, 2018

@mbed-ci
Copy link

mbed-ci commented Jul 18, 2018

@cmonr cmonr merged commit 3dd7d88 into ARMmbed:master Jul 18, 2018
@0xc0170 0xc0170 removed the needs: CI label Jul 18, 2018
pan- pushed a commit to pan-/mbed that referenced this pull request Aug 22, 2018
Remove LoRaWAN license from top-level directory
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants