qBittorrent icon indicating copy to clipboard operation
qBittorrent copied to clipboard

Backport changes to v5.0.x branch

Open glassez opened this issue 1 year ago • 15 comments
trafficstars

  • #21220
  • #21295
  • #21480
  • #21481
  • #21488
  • #21528
  • #21553
  • #21596
  • #21628

glassez avatar Oct 04 '24 11:10 glassez

#21064

xavier2k6 avatar Oct 05 '24 16:10 xavier2k6

Is it possible to backport this small feature customizing the save statistics time interval #21291 into v5.0.x?

jibwf avatar Oct 06 '24 12:10 jibwf

@glassez I would suggest not backport #21064 and instead investigate the issue thoroughly. Backport should be the last resort.

Chocobo1 avatar Oct 06 '24 14:10 Chocobo1

I would suggest not backport #21064 and instead investigate the issue thoroughly. Backport should be the last resort.

No problems. I will remove it from here as soon as another solution is provided.

@Chocobo1 BTW, another quick solution that can come to my mind is to simply roll back all the basic scripts (nova2, helpers etc.) to the state of the 4.6.x branch.The situation is not so clear about the plugins themselves. It turns out that we need to keep them compatible with the release branch (aka v.5.0.x currently), and not just with the master.

glassez avatar Oct 06 '24 14:10 glassez

BTW, another quick solution that can come to my mind is to simply roll back all the basic scripts (nova2, helpers etc.) to the state of the 4.6.x branch.The situation is not so clear about the plugins themselves. It turns out that we need to keep them compatible with the release branch (aka v.5.0.x currently), and not just with the master.

It is already targeted to release branch (v.5.0.x) correctly. The plugins repo still run CI with older python versions.

No problems. I will remove it from here as soon as another solution is provided.

It is unclear what is the actual issue as I couldn't find a proper issue report about it and I couldn't reproduce either. I suspect it doesn't affect the masses but only a few (which is the usual with a new release). I mean, until the symptom is identified it is unlikely to have a 'fix'.

Chocobo1 avatar Oct 06 '24 16:10 Chocobo1

It is unclear what is the actual issue as I couldn't find a proper issue report about it and I couldn't reproduce either. I suspect it doesn't affect the masses but only a few (which is the usual with a new release). I mean, until the symptom is identified it is unlikely to have a 'fix'.

It sounds strange... This is one of the top issues since the v5.0 release. Just try to setup environment with clean qBittorrent v4.6.7 + Pyton v3.8.x (e.g. v3.8.10) and download official plugins (make sure that they are listed in qBittorrent plugins dialog). Then update it to v5.0. https://github.com/qbittorrent/qBittorrent/issues/21445#issuecomment-2395115393 https://github.com/qbittorrent/qBittorrent/issues/21445#issuecomment-2395118111

glassez avatar Oct 06 '24 17:10 glassez

@Chocobo1 It's not an upgrade issue anyway from 4.6.x -> 5.0.0, it can be re-produced with fresh profile/portable mode & python 3.8.10

  1. Download/extract qBittorrent 5.0.0
  2. Create profile folder
  3. Ensure python 3.8.10 is installed & only version installed!
  4. Run qBittorrent
  5. View->Search
  6. Select "Search plugins..."
  7. Check for updates
  8. Met with Couldn't install "xxxxxx" search engine plugin. Plugin is not supported.
  9. It simply will not install the 8x official plugins.
  10. Installing Python 3.9.0 by itself or selecting it from advanced options....resolves the issue.

xavier2k6 avatar Oct 06 '24 19:10 xavier2k6

@Chocobo1 It's not an upgrade issue anyway from 4.6.x -> 5.0.0, it can be re-produced with fresh profile/portable mode & python 3.8.10

Thanks, I think I found the root cause. I'll submit a fix when ready.

https://github.com/qbittorrent/qBittorrent/issues/21445#issuecomment-2395115393 Therefore, https://github.com/qbittorrent/qBittorrent/pull/21064 definitely needs to be backported to v5.0.x branch.

It is not necessary. I'll fix it another way. See PR #21539.

Chocobo1 avatar Oct 07 '24 04:10 Chocobo1

Is it possible to backport this small feature customizing the save statistics time interval #21291 into v5.0.x?

By default only bugfixes are eligible for backporting into current release branch. All questions are for @Chocobo1. He is the unspoken guardian of the release policy.

glassez avatar Oct 09 '24 12:10 glassez

Is it possible to add https://github.com/qbittorrent/qBittorrent/pull/21296 ?

stalkerok avatar Oct 10 '24 14:10 stalkerok

#21220? -> https://github.com/qbittorrent/qBittorrent/pull/21220#issuecomment-2345833787

xavier2k6 avatar Oct 11 '24 18:10 xavier2k6

#21220? -> #21220 (comment)

Done.

glassez avatar Oct 14 '24 08:10 glassez

Is it possible to add #21296 ?

@Chocobo1 ?

stalkerok avatar Oct 14 '24 09:10 stalkerok

https://github.com/qbittorrent/qBittorrent/pull/21300 ?

KyleSanderson avatar Oct 16 '24 01:10 KyleSanderson

#21300 ?

I don't mind. The question is up to @Chocobo1. https://github.com/qbittorrent/qBittorrent/pull/21496#issuecomment-2402201337

glassez avatar Oct 16 '24 05:10 glassez

#21300 ?

I'm ok with it.

Chocobo1 avatar Oct 21 '24 06:10 Chocobo1

Is it possible to add #21296 ?

@Chocobo1 ?

What about this one?

stalkerok avatar Oct 21 '24 07:10 stalkerok

Is it possible to add #21296 ?

@Chocobo1 ?

What about this one?

It isn't critical, so probably not. ps. I would only reply with the OK ones.

Chocobo1 avatar Oct 21 '24 08:10 Chocobo1

I pushed #21605 to this.

sledgehammer999 avatar Oct 21 '24 17:10 sledgehammer999