NoQ added inline comments.
================ Comment at: clang/utils/analyzer/Dockerfile:15 +# test system dependencies +RUN apt-get update && apt-get install -y \ + git \ ---------------- NoQ wrote: > Maybe put `apt-get update` on a separate line? > > Also i think you mentioned offline that you want to freeze package versions > here, can you add a FIXME? > can you add a FIXME? Wait, you already did that in D81600, nvm. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81571/new/ https://reviews.llvm.org/D81571 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
