Radek Vykydal wrote:
Fixes editing of Install Repo, added repos, and methodstr dialog raised e.g. when incorrect repo= nfs option is added. Works with assumption that install (repo/method) url points to a single repo, which is true for Fedora (as opposed to RHEL5 where we various repos in directories). dgregor told me that the target for RHEL6 is to have just one repo and handle product differentiation by comps. If we had to work with multirepo install urls, the UI for repo editing would need to be redesigned and rewritten because now we are mixing install urls with repo urls there. Installation Repo is handled as other repos, not by using "Base" yuminstall functions which is incompatible with editing by means of creating temporary repo object. Our repo= url is stored in AnacondaYumRepo so that e.g nfs: type of target can be edited (instead of having mountpoint url in the dialog). I remember that we wanted to get rid of the class in yuminstall cleanup, but couldn't come with better solution. cdrom: repo option is still broken. Change to CD/DVD repo target in non-media install too. It may be next step that would probably involve refreshing of format information of the device. Change to CD/DVD in dialogs after booting from media is fixed. The patch is tested. --- gui.py | 4 +- iw/task_gui.py | 141 +++++++++++++++++++++++++++++++------------------------ yuminstall.py | 47 ++++++++++++------ 3 files changed, 112 insertions(+), 80 deletions(-)
I'll send patch broken into smaller pieces. Radek _______________________________________________ Anaconda-devel-list mailing list Anaconda-devel-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/anaconda-devel-list
