Skip to content

Commit

Permalink
[libcxx] Fix default argument for merge_archives.py -L flag
Browse files Browse the repository at this point in the history
If we use the default of None, we get a python exception in
find_and_diagnose_missing() instead of printing a sensible error message.

Reviewed By: #libc, ldionne

Differential Revision: https://reviews.llvm.org/D84342
  • Loading branch information
arichardson committed Jul 22, 2020
1 parent b98f902 commit e59778a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion libcxx/utils/merge_archives.py
Expand Up @@ -93,7 +93,7 @@ def main():
parser.add_argument(
'-L', dest='search_paths',
help='Paths to search for the libraries along', action='append',
nargs=1)
nargs=1, default=[])
parser.add_argument(
'--ar', dest='ar_exe', required=False,
help='The ar executable to use, finds \'ar\' in the path if not given',
Expand Down

0 comments on commit e59778a

Please sign in to comment.