On Tue, 4 Oct 2022 17:52:05 GMT, Phil Race <[email protected]> wrote:

>> Damon Nguyen has updated the pull request with a new target base due to a 
>> merge or a rebase. The incremental webrev excludes the unrelated changes 
>> brought in by the merge/rebase. The pull request contains two additional 
>> commits since the last revision:
>> 
>>  - Merge branch 'openjdk:master' into 8293672/updateFreeTypeMd
>>  - Update md file
>
> src/java.desktop/share/legal/freetype.md line 542:
> 
>> 540: /*
>> 541:  * Copyright 2000 Computing Research Labs, New Mexico State University
>> 542:  * Copyright 2001-2015
> 
> Why did this notice need to be moved down to the end of the file ?
> It makes the diff look bigger than it actually is and I can't imagine any 
> reason we would need to change the order.

This is due to the addition of the MIT license being treated as a separate 
license from the freetype license. In the modified tool, licenses are separated 
by license type. The additional freetype licenses seems more logically related 
to freetype licenses so it follows the freetype & GPL license. Then, other 
licenses (such as MIT in this case) follow afterwards.

This can be manually modified to maintain the old order but I believed it made 
more sense this way in the tool since "variants" of the primary license 
(freetype) would follow the primary license.

-------------

PR: https://git.openjdk.org/jdk/pull/10561

Reply via email to