Public bug reported:

Description:
I am reporting a packaging bug in the visp package (version 3.7.0-10) currently 
in Ubuntu 26.04.

The issue lies in the override_dh_auto_install target in debian/rules.
Currently, the architecture-dependent build (arch build) is installed
before the architecture-independent documentation build (doc build). As
a result, when the doc build runs, it overwrites critical configuration
files in the install directory with files generated during the doc
build, which lacks support for 3rd-party dependencies (OpenCV, X11,
etc.).

Impact:
Because of this, libvisp-dev installs incorrect configuration files 
(VISPConfig.cmake, visp.pc, etc.) that lack necessary 3rd-party integration. 
This breaks downstream projects relying on these configuration files to locate 
ViSP dependencies.

Proposed Fix:
The order of operations in debian/rules should be swapped so that the 
architecture-dependent build runs last, ensuring it correctly populates the 
final directory structure.

Here is the recommended change in debian rules:

override_dh_auto_install:
        if [ -z "$(WITHOUT_DOC)" ]; then \
                if [ -d "$(BUILDDIR_DOC)" ]; then \
                        dh_auto_install --indep 
--builddirectory=$(BUILDDIR_DOC); \
                fi; \
        fi
        if [ -d "$(BUILDDIR)" ]; then \
                dh_auto_install --arch --builddirectory=$(BUILDDIR); \
        fi

Additional Information:
This mirrors the fix required to ensure that VISPConfig.cmake, 
VISPConfig-version.cmake, VISPUse.cmake, visp-config, visp.pc, and vpConfig.h 
correctly reflect the full 3rd-party build (OpenCV, X11, etc.) instead of being 
overwritten by the doc build's limited configuration.

Upstream/Debian Status:
This issue has already been identified and addressed in the Debian Science 
repository. The fix, which involves swapping the order of the 
architecture-dependent and architecture-independent builds in debian/rules, has 
been implemented in the following commit:
https://salsa.debian.org/science-team/visp/-/commit/aa12a338

** Affects: visp (Ubuntu)
     Importance: Undecided
         Status: New


** Tags: found-in-26.04 packaging patch

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/2158276

Title:
  libvisp-dev: package files overwritten by doc build in debian/rules

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/visp/+bug/2158276/+subscriptions


-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to