d/control: fix lua dependencies for librgw2

See inner commit message for details.

Semi-related: ceph-common, which pulls in python3-rgw in the first
place, should really split up in a handful of packages, which would
avoid this issue for us too, as we do not require RGW tooling by
default.

Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
This commit is contained in:
Thomas Lamprecht 2024-11-06 16:26:27 +01:00
parent 6179f7fae5
commit 7c1f1263cd
2 changed files with 49 additions and 0 deletions

View File

@ -0,0 +1,48 @@
From 0000000000000000000000000000000000000000 Mon Sep 17 00:00:00 2001
From: Thomas Lamprecht <t.lamprecht@proxmox.com>
Date: Wed, 6 Nov 2024 15:06:09 +0100
Subject: [PATCH] d/control: fix lua dependencies for librgw2
One can attach lua scripts as sort of hooks to implement dynamic
checks or transformations of RGW requests since Ceph Pacific. A lua
library is now required for base support and optionally one can use
the luarocks deployment and management system for Lua modules to use
more advanced scripts/modules.
With commit 46500cace6c ("rgw/test/lua: add lua integration tests
suite") the dependency relations where cleaned up, as the respective
entries where missing from debian/control.
But that commit is pulling in much more than required due to adding
the devel package `liblua5.3-dev` instead of the library-only
`liblua5.3-0` one, and having `luarocks` as hard dependency compared
to an optional Suggests. Fixing that avoids pulling in a whole
build/compiler/autotools/... stack just when one wants to use librgw2
or python3-rgw for simple RGW requests.
This is a result of some discussion on the original PR [0].
[0]: https://github.com/ceph/ceph/pull/52931#issuecomment-2441253989
Signed-off-by: Thomas Lamprecht <t.lamprecht@proxmox.com>
---
debian/control | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/debian/control b/debian/control
index 041dfcc7980..0a519635f47 100644
--- a/debian/control
+++ b/debian/control
@@ -921,10 +921,10 @@ Package: librgw2
Architecture: linux-any
Section: libs
Depends: librados2 (= ${binary:Version}),
+ liblua5.3-0,
${misc:Depends},
${shlibs:Depends},
- liblua5.3-dev,
- luarocks,
+Suggests: luarocks,
Description: RADOS Gateway client library
RADOS is a distributed object store used by the Ceph distributed
storage system. This package provides a REST gateway to the

View File

@ -13,3 +13,4 @@
0013-debian-recursively-adjust-permissions-of-var-lib-cep.patch 0013-debian-recursively-adjust-permissions-of-var-lib-cep.patch
0014-ceph-crash-change-order-of-client-names.patch 0014-ceph-crash-change-order-of-client-names.patch
0017-debian-pkg-record-python3-packaging-dependency-for-c.patch 0017-debian-pkg-record-python3-packaging-dependency-for-c.patch
0018-d-control-fix-lua-dependencies-for-librgw2.patch