GNU bug report logs - #5642
Reporting a bug loads many Gnus libraries

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 5642 in the body.
You can then email your comments to 5642 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #3 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: submit <at> debbugs.gnu.org
Subject: Reporting a bug loads many Gnus libraries
Date: Wed, 24 Feb 2010 13:31:24 -0500

Current trunk on GNU/Linux:

emacs -Q
M-x report-emacs-bug

Typing C-c C-c to send the bug loads 22 extra libraries, including
several Gnus ones:

    gnus-msg gnus-art mm-uu mml2015 epg-config mm-view smime dig gnus-sum
    nnoo gnus-group gnus-undo nnmail mail-source format-spec gnus-start
    gnus-spec gnus-int gnus-range gnus-win gnus gnus-ems

This extra complexity leads to unexpected bugs like #5299.

In contrast, with mail-user-agent = sendmail-user-agent, sending a
report loads exactly 1 extra library (mail-utils).

After sending a bug report with message-user-agent, there are 146
libraries loaded. With sendmail-user-agent, the number is 93.

Message #6 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Wed, 24 Feb 2010 16:32:00 -0500

> Current trunk on GNU/Linux:

> emacs -Q
> M-x report-emacs-bug

> Typing C-c C-c to send the bug loads 22 extra libraries, including
> several Gnus ones:

>     gnus-msg gnus-art mm-uu mml2015 epg-config mm-view smime dig gnus-sum
>     nnoo gnus-group gnus-undo nnmail mail-source format-spec gnus-start
>     gnus-spec gnus-int gnus-range gnus-win gnus gnus-ems

I'm not worried about loading more libraries: the change is specifically
to provide more features so extra code is to be expected.  It'd be good
to try and reduce the reliance on Gnus libraries there, tho, or to
rename/restructure them to make it more clear that they're actually
useful independently from the Gnus MUA.

> This extra complexity leads to unexpected bugs like #5299.

Actually, I see no evidence that #5299 is linked to any of
that complexity.


        Stefan

Message #9 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Thu, 25 Feb 2010 13:24:50 -0500

Stefan Monnier wrote:

> I'm not worried about loading more libraries: the change is specifically
> to provide more features so extra code is to be expected.

For fun, I estimated the difference as ~ 75000 extra lines. This is 3
times the size of the entire lisp/mail directory, and about 50% of the
size of the combined mail+gnus directories.

I think that, just maybe, some of those extra libraries aren't really
necessary to send my bug report.

Message #12 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Stefan Monnier <monnier <at> iro.umontreal.ca>
Cc: rgm <at> gnu.org, 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Thu, 25 Feb 2010 21:28:40 +0200

> From: Stefan Monnier <monnier <at> iro.umontreal.ca>
> Date: Wed, 24 Feb 2010 16:32:00 -0500
> Cc: 5642 <at> debbugs.gnu.org
> 
> > Typing C-c C-c to send the bug loads 22 extra libraries, including
> > several Gnus ones:
> 
> >     gnus-msg gnus-art mm-uu mml2015 epg-config mm-view smime dig gnus-sum
> >     nnoo gnus-group gnus-undo nnmail mail-source format-spec gnus-start
> >     gnus-spec gnus-int gnus-range gnus-win gnus gnus-ems
> 
> I'm not worried about loading more libraries: the change is specifically
> to provide more features so extra code is to be expected.

Why isn't each library loaded only when the respective feature(s)
is/are needed?

Message #15 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Thu, 25 Feb 2010 14:45:24 -0500

Glenn Morris wrote:

> Typing C-c C-c to send the bug loads 22 extra libraries, including
> several Gnus ones:

AFAICS, this is all due to the call to gnus-setup-posting-charset in
message-send-mail.

AFAICS, when the message being sent is a mail message (rather than a
news one), this function just returns (nil).

Message #18 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Thu, 25 Feb 2010 16:12:37 -0500

Glenn Morris wrote:

> AFAICS, this is all due to the call to gnus-setup-posting-charset in
> message-send-mail.

Here are two alternative solutions to this.

(Fun-fact: requiring gnus-msg loads 59 libraries.)

Solution 1: (note, gnus-s-p-c is only used in message and mml)

