bug#19523: Segfault when creating thread with scm_with_guile

2016-06-23 Thread Anthonin Bonnefoy
Forget about the specific sha1, it compiled with my system-wide .so which should be on the v2.0.11 tag. I tried to reproduce the problem in a rr record (I had to change SIG_SUSPEND to SIGUSR1 in bdwgc) without success. On Thu, Jun 23, 2016 at 8:43 AM, Anthonin Bonnefoy < anthonin.bonne...@securac

bug#19540: repeated ./././ in compiled modules

2016-06-23 Thread Andy Wingo
Hi! On Tue 20 Jan 2015 22:12, l...@gnu.org (Ludovic Courtès) writes: > scheme@(guile-user)> (add-to-load-path ".") > scheme@(guile-user)> %load-path > $2 = ("." [...]) > scheme@(guile-user)> (getcwd) > $3 = "/home/ludo/src/guile/module/ice-9" > scheme@(guile-user)> (open-input-file (search-path %

bug#19581: 'sleep' return value is incorrectly rounded

2016-06-23 Thread Andy Wingo
Hi :) On Tue 13 Jan 2015 13:58, l...@gnu.org (Ludovic Courtès) writes: > I just noticed that ‘sleep’ essentially always rounds down its return > value, which makes it unreliable, as in this example: > > $ time guile -c '(sigaction SIGINT +) (call-with-new-thread (lambda () (let > loop () (kill (

bug#19770: goops - setter inheritance bug - serious

2016-06-23 Thread Andy Wingo
Hi David, Sorry for the long delay. I think this is not a bug. The reason is that you defined !width as an #:accessor. That means that each concrete class which has a `width' slot will have its own accessor method installed on it. You overrode the method for instances but not for any other co

bug#19793: please document level of R7RS[-small] support in Guile manual

2016-06-23 Thread Andy Wingo
Hi Andrew, Would you be willing to create such a document? Contributions would be happily accepted :) Take a look at doc/ref/r6rs.texi for a template (just the incompatibilities and general overview section I guess). Andy On Fri 06 Feb 2015 09:56, "and...@cachedot.net" writes: > It would be

bug#19883: Correction for backtrace

2016-06-23 Thread Andy Wingo
On Thu 26 Feb 2015 16:30, David Kastrup writes: > Try ./test 2 2000 200 I can reproduce the crash with your test case, thanks :) The patch below fixes the bug for me. WDYT Ludovic? Andy commit db30120fc3a1727d8f221cbb014314f2babf841e Author: Andy Wingo Date: Thu Jun 23 11:47:42 2016 +0200

bug#19883: Correction for backtrace

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 11:50, Andy Wingo writes: > On Thu 26 Feb 2015 16:30, David Kastrup writes: > >> Try ./test 2 2000 200 > > I can reproduce the crash with your test case, thanks :) The patch below > fixes the bug for me. WDYT Ludovic? Here's a patch with a test case. I'm going to apply as i

bug#20017: guile 2.0.11 make check failure for ppc architecture

2016-06-23 Thread Andy Wingo
Hi :) On Fri 06 Mar 2015 14:57, Normand writes: > guile 2.0.11 make check failure for ppc architecture on opensuse 13.2 > > details of the log is available at: > https://build.opensuse.org/package/show/openSUSE:Factory:PowerPC/guile > https://build.opensuse.org/build/openSUSE:Factory:PowerPC/sta

bug#20079: Fwd: Memory leak from seek/ftell with files larger than 2GB

