Skip to content

Fixes #39506 - remove invalid apt-get update option#1042

Closed
sbernhard wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:fix_39506
Closed

Fixes #39506 - remove invalid apt-get update option#1042
sbernhard wants to merge 1 commit into
theforeman:masterfrom
ATIX-AG:fix_39506

Conversation

@sbernhard

Copy link
Copy Markdown
Contributor

No description provided.

}
export DEBIAN_FRONTEND=noninteractive
apt-get -y update
apt-get update

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I disagree:

from the man page

      -y, --yes, --assume-yes
           Automatic yes to prompts; assume "yes" as answer to all prompts and run non-interactively. If an undesirable situation, such as changing a held package, trying to install an
           unauthenticated package or removing an essential package occurs then apt-get will abort. Configuration Item: APT::Get::Assume-Yes.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Isn't this for PKG install/ u install etc.?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Actually, I had the same impression than you. I had a look at the apt-get source and asked AI - the opton “-y” is not yet used in apt-get update. BUT, its is not YET used. Maybe somewhere in the future. It does not hurt right now - so, let is as it yes.

I will close this PR - Dont fix whats not broken!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants