tldr-pages / tldr-c-client

C command-line client for tldr pages
MIT License
296 stars 49 forks source link

Assume yes for apt installation #40

Closed packrat386 closed 5 years ago

packrat386 commented 5 years ago

What does it do?

Runs apt-get install with the -y option to automatically approve installing new packages.

Why the change?

I'm trying to automate installing this package, but I can't run deps.sh from a script very easily because I have to say yes to allow apt to install new packages. I'm already running this as sudo, it seems fair to say that I want to install these packages.

How can this be tested?

Attempt to run this in a VM that uses apt where the necessary packages are not already installed. Note that the script runs completely without prompting the user to approve installation of new

Where to start code review?

Line 23 is the only line changed, I'd start there 😛

Relevant tickets?

None

Questions?

Nope 😃

Leandros commented 5 years ago

Thanks.