For the record:
<pitti> curious - the place you patched out of osspec.c already tries to not
overwrite the value with 'unknown'
<pitti> this almost looks as if the inner if should be fixed
<pitti> if (strcmp
(hal_device_property_get_string (d, "system.formfactor"), "unknown"))
<pitti> -> to !strcmp
<pitti> and an alternative added, for the case if the property does not exist
yet
<pitti> anyway, I'm applying your patch for now, but can you please take
another look at this?
<pitti> it should be something like
<pitti> if (!hal_device_property_exists(formfactor) ||
!strcmp(h_d_p_get_string(formfactor), "unknown") { overwrite formfactor };
** Changed in: hal (Ubuntu)
Status: In Progress => Fix Committed
--
hal don't find that I'm on a laptop
https://launchpad.net/bugs/59342
--
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
