Bug 1377910

Summary:	Thunderbird should not be build against system cairo
Product:	[Fedora] Fedora	Reporter:	Hubert Figuiere <hub+rhbz>
Component:	thunderbird	Assignee:	Gecko Maintainer <gecko-bugs-nobody>
Status:	CLOSED NOTABUG	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	24	CC:	gecko-bugs-nobody, jhorak, pjasicek
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2016-09-21 09:27:15 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:

Description Hubert Figuiere 2016-09-21 01:28:12 UTC

Thunderbird should not be build against system cairo

Like Firefox, it should be build against the provided version of cairo. Otherwise it crashes very often.

See https://bugzilla.mozilla.org/show_bug.cgi?id=1174997#c5

Comment 1 Jan Horak 2016-09-21 09:27:15 UTC

Hm, it seems that TB is build without system cairo by default. Why do you presume that it is build against system cairo? You are correct that we should not build against system cairo. Do you miss the explicit --disable-system-cairo in the .mozconfig?

The configuration is implemented by MOZ_TREE_CAIRO variable, which according to build log [1] is set to 1, ie using in-tree cairo. Some build flags related to cairo are introduced by GTK2 but that should not affect actual build.

Please reopen if you have some proof that I'm wrong here. Thank you.

[1] https://kojipkgs.fedoraproject.org//packages/thunderbird/45.3.0/1.fc24/data/logs/i686/build.log

Comment 2 Hubert Figuiere 2016-09-21 11:57:21 UTC

there is a direct dependency on cairo-devel in the .spec file that Firefox doesn't. I may have been mistaken. Sorry.

Comment 3 Jan Horak 2016-09-22 07:38:46 UTC

Okay, fair enough, I'll remove that dependency, thanks for letting us know.