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

Maybe use of Debian image would be better because lighter ? #68

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Maybe use of Debian image would be better because lighter ? #68

wants to merge 2 commits into from

Conversation

tigerblue77
Copy link

Just an idea...
Don't hesitate to tell me if I'm wrong and why. Also, I removed "unrar" package as it is not available on default Debian repositories ; if it was necessary (I didn't see where) but "tar" command could be use without needing any package installation.

"apt does not have a stable CLI interface. Use with caution in scripts."
Use of the Debian 11- image (bullseye), lighter than Ubuntu and removal of "unrar" package, not available in the official Debian repositories and which does not seem to be used for the creation of the Docker image
@gbooker
Copy link
Contributor

gbooker commented Oct 13, 2021

Don't hesitate to tell me if I'm wrong and why.

This change is unlikely to go forward because if we change the base, most likely it will be to alpine (something we have investigated but it's in a queue behind a bunch of other things).

I removed "unrar" package as it is not available on default Debian repositories ; if it was necessary (I didn't see where) but "tar" command could be use without needing any package installation.

See #33 (can be found by using blame on the Dockerfile and examining the commit. It links to that PR).

@tigerblue77
Copy link
Author

Ok, thanks for your answer ! It could be a temporary improvment before switching to Alpine.
I saw that unrar could be installed on Debian by activating non-free packages : https://packages.debian.org/bullseye/unrar
Have a nice day !

@Re4zOon
Copy link

Re4zOon commented Oct 20, 2022

Hi,

Just a quick question: Why are you using apt-get instead of apt?
Is it just preference, or?
I dont see the reason to use the old/outdated apt-get anymore, specially on fresh distros.

@tigerblue77
Copy link
Author

Hi,

Just a quick question: Why are you using apt-get instead of apt? Is it just preference, or? I dont see the reason to use the old/outdated apt-get anymore, specially on fresh distros.

Hello,
"old/outdated apt-get" ? Sorry but I think you're wrong. apt is using apt-get and is just the recommended command for interactive use by administrators, not for use in shell scripts. That's why, when you use apt in a bash script you get the following message :

WARNING: apt does not have a stable CLI interface. Use with caution in scripts.

@Re4zOon
Copy link

Re4zOon commented Oct 20, 2022

Sorry, you are right. I never really looked into the differences and whatnot.
I was coming from ansible, that has a force option to use apt-get, instead of apt.
I guess the ansible package actually prefers apt for some reason. In any case, I've yet to run into trouble.

Copy link

@Randyblo7 Randyblo7 left a comment

Choose a reason for hiding this comment

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

Nice

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.

4 participants