davmac314 / dinit

Service monitoring / "init" system
Apache License 2.0
583 stars 45 forks source link

configure: use `try_optional_cxx_argument()` when flag is not required for less confusion #283

Closed mobin-2008 closed 7 months ago

mobin-2008 commented 7 months ago

Previously, All flags get checked through try_cxx_argument() which can return 1 on unsupported flag and its return should be ignored but it's made into confusion.

See https://github.com/davmac314/dinit/issues/281 for a example.

This commit creates a new function that will ignore try_cxx_argument() return by default and this commit replaces all uses of try_cxx_argument() on optional flags with this new function.

Fixes #281

Signed-off-by: Mobin "Hojjat" Aydinfar <mobin@mobintestserver.ir>