GNU bug report logs - #48021
comp.el cannot be loaded without native compilation

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 48021 in the body.
You can then email your comments to 48021 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #3 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: submit <at> debbugs.gnu.org
Subject: comp.el cannot be loaded without native compilation
Date: Sun, 25 Apr 2021 20:43:31 -0400

Package: emacs
Version: 28.0.50

At b7c22fab7d, in a default build:

./src/emacs -Q -l ./lisp/emacs-lisp/comp.el

   memq: Symbol's value as variable is void: comp-ctxt

(Presumably it works if --with-native-compilation is used.)

This causes test failures for custom tests, and multiple failures for
comp-cstr-tests (perhaps those should be tagged :nativecomp, I don't know.)

Ref eg https://hydra.nixos.org/build/142074384

Message #6 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Andrea Corallo <akrl <at> sdf.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Mon, 26 Apr 2021 14:55:26 +0000

Glenn Morris <rgm <at> gnu.org> writes:

> Package: emacs
> Version: 28.0.50
>
> At b7c22fab7d, in a default build:
>
> ./src/emacs -Q -l ./lisp/emacs-lisp/comp.el
>
>    memq: Symbol's value as variable is void: comp-ctxt
>
> (Presumably it works if --with-native-compilation is used.)
>
> This causes test failures for custom tests, and multiple failures for
> comp-cstr-tests (perhaps those should be tagged :nativecomp, I don't know.)
>
> Ref eg https://hydra.nixos.org/build/142074384

Hi Glenn,

with a390a4965f comp-cstr-tests run clean for me also on vanilla build,
please let me know if this is sufficient for the case you are testing.

Thanks

  Andrea

Message #9 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Mon, 26 Apr 2021 15:18:34 -0400

Andrea Corallo wrote:

>> ./src/emacs -Q -l ./lisp/emacs-lisp/comp.el
>>
>>    memq: Symbol's value as variable is void: comp-ctxt

> with a390a4965f comp-cstr-tests run clean for me also on vanilla build,
> please let me know if this is sufficient for the case you are testing.

Thanks. comp-cstr-test-70 is still failing for me.

The issue about comp.el being unloadable in builds without native
compilation remains (and hence test-custom-deps still fails).

I believe that in general, .el files should always be loadable, they
should just do nothing if not applicable.

For example, C-h f completion can try to load comp.el.

Message #12 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Andrea Corallo <akrl <at> sdf.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Mon, 26 Apr 2021 20:29:39 +0000

Glenn Morris <rgm <at> gnu.org> writes:

> Andrea Corallo wrote:
>
>>> ./src/emacs -Q -l ./lisp/emacs-lisp/comp.el
>>>
>>>    memq: Symbol's value as variable is void: comp-ctxt
>
>> with a390a4965f comp-cstr-tests run clean for me also on vanilla build,
>> please let me know if this is sufficient for the case you are testing.
>
> Thanks. comp-cstr-test-70 is still failing for me.

Could you share the invocation you are using? I'm asking cause here on
449045b2a6 I see this passing.

>
> The issue about comp.el being unloadable in builds without native
> compilation remains (and hence test-custom-deps still fails).

I didn't know about the test-custom-deps, I'll have a look.

> I believe that in general, .el files should always be loadable, they
> should just do nothing if not applicable.
>
> For example, C-h f completion can try to load comp.el.

We should load comp.el only on native compiled instances, if this is not
the case could you point me out exactly where we do this so I can have a
look?

Thanks

  Andrea

Message #15 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Andrea Corallo <akrl <at> sdf.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Mon, 26 Apr 2021 21:16:09 +0000

Andrea Corallo via "Bug reports for GNU Emacs, the Swiss army knife of
text editors" <bug-gnu-emacs <at> gnu.org> writes:

> Glenn Morris <rgm <at> gnu.org> writes:
>
>> Andrea Corallo wrote:
>>
>>>> ./src/emacs -Q -l ./lisp/emacs-lisp/comp.el
>>>>
>>>>    memq: Symbol's value as variable is void: comp-ctxt
>>
>>> with a390a4965f comp-cstr-tests run clean for me also on vanilla build,
>>> please let me know if this is sufficient for the case you are testing.
>>
>> Thanks. comp-cstr-test-70 is still failing for me.
>
> Could you share the invocation you are using? I'm asking cause here on
> 449045b2a6 I see this passing.
>
>>
>> The issue about comp.el being unloadable in builds without native
>> compilation remains (and hence test-custom-deps still fails).
>
> I didn't know about the test-custom-deps, I'll have a look.

Hi Glenn,

with 40c71e574a comp-cstr.el and comp.el are loadable for me also on
vanilla builds, please have a look.

Thanks

  Andrea

Message #18 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Tue, 27 Apr 2021 00:56:14 -0400

Andrea Corallo wrote:

>>> Thanks. comp-cstr-test-70 is still failing for me.
>>
>> Could you share the invocation you are using? I'm asking cause here on
>> 449045b2a6 I see this passing.

On CentOS 8.3 with gcc 8.3, I get:

Test comp-cstr-test-70 condition:
    (ert-test-failed
     ((should
       (equal
       (comp-cstr-test-ts ...)
       '...))
      :form
      (equal
       (not
       (member b a))
       (not
       (member a b)))

There is an identical failure on the very different system
https://hydra.nixos.org/build/142153876

> with 40c71e574a comp-cstr.el and comp.el are loadable for me also on
> vanilla builds, please have a look.

Works for me, thanks.

Message #23 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Thu, 29 Apr 2021 14:44:39 -0400

>>>> Thanks. comp-cstr-test-70 is still failing for me.
>>>
>>> Could you share the invocation you are using? I'm asking cause here on
>>> 449045b2a6 I see this passing.
>
> On CentOS 8.3 with gcc 8.3, I get:
>
> Test comp-cstr-test-70 condition:
>     (ert-test-failed
>      ((should
>        (equal
>        (comp-cstr-test-ts ...)
>        '...))
>       :form
>       (equal
>        (not
>        (member b a))
>        (not
>        (member a b)))
>
> There is an identical failure on the very different system
> https://hydra.nixos.org/build/142153876

Same failure on Debian testing.

Message #26 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Andrea Corallo <akrl <at> sdf.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Thu, 29 Apr 2021 19:08:18 +0000

Glenn Morris <rgm <at> gnu.org> writes:

>>>>> Thanks. comp-cstr-test-70 is still failing for me.
>>>>
>>>> Could you share the invocation you are using? I'm asking cause here on
>>>> 449045b2a6 I see this passing.
>>
>> On CentOS 8.3 with gcc 8.3, I get:
>>
>> Test comp-cstr-test-70 condition:
>>     (ert-test-failed
>>      ((should
>>        (equal
>>        (comp-cstr-test-ts ...)
>>        '...))
>>       :form
>>       (equal
>>        (not
>>        (member b a))
>>        (not
>>        (member a b)))
>>
>> There is an identical failure on the very different system
>> https://hydra.nixos.org/build/142153876
>
> Same failure on Debian testing.

Hi Glenn,

39ebc2689b should fix this, please have a look.

Thanks

  Andrea

Message #29 received at 48021 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Andrea Corallo <akrl <at> sdf.org>
Cc: 48021 <at> debbugs.gnu.org
Subject: Re: bug#48021: comp.el cannot be loaded without native compilation
Date: Thu, 29 Apr 2021 19:40:04 -0400

Andrea Corallo wrote:

> 39ebc2689b should fix this, please have a look.

Works for me, thanks for the fix.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.