Re: [ptxdist] python3-lxml: package seems to be broken due to libxslt config options

[Alexander Dahl <ada@thorsis.com>]

Hello,

Am Wed, Sep 29, 2021 at 02:14:08PM +0200 schrieb Michael Olbrich:
> On Sun, Sep 05, 2021 at 03:29:47PM +0200, Felix Mellmann wrote:
> > On 03.09.21 17:36, Michael Olbrich wrote:
> > > On Wed, Sep 01, 2021 at 08:37:27PM +0200, Felix Mellmann wrote:
> > > > Hi there,
> > > > 
> > > > I've just encountered a problem when using python3-lxml on PTXdist master
> > > > branch:
> > > > 
> > > > $ python3
> > > > Python 3.7.10 (default, Aug  1 2021, 00:00:00)
> > > > [GCC 10.2.1 20200822] on linux
> > > > Type "help", "copyright", "credits" or "license" for more information.
> > > > > > > from lxml import etree
> > > > Traceback (most recent call last):
> > > >    File "<stdin>", line 1, in <module>
> > > > ImportError:
> > > > /usr/lib/python3.7/site-packages/lxml/etree.cpython-37m-x86_64-linux-gnu.so:
> > > > undefined symbol: xsltGetProfileInformation
> > > > 
> > > > The problem arises through libxslt which is configured using
> > > > "--without-profiler" since commit 1940c3 .
> > > > 
> > > > Removing this option again allows to at least import etree from lxml. I
> > > > haven't done any investigation whether lxml may crash again elsewhere.
> > > > 
> > > > Are there any concerns about removing that configuration option again?
> > > You should probably change it to --with-profiler instead unless that's
> > > broken. Otherwise, I have no objections.
> > 
> > Maybe Alexander had reasons to add "--without-profiler" without any
> > configuration option. He added three more options within the same patch but
> > left only this one fixed to "without".
> > 
> > So the compromise could be to add a new configuration option with default
> > value of "n" and add this configuration option as a dependency to
> > python3-lxml?
> > 
> > But as long as we don't know Alexander's reasons to do so, compiling with
> > the profiler option could break something else.
> 
> So lets ask him: Alex, can you comment on this? Did you have a reason other
> than just disabling it because it seemed unnecessary?

I just checked diff between ./configure output of libxslt 1.1.29 and
1.1.34 and this is part of it:

+  --with-profiler        Add the profiling support (on)

This means the option was new and I probably just set it to some
default.  So yes, my reasoning was most probably it just seemed
unnecessary to me.  If you need it set to --with-profiler or made
optional, go ahead the way Michael suggested.

Greets
Alex

> 
> Michael
> 
> > > Please add a comment about breaking python3-lxml at runtime, so we know in
> > > the future why this was changed.
> > > 
> > > Michael
> > > 
> > 
> > _______________________________________________
> > ptxdist mailing list
> > ptxdist@pengutronix.de
> > To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 
> -- 
> Pengutronix e.K.                           |                             |
> Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
> 31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
> Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de