hotfix - getting rid of sudos#16
Conversation
This reverts commit e347d91.
| if [ -z ${1+x} ]; then | ||
| echo "Working from main branch."; | ||
| else | ||
| # -z tests for zero length. |
There was a problem hiding this comment.
For the love of all holy, please, turn off your white space editing.
There was a problem hiding this comment.
It's not a big deal, but it makes reviewing edits a lot easier.
There was a problem hiding this comment.
There are no mods on the functions_library.sh file, so I can't see what you're seeing.
Second, I don't have any white space editing option thing going on my editor. I don't do space-editing, I always prefer tabs, and frankly, that's done automatically.
There was a problem hiding this comment.
Seems to have disappeared.
There was a problem hiding this comment.
I did yesterday night a reverse-commit so maybe you were seeing the old mods.
|
@RobertLucian have you tested this on DexterOS yet? |
|
I have tested it on my version of DexterOS and partly on Raspbian For Robots. I'll be doing more tests anyway, because I have to bring changes to all repos. |
|
Just so you know, historically speaking, the Pi required sudo for a lot of stuff, accessing the pins for example. That's why there are sudos all over the place. |
johnisanerd
left a comment
johnisanerd
left a comment
There was a problem hiding this comment.
Gave it a test on DexterOS, R4R, and Stretch and each seemed to work ok.
CleoQc
left a comment
CleoQc
left a comment
There was a problem hiding this comment.
tested on DexterOS and Raspbian.
Only tested GoPiGo3 platform though.
3 participants
By doing this, we can install the script_tools repo/package with the right privileges as specified by the install command.