You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2020/08/04 05:38:14 UTC

[GitHub] [incubator-nuttx] masayuki2009 opened a new pull request #1503: Change shes related headers

masayuki2009 opened a new pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503


   ## Summary
   
   - This commit replaces SHES related headers with Apache License 2.0
   
   ## Impact
   
   - No impact
   
   ## Testing
   
   - Build check only
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] justinmclean commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
justinmclean commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-668407426


   HI,
   
   Why are these headers being replaced without a software grant? 
   
   Thanks,
   Justin
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] justinmclean commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
justinmclean commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-668414083


   Hi,
   > I am currently an SHES employee and I already submitted ICLA and I contributed the code.
   > so I thought it's OK to replace the headers.
   > 
   
   It is never OK to replace 3rd party headers without permission of the copyright owner. Sony is most likely the copyright owner here. Do we have documented permission on this mailing list to change the license or headers of those files? About the only case I could think of were it would be OK is if the wrong headers were accidentally put on the files in the first place, but I don’t think that’s the case here.
   
   Thanks,
   justin


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-691927574


   > @masayuki2009 any updates on this?
   
   @v01d 
   
   Legal group in our company is still reviewing SGA.
   So please wait for a while.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] jerpelea commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
jerpelea commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-748853598


   The SGA has been submitted 


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] jerpelea merged pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
jerpelea merged pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-748880955


   > @masayuki2009 and @jerpelea there still has many files with "Sony Semiconductor" or "Sony Home Entertainment" in nuttx and apps folder, should we change their copyright too?
   
   @xiaoxiang781216 
   Yes, we will create new PRs later.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-748877897


   @masayuki2009 and @jerpelea there still has many files with "Sony Semiconductor" or "Sony Home Entertainment" in nuttx and apps folder, should we change their copyright too?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-668418633


   > Hi,
   > I am currently an SHES employee and I already submitted ICLA and I contributed the code. so I thought it's OK to replace the headers.
   > It is never OK to replace 3rd party headers without permission of the copyright owner. Sony is most likely the copyright owner here. Do we have documented permission on this mailing list to change the license or headers of those files? About the only case I could think of were it would be OK is if the wrong headers were accidentally put on the files in the first place, but I don’t think that’s the case here. Thanks, justin
   
   Hi @justinmclean,
   
   Thanks for the comment.
   
   I'm still confused regarding the right process to change the license headers.
   I think I need to send email to dev@nuttx.apache.org to change requests which refer to this PR.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] v01d commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
v01d commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-691682525


   @masayuki2009 any updates on this?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-683774463


   > @masayuki2009 could you ask the lawyer to sign SGA? like we do here:
   > https://lists.apache.org/thread.html/re120dc1cd10822ad5d7d4af594961cd4e80c48d9c32285a2254e3c56%40%3Cdev.nuttx.apache.org%3E
   
   @xiaoxiang781216 
   
   Thanks for the comments.
   We will discuss SGA internally.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-683771460


   @masayuki2009 could you ask the lawyer to sign SGA? like we do here:
   https://lists.apache.org/thread.html/re120dc1cd10822ad5d7d4af594961cd4e80c48d9c32285a2254e3c56%40%3Cdev.nuttx.apache.org%3E


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-nuttx] masayuki2009 commented on pull request #1503: Change shes related headers

Posted by GitBox <gi...@apache.org>.
masayuki2009 commented on pull request #1503:
URL: https://github.com/apache/incubator-nuttx/pull/1503#issuecomment-668409920


   > HI, Why are these headers being replaced without a software grant? Thanks, Justin
   
   Hi, @justinmclean,
   
   I am currently an SHES employee and I already submitted ICLA and I contributed the code.
   so I thought it's OK to replace the headers.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org