osmith has submitted this change. ( https://gerrit.osmocom.org/c/osmo-dev/+/43616?usp=email )
Change subject: gen_makefile: remove unused "r" variable ......................................................................
gen_makefile: remove unused "r" variable
I guess the idea was to use it in the next line, but instead of the "r" variable the function gets called directly. Remove it.
Fixes: 450dac79 ("gen_makefile: allow combining several .opts files") Change-Id: Ia4c376c02ee721add5ed451b83edc281543cac69 --- M gen_makefile.py 1 file changed, 0 insertions(+), 1 deletion(-)
Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve osmith: Looks good to me, approved fixeria: Looks good to me, but someone else must approve
diff --git a/gen_makefile.py b/gen_makefile.py index 4cfc313..fd0bac0 100755 --- a/gen_makefile.py +++ b/gen_makefile.py @@ -656,7 +656,6 @@ if configure_opts_file.endswith(".deps"): print(f"WARNING: using {all_deps_file} instead of {configure_opts_file}") continue - r = read_configure_opts(configure_opts_file) configure_opts.extend_dict(read_configure_opts(configure_opts_file))
make_dir = args.make_dir