]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
ceph-volume: add "packaging" to install_requires 59202/head
authorKefu Chai <tchaikov@gmail.com>
Tue, 13 Aug 2024 22:37:57 +0000 (06:37 +0800)
committerKonstantin Shalygin <k0ste@k0ste.ru>
Wed, 4 Sep 2024 08:15:14 +0000 (15:15 +0700)
in 0985e201, "packaging" was introduced as a runtime dependency of
ceph-volume, and `ceph.spec.in` was updated accordingly to note
this new dependency. but the debian packaging was not updated.

in 80edcd40, the missing dependency was added to debian/control as
one of ceph-volume's runtime dependency.

but dh_python3 is able to figure out the dependencies by reading
the egg's metadata of the ceph-volume python module. and as a
python project, ceph-volume is using its `setup.py` for
tracking its dependencies.

so in order to be more consistent, and keep all of its dependencies
in one place, let's move this dependency to setup.py . as the
packagings in both distros are able to figure the dependencies
from egg-info.

see also
- https://manpages.debian.org/testing/dh-python/dh_python3.1.en.html#dependencies
- https://docs.fedoraproject.org/en-US/packaging-guidelines/Python_201x/#_automatically_generated_dependencies

Signed-off-by: Kefu Chai <tchaikov@gmail.com>
(cherry picked from commit 729fd8e25ff2bfbcf99790d6cd08489d1c4e2ede)

ceph.spec.in
debian/control
src/ceph-volume/setup.py

index 5f432779a89b6ac2d5da967f90eb33fabf8e153e..27b534bd5e63c12e203039a779f80c0c90dd5650 100644 (file)
@@ -927,7 +927,6 @@ Requires: parted
 Requires: util-linux
 Requires: xfsprogs
 Requires: python%{python3_pkgversion}-setuptools
-Requires: python%{python3_pkgversion}-packaging
 Requires: python%{python3_pkgversion}-ceph-common = %{_epoch_prefix}%{version}-%{release}
 %description volume
 This package contains a tool to deploy OSD with different devices like
index 7b12163875e63beb913f6b636f4a54ddc99fa617..041dfcc7980e8d6a224dcd22f9f9bed187e2c747 100644 (file)
@@ -432,7 +432,6 @@ Depends: ceph-osd (= ${binary:Version}),
          e2fsprogs,
          lvm2,
          parted,
-         python3-packaging,
          xfsprogs,
          ${misc:Depends},
          ${python3:Depends}
index 44a0d0e468997f67869b29d4fa6cb37c5f7c1e2b..fa49a95cdd057aa03ee4462f75673e9be53ac5ee 100644 (file)
@@ -14,7 +14,10 @@ setup(
     keywords='ceph volume disk devices lvm',
     url="https://github.com/ceph/ceph",
     zip_safe = False,
-    install_requires='ceph',
+    install_requires=[
+        'ceph',
+        'packaging',
+    ],
     dependency_links=[''.join(['file://', os.path.join(os.getcwd(), '../',
                                                        'python-common#egg=ceph-1.0.0')])],
     tests_require=[