Re: [Emacs-diffs] master 1f7b602 1/3: Add commentary about #$ in autoload files

Re: [Emacs-diffs] master 1f7b602 1/3: Add commentary about #$ in autoload files

[Stefan Monnier]

> +                ;; Use the `#$' to indicate current file, from which
> +                ;; we extract the directory name.  Note that
> +                ;; `package-quickstart-refresh' specifically replaces
> +                ;; `#$', so any other tricks (e.g., `load-file-name')
> +                ;; will not work for that case.

This is not true: load-file-name should work just as well
(package-quickstart-refresh surrounds the code with a let-binding of
load-file-name for that reason; sadly this let-binding doesn't work for
#$ because the #$ expansion is performed while reading the whole form
(the one that contains the let-binding) and hence before the let-binding
takes effect).

> -	    ";; no-byte-compile: t\n"
> +            ";; no-byte-compile: t\n" ;; #$ is byte-compiled into nil.

This, on the other hand, is very true, thanks.


        Stefan

Re: [Emacs-diffs] master 1f7b602 1/3: Add commentary about #$ in autoload files

[Noam Postavsky]

On Sat, 6 Jul 2019 at 18:38, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
>
> > +                ;; Use the `#$' to indicate current file, from which
> > +                ;; we extract the directory name.  Note that
> > +                ;; `package-quickstart-refresh' specifically replaces
> > +                ;; `#$', so any other tricks (e.g., `load-file-name')
> > +                ;; will not work for that case.
>
> This is not true: load-file-name should work just as well
> (package-quickstart-refresh surrounds the code with a let-binding of
> load-file-name for that reason;

Oh, hmm, then why do we use #$ instead of load-file-name?

Re: [Emacs-diffs] master 1f7b602 1/3: Add commentary about #$ in autoload files

[Stefan Monnier]

> Oh, hmm, then why do we use #$ instead of load-file-name?

I don't think there's a strong reason for that here.


        Stefan