Hi Fernando,

Am Tue, Aug 25, 2026 at 11:54:23AM +0200 schrieb Fernando Hueso González:
> no worries at all, no hurries.

:-)
I know this.  However, I follow a "try to serve newcomers first" rule.
I hope this will help in the long term to have higher chances for
onboarding newcomers.

> > To approach this, you need to patch the CMakeLists.txt file to create
> > a SOVERSION.  (We have some examples in Debian Med.)
> Thanks, I've just followed civetweb's approach and the soversion patch seems
> to work.

Good.

I've renamed library package to fit SONAME and moved shared lib to
library package.

> > Apropos patching.  Your debian/patches/cmake.diff patch is quite long,
> > contains files in .github which most probably are not needed for the
> > packaging.
> Indeed, done.

I've noticed the Description field and you also removed some .git* files.
However, I see more than just CMakeLists.txt files in

$ grep ^+++ debian/patches/cmake.diff 
+++ b/CMakeLists.txt
+++ b/extensions/PythonQt_QtAll/CMakeLists.txt
+++ b/extensions/PythonQt_QtAll/PythonQt_QtAll.cpp
+++ b/extensions/PythonQt_QtAll/PythonQt_QtAll.h
+++ b/generator/CMakeLists.txt
+++ b/src/CMakeLists.txt
+++ b/src/PythonQtConfigure.h.in
+++ b/src/PythonQtPythonInclude.h
+++ b/src/generate_configure_h.py
+++ b/src/src.pro
+++ b/tests/PythonQtTestMain.cpp

> > document the purpose in the "Description:" DEP3 field (I've just added
> > Author and Last-Update).
> Fixed!

I wonder whether you might want to split this into logical chunks.

> > This might be overridden in a lintian-overrides line.
> Done.
> > See above.  Moreover you should use multiarch installation
> > (/usr/lib/x86_64-linux-gnu/ and the according dir for other archs).
> > CMake has some command for this (which I do not know by heart -
> > just let me know in case you might struggle finding it.
> 
> I am a bit lost here. Do you mean adding to debian/rules when configuring
> something along:
> 
> |export DEB_HOST_MULTIARCH ?= $(shell dpkg-architecture
> -qDEB_HOST_MULTIARCH) ....|
> 
> |-DCMAKE_INSTALL_LIBDIR=lib/$(DEB_HOST_MULTIARCH)|
> 
> Or rather using a post-fix as done in the civetweb debian repo?

I bet there is a CMake option / configuration for this.  I'm not a CMake
expert but I'm sure I have seen this and you can find it in
codesearch.debian.net.

Apropos override_dh_auto_configure - this looks like a very manual
step I've never seen in other packages.  Are you sure you can't go
with the default dh_auto_configure + some options?
 
> https://salsa.debian.org/med-team/civetweb/-/commit/803d31208c73f414929a2c0360692750c3f4589
> 
> > You can tell Salsa to not fail on blhc failures:
> 
> I tried via 
> https://salsa.debian.org/python-team/packages/pythonqt/-/commit/0ad88987fe0ff1f9350dacaa533bead155afaa46,
> but it seems I did not do it correctly, in case you could take a look.

You need to set the configure file in
   
https://salsa.debian.org/python-team/packages/pythonqt/-/settings/ci_cd#js-general-pipeline-settings
It was formerly set to
   recipes/debian.yml@salsa-ci-team/pipeline
so your change was not used.
 
> Have fun at the conference! Thanks for the help, regards,

Thank you
    Andreas.

-- 
https://fam-tille.de

Reply via email to