Opened 22 months ago
Closed 22 months ago
#66667 closed defect (fixed)
Gegl fails after libraw update
Reported by: | pietvo (Pieter van Oostrum) | Owned by: | mascguy (Christopher Nielsen) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | josephsacco, cjones051073 (Chris Jones), michaellass (Michael Lass), someuser12, mouse07410 (Mouse), MaurizioLoreti | |
Port: | gegl |
Description
After a recent libraw update gegl
no longer compiles. (https://github.com/macports/macports-ports/commit/144bf8fbcc5417ec1d8950ff2e232d6eec8a507c#commitcomment-95694848)
Attachments (2)
Change History (16)
Changed 22 months ago by pietvo (Pieter van Oostrum)
comment:1 Changed 22 months ago by Gandoon (Erik Hedlund)
Also ImageMagick is affected by this problem. I just keep the previous versions active for the time being.
comment:2 Changed 22 months ago by josephsacco
Cc: | josephsacco added |
---|
comment:3 Changed 22 months ago by jmroot (Joshua Root)
Cc: | cjones051073 added; mascguy removed |
---|---|
Owner: | set to mascguy |
Port: | gegl added |
Status: | new → assigned |
comment:4 Changed 22 months ago by michaellass (Michael Lass)
Cc: | michaellass added |
---|
comment:5 Changed 22 months ago by Lev-GitHub
I've encountered a similar situation without the "quartz" option.
comment:6 Changed 22 months ago by someuser12
Cc: | someuser12 added |
---|
comment:7 follow-up: 8 Changed 22 months ago by lukaso (Lukas Oberhuber)
Here is the upstream issue: https://gitlab.gnome.org/GNOME/gegl/-/issues/315
Could version 0.20.2 be given it's own port?
comment:8 follow-up: 10 Changed 22 months ago by mascguy (Christopher Nielsen)
Replying to lukaso:
Here is the upstream issue: https://gitlab.gnome.org/GNOME/gegl/-/issues/315
Could version 0.20.2 be given it's own port?
That's certainly an option, depending on whether any other ports are broken.
Is the API change simple/isolated enough, that we could easily patch gegl for now? Your thoughts Lukas?
comment:9 Changed 22 months ago by ryandesign (Ryan Carsten Schmidt)
Cc: | mouse07410 MaurizioLoreti added |
---|
comment:10 Changed 22 months ago by cjones051073 (Chris Jones)
Replying to mascguy:
Replying to lukaso:
Here is the upstream issue: https://gitlab.gnome.org/GNOME/gegl/-/issues/315
Could version 0.20.2 be given it's own port?
That's certainly an option, depending on whether any other ports are broken.
Is the API change simple/isolated enough, that we could easily patch gegl for now? Your thoughts Lukas?
Adding multiple versioned ports should only be done in specific circumstances, and this does not seem like one to me. By far the preferable solution is to patch the ports that need it to work with the new API (which upstream for those ports should anyway do).
comment:11 Changed 22 months ago by pietvo (Pieter van Oostrum)
This patch is sufficient to get it compiled:
--- operations/external/raw-load.c.~1~ +++ operations/external/raw-load.c @@ -114,7 +114,7 @@ g_warning ("raw-load: Error Initializing raw library"); else { - p->LibRaw->params.shot_select = o->image_num; + p->LibRaw->rawparams.shot_select = o->image_num; p->LibRaw->params.aber[0] = 1.0; p->LibRaw->params.aber[2] = 1.0;
I can submit a PR.
comment:12 Changed 22 months ago by michaelld (Michael Dickens)
comment:13 Changed 22 months ago by josephsacco
Applying the patch allowed gel to build.
I tried running some of the examples that come with the package, but I cannot get them to work. For example:
cavandish:examples jsacco$ ./hello-world (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.227: cannot register existing type 'GeglModule' (hello-world:91432): GLib-CRITICAL **: 15:33:58.228: g_once_init_leave: assertion 'result != 0' failed (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.228: invalid cast from 'GeglModule' to '<invalid>' (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.281: Two different plugins tried to register 'GeglOpseamless-clone_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.281: Two different plugins tried to register 'GeglOppixbuf_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpff-load_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpcache_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpcast-format_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpcast-space_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpclone_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpconvert-format_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpconvert-space_c'. (hello-world:91432): GLib-GObject-WARNING **: 15:33:58.283: Two different plugins tried to register 'GeglOpcrop_c'.
Was something supposed to appear soon my screen?
This may very well be "pilot error" on my part, but... Someone who actually understands this code might want to take a look.
-Joseph
comment:14 Changed 22 months ago by michaelld (Michael Dickens)
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
log file