Page MenuHomePhorge
Differential D198

Fix ccache not being used
ClosedPublic
Actions

Authored by tusooa on Fri, Nov 8, 7:33 PM.

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rK54e05f5c333e: Fix ccache not being used
Summary

su will reset PATH and make the PATH set in the original environment invalid. We use env to pass on the PATH in the su'd environment.

Type: skip

Test Plan

Verify ccache is used by inspecting the stats in the build logs.

Diff Detail

Repository
rK kazv
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

tusooa created this revision.Fri, Nov 8, 7:33 PM
Owners added a reviewer: O1: the Kazv Project.Fri, Nov 8, 7:33 PM
tusooa updated this revision to Diff 529.Sat, Nov 9, 10:20 AM

Fix su command

Harbormaster failed remote builds in B317: Diff 526!Sat, Nov 9, 10:25 AM
Harbormaster returned this revision to the author for changes because remote builds failed.Sat, Nov 9, 10:32 AM
Harbormaster failed remote builds in B320: Diff 529!
tusooa updated this revision to Diff 530.Sat, Nov 9, 10:33 AM

Grant ccache dir to unprivileged user

Harbormaster completed remote builds in B321: Diff 530.Sat, Nov 9, 11:39 AM
tusooa requested review of this revision.Sat, Nov 9, 11:39 AM
nannanko accepted this revision.Sat, Nov 9, 5:53 PM
This revision is now accepted and ready to land.Sat, Nov 9, 5:53 PM
Closed by commit rK54e05f5c333e: Fix ccache not being used. · Explain WhySat, Nov 9, 6:01 PM
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rK54e05f5c333e: Fix ccache not being used.

Revision Contents
Changeset List

PathSizePackages
packaging/
GNU-Linux/
appimage/
3 linesO1: the Kazv Project

Diff 535

View Options

packaging/GNU-Linux/appimage/build.sh

Loading...