Created attachment 216384 [details] cd /usr/ports/net-p2p/gtk-gnutella && patch < <filename> Patch for update to latest version. Note, new build requires "gmake", other than that, a pretty bog standard update from a ports point of view Cheers, Jamie
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/170786666
I'll take ownership if this is abandoned...
testbuilds@work
It builds on amd64, but fails on current-i386, see https://people.freebsd.org/~pi/logs/net-p2p__gtk-gnutella-curi-1598951181.txt Do you want to investigate or is i386 no longer supported ?
Sorry about that. I'll investigate, and speak to the author. Cheers, Jamie
Hi Jamie, How did you go - get any feedback on why gtk-gnutella has stopped building on i386 with this update? Are you able to provide a link to an upstream bug report? Do you think the preference is to update to the latest release and mark i386 as broken, or hold off on the update? Cheers, Ben
Hi Ben. I include a new patch file to replace the previous one. I don't have an upstream bug report - I've been talking to Raphael (the author) directly. Anyway, I found the problem, and he has confirmed my fix is correct. Basically he'd made a change and missed off i386 related settings for when the system malloc is being used. The patch will be merged upstream, and is included here. This wasn't noticed because he develops (and tests extensively on FreeBSD amd64) using his own recommended malloc implementation. I'm not sure why the default setting for this port was to use system malloc, but if there are no objections, I've changed the port to now use his internal malloc by default, but added an OPTION tag so that it is configurable via the config dialog. I've also updated and tidied up pkg-help. Note, I needed to use patch -p0 < ... to apply this (and then remove the zero length files/patch-src_lib_xmalloc.c.orig manually) - if you know of a better way to create the patch file, I'm all ears! :-) Let me know if this is ok! Cheers, Jamie
Created attachment 218312 [details] New patch. Obsoletes previous one. patch -p0 < ....
ping???
Hi Jamie, sorry for not replying to your last message. I have dropped maintainership for this port due to a lack of time (as you can tell from this bug report). You previously offered to take up maintainership - still interested? I’m happy to help commit this update and maintainership change once you confirm. Thanks, Ben
Hi Ben, no worries about the delay. As for the patch, it's outdated - there's a newer version of gtk-gnutella out with some further upstream changes useful that haven't made the release yet, so when I have time, I'll need to either wait for another release, or add the latest version and the patches. So, I think the best thing to do is to close this without modification, but yeah, set the maintainer to me please. I don't actively use gtk-gnutella myself, but I know the guy who maintains the upstream project, so I tend to have an eye on its releases. Cheers, Jamie
You happy if I just commit this patch and add you as maintainer for now, and then you can submit further updates later?
Yeah, sure! Whatever's easier!
(In reply to Jamie Landeg-Jones from comment #13) Port is at version 1.2.2 so this PR can be closed. Feel free to open a PR to take maintainership. Cheers --- commit 5abb49b1898eb41d73499598e06a70aca0d3efa7 Author: Fernando Apesteguía <fernape@FreeBSD.org> Date: Sun Feb 27 17:53:48 2022 +0100 net-p2p/gtk-gnutella: update to 1.2.2 Reported by: portscout ---