From 6afa196ca0cb1604875847b1b84fa64896a06f6e Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Micha=C5=82=20G=C3=B3rny?= Date: Fri, 18 Oct 2024 15:13:06 +0200 Subject: [PATCH] dev-python/python-multipart: Hard-rename to python_multipart MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Hard-rename the Python package to `python_multipart`, to avoid conflicts with the `multipart` PyPI package. Both `multipart` and `python-multipart` packages install using `multipart` import name, therefore both cannot be installed at the same time. Historically, we have included only the latter in Gentoo, since it was pulled in as a dependency of dev-python/starlette. However, the former was recently made the recommended replacement for the deprecated and removed `cgi` standard library module, therefore other packages started depending on it. Given that both packages intend to be maintained throughout the foreseeable future, it seems that the best workaround is to rename one of them. In this case, `python-multipart` uses an import name that does not match the package and seems to be have fewer reverse dependencies at this time, so rename it. There is already an open pull request upstream to do exactly that, so reuse parts of it. That said, since we can simply patch reverse dependencies, we do not need the compatibility layer (and probably do not want it, as it could yield confusing errors if the wrong package is installed). Bug: https://github.com/pypa/packaging-problems/issues/818 Pull-Request: https://github.com/Kludex/python-multipart/pull/166 Signed-off-by: Michał Górny --- .../python-multipart-0.0.12-rename.patch | 54 +++++++++++++++++++ .../python-multipart-0.0.12-r100.ebuild | 47 ++++++++++++++++ 2 files changed, 101 insertions(+) create mode 100644 dev-python/python-multipart/files/python-multipart-0.0.12-rename.patch create mode 100644 dev-python/python-multipart/python-multipart-0.0.12-r100.ebuild diff --git a/dev-python/python-multipart/files/python-multipart-0.0.12-rename.patch b/dev-python/python-multipart/files/python-multipart-0.0.12-rename.patch new file mode 100644 index 0000000000000..cee51b2df1c81 --- /dev/null +++ b/dev-python/python-multipart/files/python-multipart-0.0.12-rename.patch @@ -0,0 +1,54 @@ +From d6e30eb0269fa04d4a16133bd94405f10240aeb0 Mon Sep 17 00:00:00 2001 +From: Henry Schreiner +Date: Fri, 11 Oct 2024 17:11:21 -0400 +Subject: [PATCH 1/2] refactor: rename to python_multipart + +Signed-off-by: Henry Schreiner + +diff --git a/pyproject.toml b/pyproject.toml +index fb03f83..1a81077 100644 +--- a/pyproject.toml ++++ b/pyproject.toml +@@ -62,13 +65,10 @@ Changelog = "https://github.com/Kludex/python-multipart/blob/master/CHANGELOG.md + Source = "https://github.com/Kludex/python-multipart" + + [tool.hatch.version] +-path = "multipart/__init__.py" +- +-[tool.hatch.build.targets.wheel] +-packages = ["multipart"] ++path = "python_multipart/__init__.py" + + [tool.hatch.build.targets.sdist] +-include = ["/multipart", "/tests", "CHANGELOG.md", "LICENSE.txt"] ++include = ["/python_multipart", "/tests", "CHANGELOG.md", "LICENSE.txt", "_python_multipart.pth", "_python_multipart_loader.py"] + + [tool.mypy] + strict = true +diff --git a/tests/test_multipart.py b/tests/test_multipart.py +index b824f19..f5f8e7e 100644 +--- a/tests/test_multipart.py ++++ b/tests/test_multipart.py +@@ -11,9 +11,9 @@ + + import yaml + +-from multipart.decoders import Base64Decoder, QuotedPrintableDecoder +-from multipart.exceptions import DecodeError, FileError, FormParserError, MultipartParseError, QuerystringParseError +-from multipart.multipart import ( ++from python_multipart.decoders import Base64Decoder, QuotedPrintableDecoder ++from python_multipart.exceptions import DecodeError, FileError, FormParserError, MultipartParseError, QuerystringParseError ++from python_multipart.multipart import ( + BaseParser, + Field, + File, +@@ -31,7 +31,7 @@ + if TYPE_CHECKING: + from typing import Any, Iterator, TypedDict + +- from multipart.multipart import FieldProtocol, FileConfig, FileProtocol ++ from python_multipart.multipart import FieldProtocol, FileConfig, FileProtocol + + class TestParams(TypedDict): + name: str + diff --git a/dev-python/python-multipart/python-multipart-0.0.12-r100.ebuild b/dev-python/python-multipart/python-multipart-0.0.12-r100.ebuild new file mode 100644 index 0000000000000..ea54218ea5645 --- /dev/null +++ b/dev-python/python-multipart/python-multipart-0.0.12-r100.ebuild @@ -0,0 +1,47 @@ +# Copyright 2022-2024 Gentoo Authors +# Distributed under the terms of the GNU General Public License v2 + +EAPI=8 + +DISTUTILS_USE_PEP517=hatchling +PYTHON_COMPAT=( pypy3 python3_{10..13} ) + +inherit distutils-r1 + +DESCRIPTION="A streaming multipart parser for Python" +HOMEPAGE=" + https://github.com/Kludex/python-multipart/ + https://pypi.org/project/python-multipart/ +" +SRC_URI=" + https://github.com/Kludex/python-multipart/archive/${PV}.tar.gz + -> ${P}.gh.tar.gz +" + +LICENSE="Apache-2.0" +SLOT="0" +KEYWORDS="~amd64 ~arm ~arm64 ~hppa ~loong ~ppc ~ppc64 ~riscv ~s390 ~sparc ~x86" + +BDEPEND=" + test? ( + dev-python/pyyaml[${PYTHON_USEDEP}] + ) +" + +distutils_enable_tests pytest + +src_prepare() { + local PATCHES=( + # https://github.com/Kludex/python-multipart/pull/166 + "${FILESDIR}/${P}-rename.patch" + ) + + distutils-r1_src_prepare + + mv multipart python_multipart || die +} + +python_test() { + local -x PYTEST_DISABLE_PLUGIN_AUTOLOAD=1 + epytest +}