This is an archive of the discontinued LLVM Phabricator instance.

Differential D106416

[analyzer] Fix build dependency issues for SATest
ClosedPublic

Authored by manas on Jul 20 2021, 7:31 PM.

Details

Reviewers
teemperor
vsavchenko
NoQ
xazax.hun
Summary

It comprises of two specific changes:

  1. Fixes cmake's unmet dependency by upgrading it.
  2. Adds ccache to the build dependency list, which can be used for LLVM_CCACHE_BUILD cmake option for boosting rebuilds.

Diff Detail

Unit TestsFailed

TimeTest
60 msx64 debian > ORC-x86_64-linux.TestCases/Linux/x86-64::trivial-cxa-atexit.S
90 msx64 debian > ORC-x86_64-linux.TestCases/Linux/x86-64::trivial-static-initializer.S
100 msx64 debian > ORC-x86_64-linux.TestCases/Linux/x86-64::trivial-tls.S

Event Timeline

manas created this revision.Jul 20 2021, 7:31 PM
Herald added a reviewer: teemperor. · View Herald TranscriptJul 20 2021, 7:31 PM
Herald added subscribers: steakhal, ASDenysPetrov, dkrupp and 7 others. · View Herald Transcript
manas requested review of this revision.Jul 20 2021, 7:31 PM
Herald added a project: Restricted Project. · View Herald TranscriptJul 20 2021, 7:31 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
manas added reviewers: vsavchenko, NoQ, xazax.hun.Jul 20 2021, 7:32 PM
Herald added a subscriber: rnkovacs. · View Herald TranscriptJul 20 2021, 7:32 PM
Harbormaster completed remote builds in B115238: Diff 360345.Jul 20 2021, 8:16 PM
vsavchenko accepted this revision.Jul 22 2021, 2:01 AM

Awesome! Thanks!

This revision is now accepted and ready to land.Jul 22 2021, 2:01 AM
manas updated this revision to Diff 363572.Aug 2 2021, 2:15 PM

upgrade cmake-3.21.0 to cmake-3.21.1

manas added a comment.Aug 2 2021, 2:16 PM

Fixing another broken dependency packages error.

Harbormaster completed remote builds in B117521: Diff 363572.Aug 2 2021, 2:49 PM
manas updated this revision to Diff 375017.Sep 25 2021, 1:04 AM

Upgrade cmake to 3.21.3

manas added a comment.Sep 25 2021, 1:04 AM

@vsavchenko should we land this?

Harbormaster completed remote builds in B125687: Diff 375017.Sep 25 2021, 1:21 AM
manas closed this revision.Apr 10 2022, 2:09 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 10 2022, 2:09 PM

Revision Contents

PathSize
clang/
utils/
analyzer/
5 lines

Diff 375017

clang/utils/analyzer/Dockerfile

Show All 11 Lines
RUN apt-add-repository -y 'deb https://apt.kitware.com/ubuntu/ bionic main' RUN apt-add-repository -y 'deb https://apt.kitware.com/ubuntu/ bionic main'
# test system dependencies # test system dependencies
RUN apt-get update && apt-get install -y \ RUN apt-get update && apt-get install -y \
git=1:2.17.1* \ git=1:2.17.1* \
gettext=0.19.8.1* \ gettext=0.19.8.1* \
python3=3.6.7-1~18.04 \ python3=3.6.7-1~18.04 \
python3-pip=9.0.1-2.3* \ python3-pip=9.0.1-2.3* \
cmake=3.20.5* \ cmake=3.21.3* \
ninja-build=1.8.2-1 ninja-build=1.8.2-1 \
ccache=3.4*
# box2d dependencies # box2d dependencies
RUN apt-get install -y \ RUN apt-get install -y \
libx11-dev=2:1.6.4-3* \ libx11-dev=2:1.6.4-3* \
libxrandr-dev=2:1.5.1-1 \ libxrandr-dev=2:1.5.1-1 \
libxinerama-dev=2:1.1.3-1 \ libxinerama-dev=2:1.1.3-1 \
libxcursor-dev=1:1.1.15-1 \ libxcursor-dev=1:1.1.15-1 \
libxi-dev=2:1.7.9-1 libxi-dev=2:1.7.9-1
▲ Show 20 Lines • Show All 41 Lines • Show Last 20 Lines