This is a follow up for bug 1312481 The following should be made consistent. 1) We have an extra license files for react libs in the directory (e.g. REDUX_LICENSE and REACT_REDUX_LICENSE). But other libs in the directory don't do that. 2) Some files in the directory (e.g. REACT_VIRTUALIZED_UPGRADING file) doesn't have Mozilla license header at the top. Other files have that. Honza

I added the Mozilla license to the *_UPGRADING files and removed the duplicate LICENSE file.

Comment on attachment 8808158 [details] [diff] [review] Bug1314539.patch - rev1 Review of attachment 8808158 [details] [diff] [review]: ----------------------------------------------------------------- Please keep REACT_REDUX_LICENSE file in. I believe it should be there. It's the same license, but for for different module (React vs. React-redux) Honza

Gerv, I was quickly looking at: https://www.mozilla.org/en-US/MPL/headers/ and the page talks about (a) new code and (b) test scripts and other short code examples. They both have slightly different license headers. What license header should be used for a file that contains mostly information about how to upgrade things. See e.g. this file: https://dxr.mozilla.org/mozilla-central/source/devtools/client/shared/vendor/REACT_UPGRADING The one for 'new code' or 'test scripts and other short code examples' ? Honza

(In reply to Jan Honza Odvarko [:Honza] from comment #3) > The one for 'new code' or 'test scripts and other short code examples' ? Anything which is not a "test script and other short code example" falls into the "new code" bucket. Gerv

@Dalimil: the patch looks good, but do not remove the REACT_REDUX_LICENSE file (see also my comment #2) Honza

OK, I put the license file back.

Comment on attachment 8810881 [details] [diff] [review] Bug1314539.patch - rev2 Review of attachment 8810881 [details] [diff] [review]: ----------------------------------------------------------------- Thanks! Honza

Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/60bb7c019127 Unify license files and Mozilla license headers in vendor directory. r=odvarko