[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] examples: hello-c: Include config.h if HAVE_CONFIG_H is defi
From: |
Bruno Haible |
Subject: |
Re: [PATCH] examples: hello-c: Include config.h if HAVE_CONFIG_H is defined |
Date: |
Fri, 16 Sep 2022 19:11:26 +0200 |
Marvin Schmidt wrote:
> You're right, I was looking at this from the `autopoint-3` test which creates
> a custom configure.ac, which contains `AC_CONFIG_HEADERS([config.h])`
> but lacks a check for the `unistd.h` header.
>
> The configure.ac of the hello-c example on the other hand does have a check
> for the `unistd.h` but includes it in `hello.c` only if
> `HAVE_UNISTD_H` is defined
Oh, I see. Thanks; I had ignored part of your first mail.
You are right that it's best if the example and the autopoint-3 test are
written in the same manner. I'll apply the attached patch.
Thanks for your help!
Bruno
0001-Fix-a-test-failure-of-autopoint-3.patch
Description: Text Data