> I know, the problem here is that we are calling _applyMedia (via > methodstr editing dialog) > referring to ayum object from ayum's __init__ (-> traceback). > > Another solution I tried was to move all the scanMedia thing from the > _applyMedia > method to configBaseURL (where it belongs I think - the dialog should > only change methodstr in case of configBaseURL fail). It required > quite a lot of code to be changed that would conflict with repo editing > dialog > flow in that stage of patches [4/14]. > > Yet another option was to pass the value to the dialog as parameter but I > didn't like it - it seemed less transparent and more ugly than using > "/mnt/source" to me, but I can do it this way. Yeah, that's a bit of a mess. One option might be for us to stop doing so much in AnacondaYum.__init__. You could make the argument that all you really should be doing in the __init__ method is initializing variables. All the configBaseURL and doConfigSetup stuff could perhaps get moved into another method. - Chris _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list
