Bug 277151 - net-p2p/qbittorrent: Unable to build with net-p2p/librasterbar2 support
Summary: net-p2p/qbittorrent: Unable to build with net-p2p/librasterbar2 support
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 Only Me
Assignee: Yuri Victorovich
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2024-02-19 01:02 UTC by Henrich Hartzer
Modified: 2024-03-15 18:27 UTC (History)
1 user (show)

See Also:
bugzilla: maintainer-feedback? (yuri)


Attachments
net-im/qbittorrent: Fixes libtorrent-rasterbar2 build (2.22 KB, patch)
2024-03-01 16:59 UTC, Henrich Hartzer
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Henrich Hartzer 2024-02-19 01:02:09 UTC
I was hoping to test qbittorrent's Bittorrent v2 functionality, but found out that it ships with librasterbar v1. I attempted to compile the port myself, but however when I select librasterbar v2, I get this error in the process:

```
===>  Patching for qbittorrent-4.6.3
===>  Applying extra patch /usr/ports/net-p2p/qbittorrent/files/extrapatch-libtorrent-rasterbar2-settings
1 out of 1 hunks failed--saving rejects to src/base/bittorrent/sessionimpl.cpp.rej
===>  FAILED Applying extra patch /usr/ports/net-p2p/qbittorrent/files/extrapatch-libtorrent-rasterbar2-settings
*** Error code 1
```

I see this with 4.6.3 on main, and 4.6.2 on 2024Q1. I do wonder if 4.6.3 should be backported to 2024Q1 as well.

I'd also like to ask if librasterbar v2 should become the default, but have no experience with either v1 or v2 so certainly can't say myself.

Thank you!
Comment 1 Henrich Hartzer 2024-02-29 23:08:00 UTC
I removed the patch and was able to build 4.6.3 with net-p2p/librasterbar2. So far it seems to be working fine.

Not sure what the best approach is here.
Comment 2 Henrich Hartzer 2024-03-01 16:59:45 UTC
Created attachment 248851 [details]
net-im/qbittorrent: Fixes libtorrent-rasterbar2 build
Comment 3 Henrich Hartzer 2024-03-01 17:01:56 UTC
Still not sure about if libtorret-rasterbar2 should be the default or not, but the attached patch fixes the build with it.

I don't know the intent of the original patch which this removes, so there may be sideffects I don't know about. Initial testing has worked fine for me, however.

I did only test this with qbittorrent 4.6.3 and not 4.6.2, but suspect it works with both. I don't think there would be an issue in backporting 4.6.3 to quarterly if that might be a concern.
Comment 4 Daniel Engberg freebsd_committer freebsd_triage 2024-03-14 22:15:48 UTC
Hi,

https://cgit.freebsd.org/ports/commit/net-p2p/qbittorrent?id=388ecd7ffef779aeb1b8e867c13217c061def9ee should fix your issue as far as I can tell?

Best regards,
Daniel
Comment 5 Yuri Victorovich freebsd_committer freebsd_triage 2024-03-14 22:24:25 UTC
Hi Henrich,


> Still not sure about if libtorret-rasterbar2 should be the default or not, but the attached patch fixes the build with it.

Not until libtorret-rasterbar2 would solve the memory problem. It looks like they change strategy and now mmap all torrent files, which leads to an extremely high memory usage. It's fine to use libtorret-rasterbar2 when your torrents volume is much lower than your RAM, but otherwise torrent client would run in extremely high memory.

----------------

The commit that Daniel pointed out has already fixed the problem that you are reporting here.
Please update your ports tree.


Thanks,
Yuri
Comment 6 Henrich Hartzer 2024-03-15 18:27:08 UTC
Hi Yuri and Daniel,

Thank you for the information! That should work.

-Henrich