2016-06-23 Thread Andy Wingo
Very strange bug! I can reproduce it with this file: (use-modules (rnrs bytevectors) (ice-9 binary-ports)) (define MAX_SIGNED_INT 2147483647) (define BYTES_TO_READ 10) (define (traverse port) (let* ((file-sz (stat:size (stat port))) (ua (make-bytevector BYTES_TO_READ 0))

bug#20079: Fwd: Memory leak from seek/ftell with files larger than 2GB

2016-06-23 Thread Andy Wingo
Hi, Thank you very much for this one! Turns out we had an incredibly embarrassing bug in which we forgot to attach finalizers for bignums created by scm_from_{uint64,int64} on 32-bit platforms. Fixed in master and stable-2.0. Cheers, Andy On Wed 11 Mar 2015 13:38, Anand Mohanadoss writes: >

bug#19540: repeated ./././ in compiled modules

2016-06-23 Thread Ludovic Courtès
Andy Wingo skribis: > commit 9a951678713557b548415d32eae6d63d039bf652 > Author: Andy Wingo > Date: Thu Jun 23 10:03:10 2016 +0200 > > Fix relative file name canonicalization on paths with "." > > * libguile/filesys.c (scm_i_relativize_path): Canonicalize the file > names ele

bug#19883: Correction for backtrace

2016-06-23 Thread Ludovic Courtès
Andy Wingo skribis: > On Thu 26 Feb 2015 16:30, David Kastrup writes: > >> Try ./test 2 2000 200 > > I can reproduce the crash with your test case, thanks :) The patch below > fixes the bug for me. WDYT Ludovic? > > Andy > > commit db30120fc3a1727d8f221cbb014314f2babf841e > Author: Andy Wingo

bug#19581: 'sleep' return value is incorrectly rounded

2016-06-23 Thread Ludovic Courtès
Hello, Andy Wingo skribis: > On Tue 13 Jan 2015 13:58, l...@gnu.org (Ludovic Courtès) writes: > >> I just noticed that ‘sleep’ essentially always rounds down its return >> value, which makes it unreliable, as in this example: >> >> $ time guile -c '(sigaction SIGINT +) (call-with-new-thread (lam

bug#19883: Correction for backtrace

2016-06-23 Thread Ludovic Courtès
Andy Wingo skribis: > On Thu 23 Jun 2016 11:50, Andy Wingo writes: > >> On Thu 26 Feb 2015 16:30, David Kastrup writes: >> >>> Try ./test 2 2000 200 >> >> I can reproduce the crash with your test case, thanks :) The patch below >> fixes the bug for me. WDYT Ludovic? > > Here's a patch with a t

bug#20087: gensym

2016-06-23 Thread Andy Wingo
On Thu 24 Mar 2016 09:45, l...@gnu.org (Ludovic Courtès) writes: > Mark H Weaver skribis: > >> It turns out the manual already has the following text in the ‘gensym’ >> entry, which I think is sufficient. >> >> The symbols generated by ‘gensym’ are _likely_ to be unique, since >> their nam

bug#20087: gensym

2016-06-23 Thread Ludovic Courtès
Andy Wingo skribis: > On Thu 24 Mar 2016 09:45, l...@gnu.org (Ludovic Courtès) writes: > >> Mark H Weaver skribis: >> >>> It turns out the manual already has the following text in the ‘gensym’ >>> entry, which I think is sufficient. >>> >>> The symbols generated by ‘gensym’ are _likely_ to

bug#19235: make-fresh-user-module procedure leaks memory

2016-06-23 Thread Mark H Weaver
Andy Wingo writes: > In many ways I think Ludovic was right in #15602 -- we should allow > excursions to isolate changes to the module tree. Sometimes you want an > excursion to never add a module to the tree. Sometimes you do, but > maybe all in one go and with a mutex, to avoid races -- like,

bug#20079: Fwd: Memory leak from seek/ftell with files larger than 2GB

2016-06-23 Thread Anand Mohanadoss
Hi Andy, Thanks a lot for the fix! Anand On Thu, Jun 23, 2016 at 6:31 PM, Andy Wingo wrote: > Hi, > > Thank you very much for this one! Turns out we had an incredibly > embarrassing bug in which we forgot to attach finalizers for bignums > created by scm_from_{uint64,int64} on 32-bit platform

bug#20017: guile 2.0.11 make check failure for ppc architecture

2016-06-23 Thread Normand
On 23/06/2016 13:45, Andy Wingo wrote: Hi :) On Fri 06 Mar 2015 14:57, Normand writes: guile 2.0.11 make check failure for ppc architecture on opensuse 13.2 details of the log is available at: https://build.opensuse.org/package/show/openSUSE:Factory:PowerPC/guile https://build.opensuse.org

bug#20093: master: setting merge-generics duplicate-binding-handler @ expand time raises an error

