xiaoxiang781216 commented on issue #528: sched/mqueue: Correct/update license headers. URL: https://github.com/apache/incubator-nuttx/pull/528#issuecomment-597257576 > > Maybe we can reuse and improve @adamfeuer 's tool, @adamfeuer are you interesting with the work? > > Sounds like a good idea to me. But we must be much more careful that last time. I think the tool should preserved the first N lines everything from: > > ``` > /**************************************************************************** > ``` > > To the first occurence of > > ``` > * > ``` > > We need to retain the "ruler" width, the file name as well as any optional descriptioni of the file which may be more than one line. Then retain everything at the end of the license: > > ``` > * > ****************************************************************************/ > ``` > > There are some cases I have seen where people put information in the licsned comment block after the license text proper. That should be retained as well if present. > > Plus, as I mentioned before, we must carefully select the files to operate on so that the contain the name are replaced ("Gregory Nutt", or "Pinecone", or "Sony") and that the the Word Copyright only appears in the file. > > And we must carefully review each file so that we are 100% that the tool did nothing reckless. Yes, we should try to run @adamfeuer 's tool with different cases and verify the result before we really apply the change.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
