From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: What holds the release Date: Wed, 15 Jun 2005 12:21:43 -0400 Message-ID: References: <8564wi2ag7.fsf@lola.goethe.zz> <17070.36649.525270.871681@farnswood.snap.net.nz> <3158.220.255.175.166.1118847958.squirrel@www.stupidchicken.com> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii X-Trace: sea.gmane.org 1118852700 22868 80.91.229.2 (15 Jun 2005 16:25:00 GMT) X-Complaints-To: usenet@sea.gmane.org NNTP-Posting-Date: Wed, 15 Jun 2005 16:25:00 +0000 (UTC) Cc: Juanma Barranquero , Eli Zaretskii , snogglethorpe@gmail.com, emacs-devel@gnu.org, miles@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Wed Jun 15 18:24:57 2005 Return-path: Original-Received: from lists.gnu.org ([199.232.76.165]) by ciao.gmane.org with esmtp (Exim 4.43) id 1DiagU-00069H-OF for ged-emacs-devel@m.gmane.org; Wed, 15 Jun 2005 18:24:11 +0200 Original-Received: from localhost ([127.0.0.1] helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1Dialj-0007Dp-HQ for ged-emacs-devel@m.gmane.org; Wed, 15 Jun 2005 12:29:35 -0400 Original-Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1DiakK-0006vD-4F for emacs-devel@gnu.org; Wed, 15 Jun 2005 12:28:08 -0400 Original-Received: from exim by lists.gnu.org with spam-scanned (Exim 4.43) id 1DiakE-0006sD-HA for emacs-devel@gnu.org; Wed, 15 Jun 2005 12:28:03 -0400 Original-Received: from [199.232.76.173] (helo=monty-python.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1DiakE-0006qw-04 for emacs-devel@gnu.org; Wed, 15 Jun 2005 12:28:02 -0400 Original-Received: from [132.204.24.67] (helo=mercure.iro.umontreal.ca) by monty-python.gnu.org with esmtp (Exim 4.34) id 1Diafk-0007aQ-29; Wed, 15 Jun 2005 12:23:24 -0400 Original-Received: from hidalgo.iro.umontreal.ca (hidalgo.iro.umontreal.ca [132.204.27.50]) by mercure.iro.umontreal.ca (Postfix) with ESMTP id 74F0E2CF503; Wed, 15 Jun 2005 12:21:48 -0400 (EDT) Original-Received: from asado.iro.umontreal.ca (asado.iro.umontreal.ca [132.204.24.84]) by hidalgo.iro.umontreal.ca (Postfix) with ESMTP id 702604AC008; Wed, 15 Jun 2005 12:21:44 -0400 (EDT) Original-Received: by asado.iro.umontreal.ca (Postfix, from userid 20848) id 5891DE6BD3; Wed, 15 Jun 2005 12:21:44 -0400 (EDT) Original-To: "Chong Yidong" In-Reply-To: <3158.220.255.175.166.1118847958.squirrel@www.stupidchicken.com> (Chong Yidong's message of "Wed, 15 Jun 2005 11:05:58 -0400 (EDT)") User-Agent: Gnus/5.11 (Gnus v5.11) Emacs/22.0.50 (gnu/linux) X-DIRO-MailScanner-Information: Please contact the ISP for more information X-DIRO-MailScanner: Found to be clean X-DIRO-MailScanner-SpamCheck: n'est pas un polluriel, SpamAssassin (score=-4.844, requis 5, autolearn=not spam, AWL 0.06, BAYES_00 -4.90) X-MailScanner-From: monnier@iro.umontreal.ca X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:38891 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:38891 >> Make VC-over-Tramp work where possible, or at least fail >> gracefully if something isn't supported over Tramp. >> To be done by Andre Spiegel . > This has been sitting there for several months now. Has it already been > done? Is it a major inconvenience worth delaying 22.1 for? I'd happily drop it. >> define-minor-mode should not put :require into defcustom. >> See msg from rms to emacs-devel on 21 Dec. > The relevant URL is > http://lists.gnu.org/archive/html/emacs-devel/2004-12/msg00732.html > I've tried working on this, with no result. I couldn't find a clean way > to implement RMS's suggestion, and I'm not sure it even makes sense. A > workaround for the particular problem originally reported by Stephen Stahl > is to add (require 'font-lock) to font-core.el and a ":require 'font-lock" > tag to the definition of global-font-lock-mode. Maybe we should just do > that, and wait for 22.2 for whatever general solution is required. (It > would be *nice* to get it into 22.1, but what it would *not* be nice to > delay 22.1 into 2006 just for that.) Looking at it some more, here is my thoughts about it: - this problem is only relevant for global minor modes (buffer-local minor modes can't be enabled/disabled via custom). - every global minor mode is either pre-loaded or autoloaded. So we should simply never add thje :require directive and we just need to make sure that the :setter info (custom-set-minor-mode) is included in loaddefs.el for the autoloaded vars so that enabling/disabling will go through the minor mode function and trigger the autoloading. I.e. the sample patch below seems to fix the problem. >> ** Update Speedbar. > The relevant URL is > http://lists.gnu.org/archive/html/emacs-devel/2005-05/msg00180.html > It sounds like this will be fixed soon. Hopefully, yes. >> Enhance scroll-bar to handle tall line (similar to line-move). > This is a feature request, not a show-stopping bug. In the real world, > 99.9% of users don't use Emacs as an image viewer; they use a specialized > image viewing program. Emacs is currently a mediocre image viewer; it > would be nice if it were a great image viewer, but not essential for the > release. 100% agreement. >> Adapt mouse-sel-mode to mouse-1-click-follows-link. > I fixed this a couple weeks ago. This entry should be removed. Done, thanks. >> Make GTK scrollbars behave like others w.r.t. overscrolling. > I don't experience any problem with GTK scrollbars. It does not seem to > be a show-stopper. Agreed. >> Avoid unbreakable loops in redisplay. > This is an "it would be nice" feature, not a show-stopper. It would be > nice to have a safety feature to avoid running inappropriate display > properties, but AFAIK people aren't actually being affected by such bugs. > This shouldn't block the release. Agreed. Stefan Index: lisp/emacs-lisp/easy-mmode.el =================================================================== RCS file: /cvsroot/emacs/emacs/lisp/emacs-lisp/easy-mmode.el,v retrieving revision 1.65 diff -u -r1.65 easy-mmode.el --- lisp/emacs-lisp/easy-mmode.el 8 Jun 2005 15:54:43 -0000 1.65 +++ lisp/emacs-lisp/easy-mmode.el 15 Jun 2005 16:18:23 -0000 @@ -201,10 +201,7 @@ :type 'boolean ,@(cond ((not (and curfile require)) nil) - ((not (eq require t)) `(:require ,require)) - (t `(:require - ',(intern (file-name-nondirectory - (file-name-sans-extension curfile)))))) + ((not (eq require t)) `(:require ,require))) ,@(nreverse extra-keywords)))) ;; The actual function. Index: lisp/emacs-lisp/autoload.el =================================================================== RCS file: /cvsroot/emacs/emacs/lisp/emacs-lisp/autoload.el,v retrieving revision 1.104 diff -u -r1.104 autoload.el --- lisp/emacs-lisp/autoload.el 31 Mar 2005 21:17:40 -0000 1.104 +++ lisp/emacs-lisp/autoload.el 15 Jun 2005 16:18:23 -0000 @@ -1,7 +1,7 @@ ;; autoload.el --- maintain autoloads in loaddefs.el -;; Copyright (C) 1991,92,93,94,95,96,97, 2001,02,03,04 -;; Free Software Foundation, Inc. +;; Copyright (C) 1991, 1992, 1993, 1994, 1995, 1996, 1997, 2001, 2002, 2003, +;; 2004, 2005 Free Software Foundation, Inc. ;; Author: Roland McGrath ;; Keywords: maint @@ -123,7 +123,17 @@ ) `(progn (defvar ,varname ,init ,doc) - (custom-autoload ',varname ,file)))) + (custom-autoload ',varname ,file) + ;; The use of :require in a defcustom can be annoying, especially + ;; when defcustoms are moved from one file to another between + ;; releases because the :require arg gets placed in the user's + ;; .emacs. In order for autoloaded minor modes not to need the + ;; use of :require, we arrange to store their :setter. + ,(let ((setter (condition-case nil + (cadr (memq :set form)) + (error nil)))) + (if (equal setter ''custom-set-minor-mode) + `(put ',varname 'custom-set 'custom-set-minor-mode)))))) ;; nil here indicates that this is not a special autoload form. (t nil)))) @@ -566,5 +576,5 @@ (provide 'autoload) -;;; arch-tag: 00244766-98f4-4767-bf42-8a22103441c6 +;; arch-tag: 00244766-98f4-4767-bf42-8a22103441c6 ;;; autoload.el ends here