* bug#16998: with-slots should expand to cl-symbol-macrolet
@ 2014-03-12 14:22 Jonas Bernoulli
2014-03-12 15:12 ` Juanma Barranquero
` (2 more replies)
0 siblings, 3 replies; 10+ messages in thread
From: Jonas Bernoulli @ 2014-03-12 14:22 UTC (permalink / raw
To: 16998
Currently with-slots expands to symbol-macrolet
but should instead expand to cl-symbol-macrolet.
Best regards,
Jonas
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 14:22 bug#16998: with-slots should expand to cl-symbol-macrolet Jonas Bernoulli
@ 2014-03-12 15:12 ` Juanma Barranquero
2014-03-12 15:35 ` Stefan Monnier
2020-08-16 18:57 ` Jonas Bernoulli
2 siblings, 0 replies; 10+ messages in thread
From: Juanma Barranquero @ 2014-03-12 15:12 UTC (permalink / raw
To: Jonas Bernoulli; +Cc: 16998
> Currently with-slots expands to symbol-macrolet
> but should instead expand to cl-symbol-macrolet.
I've committed this change.
But still, eieio.el should require 'cl-lib, or `cl-symbol-macrolet'
should have an autoload.
(progn
(require 'eieio)
(defclass test nil ((test-slot)))
(with-slots ((s test-slot)) (make-instance 'test)
(message "Slot = %s" s)))
Debugger entered--Lisp error: (void-function cl-symbol-macrolet)
Stefan?
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 14:22 bug#16998: with-slots should expand to cl-symbol-macrolet Jonas Bernoulli
2014-03-12 15:12 ` Juanma Barranquero
@ 2014-03-12 15:35 ` Stefan Monnier
2014-03-12 16:04 ` Juanma Barranquero
2014-03-12 18:00 ` David Engster
2020-08-16 18:57 ` Jonas Bernoulli
2 siblings, 2 replies; 10+ messages in thread
From: Stefan Monnier @ 2014-03-12 15:35 UTC (permalink / raw
To: Jonas Bernoulli; +Cc: 16998
> Currently with-slots expands to symbol-macrolet
> but should instead expand to cl-symbol-macrolet.
Current EIEIO still uses `cl' rather than `cl-lib', because it's part of
CEDET which wants to run on older Emacsen.
I'm waiting for the CEDET guys to drop EIEIO from CEDET (and rely on
Emacs's builtin EIEIO instead) to move EIEIO to `cl-lib'.
Some rumors make me think it could happen "real soon now", but it's been
that way for a while now, so maybe the rumor is wrong, or my
understanding of it is wrong.
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 15:35 ` Stefan Monnier
@ 2014-03-12 16:04 ` Juanma Barranquero
2014-03-12 18:00 ` David Engster
1 sibling, 0 replies; 10+ messages in thread
From: Juanma Barranquero @ 2014-03-12 16:04 UTC (permalink / raw
To: Stefan Monnier; +Cc: Jonas Bernoulli, 16998
On Wed, Mar 12, 2014 at 4:35 PM, Stefan Monnier
<monnier@iro.umontreal.ca> wrote:
> Current EIEIO still uses `cl' rather than `cl-lib', because it's part of
> CEDET which wants to run on older Emacsen.
My change shouldn't cause problems, because if you require 'cl you
require cl-lib.
But if you want to revert it, we still have a problem that an autoload
for symbol-macrolet or a (require 'cl) is needed.
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 15:35 ` Stefan Monnier
2014-03-12 16:04 ` Juanma Barranquero
@ 2014-03-12 18:00 ` David Engster
2014-03-12 18:46 ` Stefan Monnier
2014-03-12 22:12 ` Eric M. Ludlam
1 sibling, 2 replies; 10+ messages in thread
From: David Engster @ 2014-03-12 18:00 UTC (permalink / raw
To: Stefan Monnier; +Cc: Jonas Bernoulli, 16998, Eric M. Ludlam
Stefan Monnier writes:
>> Currently with-slots expands to symbol-macrolet
>> but should instead expand to cl-symbol-macrolet.
>
> Current EIEIO still uses `cl' rather than `cl-lib', because it's part of
> CEDET which wants to run on older Emacsen.
>
> I'm waiting for the CEDET guys to drop EIEIO from CEDET (and rely on
> Emacs's builtin EIEIO instead) to move EIEIO to `cl-lib'.
>
> Some rumors make me think it could happen "real soon now", but it's been
> that way for a while now, so maybe the rumor is wrong, or my
> understanding of it is wrong.
Yes, I'm sorry about causing this confusion. I tried to remove it, but
it turned out to be more difficult than it looked, mostly because of the
changes we made w.r.t. the EDE security fix, which introduced
type-checking for slots. Older EIEIO versions in Emacs <23.4 don't have
that, which make them pretty much incompatible with current CEDET, which
is why we still have it upstream.
However, I think that there's no problem to "officially" state that
EIEIO development now happens in Emacs, and that the version in CEDET
will not get updated anymore, but will remain there as a fallback
library for older Emacsen (similar to what Gnus does).
Eric, would you be OK with that? That would allow us to update the
built-in EIEIO to current Emacs coding standards without breaking
CEDET-from-bzr on older Emacsen.
-David
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 18:00 ` David Engster
@ 2014-03-12 18:46 ` Stefan Monnier
2014-03-12 22:12 ` Eric M. Ludlam
1 sibling, 0 replies; 10+ messages in thread
From: Stefan Monnier @ 2014-03-12 18:46 UTC (permalink / raw
To: David Engster; +Cc: Jonas Bernoulli, 16998, Eric M. Ludlam
> Yes, I'm sorry about causing this confusion.
No need to be sorry. It's not an urgent task anyway.
> I tried to remove it, but it turned out to be more difficult than it
> looked, mostly because of the changes we made w.r.t. the EDE security
> fix, which introduced type-checking for slots. Older EIEIO versions in
> Emacs <23.4 don't have that, which make them pretty much incompatible
> with current CEDET, which is why we still have it upstream.
Aha, so the rumor was wrong ;-)
> However, I think that there's no problem to "officially" state that
> EIEIO development now happens in Emacs, and that the version in CEDET
> will not get updated anymore, but will remain there as a fallback
> library for older Emacsen (similar to what Gnus does).
That would be fine by me. I just want to avoid having two divergent
versions, so if the version bundled with CEDET is frozen it's fine since
it means it won't diverge.
Stefan
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 18:00 ` David Engster
2014-03-12 18:46 ` Stefan Monnier
@ 2014-03-12 22:12 ` Eric M. Ludlam
2014-03-12 22:52 ` David Engster
1 sibling, 1 reply; 10+ messages in thread
From: Eric M. Ludlam @ 2014-03-12 22:12 UTC (permalink / raw
To: David Engster; +Cc: Jonas Bernoulli, 16998
On 03/12/2014 02:00 PM, David Engster wrote:
> Stefan Monnier writes:
>>> Currently with-slots expands to symbol-macrolet
>>> but should instead expand to cl-symbol-macrolet.
>>
>> Current EIEIO still uses `cl' rather than `cl-lib', because it's part of
>> CEDET which wants to run on older Emacsen.
>>
>> I'm waiting for the CEDET guys to drop EIEIO from CEDET (and rely on
>> Emacs's builtin EIEIO instead) to move EIEIO to `cl-lib'.
>>
>> Some rumors make me think it could happen "real soon now", but it's been
>> that way for a while now, so maybe the rumor is wrong, or my
>> understanding of it is wrong.
>
> Yes, I'm sorry about causing this confusion. I tried to remove it, but
> it turned out to be more difficult than it looked, mostly because of the
> changes we made w.r.t. the EDE security fix, which introduced
> type-checking for slots. Older EIEIO versions in Emacs<23.4 don't have
> that, which make them pretty much incompatible with current CEDET, which
> is why we still have it upstream.
>
> However, I think that there's no problem to "officially" state that
> EIEIO development now happens in Emacs, and that the version in CEDET
> will not get updated anymore, but will remain there as a fallback
> library for older Emacsen (similar to what Gnus does).
>
> Eric, would you be OK with that? That would allow us to update the
> built-in EIEIO to current Emacs coding standards without breaking
> CEDET-from-bzr on older Emacsen.
This proposal is fine by me. Short term we will just need to find a
good way to have standalone CEDET bootstrap in with either the Emacs
version, or with the one in CEDET. Eventually this will become
unnecessary.
The CEDET bootstrapping still cares about some features missing from
Emacs (according to the comments) such as:
;; Currently, Emacs proper doesn't track EIEIO methods. Until it
;; does, we have to advice `describe-variable' and `describe-function'
;; for EIEIO methods to get better help buffers.
and
;; This adds further formatting and hyperlinks.
(add-hook 'temp-buffer-show-hook 'eieio-help-mode-augmentation-maybee t)
I think we will just need to check the Emacs version, and change how we
bootstrap and compile. I'm interested in any advice on this.
Thanks
Eric
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 22:12 ` Eric M. Ludlam
@ 2014-03-12 22:52 ` David Engster
0 siblings, 0 replies; 10+ messages in thread
From: David Engster @ 2014-03-12 22:52 UTC (permalink / raw
To: Eric M. Ludlam; +Cc: Jonas Bernoulli, 16998
Eric M. Ludlam writes:
> On 03/12/2014 02:00 PM, David Engster wrote:
>> However, I think that there's no problem to "officially" state that
>> EIEIO development now happens in Emacs, and that the version in CEDET
>> will not get updated anymore, but will remain there as a fallback
>> library for older Emacsen (similar to what Gnus does).
>>
>> Eric, would you be OK with that? That would allow us to update the
>> built-in EIEIO to current Emacs coding standards without breaking
>> CEDET-from-bzr on older Emacsen.
>
> This proposal is fine by me. Short term we will just need to find a
> good way to have standalone CEDET bootstrap in with either the Emacs
> version, or with the one in CEDET.
Yes. I'd move EIEIO upstream in some 'fallback-libs' directory or
similar, and put this in load-path when Emacs version is <23.4 during
compilation as well as when CEDET is loaded.
> Eventually this will become unnecessary.
Yes. Our minimum version currently is 23.2, and as long as it does not
become too painful, there's no need to bump it.
> The CEDET bootstrapping still cares about some features missing from
> Emacs (according to the comments) such as:
>
> ;; Currently, Emacs proper doesn't track EIEIO methods. Until it
> ;; does, we have to advice `describe-variable' and `describe-function'
> ;; for EIEIO methods to get better help buffers.
>
> and
>
> ;; This adds further formatting and hyperlinks.
> (add-hook 'temp-buffer-show-hook 'eieio-help-mode-augmentation-maybee t)
Actually, that feature will be part of the upcoming 24.4 release, so it
won't be needed there.
> I think we will just need to check the Emacs version, and change how
> we bootstrap and compile. I'm interested in any advice on this.
I don't think this should be too difficult, but if it turns out to be a
nightmare, I think we could bump the minimum version to 23.4 without
alienating too many people...
-David
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2014-03-12 14:22 bug#16998: with-slots should expand to cl-symbol-macrolet Jonas Bernoulli
2014-03-12 15:12 ` Juanma Barranquero
2014-03-12 15:35 ` Stefan Monnier
@ 2020-08-16 18:57 ` Jonas Bernoulli
2020-08-16 19:09 ` Lars Ingebrigtsen
2 siblings, 1 reply; 10+ messages in thread
From: Jonas Bernoulli @ 2020-08-16 18:57 UTC (permalink / raw
To: 16998
>> Currently with-slots expands to symbol-macrolet
>> but should instead expand to cl-symbol-macrolet.
> I've committed this change.
This is what this issue was about and it has been taken care of a long
long time ago.
> But still, eieio.el should require 'cl-lib, or `cl-symbol-macrolet'
> should have an autoload.
Nowadays it requires `eieio-core', which requires `cl-lib', so this has
been taken care of as well.
> I'm waiting for the CEDET guys to drop EIEIO from CEDET (and rely on
> Emacs's builtin EIEIO instead) to move EIEIO to `cl-lib'.
Done too.
This issue can be closed.
Jonas
^ permalink raw reply [flat|nested] 10+ messages in thread
* bug#16998: with-slots should expand to cl-symbol-macrolet
2020-08-16 18:57 ` Jonas Bernoulli
@ 2020-08-16 19:09 ` Lars Ingebrigtsen
0 siblings, 0 replies; 10+ messages in thread
From: Lars Ingebrigtsen @ 2020-08-16 19:09 UTC (permalink / raw
To: Jonas Bernoulli; +Cc: 16998
Jonas Bernoulli <jonas@bernoul.li> writes:
> This issue can be closed.
OK; closing.
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2020-08-16 19:09 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-03-12 14:22 bug#16998: with-slots should expand to cl-symbol-macrolet Jonas Bernoulli
2014-03-12 15:12 ` Juanma Barranquero
2014-03-12 15:35 ` Stefan Monnier
2014-03-12 16:04 ` Juanma Barranquero
2014-03-12 18:00 ` David Engster
2014-03-12 18:46 ` Stefan Monnier
2014-03-12 22:12 ` Eric M. Ludlam
2014-03-12 22:52 ` David Engster
2020-08-16 18:57 ` Jonas Bernoulli
2020-08-16 19:09 ` Lars Ingebrigtsen
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).