Closed Bug 1634788 Opened 5 years ago Closed 3 years ago

build-clang shouldn't symlink repos anymore

Categories

(Firefox Build System :: Toolchains, defect, P3)

defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1774554

People

(Reporter: away, Unassigned)

References

(Regression)

Details

(Keywords: regression)

The symlinks are a leftover from the multi-repo SVN times. Nowadays the monorepo knows what to do.

This makes build-clang not produce the same results as a local cmake invocation. This caused me a lot of lost time and frustration today while trying to track down this difference.

Once we remove the symlinks, the build will be controlled by LLVM_ENABLE_PROJECTS, which means we'll also have to fix up last-stage builds to set that variable, otherwise we'll only build core LLVM.

While we're here, LLVM_TOOL_LIBCXX_BUILD=ON is also outdated, that too should be controlled by LLVM_ENABLE_PROJECTS.

Don't forget we still build clang 5.0 with that script.

Priority: -- → P3
Has Regression Range: --- → yes
Status: NEW → RESOLVED
Closed: 3 years ago
Duplicate of bug: 1774554
Resolution: --- → DUPLICATE
You need to log in before you can comment on or make changes to this bug.