brltty ftbfs in jammy

[Matthias Klose]

brltty ftbfs in jammy, new version in Debian unstable

[...]
config.status: executing build-configure commands
configure: WARNING: unrecognized options: --without-viavoice
dh_testdir
/usr/bin/make -C build-py3.10/Bindings/Python
make[1]: Entering directory '/<<BUILDDIR>>/brltty-6.3+dfsg/build-py3.10/Bindings/Python'
make[1]: *** No targets specified and no makefile found. Stop.
make[1]: Leaving directory '/<<BUILDDIR>>/brltty-6.3+dfsg/build-py3.10/Bindings/Python'
make: *** [debian/rules:136: build-py3.10-stamp] Error 2

[Samuel thibault (samuel-thibault)]

Please provide the whole build log.

[Sebastien Bacher (seb128)]

@Samuel, https://salsa.debian.org/a11y-team/brltty/-/blob/master/doc/ChangeLog#L378 states that the option has been removed in 6.0 but it's still used in debian/rules, it is probably newer python versions being stricter with the use of non existing options?

[Samuel thibault (samuel-thibault)]

> it is probably newer python versions being stricter with the use of non existing options?

The --without-viavoice is not passed to the python bindings, so that cannot be the problem.

That said I have now dropped the option from debian/rules, thanks :)

[Sebastien Bacher (seb128)]

Thanks. To reply to the previous question, the build log from the failed build Matthias got with his untested no change rebuild is

https://launchpadlibrarian.net/568862273/buildlog_ubuntu-jammy-amd64.brltty_6.3+dfsg-1ubuntu3_BUILDING.txt.gz

[Sebastien Bacher (seb128)]

> The --without-viavoice is not passed to the python bindings, so that cannot be the problem.

indeed that's not the issue, checking the log it has

configure:9649: WARNING: Python module not found: distutils.sysconfig: <string>:1: DeprecationWarning: The distutils package is deprecated and slated for removal in Python 3.12. Use setuptools or check PEP 632 for potential alternatives
<string>:1: DeprecationWarning: The distutils.sysconfig module is deprecated, use sysconfig instead

which seems to lead to python to be disabled as described on https://github.com/brltty/brltty/pull/329

[Samuel thibault (samuel-thibault)]

Ok, thanks!

The corresponding commit upstream is e6ed2372887ac0106db8854f70e9b2fdcadec74f, attached here, could somebody check that this fixes the build issue on Ubuntu? I'll then integrate the patch in Debian as well.

[Samuel thibault (samuel-thibault)]

Ah, the patch is already in brltty 6.4 actually, so no change on the Debian side actually :)

[Sebastien Bacher (seb128)]

The patch needs some other changes but trying by copying https://raw.githubusercontent.com/brltty/brltty/master/Bindings/Python/bindings.m4 the issue is resolved so Debian should be fine if the new version is out before the python 3.10 transition

[Olivier Tilloy (osomon)]

I just successfully built brltty 6.4 from the debian master branch in a jammy-proposed chroot, so I confirm the problem with Python 3.10 is resolved there.

[Sebastien Bacher (seb128)]

Sorry I didn't realize that the new version was already out and that Debian had it, thanks Olivier for confirming!

[Launchpad Janitor (janitor)]

This bug was fixed in the package brltty - 6.4-2ubuntu1

---------------
brltty (6.4-2ubuntu1) jammy; urgency=medium

  * Resynchronize on Debian (fixes LP: #1950488)
    - Dropped delta:
      + udeb-specific changes, because Ubuntu doesn't build udebs any longer
        (LP: #1884836)
      + Ubuntu-specific entries in deprecated debian/NEWS file
      + JAVA_HOME additions to debian/rules (unneeded)
      + noudeb profiles changes to debian/{control,rules} that were upstreamed
      + changes to debian/{brltty,xbrlapi}.manpages that were upstreamed
      + debian/brltty.sh that replaced /sbin/brltty by /bin/brltty
    - Remaining changes:
      + add brltty-setup
        - debian/brltty-setup
        - debian/brltty.install
        - debian/rules
      + add initramfs integration to run brltty-setup if necessary before
        plymouth starts
        - debian/control
        - debian/initramfs/brltty.sh
        - debian/initramfs/hooks/brltty.in
        - debian/initramfs/scripts/init-premount/brltty
        - debian/initramfs/scripts/init-top/brltty
        - debian/brltty.dirs
        - debian/brltty.install
        - debian/brltty.postinst
      + add ubiquity integration to propagate any brltty configuration to the
        target system
        - debian/31brltty
        - debian/brltty.dirs
        - debian/brltty.install
      + add udev rules file that uses brltty-udev.service to activate via
        systemd
        - debian/brltty-udev.service
        - debian/brltty.dirs
        - debian/brltty.udev.rules
        - debian/rules
      + disable brltty.service by default, but enable it if the user configures
        Braille at install time for a non-USB display
        - debian/31brltty
        - debian/rules
      + don't install /etc/brltty.conf in the package
        - debian/31brltty
  * Rebase udev rules (debian/brltty.udev.rules) on upstream ones

 -- Olivier Tilloy <email address hidden> Wed, 17 Nov 2021 13:15:55 +0100