Skip to content

Commit 3c30d8c

Browse files
committed
compiler-builtins: Eliminate symlinks
compiler-builtins has a symlink to the `libm` source directory so the two crates can share files but still act as two separate crates. This causes problems with some sysroot-related tooling, however, since directory symlinks seem to not be supported. The reason this was a symlink in the first place is that there isn't an easy for Cargo to publish two crates that share source (building works fine but publishing rejects `include`d files from parent directories, as well as nested package roots). However, after the switch to a subtree, we no longer need to publish compiler-builtins; this means that we can eliminate the link and just use `#[path]`. Similarly, the LICENSE file was symlinked so it could live in the repository root but be included in the package. This is also removed as it caused problems with the dist job (error from bootstrap's `tarball.rs`, "generated a symlink in a tarball"). If we need to publish compiler-builtins again for any reason, it would be easy to revert these changes in a preprocess step.
1 parent 2334344 commit 3c30d8c

File tree

3 files changed

+1
-2
lines changed

3 files changed

+1
-2
lines changed

compiler-builtins/LICENSE.txt

Lines changed: 0 additions & 1 deletion
This file was deleted.

compiler-builtins/src/math/libm_math

Lines changed: 0 additions & 1 deletion
This file was deleted.

compiler-builtins/src/math/mod.rs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22
#[allow(dead_code)]
33
#[allow(unused_imports)]
44
#[allow(clippy::all)]
5+
#[path = "../../../libm/src/math/mod.rs"]
56
pub(crate) mod libm_math;
67

78
macro_rules! libm_intrinsics {

0 commit comments

Comments
 (0)