mirror of
https://github.com/qbittorrent/qBittorrent.git
synced 2026-03-02 22:57:32 -05:00
Move plugins to another folder? #4731
Labels
No labels
Accessibility
AppImage
Bounty
Build system
CI
Can't reproduce
Code cleanup
Confirmed bug
Confirmed bug
Core
Crash
Data loss
Discussion
Docker
Documentation
Duplicate
Feature
Feature request
Feature request
Feature request
Filters
Flatpak
GUI
Has workaround
I2P
Invalid
Libtorrent
Look and feel
Meta
NSIS
Network
Not an issue
OS: *BSD
OS: Linux
OS: Windows
OS: macOS
PPA
Performance
Project management
Proxy/VPN
Qt bugs
Qt6 compat
RSS
Search engine
Security
Temp folder
Themes
Translations
Triggers
Waiting diagnosis
Waiting info
Waiting upstream
Waiting web implementation
Watched folders
WebAPI
WebUI
autoCloseOldIssue
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/qBittorrent#4731
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @sledgehammer999 on GitHub (Sep 23, 2016).
Originally assigned to: @sledgehammer999 on GitHub.
Hey now that I pushed
github.com/qbittorrent/qBittorrent@b3a7954363we can move the plugin folder around in the repo starting with 3.4.0.Starting with 3.3.8 all version till 3.4.0 will include the new URL, so hopefully we will break very few older clients.
@sledgehammer999 commented on GitHub (Sep 23, 2016):
@qbittorrent/qbittorrent-frequent-contributors is there any place to move in the folder structure?
I vaguely remember that @glassez wanted to move it around in the past.
PS: This is a discussion. If the current position is ok, I don't have a problem closing this issue without modification.
@Chocobo1 commented on GitHub (Sep 24, 2016):
maybe host it in another git repo?
then its possible to promote others to have write access.
@ngosang commented on GitHub (Oct 3, 2016):
+1
@glassez commented on GitHub (May 9, 2017):
IMO, leaving only plugin engine in qBittorrent is good idea. We should get rid from "official" plugins. Someone can manages separate plugins repo.
@glassez commented on GitHub (May 9, 2017):
I would also replace nova3 based plugin engine with some common plugin interface that allow call any compatible executable.
@zeule commented on GitHub (May 9, 2017):
Makes sense, +1. Any script language can be used then.
@sledgehammer999 commented on GitHub (May 9, 2017):
OK, I leave this open in tab. I'll create a repo later. Do you guys have name sugggestions? eg
search-plugins?@zeule commented on GitHub (May 9, 2017):
qBittorrent-search-pluginsto be in line with the web-site repo.@Chocobo1 commented on GitHub (May 9, 2017):
I was hoping @ngosang could adopt the current plugins in his own repo... if this is not too much to ask.
@sledgehammer999 commented on GitHub (May 9, 2017):
qBittorrent-search-pluginsSome reason this looks ugly in my eyes.
I could make him admin in the new repo...
@Chocobo1 commented on GitHub (May 9, 2017):
I mean moving to an unoffical repo
@glassez commented on GitHub (May 9, 2017):
+1
@glassez commented on GitHub (May 9, 2017):
We shouldn't distribute any plugins with qBittorrent.
@glassez commented on GitHub (May 9, 2017):
And we shouldn't have official plugins repo, of course.
@sledgehammer999 commented on GitHub (May 9, 2017):
I am not being negative, just curious. Why shouldn't we distribute plugins or have official repo?
The reviews I have read in the past indicated that his was the biggest advantage of this client.
@thalieht commented on GitHub (May 9, 2017):
Plausible deniability i guess.
@Chocobo1 commented on GitHub (May 9, 2017):
spot on.
Personally, I don't want anyone to think we are affiliated with any of those tracker operations.
@glassez commented on GitHub (May 9, 2017):
I don't think that it will greatly reduce the advantage, if plugins are distributed separately.
@Chocobo1 commented on GitHub (May 9, 2017):
I understand, we won't drop any of its functionality, we just let users install it manually, that's all.
@sledgehammer999 commented on GitHub (May 9, 2017):
I had missed https://github.com/qbittorrent/qBittorrent/pull/6762#issuecomment-300154738
Personally, I find it dubious legally that we are responsible on what ours users do. Google isn't held responsible for their search results. But in any case, I won't oppose this. It is less work for me and probably others.
And on the other hand it will allow the propagation of plugin fixes and inclusion to be more rapid.
Question is how far should we go?
http://searchplugins.qbittorrent.orgto redirect to plugins. Alternatively make the URL user configurable. User inputs URL of repo/site/service that has the expected layout by the search engine to find/update plugins. Also a link to wiki explaining stuff and having links to such repos/sites/services.Finally, even if we agree to any of the above, should we first wait for someone to explicitly adopt the plugin repo or just delete them from our repo?
@Chocobo1 commented on GitHub (May 10, 2017):
Yes
Yes
Yes
This is too much work for little gain, IMO the installed plugins should give qbt an URL to update/check version (maybe in the future).
Maybe not... too much freedom in this case, could bring up issues like: i18n translations, different term for the same categories (e.g. books, ebooks).
We should maintain a predefined set.
Either way will work, the new maintainer can checkout an older version of the master branch and get the plugins.
@glassez commented on GitHub (May 10, 2017):
You're right... So I've updated my answers.
We should delete them from our repo/distributions first. I don't think that creating a repository of plugins is too difficult and long. Moreover, we have someone who is willing to do it. @ngosang, isn't it? I hope he even make it to 3.4 final release.
@ngosang commented on GitHub (May 13, 2017):
Sorry, I'm late..
I don't care about the core, but the plugins should be in powerful scripting language like Python, Lua... I can combine nova2 & nova3 code to make it compatible with Python2 and Python3 so we don't need to duplicate code...
I prefer an official repo with admin permissions. That will make plugins more trustworthy..
I'm strongly opposed, as sledgehammer said, this is the only feature that makes qBittorrent unique. Things should work out the box, without manual installing. Today the search feature is already disabled by default, if you ask the user to manually install plugins this won't be used by anyone...
I'm not going to maintain plugins if they are so hidden that no body uses it. I can make my own scripts for that...
We aren't having legal throubles because we are not supporting piracy, just using some public sites to find legit torrents. We are not liable for the content of these sites and we are not making any profit with that.
@ArionMiles commented on GitHub (May 17, 2017):
This is worth discussing over. @Chocobo1 initially wanted to remove the plugins because of what happened to PopCorn Time, but those guys were directly promoting piracy through their tool. It isn't the same for qBit.
OTOH, While we do not have legal troubles now, there may be a time in future, though.
@ngosang commented on GitHub (May 21, 2017):
The must send you a DCMA requesting the removal of torrents/features before legal actions. If you comply and won't be any trial.
@ngosang commented on GitHub (May 21, 2017):
@qbittorrent/frequent-contributors any more thoughts about this topic? The development of 2 plugins is hold on until we have an agreement on that.
@Chocobo1 commented on GitHub (May 21, 2017):
I can agree we move to another repo (official one or not) as a first step.
@glassez commented on GitHub (May 21, 2017):
At least I would manage all plugins the same way (i.e. I wouldn't distribute official plugins inside app binaries) and I would move it outside the main app sources tree.
@glassez commented on GitHub (May 21, 2017):
I mean, they should be installed by app installer (if we choose to distribute it with app) or by app itself from repo.
@ngosang commented on GitHub (May 21, 2017):
What about a build-in downloader in qBittorrent? The installer wouldn't contains any plugin. The Search plugins list will show all "official" plugins and let you install with one click.
@MadeOfMagicAndWires commented on GitHub (May 21, 2017):
Sorry to chime in as a unofficial plugin creator but I agree with @ngosang that making installing plugins a completely manual process or hiding them makes it more or less useless. People will at that point just switch to searching on the sites instead.
The problem with Popcorn Time and others is that it's software tailored towards TV Series and Films. There are no legit downloads for TV series or films (save like 5).
qBittorrent doesn't have that same focus and even if some of the sites these plugins are made for also host torrent files to copyrighted material, they can still be solely used searching for legit content.
Have a repostory; just don't add any plugins that are tailored towards copyrighted material like you've always done.
I may misinterpret what you're going for, but as far as nova is concerned, I'm not opposed to moving away from it on principle; but it's been nice having the prettyPrinter and helper functions.
I think it'd be hard to provide that same utility with anything that is still also fully compatible with all sort of calls, apart from saving and reading search results from files in some universal format like json.
By not having those helper functions, you then make things less accessible to less experienced users from contributing by making new search engines themselves.
(edited for better reading)
@glassez commented on GitHub (May 22, 2017):
I meant to give up the current 3 tier architecture (qBittorrent manages plugins via nova3), but leave it (nova3) as a helper library. But this far-reaching plans...
Now I propose to do the following (assuming we still have the official plug-ins):
@ArionMiles commented on GitHub (May 22, 2017):
If we're doing this, can we at least have the Search tab enabled by default? There's no way newer users will know about the feature, much less use it, if they don't see it.
@glassez commented on GitHub (May 22, 2017):
Maybe it makes sense... but how this your offer is related to plugin distribution method?
@ArionMiles commented on GitHub (May 22, 2017):
It doesn't. It just came up in my mind so I made a suggestion.
@sledgehammer999 commented on GitHub (May 29, 2017):
I had been reading this thread and thinking. Finally, I came here to write my decision and I discover to my amazement that already the general consensus is in the same direction.
Here is the decision:
This is off by design. First time users on Windows will get multiple messageboxes. One to agree to license(?) and one to install Python. Seeing the message about Python will confuse them that it is a hard dependency of qBittorrent and that it will not function properly without it.
IMO, the search function is already quite popular without having that tab on by default. So I don't see any reason to change it. Users, aren't THAT dumb. They take a look at the menus.
@glassez commented on GitHub (May 29, 2017):
Agree with @sledgehammer999 plan.
I have some questions. Who will perform this changes? Do you want to make it yourself? Or someone can help you? Or we should split this task into several jobs? IIRC, this issue is only blocking v3.4 release.
@sledgehammer999 commented on GitHub (May 29, 2017):
@glassez, I'll do it myself. Today, I am starting to cherry-pick commits for v3.3.13. Then I'll do this(while building). And then 3.4.0 beta can proceed.
@ngosang commented on GitHub (May 29, 2017):
But plugins could be downloaded from qBittorrent with one click? Or the user has to open the Browser and install plugins manually?
@sledgehammer999 commented on GitHub (May 30, 2017):
In the search tab there will be informative text for the user to click the "Search plugins..." button.
In that dialog the user can click the "Update" button and get any plugins available in the repo.
Behavior will be changed to ensure that auto-update doesn't happen before the user explicitly opens and downloads the list first.
Does the above make sense?
@ngosang commented on GitHub (May 31, 2017):
yes
@sledgehammer999 commented on GitHub (Jun 1, 2017):
Take a look at #6883
@sledgehammer999 commented on GitHub (Jun 25, 2017):
Code merged.
New repo created: https://github.com/qbittorrent/search-plugins
Subdomain updated too. (GoDaddy servers are unresponsive though atm.)
I added @Chocobo1 and @ngosang as admins to that repo. If you don't want then remove yourself. As admins you can accept new people with
writeprivileges as you fit. And of course you can manage that repo.I opted for
search-pluginsas the name of the repo without adding "qBittorrent" in it, in an attempt to denote it as not an official list.