*** lisp/gnus/mml.el	2010-01-13 08:35:10 +0000
--- lisp/gnus/mml.el	2010-02-25 20:59:43 +0000
***************
*** 35,41 ****
  (eval-when-compile (require 'cl))
  
  (autoload 'message-make-message-id "message")
! (autoload 'gnus-setup-posting-charset "gnus-msg")
  (autoload 'gnus-make-local-hook "gnus-util")
  (autoload 'message-fetch-field "message")
  (autoload 'message-mark-active-p "message")
--- 35,41 ----
  (eval-when-compile (require 'cl))
  
  (autoload 'message-make-message-id "message")
! (declare-function gnus-setup-posting-charset "gnus-msg" (group))
  (autoload 'gnus-make-local-hook "gnus-util")
  (autoload 'message-fetch-field "message")
  (autoload 'message-mark-active-p "message")
***************
*** 1445,1450 ****
--- 1445,1451 ----
    (setq mml-preview-buffer (generate-new-buffer
  			    (concat (if raw "*Raw MIME preview of "
  				      "*MIME preview of ") (buffer-name))))
+   (require 'gnus-msg)		      ; for gnus-setup-posting-charset
    (save-excursion
      (let* ((buf (current-buffer))
  	   (message-options message-options)


Solution 2:

*** lisp/gnus/message.el	2010-02-24 03:12:54 +0000
--- lisp/gnus/message.el	2010-02-25 20:55:53 +0000
***************
*** 4417,4423 ****
  	 (mailbuf (current-buffer))
  	 (message-this-is-mail t)
  	 (message-posting-charset
! 	  (if (fboundp 'gnus-setup-posting-charset)
  	      (gnus-setup-posting-charset nil)
  	    message-posting-charset))
  	 (headers message-required-mail-headers))
--- 4422,4429 ----
  	 (mailbuf (current-buffer))
  	 (message-this-is-mail t)
  	 (message-posting-charset
! 	  (if (and (fboundp 'gnus-setup-posting-charset)
! 		   (boundp 'gnus-group-posting-charset-alist))
  	      (gnus-setup-posting-charset nil)
  	    message-posting-charset))
  	 (headers message-required-mail-headers))

Message #21 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Glenn Morris <rgm <at> gnu.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 5642 <at> debbugs.gnu.org, Stefan Monnier <monnier <at> iro.umontreal.ca>
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Thu, 25 Feb 2010 23:47:10 -0500

Eli Zaretskii wrote:

> Why isn't each library loaded only when the respective feature(s)
> is/are needed?

With a bit of playing around in this way, I managed to avoid 30
libraries being loaded. I'll install these changes after 23.2.

Message #24 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Stefan Monnier <monnier <at> iro.umontreal.ca>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 5642 <at> debbugs.gnu.org, Eli Zaretskii <eliz <at> gnu.org>
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Fri, 26 Feb 2010 00:24:22 -0500

>> Why isn't each library loaded only when the respective feature(s)
>> is/are needed?

> With a bit of playing around in this way, I managed to avoid 30
> libraries being loaded. I'll install these changes after 23.2.

Thank you,


        Stefan

Message #27 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Richard Stallman <rms <at> gnu.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 5642 <at> debbugs.gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Fri, 26 Feb 2010 01:01:52 -0500

    Typing C-c C-c to send the bug loads 22 extra libraries, including
    several Gnus ones:

	gnus-msg gnus-art mm-uu mml2015 epg-config mm-view smime dig gnus-sum
	nnoo gnus-group gnus-undo nnmail mail-source format-spec gnus-start
	gnus-spec gnus-int gnus-range gnus-win gnus gnus-ems

This is one of the bad things about message mode
which should have been fixed before considering making it the default.

Message #30 received at 5642 <at> debbugs.gnu.org (full text, mbox):

From: Richard Stallman <rms <at> gnu.org>
To: Glenn Morris <rgm <at> gnu.org>
Cc: 5642 <at> debbugs.gnu.org, eliz <at> gnu.org
Subject: Re: bug#5642: Reporting a bug loads many Gnus libraries
Date: Fri, 26 Feb 2010 21:32:41 -0500

    With a bit of playing around in this way, I managed to avoid 30
    libraries being loaded. I'll install these changes after 23.2.

I think you should install them now, because they are a bug fix.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.