After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 702682 - FTBFS: master doesn't to build pangocairo in 3.10
FTBFS: master doesn't to build pangocairo in 3.10
Status: RESOLVED FIXED
Product: jhbuild
Classification: Infrastructure
Component: module sets
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Jhbuild maintainers
Jhbuild QA
Depends on:
Blocks:
 
 
Reported: 2013-06-19 19:11 UTC by Germán Poo-Caamaño
Modified: 2013-11-02 21:36 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
3.10: cairo 1.12.10 needed by pangocairo, moved to -core-deps-3.10 (2.82 KB, patch)
2013-06-19 19:11 UTC, Germán Poo-Caamaño 		needs-work Details | Review
3.10: cairo 1.12.10 needed by pangocairo, moved to -core-deps-3.10 (2.27 KB, patch)
2013-06-20 16:53 UTC, Germán Poo-Caamaño 		committed Details | Review

Description Germán Poo-Caamaño 2013-06-19 19:11:00 UTC 
Created attachment 247281 [details] [review]
3.10: cairo 1.12.10 needed by pangocairo, moved to -core-deps-3.10

pango in 3.10 requires cairo 1.12.10, however -sysdeps asks for 1.10.2.

As a consequence, if cairo 1.10.2 is installed, pangocairo is not built and librsvg fails (=> no gnome-themes-standard).

The patch fixes the issue, but I moved cairo back to -gnome-core-deps, because not all systems have 1.12.10 yet.  The patch also updates cairomm.
Comment 1 Germán Poo-Caamaño 2013-06-20 06:41:48 UTC 
FWIW, this is the commit where pango bumped the cairo version required: https://git.gnome.org/browse/pango/commit/?id=49712d6381f25fe3785694fea7804988c1067a24
Comment 2 Javier Jardón (IRC: jjardon) 2013-06-20 14:58:06 UTC 
Review of attachment 247281 [details] [review]:

Looks good to me.
But instead moving cairomm simply bump the requirement in the sysdeps definition, cairomm 1.10 is available in current distros
Comment 3 Germán Poo-Caamaño 2013-06-20 16:53:44 UTC 
Created attachment 247357 [details] [review]
3.10: cairo 1.12.10 needed by pangocairo, moved to -core-deps-3.10
Comment 4 Javier Jardón (IRC: jjardon) 2013-07-09 16:41:44 UTC 
Comment on attachment 247357 [details] [review]
3.10: cairo 1.12.10 needed by pangocairo, moved to -core-deps-3.10

commit c0aebe3eebc199ff23665e7db536281bf3453e56

Thanks German!