Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ConvertUTF.cpp is shipped with a non "University of Illinois Open Source License" compatible license #32309

Closed
sylvestre opened this issue May 8, 2017 · 7 comments
Labels
bugzilla Issues migrated from bugzilla llvm:core

Comments

@sylvestre
Copy link
Collaborator

Bugzilla Link 32962
Version trunk
OS Linux
CC @dwblaikie,@DimitryAndric,@jlebar,@MatzeB

Extended Description

Detected by lintian, the Debian static analyzer:

lib/Support/ConvertUTF.cpp include material under a non-free
license from Unicode Inc. Therefore, it is not possible to ship this in
main or contrib.

This license does not grant any permission to modify the files (thus
failing DFSG#3). Moreover, the license grant to attempt to restrict use
to "products supporting the Unicode Standard" (thus failing DFSG#6).

In this case a solution is to use libicu and to remove this code by
repacking.

Refer to https://bugs.debian.org/823100 for details.

And https://en.wikipedia.org/wiki/Debian_Free_Software_Guidelines for Debian information.

@DimitryAndric
Copy link
Collaborator

Interesting... This file seems to have been added a long time ago, here:

https://reviews.llvm.org/rL68208

Originally it was in cfe/trunk/lib/Basic, then it got moved around and edited a bit, but the original license was always in there.

@DimitryAndric
Copy link
Collaborator

Also note that there also appear to be older IBM versions of this file, with a completely different license, written by the original author, Mark E. Davis. I'm not sure how the Unicode org managed to change the license. :)

@sylvestre
Copy link
Collaborator Author

Looks like there is a license compatible version in breakpad:
https://raw.githubusercontent.com/google/breakpad/master/src/common/convert_UTF.h

I am trying to fix that here:
https://reviews.llvm.org/D64740

@llvmbot llvmbot transferred this issue from llvm/llvm-bugzilla-archive Dec 10, 2021
@sylvestre
Copy link
Collaborator Author

well done :)

@bkmgit
Copy link

bkmgit commented Sep 18, 2022

@jyknight Discussion at AmokHuginnsson/replxx#12 indicates these can be replaced by https://github.com/nemtrif/utfcpp

@jyknight
Copy link
Member

jyknight commented Sep 19, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugzilla Issues migrated from bugzilla llvm:core
Projects
None yet
Development

No branches or pull requests

5 participants