debian-setup: Install clang-8, lld-8 and llvm-8 with --no-install-rec…

[nickdesaulniers]

…ommends option

Check via:

$ apt-get install clang-8 lld-8 -t llvm-toolchain -s

$ apt-get install clang-8 lld-8 llvm-8 --no-install-recommends -t llvm-toolchain -s

CC: Nick Desaulniers
CC: Nathan Chancellor
Suggested-by: Sylvestre Ledru
Signed-off-by: Sedat Dilek

[nathanchance]

Seems reasonable. I'm testing the same change on all apt-get install commands in the Dockerfile to see how much it slims up the image: https://travis-ci.com/nathanchance/dockerimage/builds/92048750

[nathanchance]

Holy moly --no-install-recommends cut the size of the image down by 50%...

ClangBuiltLinux/dockerimage@23d6973

https://hub.docker.com/r/nathanchance/cbl/tags/

https://hub.docker.com/r/clangbuiltlinux/debian/tags/

I'll run some tests now to see if anything is broken but that's a really good change imo.

[nickdesaulniers]

Want me to wait until you're done testing to merge? If not, please just merge away.

[nathanchance]

This can go in now, I've locally rebased #62 on top of this and have made the changes to the Dockerfile, which I will push for a pull here soon. Going to post an update on #62 shortly.

[dileks]

Great to hear this reduces the image sizes. Not sure if this is a wise and equivalent replacement... [ /etc/apt/apt.conf or better /etc/apt/apt.conf.d/01norecommend] APT::Install-Recommends "false"; The man-page of apt-get says: All command line options may be set using the configuration file, the descriptions indicate the configuration option to set. For boolean options you can override the config file by using something like -f-,--no-f, -f=no or several other variations.

…

--no-install-recommends Do not consider recommended packages as a dependency for installing. Configuration Item: APT::Install-Recommends. Don't know if you can put a list of package-names explicitly as the above setting is active for all 'apt-get install <pkgname>' operations. Bonne chance, - sed@ -

On Tue, Nov 20, 2018 at 9:58 PM Nathan Chancellor ***@***.***> wrote: This can go in now, I've locally rebased #62 <#62> on top of this and have made the changes to the Dockerfile, which I will push for a pull here soon. Going to post an update on #62 <#62> shortly. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#67 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAsJyPFgvXp8sziPeZqiN8ZV3e9GHD6mks5uxGz3gaJpZM4YrmR3> .