Skip to content

Commit

Permalink
Rollup merge of #132054 - onur-ozkan:cargo-config, r=Kobzol
Browse files Browse the repository at this point in the history
do not remove `.cargo` directory

If vendoring isn't used bootstrap removes `.cargo` directory, which prevents developers from setting certain options in the `.cargo/config.toml` file. This was introduced in #97513 (specifically in [this commit](345eb14)). Also, since #123942, vendoring is now possible even in git sources, which means we shouldn't remove `.cargo` directory in git sources anymore.
  • Loading branch information
fmease authored Oct 23, 2024
2 parents 03cb7de + 47d6667 commit bb65499
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions src/bootstrap/bootstrap.py
Original file line number Diff line number Diff line change
Expand Up @@ -1092,9 +1092,6 @@ def check_vendored_status(self):
if not os.path.exists(cargo_dir):
eprint('ERROR: vendoring required, but .cargo/config does not exist.')
raise Exception("{} not found".format(cargo_dir))
else:
if os.path.exists(cargo_dir):
shutil.rmtree(cargo_dir)

def parse_args(args):
"""Parse the command line arguments that the python script needs."""
Expand Down

0 comments on commit bb65499

Please sign in to comment.