add support for PEP 639 License Clarity

Author: radoering

docs/libraries.md

@@ -62,16 +62,19 @@ poetry build
 This command will package your library in two different formats: `sdist` which is
 the source format, and `wheel` which is a `compiled` package.
 
-Poetry will automatically include some metadata files when building a package. When building
-a `wheel`, the following files are included in the `.dist-info` directory:
-- `LICENSE`
-- `LICENSE.*`
-- `COPYING`
-- `COPYING.*`
-- `LICENSES/**`
-
-When building an `sdist`, the following files will be included in the root folder:
-  - `LICENSE*`
+Poetry will automatically include some license-related files when building a package -
+in the `.dist-info/licenses` directory when building a `wheel`,
+and in the root folder when building an `sdist`:
+- `LICENSE*`
+- `LICENCE*`
+- `COPYING*`
+- `AUTHORS*`
+- `NOTICE*`
+- `LICENSES/**/*`
+
+You can override this behavior by specifying
+[`license-files`]({{< relref "pyproject/#license-files" >}})
+in the `pyproject.toml` file.
 
 ### Alternative build backends
 

docs/pyproject.md

@@ -71,7 +71,8 @@ description = "A short description of the package."
 
 ### license
 
-The license of the package.
+An [SPDX expression](https://packaging.python.org/en/latest/glossary/#term-License-Expression)
+representing the license of the package.
 
 The recommended notation for the most common licenses is (alphabetical):
 
@@ -93,20 +94,43 @@ Optional, but it is highly recommended to supply this.
 More identifiers are listed at the [SPDX Open Source License Registry](https://spdx.org/licenses/).
 
 ```toml
-license = { text = "MIT" }
+license = "MIT"
 ```
-{{% note %}}
-If your project is proprietary and does not use a specific license, you can set this value as `Proprietary`.
-{{% /note %}}
 
-You can also specify a license file. However, when doing this, the complete license text
-will be added to the metadata and the License classifier cannot be determined
-automatically so that you have to add it manually.
+{{% warning %}}
+Specifying license as a table, e.g. `{ text = "MIT" }` is deprecated.
+If you used to specify a license file, e.g. `{ file = "LICENSE" }`,
+use `license-files` instead.
+{{% /warning %}}
+
+### license-files
+
+A list of glob patterns that match the license files of the package
+relative to the root of the project source tree.
 
 ```toml
-license = { file = "LICENSE" }
+[project]
+# ...
+license-files = [
+    "*-LICENSE",
+    "CONTRIBUTORS",
+    "MY-SPECIAL-LICENSE-DIR/**/*"
+]
 ```
 
+Per default, Poetry will include the following files:
+- `LICENSE*`
+- `LICENCE*`
+- `COPYING*`
+- `AUTHORS*`
+- `NOTICE*`
+- `LICENSES/**/*`
+
+{{% note %}}
+The default applies only if the `license-files` field is not specified.
+Specifying an empty list results in no license files being included.
+{{% /note %}}
+
 ### readme
 
 A path to the README file or the content.
@@ -198,7 +222,7 @@ classifiers = [
 ```
 
 {{% warning %}}
-Note that suitable classifiers based on your `python` requirement and `license`
+Note that suitable classifiers based on your `python` requirement
 are **not** automatically added for you if you define classifiers statically
 in the `project` section.
 
@@ -419,9 +443,6 @@ More identifiers are listed at the [SPDX Open Source License Registry](https://s
 ```toml
 license = "MIT"
 ```
-{{% note %}}
-If your project is proprietary and does not use a specific licence, you can set this value as `Proprietary`.
-{{% /note %}}
 
 ### authors
 
@@ -545,11 +566,8 @@ classifiers = [
 Note that Python classifiers are automatically added for you
 and are determined by your `python` requirement.
 
-The `license` property will also set the License classifier automatically.
-
 If you do not want Poetry to automatically add suitable classifiers
-based on the `python` requirement and `license` property,
-use `project.classifiers` instead of this setting.
+based on the `python` requirement, use `project.classifiers` instead of this setting.
 {{% /note %}}
 
 ### packages

poetry.lock

@@ -4,7 +4,7 @@ version = "2.2.0.dev0"
 description = "Python dependency management and packaging made easy."
 requires-python = ">=3.9,<4.0"
 dependencies = [
-    "poetry-core @ git+https://github.com/python-poetry/poetry-core.git",
+    "poetry-core @ git+https://github.com/radoering/poetry-core.git@pep639",
     "build (>=1.2.1,<2.0.0)",
     "cachecontrol[filecache] (>=0.14.0,<0.15.0)",
     "cleo (>=2.1.0,<3.0.0)",
@@ -16,7 +16,7 @@ dependencies = [
     "installer (>=0.7.0,<0.8.0)",
     "keyring (>=25.1.0,<26.0.0)",
     # packaging uses calver, so version is unclamped
-    "packaging (>=24.0)",
+    "packaging (>=24.2)",  # PEP 639 support was added in 24.2
     "pkginfo (>=1.12,<2.0)",
     "platformdirs (>=3.0.0,<5)",
     "pyproject-hooks (>=1.0.0,<2.0.0)",
@@ -45,7 +45,7 @@ maintainers = [
     { name = "finswimmer", email = "finswimmer77@gmail.com" },
     { name = "Bartosz Sokorski", email = "b.sokorski@gmail.com" },
 ]
-license = { text = "MIT" }
+license = "MIT"
 readme = "README.md"
 keywords = ["packaging", "dependency", "poetry"]
 # classifieres is dynamic because we want to create Python classifiers automatically

pyproject.toml

@@ -173,6 +173,8 @@ def test_check_invalid(
 Error: Invalid source "not-exists" referenced in dependencies.
 Error: Invalid source "not-exists2" referenced in dependencies.
 Error: poetry.lock was not found.
+Warning: [project.license] is not a valid SPDX identifier.\
+ This is deprecated and will raise an error in the future.
 Warning: A wildcard Python dependency is ambiguous.\
  Consider specifying a more explicit one.
 Warning: The "pendulum" dependency specifies the "allows-prereleases" property,\

tests/console/commands/test_check.py

@@ -1,7 +1,7 @@
 [project]
 name = "invalid"
 version = "1.0.0"
-license = { text = "INVALID" }
+license = "INVALID"
 classifiers = [
     "Environment :: Console",
     "Intended Audience :: Clowns",

tests/fixtures/invalid_pyproject/pyproject.toml

@@ -5,7 +5,7 @@ description = "Some description."
 authors = [
     { name = "Sébastien Eustace", email = "sebastien@eustace.io" }
 ]
-license = { text = "MIT" }
+license = "MIT"
 readme = "README.rst"
 keywords = ["packaging", "dependency", "poetry"]
 dynamic = [ "classifiers", "dependencies", "requires-python" ]

tests/fixtures/simple_project/pyproject.toml

@@ -192,7 +192,11 @@ def test_builder_installs_proper_files_for_standard_packages(
 ==========
 
 """
-    assert metadata == dist_info.joinpath("METADATA").read_text(encoding="utf-8")
+    if project == "simple_project":
+        metadata = metadata.replace("License:", "License-Expression:").replace(
+            "Classifier: License :: OSI Approved :: MIT License\n", ""
+        )
+    assert dist_info.joinpath("METADATA").read_text(encoding="utf-8") == metadata
 
     with open(dist_info.joinpath("RECORD"), encoding="utf-8", newline="") as f:
         reader = csv.reader(f)
@@ -220,7 +224,7 @@ def test_builder_installs_proper_files_for_standard_packages(
     sys.exit(baz.boom.bim())
 """
 
-    assert baz_script == tmp_venv._bin_dir.joinpath("baz").read_text(encoding="utf-8")
+    assert tmp_venv._bin_dir.joinpath("baz").read_text(encoding="utf-8") == baz_script
 
     foo_script = f"""\
 #!{tmp_venv.python}
@@ -231,7 +235,7 @@ def test_builder_installs_proper_files_for_standard_packages(
     sys.exit(bar())
 """
 
-    assert foo_script == tmp_venv._bin_dir.joinpath("foo").read_text(encoding="utf-8")
+    assert tmp_venv._bin_dir.joinpath("foo").read_text(encoding="utf-8") == foo_script
 
     fox_script = f"""\
 #!{tmp_venv.python}
@@ -242,7 +246,7 @@ def test_builder_installs_proper_files_for_standard_packages(
     sys.exit(bar.baz())
 """
 
-    assert fox_script == tmp_venv._bin_dir.joinpath("fox").read_text(encoding="utf-8")
+    assert tmp_venv._bin_dir.joinpath("fox").read_text(encoding="utf-8") == fox_script
 
 
 def test_builder_falls_back_on_setup_and_pip_for_packages_with_build_scripts(