Try to fix the pixbuf problem.

[cdepillabout]

Try to fix the pixbuf problem from #107.

[jtojnar]

I do not think this will help for NixOS/nixpkgs#54278. The issue is caused by mismatch between the gdk-pixbuf the project is linked against and the loaders picked up from GDK_PIXBUF_MODULE_FILE environment variable. And wrapGAppsHook should already take care of wrapping all executables with the correct variable.

[jtojnar]

Right, you are overriding the buildCommand therefore the envHooks necessary for gdk-pixbuf setup hook will not run.

You could try to set it manually but I would just suggest to rely on the default builder. Something like

dontBuild = true;
unpackPhase = ":";
installPhase = ''
  runHook preInstall
  mkdir -p $out/bin
  ln -sf ${env}/bin/termonad $out/bin/termonad
  runHook postInstall
'';

instead of the buildCommand.

[jtojnar]

There is also some kind of interaction between setup hooks and cross-compilation offsets making the example above no work: NixOS/nixpkgs#56943

[cdepillabout]

@jtojnar Thanks for the info! That is very helpful.

@craigem @romanofski I added an additional commit to this PR (0846a8f) fixing up the build as suggested by @jtojnar. Could you try it out and see if it fixes your problem with Termonad segfaulting?

[romanofski]

Seems to be working for me. I tried c1d136b and yep, still segfaults. So 0846a8f seems to do the trick. Is this a workaround until NixOS/nixpkgs#56943 is fixed?

[jtojnar]

The gdk_pixbuf insertion is the workaround, the rest is normal – we do not recommend overriding builder unless you are fine with and none of the generic builder procedures and setup hooks working.

[cdepillabout]

Okay, since this seems to be working, I will go ahead and merge it in.

@romanofski Thanks for your help with trying out various things. I'm glad we finally got this working for you.

@jtojnar Thanks for the suggestion to stop overriding the builder!