Hi Raphael,

On Fri, 21 Sep 2018 14:16:15 +0200, Raphael Hertzog <[email protected]> wrote:
> Hello Frédéric,
> 
> On Thu, 20 Sep 2018, Frédéric Bonnard wrote:
> > This was version in 5.11.0-2. But in both case, the patch
> > update-patchelf.patch hardcodes PAGESIZE which is not 4k on ppc64el.
> 
> Ah, that's a good hint.
> 
> > This was mentionned here btw :
> > https://bugreports.qt.io/browse/PYSIDE-740
> 
> Indeed, but I did not make the relation between this and the failing build
> on ppc64el.
> 
> > I tried compiling pyside2 after dropping update-patchelf.patch and
> > pulling
> > https://codereview.qt-project.org/gitweb?p=pyside/pyside-setup.git;a=patch;h=0479cbd6abc8fffb9ba8b5f94de067e1670ffffa
> > so that the correct PAGESIZE gets defined and it builds fine with the 
> > system patchelf.
> > 
> > Upstream 5.11.1 has the latter patch, so I guess, upgrading to the
> > latest 5.11.1 and removing update-patchelf.patch should be enough to get
> > rid of the FTBFS.
> 
> Using the system-wide patchelf brings another problem:
> https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=903396
> 
> Maybe this problem does not appear on ppc64el? Did you try building your
> modified source package on amd64?

It actually builds on amd64 and uses the system's patchelf.
I didn't see any output related to a stripping issue. dbg .debs were
created and are not empty...

> Can you prepare an improved update-patchelf.patch that does pass the
> correct PAGESIZE? Merge request welcome.

Sent : https://salsa.debian.org/qt-kde-team/qt/pyside2/merge_requests/1
The latter may be safer.

F.

> Cheers,
> -- 
> Raphaël Hertzog ◈ Debian Developer
> 
> Support Debian LTS: https://www.freexian.com/services/debian-lts.html
> Learn to master Debian: https://debian-handbook.info/get/

Attachment: pgpClyQQToosS.pgp
Description: PGP signature

Reply via email to