Are source locations broken?

Are source locations broken?

[Jean Abou Samra]

Hi,

Please have a look at:

$ ./libguile/guile test.scm
;;; note: auto-compilation is enabled, set GUILE_AUTO_COMPILE=0
;;;       or pass the --no-auto-compile argument to disable.
;;; compiling /home/jean/Bureau/ENS-1A/dm_anac/test.scm
;;; <unknown-location>: warning: possibly unbound variable 
`unbound-variable'
;;; compiled 
/home/jean/.cache/guile/ccache/3.0-LE-8-4.6/home/jean/Bureau/ENS-1A/dm_anac/test.scm.go 

Backtrace:
[snipped]


Note the <unknown-location>. In LilyPond, I suddenly lost
all locations for compilation warnings when upgrading from
Guile 3.0.5 to 3.0.8. The above is with the main branch.

Re: Are source locations broken?

[Maxime Devos]

[-- Attachment #1: Type: text/plain, Size: 974 bytes --]

Jean Abou Samra schreef op vr 25-03-2022 om 21:11 [+0100]:
> Please have a look at:
> 
> $ ./libguile/guile test.scm
> ;;; note: auto-compilation is enabled, set GUILE_AUTO_COMPILE=0
> ;;;       or pass the --no-auto-compile argument to disable.
> ;;; compiling /home/jean/Bureau/ENS-1A/dm_anac/test.scm
> ;;; <unknown-location>: warning: possibly unbound variable 
> `unbound-variable'
> ;;; compiled 
> /home/jean/.cache/guile/ccache/3.0-LE-8-4.6/home/jean/Bureau/ENS-1A/dm_anac/test.scm.go 
> 
> Backtrace:
> [snipped]
> 
> 
> Note the <unknown-location>. In LilyPond, I suddenly lost
> all locations for compilation warnings when upgrading from
> Guile 3.0.5 to 3.0.8. The above is with the main branch.

Here's a minimal reproducer:

$ cat trtr.scm:
> foo

$ guild compile trtr.scm
> <unknown-location>: warning: possibly unbound variable `foo'
> wrote `$HOME/.cache/guile/ccache/3.0-LE-8-4.6/$HOME/trtr.scm.go'

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

Re: Are source locations broken?

[Maxime Devos]

[-- Attachment #1: Type: text/plain, Size: 831 bytes --]

Maxime Devos schreef op vr 25-03-2022 om 21:22 [+0100]:
Here's a minimal reproducer:

$ cat trtr.scm:
> foo

$ guild compile trtr.scm
> <unknown-location>: warning: possibly unbound variable `foo'
> wrote `$HOME/.cache/guile/ccache/3.0-LE-8-4.6/$HOME/trtr.scm.go'

Curiously, it seems that source locations were actually recorded:

(define-syntax pk-the-source-location
  (lambda (s)
    (pk 'invocation (syntax-source s))
    (syntax-case s ()
      ((_ stuff)
       (pk 'use (syntax-source #'stuff))
       #'stuff))))

(lambda () (pk-the-source-location foo))

;;; [...]
;;; (invocation ((filename . "$HOME/trtr.scm") (line . 8) (column . 11)))

;;; (use ((filename . "$HOME/trtr.scm") (line . 8) (column . 35)))
;;; <unknown-location>: warning: possibly unbound variable `foo'
;;; compiled [...]


[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

Re: Are source locations broken?

[Olivier Dion via Developers list for Guile, the GNU extensibility library]

On Fri, 25 Mar 2022, Maxime Devos <maximedevos@telenet.be> wrote:
> Maxime Devos schreef op vr 25-03-2022 om 21:22 [+0100]:
> Here's a minimal reproducer:
>
> $ cat trtr.scm:
>> foo
>
> $ guild compile trtr.scm
>> <unknown-location>: warning: possibly unbound variable `foo'
>> wrote `$HOME/.cache/guile/ccache/3.0-LE-8-4.6/$HOME/trtr.scm.go'

Any news on that bug?

-- 
Olivier Dion
oldiob.dev

Re: Are source locations broken?

[Jean Abou Samra]

Le 01/05/2022 à 19:04, Olivier Dion a écrit :
> Any news on that bug?


Last time I looked, it seemed that there had been an oversight with
source locations having changed representation (becoming 3-element
vectors instead of alists with symbol keys). But since the bugfix
patch I have submitted for curried definitions with docstrings has
received no reaction, I didn't want to invest time without much
hope that the effort would pay off :-(

Jean

Re: Are source locations broken?

[Maxime Devos]

[-- Attachment #1: Type: text/plain, Size: 610 bytes --]

Olivier Dion schreef op zo 01-05-2022 om 13:04 [-0400]:
> On Fri, 25 Mar 2022, Maxime Devos <maximedevos@telenet.be> wrote:
> > Maxime Devos schreef op vr 25-03-2022 om 21:22 [+0100]:
> > Here's a minimal reproducer:
> > 
> > $ cat trtr.scm:
> > > foo
> > 
> > $ guild compile trtr.scm
> > > <unknown-location>: warning: possibly unbound variable `foo'
> > > wrote `$HOME/.cache/guile/ccache/3.0-LE-8-4.6/$HOME/trtr.scm.go'
> 
> Any news on that bug?

AFAICT, no news.  Also, I recommend to send bug reports to
bug-guile@gnu.org such that they appear in the bug tracker.

Greetings,
Maxime.

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 260 bytes --]