[PATCH] D66390: [PATCH] [llvm] [Support] Update license on ConvertUTF.* per upstream license change

Thu Jun 2 13:21:49 PDT 2022

jyknight added a comment.
Herald added a project: All.

Indeed, legal issues are complicated.

Looking into this today, I'm personally fairly confident this code is indeed re-licensed and that, therefore, this change is OK. I say that based on analysis as follows:

1. https://web.archive.org/web/20080831201118/http://www.unicode.org/copyright.html (archived 2008-08-31) has the new license text in place, and says it applies to all source code distributed under http://www.unicode.org/Public/.
2. You can find the ConvertUTF.c and ConvertUTF.h files at https://web.archive.org/web/20080913080605/http://www.unicode.org:80/Public/PROGRAMS/CVTUTF/ (archive link as of 2008-09-13, but file content unchanged since 2004). That is: they were being distributed, then, under this "new" license. Note that the files were subsequently deleted, in 2009, due to being buggy and unmaintained -- so you cannot find them on unicode.org today.
3. LLVM's initial commit of these files (https://github.com/llvm/llvm-project/commit/29cae66bde8b9d1694d85e6fa43fcf5a3ab46e2d) contains files derived -- with only LLVM-side changes -- from the version of the software distributed in that link.

I'm not giving an "Approval" to commit this change yet, though -- I believe LLVM Foundation lawyer will be double-checking this is a correct action, first.

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66390/new/

https://reviews.llvm.org/D66390