I can confirm that this build works (still on NixOS though)

Marking as WIP as upstream has an emergency release this weekend. Will remove WIP and increment when the new release occurs (provisionally 0.38.1)

Emergency Release as mentioned above.

Fixes a double-free bug.

Oh, that's some great solutions - I'll implement and test them now. Thank you so much for your help thus far!

I'm having problems with the substituteAll suggestion - getting

[red@evil:~/projects]$ nix-shell -I nixpkgs=nixpkgs -p ponyc
error: syntax error, unexpected '{', expecting '.' or '=', at /home/red/projects/nixpkgs/pkgs/development/compilers/ponyc/default.nix:38:17
(use '--show-trace' to show detailed location information)

Line 38: is the first line in:

  substituteAll {
    src = ./make-safe-for-sandbox.patch;
    googletest = fetchurl {
      url = https://github.com/google/googletest/archive/release-1.8.1.tar.gz;
      sha256 = "17147961i01fl099ygxjx4asvjanwdd446nwbq9v8156h98zxwcv";
      name = "release-1.8.1.tar.gz";
    };
  };

I guess I don't know where it should go?

I think you'll need to use the substituteAll as the expression instead of the original patch? As is it seems like the parse error is coming up because it's just a free-floating expression in the middle of your record

I'm looking through examples in nixpkgs and I think you're right. Lemme try that in patches...

Okay - got it. Testing and pushing

@veprbl - Re-requested a review as I've applied all suggested improvements. Is there anything else you need from me?

@veprbl - this commit should be interesting as I applied all your changes and re-pushed... but the build hash didn't change.

I am both surprised and impressed that s/clangStdenv/stdenv/ in the .nix file and adding stdenv = clangStdenv as an override in all-packages resulted in the same output!

@NixOS/ofborg

@GrahamcOfBorg eval

@redvers The s/clangStdenv/stdenv/ is just a variable rename, it does not change the computed derivation, so should not be too surprising. But I agree, it is quite nice, how it works.

@veprbl - it's still failing, even though that commit to maintainers was reverted. I guess I can change a comment in a patch or something to force a new hash to force it to calculate?

Success!

Was able to build this as an overlay on MacOS 👍

@GrahamcOfBorg build ponyc pony-stable

@veprbl - stable was the name of the package-manager. It's not a stable version of pony.

It's also being deprecated and replaced by corral.

See: #98601

@veprbl - I have no idea why this is still executing after 6 hours. All the logs appear to be complete.

Our darwin builder is offline right now. I don't know why the "Wait for ofborg" status still hasn't cleared.