Re: feature freeze

Re: feature freeze

[David Kastrup]

Reiner Steib <reinersteib+gmane@imap.cc> writes:

> On Wed, Oct 27 2004, David Kastrup wrote:
>
>> storm@cua.dk (Kim F. Storm) writes:
>>
>>> That's a problem of a feature freeze that's in place for more than 1
>>> year.
>>
>> IIRC, it was more or less this August, give or take a month or two.
>> Certainly not a year.
>
> It was in May, see e.g.
> <URL:http://thread.gmane.org/x5wu3kn0nl.fsf@lola.goethe.zz> or
> <URL:http://thread.gmane.org/gmane.emacs.devel/21327>.

Still quite a short year.

-- 
David Kastrup, Kriemhildstr. 15, 44793 Bochum

Feature Freeze

[Nick Roberts]

The last three bugs that I've seen are due to changes made in the last few
days.  I don't know what problems they solved and clearly the bugs they've
introduced is more apparent than the ones they've fixed.

If the intention really is to make a release how about introducing a freeze
where *all* developers must get approval before making *any* changes?  Or
alternatively let's forget the freeze and allow any changes.  This intermediate
state is a nonsense.


-- 
Nick                                           http://www.inet.net.nz/~nickrob

Re: Feature Freeze

[Chong Yidong]

> If the intention really is to make a release how about introducing a
> freeze where *all* developers must get approval before making *any*
> changes?

The changes to sit-for were discussed at length on this list.

Re: Feature Freeze

[Nick Roberts]

Chong Yidong writes:
 > > If the intention really is to make a release how about introducing a
 > > freeze where *all* developers must get approval before making *any*
 > > changes?
 > 
 > The changes to sit-for were discussed at length on this list.

I'm talking generally.  Fixing a bug introduced into Emacs three years ago
seems like progress.  Fixing a bug introduced three days ago doesn't.


-- 
Nick                                           http://www.inet.net.nz/~nickrob

Feature freeze

[Chong Yidong]

Hi people,

It's time to begin the feature freeze.

A couple of people still have small non-bugfix patches scheduled to go
into CVS; please check these in over the 24 hours, or email me if you
have problems.  Otherwise, don't commit any more new features.  If in
doubt, email emacs-devel for discussion.

Two exceptions: we'll allow the proced.el and rmail-mbox changes.

There currently isn't any detailed schedule for the release.  My hope
(and I think Stefan concurs) is to have 23.1 out in about six months
from now.  Maybe this is optimistic; we'll see.

My rough plan is to go through the bug list and finish those pertinent
to both Emacs 22 and Emacs 23 first.  About 2 weeks from now, I'll start
the Emacs 22.3 pretest, which hopefully won't take much effort and can
be completed in a month or two.  Once 22.3 is released, we'll take
another look at the bug count and see how soon afterwards we can start
the 23.1 pretest.

There are 180+ outstanding bugs, which can be browsed at the bugtracker
here:

http://emacsbugs.donarmstrong.com/cgi-bin/pkgreport.cgi?which=pkg&data=emacs&archive=no&version=&dist=unstable

Let's get to work on these.  If you need help using the bug tracker,
please email me.

Thanks!

Re: Feature freeze

[Kenichi Handa]

In article <87tze6mz3c.fsf@stupidchicken.com>, Chong Yidong <cyd@stupidchicken.com> writes:

> Hi people,
> It's time to begin the feature freeze.

> A couple of people still have small non-bugfix patches scheduled to go
> into CVS; please check these in over the 24 hours, or email me if you
> have problems.  Otherwise, don't commit any more new features.  If in
> doubt, email emacs-devel for discussion.

> Two exceptions: we'll allow the proced.el and rmail-mbox changes.

> There currently isn't any detailed schedule for the release.  My hope
> (and I think Stefan concurs) is to have 23.1 out in about six months
> from now.  Maybe this is optimistic; we'll see.

> My rough plan is to go through the bug list and finish those pertinent
> to both Emacs 22 and Emacs 23 first.  About 2 weeks from now, I'll start
> the Emacs 22.3 pretest, which hopefully won't take much effort and can
> be completed in a month or two.  Once 22.3 is released, we'll take
> another look at the bug count and see how soon afterwards we can start
> the 23.1 pretest.

As I wrote before, I'm now working on the new implementation
of automatic-character-composition that doesn't use text
property.  The work is not for a new feature, but I realized
that an element in composition-function-table must be
changed slightly, and a few new Lisp functions must be
added (in a broad sense, it's a new feature).  Please let me
check-in the necessary changes for that work even after the
feature freeze.

---
Kenichi Handa
handa@ni.aist.go.jp

Feature freeze

[Stefan Monnier]

> But... a formal announcemente about which state are we just now and
> when do we <em>expect</em> to release the first pretest tarball would
> be welcome.

OK, so to clarify:
- We started the feature freeze on July 1.
- The freeze is somewhat progressive/soft and we hope to harden it by end
  of July.
- The first pretest will come out as soon as we don't have enough bugs
  open to keep us busy (more or less).  How long that will take after
  the hard feature freeze is not yet determined.


        Stefan

Feature freeze

[Stefan Monnier]

If you want to install some new features in Emacs's trunk, do it now,
because the trunk will be feature-frozen starting next week-end.


        Stefan "who wouldn't want Santa to come and install some
                undiscussed new feature while we sleep"

Re: Feature freeze

[Jambunathan K]

Stefan

I would like to merge my pending changes to ox-odt.el directly to the
Emacs trunk.

A recent change in LibreOffice 4.1 is causing the documents produced by
the ODT exporter to be "un-openable".  See
http://lists.gnu.org/archive/html/emacs-orgmode/2013-12/msg00047.html

If you are interested you can send me the instruction off the list.

NOTE: In principle, I will not commit my changes to Org-mode repo.  The
rest can be negotiated.

Jambunathan K.

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> If you want to install some new features in Emacs's trunk, do it now,
> because the trunk will be feature-frozen starting next week-end.
>
>
>         Stefan "who wouldn't want Santa to come and install some
>                 undiscussed new feature while we sleep"

Re: Feature freeze

[Stefan Monnier]

> I would like to merge my pending changes to ox-odt.el directly to the
> Emacs trunk.

AFAIK, currently, you haven't signed the necessary copyright paperwork,
so we can't accept your changes.  If you want to contribute patches to
Emacs, please address this first.

As for changes to Org, you can send them via M-x report-emacs-bug or
send them to Org maintainers directly.  The Org code will be sync'd back
to Emacs (they got a special "feature freeze exemption"), so it will
work as well.


        Stefan

Re: Feature freeze

[Jambunathan K]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> I would like to merge my pending changes to ox-odt.el directly to the
>> Emacs trunk.

> If you want to contribute patches to Emacs, please address this first.

Will do (See "Conditions Apply" below).

> As for changes to Org, you can send them via M-x report-emacs-bug or
> send them to Org maintainers directly.  The Org code will be sync'd back
> to Emacs (they got a special "feature freeze exemption"), so it will
> work as well.

As the original author of ox-odt.el, I would like to manage the changes
directly in Emacs trunk.  I don't want to deal with any intermediaries
committing changes on my behalf.

Re: Feature freeze

[Glenn Morris]

Stefan Monnier wrote:

> The Org code will be sync'd back to Emacs (they got a special "feature
> freeze exemption")

Any special reason for this exception?
Is it just "lives in an external repo and is big"?
Because there were large non-bugfix Org mode changes late during
the 24.3 cycle, which were so big they were basically impossible to
check, and they caused some problems:

http://lists.gnu.org/archive/html/emacs-devel/2013-03/msg00108.html

So personally I'm disappointed to see the same thing on the cards again.

Re: Feature freeze

[Stefan Monnier]

> Any special reason for this exception?
> Is it just "lives in an external repo and is big"?

Pretty much, yes.

> Because there were large non-bugfix Org mode changes late during
> the 24.3 cycle, which were so big they were basically impossible to
> check, and they caused some problems:
> http://lists.gnu.org/archive/html/emacs-devel/2013-03/msg00108.html
> So personally I'm disappointed to see the same thing on the cards again.

This "feature freeze exemption" isn't meant to imply "commit any time
you like".  Basically, what it means for me is that if/when they have
code ready to merge into Emacs, we may accept it even after the freeze,
as long as there's still enough time before the release.

E.g. installing a new version in January should be fine, February,
probably, and after that, less likely, tho it depends on how well the
release process is going.  Does that sound good enough for you?


        Stefan

Re: Feature freeze

[Eli Zaretskii]

> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Date: Mon, 16 Dec 2013 13:49:25 -0500
> Cc: emacs-devel@gnu.org
> 
> This "feature freeze exemption" isn't meant to imply "commit any time
> you like".  Basically, what it means for me is that if/when they have
> code ready to merge into Emacs, we may accept it even after the freeze,
> as long as there's still enough time before the release.

How is this different from what I might have in my local feature
branch?

Re: Feature freeze

[Michael Albinus]

Jambunathan K <kjambunathan@gmail.com> writes:

> I would like to merge my pending changes to ox-odt.el directly to the
> Emacs trunk.

[...]

> NOTE: In principle, I will not commit my changes to Org-mode repo.  The
> rest can be negotiated.

IIRC, you have vetoed applying any of your patches to the org-mode
project. Do you have changed your mind?

If not, I believe your patches are not applicable to Emacs proper as well.

> Jambunathan K.

Best regards, Michael.

Re: Feature freeze

[Jambunathan K]

Michael Albinus <michael.albinus@gmx.de> writes:

>> NOTE: In principle, I will not commit my changes to Org-mode repo.  The
>> rest can be negotiated.
>
> IIRC, you have vetoed applying any of your patches to the org-mode
> project.

There is no such veto.

I will not personally commit to Orgmode repo.  If the code gets merged
from Emacs repo back to Orgmode repo (by someone else) I have nothing
against it.

> Do you have changed your mind?

I have never changed my mind.

ODT exporter was created as a "donation" to Emacs.  It will remain so.
I still answer questions on the mailing list and accumulate patches in a
private repo.

Re: Feature freeze

[Michael Albinus]

Jambunathan K <kjambunathan@gmail.com> writes:

> Michael Albinus <michael.albinus@gmx.de> writes:
>
>>> NOTE: In principle, I will not commit my changes to Org-mode repo.  The
>>> rest can be negotiated.
>>
>> IIRC, you have vetoed applying any of your patches to the org-mode
>> project.
>
> There is no such veto.
>
> I will not personally commit to Orgmode repo.  If the code gets merged
> from Emacs repo back to Orgmode repo (by someone else) I have nothing
> against it.
>
>> Do you have changed your mind?
>
> I have never changed my mind.

Then I must have misunderstood you:

"ox-odt.el/ox-freemind.el: Please remove it from Org core and don't
 distribute these files with Org."

<http://thread.gmane.org/gmane.emacs.orgmode/68127>

"I would like to remove ox-html.el from Org distribution."

<http://thread.gmane.org/gmane.emacs.orgmode/68128>

Best regards, Michael.

Re: Feature freeze

[Jambunathan K]

Michael Albinus <michael.albinus@gmx.de> writes:

> Then I must have misunderstood you:

Yes.  But you are not alone in that camp...

Re: Feature freeze

[Jambunathan K]

> "ox-odt.el/ox-freemind.el: Please remove it from Org core and don't
>  distribute these files with Org."
>
> <http://thread.gmane.org/gmane.emacs.orgmode/68127>
>
> "I would like to remove ox-html.el from Org distribution."
>
> <http://thread.gmane.org/gmane.emacs.orgmode/68128>

Irrelevant.  The code is part of Org-mode repo and it is part of Emacs.

Re: Feature freeze

[Stefan Monnier]

>> This "feature freeze exemption" isn't meant to imply "commit any time
>> you like".  Basically, what it means for me is that if/when they have
>> code ready to merge into Emacs, we may accept it even after the freeze,
>> as long as there's still enough time before the release.
> How is this different from what I might have in my local feature
> branch?

The code coming in from Org's repository has generally gone through some
reasonable beta-testing (and normally, the rest of Emacs
doesn't/shouldn't depend on Org's code, so a problem in Org mode
shouldn't affect the rest of the release too significantly).



        Stefan

Re: Feature freeze

[Michael Albinus]

Jambunathan K <kjambunathan@gmail.com> writes:

>> "ox-odt.el/ox-freemind.el: Please remove it from Org core and don't
>>  distribute these files with Org."
>>
>> <http://thread.gmane.org/gmane.emacs.orgmode/68127>
>>
>> "I would like to remove ox-html.el from Org distribution."
>>
>> <http://thread.gmane.org/gmane.emacs.orgmode/68128>
>
> Irrelevant.  The code is part of Org-mode repo and it is part of Emacs.

I remember you did a lot of people busy with this kind of requests. It
would be annoying to see such messages from you again, that's why I have
asked for your today's position.

Until you claim that those games are over now from your side, I would
recommend not to accept patches from you. But of course, I'm neither the
Emacs nor the Org-mode maintainer.

Best regards, Michael.

Re: Feature freeze

[Jambunathan K]

Michael Albinus <michael.albinus@gmx.de> writes:

> But of course, I'm neither the Emacs nor the Org-mode maintainer.

Yes.

Re: Feature freeze

[Juanma Barranquero]

On Tue, Dec 17, 2013 at 7:52 AM, Michael Albinus <michael.albinus@gmx.de> wrote:

> Until you claim that those games are over now from your side, I would
> recommend not to accept patches from you.

+1

Re: Feature freeze

[Jambunathan K]

Juanma Barranquero <lekktu@gmail.com> writes:

> On Tue, Dec 17, 2013 at 7:52 AM, Michael Albinus <michael.albinus@gmx.de> wrote:
>
>> Until you claim that those games are over now from your side, I would
>> recommend not to accept patches from you.
>
> +1

What sort of absurdity is this.  You take CA or Disclaimer, you put the
patch in Emacs and forget about it.

Hint: Jambunathan K is a GIGO machine.  Next time you talk to me, try
putting in something other than Garbage.

Re: Feature freeze

[Jay Belanger]

Jambunathan K <kjambunathan@gmail.com> writes:

> Juanma Barranquero <lekktu@gmail.com> writes:
>
>> On Tue, Dec 17, 2013 at 7:52 AM, Michael Albinus <michael.albinus@gmx.de> wrote:
>>
>>> Until you claim that those games are over now from your side, I would
>>> recommend not to accept patches from you.
>>
>> +1
>
> What sort of absurdity is this.

The sort that is based on your previous behavior.

Re: Feature freeze

[Jambunathan K]

Jay Belanger <jay.p.belanger@gmail.com> writes:

> The sort that is based on your previous behavior.

What was that previous behaviour based on?  Could you summarize in your
own words why I cancelled my assignment in first place?  Let me see how
much of a context you can provide.

Jambunathan K.

Re: Feature freeze

[Jay Belanger]

> What was that previous behaviour based on?

Your immaturity, I would assume.
Since you refuse to say that your games are over now, I take it you
leave open the possibility that they might continue.

> Could you summarize in your own words why I cancelled my assignment in
> first place?

It was the hissy fit and your stated desire to cause problems that
bothered people, which were not acceptable behavior under any
circumstances.

Re: Feature freeze

[Jambunathan K]

Jay

Don't take covers.  Answer my question or leave the room.

Jambunathan K.

Jay Belanger <jay.p.belanger@gmail.com> writes:

>> What was that previous behaviour based on?
>
> Your immaturity, I would assume.
> Since you refuse to say that your games are over now, I take it you
> leave open the possibility that they might continue.
>
>> Could you summarize in your own words why I cancelled my assignment in
>> first place?
>
> It was the hissy fit and your stated desire to cause problems that
> bothered people, which were not acceptable behavior under any
> circumstances.

Re: Feature freeze

[Glenn Morris]

People, please stop rising to the bait.

Re: Feature freeze

[Jay Belanger]

Jambunathan K <kjambunathan@gmail.com> writes:

> Answer my question or leave the room.

The only worthwhile question that you asked was why people were agreeing
with the statement:

>>> Until you claim that those games are over now from your side, I would
>>> recommend not to accept patches from you.

I explained why people have a problem with your behavior, but you seem
to be ignoring that, and as before, you are behaving poorly.  You seem
intent on throwing another hissy fit, and the concern that you might
again purposely try to cause problems is real.
I'll let others worry about that; as for you:
  *plonk*

Re: Feature freeze

[Jambunathan K]

Jay Belanger <jay.p.belanger@gmail.com> writes:

> I'll let others worry about that

"Better done than said"

Re: Feature freeze

[Jambunathan K]

Stefan

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> I would like to merge my pending changes to ox-odt.el directly to the
>> Emacs trunk.
>
> AFAIK, currently, you haven't signed the necessary copyright paperwork,
> so we can't accept your changes.  If you want to contribute patches to
> Emacs, please address this first.

My offer stands.

Don't hesitate to write to me if there is a felt need for having my
patches in Emacs.  I will oblige in no time.

Jambunathan K

Re: Feature freeze

[Jambunathan K]

Glenn Morris <rgm@gnu.org> writes:

> People, please stop rising to the bait.

Knee-jerk reaction from one and all, save from some quarters.

Re: Feature freeze

[Stefan Monnier]

People, can we please move on to other things than bashing each other?
Thank you,


        Stefan

Re: Feature freeze

[Bozhidar Batsov]

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

Doesn’t it make more sense to have the feature freeze in the beginning of January? Many people will take some time off work around Christmas and will probably have a bit of extra time to work on things they want to include in Emacs 24.4. I was planning to integrate ruby-tools in ruby-mode, but I’m not sure I’ll be able to do it by the end of the week.

--  
Cheers,
Bozhidar


On Sunday, December 15, 2013 at 3:35 PM, Stefan Monnier wrote:

> If you want to install some new features in Emacs's trunk, do it now,
> because the trunk will be feature-frozen starting next week-end.
>  
>  
> Stefan "who wouldn't want Santa to come and install some
> undiscussed new feature while we sleep"
>  
>  



[-- Attachment #2: Type: text/html, Size: 1413 bytes --]

Feature freeze

[Stefan Monnier]

The trunk is now officially "frozen for new features".
IOW, feel free to keep installing bug-fixes, manual updates, and things
like that, but refrain from installing changes that introduce new
features.


        Stefan

Re: Feature freeze

[Dmitry Gutov]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> The trunk is now officially "frozen for new features".
> IOW, feel free to keep installing bug-fixes, manual updates, and things
> like that, but refrain from installing changes that introduce new
> features.

Is it too late to install the "make electric-pair-mode smarter/more
useful" patch? It's been discussed for some time.

Re: Feature freeze

[Leo Liu]

On 2013-12-24 12:49 +0800, Dmitry Gutov wrote:
> Is it too late to install the "make electric-pair-mode smarter/more
> useful" patch? It's been discussed for some time.

I also wish it applied.

Leo

Re: Feature freeze

[Bozhidar Batsov]

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

I'm also in favour of installing the electric-pair-mode patch. It will be a
big improvement for newcomers who are unlikely to use a third-party
alternative (at least not from day 1).


On 24 December 2013 08:08, Leo Liu <sdl.web@gmail.com> wrote:

> On 2013-12-24 12:49 +0800, Dmitry Gutov wrote:
> > Is it too late to install the "make electric-pair-mode smarter/more
> > useful" patch? It's been discussed for some time.
>
> I also wish it applied.
>
> Leo
>
>

[-- Attachment #2: Type: text/html, Size: 890 bytes --]

Re: Feature freeze

[Stefan Monnier]

>> The trunk is now officially "frozen for new features".
>> IOW, feel free to keep installing bug-fixes, manual updates, and things
>> like that, but refrain from installing changes that introduce new
>> features.
> Is it too late to install the "make electric-pair-mode smarter/more
> useful" patch? It's been discussed for some time.

No, this one is "already accepted", so it can be installed.


        Stfean

Re: Feature freeze

[David Engster]

Stefan Monnier writes:
> The trunk is now officially "frozen for new features".
> IOW, feel free to keep installing bug-fixes, manual updates, and things
> like that, but refrain from installing changes that introduce new
> features.

I would very much like to get the better help support for EIEIO methods
and classes in the next release. We discussed it here in February:

http://thread.gmane.org/gmane.emacs.devel/156807

I got sidetracked at that time, but I'm currently working it. The
essential functionality won't change, though: the main work is done in
eieio.el and eieio-opt.el. The only things that get's added to
help-fns.el is the new `help-fns-describe-function-functions' hook.

Would it be OK to apply it in the coming days?

Also, I'd like to do another sync from CEDET upstream, but AFAICS that's
only bugfixes.

-David

Re: Feature freeze

[João Távora]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>>> The trunk is now officially "frozen for new features".
>>> IOW, feel free to keep installing bug-fixes, manual updates, and things
>>> like that, but refrain from installing changes that introduce new
>>> features.
>> Is it too late to install the "make electric-pair-mode smarter/more
>> useful" patch? It's been discussed for some time.
>
> No, this one is "already accepted", so it can be installed.

I can do it myself if someone grants me write permission on Savannah.

BTW that could also help with the (unrelated) synch'ing of yasnippet's
upstream and GNU ELPA's.

João

Re: Feature freeze

[Stefan Monnier]

> I can do it myself if someone grants me write permission on Savannah.

Good idea.  Please request membership in the "emacs" group from your
savannah account,


        Stefan

Re: Feature freeze

[Stefan Monnier]

> I got sidetracked at that time, but I'm currently working it. The
> essential functionality won't change, though: the main work is done in
> eieio.el and eieio-opt.el. The only things that get's added to
> help-fns.el is the new `help-fns-describe-function-functions' hook.
> Would it be OK to apply it in the coming days?

I remember discussions around that, yes.  Send us the patch and we'll
see if that can be installed.

> Also, I'd like to do another sync from CEDET upstream, but AFAICS that's
> only bugfixes.

If it's only bug-fixes then it's fine, yes.


        Stefan

Re: Feature freeze

[Stefan Monnier]

> The trunk is now officially "frozen for new features".

BTW, the plan is to keep it frozen for a month or so, after which we'll
make a release branch and open up the trunk for changes again.


        Stefan

Re: Feature freeze

[João Távora]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

>> I can do it myself if someone grants me write permission on Savannah.
>
> Good idea.  Please request membership in the "emacs" group from your
> savannah account,

So I did and I commited the new features in revno: 115759. Have a look
and tell me how much I messed up :-) Not much I hope...

João

Re: Feature freeze

[João Távora]

>>> I can do it myself if someone grants me write permission on Savannah.
>> Good idea.  Please request membership in the "emacs" group from your
>> savannah account,
> So I did and I commited the new features in revno: 115759. Have a look
> and tell me how much I messed up :-) Not much I hope...

Two things:

* Somehow my patch didn't make it to emacs-diffs, but bzr log reports
  it.

  However, and curiously, I can see it parenting a revision pushed shortly
  thereafter, http://thread.gmane.org/gmane.emacs.diffs/123754.

  Did anything funny happen? I followed the instructions at
  http://www.emacswiki.org/emacs/BzrForEmacsDevs#toc11.

* I noticed only now that the new tests for the "autowrapping"
  functionality in test/automated/electric-tests.el (i.e. the ones that
  use `region-active-p') are broken, but only when run from "make
  check", a target that I became aware of just now.

  Should these be skipped in that situation?

João

Re: Feature freeze

[Dmitry Gutov]

On 27.12.2013 00:16, João Távora wrote:
> So I did and I commited the new features in revno: 115759. Have a look
> and tell me how much I messed up :-) Not much I hope...

Works quite well so far, thank you.

The commit not showing up in emacs-diffs is weird, though. I've seen my 
commits appear out of order, but they did show up eventually.

Re: Feature freeze

[Eli Zaretskii]

> From: joaotavora@gmail.com (João Távora)
> Date: Thu, 26 Dec 2013 23:46:01 +0000
> Cc: emacs-devel@gnu.org, João Távora <joaot@siscog.pt>,
> 	Dmitry Gutov <dgutov@yandex.ru>
> 
> * Somehow my patch didn't make it to emacs-diffs, but bzr log reports
>   it.

emacs-diffs is a moderated list, so messages by non-members get held
until approved by the moderator (who is a human, and needs to sleep
from time to time).

>   However, and curiously, I can see it parenting a revision pushed shortly
>   thereafter, http://thread.gmane.org/gmane.emacs.diffs/123754.
> 
>   Did anything funny happen? I followed the instructions at
>   http://www.emacswiki.org/emacs/BzrForEmacsDevs#toc11.

Nothing funny, thank you.  Your commit was delivered to the repository
as expected, but the email message to emacs-diffs, generated by bzr on
your behalf, was held (until a few minutes ago).

Re: Feature freeze

[Michael Albinus]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> The trunk is now officially "frozen for new features".
> IOW, feel free to keep installing bug-fixes, manual updates, and things
> like that, but refrain from installing changes that introduce new
> features.

Do we have a list of bug numbers, which shall be fixed before the
release? Preferred as (user) tags in debbugs or alike, that the list can
be kept up-to-date.

>         Stefan

Best regards, Michael.

Re: Feature freeze

[Stefan Monnier]

>> The trunk is now officially "frozen for new features".
>> IOW, feel free to keep installing bug-fixes, manual updates, and things
>> like that, but refrain from installing changes that introduce new
>> features.
> Do we have a list of bug numbers, which shall be fixed before the
> release? Preferred as (user) tags in debbugs or alike, that the list can
> be kept up-to-date.

No, but I encourage everyone to tag as "important" any bug that they
think should be fixed.  Of course, that should be someone else's bug.
If you think your own bug-report is important, send me an email.


        Stefan

Re: Feature freeze

[Michael Albinus]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> No, but I encourage everyone to tag as "important" any bug that they
> think should be fixed.  Of course, that should be someone else's bug.
> If you think your own bug-report is important, send me an email.

That will work fine if the bug is related to Emacs only. When it is
assigned to several packages in parallel, like Emacs and Gnus, it is not
obvious whether it must be fixed in the next Emacs release or the next
Gnus release.

But let's start with this. Conflicts in intended release versionss the
bug shall be fixed could be solved when they happen. With the debbugs
package from GNU ELPA, we get the list of bugs:

(debbugs-gnu '("serious" "important") '("emacs") nil t)

Admittedly, the last one has been submitted by me. Maybe somebody else
could check, whether the severity is proper. It is a security flaw, so I
believe it shall be fixed soon.

If one prefers the bug list as TODO items of org-mode, one could apply

(debbugs-org '("serious" "important") '("emacs") nil)

>         Stefan

Best regards, Michael.

Re: Feature freeze

[David Engster]

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

Stefan Monnier writes:
>> I got sidetracked at that time, but I'm currently working it. The
>> essential functionality won't change, though: the main work is done in
>> eieio.el and eieio-opt.el. The only things that get's added to
>> help-fns.el is the new `help-fns-describe-function-functions' hook.
>> Would it be OK to apply it in the coming days?
>
> I remember discussions around that, yes.  Send us the patch and we'll
> see if that can be installed.

Much to my surprise I didn't get much done over the holidays, but
anyway, here it is.

The patch looks large, but as you can see, most of the stuff is just for
displaying the class/constructor/slot descriptions. The real core of the
patch is, as I've written above, the new hook in help-fns.el.

Is it OK to install, or will it have to wait?

-David


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: eieio-help-fns-patch.diff --]
[-- Type: text/x-diff, Size: 16935 bytes --]

=== modified file 'lisp/emacs-lisp/eieio-opt.el'
--- lisp/emacs-lisp/eieio-opt.el	2014-01-01 07:43:34 +0000
+++ lisp/emacs-lisp/eieio-opt.el	2014-01-06 21:32:16 +0000
@@ -77,101 +77,76 @@
 ;;;###autoload(defalias 'describe-class 'eieio-describe-class)
 
 ;;;###autoload
-(defun eieio-describe-class (class &optional headerfcn)
+(defun eieio-describe-class (class)
   "Describe a CLASS defined by a string or symbol.
-If CLASS is actually an object, then also display current values of that object.
-Optional HEADERFCN should be called to insert a few bits of info first."
-  (interactive (list (eieio-read-class "Class: ")))
-  (with-output-to-temp-buffer (help-buffer) ;"*Help*"
-    (help-setup-xref (list #'eieio-describe-class class headerfcn)
-		     (called-interactively-p 'interactive))
-
-    (when headerfcn (funcall headerfcn))
-    (prin1 class)
-    (princ " is a")
-    (if (class-option class :abstract)
-	(princ "n abstract"))
-    (princ " class")
-    ;; Print file location
-    (when (get class 'class-location)
-      (princ " in `")
-      (princ (file-name-nondirectory (get class 'class-location)))
-      (princ "'"))
-    (terpri)
-    ;; Inheritance tree information
-    (let ((pl (eieio-class-parents class)))
-      (when pl
-	(princ " Inherits from ")
-	(while pl
-	  (princ "`") (prin1 (car pl)) (princ "'")
-	  (setq pl (cdr pl))
-	  (if pl (princ ", ")))
-	(terpri)))
-    (let ((ch (eieio-class-children class)))
-      (when ch
-	(princ " Children ")
-	(while ch
-	  (princ "`") (prin1 (car ch)) (princ "'")
-	  (setq ch (cdr ch))
-	  (if ch (princ ", ")))
-	(terpri)))
-    (terpri)
-    ;; System documentation
-    (let ((doc (documentation-property class 'variable-documentation)))
-      (when doc
-	(princ "Documentation:")
-	(terpri)
-	(princ doc)
-	(terpri)
-	(terpri)))
-    ;; Describe all the slots in this class
-    (eieio-describe-class-slots class)
-    ;; Describe all the methods specific to this class.
-    (let ((methods (eieio-all-generic-functions class))
-	  (doc nil))
-      (if (not methods) nil
-	(princ "Specialized Methods:")
-	(terpri)
-	(terpri)
-	(while methods
-	  (setq doc (eieio-method-documentation (car methods) class))
-	  (princ "`")
-	  (prin1 (car methods))
-	  (princ "'")
-	  (if (not doc)
-	      (princ "  Undocumented")
-	    (if (car doc)
-		(progn
-		  (princ "  :STATIC ")
-		  (prin1 (car (car doc)))
-		  (terpri)
-		  (princ (cdr (car doc)))))
-	    (setq doc (cdr doc))
-	    (if (car doc)
-		(progn
-		  (princ "  :BEFORE ")
-		  (prin1 (car (car doc)))
-		  (terpri)
-		  (princ (cdr (car doc)))))
-	    (setq doc (cdr doc))
-	    (if (car doc)
-		(progn
-		  (princ "  :PRIMARY ")
-		  (prin1 (car (car doc)))
-		  (terpri)
-		  (princ (cdr (car doc)))))
-	    (setq doc (cdr doc))
-	    (if (car doc)
-		(progn
-		  (princ "  :AFTER ")
-		  (prin1 (car (car doc)))
-		  (terpri)
-		  (princ (cdr (car doc)))))
-	    (terpri)
-	    (terpri))
-	  (setq methods (cdr methods))))))
-  (with-current-buffer (help-buffer)
-    (buffer-string)))
+If CLASS is actually an object, then also display current values of that object."
+  ;; Header line
+  (prin1 class)
+  (insert " is a"
+	  (if (class-option class :abstract)
+	      "n abstract"
+	    "")
+	  " class")
+  (let ((location (get class 'class-location)))
+    (when location
+      (insert " in `")
+      (help-insert-xref-button
+       (file-name-nondirectory location)
+       'eieio-class-def class location)
+      (insert "'")))
+  (insert ".\n")
+  ;; Parents
+  (let ((pl (eieio-class-parents class))
+	cur)
+    (when pl
+      (insert " Inherits from ")
+      (while (setq cur (pop pl))
+	(insert "`")
+	(help-insert-xref-button (symbol-name cur)
+				 'help-function cur)
+	(insert (if pl "', " "'")))
+      (insert ".\n")))
+  ;; Children
+  (let ((ch (eieio-class-children class))
+	cur)
+    (when ch
+      (insert " Children ")
+      (while (setq cur (pop ch))
+	(insert "`")
+	(help-insert-xref-button (symbol-name cur)
+				 'help-function cur)
+	(insert (if ch "', " "'")))
+      (insert ".\n")))
+  ;; System documentation
+  (let ((doc (documentation-property class 'variable-documentation)))
+    (when doc
+      (insert "\n" doc "\n\n")))
+  ;; Describe all the slots in this class
+  (eieio-describe-class-slots class)
+  ;; Describe all the methods specific to this class.
+  (let ((methods (eieio-all-generic-functions class))
+	(type [":STATIC" ":BEFORE" ":PRIMARY" ":AFTER"])
+	counter doc argshl dochl)
+    (when methods
+      (insert (propertize "Specialized Methods:\n\n" 'face 'bold))
+      (while methods
+	(setq doc (eieio-method-documentation (car methods) class))
+	(insert "`")
+	(help-insert-xref-button (symbol-name (car methods))
+				 'help-function (car methods))
+	(insert "'")
+	(if (not doc)
+	    (insert "  Undocumented")
+	  (setq counter 0)
+	  (dolist (cur doc)
+	    (when cur
+	      (insert " " (aref type counter) " "
+		      (prin1-to-string (car cur) (current-buffer))
+		      "\n"
+		      (cdr cur)))
+	    (setq counter (1+ counter))))
+	(insert "\n\n")
+	(setq methods (cdr methods))))))
 
 (defun eieio-describe-class-slots (class)
   "Describe the slots in CLASS.
@@ -185,28 +160,27 @@
 	 (i      0)
 	 (prot   (eieio--class-protection cv))
 	 )
-    (princ "Instance Allocated Slots:")
-    (terpri)
-    (terpri)
+    (insert (propertize "Instance Allocated Slots:\n\n"
+			'face 'bold))
     (while names
-      (if (car prot) (princ "Private "))
-      (princ "Slot: ")
-      (prin1 (car names))
-      (when (not (eq (aref types i) t))
-	(princ "    type = ")
-	(prin1 (aref types i)))
-      (unless (eq (car deflt) eieio-unbound)
-	(princ "    default = ")
-	(prin1 (car deflt)))
-      (when (car publp)
-	(princ "    printer = ")
-	(prin1 (car publp)))
-      (when (car docs)
-	(terpri)
-	(princ "  ")
-	(princ (car docs))
-	(terpri))
-      (terpri)
+      (insert
+       (concat
+	(when (car prot)
+	  (propertize "Private " 'face 'bold))
+	(propertize "Slot: " 'face 'bold)
+	(prin1-to-string (car names))
+	(unless (eq (aref types i) t)
+	  (concat "    type = "
+		  (prin1-to-string (aref types i))))
+	(unless (eq (car deflt) eieio-unbound)
+	  (concat "    default = "
+		  (prin1-to-string (car deflt))))
+	(when (car publp)
+	  (concat "    printer = "
+		  (prin1-to-string (car publp))))
+	(when (car docs)
+	  (concat "\n  " (car docs) "\n"))
+	"\n"))
       (setq names (cdr names)
 	    docs (cdr docs)
 	    deflt (cdr deflt)
@@ -219,61 +193,30 @@
 	  i     0
 	  prot  (eieio--class-class-allocation-protection cv))
     (when names
-	(terpri)
-	(princ "Class Allocated Slots:"))
-	(terpri)
-	(terpri)
+      (insert (propertize "\nClass Allocated Slots:\n\n" 'face 'bold)))
     (while names
-      (when (car prot)
-	(princ "Private "))
-      (princ "Slot: ")
-      (prin1 (car names))
-      (unless (eq (aref types i) t)
-	(princ "    type = ")
-	(prin1 (aref types i)))
-      (condition-case nil
-	  (let ((value (eieio-oref class (car names))))
-	    (princ "   value = ")
-	    (prin1 value))
+      (insert
+       (concat
+	(when (car prot)
+	  "Private ")
+	"Slot: "
+	(prin1-to-string (car names))
+	(unless (eq (aref types i) t)
+	  (concat "    type = "
+		  (prin1-to-string (aref types i))))
+	(condition-case nil
+	    (let ((value (eieio-oref class (car names))))
+	      (concat "   value = "
+		      (prin1-to-string value)))
 	  (error nil))
-      (when (car docs)
-	(terpri)
-	(princ "  ")
-	(princ (car docs))
-	(terpri))
-      (terpri)
+	(when (car docs)
+	  (concat "\n\n " (car docs) "\n"))
+	"\n"))
       (setq names (cdr names)
 	    docs (cdr docs)
 	    prot (cdr prot)
 	    i (1+ i)))))
 
-;;;###autoload
-(defun eieio-describe-constructor (fcn)
-  "Describe the constructor function FCN.
-Uses `eieio-describe-class' to describe the class being constructed."
-  (interactive
-   ;; Use eieio-read-class since all constructors have the same name as
-   ;; the class they create.
-   (list (eieio-read-class "Class: ")))
-  (eieio-describe-class
-   fcn (lambda ()
-	 ;; Describe the constructor part.
-	 (prin1 fcn)
-	 (princ " is an object constructor function")
-	 ;; Print file location
-	 (when (get fcn 'class-location)
-	   (princ " in `")
-	   (princ (file-name-nondirectory (get fcn 'class-location)))
-	   (princ "'"))
-	 (terpri)
-	 (princ "Creates an object of class ")
-	 (prin1 fcn)
-	 (princ ".")
-	 (terpri)
-	 (terpri)
-	 ))
-  )
-
 (defun eieio-build-class-list (class)
   "Return a list of all classes that inherit from CLASS."
   (if (class-p class)
@@ -330,87 +273,99 @@
 ;;;###autoload(defalias 'describe-generic 'eieio-describe-generic)
 (defalias 'eieio-describe-method 'eieio-describe-generic)
 
-;;;###autoload
-(defun eieio-describe-generic (generic)
-  "Describe the generic function GENERIC.
-Also extracts information about all methods specific to this generic."
-  (interactive (list (eieio-read-generic "Generic Method: ")))
-  (eieio--check-type generic-p generic)
-  (with-output-to-temp-buffer (help-buffer) ; "*Help*"
-    (help-setup-xref (list #'eieio-describe-generic generic)
-		     (called-interactively-p 'interactive))
-
-    (prin1 generic)
-    (princ " is a generic function")
-    (when (generic-primary-only-p generic)
-      (princ " with only ")
-      (when (generic-primary-only-one-p generic)
-	(princ "one "))
-      (princ "primary method")
-      (when (not (generic-primary-only-one-p generic))
-	(princ "s"))
-      )
-    (princ ".")
-    (terpri)
-    (terpri)
-    (let ((d (documentation generic)))
-      (if (not d)
-	  (princ "The generic is not documented.\n")
-	(princ "Documentation:")
-	(terpri)
-	(princ d)
-	(terpri)
-	(terpri)))
-    (princ "Implementations:")
-    (terpri)
-    (terpri)
-    (let ((i 4)
-	  (prefix [ ":STATIC" ":BEFORE" ":PRIMARY" ":AFTER" ] ))
-      ;; Loop over fanciful generics
-      (while (< i 7)
-	(let ((gm (aref (get generic 'eieio-method-tree) i)))
-	  (when gm
-	    (princ "Generic ")
-	    (princ (aref prefix (- i 3)))
-	    (terpri)
-	    (princ (or (nth 2 gm) "Undocumented"))
-	    (terpri)
-	    (terpri)))
-	(setq i (1+ i)))
-      (setq i 0)
-      ;; Loop over defined class-specific methods
-      (while (< i 4)
-	(let ((gm (reverse (aref (get generic 'eieio-method-tree) i)))
-	      location)
-	  (while gm
-	    (princ "`")
-	    (prin1 (car (car gm)))
-	    (princ "'")
-	    ;; prefix type
-	    (princ " ")
-	    (princ (aref prefix i))
-	    (princ " ")
-	    ;; argument list
-	    (let* ((func (cdr (car gm)))
-		   (arglst (eieio-lambda-arglist func)))
-	      (prin1 arglst))
-	    (terpri)
-	    ;; 3 because of cdr
-	    (princ (or (documentation (cdr (car gm)))
-		       "Undocumented"))
-	    ;; Print file location if available
-	    (when (and (setq location (get generic 'method-locations))
-		       (setq location (assoc (caar gm) location)))
-	      (setq location (cadr location))
-	      (princ "\n\nDefined in `")
-	      (princ (file-name-nondirectory location))
-	      (princ "'\n"))
-	    (setq gm (cdr gm))
-	    (terpri)
-	    (terpri)))
-	(setq i (1+ i)))))
-  (with-current-buffer (help-buffer)
-    (buffer-string)))
+(define-button-type 'eieio-method-def
+  :supertype 'help-xref
+  'help-function (lambda (class method file)
+		   (eieio-help-find-method-definition class method file))
+  'help-echo (purecopy "mouse-2, RET: find method's definition"))
+
+(define-button-type 'eieio-class-def
+  :supertype 'help-xref
+  'help-function (lambda (class file)
+		   (eieio-help-find-class-definition class file))
+  'help-echo (purecopy "mouse-2, RET: find class definition"))
+
+;;;###autoload
+(defun eieio-help-constructor (ctr)
+  "Describe CTR if it is a class constructor."
+  (when (class-p ctr)
+    (let ((location (get ctr 'class-location)))
+      (goto-char (point-min))
+      (delete-region (point) (point-at-eol))
+      (prin1 ctr)
+      (insert " is an object constructor function in `")
+      (help-insert-xref-button
+       (file-name-nondirectory location)
+       'eieio-class-def ctr location)
+      (insert "'.\nCreates an object of class " (symbol-name ctr) ".")
+      (goto-char (point-max))
+      (save-excursion
+	(insert (propertize "\n\nClass description:\n" 'face 'bold))
+	(eieio-describe-class ctr))
+      )))
+
+
+;;;###autoload
+(defun eieio-help-generic (generic)
+  "Describe GENERIC if it is a generic function."
+  (when (generic-p generic)
+    (save-excursion
+      (goto-char (point-min))
+      (when (re-search-forward " in `.+'.$" nil t)
+	(replace-match ".")))
+    (save-excursion
+      (insert "\n\nThis is a generic function"
+	      (cond
+	       ((and (generic-primary-only-p generic)
+		     (generic-primary-only-one-p generic))
+		" with only one primary method")
+	       ((generic-primary-only-p generic)
+		" with only primary methods")
+	       (t ""))
+	      ".\n\n")
+      (insert (propertize "Implementations:\n\n" 'face 'bold))
+      (let ((i 4)
+	    (prefix [ ":STATIC" ":BEFORE" ":PRIMARY" ":AFTER" ] ))
+	;; Loop over fanciful generics
+	(while (< i 7)
+	  (let ((gm (aref (get generic 'eieio-method-tree) i)))
+	    (when gm
+	      (insert "Generic "
+		      (aref prefix (- i 3))
+		      "\n"
+		      (or (nth 2 gm) "Undocumented")
+		      "\n\n")))
+	  (setq i (1+ i)))
+	(setq i 0)
+	;; Loop over defined class-specific methods
+	(while (< i 4)
+	  (let* ((gm (reverse (aref (get generic 'eieio-method-tree) i)))
+		 cname location)
+	    (while gm
+	      (setq cname (caar gm))
+	      (insert "`")
+	      (help-insert-xref-button (symbol-name cname)
+				       'help-variable cname)
+	      (insert "' " (aref prefix i) " ")
+	      ;; argument list
+	      (let* ((func (cdr (car gm)))
+		     (arglst (eieio-lambda-arglist func)))
+		(prin1 arglst (current-buffer)))
+	      (insert "\n"
+		      (or (documentation (cdr (car gm)))
+			  "Undocumented"))
+	      ;; Print file location if available
+	      (when (and (setq location (get generic 'method-locations))
+			 (setq location (assoc cname location)))
+		(setq location (cadr location))
+		(insert "\n\nDefined in `")
+		(help-insert-xref-button
+		 (file-name-nondirectory location)
+		 'eieio-method-def cname generic location)
+		(insert "'\n"))
+	      (setq gm (cdr gm))
+	      (insert "\n")))
+	  (setq i (1+ i)))))))
 
 (defun eieio-lambda-arglist (func)
   "Return the argument list of FUNC, a function body."
@@ -584,21 +539,13 @@
 
 ;;; HELP AUGMENTATION
 ;;
-(define-button-type 'eieio-method-def
-  :supertype 'help-xref
-  'help-function (lambda (class method file)
-		   (eieio-help-find-method-definition class method file))
-  'help-echo (purecopy "mouse-2, RET: find method's definition"))
-
-(define-button-type 'eieio-class-def
-  :supertype 'help-xref
-  'help-function (lambda (class file)
-		   (eieio-help-find-class-definition class file))
-  'help-echo (purecopy "mouse-2, RET: find class definition"))
-
 (defun eieio-help-find-method-definition (class method file)
   (let ((filename (find-library-name file))
 	location buf)
+    (when (symbolp class)
+      (setq class (symbol-name class)))
+    (when (symbolp method)
+      (setq method (symbol-name method)))
     (when (null filename)
       (error "Cannot find library %s" file))
     (setq buf (find-file-noselect filename))
@@ -622,6 +569,8 @@
       (beginning-of-line))))
 
 (defun eieio-help-find-class-definition (class file)
+  (when (symbolp class)
+    (setq class (symbol-name class)))
   (let ((filename (find-library-name file))
 	location buf)
     (when (null filename)

=== modified file 'lisp/emacs-lisp/eieio.el'
--- lisp/emacs-lisp/eieio.el	2014-01-01 07:43:34 +0000
+++ lisp/emacs-lisp/eieio.el	2014-01-04 09:04:08 +0000
@@ -865,6 +865,10 @@
 of `eq'."
   (error "EIEIO: `change-class' is unimplemented"))
 
+;; Hook ourselves into help system for describing classes and methods.
+(add-hook 'help-fns-describe-function-functions 'eieio-help-generic)
+(add-hook 'help-fns-describe-function-functions 'eieio-help-constructor)
+
 ;;; Interfacing with edebug
 ;;
 (defun eieio-edebug-prin1-to-string (object &optional noescape)

=== modified file 'lisp/help-fns.el'
--- lisp/help-fns.el	2014-01-01 07:43:34 +0000
+++ lisp/help-fns.el	2014-01-04 09:04:08 +0000
@@ -32,6 +32,12 @@
 
 ;;; Code:
 
+(defvar help-fns-describe-function-functions nil
+  "List of functions to run in help buffer in `describe-function'.
+Those functions will be run after the header line and argument
+list was inserted, and before the documentation will be inserted.
+The functions will receive the function name as argument.")
+
 ;; Functions
 
 ;;;###autoload
@@ -653,7 +659,7 @@
           (help-fns--compiler-macro function)
           (help-fns--parent-mode function)
           (help-fns--obsolete function)
-
+	  (run-hook-with-args 'help-fns-describe-function-functions function)
           (insert "\n"
                   (or doc "Not documented.")))))))
 

Re: Feature freeze

[Stefan Monnier]

> Is it OK to install, or will it have to wait?

Looks OK.  Please install, tho see nitpicks below.

> +  ;; Describe all the slots in this class

Please punctuate the comments (IIUC you didn't actually change this
comment, but while you're there, you might as well do it ;-).

> +(defvar help-fns-describe-function-functions nil
> +  "List of functions to run in help buffer in `describe-function'.
> +Those functions will be run after the header line and argument
> +list was inserted, and before the documentation will be inserted.
> +The functions will receive the function name as argument.")
> +
>  ;; Functions
> 
>  ;;;###autoload
> @@ -653,7 +659,7 @@
>            (help-fns--compiler-macro function)
>            (help-fns--parent-mode function)
>            (help-fns--obsolete function)
> -
> +	  (run-hook-with-args 'help-fns-describe-function-functions function)

Looks good, but please move help-fns--compiler-macro,
help-fns--parent-mode, help-fns--obsolete to
help-fns-describe-function-functions while you're there.


        Stefan

Re: Feature freeze

[David Engster]

Stefan Monnier writes:
>>  ;;;###autoload
>> @@ -653,7 +659,7 @@
>>            (help-fns--compiler-macro function)
>>            (help-fns--parent-mode function)
>>            (help-fns--obsolete function)
>> -
>> +	  (run-hook-with-args 'help-fns-describe-function-functions function)
>
> Looks good, but please move help-fns--compiler-macro,
> help-fns--parent-mode, help-fns--obsolete to
> help-fns-describe-function-functions while you're there.

Yes, will do.

One more question which I forgot to ask: I did rename
`eieio-describe-constructor' and `eieio-describe-generic' to
`eieio-help-constructor' and `eieio-help-generic', since they are not
interactive anymore but augment an existing help buffer. Those two
however had aliases `describe-constructor' and `describe-generic', which
I'd like to remove. Would that be OK? I don't think anyone was using
those, and calling the normal `describe-function' on a constructor or
generic will now give the same information.

If you'd like to keep them, I'd simply re-implement them as small
wrappers for `describe-function'.

-David

Re: Feature freeze

[Stefan Monnier]

> interactive anymore but augment an existing help buffer. Those two
> however had aliases `describe-constructor' and `describe-generic', which
> I'd like to remove. Would that be OK?

Yes, thanks,


        Stefan

Re: Feature freeze

[David Engster]

Stefan Monnier writes:
>> interactive anymore but augment an existing help buffer. Those two
>> however had aliases `describe-constructor' and `describe-generic', which
>> I'd like to remove. Would that be OK?
>
> Yes, thanks,

Pushed. I will update NEWS tomorrow.

What I didn't find out is how to update the extracted autoloads at the
end of lisp/emacs-lisp/eieio.el, so those are still wrong. 'make
autoloads' doesn't do the trick. Any hints?

-David

Re: Feature freeze

[Glenn Morris]

David Engster wrote:

> What I didn't find out is how to update the extracted autoloads at the
> end of lisp/emacs-lisp/eieio.el, so those are still wrong. 'make
> autoloads' doesn't do the trick. Any hints?

There's something weird there.
How did those autoloads get into eieio.el in the first place, when it is
not the value of any other file's generated-autoload-file setting?
(They are also in loaddefs.el, which is where `make autoloads' will
update them.)

Maybe there was such a setting at some point in the past, but it was
removed? Or were those autoloads added to eieio.el by hand?

I suggest you fix it up by hand.

Re: Feature freeze

[Glenn Morris]

Glenn Morris wrote:

> Maybe there was such a setting at some point in the past, but it was
> removed?

Yes: I intentionally added it in r103332.
It was removed it r110325 "Update CEDET from upstream".
I guess that was unintentional.

I suggest you check that revision and make any necessary fixes.


In a way, this is an autoloads bug:
If a file foo.el used to contain a file-local generated-autoload-file: bar.el
setting, but then it gets removed, the autoloads will not be removed
from bar.el.

Re: Feature freeze

[Eli Zaretskii]

> From: Glenn Morris <rgm@gnu.org>
> Date: Wed, 08 Jan 2014 17:37:31 -0500
> 
> In a way, this is an autoloads bug:
> If a file foo.el used to contain a file-local generated-autoload-file: bar.el
> setting, but then it gets removed, the autoloads will not be removed
> from bar.el.

Perhaps we should fix this, either in the Makefile or maybe via some
file-local variables wizardry.

Re: Feature freeze

[David Engster]

Glenn Morris writes:
> Glenn Morris wrote:
>
>> Maybe there was such a setting at some point in the past, but it was
>> removed?
>
> Yes: I intentionally added it in r103332.
> It was removed it r110325 "Update CEDET from upstream".
> I guess that was unintentional.

Yes, that was my fault. Thanks for looking into this.

> I suggest you check that revision and make any necessary fixes.

But I guess there must be some reason why you used 'eieio.el' as
generated autoload file instead of using loaddefs.el? Is it OK to just
remove the autoloads from eieio.el and fix AUTOGEN_VCS in Makefile.in,
or should I restore it to the way it was?

-David

Re: Feature freeze

[Glenn Morris]

David Engster wrote:

> But I guess there must be some reason why you used 'eieio.el' as
> generated autoload file instead of using loaddefs.el? 

I did that because those autoloads were already in eieio.el, but
hand-written rather than auto-generated.

You know better than I where those autoloads need to be.
Do they only make sense after eieio is loaded, or are they external
access points.

Re: Feature freeze

[David Engster]

Glenn Morris writes:
> David Engster wrote:
>> But I guess there must be some reason why you used 'eieio.el' as
>> generated autoload file instead of using loaddefs.el? 
>
> I did that because those autoloads were already in eieio.el, but
> hand-written rather than auto-generated.

Ah yes, I now dimly remember writing those...

> You know better than I where those autoloads need to be.
> Do they only make sense after eieio is loaded, or are they external
> access points.

They really only make sense when EIEIO is already loaded, so I restored
the old behavior.

-David

Re: Feature freeze

[Nix]

On 6 Jan 2014, David Engster outgrape:
> Much to my surprise I didn't get much done over the holidays, but
> anyway, here it is.

As an eieio user long frustrated by the opacity caused by its poor help
integration:

Best patch so far this year! Thank you thank you!

(not that that's really saying *that* much, what with freeze and New
Year and all. But still.)

Feature freeze

[Stefan Monnier]

Time is up!

I think it's about time we freeze the features for Emacs-25.  There are
a few things I hope can still make it into this new release, (such as
the xwidget branch (long overdue) and the dynload/modules, for example),
so the exact schedule and details are still up for discussion.

But I'll leave these decisions to someone else, because I also take this
opportunity to step down as head maintainer.

It's time for me to move on, and it's time for new blood to take
the lead.  I'm not about to disappear, but I won't be reading
emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
something from me, put me explicitly in the Cc.

Thank you all for bearing with me as head maintainer, it's been a great
ride, I hope you enjoyed it as much as I did.


        Stefan

Re: Feature freeze

[Kaushal Modi]

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

Hey Stefan,

Wishing you success with your next venture too. It was great to have
discussions and learn from you on this list.

You will be missed. Looking forward to what new ideas are being put into
the pipeline with the help and support of the next head maintainer.

Kaushal
On Sep 21, 2015 4:43 PM, "Stefan Monnier" <monnier@iro.umontreal.ca> wrote:

> Time is up!
>
> I think it's about time we freeze the features for Emacs-25.  There are
> a few things I hope can still make it into this new release, (such as
> the xwidget branch (long overdue) and the dynload/modules, for example),
> so the exact schedule and details are still up for discussion.
>
> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.
>
> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.
>
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.
>
>
>         Stefan
>
>

[-- Attachment #2: Type: text/html, Size: 1590 bytes --]

Re: Feature freeze

[John Yates]

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

I regularly describe to fellow programmers how vibrant the emacs project
has become.  You deserve the lion's share of the credit.  May we be blessed
with a new head maintainer as competent, creative, courteous and visionary
as you have been

Best of luck,

/john

[-- Attachment #2: Type: text/html, Size: 374 bytes --]

Re: Feature freeze

[Rasmus]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> Time is up!

Cool! 

> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.
>
> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.
>
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

Damn, I hate Mondays...  Well, best of luck!  And thanks a lot for all of
your contributions thus far and also future contributions.

Rasmus

-- 
Dobbelt-A

Re: Feature freeze

[Xue Fuqiao]

Hi Stefan,

Thanks for your great 7 years of being an elite technical leader, and
pulling Emacs forward for 10+ years!  Especially thanks for your
contributions to vc-svn, Emacs server, PCL-CVS (just like Magit for
Git!), smerge-mode and diff-mode, mpc, smie, pcase, cl-lib (CL at
runtime!), gv, nadvice (which is really easy to use) and countless
contributions to GNU ELPA, *.texi and doc strings, Unicode support,
CEDET, plus many other places all over Emacs.

Personally, I'd like to express my gratitude for your guidance and
patience, which makes me more proficient in Emacs development.  Wish you
success with your next venture!

Re: Feature freeze

[Eli Zaretskii]

> From: Stefan Monnier <monnier@IRO.UMontreal.CA>
> Date: Mon, 21 Sep 2015 15:39:55 -0400
> 
> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.
> 
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

We did enjoy that ride, and you _will_ be missed.  Thank you for those
wonderful years, and I hope your "infrequent" appearances here will be
more frequent.

Re: Feature freeze

[martin rudalics]

 > I also take this opportunity to step down as head maintainer.

I was afraid this would happen.

Thanks for bearing with us, martin

Re: Feature freeze

[Zack Piper]

On Mon, 21 Sep 2015 21:39:55 +0200,
Stefan Monnier wrote:
> 
> Time is up!
> 
> I think it's about time we freeze the features for Emacs-25.  There are
> a few things I hope can still make it into this new release, (such as
> the xwidget branch (long overdue) and the dynload/modules, for example),
> so the exact schedule and details are still up for discussion.

I'm looking forward to these features.

> 
> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.

Thank you for your hard work, and good luck in your future endeavors.

> 
> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.
> 
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.
> 
> 
>         Stefan
> 

Re: Feature freeze

[Aurélien Aptel]

On Mon, Sep 21, 2015 at 9:39 PM, Stefan Monnier
<monnier@iro.umontreal.ca> wrote:
> I think it's about time we freeze the features for Emacs-25.  There are
> a few things I hope can still make it into this new release, (such as
> the xwidget branch (long overdue) and the dynload/modules, for example),
> so the exact schedule and details are still up for discussion.

Yeah I would also like to have the modules in too! I'll update the
relevant thread.

> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

Aww :( You will be missed.
You always took time to explain stuff when I asked dumb questions,
really appreciated that :D

Re: Feature freeze

[Artur Malabarba]

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

Thanks for all your work and time, Stefan. You've done a brilliant job. And
thanks for dragging me onto this mess as well.

Best of luck,
Artur

[-- Attachment #2: Type: text/html, Size: 187 bytes --]

Re: Feature freeze

[Eric Abrahamsen]

Aurélien Aptel <aurelien.aptel+emacs@gmail.com> writes:

> On Mon, Sep 21, 2015 at 9:39 PM, Stefan Monnier
> <monnier@iro.umontreal.ca> wrote:
>> I think it's about time we freeze the features for Emacs-25.  There are
>> a few things I hope can still make it into this new release, (such as
>> the xwidget branch (long overdue) and the dynload/modules, for example),
>> so the exact schedule and details are still up for discussion.
>
> Yeah I would also like to have the modules in too! I'll update the
> relevant thread.
>
>> Thank you all for bearing with me as head maintainer, it's been a great
>> ride, I hope you enjoyed it as much as I did.
>
> Aww :( You will be missed.
> You always took time to explain stuff when I asked dumb questions,
> really appreciated that :D

Amen to this. I got a lot of assistance and encouragement, and that made
the process of contributing to (and using) Emacs really pleasurable.
Thanks, and stick around.

Re: Feature freeze

[Alan Mackenzie]

Hello, Stefan.

On Mon, Sep 21, 2015 at 03:39:55PM -0400, Stefan Monnier wrote:
> Time is up!

> I think it's about time we freeze the features for Emacs-25.  There are
> a few things I hope can still make it into this new release, (such as
> the xwidget branch (long overdue) and the dynload/modules, for example),
> so the exact schedule and details are still up for discussion.

> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.

> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.

> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

Yes, all good things come to an end.  Thanks for all the hard work over
the years (and decades), and wishing you a more relaxed and less
stressful future.

We're going to miss you!

>         Stefan

-- 
Alan Mackenzie (Nuremberg, Germany).

Re: Feature freeze

[Tassilo Horn]

Artur Malabarba <bruce.connor.am@gmail.com> writes:

> Thanks for all your work and time, Stefan.  You've done a brilliant
> job.

I second this.  You've been a fantastic maintainer!

> And thanks for dragging me onto this mess as well.

Oh, yes.  Thanks also for dragging Artur into this mess! :-)

Bye,
Tassilo

Re: Feature freeze

[Oleh Krehel]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

You were a great maintainer, thanks for all your work.
And best of luck in your new projects.

    Oleh

Re: Feature freeze

[Dmitry Gutov]

Hello Stefan,

On 09/21/2015 10:39 PM, Stefan Monnier wrote:

> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

Thank you very much for your work, patience and fruitful discussions 
over the years (as well as the less fruitful ones, all of those are 
surely my fault).

It's hard for me to imagine Emacs development without your guiding hand, 
but I hope we'll manage somehow.

Best of luck in all your endeavors. Come back to govern anytime. ;)

Re: Feature freeze

[Paul Eggert]

On 09/21/2015 12:39 PM, Stefan Monnier wrote:
> Thank you all for bearing with me as head maintainer

It was more the other way round: you were bearing us, and doing it so 
well that we all got a little spoiled.  Thanks for the ride, and please 
don't be a stranger.

Re: Feature freeze

[Nicolas Petton]

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

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

Hi Stefan,

> I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.

I'm wondering how it will work from there.  Will you look for a new head
maintainer, or is this task up to somebody else?  Or maybe we already
have a new head maintainer and I missed it?

Cheers,
Nico

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 512 bytes --]

Re: Feature freeze

[Richard Stallman]

[[[ To any NSA and FBI agents reading my email: please consider    ]]]
[[[ whether defending the US Constitution against all enemies,     ]]]
[[[ foreign or domestic, requires you to follow Snowden's example. ]]]

We need a new Emacs maintainer (better, two).  I have asked Stefan to
help recruit the new maintainers and work with them in a transition.

-- 
Dr Richard Stallman
President, Free Software Foundation (gnu.org, fsf.org)
Internet Hall-of-Famer (internethalloffame.org)
Skype: No way! See stallman.org/skype.html.

Re: Feature freeze

[Bastien]

Paul Eggert <eggert@cs.ucla.edu> writes:

> On 09/21/2015 12:39 PM, Stefan Monnier wrote:
>> Thank you all for bearing with me as head maintainer
>
> It was more the other way round: you were bearing us, and doing it so
> well that we all got a little spoiled.*

Exactly my feelings.  *Thanks* Stefan!

-- 
 Bastien

Re: Feature freeze

[Leo Liu]

On 2015-09-22 03:39 +0800, Stefan Monnier wrote:
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.

Stefan, thanks for the generous contribution to emacs which has become
my favourite development environment. Thanks also for teaching me many
things emacs ;)

Leo

Re: Feature freeze

[Daniel Colascione]

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

On 09/21/2015 12:39 PM, Stefan Monnier wrote:
> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.

Thanks for your hard work and numerous improvements to Emacs. Good luck
with everything!


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

Re: Feature freeze

[John Wiegley]

>>>>> Daniel Colascione <dancol@dancol.org> writes:

> On 09/21/2015 12:39 PM, Stefan Monnier wrote:
>> But I'll leave these decisions to someone else, because I also take this
>> opportunity to step down as head maintainer.

> Thanks for your hard work and numerous improvements to Emacs. Good luck with
> everything!

Indeed, Stefan, I can't thank you enough. Emacs is the center of my electronic
life!

John

Re: Feature freeze

[joakim]

Stefan Monnier <monnier@IRO.UMontreal.CA> writes:

> Time is up!
>
> I think it's about time we freeze the features for Emacs-25.  There are
> a few things I hope can still make it into this new release, (such as
> the xwidget branch (long overdue) and the dynload/modules, for example),
> so the exact schedule and details are still up for discussion.

It would be pretty nice to finally merge the Xwidget branch.

I'm not sure how to proceed though. I'm pretty swamped with work at the
moment, and I would likely need some assistance.

> But I'll leave these decisions to someone else, because I also take this
> opportunity to step down as head maintainer.
>
> It's time for me to move on, and it's time for new blood to take
> the lead.  I'm not about to disappear, but I won't be reading
> emacs-devel (nor bug-gnu-emacs) at least for a while, so if you need
> something from me, put me explicitly in the Cc.
>
> Thank you all for bearing with me as head maintainer, it's been a great
> ride, I hope you enjoyed it as much as I did.
>
>
>         Stefan
>

-- 
Joakim Verona