Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Build r-base with cairo & pango #1399

Open
wants to merge 16 commits into
base: main
Choose a base branch
from
Open

Build r-base with cairo & pango #1399

wants to merge 16 commits into from

Conversation

anutosh491
Copy link
Collaborator

No description provided.

@anutosh491 anutosh491 marked this pull request as ready for review October 25, 2024 12:17
Copy link
Member

@IsabelParedes IsabelParedes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You probably need to add --with-static-cairo to the configure step.

And perhaps it might need some more tweaking.

checking whether pkg-config knows about cairo and pango... no
checking whether pkg-config knows about cairo... no
Capabilities skipped:        JPEG, NLS, cairo, ICU

This test requires all of these libraries to pass:
- pangocairo-1.0
- thai
- datrie
- Xft
- Xrender
- xcb
- Xau
- Xdmcp
- harfbuzz-gobject
- gio-2.0
- gmodule-2.0
- mount
- blkid
- gobject-2.0
- ffi
- fribidi
- fontconfig
- expat
- freetype
- harfbuzz
Copy link
Member

@IsabelParedes IsabelParedes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The cairo-svg.h file was not included in the cairo package.

  • include/cairo/cairo-deprecated.h
  • include/cairo/cairo-features.h
  • include/cairo/cairo-ft.h
  • include/cairo/cairo-gobject.h
  • include/cairo/cairo-pdf.h
  • include/cairo/cairo-ps.h
  • include/cairo/cairo-script-interpreter.h
  • include/cairo/cairo-script.h
  • include/cairo/cairo-tee.h
  • include/cairo/cairo-version.h
  • include/cairo/cairo.h

IsabelParedes
IsabelParedes previously approved these changes Oct 25, 2024
@IsabelParedes IsabelParedes dismissed their stale review October 25, 2024 15:40

Changes needed

@IsabelParedes
Copy link
Member

IsabelParedes commented Oct 25, 2024

I don't know why pkg-config is not finding cairo Fixed!

checking whether pkg-config knows about cairo and pango... no
checking whether pkg-config knows about cairo... no

@IsabelParedes
Copy link
Member

@anutosh491 The issue with pkg-config is that the dependencies for those packages were not being met. This is probably also what was happening with harfbuzz, maybe we should rebuild harfbuzz and move the dependencies back to the host environment.

$PREFIX/lib/pkgconfig/cairo.pc

Name: cairo
Description: Multi-platform 2D graphics library
Version: 1.18.0
Requires: fontconfig >=  2.2.95, freetype2 >=  9.7.3, pixman-1 >=  0.36.0
Libs: -L${libdir} -lcairo -lm -ldl
Cflags: -I${includedir}/cairo

@IsabelParedes
Copy link
Member

IsabelParedes commented Oct 25, 2024

It seems that cairo and pango need to be rebuilt before they can be integrated into r-base :(

From cairo we need cairo-png

error: call to undeclared function 'cairo_surface_write_to_png'; 

And from pango we need libpangocairo.a.

checking whether pkg-config knows about cairo and pango... no

@IsabelParedes IsabelParedes mentioned this pull request Dec 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants