[tor-bugs] #12868 [metrics-lib]: cyclic dependency

[Tor Bug Tracker & Wiki]

#12868: cyclic dependency
-----------------------------+---------------------
     Reporter:  iwakeh       |      Owner:  karsten
         Type:  defect       |     Status:  new
     Priority:  normal       |  Milestone:
    Component:  metrics-lib  |    Version:
   Resolution:               |   Keywords:
Actual Points:               |  Parent ID:
       Points:               |
-----------------------------+---------------------

Comment (by iwakeh):

 Why not make the decision about the actual Parser/Reader/Downloader at
 runtime?

 The class names could be provided as properties and the
 DescriptorSourceFactory
 loads the named classes defaulting to the current ones in case the
 properties are missing.

 Thus, the factory doesn't need to now the impl package at compile time.

 It doesn't seem like changing Parser/Reader/Downloader was intended from
 the beginning of metric-lib,
 but might it be a good idea to provide modularization now?

--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/12868#comment:3>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online