-
Notifications
You must be signed in to change notification settings - Fork 55
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
Install Package Manager with apt instead of gdebi #290
Conversation
This has a few advantages: * We don't have to install (and later remove) gdebi. * apt allows us to skip installing Recommended packages. RSPM depends on git and git recommends "less" and "ssh-client" (yes, really), so we were previously installing an SSH client into the image. With this change neither of these packages nor their transitive dependencies are installed. This shaves about 8M off the resulting image, too. Signed-off-by: Aaron Jacobs <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The license manager doesn't seem to work with this change, as the Ubuntu 18 build of it requires libssl1.0.0
?
/opt/rstudio-pm/bin/../ext/activation/license-manager: error while loading shared libraries: libcrypto.so.1.0.0: cannot open shared object file: No such file or directory
I didn't realize/forgot that RSPM actually does require libssl via license-manager and other RSW dependencies - rsandbox, launcher. libssl1.0.0
seems to be coming in as a Recommends of git, so that's not great, we should make that an explicit dependency.
And also, the next release of RSPM on Ubuntu 18 uses RSW dependencies built against libssl1.1
, so there's some maintenance pain with including libssl manually (e.g. #285).
What do you think of keeping the gdebi change, but not using --no-install-recommends
for now? Then we could fix the dependency issues with the RSPM packages for the next release.
Ah, so really there was an issue with #285 it was just masked by getting pulled in with |
This should most likely be removed with future releases. Signed-off-by: Aaron Jacobs <[email protected]>
I think we want to keep the Docker installation as close to the admin guide as possible, because that's the path the RSPM team is testing. If we want to make a change limiting dependencies, it should happen in the main codebase. |
This would be nice to do. Using |
We should be able to do this now that Ubuntu 18.04 is EOL. |
|
Closing this out since we are using now
|
This has a few advantages:
We don't have to install (and later remove)
gdebi
.ap
t allows us to skip installing Recommended packages. RSPM depends ongit
andgit
recommendsless
andssh-client
(yes, really), so we were previously installing an SSH client into the image. With this change neither of these packages nor their transitive dependencies are installed.This shaves about 8M off the resulting image, too.