Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hide apt-get update output #1049

Closed
wants to merge 1 commit into from
Closed

Conversation

yuvipanda
Copy link
Collaborator

apt-get update produces a lot of output that's mostly
irrelevant unless there is an error. This PR will suppress
stdout, but stderr will still show - so we can see errors
if they happen.

`apt-get update` produces a lot of output that's mostly
irrelevant unless there is an error. This PR will suppress
stdout, but stderr will still show - so we can see errors
if they happen.
@yuvipanda yuvipanda changed the title Hide apt-get update output [MRG] Hide apt-get update output Jun 29, 2021
@yuvipanda yuvipanda requested review from manics and betatim June 29, 2021 12:21
@@ -682,7 +682,7 @@ def get_preassemble_scripts(self):
"root",
# This apt-get install is *not* quiet, since users explicitly asked for this
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we remove this comment? Or does it not refer to the apt-get below?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@betatim the apt-get install is still not quiet. Just the apt-get update is

@manics
Copy link
Member

manics commented Jul 2, 2021

Is there some other Docker/apt config hiding somewhere? When I run apt-get -qq update interactively I don't get any output, whereas I do with apt-get update:

root@385cd7180d59:/# apt-get -qq update
root@385cd7180d59:/# apt-get update
Hit:1 http://archive.ubuntu.com/ubuntu bionic InRelease
Hit:2 http://archive.ubuntu.com/ubuntu bionic-updates InRelease
Hit:3 http://archive.ubuntu.com/ubuntu bionic-backports InRelease              
Hit:4 http://security.ubuntu.com/ubuntu bionic-security InRelease              
Reading package lists... Done                               

@consideRatio consideRatio changed the title [MRG] Hide apt-get update output Hide apt-get update output Oct 30, 2022
@consideRatio
Copy link
Member

I verified what @manics described, that with -qq as we already have, adding > /dev/null doesn't reduce something further - the output is blank.

(base) ➜  repo2docker-service git:(main) sudo apt-get -qq update
[sudo] password for erik: 
(base) ➜  repo2docker-service git:(main)

Closing this to tidy up our list of open PRs!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants