[Github-comments] [geany/geany] WIP: Meson build system (#2761)
Eli Schwartz
notifications at xxxxx
Tue Dec 7 22:35:04 UTC 2021
@eli-schwartz requested changes on this pull request.
> + ['memcpy', '#include <string.h>'],
+ ['mkstemp', '#include <stdlib.h>'],
+ ['regcomp', '#include <regex.h>'],
+ ['socket', '#include <sys/socket.h>'],
+ ['strerror', '#include <string.h>'],
+ ['strstr', '#include <string.h>'],
+ ['wcrtomb', '#include <wchar.h>'],
+ ['wcscoll', '#include <wchar.h>'],
+]
+
+foreach h : check_headers
+ define = 'HAVE_' + h.underscorify().to_upper()
+ if cc.has_header(h)
+ cdata.set(define, 1)
+ else
+ cdata.set(define, false)
Several times you do `cdata.get()` on values that could be either int `1` or bool `false`, and that evaluates to a meson syntax error when trying to check the "truthiness" of an integer. It doesn't build by default for me.
Which systems was this tested on? Seems like a system that has none of: glibc, fnmatch, regcomp (all 3 need to evaluate to false for configuration to pass).
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/2761#pullrequestreview-825838658
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20211207/9ee44500/attachment.htm>
More information about the Github-comments
mailing list