Skip to content

Dockerfile: use xx-apt-get instead of xx-apt - #3491

Merged
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:xx_apt_get
Mar 26, 2022
Merged

Dockerfile: use xx-apt-get instead of xx-apt#3491
thaJeztah merged 1 commit into
docker:masterfrom
thaJeztah:xx_apt_get

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

The apt command is meant to provide a user-friendly interface, but does not
have a stable interface, and not recommended for scripting, see:

#54 [linux/amd64 build-buster 1/1] RUN xx-apt install --no-install-recommends -y libc6-dev libgcc-8-dev
#54 0.706 Hit:1 http://security.debian.org/debian-security buster/updates InRelease
#54 0.707 Hit:2 http://deb.debian.org/debian buster InRelease
#54 0.708 Hit:3 http://deb.debian.org/debian buster-updates InRelease
#54 2.149 Reading package lists...
#54 4.917 + apt install --no-install-recommends -y libc6-dev libgcc-8-dev
#54 4.934
#54 4.934 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

This changes the command to use apt-get instead

- A picture of a cute animal (not mandatory but encouraged)

The `apt` command is meant to provide a user-friendly interface, but does not
have a stable interface, and not recommended for scripting, see:
docker#54 [linux/amd64 build-buster 1/1] RUN xx-apt install --no-install-recommends -y libc6-dev libgcc-8-dev
docker#54 0.706 Hit:1 http://security.debian.org/debian-security buster/updates InRelease
docker#54 0.707 Hit:2 http://deb.debian.org/debian buster InRelease
docker#54 0.708 Hit:3 http://deb.debian.org/debian buster-updates InRelease
docker#54 2.149 Reading package lists...
docker#54 4.917 + apt install --no-install-recommends -y libc6-dev libgcc-8-dev
docker#54 4.934
docker#54 4.934 WARNING: apt does not have a stable CLI interface. Use with caution in scripts.
This changes the command to use `apt-get` instead
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

codecov-commenter commented Mar 26, 2022

Copy link
Copy Markdown

Codecov Report

Merging #3491 (3a1fc21) into master (3897c3f) will not change coverage.
The diff coverage is n/a.

@@ Coverage Diff @@## master #3491 +/- ##
=======================================
Coverage 59.01% 59.01% =======================================
Files 284 284 Lines 23833 23833 =======================================
Hits 14066 14066 Misses 8908 8908 Partials 859 859 

@thaJeztah
thaJeztah merged commit a23dc4f into docker:masterMar 26, 2022
@thaJeztah
thaJeztah deleted the xx_apt_get branch March 26, 2022 15:48
Sign up for freeto 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.

3 participants

@thaJeztah@codecov-commenter@crazy-max