When a value could either be defined or not, we used to call either set(varname, 1) or set(varname, false) respectively. This however leads to the value type of varname to be conditional, making it virtually unusable for testing its value.
Instead, just do not emit anything when we used to emit false.

The only difference is that it will not generate any entry for the disabled values, but that is what Autotools do already anyway -- although Autotools emit a comment.

This fixes build if either HAVE_FNMATCH or HAVE_REGCOMP is false (that is, if either fmatch() or regcomp() functions are not found on the system).

Fixes #3618.


You can view, comment on, or merge this pull request online at:

  https://github.com/geany/geany/pull/3642

Commit Summary

File Changes

(1 file)

Patch Links:


Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: <geany/geany/pull/3642@github.com>