https://bugzilla.novell.com/show_bug.cgi?id=679435
https://bugzilla.novell.com/show_bug.cgi?id=679435#c35
Lukas Ocilka changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEEDINFO |NEW
InfoProvider|locilka@novell.com |
--- Comment #35 from Lukas Ocilka 2011-03-21 16:00:32 UTC ---
IMO the proposed fix should work for this particular case but it's not
generic enough. A few lines above, you have:
--- cut ---
if ...
ProductControl::DisableModule ("add-on");
if ...
ProductControl::EnableModule ("productsources");
ProductControl::DisableModule ("root");
ProductControl::DisableModule ("user");
ProductControl::DisableModule ("user_non_interactive");
ProductControl::DisableModule ("auth");
--- cut ---
Additionally, after that AdjustAutomaticConfiguration() is called
that contains:
--- cut ---
ProductControl::DisableAllModulesAndProposals ("installation", "continue");
ProductControl::EnableModule ("automatic_configuration");
EnableRequiredModules();
or
ProductControl::UnDisableAllModulesAndProposals ("installation", "continue");
ProductControl::DisableModule ("automatic_configuration");
--- cut ---
IMO we should do something like this
--- cut ---
if (Mode::autoinst()) {
InitAC(false); // Make it disabled
ClearAllDisabledModulesAndProposals();
} else {
SetModules() // Enabled / Disabled
InitAC();
AdjustAutomaticConfiguration();
}
--- cut ---
Plus AdjustAutomaticConfiguration() should contain:
--- cut ---
if (Mode::autoinst()) {
y2milestone ("AutoYaST, ignoring AC...");
// Maybe even: ClearAllDisabledModulesAndProposals();
return true;
}
--- cut ---
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.