Skip to content

Commit

Permalink
subprojects: iniparser: Address meson run_command() warning
Browse files Browse the repository at this point in the history
The `run_command()` invocations in the iniparser meson.build lacked the
`check` parameter:

```
iniparser| WARNING: You should add the boolean check kwarg to the run_command call.
iniparser| It currently defaults to false,
iniparser| but it will default to true in future releases of meson.
iniparser| See also: mesonbuild/meson#9300
```

Add `check` and set it to true to make sure the build configuration only
succeeds if these commands also succeed.

Fixes: 1e04f44 ("use iniparser dependency for config file parsing")
Change-Id: I97841398fc899e876b97e4e28e18cd8dd3b13222
Signed-off-by: Andrew Jeffery <[email protected]>
  • Loading branch information
amboar committed Jul 9, 2024
1 parent 1e04f44 commit ab64dad
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions subprojects/packagefiles/iniparser/meson.build
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@ project('iniparser', 'c')

inc = 'iniparser-include'

run_command('sh', '-c', 'mkdir -p iniparser-include/iniparser')
run_command('sh', '-c', 'cp -r src/* iniparser-include/iniparser/')
run_command('sh', '-c', 'mkdir -p iniparser-include/iniparser', check: true)
run_command('sh', '-c', 'cp -r src/* iniparser-include/iniparser/', check: true)

iniparser = shared_library('iniparser',
'src/dictionary.c',
Expand Down

0 comments on commit ab64dad

Please sign in to comment.