> > I just tried the new MAC stuff, the auto-assign thing is neat, but it
> > re-orders the MACs when it assigns them. For example:
>
> This should be fixed tomorrow.  It isn't re-ordering them, it's just
> assigning them backwards.

Ok, great. I realized it was going backwards once I had more than 2
entries, thanks.
As promised, this is now fixed.

> The uninitialized value errors are due to SIS code:
>
> Use of uninitialized value in concatenation (.) at
> /usr/lib/systeminstaller/SIS/Adapter.pm line 74.
>
> and they are spouted out generously while using the window.  Convince them
> to fix it and we'll all be happier.
>
> The offending line referenced is:
>
> $this->{_cache}->{_primkey} = $this->{_vars}->{_devname} . ":" .
> $this->{_vars}->{_client};

As discussed in another thread, the SIS calls are fixed in CVS, we just
haven't made a new release of SIS for it yet. The errors I was referring
to were slightly different and caused in MAC.pm. The add_mac_to_hash
routine was doing lots of comparisons on the mac, but it was often
getting called with an undefined mac when it wasn't in the database yet.
I checked a fix for this in earlier that only calls add_mac_to_hash if
the mac is defined.
SIS should put in the new release then!!! The errors you found are probably fixed now... but let me know if you find any more problems. I've tried to test all the erroneous cases (as well as reasonable ones) I could think of, so let me know if you find any additional issues with the Assign All function.

Jason



-------------------------------------------------------
This SF.NET email is sponsored by:
SourceForge Enterprise Edition + IBM + LinuxWorld = Something 2 See!
http://www.vasoftware.com
_______________________________________________
Oscar-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/oscar-devel

Reply via email to