Alan Maguire wrote:
> Michael Hunter wrote:
>> Recently I've been starting up a virtual box with
>> network/physical:default enabled and network/physical:nwam disabled and
>> then later temporarily disabling default and temporarily enabling
>> nwam.  This fails our exclusion test.
>>
>>   
> I suspect the issue here is that temporary disable
> doesn't effect the general/enabled value (which is what
> we base our mutual exlusivity test off) - rather
> it masks the general/enabled value with a
> general_ovr/enabled property that the restarter
> uses to dictate state if it is defined. The solution
> is to ensure that the nwam_is_enabled() and
> physical_default_is_enabled() tests in the start
> methods of default and nwam also consult the
> general_ovr/enabled value for state, using its
> value if it's there, otherwise falling back to
> general/enabled. I'll fix this in my next wad.
Yep, I remember talking about this during the MPK week.  Didn't make 
note of anywhere and skipped the dev tasks.  Thanks for the quick fix Alan.

Anurag


Reply via email to