[cli] do not export the contents of .inferconfig to INFER_ARGS

Summary:
Propagating the arguments read in .inferconfig shouldn't be necessary as they
are parsed by each executable. There are corner cases where I think this diff
could change the behaviour of infer (eg, .inferconfig redefines project-root
and subsequent exes read a different .inferconfig thanks to the project-root in
INFER_ARGS), but I don't think there are good use cases like that.

Reviewed By: jberdine

Differential Revision: D4475092

fbshipit-source-id: 5c020d0
master
Jules Villard 8 years ago committed by Facebook Github Bot
parent 4584bf9a4c
commit 93f33d07b8

@ -776,7 +776,8 @@ let parse ?(incomplete=false) ?config_file ~usage action =
args_to_export := arg_string in
(* read .inferconfig first, then env vars, then command-line options *)
parse_args ~incomplete ~usage (Infer Driver) inferconfig_args |> ignore;
if not incomplete then add_parsed_args_to_args_to_export ();
(* NOTE: do not add the contents of .inferconfig to INFER_ARGS. This helps avoid hitting the
command line size limit. *)
parse_args ~incomplete ~usage (Infer Driver) env_args |> ignore;
if not incomplete then add_parsed_args_to_args_to_export ();
let curr_usage =

Loading…
Cancel
Save