Add $CONTAINER_USER to sudo users

esp-rs / esp-idf-template

A "Hello, world!" template of a Rust binary crate for the ESP-IDF framework.

372 stars 43 forks source link

Add $CONTAINER_USER to sudo users #175

Closed yagop closed 6 months ago

Vollbrecht commented 6 months ago

what is your reason here? Its generally bad practice to install sudo if its not needed. Normally the container should run as is, if you encounter any problems or missing a feature please tell us about it, and the reason it id needed.

yagop commented 6 months ago

Problem is it can't access /dev/ttyACM0 so need to run as root some commands.

It also help to install dependencies if needed

SergioGasquez commented 6 months ago

Thanks for the contribution but, I will be closing the PR as I don't think the template should provide sudo to the user, users can always modify the generated template for their specific purpose