Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Install bootstrap fails to use configured prefix due to Tarball add_renamed_file #111653

Closed
okeuday opened this issue May 16, 2023 · 5 comments · Fixed by #110906
Closed

Install bootstrap fails to use configured prefix due to Tarball add_renamed_file #111653

okeuday opened this issue May 16, 2023 · 5 comments · Fixed by #110906
Labels
C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)

Comments

@okeuday
Copy link

okeuday commented May 16, 2023

I tried this code, due to installing from source:

tarball.add_renamed_file(etc.join("cargo.bashcomp.sh"), "etc/bash_completion.d", "cargo");

with the installation commands:

./configure --prefix=/path/to/installation/using/a/non/root/user/rust-1.69.0 --enable-extended
python3 ./x.py build
python3 ./x.py install

I expected to see the installation succeed.

Instead, the installation failed when processing the cargo installation. Only the cargo installation uses the Tarball add_renamed_file function, so the problem appears isolated to that single function call, due to the implementation of add_renamed_file.

Meta

rustc --version --verbose:

rustc 1.69.0-dev (84c898d65 2023-04-16) (built from a source tarball)
binary: rustc
commit-hash: 84c898d65adf2f39a5a98507f1fe0ce10a2b8dbc
commit-date: 2023-04-16
host: x86_64-unknown-linux-gnu
release: 1.69.0-dev
LLVM version: 15.0.7
Backtrace

The output only describes the installation failing with only cargo installed.  I didn't save the output.

@okeuday okeuday added the C-bug Category: This is a bug. label May 16, 2023
@albertlarsan68
Copy link
Member

Seems related to #110906

@clubby789 clubby789 added the T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) label May 17, 2023
@okeuday
Copy link
Author

okeuday commented May 17, 2023

@albertlarsan68 Thank you for pointing-out #110906 ! It is the same problem. Closing this as a duplicate.

@okeuday okeuday closed this as completed May 17, 2023
@albertlarsan68
Copy link
Member

It is not an issue, but this PR will fix this issue. Reopening to keep track of progress.

@rust-lang rust-lang locked and limited conversation to collaborators May 17, 2023
@rust-lang rust-lang unlocked this conversation May 17, 2023
@albertlarsan68
Copy link
Member

@okeuday Does the PR fix the issue?

@okeuday
Copy link
Author

okeuday commented May 24, 2023

@albertlarsan68 When #110906 is combined with #109256 , the 1.69.0 rustc release is able to compile and install successfully (from https://static.rust-lang.org/dist/rustc-1.69.0-src.tar.gz linked at https://forge.rust-lang.org/infra/other-installation-methods.html#source-code) as shown below:

wget https://static.rust-lang.org/dist/rustc-1.69.0-src.tar.gz
tar zxvf rustc-1.69.0-src.tar.gz
cd rustc-1.69.0-src
wget https://github.com/rust-lang/rust/pull/109256.patch
wget https://github.com/rust-lang/rust/pull/110906.patch
patch -p1 < 109256.patch
patch -p1 < 110906.patch
./configure --prefix=/path/to/installation/using/a/non/root/user/rust-1.69.0 --enable-extended
python3 ./x.py build
python3 ./x.py install

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-bug Category: This is a bug. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants