This is an archive of the discontinued LLVM Phabricator instance.

[analyzer] Fix build dependency issues for SATest
ClosedPublic

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

Download Raw Diff

Details

Reviewers

teemperor
vsavchenko
NoQ
xazax.hun

Summary

It comprises of two specific changes:

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

Diff Detail

Repository: rG LLVM Github Monorepo

Unit TestsFailed

	Time	Test
	4,420 ms	x64 debian > libarcher.races::task-dependency.c

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

Awesome! Thanks!

This revision is now accepted and ready to land.Jul 22 2021, 2:01 AM

upgrade cmake-3.21.0 to cmake-3.21.1

Fixing another broken dependency packages error.

Harbormaster completed remote builds in B117521: Diff 363572.Aug 2 2021, 2:49 PM

Upgrade cmake to 3.21.3

@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

Path

Size

clang/

utils/

analyzer/

Dockerfile

5 lines

Diff 363572

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.1* \
	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