From 07d2bd751708bd438975f80b3db6aa648baad2a3 Mon Sep 17 00:00:00 2001 From: Sebastian Thiel Date: Thu, 18 Apr 2024 18:44:59 +0200 Subject: [PATCH] reproduce failure when packaging a path-dependency inside a symlinked git repository (#13773) --- crates/cargo-test-support/src/git.rs | 9 ++++- tests/testsuite/build.rs | 56 +++++++++++++++++++++++++++- 2 files changed, 62 insertions(+), 3 deletions(-) diff --git a/crates/cargo-test-support/src/git.rs b/crates/cargo-test-support/src/git.rs index 236011ca1..2b838e8c9 100644 --- a/crates/cargo-test-support/src/git.rs +++ b/crates/cargo-test-support/src/git.rs @@ -38,7 +38,7 @@ use some of the helper functions in this file to interact with the repository. */ -use crate::{path2url, project, Project, ProjectBuilder}; +use crate::{path2url, project, Project, ProjectBuilder, SymlinkBuilder}; use std::fs; use std::path::{Path, PathBuf}; use std::sync::Once; @@ -76,6 +76,13 @@ impl RepoBuilder { me } + /// Create a symlink to a directory + pub fn nocommit_symlink_dir>(self, dst: T, src: T) -> Self { + let workdir = self.repo.workdir().unwrap(); + SymlinkBuilder::new_dir(workdir.join(dst), workdir.join(src)).mk(); + self + } + /// Add a file that will be left in the working directory, but not added /// to the repository. pub fn nocommit_file(self, path: &str, contents: &str) -> RepoBuilder { diff --git a/tests/testsuite/build.rs b/tests/testsuite/build.rs index 507d6cb19..daa605fdf 100644 --- a/tests/testsuite/build.rs +++ b/tests/testsuite/build.rs @@ -6,14 +6,14 @@ use cargo::{ ops::CompileOptions, GlobalContext, }; -use cargo_test_support::compare; use cargo_test_support::paths::{root, CargoPathExt}; use cargo_test_support::registry::Package; -use cargo_test_support::tools; use cargo_test_support::{ basic_bin_manifest, basic_lib_manifest, basic_manifest, cargo_exe, git, is_nightly, main_file, paths, process, project, rustc_host, sleep_ms, symlink_supported, t, Execs, ProjectBuilder, }; +use cargo_test_support::{cargo_process, compare}; +use cargo_test_support::{git_process, tools}; use cargo_util::paths::dylib_path_envvar; use std::env; use std::fs; @@ -33,6 +33,58 @@ fn cargo_compile_simple() { p.process(&p.bin("foo")).with_stdout("i am foo\n").run(); } +#[cargo_test] +fn build_with_symlink_to_path_dependency_with_build_script_in_git() { + if !symlink_supported() { + return; + } + + let root = paths::root(); + git::repo(&root) + .nocommit_file( + "Cargo.toml", + r#" + [package] + name = "foo" + version = "0.1.0" + edition = "2021" + + [dependencies] + # the path leads through a symlink, 'symlink-to-original' is a worktree root, + # and symlink-to-dir/ is a symlink to a sub-directory to be stepped through. + lib = { version = "0.1.0", path = "symlink-to-original/symlink-to-dir/lib" } + "#, + ) + .nocommit_file("src/main.rs", "fn main() { }") + .nocommit_file("original/dir/lib/build.rs", "fn main() {}") + .nocommit_file( + "original/dir/lib/Cargo.toml", + r#" + [package] + name = "lib" + version = "0.1.0" + edition = "2021" + "#, + ) + .nocommit_file("original/dir/lib/src/lib.rs", "") + .nocommit_symlink_dir("original", "symlink-to-original") + .nocommit_symlink_dir("original/dir", "original/symlink-to-dir") + .build(); + + // It is necessary to have a sub-repository and to add files so there is an index. + git_process("init") + .cwd(root.join("original")) + .build_command() + .status() + .unwrap(); + git_process("add .") + .cwd(root.join("original")) + .build_command() + .status() + .unwrap(); + cargo_process("build").run() +} + #[cargo_test] fn cargo_fail_with_no_stderr() { let p = project()