Comment 4 for bug 487009

Revision history for this message
Michael Nelson (michael.nelson) wrote :

<BjornT_> noodles775: it seems a bit odd to adapt a job type. adapting an actual job seems more natural
<noodles775> BjornT_: yes - that would be much better (I'd headed down that path as I initially had a factory there).
<noodles775> BjornT_: although, that would then require adding a factory that knows about the different types, wouldn't it?
<noodles775> That was what I was trying to avoid too (having multiple places that need to know about the different types)
<BjornT_> noodles775: no. you can mark the job as providing the right interface, depending on which job type it is
<noodles775> BjornT_: ah, great!
<BjornT_> noodles775: you can see an example in BugTask._init()
* noodles775 looks
<bigjools-afk> noodles775: same sort of thing happens in Archive.__init__
<noodles775> Right.