Added new dependent arguments, specifically llvm_tar and llvm_binary as well as llvm_tar and llvm_url. I added these dependent arguments because they were listed as a rewrite step for the argument parser in the meta-issue list. Also, it wouldn't make sense to download both the LLVM binary and tar. I also added the tar and url dependency because there was a binary and url dependency.
This commit is contained in:
parent
39738dc98f
commit
3feec559a9
@ -1971,6 +1971,12 @@ elif args['with_llvm_binary'] is False and args['with_llvm_url']:
|
||||
LLVM_GIT_URL = args['with_llvm_url']
|
||||
else:
|
||||
LLVM_GIT_URL = "http://root.cern.ch/git/llvm.git"
|
||||
|
||||
if args['with_binary_llvm'] and args['with_llvm_tar']:
|
||||
raise Exception("Cannot specify flags --with-binary-llvm and --with-llvm-tar together")
|
||||
|
||||
if args['with_llvm_tar'] and args['with_llvm_url']:
|
||||
raise Exception("Cannot specify flags --with-llvm-tar and --with-llvm-url together")
|
||||
|
||||
if args['with_llvm_tar']:
|
||||
tar_required = True
|
||||
|
Loading…
Reference in New Issue
Block a user