Johannes Thyssen Tishman wrote (2023-05-14 23:40 CEST):
> > $ solvespace
> > Segmentation fault
> 
> Sorry for that. After some testing we found out it was due to the
> MI_OVERRIDE compile option. Disabling it should fix the Segmentation
> fault. We are currently working on getting rid of the bundled mimalloc
> dependency, but for the time being the attached port should work just
> fine.

I can confirm it's working now.

> > I tried to build it with debug symbols. But the cmake build seems to
> > ignore my "DEBUG=-g make". I haven't looked further.
> 
> We will look into this too and give an update as soon as possible.
> Thanks again for the feedback.

I'm not too worried about it as long as the port works :-)

Attached is the port again with some tweaks:

* WANTLIB sorted
* Some Makefile =<space><tab> fixes
* Simplified post-extract
* Copy FindCairo.cmake from FILESDIR instead of patching it in

Reg. the patches: I prefer lines to be removed instead of commented.
This makes the patch smaller and better readable. But it's a matter
of taste, so I didn't change it.

Reg. the rand() change. I saw the Github comment on your pull request:
> Agree that we want determinism so the rand change is not useful here

Please double check if you want arc4random here or srand_deterministic
or not touch it at all.

Best Regards,
Stefan

Attachment: solvespace.tgz
Description: application/tar-gz

Reply via email to