Subject: bug#39183: closed (Re: bug#39183: Missing ‘scm_sym_’symbols in libguile-3.0.so)

From: [email-masked] (GNU bug Tracking System)
Date: Mon, 20 Jan 2020 10:48:02 +0000

Your bug report

#39183: Missing ‘scm_sym_’ symbols in libguile-3.0.so

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].

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

Subject: Re: bug#39183: Missing ‘scm_sym_’ symbols in libguile-3.0.so
From: Ludovic Courtès <[email-masked]>
To: [email-masked], [email-masked]
Date: Mon, 20 Jan 2020 11:47:15 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hi,

Ludovic Courtès <[email-masked]> skribis:

> Libguile in 3.0.0 lacks all the C exports for variables created with
> ‘SCM_GLOBAL_SYMBOL’ in ‘expand.c’:
>
> $ objdump -T /gnu/store/b9xkl8vbxbyblzkwp7xzwwmharswkbgy-guile-2.2.6-1/lib/libguile-2.2.so |grep scm_sym_lambda
> 000000000012c298 g    DO .bss	0000000000000008  GUILE_2.0   scm_sym_lambda
> $ objdump -T /gnu/store/7z0zzqbchcjmc1krwhy0jd5hsny5y77w-guile-next-3.0.0/lib/libguile-3.0.so |grep scm_sym_lambda
>
> That’s because they accidentally ended up having the “hidden”
> visibility, which is fixed by the patch below.
>
> This will obviously augment the ABI.  Should we push?

Pushed in 228ae549ca651d9e9e7a7b02d282d0408bd6c9e6.

Ludo’.

Subject: Missing ‘scm_sym_’ symbols in libguile-3.0.so
From: Ludovic Courtès <[email-masked]>
To: [email-masked]
Date: Sat, 18 Jan 2020 23:20:27 +0100
List-Id: <debbugs-submit.debbugs.gnu.org>

Hello,

Libguile in 3.0.0 lacks all the C exports for variables created with
‘SCM_GLOBAL_SYMBOL’ in ‘expand.c’:

--8<---------------cut here---------------start------------->8---
$ objdump -T /gnu/store/b9xkl8vbxbyblzkwp7xzwwmharswkbgy-guile-2.2.6-1/lib/libguile-2.2.so |grep scm_sym_lambda
000000000012c298 g    DO .bss	0000000000000008  GUILE_2.0   scm_sym_lambda
$ objdump -T /gnu/store/7z0zzqbchcjmc1krwhy0jd5hsny5y77w-guile-next-3.0.0/lib/libguile-3.0.so |grep scm_sym_lambda
--8<---------------cut here---------------end--------------->8---

That’s because they accidentally ended up having the “hidden”
visibility, which is fixed by the patch below.

This will obviously augment the ABI.  Should we push?

Thanks,
Ludo’.

diff --git a/libguile/expand.c b/libguile/expand.c
index 11e43c2b9..6ba65ffa2 100644
--- a/libguile/expand.c
+++ b/libguile/expand.c
@@ -1,4 +1,4 @@
-/* Copyright 1995-2014,2016,2018-2019
+/* Copyright 1995-2014,2016,2018-2020
      Free Software Foundation, Inc.
 
    This file is part of Guile.
@@ -32,6 +32,7 @@
 #include "keywords.h"
 #include "list.h"
 #include "macros.h"
+#include "memoize.h"    /* for the SCM_API declarations of 'scm_sym_' */
 #include "modules.h"
 #include "pairs.h"
 #include "ports.h"