2016-06-23 Thread Andy Wingo
Hi, This was interesting -- turned out that GOOPS imports `map' from (srfi srfi-1), and it's also there from (guile-user). SRFI-1 specified it via #:replace so all is well, no warnings. However merge-generics is itself a generic which needs `map' to dispatch over its types (at first) and then th

bug#19540: repeated ./././ in compiled modules

2016-06-23 Thread Andy Wingo
Hi :) On Thu 23 Jun 2016 15:06, l...@gnu.org (Ludovic Courtès) writes: > Andy Wingo skribis: > >> commit 9a951678713557b548415d32eae6d63d039bf652 >> Author: Andy Wingo >> Date: Thu Jun 23 10:03:10 2016 +0200 >> >> Fix relative file name canonicalization on paths with "." >> >> *

bug#20087: gensym

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 16:13, l...@gnu.org (Ludovic Courtès) writes: > Andy Wingo skribis: > >> I just pushed something to master to error when serializing an >> uninterned symbol. Otherwise compiling an uninterned symbol effectively >> interns it! I am not sure that we can apply such a fix in 2.0

bug#20017: guile 2.0.11 make check failure for ppc architecture

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 17:28, Normand writes: >> (expt 2 1/2) >> (expt 2.0 1/2) >> (sqrt 2) > > The bugzilla should probably be closed now, > because since then it seems that problem disapeared. > If I do the manual tests as suggested there is no differences in expressions > outputs as reported belo

bug#20109: Incompatible API change in 2.0 series for string port encoding

2016-06-23 Thread Andy Wingo
On Fri 17 Apr 2015 07:17, Mark H Weaver writes: > David Kastrup writes: > >> In 2.0.9, the following patch/code for getting what amounts to a binary >> string port worked. >> >> commit 7f7a124d3470b0d566f796e88f4e2ad5aa043f16 >> Author: David Kastrup >> Date: Sun Sep 21 18:40:06 2014 +0200 >>

bug#20209: GUILE 2.0.11: crash in set_port_filename_x for bytevector ports

2016-06-23 Thread Andy Wingo
On Sun 29 Mar 2015 15:06, l...@gnu.org (Ludovic Courtès) writes: > Mark H Weaver skribis: > >> For these reasons, I decided against recommending those undocumented >> initialization functions. Instead, I suggested that David initialize >> binary ports by loading (ice-9 binary-ports): >> >> (vo

bug#20109: Incompatible API change in 2.0 series for string port encoding

2016-06-23 Thread David Kastrup
Andy Wingo writes: > On Fri 17 Apr 2015 07:17, Mark H Weaver writes: > >> David Kastrup writes: >> >>> In 2.0.9, the following patch/code for getting what amounts to a binary >>> string port worked. >>> >>> commit 7f7a124d3470b0d566f796e88f4e2ad5aa043f16 >>> Author: David Kastrup >>> Date: S

bug#20209: GUILE 2.0.11: crash in set_port_filename_x for bytevector ports

2016-06-23 Thread David Kastrup
Andy Wingo writes: > On Sun 29 Mar 2015 15:06, l...@gnu.org (Ludovic Courtès) writes: > >> Mark H Weaver skribis: >> >>> For these reasons, I decided against recommending those undocumented >>> initialization functions. Instead, I suggested that David initialize >>> binary ports by loading (ice

bug#20645: sxml xpath exports "filter"

2016-06-23 Thread Mark H Weaver
Mark H Weaver writes: > Matt Wette writes: > >> This is a very annoying "bug". The (sxml xpath) “filter” procedure >> overrides the builtin guile “filter”. >> >> I believe the fix can be implemented with the following changes to (sxml >> xpath): >> 1) (define node-filter filter) >> 2) In the

bug#16357: insufficient print abbreviation in error messages

2016-06-23 Thread Mark H Weaver
Andy Wingo writes: > Or, we could use print states. But print states are not so great and > ideally we would remove them eventually. We will need print states, or something like them, to support writing cyclic data structures as required by R7RS. Mark

