MAINT: remove workarounds for bugs fixed in pyproject-metadata 0.8.0

Author: dnicolodi

mesonpy/__init__.py

@@ -234,18 +234,12 @@ class MesonBuilderError(Error):
 
 class Metadata(pyproject_metadata.StandardMetadata):
     def __init__(self, name: str, *args: Any, **kwargs: Any):
-        super().__init__(name, *args, **kwargs)
-        # Local fix for https://github.com/FFY00/python-pyproject-metadata/issues/60
-        self.name = self._validate_name(name)
-
-    @staticmethod
-    def _validate_name(name: str) -> str:
         # See https://packaging.python.org/en/latest/specifications/core-metadata/#name
         if not re.match(r'^([A-Z0-9]|[A-Z0-9][A-Z0-9._-]*[A-Z0-9])$', name, re.IGNORECASE):
             raise pyproject_metadata.ConfigurationError(
                 f'Invalid project name "{name}". A valid name consists only of ASCII letters and '
                 f'numbers, period, underscore and hyphen. It must start and end with a letter or number')
-        return name
+        super().__init__(name, *args, **kwargs)
 
     @classmethod
     def from_pyproject(cls, data: Mapping[str, Any], project_dir: Path) -> Metadata:  # type: ignore[override]
@@ -255,11 +249,6 @@ def from_pyproject(cls, data: Mapping[str, Any], project_dir: Path) -> Metadata:
 
         metadata = super().from_pyproject(data, project_dir)
 
-        # Check for missing version field.
-        if not metadata.version and 'version' not in metadata.dynamic:
-            raise pyproject_metadata.ConfigurationError(
-                'Required "project.version" field is missing and not declared as dynamic')
-
         # Check for unsupported dynamic fields.
         unsupported_dynamic = set(metadata.dynamic) - {'version', }
         if unsupported_dynamic:
@@ -268,17 +257,6 @@ def from_pyproject(cls, data: Mapping[str, Any], project_dir: Path) -> Metadata:
 
         return metadata  # type: ignore[return-value]
 
-    # Local fix for a bug in pyproject-metadata. See
-    # https://github.com/mesonbuild/meson-python/issues/454
-    def _update_dynamic(self, value: Any) -> None:
-        if value and 'version' in self.dynamic:
-            self.dynamic.remove('version')
-
-    @property
-    def canonical_name(self) -> str:
-        # See https://packaging.python.org/en/latest/specifications/name-normalization/#normalization
-        return packaging.utils.canonicalize_name(self.name)
-
     @property
     def distribution_name(self) -> str:
         """Name to be used in wheel and sdist file names."""

tests/packages/full-metadata/pyproject.toml

@@ -13,7 +13,7 @@ description = 'Some package with all of the PEP 621 metadata'
 readme = 'README.md'
 requires-python = '>=3.7'
 license = {file = 'LICENSE'}
-keywords = ['full', 'metadata']
+keywords = ['full', 'metadata', 'keyword with spaces']
 authors = [
   {email = '[email protected]'},
   {name = 'Jane Doe'}

tests/test_metadata.py

@@ -48,5 +48,5 @@ def test_missing_version(package_missing_version):
     pyproject = {'project': {
         'name': 'missing-version',
     }}
-    with pytest.raises(pyproject_metadata.ConfigurationError, match='Required "project.version" field is missing'):
+    with pytest.raises(pyproject_metadata.ConfigurationError, match='Field "project.version" missing'):
         Metadata.from_pyproject(pyproject, pathlib.Path())

tests/test_sdist.py

@@ -35,7 +35,7 @@ def test_pep621(sdist_full_metadata):
         Name: full-metadata
         Version: 1.2.3
         Summary: Some package with all of the PEP 621 metadata
-        Keywords: full metadata
+        Keywords: full,metadata,keyword with spaces
         Home-page: https://example.com
         Author: Jane Doe
         Author-Email: Unknown <[email protected]>