Bug 247914 - net-p2p/gtk-gnutella - Update to 1.2.0
Summary: net-p2p/gtk-gnutella - Update to 1.2.0
Status: Closed Overcome By Events
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: freebsd-ports-bugs (Nobody)
URL:
Keywords: buildisok, patch
Depends on:
Blocks:
 
Reported: 2020-07-11 18:51 UTC by Jamie Landeg-Jones
Modified: 2022-07-17 00:30 UTC (History)
4 users (show)

See Also:


Attachments
cd /usr/ports/net-p2p/gtk-gnutella && patch < <filename> (1.17 KB, patch)
2020-07-11 18:51 UTC, Jamie Landeg-Jones
no flags Details | Diff
New patch. Obsoletes previous one. patch -p0 < .... (6.11 KB, patch)
2020-09-26 07:19 UTC, Jamie Landeg-Jones
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jamie Landeg-Jones 2020-07-11 18:51:59 UTC
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
Comment 1 Automation User 2020-07-26 00:39:51 UTC
Build info is available at https://gitlab.com/swills/freebsd-ports/pipelines/170786666
Comment 2 Jamie Landeg-Jones 2020-08-31 20:36:45 UTC
I'll take ownership if this is abandoned...
Comment 3 Kurt Jaeger freebsd_committer freebsd_triage 2020-09-01 09:06:34 UTC
testbuilds@work
Comment 4 Kurt Jaeger freebsd_committer freebsd_triage 2020-09-01 12:00:31 UTC
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 ?
Comment 5 Jamie Landeg-Jones 2020-09-02 22:25:19 UTC
Sorry about that. I'll investigate, and speak to the author.

Cheers, Jamie
Comment 6 Ben Woods freebsd_committer freebsd_triage 2020-09-21 23:00:24 UTC
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
Comment 7 Jamie Landeg-Jones 2020-09-26 07:18:39 UTC
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
Comment 8 Jamie Landeg-Jones 2020-09-26 07:19:52 UTC
Created attachment 218312 [details]
New patch. Obsoletes previous one. patch -p0 < ....
Comment 9 Jamie Landeg-Jones 2021-03-05 22:52:40 UTC
ping???
Comment 10 Ben Woods freebsd_committer freebsd_triage 2021-09-09 22:55:44 UTC
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
Comment 11 Jamie Landeg-Jones 2021-09-10 12:01:15 UTC
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
Comment 12 Ben Woods freebsd_committer freebsd_triage 2021-09-11 01:15:02 UTC
You happy if I just commit this patch and add you as maintainer for now, and then you can submit further updates later?
Comment 13 Jamie Landeg-Jones 2021-09-11 03:53:17 UTC
Yeah, sure! Whatever's easier!
Comment 14 Nuno Teixeira freebsd_committer freebsd_triage 2022-07-17 00:30:15 UTC
(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
---