Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

[Bastien Guerry]

Ihor Radchenko  writes:

> And they reverted the warning at the end.
> https://git.savannah.gnu.org/cgit/emacs.git/commit?id=d51de0c5d90117bc1dc4bc5bc700253d71cd4579

Yes, it makes sense.

-- 
 Bastien Guerry

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

[Ihor Radchenko]

Ihor Radchenko  writes:

>>   ⛔ Warning (lexical-binding): File
>>   /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding'
>>   directive on its first line
>>
>> The attached patch should fix this.  Is it okay to add it to bugfix?
>
> I think that it is premature.
> This is a warning recently introduced on Emacs master, and it is still
> being debated upon. See
> https://yhetil.org/emacs-devel/50e29988-d354-4d10-990f-31828dbe6...@gmail.com/t/#u

And they reverted the warning at the end.
https://git.savannah.gnu.org/cgit/emacs.git/commit?id=d51de0c5d90117bc1dc4bc5bc700253d71cd4579

Canceled.

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at .
Support Org development at ,
or support my work at 

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

[Bastien Guerry]

Ihor Radchenko  writes:

> I think that it is premature.
> This is a warning recently introduced on Emacs master, and it is still
> being debated upon. See
> https://yhetil.org/emacs-devel/50e29988-d354-4d10-990f-31828dbe6...@gmail.com/t/#u

Okay, fair enough.

-- 
 Bastien Guerry

Re: [PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

[Ihor Radchenko]

Bastien  writes:

> Recent versions of Emacs complain about org-clock-save.el:
>
>   ⛔ Warning (lexical-binding): File
>   /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding'
>   directive on its first line
>
> The attached patch should fix this.  Is it okay to add it to bugfix?

I think that it is premature.
This is a warning recently introduced on Emacs master, and it is still
being debated upon. See
https://yhetil.org/emacs-devel/50e29988-d354-4d10-990f-31828dbe6...@gmail.com/t/#u

-- 
Ihor Radchenko // yantar92,
Org mode contributor,
Learn more about Org mode at .
Support Org development at ,
or support my work at 

[PATCH] Remove warning about org-clock-save.el lacking lexical-binding directive

[Bastien]

Recent versions of Emacs complain about org-clock-save.el:

  ⛔ Warning (lexical-binding): File
  /home/bzg/.emacs.d/org-clock-save.el lacks `lexical-binding'
  directive on its first line

The attached patch should fix this.  Is it okay to add it to bugfix?

Thanks,

-- 
 Bastien
diff --git a/lisp/org-clock.el b/lisp/org-clock.el
index 67feb6b35..806bc02de 100644
--- a/lisp/org-clock.el
+++ b/lisp/org-clock.el
@@ -3233,7 +3233,7 @@ The details of what will be saved are regulated by the variable
 		 org-clock-has-been-used
 		 (not (file-exists-p org-clock-persist-file
 (with-temp-file org-clock-persist-file
-  (insert (format ";; %s - %s at %s\n"
+  (insert (format ";; %s - %s at %s -*- lexical-binding: t; -*-\n"
 		  (file-name-nondirectory org-clock-persist-file)
 		  (system-name)
 		  (format-time-string (org-time-stamp-format t