bug#16357: insufficient print abbreviation in error messages

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 18:57, Mark H Weaver writes: > Andy Wingo writes: >> Or, we could use print states. But print states are not so great and >> ideally we would remove them eventually. > > We will need print states, or something like them, to support writing > cyclic data structures as required

bug#20109: Incompatible API change in 2.0 series for string port encoding

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 18:46, David Kastrup writes: > With regard to Guile 2.0, I cannot provide anything that would warrant > keeping this report open. Okeydoke, will close. Thanks :) Andy

bug#20209: GUILE 2.0.11: crash in set_port_filename_x for bytevector ports

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 18:59, David Kastrup writes: > I don't see anything protecting sym_big or sym_little (more accurately, > 'big or 'little which are non-immediate SCM values) from collection > which would make sym_big and sym_little useless for comparison. > > I'm assuming that not the whole bss

bug#20209: GUILE 2.0.11: crash in set_port_filename_x for bytevector ports

2016-06-23 Thread David Kastrup
Andy Wingo writes: > On Thu 23 Jun 2016 18:59, David Kastrup writes: > >> I don't see anything protecting sym_big or sym_little (more accurately, >> 'big or 'little which are non-immediate SCM values) from collection >> which would make sym_big and sym_little useless for comparison. >> >> I'm as

bug#20272: Support reproducible builds

2016-06-23 Thread Andy Wingo
On Mon 20 Jun 2016 17:48, Andy Wingo writes: >>> Currently, we include both the session-id and a global gensym counter in >>> the names of freshly generated marks and labels in psyntax.scm. >>> Instead, let them include the module name and a per-module counter. >> >> I've attached a preliminary i

bug#19459: #:export does not honor the merge-generics contract

2016-06-23 Thread David Pirotte
Hi Andy, > > (define-module (a) > > #:use-module (oop goops) > > #:export ( > > !width > > get-width > > set-width)) > Here you export four bindings: one class and three generics. Those > three generics have methods on . > > (define-module (b) > > #:use-module (o

bug#20219: crash of the repl

2016-06-23 Thread Andy Wingo
On Sat 28 Mar 2015 17:18, Andrew Engelbrecht writes: > i opened guile, then pressed ctrl-c, then a bunch of other buttons > including ctrl-d and enter. eventually it crashed: This is an amusing bug report :) On the other hand it does seem to be entirely reproducible if I do meta/guile -q and C-c

bug#20339: sxml simple: sxml->xml mishandles namespaces?

2016-06-23 Thread Andy Wingo
See thread here as well: http://thread.gmane.org/gmane.lisp.guile.devel/17709 I like Ricardo's patch but have some comments here: http://article.gmane.org/gmane.lisp.guile.devel/18384 Andy

