#67712 closed defect (fixed)

gegl @0.4.46: error: unknown type name 'RsvgRectangle'; did you mean 'GeglRectangle'?

Reported by: barracuda156 Owned by: mascguy (Christopher Nielsen)
Priority: Normal Milestone:
Component: ports Version: 2.8.1
Keywords: powerpc, tiger, leopard Cc:
Port: gegl

Description

../gegl-0.4.46/operations/external/svg-load.c:101:5: error: unknown type name 'RsvgRectangle'; did you mean 'GeglRectangle'?
  101 |     RsvgRectangle      svg_rect = {0.0, 0.0, width, height};
      |     ^~~~~~~~~~~~~
      |     GeglRectangle

0.4.44 built fine. That was pretty recent, so I think nothing dramatic happened with the toolchain since then.

Change History (7)

comment:1 Changed 10 months ago by ryandesign (Ryan Carsten Schmidt)

Keywords: tiger added; snowleopard removed
Owner: set to mascguy
Status: newassigned
Summary: Something got broken in the latest gegl: it does not build on PPC: error: unknown type name 'RsvgRectangle'; did you mean 'GeglRectangle'?gegl @0.4.46: error: unknown type name 'RsvgRectangle'; did you mean 'GeglRectangle'?

The use of RsvgRectangle was introduced into gegl 3 weeks ago when they migrated from the old rsvg_handle_render_cairo API to the new rsvg_handle_render_document API as described in the librsvg migration documentation.

According to the migration document, the new methods were introduced in librsvg 2.46.

The librsvg port uses the current version on systems that can use rust, and the older 2.40.20 version on old systems not capable of using rust (PowerPCs, and Leopard or earlier).

So, sorry, gegl @0.4.46 is not compatible with old systems that require the use of the older librsvg. Maybe the gegl port can be enhanced to use an older version on those old systems.

comment:2 in reply to:  1 Changed 10 months ago by barracuda156

Replying to ryandesign:

The use of RsvgRectangle was introduced into gegl 3 weeks ago when they migrated from the old rsvg_handle_render_cairo API to the new rsvg_handle_render_document API as described in the librsvg migration documentation.

According to the migration document, the new methods were introduced in librsvg 2.46.

The librsvg port uses the current version on systems that can use rust, and the older 2.40.20 version on old systems not capable of using rust (PowerPCs, and Leopard or earlier).

So, sorry, gegl @0.4.46 is not compatible with old systems that require the use of the older librsvg. Maybe the gegl port can be enhanced to use an older version on those old systems.

Oops, that is a problem.

We either need to fix librsvg, or come up with fallbacks, or in the worst case peg an older gegl, for now. Otherwise all dependencies are broken.

comment:3 Changed 10 months ago by barracuda156

I guess little hope, but just in case: https://gitlab.gnome.org/GNOME/gegl/-/issues/333

In the meanwhile, what will be a better fix for now? Maybe just revert breaking commit in Gegl?

comment:4 in reply to:  3 ; Changed 10 months ago by mascguy (Christopher Nielsen)

Replying to barracuda156:

I guess little hope, but just in case: https://gitlab.gnome.org/GNOME/gegl/-/issues/333

In the meanwhile, what will be a better fix for now? Maybe just revert breaking commit in Gegl?

I'll probably pin gegl to the previous version for older systems, but I'm still dealing with other higher-priority items.

Since I'm the gatekeeper on this, there's no need to send a message to the Dev mailing list...

comment:5 in reply to:  4 Changed 10 months ago by barracuda156

Replying to mascguy:

Replying to barracuda156:

I guess little hope, but just in case: https://gitlab.gnome.org/GNOME/gegl/-/issues/333

In the meanwhile, what will be a better fix for now? Maybe just revert breaking commit in Gegl?

I'll probably pin gegl to the previous version for older systems, but I'm still dealing with other higher-priority items.

Since I'm the gatekeeper on this, there's no need to send a message to the Dev mailing list...

Upstream fixed this: https://gitlab.gnome.org/GNOME/gegl/-/issues/333 We can just pick the patch.

I.e. https://gitlab.gnome.org/GNOME/gegl/-/commit/a99a93e5c9013bd4101f5058cdee7d0cf30234fe

Last edited 10 months ago by barracuda156 (previous) (diff)

comment:6 Changed 10 months ago by Christopher Nielsen <mascguy@…>

In cb20d7ec955008abd035bce1929639b745ed1794/macports-ports (master):

gegl-devel: patch to support non-rust librsvg
See: #67712

comment:7 Changed 10 months ago by Christopher Nielsen <mascguy@…>

Resolution: fixed
Status: assignedclosed

In c8ff2312c9f8c1695bd0b94492e3f31d2c120a87/macports-ports (master):

gegl: patch to support non-rust librsvg
Closes: #67712

Note: See TracTickets for help on using tickets.