spacemit: shellfmt again after quoting fixes

[rpardini]

spacemit: shellfmt again after quoting fixes

• spacemit: revert shellfmt that messed up
  • revert spacemit.conf changes from 2f63a9c
• spacemit: apply quoting to fix more glaring problems
  • add a TODO for the original author of a "if $(...)" statement which I can't understand; it's still wrong
• spacemit: shellfmt again after quoting fixes
  • problematic if $(...) remains; just indent changed

[rpardini]

@The-going identified the mess introduced by shellfmt. I've reverted the shellfmt, fixed a few quoting issues, and re-ran it. Thanks @The-going for finding and reporting it

[The-going]

spacemit: shellfmt again after quoting fixes

shellfmt script\program is most likely an assistant for a person who can find typos and suggest something,
but he is very often mistaken.
I interpret everything that shellfmt writes\prompts like this:
"Hey, man, I think there's something wrong with this place. Pay attention to this."
But no more.

I'll check the changes in more detail tomorrow.

[The-going]

I understand your desire to make the script better.
But! But this is a system file!
And if we make any changes, we are obliged to test these changes on various
possible operating systems.

I do not know for sure in which environment this file will be executed.
With which environment variables and how a completely specific
operating system shell will handle the disclosure of double surrounding quotes.
This file was written by two people and checked each other after making changes.
Personally, I tested it in debian and openSUSE.