cpp-linter / cpp-linter-action

A Github Action for linting C/C++ code integrating clang-tidy and clang-format to collect feedback provided in the form of file-annotations, thread-comments, workflow step-summary, and Pull Request reviews.

Home Page:https://cpp-linter.github.io/cpp-linter-action/

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

apt-get is missing -y and gets stuck due to interactive mode

lrishi opened this issue · comments

The following additional packages will be installed:
  clang-15 clang-tools-15 cpp-11 g++-11 gcc-11 gcc-11-base icu-devtools
  lib32gcc-s1 lib32stdc++6 libasan6 libc6-i386 libclang-common-15-dev
  libclang-cpp15 libclang1-15 libffi-dev libgc1 libgcc-11-dev libicu-dev
  libllvm15 libncurses-dev libobjc-11-dev libobjc4 libpfm4 libstdc++-11-dev
  libtinfo-dev libtsan0 libxml2 libxml2-dev libz3-4 libz3-dev llvm-15
  llvm-15-dev llvm-15-linker-tools llvm-15-runtime llvm-15-tools
  python3-pygments python3-yaml
Suggested packages:
  clang-15-doc wasi-libc gcc-11-locales g++-11-multilib gcc-11-doc
  gcc-11-multilib icu-doc ncurses-doc libstdc++-11-doc pkg-config llvm-15-doc
  python-pygments-doc ttf-bitstream-vera
The following NEW packages will be installed:
  clang-15 clang-format-15 clang-tidy-15 clang-tools-15 icu-devtools
  lib32gcc-s1 lib32stdc++6 libc6-i386 libclang-common-15-dev libclang-cpp15
  libclang1-15 libffi-dev libgc1 libicu-dev libllvm15 libncurses-dev
  libobjc-11-dev libobjc4 libpfm4 libtinfo-dev libxml2-dev libz3-4 libz3-dev
  llvm-15 llvm-15-dev llvm-15-linker-tools llvm-15-runtime llvm-15-tools
  python3-pygments python3-yaml
The following packages will be upgraded:
  cpp-11 g++-11 gcc-11 gcc-11-base libasan6 libgcc-11-dev libstdc++-11-dev
  libtsan0 libxml2
9 upgraded, 30 newly installed, 0 to remove and 50 not upgraded.
Need to get 196 MB of archives.
After this operation, 800 MB of additional disk space will be used.
Do you want to continue? [Y/n] Abort.

This was working until yesterday. :S

Strange, we didn't come accross this in our tests. I think this is usually set already by the runner image's env. Are you using an additional container?

Nonetheless, this is a good idea.

Yes, we use self hosted runners :S I'm going to try by adding DEBIAN_FRONTEND=noninteractive

Thanks for jumping on this quickly 🙏🏽

Yes, we use self hosted runners :S I'm going to try by adding DEBIAN_FRONTEND=noninteractive

I'm curious, did this work for you?

Yes, we use self hosted runners :S I'm going to try by adding DEBIAN_FRONTEND=noninteractive

I'm curious, did this work for you?

No, it didn't. setting that envvar across jobs and executions of sudo was just too much. Any way we can get the fix released soon? :D

Very soon. We just merged a revision to the thread-comments and added a step-summary feature. Now we just need to polish it off with #151.