zathura: missing link numbering

[zspher]

Nixpkgs version

• Unstable (26.11)

Describe the bug

zathura link number overlay (press f and look at links) is missing.

I've managed to track down the PR causing the issue: #520766. The PR sets __structuredAttrs = true; which removes the webp-pixbuf-loader and gdk-pixbuf-loaders.cache dependencies which seems to be required for link numbering.

Steps to reproduce

1. open a pdf file (e.g. https://www.bu.edu/math/files/2013/08/tikzpgfmanual.pdf) with zathura
2. press f
3. look for number overlay

Expected behaviour

there should be a number overlay

Screenshots

numbered links missing:

numbered links present:

Relevant log output

Additional context

No response

System metadata

• system: "x86_64-linux"
• host os: Linux 6.18.32, NixOS, 26.05 (Yarara), 26.05.20260523.2991645
• multi-user?: yes
• sandbox: yes
• version: `nix-env (Lix, like Nix) 2.95.2

Notify maintainers

---

@MithicSpirit @TethysSvensson
Note for maintainers: Please tag this issue in your pull request description. (i.e. Resolves #ISSUE.)

I assert that this issue is relevant for Nixpkgs

• I assert that this is a bug and not a support request.
• I assert that this is not a duplicate of an existing issue.
• I assert that I have read the NixOS Code of Conduct and agree to abide by it.
• I assert that I have read the automation/AI policy and that this issue report complies with it.

Is this issue important to you?

Add a 👍 reaction to issues you find important.

[zspher]

using the ff. overlay fixes the problem

zathuraPkgs = prev.zathuraPkgs.overrideScope (
  selfx: prevx: {
    # FIX: missing link numbering
    zathura_core = prevx.zathura_core.overrideAttrs (oldAttrs: {
      __structuredAttrs = false;
    });
  }
);

[SandaruKasa]

Wouldn't the problem be caused by strictDeps and not __structuredAttrs?

[zspher]

Wouldn't the problem be caused by strictDeps and not __structuredAttrs?

setting

            zathura_core = prevx.zathura_core.overrideAttrs (oldAttrs: {
              __structuredAttrs = true;
              strictDeps = false;
            });

still results in the issue

[MithicSpirit]

Okay I'm trawling through diffs and I finally found something: wrapGAppsHook3 doesn't seem to be working with __structuredAttrs since the log output is missing "Wrapping program …" lines. It seems like this is being fixed in staging (#511330), so I'm not sure it's worth reverting just to make this change again when staging is merged.

[MithicSpirit]

Can confirm that cherry-picking a0ee897 (#511330) fixes this issue for me, so this will probably be fixed in master with the next staging merge (I haven't tested directly on staging due to prohibitively long build times).

[MithicSpirit]

The necessary changes appear to be included in #535855.