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

license issue: restricted unicode license headers #11300

Closed
pgajdos opened this issue May 23, 2023 · 3 comments
Closed

license issue: restricted unicode license headers #11300

pgajdos opened this issue May 23, 2023 · 3 comments

Comments

@pgajdos
Copy link

pgajdos commented May 23, 2023

Description

$ grep -r 'specifically excludes the right'
php-8.1.19/ext/mbstring/tests/data/KSX1001.txt:#       specifically excludes the right to re-distribute this file directly
php-8.1.19/ext/mbstring/tests/data/JISX0212.txt:#	specifically excludes the right to re-distribute this file directly
php-8.1.19/ext/mbstring/tests/data/BIG5.txt:#	specifically excludes the right to re-distribute this file directly
php-8.1.19/ext/mbstring/tests/data/JISX0201.txt:#	specifically excludes the right to re-distribute this file directly
$

The full wording:

#    Date:          08/16/99
#    Authors:       Jungshik Shin at jshin@pantheon.yale.edu
#    General notes: none
#
#       This file is provided as-is by Unicode, Inc. (The Unicode Consortium).
#       No claims are made as to fitness for any particular purpose.  No
#       warranties of any kind are expressed or implied.  The recipient
#       agrees to determine applicability of information provided.  If this
#       file has been provided on magnetic media by Unicode, Inc., the sole
#       remedy for any claim will be exchange of defective media within 90
#       days of receipt.
#
#       Recipient is granted the right to make copies in any form for
#       internal distribution and to freely use the information supplied
#       in the creation of products supporting Unicode.  Unicode, Inc.
#       specifically excludes the right to re-distribute this file directly
#       to third parties or other organizations whether for profit or not.

If I understand correctly, these files should not be distributed as they are.

PHP Version

PHP 8.1.19

Operating System

No response

@alexdowad
Copy link
Contributor

@pgajdos Thanks for pointing this out and sorry for the oversight. Thanks also to @KapitanOczywisty.

If someone wants to open a PR replacing these files (used for test data) with the new ones, that would be great. If no-one does it, I will take it up.

@pgajdos
Copy link
Author

pgajdos commented May 25, 2023

Thanks @KapitanOczywisty, @alexdowad and @andreasstieger for the quick fix.

bmwiedemann pushed a commit to bmwiedemann/openSUSE that referenced this issue Jun 16, 2023
https://build.opensuse.org/request/show/1093234
by user pgajdos + dimstar_suse
- version update to 8.1.20
  * This is a security release.
  * https://www.php.net/ChangeLog-8.php#8.1.20
- force to repack tarball after update
  php/php-src#11300
- session.save_path set to /var/lib/php8/sessions in mod_php8.conf
  and www.conf php-fpm pool example
- modified sources
  % mod_php8.conf
- added sources
  + repack.sh
  + php-unicode-allow-redistribution.patch
nielsdos added a commit to nielsdos/php-src that referenced this issue Jul 1, 2023
nielsdos added a commit that referenced this issue Jul 1, 2023
* PHP-8.1:
  Fix GH-11300: license issue: restricted unicode license headers
nielsdos added a commit that referenced this issue Jul 1, 2023
* PHP-8.2:
  Fix GH-11300: license issue: restricted unicode license headers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants