Skip to content

Commit

Permalink
Rollup merge of #110123 - madsravn:master, r=jyn514
Browse files Browse the repository at this point in the history
'./configure' now checks if 'config.toml' exists before writing to that destination

Fixes #110109

Instead of overwriting the current `config.toml` file, exit the `./configure` script with a message stating why.
  • Loading branch information
matthiaskrgr authored Apr 19, 2023
2 parents 3a5c8e9 + 010d797 commit 73bfa18
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions src/bootstrap/configure.py
Original file line number Diff line number Diff line change
Expand Up @@ -521,8 +521,14 @@ def write_config_toml(writer, section_order, targets, sections):
else:
writer = write_uncommented(sections[section], writer)

def quit_if_file_exists(file):
if os.path.isfile(file):
err("Existing '" + file + "' detected.")

if __name__ == "__main__":
# If 'config.toml' already exists, exit the script at this point
quit_if_file_exists('config.toml')

p("processing command line")
# Parse all known arguments into a configuration structure that reflects the
# TOML we're going to write out
Expand Down

0 comments on commit 73bfa18

Please sign in to comment.