[oe] [meta-qt5] qtwebkit 5.8 build issues?

[Andre McCurdy]

On Wed, Apr 12, 2017 at 5:11 PM, Andreas Oberritter
<obi at opendreambox.org> wrote:
> On Wed, 12 Apr 2017 16:38:58 -0700
> Andre McCurdy <armccurdy at gmail.com> wrote:
>
>> On Wed, Apr 12, 2017 at 3:45 PM, Andreas Oberritter
>> <obi at opendreambox.org> wrote:
>> > On Wed, 29 Mar 2017 19:06:14 -0400
>> > Denys Dmytriyenko <denis at denix.org> wrote:
>> >
>> >> On Sun, Mar 26, 2017 at 04:30:53PM -0400, Denys Dmytriyenko wrote:
>> >> > Hi,
>> >> >
>> >> > I've been having the following build issues lately with qtwebkit 5.8 from
>> >> > master:
>> >> >
>> >> > | make[2]: Entering directory '/OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/build/Source'
>> >> > | rm -f libQt5WebKit.so.5.8.0 libQt5WebKit.so libQt5WebKit.so.5 libQt5WebKit.so.5.8
>> >> > | linking ../lib/libQt5WebKit.so.5.8.0
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `__bss_start'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `__bss_start'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `_edata'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `_edata'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `_end'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Core.so:(*IND*+0x0): multiple definition of `_end'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Gui.so:(*IND*+0x0): multiple definition of `__bss_start'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Gui.so:(*IND*+0x0): multiple definition of `_edata'
>> >> > | /OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/recipe-sysroot/usr/lib/libQt5Gui.so:(*IND*+0x0): multiple definition of `_end'
>> >> > | collect2: error: ld returned 1 exit status
>> >> > | Makefile.api:92: recipe for target '../lib/libQt5WebKit.so.5.8.0' failed
>> >> > | make[2]: *** [../lib/libQt5WebKit.so.5.8.0] Error 1
>> >> > | make[2]: Leaving directory '/OE/master/build/tmp/work/armv7ahf-neon-oe-linux-gnueabi/qtwebkit/5.8.0+gitAUTOINC+74ac5b0f34-r0/build/Source'
>> >> > | Makefile.QtWebKit:44: recipe for target 'sub-api-pri-make_first-ordered' failed
>> >> > | make[1]: *** [sub-api-pri-make_first-ordered] Error 2
>> >> >
>> >> > I believe I was able to build 5.8/master before, so I'm suspecting recent
>> >> > binutils upgrade... But I can be wrong. Would really appreciate some help
>> >> > here. Thanks.
>> >>
>> >> Anyone else see this?
>> >>
>> >
>> > I do. Have you been able to solve it in the meantime?
>>
>> Do you both have gold enabled?
>>
>>   https://bugs.launchpad.net/ubuntu/+source/qtwebkit-opensource-src/+bug/1653529
>
> I don't use the ld-is-gold flag, but gold gets built nevertheless. It's just not
> the default linker. I guess qtwebkit forces its use. So the question becomes whether
> I should set ld-is-gold or teach qtwebkit not to use gold, in order not to mix both
> linkers.

Assuming qtwebkit has a configure option to explicitly enable/disable
gold, I'd say add a PACKAGECONFIG option.

In the short term keep the PACKAGECONFIG option disabled and in the
longer term (once qtwebkit builds successfully with gold enabled) set
the PACKAGECONFIG option based on the testing the distro feature.

> Regards,
> Andreas