Opened 13 years ago

Closed 8 years ago

Last modified 6 years ago

#1974 closed patch (Fixed)

Core shouldn't have to know about the clients in advance

Reported by: jumentous Owned by:
Priority: minor Milestone: 2.0.0
Component: Unknown Version: master
Keywords: Cc: jamielennox@gmail.com, Bro

Description

So, #1971, #1972, #1973 are bugs that were discovered in the trying to fix this point.

I am looking at adding a new UI to deluge, and in the process noticed that the core currently matches the chosen UI and launches the appropriate client. This makes it impossible to add a client developed out of the deluge tree.

This patch modifies setup to export the UI classes as entry points and load the entry point for the class. Details for this method can be found here: http://aroberge.blogspot.com/2008/12/plugins-part-6-setuptools-based.html and http://peak.telecommunity.com/DevCenter/PkgResources#locating-plugins

Note that it doesn't actually do a find_plugins yet, it's just moving towards it.

Now i realize that there is already a plugin format for deluge however i thought this more appropriate for something that will be relatively static and can be deployed by rpm/deb.

Obviously, comments/revisions are encouraged.

Attachments (1)

0001-Decouple-GUI-selection-from-core.patch (17.6 KB ) - added by jumentous 13 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by jumentous, 13 years ago

Cc: jamielennox@gmail.com added

I've put the branch on github for the patches #1971 #1972 #1973 and this under detach-ui

https://github.com/jamielennox/deluge/tree/detach-ui

comment:2 by Calum, 13 years ago

Milestone: Future1.4.0
Type: bugpatch
Version: other (please specify)git master

comment:3 by Bro, 9 years ago

Cc: Bro added

Add bro to cc

comment:4 by Calum, 8 years ago

Milestone: 2.0.x2.0
Resolution: Fixed
Status: newclosed

Merged in develop: [6343f32d70408ba6ab]

comment:5 by Calum, 6 years ago

Milestone: 2.02.0.0

Milestone renamed

Note: See TracTickets for help on using tickets.