On 8/27/20 11:43 AM, Alex Kiernan wrote: > This reverts commit 0f036bf4b87e6416f5c4d23865a62a62d9073c20. > > With this change applied and CONFIG_ENV_ACCESS_IGNORE_FORCE disabled, > the warning appears on every force overwrite, but the variable is then > written to: > > => env print ethaddr > ethaddr=00:1C:2B:08:AF:65 > => env set ethaddr 00:00:00:00:00:00 > ## Error: Can't overwrite "ethaddr" > ## Error inserting "ethaddr" variable, errno=1 > => env print ethaddr > ethaddr=00:1C:2B:08:AF:65 > => env set -f ethaddr 00:00:00:00:00:00 > ## Error: Can't force access to "ethaddr" > => env print ethaddr > ethaddr=00:00:00:00:00:00 > > Signed-off-by: Alex Kiernan <alex.kier...@gmail.com> > --- > As I noted in my email, I can't see an immediately obvious way to make > this work as intended and given we're at -rc3, I think a revert is the > most appropriate way forward.
Can you please spend some more time on proper fix, instead of just reverting ? Reverting isn't the way forward most of the time.