bug#19459: #:export does not honor the merge-generics contract

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 21:23, David Pirotte writes: > Hi Andy, > >> > (define-module (a) >> > #:use-module (oop goops) >> > #:export ( >> >!width >> >get-width >> >set-width)) > >> Here you export four bindings: one class and three generics. Those >> three generics have

bug#20375: doc. error in v 2.0.11, Section 7.6.2.5

2016-06-23 Thread Andy Wingo
On Mon 20 Apr 2015 04:20, Matt Wette writes: > The reference manual for guile 2.0.11 has an error in Section 7.6.2.5 > on rnrs lists. The statement "identical to the fold ..." is not quite > correct. The section says: > > fold-left combine nil list1 list2 . . . [Scheme Procedure] fold-right > c

bug#20423: goops - inheritance of slot options

2016-06-23 Thread Andy Wingo
Hi, For what it's worth this is a part of GOOPS's design AFAIU: all slot definitions are unique. Two slots named S declared on classes A and B are distinct, even if A is a superclass of B. I don't know if we can change this. I guess maybe. There would have to be a design though. I guess flesh

bug#20546: Syntax bug in master branch (till d99fedc5fd38c0)

2016-06-23 Thread Andy Wingo
On Mon 11 May 2015 12:29, Nala Ginrut writes: > I'm using master branch (till d99fedc5fd38c0), and I found this code > can't run in 2.1, but works in 2.0: > > (define-syntax define-art > (lambda (x) >(syntax-case x () > ((_ name) (identifier? #'name) > #`(begin > (define na

bug#20551: Missing dokumentation for weak hash tables

2016-06-23 Thread Andy Wingo
On Mon 11 May 2015 22:00, Stefan Israelsson Tampe writes: > Weak hash tables has an undokumented constraint in order for it to > work. The key > of a weak key value table / double weak table cannot be part of the > value and expect the behavior in > the current dokumentation to apply,, and vice

bug#20655: According to docs non-greedy regexp should work in guile but they don't

2016-06-23 Thread Andy Wingo
On Mon 25 May 2015 23:58, Michael Gerdau writes: > the guile documentation under > https://www.gnu.org/software/guile/manual/html_node/Regular-Expressions.html#index-regular-expressions > directly references Regular expressions used in Emacs which do > support non-greedy regexp. > > However in th

bug#20551: Missing dokumentation for weak hash tables

2016-06-23 Thread Andy Wingo
thanks

bug#19459: #:export does not honor the merge-generics contract

2016-06-23 Thread David Pirotte
> > Hi Andy, > > > >> > (define-module (a) > >> > #:use-module (oop goops) > >> > #:export ( > >> > !width > >> > get-width > >> > set-width)) > > > >> Here you export four bindings: one class and three generics. Those > >> three generics have methods on . > > > >>

bug#20423: goops - inheritance of slot options

2016-06-23 Thread David Pirotte
Hi Andy, > For what it's worth this is a part of GOOPS's design AFAIU: all slot > definitions are unique. Two slots named S declared on classes A and B > are distinct, even if A is a superclass of B. Well, as I explained in the original report, the only specification we have is CLOS: Stklos, up

bug#20423: goops - inheritance of slot options

2016-06-23 Thread dsmich
David Pirotte wrote: > Hi Andy, > > > For what it's worth this is a part of GOOPS's design AFAIU: all slot > > definitions are unique. Two slots named S declared on classes A and B > > are distinct, even if A is a superclass of B. > > Well, as I explained in the original report, the only

bug#20423: goops - inheritance of slot options

2016-06-23 Thread Andy Wingo
Hi, On Fri 24 Jun 2016 01:08, David Pirotte writes: > It actually is my intention, it has been for years, to study, dig into and > work on > our GOOPS implementation and start to help maintain it ... if I only had more > time, > I would have done it already. > > Till then we rely on you I don

bug#20423: goops - inheritance of slot options

2016-06-23 Thread Andy Wingo
Hi, On Fri 24 Jun 2016 01:08, David Pirotte writes: > It actually is my intention, it has been for years, to study, dig into and > work on > our GOOPS implementation and start to help maintain it ... if I only had more > time, > I would have done it already. > > Till then we rely on you I don

bug#19459: #:export does not honor the merge-generics contract

2016-06-23 Thread Andy Wingo
On Thu 23 Jun 2016 23:11, David Pirotte writes: >> However... I believe merge-generics is intended to merge duplicate >> imported bindings. It does not provide a copy-on-write version of an >> imported generic, if that generic was not duplicated in the imports. >> There is no facility in GOOPS t

bug#20567: guile 2.0.11 - test failures on mips64 big-endian with n32 ABI

2016-06-23 Thread Andy Wingo
Mark does this ring a bell to you? I know you have a mip64 n32 machine that you use sometimes. Andy On Wed 13 May 2015 20:31, "Schaefer, Frank" writes: > I've encountered two test failures with guile 2.0.11 on mips64 > big-endian, whenever I build with -mabi=n32 (these test failures do > not

bug#20822: environment mangled by locale

2016-06-23 Thread Andy Wingo
On Tue 16 Jun 2015 06:17, Zefram writes: > When guile-2.0 is asked to read environment variables, via getenv, > it always decodes the underlying octet string according to the current > locale's nominal character encoding. This is a problem, because the > environment variable's value is not neces

bug#20823: argv mangled by locale

2016-06-23 Thread Andy Wingo
On Tue 16 Jun 2015 06:33, Zefram writes: > I don't see any Scheme interface that reliably retrieves the command > line arguments without locale decoding. [...] > The actual data passed between processes is an octet string, and > there really needs to be some reliable way to access that octet stri