paraview 5.12.1+dfsg-3 FTBFS on Oracular

[Zixing Liu]

paraview 5.12.1+dfsg-3 FTBFS on Oracular on ppc64el and s390x due to compiler unable to encode the jumps or symbols.

[Ubuntu Foundations Team Bug Bot (crichton)]

The attachment "paraview-5.12.1+dfsg-3ubuntu1.debdiff" seems to be a debdiff. The ubuntu-sponsors team has been subscribed to the bug report so that they can review and hopefully sponsor the debdiff. If the attachment isn't a patch, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are member of the ~ubuntu-sponsors, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issue please contact him.]

[Sergio Durigan Junior (sergiodj)]

Thanks for the patch, Zixing Liu.

I'd like to ask you adjust a few things before we proceed with the upload:

1) Since you're adding a delta, you also need to run update-maintainer in order to have the Maintainer field set to Ubuntu Developers instead of the regular Debian maintainer.

2) You're also reorganizing the definitions of the compiler flags. Could you please mention this in the changelog entry?

3) While the CMAKE_BUILD_TYPE change looks sensible to me, it would be great if you could provide a PPA where the modified package builds properly.

I will remove ubuntu-sponsors from the notification list for now. Please re-add it when the package is ready to be reviewed again.

Thanks!

[Sergio Durigan Junior (sergiodj)]

Something else I forgot to ask/mention: does the package also FTBFS on Debian? If yes, then it's worth considering forwarding the delta to them as well.

Also, I might be wrong here but I believe you don't have to manually set the compiler flags like that anymore. This used to be the case a long time ago when debhelper's cmake module couldn't handle the flags very well, but nowadays things should just work. Might be worth giving it a try.