Skip to content

Commit

Permalink
env2mfile: Only consume LDFLAGS for relevant languages
Browse files Browse the repository at this point in the history
It makes no sense to add them into vala_link_args, for example.
  • Loading branch information
oleavr committed Apr 4, 2024
1 parent e47bd3a commit 4b338b4
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions mesonbuild/scripts/env2mfile.py
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,6 @@ def write_machine_file(infos: MachineInfo, ofilename: str, write_system_info: bo
os.replace(tmpfilename, ofilename)

def detect_language_args_from_envvars(langname: str, envvar_suffix: str = '') -> T.Tuple[T.List[str], T.List[str]]:
ldflags = tuple(shlex.split(os.environ.get('LDFLAGS' + envvar_suffix, '')))
compile_args = []
if langname in compilers.CFLAGS_MAPPING:
compile_args = shlex.split(os.environ.get(compilers.CFLAGS_MAPPING[langname] + envvar_suffix, ''))
Expand All @@ -282,7 +281,10 @@ def detect_language_args_from_envvars(langname: str, envvar_suffix: str = '') ->
lang_compile_args = list(cppflags) + compile_args
else:
lang_compile_args = compile_args
lang_link_args = list(ldflags) + compile_args
lang_link_args = []
if langname in compilers.LANGUAGES_USING_LDFLAGS:
lang_link_args += shlex.split(os.environ.get('LDFLAGS' + envvar_suffix, ''))
lang_link_args += compile_args
return (lang_compile_args, lang_link_args)

def detect_compilers_from_envvars(envvar_suffix: str = '') -> MachineInfo:
Expand Down

0 comments on commit 4b338b4

Please sign in to comment.