Re: master 05d240997a 2/2: Add native-compile-prune-cache command

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Sat,  2 Jul 2022 06:59:56 -0400 (EDT), Lars Ingebrigtsen <larsi@gnus.org> said:

    Lars> branch: master
    Lars> commit 05d240997aeb6ffe3db21e7f4c8e2f181edc37b6
    Lars> Author: Lars Ingebrigtsen <larsi@gnus.org>
    Lars> Commit: Lars Ingebrigtsen <larsi@gnus.org>

    Lars>     Add native-compile-prune-cache command
    
    Lars>     * lisp/emacs-lisp/comp.el (native-compile-prune-cache): New
    Lars>     command (bug#48108).


Should we wrap this (and perhaps `startup-redirect-eln-cache') in
(when native-comp-available-p) ? In a non-native compile Emacs they
signal an error:

Debugger entered--Lisp error: (void-variable native-comp-eln-load-path)
  (let ((--dolist-tail-- native-comp-eln-load-path)) (while --dolist-tail-- (let ((dir (car --dolist-tail--))) (setq dir (expand-file-name dir invocation-directory)) (if (file-exists-p dir) (progn (let ((--dolist-tail-- ...)) (while --dolist-tail-- (let ... ... ...))))) (setq --dolist-tail-- (cdr --dolist-tail--)))))
  native-compile-prune-cache()
  eval((native-compile-prune-cache) t)
  eval-expression((native-compile-prune-cache) nil nil 4194303)
  funcall-interactively(eval-expression (native-compile-prune-cache) nil nil 4194303)
  call-interactively(eval-expression nil nil)
  command-execute(eval-expression)

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> Debugger entered--Lisp error: (void-variable native-comp-eln-load-path)

BTW, I think we should try to make those native-compile-only functions
and variables usable when native-compile is not included.
E.g. native-comp-eln-load-path could simply be nil in such builds.


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Robert Pluim <rpluim@gmail.com> writes:

>>>>>> On Sat,  2 Jul 2022 06:59:56 -0400 (EDT), Lars Ingebrigtsen <larsi@gnus.org> said:
>
>     Lars> branch: master
>     Lars> commit 05d240997aeb6ffe3db21e7f4c8e2f181edc37b6
>     Lars> Author: Lars Ingebrigtsen <larsi@gnus.org>
>     Lars> Commit: Lars Ingebrigtsen <larsi@gnus.org>
>
>     Lars>     Add native-compile-prune-cache command
>     
>     Lars>     * lisp/emacs-lisp/comp.el (native-compile-prune-cache): New
>     Lars>     command (bug#48108).
>
>
> Should we wrap this (and perhaps `startup-redirect-eln-cache') in
> (when native-comp-available-p) ? In a non-native compile Emacs they
> signal an error:

Hi Robert,

do we support loading comp.el and using in non native builds?

I thought is not the case, because otherwise most of the functions there
ATM are probably broken for such a build no?

Thanks

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Fri, 22 Jul 2022 12:45:03 +0000, Andrea Corallo <akrl@sdf.org> said:
    >> Should we wrap this (and perhaps `startup-redirect-eln-cache') in
    >> (when native-comp-available-p) ? In a non-native compile Emacs they
    >> signal an error:

    Andrea> Hi Robert,

    Andrea> do we support loading comp.el and using in non native builds?

(require 'comp)

certainly works (although youʼre right, the functions
in comp.el are not available by default, which means I must have
loaded comp.el myself at some point).

    Andrea> I thought is not the case, because otherwise most of the functions there
    Andrea> ATM are probably broken for such a build no?

Right. The question is if we want to support such a case (and if so,
exactly how).

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Robert Pluim <rpluim@gmail.com> writes:

>>>>>> On Fri, 22 Jul 2022 12:45:03 +0000, Andrea Corallo <akrl@sdf.org> said:
>     >> Should we wrap this (and perhaps `startup-redirect-eln-cache') in
>     >> (when native-comp-available-p) ? In a non-native compile Emacs they
>     >> signal an error:
>
>     Andrea> Hi Robert,
>
>     Andrea> do we support loading comp.el and using in non native builds?
>
> (require 'comp)
>
> certainly works (although youʼre right, the functions
> in comp.el are not available by default, which means I must have
> loaded comp.el myself at some point).
>
>     Andrea> I thought is not the case, because otherwise most of the functions there
>     Andrea> ATM are probably broken for such a build no?
>
> Right. The question is if we want to support such a case (and if so,
> exactly how).

Yep, I really thought the answer was no, at least for now, but I might
be wrong.

BTW assuming the answer is no, should we add a load time check with
associated error or warning for comp.el to gate all these possible
issues?

Thanks!

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Fri, 22 Jul 2022 13:51:04 +0000, Andrea Corallo <akrl@sdf.org> said:
    >> Right. The question is if we want to support such a case (and if so,
    >> exactly how).

    Andrea> Yep, I really thought the answer was no, at least for now, but I might
    Andrea> be wrong.

    Andrea> BTW assuming the answer is no, should we add a load time check with
    Andrea> associated error or warning for comp.el to gate all these possible
    Andrea> issues?

Iʼd lean towards a warning myself (both for loading and for running
any of the commands inside comp.el)

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Robert Pluim <rpluim@gmail.com> writes:

>>>>>> On Fri, 22 Jul 2022 13:51:04 +0000, Andrea Corallo <akrl@sdf.org> said:
>     >> Right. The question is if we want to support such a case (and if so,
>     >> exactly how).
>
>     Andrea> Yep, I really thought the answer was no, at least for now, but I might
>     Andrea> be wrong.
>
>     Andrea> BTW assuming the answer is no, should we add a load time check with
>     Andrea> associated error or warning for comp.el to gate all these possible
>     Andrea> issues?
>
> Iʼd lean towards a warning myself (both for loading and for running
> any of the commands inside comp.el)

FWIW I'd probably go just for the load-time warning.  I fear gating
every function would probably bloat the code for not getting much back.

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Fri, 22 Jul 2022 15:14:21 +0000, Andrea Corallo <akrl@sdf.org> said:

    Andrea> Robert Pluim <rpluim@gmail.com> writes:
    >>>>>>> On Fri, 22 Jul 2022 13:51:04 +0000, Andrea Corallo <akrl@sdf.org> said:
    >> >> Right. The question is if we want to support such a case (and if so,
    >> >> exactly how).
    >> 
    Andrea> Yep, I really thought the answer was no, at least for now, but I might
    Andrea> be wrong.
    >> 
    Andrea> BTW assuming the answer is no, should we add a load time check with
    Andrea> associated error or warning for comp.el to gate all these possible
    Andrea> issues?
    >> 
    >> Iʼd lean towards a warning myself (both for loading and for running
    >> any of the commands inside comp.el)

    Andrea> FWIW I'd probably go just for the load-time warning.  I fear gating
    Andrea> every function would probably bloat the code for not getting much back.

Something like this (untested) in `normal-top-level'?

    (unless (featurep 'native-compile)
      (with-eval-after-load 'comp
        (display-warning 'comp "This emacs does not support native compilation" :warning)))

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Robert Pluim <rpluim@gmail.com>
> Cc: emacs-devel@gnu.org,  Lars Ingebrigtsen <larsi@gnus.org>
> Date: Fri, 22 Jul 2022 17:36:16 +0200
> 
> Something like this (untested) in `normal-top-level'?
> 
>     (unless (featurep 'native-compile)
>       (with-eval-after-load 'comp
>         (display-warning 'comp "This emacs does not support native compilation" :warning)))

Why do we need this, again?

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Fri, 22 Jul 2022 18:41:09 +0300, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Cc: emacs-devel@gnu.org,  Lars Ingebrigtsen <larsi@gnus.org>
    >> Date: Fri, 22 Jul 2022 17:36:16 +0200
    >> 
    >> Something like this (untested) in `normal-top-level'?
    >> 
    >> (unless (featurep 'native-compile)
    >> (with-eval-after-load 'comp
    >> (display-warning 'comp "This emacs does not support native compilation" :warning)))

    Eli> Why do we need this, again?

I had comp.el loaded in my non-native compile capable Emacs, and tried
to run `native-compile-prune-cacheʼ, which gave me an error. Either we
accept that I did something that shouldnʼt be done, or we try to warn
people that they shouldnʼt do it (or on the gripping hand we add code
to check 'native-compile to a bunch of stuff in comp.el).

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Robert Pluim <rpluim@gmail.com>
> Cc: akrl@sdf.org,  emacs-devel@gnu.org,  larsi@gnus.org
> Date: Fri, 22 Jul 2022 17:46:07 +0200
> 
> >>>>> On Fri, 22 Jul 2022 18:41:09 +0300, Eli Zaretskii <eliz@gnu.org> said:
> 
>     >> From: Robert Pluim <rpluim@gmail.com>
>     >> Cc: emacs-devel@gnu.org,  Lars Ingebrigtsen <larsi@gnus.org>
>     >> Date: Fri, 22 Jul 2022 17:36:16 +0200
>     >> 
>     >> Something like this (untested) in `normal-top-level'?
>     >> 
>     >> (unless (featurep 'native-compile)
>     >> (with-eval-after-load 'comp
>     >> (display-warning 'comp "This emacs does not support native compilation" :warning)))
> 
>     Eli> Why do we need this, again?
> 
> I had comp.el loaded in my non-native compile capable Emacs, and tried
> to run `native-compile-prune-cacheʼ, which gave me an error. Either we
> accept that I did something that shouldnʼt be done, or we try to warn
> people that they shouldnʼt do it (or on the gripping hand we add code
> to check 'native-compile to a bunch of stuff in comp.el).

What would be the reason to load comp.el in such a session?

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Robert Pluim]

>>>>> On Fri, 22 Jul 2022 19:18:19 +0300, Eli Zaretskii <eliz@gnu.org> said:

    >> From: Robert Pluim <rpluim@gmail.com>
    >> Cc: akrl@sdf.org,  emacs-devel@gnu.org,  larsi@gnus.org
    >> Date: Fri, 22 Jul 2022 17:46:07 +0200
    >> 
    >> >>>>> On Fri, 22 Jul 2022 18:41:09 +0300, Eli Zaretskii <eliz@gnu.org> said:
    >> 
    >> >> From: Robert Pluim <rpluim@gmail.com>
    >> >> Cc: emacs-devel@gnu.org,  Lars Ingebrigtsen <larsi@gnus.org>
    >> >> Date: Fri, 22 Jul 2022 17:36:16 +0200
    >> >> 
    >> >> Something like this (untested) in `normal-top-level'?
    >> >> 
    >> >> (unless (featurep 'native-compile)
    >> >> (with-eval-after-load 'comp
    >> >> (display-warning 'comp "This emacs does not support native compilation" :warning)))
    >> 
    Eli> Why do we need this, again?
    >> 
    >> I had comp.el loaded in my non-native compile capable Emacs, and tried
    >> to run `native-compile-prune-cacheʼ, which gave me an error. Either we
    >> accept that I did something that shouldnʼt be done, or we try to warn
    >> people that they shouldnʼt do it (or on the gripping hand we add code
    >> to check 'native-compile to a bunch of stuff in comp.el).

    Eli> What would be the reason to load comp.el in such a session?

User error. Some people might write (require ʼcomp) in their
.emacs. Iʼm sure we can come up with lots of scenarios.

Robert
-- 

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Robert Pluim <rpluim@gmail.com>
> Cc: akrl@sdf.org,  emacs-devel@gnu.org,  larsi@gnus.org
> Date: Fri, 22 Jul 2022 18:41:40 +0200
> 
> >>>>> On Fri, 22 Jul 2022 19:18:19 +0300, Eli Zaretskii <eliz@gnu.org> said:
> 
>     >> From: Robert Pluim <rpluim@gmail.com>
>     >> Cc: akrl@sdf.org,  emacs-devel@gnu.org,  larsi@gnus.org
>     >> Date: Fri, 22 Jul 2022 17:46:07 +0200
>     >> 
>     >> >>>>> On Fri, 22 Jul 2022 18:41:09 +0300, Eli Zaretskii <eliz@gnu.org> said:
>     >> 
>     >> >> From: Robert Pluim <rpluim@gmail.com>
>     >> >> Cc: emacs-devel@gnu.org,  Lars Ingebrigtsen <larsi@gnus.org>
>     >> >> Date: Fri, 22 Jul 2022 17:36:16 +0200
>     >> >> 
>     >> >> Something like this (untested) in `normal-top-level'?
>     >> >> 
>     >> >> (unless (featurep 'native-compile)
>     >> >> (with-eval-after-load 'comp
>     >> >> (display-warning 'comp "This emacs does not support native compilation" :warning)))
>     >> 
>     Eli> Why do we need this, again?
>     >> 
>     >> I had comp.el loaded in my non-native compile capable Emacs, and tried
>     >> to run `native-compile-prune-cacheʼ, which gave me an error. Either we
>     >> accept that I did something that shouldnʼt be done, or we try to warn
>     >> people that they shouldnʼt do it (or on the gripping hand we add code
>     >> to check 'native-compile to a bunch of stuff in comp.el).
> 
>     Eli> What would be the reason to load comp.el in such a session?
> 
> User error. Some people might write (require ʼcomp) in their
> .emacs. Iʼm sure we can come up with lots of scenarios.

If we want to handle user error of loading the file, then placing the
code you suggested in comp.el itself would be TRT, I think.  Not in
startup.el.

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andreas Schwab]

On Jul 22 2022, Eli Zaretskii wrote:

> What would be the reason to load comp.el in such a session?

Customize can do that.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 7578 EB47 D4E5 4D69 2510  2552 DF73 E780 A9DA AEC1
"And now for something completely different."

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Andreas Schwab <schwab@linux-m68k.org>
> Cc: Robert Pluim <rpluim@gmail.com>,  akrl@sdf.org,  emacs-devel@gnu.org,
>   larsi@gnus.org
> Date: Fri, 22 Jul 2022 18:42:28 +0200
> 
> On Jul 22 2022, Eli Zaretskii wrote:
> 
> > What would be the reason to load comp.el in such a session?
> 
> Customize can do that.

In that case, I see no reason to produce any warnings.

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> I had comp.el loaded in my non-native compile capable Emacs, and tried
> to run `native-compile-prune-cacheʼ, which gave me an error.

Let's fix it so it doesn't given an error, then.
It could just give a warning if it's called interactively and there's no
file to prune (that'd be useful also for builds *with* native
compilation).


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

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

Robert Pluim <rpluim@gmail.com> writes:

>>>>>> On Fri, 22 Jul 2022 15:14:21 +0000, Andrea Corallo <akrl@sdf.org> said:
>
>     Andrea> Robert Pluim <rpluim@gmail.com> writes:
>     >>>>>>> On Fri, 22 Jul 2022 13:51:04 +0000, Andrea Corallo <akrl@sdf.org> said:
>     >> >> Right. The question is if we want to support such a case (and if so,
>     >> >> exactly how).
>     >> 
>     Andrea> Yep, I really thought the answer was no, at least for now, but I might
>     Andrea> be wrong.
>     >> 
>     Andrea> BTW assuming the answer is no, should we add a load time check with
>     Andrea> associated error or warning for comp.el to gate all these possible
>     Andrea> issues?
>     >> 
>     >> Iʼd lean towards a warning myself (both for loading and for running
>     >> any of the commands inside comp.el)
>
>     Andrea> FWIW I'd probably go just for the load-time warning.  I fear gating
>     Andrea> every function would probably bloat the code for not getting much back.
>
> Something like this (untested) in `normal-top-level'?
>
>     (unless (featurep 'native-compile)
>       (with-eval-after-load 'comp
>         (display-warning 'comp "This emacs does not support native compilation" :warning)))

Yes something similar, wouldn't the attached be simpler?

BR

  Andrea


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Warning-the-user-if-loading-comp-on-a-non-native-cap.patch --]
[-- Type: text/x-diff, Size: 798 bytes --]

From 81bfe715e2137f0572bb17a51bdac3e4f8cd106d Mon Sep 17 00:00:00 2001
From: Andrea Corallo <akrl@sdf.org>
Date: Fri, 22 Jul 2022 18:46:33 +0200
Subject: [PATCH] * Warning the user if loading comp on a non native capable
 Emacs.

* lisp/emacs-lisp/comp.el: Warning the user if loading on a non native
capable Emacs.
---
 lisp/emacs-lisp/comp.el | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/lisp/emacs-lisp/comp.el b/lisp/emacs-lisp/comp.el
index 9a635a4776..b9de0289ee 100644
--- a/lisp/emacs-lisp/comp.el
+++ b/lisp/emacs-lisp/comp.el
@@ -28,6 +28,10 @@
 
 ;;; Code:
 
+(unless (featurep 'native-compile)
+  (display-warning 'comp "This emacs does not support native compilation"
+                   :warning))
+
 (require 'bytecomp)
 (require 'cl-extra)
 (require 'cl-lib)
-- 
2.25.1

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Andrea Corallo <akrl@sdf.org>
> Cc: emacs-devel@gnu.org, Lars Ingebrigtsen <larsi@gnus.org>
> Date: Fri, 22 Jul 2022 16:50:24 +0000
> 
> > Something like this (untested) in `normal-top-level'?
> >
> >     (unless (featurep 'native-compile)
> >       (with-eval-after-load 'comp
> >         (display-warning 'comp "This emacs does not support native compilation" :warning)))
> 
> Yes something similar, wouldn't the attached be simpler?

What will that do if comp.el is loaded by custom.el?

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Andrea Corallo <akrl@sdf.org>
>> Cc: emacs-devel@gnu.org, Lars Ingebrigtsen <larsi@gnus.org>
>> Date: Fri, 22 Jul 2022 16:50:24 +0000
>> 
>> > Something like this (untested) in `normal-top-level'?
>> >
>> >     (unless (featurep 'native-compile)
>> >       (with-eval-after-load 'comp
>> >         (display-warning 'comp "This emacs does not support native compilation" :warning)))
>> 
>> Yes something similar, wouldn't the attached be simpler?
>
> What will that do if comp.el is loaded by custom.el?

AFAIU it should just emit the warning when comp.el is loaded.

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Andrea Corallo <akrl@sdf.org>
> Cc: rpluim@gmail.com, emacs-devel@gnu.org, larsi@gnus.org
> Date: Fri, 22 Jul 2022 18:24:51 +0000
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> From: Andrea Corallo <akrl@sdf.org>
> >> Cc: emacs-devel@gnu.org, Lars Ingebrigtsen <larsi@gnus.org>
> >> Date: Fri, 22 Jul 2022 16:50:24 +0000
> >> 
> >> > Something like this (untested) in `normal-top-level'?
> >> >
> >> >     (unless (featurep 'native-compile)
> >> >       (with-eval-after-load 'comp
> >> >         (display-warning 'comp "This emacs does not support native compilation" :warning)))
> >> 
> >> Yes something similar, wouldn't the attached be simpler?
> >
> > What will that do if comp.el is loaded by custom.el?
> 
> AFAIU it should just emit the warning when comp.el is loaded.

Is that a good idea for such a scenario?

Perhaps we should have a variable to inhibit the warning, which
custom.el could bind?

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Andrea Corallo <akrl@sdf.org>
>> Cc: rpluim@gmail.com, emacs-devel@gnu.org, larsi@gnus.org
>> Date: Fri, 22 Jul 2022 18:24:51 +0000
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> >> From: Andrea Corallo <akrl@sdf.org>
>> >> Cc: emacs-devel@gnu.org, Lars Ingebrigtsen <larsi@gnus.org>
>> >> Date: Fri, 22 Jul 2022 16:50:24 +0000
>> >> 
>> >> > Something like this (untested) in `normal-top-level'?
>> >> >
>> >> >     (unless (featurep 'native-compile)
>> >> >       (with-eval-after-load 'comp
>> >> >         (display-warning 'comp "This emacs does not support native compilation" :warning)))
>> >> 
>> >> Yes something similar, wouldn't the attached be simpler?
>> >
>> > What will that do if comp.el is loaded by custom.el?
>> 
>> AFAIU it should just emit the warning when comp.el is loaded.
>
> Is that a good idea for such a scenario?

I'm not sure why custom should load it (not a custom expert here), but
assuming it is for making any use of it I'd say it is.

Maybe an example of why custom should load comp would make my ideas more
clear on this.

Thanks

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> I'm not sure why custom should load it (not a custom expert here), but
> assuming it is for making any use of it I'd say it is.
>
> Maybe an example of why custom should load comp would make my ideas more
> clear on this.

It's not just custom, there are all kinds of cases.
E.g. earlier, I loaded `comp.elc` into my no-native-compiler Emacs
session because I did `C-h o native-compile-prune-cache RET` which loaded
the file to find the docstring.
[ Actually, it loaded it before I hit RET, because I used completion,
  which triggered loading of `comp.elc` in order to provide completions
  for the `native-compile-` prefix.  ]


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> I'm not sure why custom should load it (not a custom expert here), but
>> assuming it is for making any use of it I'd say it is.
>>
>> Maybe an example of why custom should load comp would make my ideas more
>> clear on this.
>
> It's not just custom, there are all kinds of cases.
> E.g. earlier, I loaded `comp.elc` into my no-native-compiler Emacs
> session because I did `C-h o native-compile-prune-cache RET` which loaded
> the file to find the docstring.
> [ Actually, it loaded it before I hit RET, because I used completion,
>   which triggered loading of `comp.elc` in order to provide completions
>   for the `native-compile-` prefix.  ]

I see, yeah than I defently agree we need to support loading comp.el
also on a non native compiled Emacs.

BTW, now that we have sympol positions, I'd like to extend comp.el for
compile time analysis/diagnostic and this should be useful to all Emacs
confs, so before or later loading (but also using) comp on all confs is
probably something we might want to do.

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> Iʼd lean towards a warning myself (both for loading and for running
> any of the commands inside comp.el)

Please don't warn when loading.  There can be all kinds of valid reasons
to load a file whose functions you have no intention to call.


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> do we support loading comp.el and using in non native builds?

We should support loading `comp.el`, yes.
Using it (e.g. calling functions therein) probably not unless the call
can be given a reasonable semantics.


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Andrea Corallo]

Stefan Monnier <monnier@iro.umontreal.ca> writes:

>> do we support loading comp.el and using in non native builds?
>
> We should support loading `comp.el`, yes.
> Using it (e.g. calling functions therein) probably not unless the call
> can be given a reasonable semantics.

Sounds sensible, but then should we protect all functions or just say
that using them not supported?

  Andrea

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

Andrea Corallo [2022-07-22 23:17:44] wrote:
> Stefan Monnier <monnier@iro.umontreal.ca> writes:
>>> do we support loading comp.el and using in non native builds?
>>
>> We should support loading `comp.el`, yes.
>> Using it (e.g. calling functions therein) probably not unless the call
>> can be given a reasonable semantics.
>
> Sounds sensible, but then should we protect all functions or just say
> that using them not supported?

I don't think we can/should have a general policy in this respect.
I'd rather do it on a case-by-case basis.

Regarding "prune cache" I earlier suggested that it could just "prune
nothing" when called from an Emacs with no native-compiler, but I later
saw that the docstring says:

    Remove .eln files that aren’t applicable to the current Emacs invocation.

so another valid option would be for it to delete *all* the .eln files
when called from an Emacs without support for native compilation
(because none of the .eln files are "applicable to the current Emacs
invocation").


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Eli Zaretskii]

> From: Stefan Monnier <monnier@iro.umontreal.ca>
> Cc: Robert Pluim <rpluim@gmail.com>,  emacs-devel@gnu.org,  Lars
>  Ingebrigtsen <larsi@gnus.org>
> Date: Fri, 22 Jul 2022 19:38:09 -0400
> 
> Regarding "prune cache" I earlier suggested that it could just "prune
> nothing" when called from an Emacs with no native-compiler, but I later
> saw that the docstring says:
> 
>     Remove .eln files that aren’t applicable to the current Emacs invocation.
> 
> so another valid option would be for it to delete *all* the .eln files
> when called from an Emacs without support for native compilation
> (because none of the .eln files are "applicable to the current Emacs
> invocation").

I think this would be a very dangerous thing to do, especially if
someone has more than one build of the same Emacs version on the
system.  (I also don't think it could be possible, because important
variables needed by that function are undefined in such a build, which
is IMO fortunate.)

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Stefan Monnier]

> I think this would be a very dangerous thing to do,

That's also my opinion, BTW :-)


        Stefan

Re: master 05d240997a 2/2: Add native-compile-prune-cache command

[Lars Ingebrigtsen]

Robert Pluim <rpluim@gmail.com> writes:

> Should we wrap this (and perhaps `startup-redirect-eln-cache') in
> (when native-comp-available-p) ? In a non-native compile Emacs they
> signal an error:
>
> Debugger entered--Lisp error: (void-variable native-comp-eln-load-path)
>   (let ((--dolist-tail-- native-comp-eln-load-path)) (while

I've now improved the error message in native-compile-prune-cache.

I don't think issuing warnings etc when loading comp.el in a
non-nativecomp build is warranted -- you should always be able to load
any .el file in any Emacs version without being bothered by things like
that.

-- 
(domestic pets only, the antidote for overdose, milk.)
   bloggy blog: http://lars.ingebrigtsen.no