| Summary: | Newline issue in configure.ac when checking minimum memory pointer alignment | ||
|---|---|---|---|
| Product: | Ghostscript | Reporter: | Nick Gaya <nicholasgaya+bugzilla> |
| Component: | Build Process | Assignee: | Chris Liddell (chrisl) <chris.liddell> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | chris.liddell |
| Priority: | P4 | ||
| Version: | master | ||
| Hardware: | Macintosh | ||
| OS: | MacOS X | ||
| Customer: | Word Size: | --- | |
| Attachments: | configure.ac.patch | ||
Fixed in: https://git.ghostscript.com/?p=ghostpdl.git;a=commitdiff;h=054255c6ad21 Apologies, for not crediting you in the commit message - I didn't use your patch directly because I did away with the line continuations in the code snippet, too, and I plain forgot to give you a shout out in the commit. Thanks for reporting and supplying the patch, I appreciate it. Revised link: https://git.ghostscript.com/?p=ghostpdl.git;a=commitdiff;h=6e039df2621 (I'd accidentally included a work in progress earlier). |
Created attachment 21017 [details] configure.ac.patch I noticed an unexpected warning in config.log when building ghostscript: configure:13760: checking minimum memory pointer alignment ... conftest.c:89:20: warning: extra tokens at end of #include directive [-Wextra-tokens] #include <stdio.h> #include <stdlib.h> ^ The cause is this definition in configure.ac ALIGN_TEXT_PROG_INCS="\ #include <stdio.h> \ #include <stdlib.h>" The trailing backslash on the second line causes the newline to be incorrectly omitted, leading to problems in the configure check. The fix is to remove the backslash, thereby defining a multiline string (see patchfile).