Subject: bug#60022: closed (Re: bug#60022: configure.ac fix for C99 compatibility)

From: [email-masked] (GNU bug Tracking System)
Date: Mon, 16 Jan 2023 22:20:02 +0000

Your bug report

#60022: configure.ac fix for C99 compatibility

which was filed against the guile package, has been closed.

The explanation is attached below, along with your original report.
If you require more details, please reply to [email-masked].

-- 
60022: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=60022
GNU Bug Tracking System
Contact [email-masked] with problems

Subject: Re: bug#60022: configure.ac fix for C99 compatibility
From: Ludovic Courtès <[email-masked]>
To: Florian Weimer <[email-masked]>
Cc: [email-masked]
Date: Mon, 16 Jan 2023 23:19:27 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi Florian,

Florian Weimer <[email-masked]> skribis:

> This patch is needed to improve C99 compatibility:
>
> diff --git a/configure.ac b/configure.ac
> index b3879df1f..cc865a028 100644
> --- a/configure.ac
> +++ b/configure.ac

Applied, thanks.

Ludo’.

Subject: configure.ac fix for C99 compatibility
From: Florian Weimer <[email-masked]>
To: [email-masked]
Date: Mon, 12 Dec 2022 22:35:33 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

This patch is needed to improve C99 compatibility:

diff --git a/configure.ac b/configure.ac
index b3879df1f..cc865a028 100644
--- a/configure.ac
+++ b/configure.ac
@@ -904,7 +904,7 @@ AC_CACHE_VAL(guile_cv_localtime_cache,
 AC_RUN_IFELSE([AC_LANG_SOURCE([[#include <time.h>
 #include <stdlib.h>
 extern char **environ;
-unset_TZ ()
+int unset_TZ ()
 {
   char **from, **to;
   for (to = from = environ; (*to = *from); from++)
@@ -913,7 +913,7 @@ unset_TZ ()
 }
 char TZ_GMT0[] = "TZ=GMT0";
 char TZ_PST8[] = "TZ=PST8";
-main()
+int main()
 {
   time_t now = time ((time_t *) 0);
   int hour_GMT0, hour_unset;

Future compilers are likely to reject implicit int types by default.

Found via:

  <https://fedoraproject.org/wiki/Changes/PortingToModernC>
  <https://fedoraproject.org/wiki/Toolchain/PortingToModernC>

Thanks,
Florian