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

add st license #900

Closed
wants to merge 5 commits into from
Closed

add st license #900

wants to merge 5 commits into from

Conversation

inishchith
Copy link
Contributor

@inishchith inishchith commented Jan 27, 2018

PR for issue : #662
Signed-off-by: inishchith inishchith@gmail.com

@pombredanne need your review . I figured out that it's a Permissive Free License .

@codecov
Copy link

codecov bot commented Jan 27, 2018

Codecov Report

Merging #900 into develop will decrease coverage by 0.24%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop     #900      +/-   ##
===========================================
- Coverage    79.07%   78.83%   -0.25%     
===========================================
  Files           93       93              
  Lines        11456    11456              
===========================================
- Hits          9059     9031      -28     
- Misses        2397     2425      +28
Impacted Files Coverage Δ
src/scancode/api.py 59.28% <0%> (-16.77%) ⬇️
src/scancode/cli.py 93.49% <0%> (ø) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1009508...d0bead6. Read the comment docs.

Signed-off-by: inishchith <inishchith@gmail.com>
Copy link
Member

@pombredanne pombredanne left a comment

Choose a reason for hiding this comment

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

Looking good: thank you ++
I made a few comments for your consideration

@@ -0,0 +1,34 @@
Copyright (c) 2017 STMicroelectronics – All rights reserved
Copy link
Member

Choose a reason for hiding this comment

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

Can you remove this copyright line from the reference license text? We want to avoid mixing copyrights in licenses in most cases.

SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR
OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY
OF SUCH DAMAGE.
Copy link
Member

Choose a reason for hiding this comment

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

Please add a trailing new line

owner: STMicroelectronics International N.V.
homepage_url: http://www.st.com/content/st_com/en.html
text_urls:
- https://github.com/STMicroelectronics-CentralLabs/BlueSTSDK_GUI_iOS/blob/master/LICENSE
Copy link
Member

Choose a reason for hiding this comment

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

Can you use 4 spaces for indent there?

@inishchith
Copy link
Contributor Author

@pombredanne i've made the required changes .

@pombredanne
Copy link
Member

@inishchith kind reminder that I need your "Signed-off-by: inishchith inishchith@gmail.com" line on every commit, and you could also come with more explicit commit messages (or squash them all in one?)
Thanks!

@inishchith
Copy link
Contributor Author

@pombredanne Sorry , this time i had issues with my remote branch so had no other way :)

@pombredanne
Copy link
Member

Fair enough. I take care of squashing this alright!

key: st-microelectronics
short_name: STMicroelectronics CentralLabs License
name: STMicroelectronics CentralLabs License
category: Permissive Free
Copy link
Member

Choose a reason for hiding this comment

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

Since it has some serious restrictions on redistribution of graphics, I wonder if the category should be something else. @DennisClark what do you think?

Copy link
Member

Choose a reason for hiding this comment

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

I am making this free restricted instead.... FWIW there is no such "Permissive Free" category anyway

@pombredanne
Copy link
Member

Thanks! merged with c8c0451 and 59d637a

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.

2 participants