On Wed, 20 Mar 2024 22:04:03 +0100 Lucas Nussbaum <lu...@debian.org> wrote:
Source: scons
Version: 4.5.2+dfsg-1
Severity: serious
Justification: FTBFS
Tags: trixie sid ftbfs
User: lu...@debian.org
Usertags: ftbfs-20240319 ftbfs-trixie

Hi,

During a rebuild of all packages in sid, your package failed to build
on amd64.

This is unfortunate. The docbook toolchain seems to have always generated a lot of spew from fop, which to the SCons project has seemed "out of our control". Now it seems something has changed so where the java used to throw SEVERE it is now throwing ERROR, thus failing the build - which really has nothing to do with the operation of SCons, which is just a Python package, after all. This is a failure to generate the pdf versions of the docs, which are unused by the deb package, afaict.

Relevant part (hopefully):

> [ERROR] FOUserAgent - Invalid property value encountered in margin-left="": org.apache.fop.fo.expr.PropertyException: 
file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion defined ; property:'margin-left' (See position 147:8) 
<org.apache.fop.fo.expr.PropertyException: file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion defined 
; property:'margin-left'>org.apache.fop.fo.expr.PropertyException: 
file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion defined ; property:'margin-left'
...
> [ERROR] FOUserAgent - Invalid property value encountered in margin-right="": org.apache.fop.fo.expr.PropertyException: 
file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion defined ; property:'margin-right' (See position 
147:8) <org.apache.fop.fo.expr.PropertyException: file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion 
defined ; property:'margin-right'>org.apache.fop.fo.expr.PropertyException: 
file:/<<PKGBUILDDIR>>/doc/man/scons-scons-time.fo:3:6: No conversion defined ; property:'margin-right'
>    at org.apache.fop.fo.properties.PropertyMaker.make(PropertyMaker.java:446)
>    at 
org.apache.fop.fo.PropertyList.convertAttributeToProperty(PropertyList.java:499)

The latest upstream release (4.7.0) tried to provide a way to avoid doing the pdf and Sphinx builds, but it turns out it doesn't do that in a place that the Debian packaging uses anyway. Will make a note to revisit that on our end.

Didn't realize the deb directly used the build files in doc/man and doc/user. The quickest way to patch around this would seem to be to comment out or remove the check in

doc/man/SConstruct
doc/user/SConstrcut

which checks for the presence of the 'fop' or 'xep' commands, so that 'has_pdf' remains false, and the pdf build thus isn't attempted at all. I assume you don't need the pdf files, just the scons.1 etc. files? that's all I see copied in https://salsa.debian.org/debian/scons/-/blob/master/debian/rules?ref_type=heads

Reply via email to