search

DexterInd / Raspbian_For_Robots

Raspbian image changes to make it work well with Dexter Industries robots.
http://www.dexterindustries.com/raspbian-for-robots/
Other
32 stars 18 forks source link

Various Updates Including Pi3 WiFi #167

Closed johnisanerd closed 6 years ago

RobertLucian commented 6 years ago

This PR seems to contain more than the changes we've merged with #165. The other changes are related to #161, #162. For some reason, update is not up-to-date with master. This PR will sync them.

CleoQc commented 6 years ago

btw, @RobertLucian What about all those lines similar to sudo sh -c "curl -kL dexterindustries.com/update_sensors | bash" ?

Why call sh only to pipe into bash?

RobertLucian commented 6 years ago

@CleoQc we could have gone with sudo bash -c "curl -kL dexterindustries.com/update_sensors | bash" instead of sh, but at the moment, I can't remember why I chose sh instead of bash. Was your question about this?

CleoQc commented 6 years ago

why not just sudo curl -kL dexterindustries.com/update_sensors | bash

johnisanerd commented 6 years ago

So . . . merge.

RobertLucian commented 6 years ago

Oh, 'cause piping it to bash this way (the way you shown) will only make the script get executed without root privileges no matter what you prepend to your command. You either have got to login with the root user or launch it this way in order to inherit root's privileges.